RE: [PATCH] arm64: dts: renesas: r9a08g045: Move max-frequency to SoC dtsi

Biju Das <[email protected]>
Newsgroups org.kernel.vger.linux-renesas-soc,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID <TY3PR01MB113468B2AFB2409C955D5B0CE86DC2@TY3PR01MB11346.jpnprd01.prod.outlook.com>
Hi All,

> -----Original Message-----
> From: Biju <[email protected]>
> Sent: 19 June 2026 08:56
> Subject: [PATCH] arm64: dts: renesas: r9a08g045: Move max-frequency to SoC dtsi
> 
> From: Biju Das <[email protected]>
> 
> Move the max-frequency property for SDHI0/1/2 from the board-level SMARC dtsi files into the
> r9a08g045.dtsi SoC file, since these values reflect controller/SoC limitations rather than board-
> specific.
> 
> This removes the duplicated max-frequency = <125000000> entries for
> SDHI0 (both SD and eMMC variants) and SDHI1 in rzg3s-smarc-som.dtsi and rzg3s-smarc.dtsi, and the max-
> frequency = <50000000> entry for SDHI2, consolidating them as defaults in r9a08g045.dtsi instead.
> 
> Boards needing a different limit can still override max-frequency locally.
> 
> Signed-off-by: Biju Das <[email protected]>
> ---
>  arch/arm64/boot/dts/renesas/r9a08g045.dtsi       | 3 +++
>  arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi | 3 ---
>  arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi     | 1 -
>  3 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r9a08g045.dtsi b/arch/arm64/boot/dts/renesas/r9a08g045.dtsi
> index 3a69bb246bab..ae92d45ede38 100644
> --- a/arch/arm64/boot/dts/renesas/r9a08g045.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r9a08g045.dtsi
> @@ -655,6 +655,7 @@ sdhi0: mmc@11c00000  {
>  				 <&cpg CPG_MOD R9A08G045_SDHI0_IMCLK2>,
>  				 <&cpg CPG_MOD R9A08G045_SDHI0_ACLK>;
>  			clock-names = "core", "clkh", "cd", "aclk";
> +			max-frequency = <125000000>;
>  			resets = <&cpg R9A08G045_SDHI0_IXRST>;
>  			power-domains = <&cpg>;
>  			status = "disabled";
> @@ -670,6 +671,7 @@ sdhi1: mmc@11c10000 {
>  				 <&cpg CPG_MOD R9A08G045_SDHI1_IMCLK2>,
>  				 <&cpg CPG_MOD R9A08G045_SDHI1_ACLK>;
>  			clock-names = "core", "clkh", "cd", "aclk";
> +			max-frequency = <125000000>;
>  			resets = <&cpg R9A08G045_SDHI1_IXRST>;
>  			power-domains = <&cpg>;
>  			status = "disabled";
> @@ -685,6 +687,7 @@ sdhi2: mmc@11c20000 {
>  				 <&cpg CPG_MOD R9A08G045_SDHI2_IMCLK2>,
>  				 <&cpg CPG_MOD R9A08G045_SDHI2_ACLK>;
>  			clock-names = "core", "clkh", "cd", "aclk";
> +			max-frequency = <50000000>;
>  			resets = <&cpg R9A08G045_SDHI2_IXRST>;
>  			power-domains = <&cpg>;
>  			status = "disabled";
> diff --git a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi b/arch/arm64/boot/dts/renesas/rzg3s-smarc-
> som.dtsi
> index b45acfe6288a..9039a927bc46 100644
> --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi
> @@ -184,7 +184,6 @@ &sdhi0 {
>  	bus-width = <4>;
>  	sd-uhs-sdr50;
>  	sd-uhs-sdr104;
> -	max-frequency = <125000000>;
>  	status = "okay";
>  };
>  #else
> @@ -199,7 +198,6 @@ &sdhi0 {
>  	mmc-hs200-1_8v;
>  	non-removable;
>  	fixed-emmc-driver-type = <1>;
> -	max-frequency = <125000000>;
>  	status = "okay";
>  };
>  #endif
> @@ -210,7 +208,6 @@ &sdhi2 {
>  	pinctrl-names = "default";
>  	vmmc-supply = <&vcc_sdhi2>;
>  	bus-width = <4>;
> -	max-frequency = <50000000>;
>  	status = "okay";
>  };
>  #endif
> diff --git a/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi
> index 70af605168b0..e3821d8c01e3 100644
> --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi
> +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc.dtsi
> @@ -285,7 +285,6 @@ &sdhi1 {
>  	bus-width = <4>;
>  	sd-uhs-sdr50;
>  	sd-uhs-sdr104;
> -	max-frequency = <125000000>;
>  	status = "okay";
>  };
> 
> --
> 2.43.0

Gentle ping.

Cheers,
Biju
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.