Re: [PATCH v3 1/2] ARM: dts: Add node references to 5420

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux