Hi Biju, On Fri, Jul 12, 2019 at 3:22 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote: > > Subject: Re: [PATCH 1/3] arm64: dts: renesas: hihope-common: Add PCA9654 I/O expander > > On Fri, Jul 5, 2019 at 12:20 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote: > > > Enable PCA9654 GPIO expander, so that we can configure its GPIOs later. > > > > > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > > > > --- a/arch/arm64/boot/dts/renesas/hihope-common.dtsi > > > +++ b/arch/arm64/boot/dts/renesas/hihope-common.dtsi > > > @@ -162,6 +162,13 @@ > > > clock-frequency = <400000>; > > > status = "okay"; > > > > > > + gpio_expander: gpio@20 { > > > + compatible = "onnn,pca9654"; > > > + reg = <0x20>; > > > + gpio-controller; > > > + #gpio-cells = <2>; > > > > You may want to describe that the interrupt output is tied to GP5_25? > > BT_REG_ON and WLAN_REG_ON are output pins. That is the reason, I haven't described the interrupt output. > If needed, I can add this. Please let me know. OK, it's fine for me (for now). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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