[PATCH 0/3] irqchip: meson-gpio: add support for Meson-AXG SoCs

[PATCH 0/3] irqchip: meson-gpio: add support for Meson-AXG SoCs

[Yixun Lan]

  This series try to add GPIO interrupt controller support for Meson-AXG SoCs.
  The first patch is a trivial typo fix, I can fold the first two patches
together if necessary.

Yixun Lan (3):
  dt-bindings: interrupt-controller: fix the double quotes
  dt-bindings: interrupt-controller: New binding for Meson-AXG SoC
  irqchip/meson-gpio: add support for Meson-AXG SoCs

 .../bindings/interrupt-controller/amlogic,meson-gpio-intc.txt | 11 ++++++-----
 drivers/irqchip/irq-meson-gpio.c                              |  5 +++++
 2 files changed, 11 insertions(+), 5 deletions(-)

-- 
2.16.2

[PATCH 1/3] dt-bindings: interrupt-controller: fix the double quotes

[Yixun Lan]

The double quotes seems not ASCII type, fix it here.

Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
---
 .../bindings/interrupt-controller/amlogic,meson-gpio-intc.txt  | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
index a83f9a5734ca..c753d99d43db 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
+++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
@@ -9,11 +9,11 @@ number of interrupt exposed depends on the SoC.
 
 Required properties:
 
-- compatible : must have "amlogic,meson8-gpio-intc” and either
-   “amlogic,meson8-gpio-intc” for meson8 SoCs (S802) or
-   “amlogic,meson8b-gpio-intc” for meson8b SoCs (S805) or
-   “amlogic,meson-gxbb-gpio-intc” for GXBB SoCs (S905) or
-   “amlogic,meson-gxl-gpio-intc” for GXL SoCs (S905X, S912)
+- compatible : must have "amlogic,meson8-gpio-intc" and either
+    "amlogic,meson8-gpio-intc" for meson8 SoCs (S802) or
+    "amlogic,meson8b-gpio-intc" for meson8b SoCs (S805) or
+    "amlogic,meson-gxbb-gpio-intc" for GXBB SoCs (S905) or
+    "amlogic,meson-gxl-gpio-intc" for GXL SoCs (S905X, S912)
 - interrupt-parent : a phandle to the GIC the interrupts are routed to.
    Usually this is provided at the root level of the device tree as it is
    common to most of the SoC.
-- 
2.16.2

Re: [PATCH 1/3] dt-bindings: interrupt-controller: fix the double quotes

[Rob Herring]

On Sun, Apr 08, 2018 at 02:56:58PM +0000, Yixun Lan wrote:
> The double quotes seems not ASCII type, fix it here.
> 
> Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
> ---
>  .../bindings/interrupt-controller/amlogic,meson-gpio-intc.txt  | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 2/3] dt-bindings: interrupt-controller: New binding for Meson-AXG SoC

[Yixun Lan]

Update the dt-binding documentation to support new compatible string
for the GPIO interrupt controller which found in Amlogic's Meson-AXG SoC.

Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
---
 .../devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
index c753d99d43db..89674ad8a097 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
+++ b/Documentation/devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt
@@ -14,6 +14,7 @@ Required properties:
     "amlogic,meson8b-gpio-intc" for meson8b SoCs (S805) or
     "amlogic,meson-gxbb-gpio-intc" for GXBB SoCs (S905) or
     "amlogic,meson-gxl-gpio-intc" for GXL SoCs (S905X, S912)
+    "amlogic,meson-axg-gpio-intc" for AXG SoCs (A113D, A113X)
 - interrupt-parent : a phandle to the GIC the interrupts are routed to.
    Usually this is provided at the root level of the device tree as it is
    common to most of the SoC.
-- 
2.16.2

Re: [PATCH 2/3] dt-bindings: interrupt-controller: New binding for Meson-AXG SoC

[Rob Herring]

On Sun, Apr 08, 2018 at 02:56:59PM +0000, Yixun Lan wrote:
> Update the dt-binding documentation to support new compatible string
> for the GPIO interrupt controller which found in Amlogic's Meson-AXG SoC.
> 
> Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
> ---
>  .../devicetree/bindings/interrupt-controller/amlogic,meson-gpio-intc.txt | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Rob Herring <robh@kernel.org>

