Hi, Shawn,
On Tue, Mar 8, 2011 at 11:11 AM, Shawn Guo <[email protected]> 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 <[email protected]> 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 <[email protected]>
>> >> Singed-off-by: Rob Herring <[email protected]>
>> >> ---
>> >> arch/arm/boot/dts/babbage.dts | 110
>> >> +++++++++++++++++++++++++++++++++++++++++
>> >> 1 files changed, 110 insertions(+), 0 deletions(-)
>
> --
> Regards,
> Shawn
>
>
_______________________________________________
devicetree-discuss mailing list
[email protected]
https://lists.ozlabs.org/listinfo/devicetree-discuss
