Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1643892 > unrolled thread

[PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc

Started byIcenowy Zheng <icenowy@aosc.io>
First post2017-05-18 09:20 +0200
Last post2017-05-18 10:00 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc Icenowy Zheng <icenowy@aosc.io> - 2017-05-18 09:20 +0200
    Re: [linux-sunxi] [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to  the binding doc Chen-Yu Tsai <wens@csie.org> - 2017-05-18 09:50 +0200
    Re: [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding  doc Maxime Ripard <maxime.ripard@free-electrons.com> - 2017-05-18 10:00 +0200

#1643892 — [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc

FromIcenowy Zheng <icenowy@aosc.io>
Date2017-05-18 09:20 +0200
Subject[PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc
Message-ID<tInT5-6TZ-29@gated-at.bofh.it>
The A31 NMI driver seems to be using wrong base address.

As we're going to convert to use a correct NMI base address (and
correctly name it to R_INTC as the datasheet suggests), add a new
compatible string for the "correct" R_INTC, which we will use for A64
SoC.

Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
Acked-by: Rob Herring <robh@kernel.org>
---
Changes in v6:
- Added Rob's ACK.

 .../bindings/interrupt-controller/allwinner,sunxi-nmi.txt          | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/allwinner,sunxi-nmi.txt b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sunxi-nmi.txt
index 81cd3692405e..fea0c6a6211f 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/allwinner,sunxi-nmi.txt
+++ b/Documentation/devicetree/bindings/interrupt-controller/allwinner,sunxi-nmi.txt
@@ -3,8 +3,11 @@ Allwinner Sunxi NMI Controller
 
 Required properties:
 
-- compatible : should be "allwinner,sun7i-a20-sc-nmi" or
-  "allwinner,sun6i-a31-sc-nmi" or "allwinner,sun9i-a80-nmi"
+- compatible : should be one of:
+	"allwinner,sun6i-a31-sc-nmi"
+	"allwinner,sun7i-a20-sc-nmi"
+	"allwinner,sun9i-a80-nmi"
+	"allwinner,sun50i-a64-r-intc"
 - reg : Specifies base physical address and size of the registers.
 - interrupt-controller : Identifies the node as an interrupt controller
 - #interrupt-cells : Specifies the number of cells needed to encode an
-- 
2.12.2

[toc] | [next] | [standalone]


#1643911 — Re: [linux-sunxi] [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc

FromChen-Yu Tsai <wens@csie.org>
Date2017-05-18 09:50 +0200
SubjectRe: [linux-sunxi] [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc
Message-ID<tIom6-7a7-17@gated-at.bofh.it>
In reply to#1643892
On Thu, May 18, 2017 at 3:16 PM, Icenowy Zheng <icenowy@aosc.io> wrote:
> The A31 NMI driver seems to be using wrong base address.
>
> As we're going to convert to use a correct NMI base address (and
> correctly name it to R_INTC as the datasheet suggests), add a new
> compatible string for the "correct" R_INTC, which we will use for A64
> SoC.
>
> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> Acked-by: Rob Herring <robh@kernel.org>

The subject prefix might need a bit of work, but otherwise

Reviewed-by: Chen-Yu Tsai <wens@csie.org>

[toc] | [prev] | [next] | [standalone]


#1643925 — Re: [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2017-05-18 10:00 +0200
SubjectRe: [PATCH v6 1/9] irqchip/sunxi-nmi: add A64 R_INTC to the binding doc
Message-ID<tIovM-7em-21@gated-at.bofh.it>
In reply to#1643892

[Multipart message — attachments visible in raw view] — view raw

On Thu, May 18, 2017 at 03:16:45PM +0800, Icenowy Zheng wrote:
> The A31 NMI driver seems to be using wrong base address.
> 
> As we're going to convert to use a correct NMI base address (and
> correctly name it to R_INTC as the datasheet suggests), add a new
> compatible string for the "correct" R_INTC, which we will use for A64
> SoC.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.io>
> Acked-by: Rob Herring <robh@kernel.org>

Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>

Maxime

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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web