Re: [PATCH v2 4/5] riscv: dts: thead: Add IO labels for the IO expansion on I2C1
From: Icenowy Zheng
Date: Mon Jul 20 2026 - 13:34:42 EST
在 2026-07-20一的 09:56 -0700,Drew Fustini写道:
> On Fri, Jul 17, 2026 at 12:11:23PM +0800, Icenowy Zheng wrote:
> >
> >
> > 于 2026年7月17日 GMT+08:00 04:52:19,Drew Fustini <fustini@xxxxxxxxxx>
> > 写道:
> > > On Tue, Jul 14, 2026 at 03:45:14PM +0800, Icenowy Zheng wrote:
> > > > From: Emil Renner Berthing <emil.renner.berthing@xxxxxxxxxxxxx>
> > > >
> > > > Add labels for GPIOs on the IO expansion chip on the I2C1 bus.
> > > >
> > > > Signed-off-by: Emil Renner Berthing
> > > > <emil.renner.berthing@xxxxxxxxxxxxx>
> > > > [Icenowy: Splitted off the patch adding all 3 IO expansions]
> > > > Signed-off-by: Icenowy Zheng <zhengxingda@xxxxxxxxxxx>
> > > > ---
> > > > arch/riscv/boot/dts/thead/th1520-lichee-pi-4a.dts | 8 ++++++++
> > > > 1 file changed, 8 insertions(+)
> > > >
> > > > diff --git a/arch/riscv/boot/dts/thead/th1520-lichee-pi-4a.dts
> > > > b/arch/riscv/boot/dts/thead/th1520-lichee-pi-4a.dts
> > > > index 74b120c6b063a..78f5979a56c14 100644
> > > > --- a/arch/riscv/boot/dts/thead/th1520-lichee-pi-4a.dts
> > > > +++ b/arch/riscv/boot/dts/thead/th1520-lichee-pi-4a.dts
> > > > @@ -128,6 +128,14 @@ ioexp2: gpio@18 {
> > > > reg = <0x18>;
> > > > gpio-controller;
> > > > #gpio-cells = <2>;
> > > > + gpio-line-names = "",
> > > > + "cam0_reset",
> > > > + "cam1_reset",
> > > > + "cam2_reset",
> > > > + "wl_host_wake",
> > >
> > > I am not sure the line name really matters all that, but it seems
> > > that
> > > pin 4 is also used as a reset gpio:
> >
> > The "host" here is subject, which means "this pins is used by the
> > host to wake the wl".
> >
> > Maybe I should take the schematics name, "wl_host_wake_dev"? Or use
> > another name on the page for Wi-Fi, "wl_reg_on"?
>
> I thnk "wl_host_wake_dev" is slightly better in terms of searching
> the
> PDF [1] and finding the "IO Expansion 2" section. This is minor
> change
> so I can just fix up when applying if you are okay with that.
Of course I am okay with this.
Thanks,
Icenowy
>
> Thanks,
> Drew
>
> [1] https://dl.sipeed.com/shareURL/LICHEE/licheepi4a/02_Schematic