From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [RFC PATCHv2 1/2] Export SoC info through sysfs Date: Fri, 8 Apr 2011 01:19:10 +0200 Message-ID: <201104080119.11378.arnd@arndb.de> References: <1299846911-15782-1-git-send-email-maxime.coquelin-nonst@stericsson.com> <4D9E4114.3090102@bluewatersys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4D9E4114.3090102@bluewatersys.com> 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: Ryan Mallon Cc: Nicolas Pitre , ext Nishanth Menon , ext Tony Lindgren , Greg KH , Linus Walleij , Ambresh , Saravana Kannan , Andrei Warkentin , Lee Jones , Rabin VINCENT , Russell King , Jonas ABERG , ext Kevin Hilman , Peter De-Schrijver , David Brown , Maxime Coquelin , "linux-arm-msm@vger.kernel.org" , Loic PALLARDY , Eduardo Valentin , maxime_coquelin@yahoo.fr, Linux-OMAP , linux-arm-ke List-Id: linux-arm-msm@vger.kernel.org On Friday 08 April 2011, Ryan Mallon wrote: >>>>> /sys/devices/soc/${NAME}/ >>>>> /sys/devices/platform/soc${NUMBER}/ >>>> >>>> I prefer the second format here since the path is always the same which >>>> makes it easier to write parsing tools. The name should be an entry in >>>> the directory rather than the name of the directory itself. In the first case, we would ensure that every directory under /sys/devices/soc is an SOC device, so you would not need to parse the name at all, which is even simpler. I'm open to other arguments either way, but I think this one is not particularly important. > > That would mean two separate instances of Linux, just like two systems. > > SMP across multiple SOCs doesn't make much sense, and I don't think they > > are likely to be designed for that ability either. > > So we probably don't need the ability to have multiple SoC directories > under sysfs then? There are systems that have multiple ones, they are just not as common. > For the other part of the question, do you think it makes sense for the > CPU's to be child devices of the SoC (and for the CPU devices to be > potentially symlinked from the SoC directory)? No, the CPUs are children of /sys/devices/system, and we should not change that. Symlinks sound like a good idea though. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Fri, 8 Apr 2011 01:19:10 +0200 Subject: [RFC PATCHv2 1/2] Export SoC info through sysfs In-Reply-To: <4D9E4114.3090102@bluewatersys.com> References: <1299846911-15782-1-git-send-email-maxime.coquelin-nonst@stericsson.com> <4D9E4114.3090102@bluewatersys.com> Message-ID: <201104080119.11378.arnd@arndb.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Friday 08 April 2011, Ryan Mallon wrote: >>>>> /sys/devices/soc/${NAME}/ >>>>> /sys/devices/platform/soc${NUMBER}/ >>>> >>>> I prefer the second format here since the path is always the same which >>>> makes it easier to write parsing tools. The name should be an entry in >>>> the directory rather than the name of the directory itself. In the first case, we would ensure that every directory under /sys/devices/soc is an SOC device, so you would not need to parse the name at all, which is even simpler. I'm open to other arguments either way, but I think this one is not particularly important. > > That would mean two separate instances of Linux, just like two systems. > > SMP across multiple SOCs doesn't make much sense, and I don't think they > > are likely to be designed for that ability either. > > So we probably don't need the ability to have multiple SoC directories > under sysfs then? There are systems that have multiple ones, they are just not as common. > For the other part of the question, do you think it makes sense for the > CPU's to be child devices of the SoC (and for the CPU devices to be > potentially symlinked from the SoC directory)? No, the CPUs are children of /sys/devices/system, and we should not change that. Symlinks sound like a good idea though. Arnd