diff mbox series

ARM: kirkwood: ts219: disable the SoC's RTC

Message ID 20190723194505.28060-1-uwe@kleine-koenig.org (mailing list archive)
State Mainlined
Commit 644763224169e9ca2e3010c24bf4b81a6be64959
Headers show
Series ARM: kirkwood: ts219: disable the SoC's RTC | expand

Commit Message

Uwe Kleine-König July 23, 2019, 7:45 p.m. UTC
The internal RTC doesn't work, loading the driver only yields

	rtc-mv f1010300.rtc: internal RTC not ticking

. So disable it.

Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
---
 arch/arm/boot/dts/kirkwood-ts219.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Alexandre Belloni July 23, 2019, 8:03 p.m. UTC | #1
On 23/07/2019 21:45:05+0200, Uwe Kleine-König wrote:
> The internal RTC doesn't work, loading the driver only yields
> 
> 	rtc-mv f1010300.rtc: internal RTC not ticking
> 
> . So disable it.
> 
> Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
> ---
>  arch/arm/boot/dts/kirkwood-ts219.dtsi | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/kirkwood-ts219.dtsi b/arch/arm/boot/dts/kirkwood-ts219.dtsi
> index a88eb22070a1..994cabcf4b51 100644
> --- a/arch/arm/boot/dts/kirkwood-ts219.dtsi
> +++ b/arch/arm/boot/dts/kirkwood-ts219.dtsi
> @@ -104,3 +104,11 @@
>  &pcie0 {
>  	status = "okay";
>  };
> +
> +&rtc {
> +	/*
> +	 * There is a s35390a available on the i2c bus, the internal rtc isn't
> +	 * working (probably no crystal assembled).
> +	 */
> +	status = "disabled";
> +};

You could also use the aliases to ensure rtc0 is the s35390a. This would
solve the initial issue.

> -- 
> 2.20.1
>
Uwe Kleine-König July 23, 2019, 8:24 p.m. UTC | #2
On 7/23/19 10:03 PM, Alexandre Belloni wrote:
> On 23/07/2019 21:45:05+0200, Uwe Kleine-König wrote:
>> The internal RTC doesn't work, loading the driver only yields
>>
>> 	rtc-mv f1010300.rtc: internal RTC not ticking
>>
>> . So disable it.
>>
>> Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
>> ---
>>  arch/arm/boot/dts/kirkwood-ts219.dtsi | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/kirkwood-ts219.dtsi b/arch/arm/boot/dts/kirkwood-ts219.dtsi
>> index a88eb22070a1..994cabcf4b51 100644
>> --- a/arch/arm/boot/dts/kirkwood-ts219.dtsi
>> +++ b/arch/arm/boot/dts/kirkwood-ts219.dtsi
>> @@ -104,3 +104,11 @@
>>  &pcie0 {
>>  	status = "okay";
>>  };
>> +
>> +&rtc {
>> +	/*
>> +	 * There is a s35390a available on the i2c bus, the internal rtc isn't
>> +	 * working (probably no crystal assembled).
>> +	 */
>> +	status = "disabled";
>> +};
> 
> You could also use the aliases to ensure rtc0 is the s35390a. This would
> solve the initial issue.

What do you assume is my initial issue? Adding the alias doesn't make
the above message disappear from the boot log.

Best regards
Uwe
Andrew Lunn July 23, 2019, 10:20 p.m. UTC | #3
On Tue, Jul 23, 2019 at 09:45:05PM +0200, Uwe Kleine-König wrote:
> The internal RTC doesn't work, loading the driver only yields
> 
> 	rtc-mv f1010300.rtc: internal RTC not ticking
> 
> . So disable it.
> 
> Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>

Hi Uwe

This should probably go via the mvebu maintainers, who look after the
DT files.

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew
Alexandre Belloni July 24, 2019, 8:57 a.m. UTC | #4
On 23/07/2019 22:24:18+0200, Uwe Kleine-König wrote:
> On 7/23/19 10:03 PM, Alexandre Belloni wrote:
> > On 23/07/2019 21:45:05+0200, Uwe Kleine-König wrote:
> >> The internal RTC doesn't work, loading the driver only yields
> >>
> >> 	rtc-mv f1010300.rtc: internal RTC not ticking
> >>
> >> . So disable it.
> >>
> >> Signed-off-by: Uwe Kleine-König <uwe@kleine-koenig.org>
> >> ---
> >>  arch/arm/boot/dts/kirkwood-ts219.dtsi | 8 ++++++++
> >>  1 file changed, 8 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/kirkwood-ts219.dtsi b/arch/arm/boot/dts/kirkwood-ts219.dtsi
> >> index a88eb22070a1..994cabcf4b51 100644
> >> --- a/arch/arm/boot/dts/kirkwood-ts219.dtsi
> >> +++ b/arch/arm/boot/dts/kirkwood-ts219.dtsi
> >> @@ -104,3 +104,11 @@
> >>  &pcie0 {
> >>  	status = "okay";
> >>  };
> >> +
> >> +&rtc {
> >> +	/*
> >> +	 * There is a s35390a available on the i2c bus, the internal rtc isn't
> >> +	 * working (probably no crystal assembled).
> >> +	 */
> >> +	status = "disabled";
> >> +};
> > 
> > You could also use the aliases to ensure rtc0 is the s35390a. This would
> > solve the initial issue.
> 
> What do you assume is my initial issue? Adding the alias doesn't make
> the above message disappear from the boot log.
> 

I overlooked that the rtc is not working at all (and I thought it only
needed some init in u-boot like the armada38x). Seems good to me then.
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/kirkwood-ts219.dtsi b/arch/arm/boot/dts/kirkwood-ts219.dtsi
index a88eb22070a1..994cabcf4b51 100644
--- a/arch/arm/boot/dts/kirkwood-ts219.dtsi
+++ b/arch/arm/boot/dts/kirkwood-ts219.dtsi
@@ -104,3 +104,11 @@ 
 &pcie0 {
 	status = "okay";
 };
+
+&rtc {
+	/*
+	 * There is a s35390a available on the i2c bus, the internal rtc isn't
+	 * working (probably no crystal assembled).
+	 */
+	status = "disabled";
+};