[PATCH 1/2] Added support for PRTLVT based boards (MPC5121)

Grant Likely grant.likely at secretlab.ca
Fri Jun 13 15:12:24 EST 2008


On Thu, Jun 12, 2008 at 10:19 PM, David Gibson
<david at gibson.dropbear.id.au> wrote:
> On Thu, Jun 12, 2008 at 08:10:47AM -0600, Grant Likely wrote:
> [snip]
>> > +               i2ccontrol at 1760 {
>> > +                       compatible = "fsl,mpc5121-i2c-ctrl";
>> > +                       reg = <0x1760 0x8>;
>> > +               };
>> > +
>> > +               diu at 2100 {
>>
>> (nitpick) There is a recommended practice that says node names should
>> be generic as much as possible, so I think this should probably be
>> video at 2100.  The compatible value is fine.
>
> If this is a video display device, the recommended generic name is
> "display at XX", not "video at XX".

heh; oops.  I should know better and I stand corrected.  :-)

g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.


More information about the Linuxppc-embedded mailing list