Re: [PATCH] ARM: dts: Add watchdog in exynos5440-ssdk5440 device list

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

 



On 01/19/2013 04:10 PM, Tomasz Figa wrote:
> Hi Kukjin,
> 
> The commit message is a bit confusing, because it says that something is 
> being added, while in fact the whole node is being removed.
> 
> Shouldn't it rather say something like:
> ARM: dts: exynos5440-ssdk5440: Remove disabled status from watchdog node
> 

Rather, should we not be following a consistent approach for both
EXYNSO4 and EXYNOS5.

In EXYNOS4, the default state of the nodes is 'disabled' and they are
enabled in the board-specific files whereas it is exactly the opposite
in case of EXYNOS5.

> Best regards,
> Tomasz Figa
> 
> On Friday 18 of January 2013 21:00:51 Kukjin Kim wrote:
>> From: Giridhar Maruthy <giridhar.m@xxxxxxxxxxx>
>>
>> Signed-off-by: Giridhar Maruthy <giridhar.m@xxxxxxxxxxx>
>> Signed-off-by: Kukjin Kim <kgene.kim@xxxxxxxxxxx>
>> ---
>>  arch/arm/boot/dts/exynos5440-ssdk5440.dts |    4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
>> b/arch/arm/boot/dts/exynos5440-ssdk5440.dts index 81e2c96..5239d75
>> 100644
>> --- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
>> +++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
>> @@ -36,10 +36,6 @@
>>  		status = "disabled";
>>  	};
>>
>> -	watchdog {
>> -		status = "disabled";
>> -	};
>> -
>>  	rtc {
>>  		status = "disabled";
>>  	};
> --
> 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
> 


-- 
Tushar Behera
--
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