* [PATCH] pinctrl: renesas: rzg2l: Limit 2.5V power supply to Ethernet interfaces
@ 2024-04-17 11:41 Paul Barker
2024-04-24 13:24 ` Geert Uytterhoeven
0 siblings, 1 reply; 3+ messages in thread
From: Paul Barker @ 2024-04-17 11:41 UTC (permalink / raw)
To: Geert Uytterhoeven, Linus Walleij
Cc: Paul Barker, Claudiu Beznea, linux-renesas-soc, linux-gpio, linux-kernel
The RZ/G3S SoC supports configurable supply voltages for several of its
I/O interfaces. All of these interfaces support both 1.8V and 3.3V
supplies, but only the Ethernet and XSPI interfaces support a 2.5V
supply.
Voltage selection for the XSPI interface is not yet supported, so this
leaves only the Ethernet interfaces currently supporting selection of a
2.5V supply. So we need to return an error if there is an attempt to
select a 2.5V supply for any non-Ethernet interface.
Fixes: 51996952b8b5 ("pinctrl: renesas: rzg2l: Add support to select power source for Ethernet pins")
Signed-off-by: Paul Barker <paul.barker.ct@bp.renesas.com>
---
drivers/pinctrl/renesas/pinctrl-rzg2l.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/pinctrl/renesas/pinctrl-rzg2l.c b/drivers/pinctrl/renesas/pinctrl-rzg2l.c
index eb5a8c654260..33602f0e4dad 100644
--- a/drivers/pinctrl/renesas/pinctrl-rzg2l.c
+++ b/drivers/pinctrl/renesas/pinctrl-rzg2l.c
@@ -892,6 +892,8 @@ static int rzg2l_set_power_source(struct rzg2l_pinctrl *pctrl, u32 pin, u32 caps
val = PVDD_1800;
break;
case 2500:
+ if (!(caps & (PIN_CFG_IO_VMC_ETH0 | PIN_CFG_IO_VMC_ETH1)))
+ return -EINVAL;
val = PVDD_2500;
break;
case 3300:
--
2.39.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] pinctrl: renesas: rzg2l: Limit 2.5V power supply to Ethernet interfaces
2024-04-17 11:41 [PATCH] pinctrl: renesas: rzg2l: Limit 2.5V power supply to Ethernet interfaces Paul Barker
@ 2024-04-24 13:24 ` Geert Uytterhoeven
2024-04-24 13:24 ` Geert Uytterhoeven
0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2024-04-24 13:24 UTC (permalink / raw)
To: Paul Barker
Cc: Linus Walleij, Claudiu Beznea, linux-renesas-soc, linux-gpio,
linux-kernel
On Wed, Apr 17, 2024 at 1:41 PM Paul Barker
<paul.barker.ct@bp.renesas.com> wrote:
> The RZ/G3S SoC supports configurable supply voltages for several of its
> I/O interfaces. All of these interfaces support both 1.8V and 3.3V
> supplies, but only the Ethernet and XSPI interfaces support a 2.5V
> supply.
>
> Voltage selection for the XSPI interface is not yet supported, so this
> leaves only the Ethernet interfaces currently supporting selection of a
> 2.5V supply. So we need to return an error if there is an attempt to
> select a 2.5V supply for any non-Ethernet interface.
>
> Fixes: 51996952b8b5 ("pinctrl: renesas: rzg2l: Add support to select power source for Ethernet pins")
> Signed-off-by: Paul Barker <paul.barker.ct@bp.renesas.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-clk for v6.10.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] pinctrl: renesas: rzg2l: Limit 2.5V power supply to Ethernet interfaces
2024-04-24 13:24 ` Geert Uytterhoeven
@ 2024-04-24 13:24 ` Geert Uytterhoeven
0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2024-04-24 13:24 UTC (permalink / raw)
To: Paul Barker
Cc: Linus Walleij, Claudiu Beznea, linux-renesas-soc, linux-gpio,
linux-kernel
On Wed, Apr 24, 2024 at 3:24 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Wed, Apr 17, 2024 at 1:41 PM Paul Barker
> <paul.barker.ct@bp.renesas.com> wrote:
> > The RZ/G3S SoC supports configurable supply voltages for several of its
> > I/O interfaces. All of these interfaces support both 1.8V and 3.3V
> > supplies, but only the Ethernet and XSPI interfaces support a 2.5V
> > supply.
> >
> > Voltage selection for the XSPI interface is not yet supported, so this
> > leaves only the Ethernet interfaces currently supporting selection of a
> > 2.5V supply. So we need to return an error if there is an attempt to
> > select a 2.5V supply for any non-Ethernet interface.
> >
> > Fixes: 51996952b8b5 ("pinctrl: renesas: rzg2l: Add support to select power source for Ethernet pins")
> > Signed-off-by: Paul Barker <paul.barker.ct@bp.renesas.com>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> i.e. will queue in renesas-clk for v6.10.
renesas-pinctrl, of course ;-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-04-24 13:24 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-17 11:41 [PATCH] pinctrl: renesas: rzg2l: Limit 2.5V power supply to Ethernet interfaces Paul Barker
2024-04-24 13:24 ` Geert Uytterhoeven
2024-04-24 13:24 ` Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).