diff mbox

ARM: dts: Enable RTC node for exynos5250

Message ID 1371189378-22823-1-git-send-email-tushar.behera@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Tushar Behera June 14, 2013, 5:56 a.m. UTC
The kernel dump produced by s3c-rtc driver as reported by
commit 522ccdb6fd0e ("ARM: dts: Disable the RTC by default on exynos5")
are no longer reproducible in latest kernel.

Hence it would be good to re-enable RTC support for EXYNOS5250.

Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
---
 arch/arm/boot/dts/exynos5250.dtsi |    1 -
 1 file changed, 1 deletion(-)

Comments

Doug Anderson June 14, 2013, 3:01 p.m. UTC | #1
Tushar,

On Thu, Jun 13, 2013 at 10:56 PM, Tushar Behera
<tushar.behera@linaro.org> wrote:
> The kernel dump produced by s3c-rtc driver as reported by
> commit 522ccdb6fd0e ("ARM: dts: Disable the RTC by default on exynos5")
> are no longer reproducible in latest kernel.
>
> Hence it would be good to re-enable RTC support for EXYNOS5250.
>
> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
> ---
>  arch/arm/boot/dts/exynos5250.dtsi |    1 -
>  1 file changed, 1 deletion(-)

I would prefer we not commit this.

The original commit wasn't just about avoiding the bug (though that
was a nice side effect).  My understanding is that the SoC dtsi file
ought to have most peripherals disabled by default and individual
board dts files should enable the peripherals that they want.  I think
exynos5250 is particularly bad about this right now (it starts with
most peripherals enabled) but that should probably be fixed...

If my understanding is wrong, others on the list should definitely speak up!  :)

-Doug
Tushar Behera June 17, 2013, 3:11 a.m. UTC | #2
On 14 June 2013 20:31, Doug Anderson <dianders@chromium.org> wrote:
> Tushar,
>
> On Thu, Jun 13, 2013 at 10:56 PM, Tushar Behera
> <tushar.behera@linaro.org> wrote:
>> The kernel dump produced by s3c-rtc driver as reported by
>> commit 522ccdb6fd0e ("ARM: dts: Disable the RTC by default on exynos5")
>> are no longer reproducible in latest kernel.
>>
>> Hence it would be good to re-enable RTC support for EXYNOS5250.
>>
>> Signed-off-by: Tushar Behera <tushar.behera@linaro.org>
>> ---
>>  arch/arm/boot/dts/exynos5250.dtsi |    1 -
>>  1 file changed, 1 deletion(-)
>
> I would prefer we not commit this.
>
> The original commit wasn't just about avoiding the bug (though that

I thought the intention of the original commit was just that. :-)

> was a nice side effect).  My understanding is that the SoC dtsi file
> ought to have most peripherals disabled by default and individual
> board dts files should enable the peripherals that they want.  I think
> exynos5250 is particularly bad about this right now (it starts with
> most peripherals enabled) but that should probably be fixed...
>

I agree, all the peripherals should be fixed for this, not just RTC.

> If my understanding is wrong, others on the list should definitely speak up!  :)
>

Let me know if you still prefer to keep RTC node disabled on exynos5250.dtsi.
In that case, I would enable it only for Arndale.

> -Doug


Thanks.
--
Tushar Behera
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index 0673524..ce4a150 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -188,7 +188,6 @@ 
 		interrupts = <0 43 0>, <0 44 0>;
 		clocks = <&clock 337>;
 		clock-names = "rtc";
-		status = "disabled";
 	};
 
 	tmu@10060000 {