Re: [PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-06-02 Thread Marc Zyngier 
On 16/05/17 08:57, Andrew Jeffery wrote:
> In addition to introducing the new compatible string the bindings
> description is reworked to be more generic.
> 
> Signed-off-by: Andrew Jeffery 

Queued for 4.13.

Thanks,

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

Re: [PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-06-02 Thread Marc Zyngier 
On 16/05/17 08:57, Andrew Jeffery wrote:
> In addition to introducing the new compatible string the bindings
> description is reworked to be more generic.
> 
> Signed-off-by: Andrew Jeffery 

Queued for 4.13.

Thanks,

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

Re: [PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-05-22 Thread Rob Herring 
On Tue, May 16, 2017 at 03:57:47PM +0800, Andrew Jeffery wrote:
> In addition to introducing the new compatible string the bindings
> description is reworked to be more generic.
> 
> Signed-off-by: Andrew Jeffery 
> ---
>  .../bindings/interrupt-controller/aspeed,ast2400-vic.txt | 9 
> +
>  drivers/irqchip/irq-aspeed-vic.c | 3 ++-
>  2 files changed, 7 insertions(+), 5 deletions(-)

Acked-by: Rob Herring

Re: [PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-05-22 Thread Rob Herring 
On Tue, May 16, 2017 at 03:57:47PM +0800, Andrew Jeffery wrote:
> In addition to introducing the new compatible string the bindings
> description is reworked to be more generic.
> 
> Signed-off-by: Andrew Jeffery 
> ---
>  .../bindings/interrupt-controller/aspeed,ast2400-vic.txt | 9 
> +
>  drivers/irqchip/irq-aspeed-vic.c | 3 ++-
>  2 files changed, 7 insertions(+), 5 deletions(-)

Acked-by: Rob Herring

[PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-05-16 Thread Andrew Jeffery 
In addition to introducing the new compatible string the bindings
description is reworked to be more generic.

Signed-off-by: Andrew Jeffery 
---
 .../bindings/interrupt-controller/aspeed,ast2400-vic.txt | 9 +
 drivers/irqchip/irq-aspeed-vic.c | 3 ++-
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git 
a/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt 
b/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
index 6c6e85324b9d..e3fea0758d25 100644
--- 
a/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
+++ 
b/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
@@ -1,12 +1,13 @@
 Aspeed Vectored Interrupt Controller
 
-These bindings are for the Aspeed AST2400 interrupt controller register layout.
-The SoC has an legacy register layout, but this driver does not support that
-mode of operation.
+These bindings are for the Aspeed interrupt controller. The AST2400 and
+AST2500 SoC families include a legacy register layout before a re-designed
+layout, but the bindings do not prescribe the use of one or the other.
 
 Required properties:
 
-- compatible : should be "aspeed,ast2400-vic".
+- compatible : "aspeed,ast2400-vic"
+   "aspeed,ast2500-vic"
 
 - interrupt-controller : Identifies the node as an interrupt controller
 - #interrupt-cells : Specifies the number of cells needed to encode an
diff --git a/drivers/irqchip/irq-aspeed-vic.c b/drivers/irqchip/irq-aspeed-vic.c
index d24451d5bf8a..1c908ce25a9f 100644
--- a/drivers/irqchip/irq-aspeed-vic.c
+++ b/drivers/irqchip/irq-aspeed-vic.c
@@ -227,4 +227,5 @@ static int __init avic_of_init(struct device_node *node,
return 0;
 }
 
-IRQCHIP_DECLARE(aspeed_new_vic, "aspeed,ast2400-vic", avic_of_init);
+IRQCHIP_DECLARE(ast2400_vic, "aspeed,ast2400-vic", avic_of_init);
+IRQCHIP_DECLARE(ast2500_vic, "aspeed,ast2500-vic", avic_of_init);
-- 
2.9.3

[PATCH] irqchip: aspeed: Add AST2500 compatible string

2017-05-16 Thread Andrew Jeffery 
In addition to introducing the new compatible string the bindings
description is reworked to be more generic.

Signed-off-by: Andrew Jeffery 
---
 .../bindings/interrupt-controller/aspeed,ast2400-vic.txt | 9 +
 drivers/irqchip/irq-aspeed-vic.c | 3 ++-
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git 
a/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt 
b/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
index 6c6e85324b9d..e3fea0758d25 100644
--- 
a/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
+++ 
b/Documentation/devicetree/bindings/interrupt-controller/aspeed,ast2400-vic.txt
@@ -1,12 +1,13 @@
 Aspeed Vectored Interrupt Controller
 
-These bindings are for the Aspeed AST2400 interrupt controller register layout.
-The SoC has an legacy register layout, but this driver does not support that
-mode of operation.
+These bindings are for the Aspeed interrupt controller. The AST2400 and
+AST2500 SoC families include a legacy register layout before a re-designed
+layout, but the bindings do not prescribe the use of one or the other.
 
 Required properties:
 
-- compatible : should be "aspeed,ast2400-vic".
+- compatible : "aspeed,ast2400-vic"
+   "aspeed,ast2500-vic"
 
 - interrupt-controller : Identifies the node as an interrupt controller
 - #interrupt-cells : Specifies the number of cells needed to encode an
diff --git a/drivers/irqchip/irq-aspeed-vic.c b/drivers/irqchip/irq-aspeed-vic.c
index d24451d5bf8a..1c908ce25a9f 100644
--- a/drivers/irqchip/irq-aspeed-vic.c
+++ b/drivers/irqchip/irq-aspeed-vic.c
@@ -227,4 +227,5 @@ static int __init avic_of_init(struct device_node *node,
return 0;
 }
 
-IRQCHIP_DECLARE(aspeed_new_vic, "aspeed,ast2400-vic", avic_of_init);
+IRQCHIP_DECLARE(ast2400_vic, "aspeed,ast2400-vic", avic_of_init);
+IRQCHIP_DECLARE(ast2500_vic, "aspeed,ast2500-vic", avic_of_init);
-- 
2.9.3