Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752904AbdCMLyf (ORCPT ); Mon, 13 Mar 2017 07:54:35 -0400 Received: from mga01.intel.com ([192.55.52.88]:17193 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751819AbdCMLye (ORCPT ); Mon, 13 Mar 2017 07:54:34 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,159,1486454400"; d="scan'208";a="1107855703" Date: Mon, 13 Mar 2017 13:54:15 +0200 From: Jarkko Sakkinen To: Meng.Li@windriver.com Cc: linux-kernel@vger.kernel.org, peterhuewe@gmx.de, tpmdd@selhorst.net, jgunthorpe@obsidianresearch.com, tpmdd-devel@lists.sourceforge.net Subject: Re: [PATCH] tpm: Add sysfs interface to show TPM family version Message-ID: <20170313115415.2czyr4woaq4h3ncr@intel.com> References: <1489396817-24855-1-git-send-email-Meng.Li@windriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1489396817-24855-1-git-send-email-Meng.Li@windriver.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3106 Lines: 116 On Mon, Mar 13, 2017 at 05:20:17PM +0800, Meng.Li@windriver.com wrote: > From: Limeng > > So far, there is not a sysfs interface for user space code to > check the TPM family version(TPM1.x or TPM2). So, add a > file named description in /sys/class/tpm/tpmX/ to show it. > > Signed-off-by: Meng Li > --- Is this the first or which version of the patch is this? Version number and changelog are missing :/ /Jarkko > drivers/char/tpm/tpm-chip.c | 70 +++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 70 insertions(+) > > diff --git a/drivers/char/tpm/tpm-chip.c b/drivers/char/tpm/tpm-chip.c > index c406343..b222421 100644 > --- a/drivers/char/tpm/tpm-chip.c > +++ b/drivers/char/tpm/tpm-chip.c > @@ -36,6 +36,68 @@ > dev_t tpm_devt; > > /** > + * show_description - sysfs interface for checking current TPM hardware version. > + * @dev: pointer to tpm chip device > + * @attr: unused > + * @buf: char buffer to be filled with TPM hardware version info > + * > + * Provides sysfs interface for showing current TPM hardware version. > + */ > +static ssize_t show_description(struct device *dev, > + struct device_attribute *attr, char *buf) > +{ > + struct tpm_chip *chip = (struct tpm_chip *)container_of(dev,struct tpm_chip,dev); > + int ret; > + > + if (chip->flags & TPM_CHIP_FLAG_TPM2) > + ret = sprintf(buf, "TPM 2.0"); > + else > + ret = sprintf(buf, "TPM 1.x"); > + > + return ret; > +} > + > +static struct device_attribute tpm_attrs[] = { > + __ATTR(description, S_IRUGO, show_description, NULL), > +}; > + > +/** > + * tpm_create_sysfs - Create tpm sysfs interface. > + * @dev: pointer to tpm chip device > + * > + * Create sysfs interface for checking current TPM hardware version. > + */ > +static int tpm_create_sysfs(struct device *dev) > +{ > + int r, t; > + > + for (t = 0; t < ARRAY_SIZE(tpm_attrs); t++) { > + r = device_create_file(dev, &tpm_attrs[t]); > + if (r) { > + dev_err(dev, "failed to create sysfs file\n"); > + return r; > + } > + } > + > + return 0; > +} > + > +/** > + * tpm_remove_sysfs - Remove tpm sysfs interface. > + * @dev: pointer to tpm chip device > + * > + * Remove sysfs interface for checking current TPM hardware version. > + */ > +static void tpm_remove_sysfs(struct device *dev) > +{ > + int t; > + > + for (t = 0; t < ARRAY_SIZE(tpm_attrs); t++) { > + device_remove_file(dev, &tpm_attrs[t]); > + } > +} > + > +/** > * tpm_try_get_ops() - Get a ref to the tpm_chip > * @chip: Chip to ref > * > @@ -363,6 +425,13 @@ int tpm_chip_register(struct tpm_chip *chip) > return rc; > } > > + rc = tpm_create_sysfs(&chip->dev); > + if (rc) { > + tpm_del_legacy_sysfs(chip); > + tpm_chip_unregister(chip); > + return rc; > + } > + > return 0; > } > EXPORT_SYMBOL_GPL(tpm_chip_register); > @@ -382,6 +451,7 @@ int tpm_chip_register(struct tpm_chip *chip) > */ > void tpm_chip_unregister(struct tpm_chip *chip) > { > + tpm_remove_sysfs(&chip->dev); > tpm_del_legacy_sysfs(chip); > tpm_bios_log_teardown(chip); > tpm_del_char_device(chip); > -- > 1.7.9.5 >