On Thu, Jun 12, 2008 at 10:19 PM, David Gibson
<[EMAIL PROTECTED]> wrote:
> On Thu, Jun 12, 2008 at 08:10:47AM -0600, Grant Likely wrote:
> [snip]
>> > + [EMAIL PROTECTED] {
>> > + compatible = "fsl,mpc5121-i2c-ctrl";
>> > + reg = <0x1760 0x8>;
>> > + };
>> > +
>> > + [EMAIL PROTECTED] {
>>
>> (nitpick) There is a recommended practice that says node names should
>> be generic as much as possible, so I think this should probably be
>> [EMAIL PROTECTED] The compatible value is fine.
>
> If this is a video display device, the recommended generic name is
> "[EMAIL PROTECTED]", not "[EMAIL PROTECTED]".
heh; oops. I should know better and I stand corrected. :-)
g.
--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
_______________________________________________
Linuxppc-embedded mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-embedded
