arm64: dts: s32gxxxa-rdb: add RTC module over I2C0

[PATCH] arm64: dts: s32gxxxa-rdb: add RTC module over I2C0

Posted by Ciprian Costea 10 months ago

From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>

S32G274A-RDB2 and S32G399A-RDB3 boards have a PCA85073A RTC hardware module
available and connected over I2C0. Note that the RTC module is not battery
backed.

Signed-off-by: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
---
 arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
index ba53ec622f0b..4587e1cb8835 100644
--- a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
+++ b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
@@ -153,6 +153,11 @@ pcal6524: gpio-expander@22 {
 		gpio-controller;
 		#gpio-cells = <2>;
 	};
+
+	pca85073a: rtc@51 {
+		compatible = "nxp,pca85073a";
+		reg = <0x51>;
+	};
 };
 
 &i2c2 {
-- 
2.45.2

Re: [PATCH] arm64: dts: s32gxxxa-rdb: add RTC module over I2C0

Posted by Frank Li 10 months ago

On Thu, Apr 10, 2025 at 04:30:28PM +0300, Ciprian Costea wrote:
> From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
>
> S32G274A-RDB2 and S32G399A-RDB3 boards have a PCA85073A RTC hardware module
> available and connected over I2C0. Note that the RTC module is not battery
> backed.

Suggested commit message:

arm64: dts: s32gxxxa-rdb: Add PCA85073A RTC module over I2C0

Add support for the PCA85073A RTC module connected via I2C0 on
S32G274A-RDB2 and S32G399A-RDB3 boards.

Note that the RTC module is not battery backed.

Reviewed-by: Frank Li <Frank.Li@nxp.com>

>
> Signed-off-by: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
> index ba53ec622f0b..4587e1cb8835 100644
> --- a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
> +++ b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
> @@ -153,6 +153,11 @@ pcal6524: gpio-expander@22 {
>  		gpio-controller;
>  		#gpio-cells = <2>;
>  	};
> +
> +	pca85073a: rtc@51 {
> +		compatible = "nxp,pca85073a";
> +		reg = <0x51>;
> +	};
>  };
>
>  &i2c2 {
> --
> 2.45.2
>

Re: [PATCH] arm64: dts: s32gxxxa-rdb: add RTC module over I2C0

Posted by Ciprian Marian Costea 10 months ago

On 4/10/2025 5:31 PM, Frank Li wrote:
> On Thu, Apr 10, 2025 at 04:30:28PM +0300, Ciprian Costea wrote:
>> From: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
>>
>> S32G274A-RDB2 and S32G399A-RDB3 boards have a PCA85073A RTC hardware module
>> available and connected over I2C0. Note that the RTC module is not battery
>> backed.
> 
> Suggested commit message:
> 
> arm64: dts: s32gxxxa-rdb: Add PCA85073A RTC module over I2C0
> 
> Add support for the PCA85073A RTC module connected via I2C0 on
> S32G274A-RDB2 and S32G399A-RDB3 boards.
> 
> Note that the RTC module is not battery backed.
> 
> Reviewed-by: Frank Li <Frank.Li@nxp.com>
> 

Hello Frank,

Thanks for your suggestion. I will send a V2 accordingly.

Regards,
Ciprian

>>
>> Signed-off-by: Ciprian Marian Costea <ciprianmarian.costea@oss.nxp.com>
>> ---
>>   arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
>> index ba53ec622f0b..4587e1cb8835 100644
>> --- a/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
>> +++ b/arch/arm64/boot/dts/freescale/s32gxxxa-rdb.dtsi
>> @@ -153,6 +153,11 @@ pcal6524: gpio-expander@22 {
>>   		gpio-controller;
>>   		#gpio-cells = <2>;
>>   	};
>> +
>> +	pca85073a: rtc@51 {
>> +		compatible = "nxp,pca85073a";
>> +		reg = <0x51>;
>> +	};
>>   };
>>
>>   &i2c2 {
>> --
>> 2.45.2
>>