[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<TY3PR01MB113466DE7061B60485B3BE97986852@TY3PR01MB11346.jpnprd01.prod.outlook.com>
Date: Mon, 12 Aug 2024 07:40:00 +0000
From: Biju Das <biju.das.jz@...renesas.com>
To: Prabhakar <prabhakar.csengg@...il.com>, Geert Uytterhoeven
<geert+renesas@...der.be>, Magnus Damm <magnus.damm@...il.com>, Rob Herring
<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>
CC: "linux-renesas-soc@...r.kernel.org" <linux-renesas-soc@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Fabrizio
Castro <fabrizio.castro.jz@...esas.com>, Prabhakar Mahadev Lad
<prabhakar.mahadev-lad.rj@...renesas.com>
Subject: RE: [PATCH v2 7/8] arm64: dts: renesas: r9a09g057h44-gp-evk: Enable
OSTM, I2C, and SDHI
Hi Prabhakar,
Thanks for the patch.
> -----Original Message-----
> From: Prabhakar <prabhakar.csengg@...il.com>
> Sent: Sunday, August 11, 2024 9:50 PM
> Subject: [PATCH v2 7/8] arm64: dts: renesas: r9a09g057h44-gp-evk: Enable OSTM, I2C, and SDHI
>
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
>
> Enable OSTM0-OSTM7, RIIC{0,1,2,3,6,7,8}, and SDHI1 (available on the SD2
> connector) on the RZ/V2H GP-EVK platform.
>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>
> ---
> v1->v2
> - New patch
> ---
> .../boot/dts/renesas/r9a09g057h44-gp-evk.dts | 191 ++++++++++++++++++
> 1 file changed, 191 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r9a09g057h44-gp-evk.dts
> b/arch/arm64/boot/dts/renesas/r9a09g057h44-gp-evk.dts
> index 593c48181248..11c13c85d278 100644
> --- a/arch/arm64/boot/dts/renesas/r9a09g057h44-gp-evk.dts
> +++ b/arch/arm64/boot/dts/renesas/r9a09g057h44-gp-evk.dts
> @@ -7,6 +7,8 @@
>
> /dts-v1/;
>
> +#include <dt-bindings/pinctrl/rzg2l-pinctrl.h>
> +#include <dt-bindings/gpio/gpio.h>
> #include "r9a09g057.dtsi"
>
> / {
> @@ -14,6 +16,14 @@ / {
> compatible = "renesas,gp-evk", "renesas,r9a09g057h44", "renesas,r9a09g057";
>
> aliases {
> + i2c0 = &i2c0;
> + i2c1 = &i2c1;
> + i2c2 = &i2c2;
> + i2c3 = &i2c3;
> + i2c6 = &i2c6;
> + i2c7 = &i2c7;
> + i2c8 = &i2c8;
> + mmc1 = &sdhi1;
> serial0 = &scif;
> };
>
> @@ -32,17 +42,186 @@ memory@...000000 {
> device_type = "memory";
> reg = <0x2 0x40000000 0x2 0x00000000>;
> };
> +
> + reg_3p3v: regulator1 {
> + compatible = "regulator-fixed";
> +
> + regulator-name = "fixed-3.3V";
> + regulator-min-microvolt = <3300000>;
> + regulator-max-microvolt = <3300000>;
> + regulator-boot-on;
> + regulator-always-on;
> + };
> +
> + vqmmc_sdhi1: regulator-vccq-sdhi1 {
> + compatible = "regulator-gpio";
> + regulator-name = "SDHI1 VccQ";
> + gpios = <&pinctrl RZG2L_GPIO(10, 2) GPIO_ACTIVE_HIGH>;
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <3300000>;
> + gpios-states = <0>;
> + states = <3300000 0>, <1800000 1>;
> + };
> };
>
> &audio_extal_clk {
> clock-frequency = <22579200>;
> };
>
> +&i2c0 {
> + pinctrl-0 = <&i2c0_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
clock-frequency = <100000>; in SoC dtsim
Why frequency set to 100kHz for all the i2c nodes even though SoC supports
Transfer rate up to 1MHz? Is it board limitation restricting to 100kHz?
Cheers,
Biju
> +&i2c1 {
> + pinctrl-0 = <&i2c1_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&i2c2 {
> + pinctrl-0 = <&i2c2_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&i2c3 {
> + pinctrl-0 = <&i2c3_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&i2c6 {
> + pinctrl-0 = <&i2c6_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&i2c7 {
> + pinctrl-0 = <&i2c7_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&i2c8 {
> + pinctrl-0 = <&i2c8_pins>;
> + pinctrl-names = "default";
> +
> + status = "okay";
> +};
> +
> +&ostm0 {
> + status = "okay";
> +};
> +
> +&ostm1 {
> + status = "okay";
> +};
> +
> +&ostm2 {
> + status = "okay";
> +};
> +
> +&ostm3 {
> + status = "okay";
> +};
> +
> +&ostm4 {
> + status = "okay";
> +};
> +
> +&ostm5 {
> + status = "okay";
> +};
> +
> +&ostm6 {
> + status = "okay";
> +};
> +
> +&ostm7 {
> + status = "okay";
> +};
> +
> &pinctrl {
> + i2c0_pins: i2c0 {
> + pinmux = <RZG2L_PORT_PINMUX(3, 0, 1)>, /* I2C0_SDA */
> + <RZG2L_PORT_PINMUX(3, 1, 1)>; /* I2C0_SCL */
> + };
> +
> + i2c1_pins: i2c1 {
> + pinmux = <RZG2L_PORT_PINMUX(3, 2, 1)>, /* I2C1_SDA */
> + <RZG2L_PORT_PINMUX(3, 3, 1)>; /* I2C1_SCL */
> + };
> +
> + i2c2_pins: i2c2 {
> + pinmux = <RZG2L_PORT_PINMUX(2, 0, 4)>, /* I2C2_SDA */
> + <RZG2L_PORT_PINMUX(2, 1, 4)>; /* I2C2_SCL */
> + };
> +
> + i2c3_pins: i2c3 {
> + pinmux = <RZG2L_PORT_PINMUX(3, 6, 1)>, /* I2C3_SDA */
> + <RZG2L_PORT_PINMUX(3, 7, 1)>; /* I2C3_SCL */
> + };
> +
> + i2c6_pins: i2c6 {
> + pinmux = <RZG2L_PORT_PINMUX(4, 4, 1)>, /* I2C6_SDA */
> + <RZG2L_PORT_PINMUX(4, 5, 1)>; /* I2C6_SCL */
> + };
> +
> + i2c7_pins: i2c7 {
> + pinmux = <RZG2L_PORT_PINMUX(4, 6, 1)>, /* I2C7_SDA */
> + <RZG2L_PORT_PINMUX(4, 7, 1)>; /* I2C7_SCL */
> + };
> +
> + i2c8_pins: i2c8 {
> + pinmux = <RZG2L_PORT_PINMUX(0, 6, 1)>, /* I2C8_SDA */
> + <RZG2L_PORT_PINMUX(0, 7, 1)>; /* I2C8_SCL */
> + };
> +
> scif_pins: scif {
> pins = "SCIF_TXD", "SCIF_RXD";
> renesas,output-impedance = <1>;
> };
> +
> + sd1-pwr-en-hog {
> + gpio-hog;
> + gpios = <RZG2L_GPIO(10, 3) GPIO_ACTIVE_HIGH>;
> + output-high;
> + line-name = "sd1_pwr_en";
> + };
> +
> + sdhi1_pins: sd1 {
> + sd1_data {
> + pins = "SD1DAT0", "SD1DAT1", "SD1DAT2", "SD1DAT3";
> + input-enable;
> + renesas,output-impedance = <3>;
> + slew-rate = <1>;
> + };
> +
> + sd1_cmd {
> + pins = "SD1CMD";
> + input-enable;
> + renesas,output-impedance = <3>;
> + slew-rate = <1>;
> + };
> +
> + sd1_clk {
> + pins = "SD1CLK";
> + renesas,output-impedance = <3>;
> + slew-rate = <1>;
> + };
> +
> + sd1_cd {
> + pinmux = <RZG2L_PORT_PINMUX(9, 4, 14)>; /* SD1_CD */
> + };
> + };
> };
>
> &qextal_clk {
> @@ -59,3 +238,15 @@ &scif {
>
> status = "okay";
> };
> +
> +&sdhi1 {
> + pinctrl-0 = <&sdhi1_pins>;
> + pinctrl-1 = <&sdhi1_pins>;
> + pinctrl-names = "default", "state_uhs";
> + vmmc-supply = <®_3p3v>;
> + vqmmc-supply = <&vqmmc_sdhi1>;
> + bus-width = <4>;
> + sd-uhs-sdr50;
> + sd-uhs-sdr104;
> + status = "okay";
> +};
> --
> 2.34.1
Powered by blists - more mailing lists