From: Wolfram Sang <wsa+renesas@sang-engineering.com>
To: Claudiu Beznea <claudiu.beznea@kernel.org>
Cc: geert+renesas@glider.be, linusw@kernel.org, robh@kernel.org,
krzk+dt@kernel.org, conor+dt@kernel.org, magnus.damm@gmail.com,
claudiu.beznea@tuxon.dev, linux-renesas-soc@vger.kernel.org,
linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com>
Subject: Re: [PATCH 9/9] arm64: dts: renesas: rzg3s-smarc-som: Enable I3C
Date: Fri, 22 May 2026 14:29:29 +0200 [thread overview]
Message-ID: <ahBMKWrKQDko3cG8@shikoro> (raw)
In-Reply-To: <20260522102251.1723392-10-claudiu.beznea@kernel.org>
[-- Attachment #1: Type: text/plain, Size: 731 bytes --]
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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2026-05-22 12:29 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-22 10:22 [PATCH 0/9] pinctrl: renesas: rzg2l: Add support for RZ/G3S I3C Claudiu Beznea
2026-05-22 10:22 ` [PATCH 1/9] pinctrl: renesas: rzg2l: Generalize the power source code Claudiu Beznea
2026-05-22 10:22 ` [PATCH 2/9] pinctrl: renesas: rzg2l: Drop defines present in struct rzg2l_hwcfg Claudiu Beznea
2026-05-22 10:22 ` [PATCH 3/9] pinctrl: renesas: rzg2l: Keep member documentation aligned Claudiu Beznea
2026-05-22 10:22 ` [PATCH 4/9] pinctrl: renesas: rzg2l: Use tab instead of spaces Claudiu Beznea
2026-05-22 10:22 ` [PATCH 5/9] dt-bindings: pinctrl: renesas,rzg2l-pinctrl: Document the missing I3C power source option Claudiu Beznea
2026-05-22 16:11 ` Conor Dooley
2026-05-22 10:22 ` [PATCH 6/9] pinctrl: renesas: rzg2l: Add RZ/G3S support for selecting the I3C power source Claudiu Beznea
2026-05-22 10:22 ` [PATCH 7/9] dt-bindings: pinctrl: renesas,rzg2l-pinctrl: Document the I3C standby property Claudiu Beznea
2026-05-22 12:30 ` Wolfram Sang
2026-05-22 10:22 ` [PATCH 8/9] pinctrl: renesas: rzg2l: Add RZ/G3S support for selecting the I3C standby state Claudiu Beznea
2026-05-22 10:22 ` [PATCH 9/9] arm64: dts: renesas: rzg3s-smarc-som: Enable I3C Claudiu Beznea
2026-05-22 12:29 ` Wolfram Sang [this message]
2026-05-22 13:01 ` Claudiu Beznea
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ahBMKWrKQDko3cG8@shikoro \
--to=wsa+renesas@sang-engineering.com \
--cc=claudiu.beznea.uj@bp.renesas.com \
--cc=claudiu.beznea@kernel.org \
--cc=claudiu.beznea@tuxon.dev \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=krzk+dt@kernel.org \
--cc=linusw@kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox