From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751856AbdGIHEO (ORCPT ); Sun, 9 Jul 2017 03:04:14 -0400 Received: from mga04.intel.com ([192.55.52.120]:57571 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750859AbdGIHEN (ORCPT ); Sun, 9 Jul 2017 03:04:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,333,1496127600"; d="scan'208";a="1170350540" Date: Sun, 9 Jul 2017 10:04:03 +0300 From: Jarkko Sakkinen To: Sonny Rao Cc: Enric Balletbo i Serra , Rob Herring , Peter Huewe , tpmdd-devel@lists.sourceforge.net, "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Mark Rutland , Jason Gunthorpe , Guenter Roeck Subject: Re: [PATCH v4 RESEND 1/2] Documentation: tpm: add powered-while-suspended binding documentation Message-ID: <20170709070403.drxxzllnhagnxhjn@linux.intel.com> References: <20170627102724.23861-1-enric.balletbo@collabora.com> <20170703125735.dsrl6bs7ttj27zrt@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 05, 2017 at 10:19:02AM -0700, Sonny Rao wrote: > On Mon, Jul 3, 2017 at 5:57 AM, Jarkko Sakkinen > wrote: > > On Tue, Jun 27, 2017 at 12:27:23PM +0200, Enric Balletbo i Serra wrote: > >> Add a new powered-while-suspended property to control the behavior of the > >> TPM suspend/resume. > >> > >> Signed-off-by: Enric Balletbo i Serra > >> Signed-off-by: Sonny Rao > >> Reviewed-by: Jason Gunthorpe > >> Reviewed-by: Jarkko Sakkinen > >> Acked-by: Rob Herring > >> --- > >> Changes since v3. > >> - Rebased on top of linux-next > >> Rob Herring > >> - Split DT binding from code patch as is preferred. > >> > >> Did not exist on previous versions. > >> > >> Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt | 6 ++++++ > >> 1 file changed, 6 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt b/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> index 8cb638b..85c8216 100644 > >> --- a/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> +++ b/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> @@ -8,6 +8,12 @@ Required properties: > >> the firmware event log > >> - linux,sml-size : size of the memory allocated for the firmware event log > >> > >> +Optional properties: > >> + > >> +- powered-while-suspended: present when the TPM is left powered on between > >> + suspend and resume (makes the suspend/resume > >> + callbacks do nothing). > >> + > >> Example (for OpenPower Systems with Nuvoton TPM 2.0 on I2C) > >> ---------------------------------------------------------- > >> > >> -- > >> 2.9.3 > >> > > > > So... should I apply this? > > Hi, since you applied the code part, it would make sense to apply the > documentation too. I can do it. /Jarkko From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Sakkinen Subject: Re: [PATCH v4 RESEND 1/2] Documentation: tpm: add powered-while-suspended binding documentation Date: Sun, 9 Jul 2017 10:04:03 +0300 Message-ID: <20170709070403.drxxzllnhagnxhjn@linux.intel.com> References: <20170627102724.23861-1-enric.balletbo@collabora.com> <20170703125735.dsrl6bs7ttj27zrt@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sonny Rao Cc: Enric Balletbo i Serra , Rob Herring , Peter Huewe , tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org, "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Mark Rutland , Jason Gunthorpe , Guenter Roeck List-Id: devicetree@vger.kernel.org On Wed, Jul 05, 2017 at 10:19:02AM -0700, Sonny Rao wrote: > On Mon, Jul 3, 2017 at 5:57 AM, Jarkko Sakkinen > wrote: > > On Tue, Jun 27, 2017 at 12:27:23PM +0200, Enric Balletbo i Serra wrote: > >> Add a new powered-while-suspended property to control the behavior of the > >> TPM suspend/resume. > >> > >> Signed-off-by: Enric Balletbo i Serra > >> Signed-off-by: Sonny Rao > >> Reviewed-by: Jason Gunthorpe > >> Reviewed-by: Jarkko Sakkinen > >> Acked-by: Rob Herring > >> --- > >> Changes since v3. > >> - Rebased on top of linux-next > >> Rob Herring > >> - Split DT binding from code patch as is preferred. > >> > >> Did not exist on previous versions. > >> > >> Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt | 6 ++++++ > >> 1 file changed, 6 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt b/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> index 8cb638b..85c8216 100644 > >> --- a/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> +++ b/Documentation/devicetree/bindings/security/tpm/tpm-i2c.txt > >> @@ -8,6 +8,12 @@ Required properties: > >> the firmware event log > >> - linux,sml-size : size of the memory allocated for the firmware event log > >> > >> +Optional properties: > >> + > >> +- powered-while-suspended: present when the TPM is left powered on between > >> + suspend and resume (makes the suspend/resume > >> + callbacks do nothing). > >> + > >> Example (for OpenPower Systems with Nuvoton TPM 2.0 on I2C) > >> ---------------------------------------------------------- > >> > >> -- > >> 2.9.3 > >> > > > > So... should I apply this? > > Hi, since you applied the code part, it would make sense to apply the > documentation too. I can do it. /Jarkko -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html