From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [RFC PATCHv2 1/2] Export SoC info through sysfs Date: Fri, 11 Mar 2011 07:52:57 -0800 Message-ID: <20110311155257.GA6711@kroah.com> References: <1299846911-15782-1-git-send-email-maxime.coquelin-nonst@stericsson.com> <1299846911-15782-2-git-send-email-maxime.coquelin-nonst@stericsson.com> <201103111532.52393.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <201103111532.52393.arnd@arndb.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Arnd Bergmann Cc: ext Nishanth Menon , ext Tony Lindgren , Peter De-Schrijver , Linus Walleij , Ambresh , Saravana Kannan , Jouni Hogander , Lee Jones , Rabin VINCENT , Russell King , Jonas ABERG , ext Kevin Hilman , David Brown , Maxime Coquelin , "linux-arm-msm@vger.kernel.org" , Loic PALLARDY , "eduardo.valentin@nokia.com" , maxime_coquelin@yahoo.fr, Ryan Mallon , Linux-OMAP , linux-arm-kernel@lists.infra List-Id: linux-arm-msm@vger.kernel.org On Fri, Mar 11, 2011 at 03:32:51PM +0100, Arnd Bergmann wrote: > On Friday 11 March 2011, Maxime Coquelin wrote: > > Common base to export System-on-Chip related informations through sysfs. > > > > Creation of a "socinfo" directory under /sys/. > > Creation of SoC information entries. > > > > Signed-off-by: Maxime COQUELIN > > I think it's better than the previous patch to create an > artificial device in /sys/devices/system/socinfo, but I'd > still prefer the information to be attached to a real device > that represents the SOC, as I explained in the discussion with > Linus Walleij. > > You should definitely add Greg on Cc, as he's maintaining sysfs > and certainly has an opininion here. > > Arnd > > > --- > > Documentation/ABI/testing/sysfs-socinfo | 16 ++++++ > > drivers/base/Kconfig | 3 + > > drivers/base/Makefile | 1 + > > drivers/base/soc.c | 79 +++++++++++++++++++++++++++++++ > > include/linux/sys_soc.h | 50 +++++++++++++++++++ > > 5 files changed, 149 insertions(+), 0 deletions(-) > > create mode 100644 Documentation/ABI/testing/sysfs-socinfo > > create mode 100644 drivers/base/soc.c > > create mode 100644 include/linux/sys_soc.h > > > > diff --git a/Documentation/ABI/testing/sysfs-socinfo b/Documentation/ABI/testing/sysfs-socinfo > > new file mode 100644 > > index 0000000..afd9da2 > > --- /dev/null > > +++ b/Documentation/ABI/testing/sysfs-socinfo > > @@ -0,0 +1,16 @@ > > +What: /sys/socinfo > > +Date: March 2011 > > +contact: Maxime Coquelin > > +Description: > > + The /sys/socinfo directory contains information about the > > + System-on-Chip. It is only available if platform implements it. > > + This directory contains two kind of attributes : > > + - common attributes: > > + * machine: the name of the machine. > > + * family: the family name of the SoC > > + - SoC-specific attributes: The SoC vendor can declare attributes > > + to export some strings to user-space, like the serial-number for > > + example. > > + > > +Users: > > + User-space applications which needs these kind of attributes. I thought I rejected this the last time it came around? I still fail to understand why this is needed, please provide more information about why you feel this is something that the kernel needs. thanks, greg k-h