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

[Tushar Behera <tushar.behera@linaro.org>]

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@samsung.com>
>>
>> Signed-off-by: Giridhar Maruthy <giridhar.m@samsung.com>
>> Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
>> ---
>>  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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 


-- 
Tushar Behera