All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: renesas: falcon-csi-dsi: Add GPIO extenders
@ 2021-04-13 18:14 Niklas Söderlund
  2021-04-14 19:54 ` Geert Uytterhoeven
  0 siblings, 1 reply; 2+ messages in thread
From: Niklas Söderlund @ 2021-04-13 18:14 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: linux-renesas-soc, Niklas Söderlund

We need to configure its GPIOs to power on the MAX96712.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
 .../dts/renesas/r8a779a0-falcon-csi-dsi.dtsi  | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi b/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi
index 14d3db5d6c16485d..928b182a76901276 100644
--- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi
@@ -12,4 +12,25 @@ eeprom@52 {
 		reg = <0x52>;
 		pagesize = <8>;
 	};
+
+	pca9654_a: gpio@21 {
+		compatible = "onnn,pca9654";
+		reg = <0x21>;
+		gpio-controller;
+		#gpio-cells = <2>;
+	};
+
+	pca9654_b: gpio@22 {
+		compatible = "onnn,pca9654";
+		reg = <0x22>;
+		gpio-controller;
+		#gpio-cells = <2>;
+	};
+
+	pca9654_c: gpio@23 {
+		compatible = "onnn,pca9654";
+		reg = <0x23>;
+		gpio-controller;
+		#gpio-cells = <2>;
+	};
 };
-- 
2.31.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] arm64: dts: renesas: falcon-csi-dsi: Add GPIO extenders
  2021-04-13 18:14 [PATCH] arm64: dts: renesas: falcon-csi-dsi: Add GPIO extenders Niklas Söderlund
@ 2021-04-14 19:54 ` Geert Uytterhoeven
  0 siblings, 0 replies; 2+ messages in thread
From: Geert Uytterhoeven @ 2021-04-14 19:54 UTC (permalink / raw)
  To: Niklas Söderlund; +Cc: Linux-Renesas

Hi Niklas,

Thanks for your patch!

On Tue, Apr 13, 2021 at 8:17 PM Niklas Söderlund
<niklas.soderlund+renesas@ragnatech.se> wrote:
> We need to configure its GPIOs to power on the MAX96712.

... their GPIOs ... MAX96712s?

>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

> --- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon-csi-dsi.dtsi
> @@ -12,4 +12,25 @@ eeprom@52 {
>                 reg = <0x52>;
>                 pagesize = <8>;
>         };
> +
> +       pca9654_a: gpio@21 {

Please obey numerical sort order by unit-address.

> +               compatible = "onnn,pca9654";
> +               reg = <0x21>;
> +               gpio-controller;
> +               #gpio-cells = <2>;
> +       };
> +
> +       pca9654_b: gpio@22 {
> +               compatible = "onnn,pca9654";
> +               reg = <0x22>;
> +               gpio-controller;
> +               #gpio-cells = <2>;
> +       };
> +
> +       pca9654_c: gpio@23 {
> +               compatible = "onnn,pca9654";
> +               reg = <0x23>;
> +               gpio-controller;
> +               #gpio-cells = <2>;
> +       };

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
No need to resend, will queue in renesas-devel for v5.14 with the above
fixed.

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] 2+ messages in thread

end of thread, other threads:[~2021-04-14 19:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-13 18:14 [PATCH] arm64: dts: renesas: falcon-csi-dsi: Add GPIO extenders Niklas Söderlund
2021-04-14 19:54 ` Geert Uytterhoeven

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.