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


Groups > linux.kernel > #1561604 > unrolled thread

[PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps

Started byMarc Zyngier <marc.zyngier@arm.com>
First post2017-01-18 12:20 +0100
Last post2017-01-19 18:30 +0100
Articles 4 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps Marc Zyngier <marc.zyngier@arm.com> - 2017-01-18 12:20 +0100
    Re: [PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps Arnd Bergmann <arnd@arndb.de> - 2017-01-18 13:40 +0100
      Re: [PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps Marc Zyngier <marc.zyngier@arm.com> - 2017-01-18 14:10 +0100
        Re: [PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps Tony Lindgren <tony@atomide.com> - 2017-01-19 18:30 +0100

#1561604 — [PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps

FromMarc Zyngier <marc.zyngier@arm.com>
Date2017-01-18 12:20 +0100
Subject[PATCH 0/2] ARM: DTS: Fix broken GICv2 register maps
Message-ID<t0W8d-2X3-109@gated-at.bofh.it>
For a GICv2 (which happens to be virtualization capable), the
architecture mandates the following regions:

	     GICD: 4kB
	     GICC: 8kB
	     GICH: 8kB
	     GICV: 8kB

Unfortunately, I made a mistake in one of the examples contained in
the DT binding document, and everyone duplicated that same mistake all
over the map.

This small series fixes the DT binding, and hopefully updates all the
offending DTs to be compliant with the architecture.

Marc Zyngier (2):
  dt-bindings: arm,gic: Fix binding example for a virt-capable GIC
  ARM: DTS: Fix register map for virt-capable GIC

 Documentation/devicetree/bindings/interrupt-controller/arm,gic.txt | 4 ++--
 arch/arm/boot/dts/alpine.dtsi                                      | 2 +-
 arch/arm/boot/dts/axm55xx.dtsi                                     | 2 +-
 arch/arm/boot/dts/dra7.dtsi                                        | 2 +-
 arch/arm/boot/dts/ecx-2000.dts                                     | 2 +-
 arch/arm/boot/dts/exynos3250.dtsi                                  | 2 +-
 arch/arm/boot/dts/exynos5.dtsi                                     | 2 +-
 arch/arm/boot/dts/exynos5260.dtsi                                  | 2 +-
 arch/arm/boot/dts/exynos5440.dtsi                                  | 2 +-
 arch/arm/boot/dts/imx6ul.dtsi                                      | 4 ++--
 arch/arm/boot/dts/keystone-k2g.dtsi                                | 2 +-
 arch/arm/boot/dts/keystone.dtsi                                    | 2 +-
 arch/arm/boot/dts/ls1021a.dtsi                                     | 4 ++--
 arch/arm/boot/dts/mt2701.dtsi                                      | 2 +-
 arch/arm/boot/dts/mt6580.dtsi                                      | 2 +-
 arch/arm/boot/dts/mt6589.dtsi                                      | 2 +-
 arch/arm/boot/dts/mt7623.dtsi                                      | 2 +-
 arch/arm/boot/dts/mt8127.dtsi                                      | 2 +-
 arch/arm/boot/dts/mt8135.dtsi                                      | 2 +-
 arch/arm/boot/dts/omap5.dtsi                                       | 2 +-
 arch/arm/boot/dts/r8a73a4.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7743.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7745.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7790.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7791.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7792.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7793.dtsi                                     | 2 +-
 arch/arm/boot/dts/r8a7794.dtsi                                     | 2 +-
 arch/arm/boot/dts/rk1108.dtsi                                      | 2 +-
 arch/arm/boot/dts/rk3036.dtsi                                      | 2 +-
 arch/arm/boot/dts/rk322x.dtsi                                      | 2 +-
 arch/arm/boot/dts/rk3288.dtsi                                      | 2 +-
 arch/arm/boot/dts/sun6i-a31.dtsi                                   | 2 +-
 arch/arm/boot/dts/sun7i-a20.dtsi                                   | 4 ++--
 arch/arm/boot/dts/sun8i-a23-a33.dtsi                               | 2 +-
 arch/arm/boot/dts/sun8i-a83t.dtsi                                  | 2 +-
 arch/arm/boot/dts/sun8i-h3.dtsi                                    | 2 +-
 arch/arm/boot/dts/sun9i-a80.dtsi                                   | 2 +-
 38 files changed, 42 insertions(+), 42 deletions(-)

-- 
2.1.4

[toc] | [next] | [standalone]


#1561715

FromArnd Bergmann <arnd@arndb.de>
Date2017-01-18 13:40 +0100
Message-ID<t0XGW-469-5@gated-at.bofh.it>
In reply to#1561604
On Wednesday, January 18, 2017 10:53:29 AM CET Marc Zyngier wrote:
> For a GICv2 (which happens to be virtualization capable), the
> architecture mandates the following regions:
> 
> 	     GICD: 4kB
> 	     GICC: 8kB
> 	     GICH: 8kB
> 	     GICV: 8kB
> 
> Unfortunately, I made a mistake in one of the examples contained in
> the DT binding document, and everyone duplicated that same mistake all
> over the map.
> 
> This small series fixes the DT binding, and hopefully updates all the
> offending DTs to be compliant with the architecture.
> 

Looks good to me, can you send this as a pull request to arm@kernel.org
once you have collected a reasonable number of Acks?

Acked-by: Arnd Bergmann <arnd@arndb.de>

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


#1561746

FromMarc Zyngier <marc.zyngier@arm.com>
Date2017-01-18 14:10 +0100
Message-ID<t0Y9Z-4vs-57@gated-at.bofh.it>
In reply to#1561715
On 18/01/17 12:34, Arnd Bergmann wrote:
> On Wednesday, January 18, 2017 10:53:29 AM CET Marc Zyngier wrote:
>> For a GICv2 (which happens to be virtualization capable), the
>> architecture mandates the following regions:
>>
>> 	     GICD: 4kB
>> 	     GICC: 8kB
>> 	     GICH: 8kB
>> 	     GICV: 8kB
>>
>> Unfortunately, I made a mistake in one of the examples contained in
>> the DT binding document, and everyone duplicated that same mistake all
>> over the map.
>>
>> This small series fixes the DT binding, and hopefully updates all the
>> offending DTs to be compliant with the architecture.
>>
> 
> Looks good to me, can you send this as a pull request to arm@kernel.org
> once you have collected a reasonable number of Acks?

Sure, will do.

> Acked-by: Arnd Bergmann <arnd@arndb.de>

Thanks,

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

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


#1562961

FromTony Lindgren <tony@atomide.com>
Date2017-01-19 18:30 +0100
Message-ID<t1oH8-4qY-19@gated-at.bofh.it>
In reply to#1561746
* Marc Zyngier <marc.zyngier@arm.com> [170118 05:09]:
> On 18/01/17 12:34, Arnd Bergmann wrote:
> > On Wednesday, January 18, 2017 10:53:29 AM CET Marc Zyngier wrote:
> >> For a GICv2 (which happens to be virtualization capable), the
> >> architecture mandates the following regions:
> >>
> >> 	     GICD: 4kB
> >> 	     GICC: 8kB
> >> 	     GICH: 8kB
> >> 	     GICV: 8kB
> >>
> >> Unfortunately, I made a mistake in one of the examples contained in
> >> the DT binding document, and everyone duplicated that same mistake all
> >> over the map.
> >>
> >> This small series fixes the DT binding, and hopefully updates all the
> >> offending DTs to be compliant with the architecture.
> >>
> > 
> > Looks good to me, can you send this as a pull request to arm@kernel.org
> > once you have collected a reasonable number of Acks?
> 
> Sure, will do.
> 
> > Acked-by: Arnd Bergmann <arnd@arndb.de>

Here's yet another ack to paste for both patches:

Acked-by: Tony Lindgren <tony@atomide.com>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web