Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <691102a4-e920-ef88-3c5b-8240b8f87c5b@linaro.org>
Date:   Mon, 31 Jul 2023 10:49:07 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
From:   neil.armstrong@linaro.org
Reply-To: neil.armstrong@linaro.org
Subject: Re: [PATCH V2 0/3] Add binding header file for GPIO interrupt of
 Amlogic Meson-G12A
Content-Language: en-US
To:     linus.walleij@linaro.org
Cc:     robh+dt@kernel.org, martin.blumenstingl@googlemail.com,
        jbrunet@baylibre.com, khilman@baylibre.com, conor+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org,
        Huqiang Qin <huqiang.qin@amlogic.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-gpio@vger.kernel.org
References: <20230724060108.1403662-1-huqiang.qin@amlogic.com>
Organization: Linaro Developer Services
In-Reply-To: <20230724060108.1403662-1-huqiang.qin@amlogic.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Linus,

On 24/07/2023 08:01, Huqiang Qin wrote:
> After adding this binding header file, you can intuitively find
> the interrupt number corresponding to each GPIO, and apply it
> in the format of IRQID_XXX where it needs to be used.
> 
> Associated platforms:
> - Amlogic Meson-G12A
> - Amlogic Meson-G12B
> - Amlogic Meson-SM1
> 
> [PATCH 1/3]:
>    V1 -> V2: Rename irq-meson-g12a-gpio.h to amlogic,meson-g12a-gpio-intc.h
> 
> [PATCH 2/3]:
>    V1 -> V2: Include header file changed to amlogic,meson-g12a-gpio-intc.h
> 
> [PATCH 3/3]:
>    V1 -> V2: Include header file changed to amlogic,meson-g12a-gpio-intc.h
> 
> Huqiang Qin (3):
>    dt-bindings: interrupt-controller: Add header file for Amlogic
>      Meson-G12A SoCs
>    pinctrl: Replace the IRQ number in the driver with the IRQID macro
>      definition
>    arm64: dts: Replace the IRQ number with the IRQID macro definition

What's the best way to apply this patchset ?

Thanks,
Neil

> 
>   .../boot/dts/amlogic/meson-g12-common.dtsi    |   1 +
>   .../boot/dts/amlogic/meson-g12a-x96-max.dts   |   2 +-
>   .../dts/amlogic/meson-g12b-bananapi-cm4.dtsi  |   2 +-
>   .../boot/dts/amlogic/meson-g12b-bananapi.dtsi |   2 +-
>   .../amlogic/meson-g12b-odroid-go-ultra.dts    |   4 +-
>   .../dts/amlogic/meson-g12b-odroid-n2.dtsi     |   2 +-
>   .../boot/dts/amlogic/meson-g12b-w400.dtsi     |   2 +-
>   .../dts/amlogic/meson-sm1-a95xf3-air-gbit.dts |   2 +-
>   .../boot/dts/amlogic/meson-sm1-bananapi.dtsi  |   4 +-
>   .../boot/dts/amlogic/meson-sm1-h96-max.dts    |   2 +-
>   .../boot/dts/amlogic/meson-sm1-odroid.dtsi    |   2 +-
>   .../boot/dts/amlogic/meson-sm1-sei610.dts     |  12 +-
>   .../dts/amlogic/meson-sm1-x96-air-gbit.dts    |   2 +-
>   drivers/pinctrl/meson/pinctrl-meson-g12a.c    |  35 ++---
>   .../amlogic,meson-g12a-gpio-intc.h            | 126 ++++++++++++++++++
>   15 files changed, 164 insertions(+), 36 deletions(-)
>   create mode 100644 include/dt-bindings/interrupt-controller/amlogic,meson-g12a-gpio-intc.h
> 
> 
> base-commit: c58c49dd89324b18a812762a2bfa5a0458e4f252