From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com ([192.55.52.136]:15103 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751874AbeCESIh (ORCPT ); Mon, 5 Mar 2018 13:08:37 -0500 Date: Mon, 5 Mar 2018 20:08:31 +0200 From: Jarkko Sakkinen To: Tomas Winkler Cc: Jason Gunthorpe , Alexander Usyskin , linux-integrity@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2 v3] tpm: separate cmd_ready/go_idle from runtime_pm Message-ID: <20180305180831.GB6220@linux.intel.com> References: <20180214134319.4400-1-tomas.winkler@intel.com> <20180214134319.4400-3-tomas.winkler@intel.com> <20180219115221.7kd3ks7mrzmw36mz@linux.intel.com> <1519136010.4113.10.camel@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1519136010.4113.10.camel@linux.intel.com> Sender: linux-integrity-owner@vger.kernel.org List-ID: On Tue, Feb 20, 2018 at 04:13:30PM +0200, Jarkko Sakkinen wrote: > On Mon, 2018-02-19 at 13:52 +0200, Jarkko Sakkinen wrote: > > On Wed, Feb 14, 2018 at 03:43:19PM +0200, Tomas Winkler wrote: > > > We cannot use go_idle cmd_ready commands via runtime_pm handles > > > as with the introduction of localities this is no longer an > > > optional > > > feature, while runtime pm can be not enabled. > > > Though cmd_ready/go_idle provides power saving feature, it's also > > > part of > > > TPM2 protocol and should be called explicitly. > > > This patch exposes cmd_read/go_idle via tpm class ops and removes > > > runtime pm support as it is not used by any driver. > > > > > > Signed-off-by: Tomas Winkler > > > --- > > > V2: resent > > > V3: resent > > > > Only the cover letter needs a backlog but the code change looks > > good to me. > > > > Reviewed-by: Jarkko Sakkinen > > Tested-by: Jarkko Sakkinen > > /Jarkko Applied. /Jarkko