From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757277AbcING2K (ORCPT ); Wed, 14 Sep 2016 02:28:10 -0400 Received: from mga01.intel.com ([192.55.52.88]:8357 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751764AbcING2H (ORCPT ); Wed, 14 Sep 2016 02:28:07 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,332,1470726000"; d="scan'208";a="1039597214" From: "Winkler, Tomas" To: "jarkko.sakkinen@linux.intel.com" , "jgunthorpe@obsidianresearch.com" , "tpmdd-devel@lists.sourceforge.net" CC: "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v3 0/4] tpm/tpm_crb: implement power management. Thread-Topic: [PATCH v3 0/4] tpm/tpm_crb: implement power management. Thread-Index: AQHSDPaCqzrK4jidHkKxjfSSkAPaVaB4VYuA Date: Wed, 14 Sep 2016 06:28:03 +0000 Message-ID: <1473834369.21459.4.camel@intel.com> References: <1473685461-6540-1-git-send-email-tomas.winkler@intel.com> In-Reply-To: <1473685461-6540-1-git-send-email-tomas.winkler@intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.12.87.167] Content-Type: text/plain; charset="utf-8" Content-ID: <07793DA38D74B14F8E391A64442D3BAF@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u8E6SFBK027638 On Mon, 2016-09-12 at 16:04 +0300, Tomas Winkler wrote: > Te overall platform ability to enter a low power state is also > conditioned on the ability of a tpm device to go to idle state. > This series should provide this feature. > > Unfortunately, there is a HW bug on Intel PTT devices on Skylake, > Kabylake, and Broxton devices, where certain registers lost retention > during TPM idle state. Hence this implementation takes this into > consideration and implement the feature based only on access to > registers that retain their state. This still conforms to the spec > and should be correct also on non Intle devices. > > V2: Utilize runtime_pm for driving tpm crb idle states. > V3. fix lower case corruption in the first patch > Jarkko, had you chance to test v3 series one on your side, is this okay to go? Thanks Tomas