Re: [PATCH 9/9] arm64: dts: renesas: rzg3s-smarc-som: Enable I3C

[Wolfram Sang]

Hi Claudiu,

> +	i3c_pins: i3c {
> +		pins = "I3C_SDA", "I3C_SCL";
> +#if SW_CONFIG4 == SW_ON
> +		power-source = <1200>;
> +#else
> +		power-source = <1800>;
> +#endif
> +		input-enable;
> +		renesas,i3c-standby = <0>;
> +	};
> +
> +	i3c_standby_pins: i3c-standby {
> +		pins = "I3C_SDA", "I3C_SCL";
> +#if SW_CONFIG4 == SW_ON
> +		power-source = <1200>;
> +#else
> +		power-source = <1800>;
> +#endif
> +		renesas,i3c-standby = <1>;
> +	};

That doesn't look like HW description to me because the pins are the
same for regular and standby mode. Their state is changed when powering
down, but that should not need DT encoding, I'd think. The driver should
know what to do, or? Am I missing something?

Happy hacking,

   Wolfram

Attachment: signature.asc
Description: PGP signature