Hi, Shawn, On Tue, Mar 8, 2011 at 11:11 AM, Shawn Guo <shawn....@freescale.com> wrote: > On Tue, Mar 08, 2011 at 10:20:33AM +0800, Jason Hui wrote: >> Hi, Shawn, >> >> On Mon, Mar 7, 2011 at 10:51 PM, Shawn Guo <shawn....@freescale.com> wrote: >> > Aha, Jason. Grant and I put many comments on v1 of this patch, but it >> > seems that v2 is identical to v1, except adding Rob Herring sign-off. >> >> Please compare with v1 and say it once again, It should include the > > Oops! Sorry, I messed up with my local branches last night, probably > because it was late :) > >> grant and your comments except the mxctty1 >> due to I think that it's not one issue, So, I did not take your comments. >> > I still think it's an issue that the dt kernel needs a different > console cmdline from the non-dt one.
In fact, this is user defined, I think But why we need mxctty1 in dt here, the reason is that, uart3 will be probed first and then uart0, uart1, which lead to mxctty0 ---> uart3, mxctty1---> uart0 and mxctty2 --->uart1. The debug uart of mx51 babbage board is uart0, so we need define mxctty1 in the kernel command line. We can put spba@70000000 at the back of aips@73f00000, but it will make the memory address not sorted from lower to higher. spba@70000000 { #address-cells = <1>; #size-cells = <1>; compatible = "simple-bus"; ranges = <0x0 0x70000000 0x100000>; imx-uart@c000 { compatible = "fsl,imx51-uart"; reg = <0xc000 0x1000>; interrupts = <0x21>; rts-cts; uart-clock = <&uart2_clk>, "uart"; }; }; aips@73f00000 { #address-cells = <1>; #size-cells = <1>; compatible = "simple-bus"; ranges = <0x0 0x73f00000 0x100000>; imx-uart@bc000 { compatible = "fsl,imx51-uart"; reg = <0xbc000 0x1000>; interrupts = <0x1f>; fsl,has-rts-cts; uart-clock = <&uart0_clk>, "uart"; }; imx-uart@c0000 { compatible = "imx-uart"; reg = <0xc0000 0x1000>; interrupts = <0x20>; fsl,has-rts-cts; uart-clock = <&uart1_clk>, "uart"; }; > >> But I observed that some comments as Grant suggest to use decimal not >> hex value for #address-cells and #size-cells, >> not applied to this file globally, I will fix it in the V3 patch if need. >> >> > >> > On Mon, Mar 07, 2011 at 09:05:59PM +0800, Jason Liu wrote: >> >> Signed-off-by: Jason Liu <r64...@freescale.com> >> >> Singed-off-by: Rob Herring <robherri...@gmail.com> >> >> --- >> >> arch/arm/boot/dts/babbage.dts | 110 >> >> +++++++++++++++++++++++++++++++++++++++++ >> >> 1 files changed, 110 insertions(+), 0 deletions(-) > > -- > Regards, > Shawn > > _______________________________________________ devicetree-discuss mailing list devicetree-discuss@lists.ozlabs.org https://lists.ozlabs.org/listinfo/devicetree-discuss