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