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 Tomasz,

Will make these changes.

Thanks & Regards
Arun

On 05/01/14 22:55, Tomasz Figa wrote:
> 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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux