Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758199AbbDWEz3 (ORCPT ); Thu, 23 Apr 2015 00:55:29 -0400 Received: from mga11.intel.com ([192.55.52.93]:14512 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755599AbbDWEzZ (ORCPT ); Thu, 23 Apr 2015 00:55:25 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,628,1422950400"; d="scan'208";a="484516590" Date: Thu, 23 Apr 2015 07:55:21 +0300 From: Jarkko Sakkinen To: Jason Gunthorpe Cc: peterhuewe@gmx.de, tpmdd-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Marcel Selhorst , Greg Kroah-Hartman , Tejun Heo , Al Viro , Andrew Morton , Jianyu Zhan , "Eric W. Biederman" , Rasmus Villemoes , Andrzej Hajda , NeilBrown , Guenter Roeck , Simon Wunderlich Subject: Re: [PATCH v2] tpm: enable PPI for TPM 2.0 Message-ID: <20150423045521.GA3015@intel.com> References: <1429641062-14421-1-git-send-email-jarkko.sakkinen@linux.intel.com> <20150421193338.GA31179@obsidianresearch.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150421193338.GA31179@obsidianresearch.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2463 Lines: 66 On Tue, Apr 21, 2015 at 01:33:38PM -0600, Jason Gunthorpe wrote: > On Tue, Apr 21, 2015 at 09:30:55PM +0300, Jarkko Sakkinen wrote: > > Enabled PPI interface to the character device sysfs directory accessible > > both for 1.x and 2.0 devices. > > > > The ppi group is moved from the platform device directory to the > > character device directory. In order to retain backwards compatibility > > with the 1.x devices, a symlink is created to the platform device > > directory. > > > > Signed-off-by: Jarkko Sakkinen > > > Reviewed-by: Jason Gunthorpe > > Jumping the gun a bit, there :) > But yes, the TPM bits still look OK to me. > > Ah, don't forget to update Documentation/ABI/testing/sysfs-driver-ppi > > > diff --git a/fs/kernfs/dir.c b/fs/kernfs/dir.c > > index 6acc964..749cea3 100644 > > +++ b/fs/kernfs/dir.c > > @@ -1225,6 +1225,7 @@ int kernfs_remove_by_name_ns(struct kernfs_node *parent, const char *name, > > else > > return -ENOENT; > > } > > +EXPORT_SYMBOL_GPL(kernfs_remove_by_name_ns); > > ?? > I don't see this being called > > > +/** > > + * sysfs_link_group_to_kobj - add a symlink to a kobject pointing to a group > > + * @kobj: The kobject containing the group. > > + * @target_kobj: The target kobject. > > + * @target_name: Name of the target group. > > + */ > > +int sysfs_link_group_to_kobj(struct kobject *kobj, struct kobject *target_kobj, > > + const char *target_name) > > Agree with Guenter, separate patch. > > I liked the signature I suggested earlier: > > sysfs_link_group_to_kobj(struct kobject *from_kobj, > const char *from_name, > struct kobject *target_kobj, > const char *target_name) > > 'link_group' may be too specific a name, I think this would work to > establish a link to any file in a sysfs directory? Which is good, > we'll need that for future TPM patches that relocate the other sysfs > files.. BTW, which sysfs attributes would need to be relocated? Other than PPI at least we are not interested to port for TPM 2.0 because you can just as well get the data using TPM2 protocol and /dev/tpm0. > Jason /Jarkko -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/