Hi,

> +             mpc5200_pic: interrupt-control...@500 {
> +                     // 5200 interrupts are encoded into two levels;

Grant, maybe this comment could be dropped also for the in-kernel dts-files? I
think it is sufficently described in mpc5200.txt. I could prepare a patch if
you agree.

> +             i...@3d40 {
> +                     #address-cells = <1>;
> +                     #size-cells = <0>;
> +                     compatible = "fsl,mpc5200-i2c","fsl-i2c";
> +                     reg = <0x3d40 0x40>;
> +                     interrupts = <2 16 0>;
> +                     fsl5200-clocking;

Please drop the clocking-line. It became obsolete.

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

Attachment: signature.asc
Description: Digital signature

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to