Hi,

On Tue, Dec 05, 2017 at 05:24:47PM +0800, Chen-Yu Tsai wrote:

On Fri, Dec 1, 2017 at 9:44 PM, Quentin Schulz
[off-list ref] wrote:

quoted

This adds DT node for the GPIO/pinctrl part present in AXP813/AXP818.

Signed-off-by: Quentin Schulz <quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
---
 arch/arm/boot/dts/axp81x.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/axp81x.dtsi b/arch/arm/boot/dts/axp81x.dtsi
index 73b761f..0ef959d 100644
--- a/arch/arm/boot/dts/axp81x.dtsi
+++ b/arch/arm/boot/dts/axp81x.dtsi

@@ -48,6 +48,12 @@
        interrupt-controller;
        #interrupt-cells = <1>;

+       axp_gpio: axp-gpio {
+               compatible = "x-powers,axp813-gpio";
+               gpio-controller;
+               #gpio-cells = <2>;

What about interrupt-controller for directly referenced interrupts from
the GPIO pins?

There's a bit more to it to enable interrupts. You would probably need
to set up a chained interrupt controller in the GPIO driver, and in
the DTS with a interrupt-parent and interrupts properties pointing to
the AXP device itself.

Otherwise,

Acked-by: Chen-Yu Tsai <redacted>

Applied 9 and 10, thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com