[PATCH 3/3] irqchip/meson-gpio: add support for Meson-AXG SoCs

[Yixun Lan]

The Meson-AXG SoC uses the same GPIO interrupt controller IP block as the other
Meson SoCs. A total of 100 pins can be spied on, which is the sum of:
- 255:100 Undefined(no interrupt)
- 99:84, 16 pins on bank GPIOY
- 83:61, 23 pins on bank GPIOX
- 60:40, 21 pins on bank GPIOA
- 39:25, 15 pins on bank BOOT
- 24:14, 11 pins on bank GPIOZ
- 13:0 , 14 pins in the AO domain

Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
---
 drivers/irqchip/irq-meson-gpio.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/irqchip/irq-meson-gpio.c b/drivers/irqchip/irq-meson-gpio.c
index a59bdbc0b9bb..7b531fd075b8 100644
--- a/drivers/irqchip/irq-meson-gpio.c
+++ b/drivers/irqchip/irq-meson-gpio.c
@@ -63,11 +63,16 @@ static const struct meson_gpio_irq_params gxl_params = {
 	.nr_hwirq = 110,
 };
 
+static const struct meson_gpio_irq_params axg_params = {
+	.nr_hwirq = 100,
+};
+
 static const struct of_device_id meson_irq_gpio_matches[] = {
 	{ .compatible = "amlogic,meson8-gpio-intc", .data = &meson8_params },
 	{ .compatible = "amlogic,meson8b-gpio-intc", .data = &meson8b_params },
 	{ .compatible = "amlogic,meson-gxbb-gpio-intc", .data = &gxbb_params },
 	{ .compatible = "amlogic,meson-gxl-gpio-intc", .data = &gxl_params },
+	{ .compatible = "amlogic,meson-axg-gpio-intc", .data = &axg_params },
 	{ }
 };
 
-- 
2.16.2

Re: [PATCH 0/3] irqchip: meson-gpio: add support for Meson-AXG SoCs

[Yixun Lan]

Hi Marc (or anyone else)

On 04/08/18 22:56, Yixun Lan wrote:
>   This series try to add GPIO interrupt controller support for Meson-AXG SoCs.
>   The first patch is a trivial typo fix, I can fold the first two patches
> together if necessary.
> 
> Yixun Lan (3):
>   dt-bindings: interrupt-controller: fix the double quotes
>   dt-bindings: interrupt-controller: New binding for Meson-AXG SoC
>   irqchip/meson-gpio: add support for Meson-AXG SoCs
> 
>  .../bindings/interrupt-controller/amlogic,meson-gpio-intc.txt | 11 ++++++-----
>  drivers/irqchip/irq-meson-gpio.c                              |  5 +++++
>  2 files changed, 11 insertions(+), 5 deletions(-)
> 

please consider this merely a ping..
will you take this series, or is there anything holding this?

thanks

Yixun

Re: [PATCH 0/3] irqchip: meson-gpio: add support for Meson-AXG SoCs

[Marc Zyngier]

On 16/05/18 03:50, Yixun Lan wrote:
> Hi Marc (or anyone else)
> 
> On 04/08/18 22:56, Yixun Lan wrote:
>>   This series try to add GPIO interrupt controller support for Meson-AXG SoCs.
>>   The first patch is a trivial typo fix, I can fold the first two patches
>> together if necessary.
>>
>> Yixun Lan (3):
>>   dt-bindings: interrupt-controller: fix the double quotes
>>   dt-bindings: interrupt-controller: New binding for Meson-AXG SoC
>>   irqchip/meson-gpio: add support for Meson-AXG SoCs
>>
>>  .../bindings/interrupt-controller/amlogic,meson-gpio-intc.txt | 11 ++++++-----
>>  drivers/irqchip/irq-meson-gpio.c                              |  5 +++++
>>  2 files changed, 11 insertions(+), 5 deletions(-)
>>
> 
> please consider this merely a ping..
> will you take this series, or is there anything holding this?

Sure, I'll queue that for 4.18.

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...