ARM: dts: at91: sam9x60_curiosity: Add mandatory dt property for RTT

Message ID 20230928143644.208515-1-nicolas.ferre@microchip.com
State New
Headers
Series ARM: dts: at91: sam9x60_curiosity: Add mandatory dt property for RTT |

Commit Message

Nicolas Ferre Sept. 28, 2023, 2:36 p.m. UTC
  From: Tudor Ambarus <tudor.ambarus@linaro.org>

atmel,rtt-rtc-time-reg is a mandatory property and encodes the GPBR
register used to store the time base when the RTT is used as an RTC.
Align the RTT with what's currently done for sam9x60ek and sama7g5ek,
and enable it by default even if RTC is also enabled.

Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
[nicolas.ferre@microchip.com: adapt to newer kernel]
Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>
---
 arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts | 4 ++++
 1 file changed, 4 insertions(+)
  

Comments

claudiu beznea Sept. 29, 2023, 6:06 p.m. UTC | #1
On 28.09.2023 17:36, nicolas.ferre@microchip.com wrote:
> From: Tudor Ambarus <tudor.ambarus@linaro.org>
> 
> atmel,rtt-rtc-time-reg is a mandatory property and encodes the GPBR
> register used to store the time base when the RTT is used as an RTC.
> Align the RTT with what's currently done for sam9x60ek and sama7g5ek,
> and enable it by default even if RTC is also enabled.
> 
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> [nicolas.ferre@microchip.com: adapt to newer kernel]
> Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Reviewed-by: Claudiu Beznea <claudiu.beznea@tuxon.dev>

> ---
>  arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
> index cb86a3a170ce..83372c1f291b 100644
> --- a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
> +++ b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
> @@ -439,6 +439,10 @@ &pwm0 {
>  	status = "okay";
>  };
>  
> +&rtt {
> +	atmel,rtt-rtc-time-reg = <&gpbr 0x0>;
> +};
> +
>  &sdmmc0 {
>  	bus-width = <4>;
>  	pinctrl-names = "default";
  
claudiu beznea Sept. 29, 2023, 6:10 p.m. UTC | #2
On 28.09.2023 17:36, nicolas.ferre@microchip.com wrote:
> From: Tudor Ambarus <tudor.ambarus@linaro.org>
> 
> atmel,rtt-rtc-time-reg is a mandatory property and encodes the GPBR
> register used to store the time base when the RTT is used as an RTC.
> Align the RTT with what's currently done for sam9x60ek and sama7g5ek,
> and enable it by default even if RTC is also enabled.
> 
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> [nicolas.ferre@microchip.com: adapt to newer kernel]
> Signed-off-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Applied to at91-dt, thanks!
  

Patch

diff --git a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
index cb86a3a170ce..83372c1f291b 100644
--- a/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
+++ b/arch/arm/boot/dts/microchip/at91-sam9x60_curiosity.dts
@@ -439,6 +439,10 @@  &pwm0 {
 	status = "okay";
 };
 
+&rtt {
+	atmel,rtt-rtc-time-reg = <&gpbr 0x0>;
+};
+
 &sdmmc0 {
 	bus-width = <4>;
 	pinctrl-names = "default";