On Wed, Apr 01, 2009 at 01:59:36PM +0200, Wolfgang Grandegger wrote:
> David Gibson wrote:
> > On Wed, Apr 01, 2009 at 09:31:34AM +0200, Wolfgang Grandegger wrote:
> >> Hi David,
> >>
> >> this patch went out by mistake, sorry.
> >>
> >> David Gibson wrote:
> >>> On Tue, Mar 31, 2009 at 02:37:33PM +0200, Wolfgang Grandegger wrote:
> >>>> Index: linux-2.6/arch/powerpc/boot/dts/socrates.dts
> >>>> ===================================================================
> >>>> --- linux-2.6.orig/arch/powerpc/boot/dts/socrates.dts 2009-03-31
> >>>> 13:24:38.078719288 +0200
> >>>> +++ linux-2.6/arch/powerpc/boot/dts/socrates.dts 2009-03-31
> >>>> 13:47:56.986720719 +0200
> >>>> @@ -52,6 +52,7 @@
> >>>> soc8...@e0000000 {
> >>>> #address-cells = <1>;
> >>>> #size-cells = <1>;
> >>>> + device_type = "soc";
> >>> You should not be adding device_type to a new board's dts. You
> >>> *especially* shouldn't be doing that with no patch comment explaining
> >>> why.
> >> In the final patch I have removed the device_type = "soc", but it makes
> >> trouble as some mpc85xx rely on it.
> >
> > Um.. what? Some mpc85xx? This is the dts for *one specific board*, I
> > don't see how something that affects "some mpc85xx" is relevant.
>
> Well, to be more precise, it affects all boards using CONFIG_FSL_SOC
> because they use functions searching for the device type "soc", e.g.
> fsl_get_sys_freq():
>
> http://lxr.linux.no/linux+v2.6.29/arch/powerpc/sysdev/fsl_soc.c#L80
>
> This needs to be fixed before the line 'device_type = "soc"' can be removed.
Gah. I had been under the impression that had been fixed already.
Kumar! Get your house in order!
Ok, I guess it will have to stay for now, just as long as the new
style marker in compatible is there as well.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
