Hello Vitaly,

Vitaly Bordug wrote:
> Heiko Schocher wrote:
>> the following patch adds support for the MPC852 based mgsuvd board
>> from keymile.
> Looks good overall. Please add supported/working/not working etc SoC devices 
> state along with the patch 
> description. Also a few really small comments below...

OK.

>> Signed-off-by: Heiko Schocher <[EMAIL PROTECTED]>
>> ---
[...]
>> +
>> +            PowerPC,[EMAIL PROTECTED] {
> But it's mpc852, isn't it?

Yes your are right.

>> +                    device_type = "cpu";
>> +                    reg = <0>;
>> +                    d-cache-line-size = <d#16>;
>> +                    i-cache-line-size = <d#16>;
>> +                    d-cache-size = <d#8192>;
>> +                    i-cache-size = <d#8192>;
>> +                    timebase-frequency = <0>;  /* Filled in by
>> u-boot */
>> +                    bus-frequency = <0>;  /* Filled in by u-boot
>> */
>> +                    clock-frequency = <0>; /* Filled in by
>> u-boot */
>> +                    interrupts = <f 2>;     // decrementer
>> interrupt
> I would like all the comments to be consistent C style (because that's it for 
> the most other dts'es).

Okay, I fix this, and send a new patch.

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to