Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Stephen Warren
On 03/12/2013 01:10 PM, Arnd Bergmann wrote: > On Tuesday 12 March 2013, Stephen Warren wrote: >>> You are dropping the soc_dev on the floor here by just returning. >>> >>> The idea of the soc node is to have all on-soc components be children >>> of that node, so you should instead pass it into of_

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Arnd Bergmann
On Tuesday 12 March 2013, Stephen Warren wrote: > > You are dropping the soc_dev on the floor here by just returning. > > > > The idea of the soc node is to have all on-soc components be children > > of that node, so you should instead pass it into of_platform_populate > > as the parent device. >

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Stephen Warren
On 03/12/2013 06:59 AM, Arnd Bergmann wrote: > On Tuesday 12 March 2013, Danny Huang wrote: >> >> +void __init tegra_soc_device_init(void) >> +{ >> + struct soc_device *soc_dev; >> + struct soc_device_attribute *soc_dev_attr; >> + >> + soc_dev_attr = kzalloc(sizeof(*soc_dev_attr),

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Stephen Warren
On 03/12/2013 06:01 AM, Danny Huang wrote: > Expose Tegra chip ID and revision in /sys/devices/soc for user mode > usage > diff --git a/arch/arm/mach-tegra/common.c b/arch/arm/mach-tegra/common.c > +void __init tegra_soc_device_init(void) Oh, and since this is exposing data that's initialized by

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Stephen Warren
On 03/12/2013 06:01 AM, Danny Huang wrote: > Expose Tegra chip ID and revision in /sys/devices/soc for user mode > usage > diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig > index d1c4893..b7abcfa 100644 > --- a/arch/arm/mach-tegra/Kconfig > +++ b/arch/arm/mach-tegra/Kconfig

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Arnd Bergmann
On Tuesday 12 March 2013, Danny Huang wrote: > > +void __init tegra_soc_device_init(void) > +{ > + struct soc_device *soc_dev; > + struct soc_device_attribute *soc_dev_attr; > + > + soc_dev_attr = kzalloc(sizeof(*soc_dev_attr), GFP_KERNEL); > + if (!soc_dev_attr) > +

Re: [PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Russell King - ARM Linux
On Tue, Mar 12, 2013 at 08:01:07PM +0800, Danny Huang wrote: > + if (IS_ERR_OR_NULL(soc_dev)) Sigh. NAK. Fur cryin out loud! Use the IS_ERR() version. And you don't need the following "return;" statement - that's already implied at the closing brace of a function. -- To unsubscribe from th

[PATCH v2] ARM: tegra: expose chip ID and revision

2013-03-12 Thread Danny Huang
Expose Tegra chip ID and revision in /sys/devices/soc for user mode usage Signed-off-by: Danny Huang --- arch/arm/mach-tegra/Kconfig | 3 +++ arch/arm/mach-tegra/common.c | 23 +++ 2 files changed, 26 insertions(+) diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-t