Hi Arun, Looks mostly fine, just some minor nits to improve readability. On 30.04.2014 13:08, Arun Kumar K wrote:
Adding references to nodes which do not have it yet in exynos5420.dtsi. This is done so as to use reference based node updation in board files.
To stick to correct terminology, the handle that lets you reference a node is called "label", so this patch adds labels to nodes that don't have them yet.
Signed-off-by: Arun Kumar K <arun.kk@xxxxxxxxxxx> --- arch/arm/boot/dts/exynos5420.dtsi | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/arch/arm/boot/dts/exynos5420.dtsi b/arch/arm/boot/dts/exynos5420.dtsi index c3a9a66..2058276 100644 --- a/arch/arm/boot/dts/exynos5420.dtsi +++ b/arch/arm/boot/dts/exynos5420.dtsi @@ -125,7 +125,7 @@ clock-names = "pll_ref", "pll_in", "sclk_audio", "sclk_pcm_in"; }; - codec@11000000 { + codec: codec@11000000 {
IMHO "mfc" would be a more convenient label.
compatible = "samsung,mfc-v7"; reg = <0x11000000 0x10000>; interrupts = <0 96 0>; @@ -169,7 +169,7 @@ status = "disabled"; }; - mct@101C0000 { + mct: mct@101C0000 { compatible = "samsung,exynos4210-mct"; reg = <0x101C0000 0x800>; interrupt-controller; @@ -270,7 +270,7 @@ interrupts = <0 47 0>; }; - rtc@101E0000 { + rtc: rtc@101E0000 { clocks = <&clock CLK_RTC>; clock-names = "rtc"; status = "disabled"; @@ -430,22 +430,22 @@ status = "disabled"; }; - serial@12C00000 { + serial_0: serial@12C00000 {
To match with our hardware naming conventions, uart0 would probably be a better label. Similarly for remaining serial ports.
With those fixed, feel free to add my Reviewed-by tag. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html