From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756240AbdLTSKR (ORCPT ); Wed, 20 Dec 2017 13:10:17 -0500 Received: from mail-io0-f180.google.com ([209.85.223.180]:43693 "EHLO mail-io0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756016AbdLTSKI (ORCPT ); Wed, 20 Dec 2017 13:10:08 -0500 X-Google-Smtp-Source: ACJfBot69MgkYH5psXejsvwkUJZ/e+Uchqjec032hiB0L8Npt33B450O5qBy2aYmf3fpFIQaYg9W6w== Date: Wed, 20 Dec 2017 11:10:05 -0700 From: Jason Gunthorpe To: "Shaikh, Azhar" Cc: Javier Martinez Canillas , "linux-kernel@vger.kernel.org" , James Ettle , Hans de Goede , Arnd Bergmann , Jarkko Sakkinen , Peter Huewe , Greg Kroah-Hartman , "linux-integrity@vger.kernel.org" Subject: Re: [PATCH 2/4] tpm: delete the TPM_TIS_CLK_ENABLE flag Message-ID: <20171220181005.GC22908@ziepe.ca> References: <20171220113538.16099-1-javierm@redhat.com> <20171220113538.16099-3-javierm@redhat.com> <5FFFAD06ADE1CA4381B3F0F7C6AF5828988730@ORSMSX109.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5FFFAD06ADE1CA4381B3F0F7C6AF5828988730@ORSMSX109.amr.corp.intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 20, 2017 at 03:19:19PM +0000, Shaikh, Azhar wrote: > >This flag is only used to warn if CLKRUN_EN wasn't disabled on Braswell > >systems, but the only way this can happen is if the code is not correct. > > > >So it's an unnecessary check that just makes the code harder to read. > > This code was implemented as a suggestion from Jason on the previous patches. > https://www.spinics.net/lists/linux-integrity/msg00827.html The concept was to be like ASSERT_RTNL, maybe it just needs a suitably named static inline to addrress Javier's readability concerns? Jason