Re: [PATCH v2 4/5] riscv: dts: thead: Add IO labels for the IO expansion on I2C1

[Drew Fustini]

On Tue, Jul 21, 2026 at 01:28:58AM +0800, Icenowy Zheng wrote:
> 在 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

Great, I'll fixup on apply.

Reviewed-by: Drew Fustini <fustini@xxxxxxxxxx>    

Thanks,
Drew