| Message ID | 20200507112430.183940-7-andre.przywara@arm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | dts/dt-bindings: Fix Arm Ltd. ARMv8 "boards" | expand |
On 2020-05-07 12:24, Andre Przywara wrote: > The GIC DT binding only allows a certain combination of DT compatible > strings, mostly just consisting of one name. > > Drop the somewhat awkward combination of multiple names using different > "cortex" based strings, in the fastmodel .dts files. > > Signed-off-by: Andre Przywara <andre.przywara@arm.com> > --- > arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi | 2 +- > arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi > b/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi > index 15fe81738e94..61a1750fcdd6 100644 > --- a/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi > +++ b/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi > @@ -6,7 +6,7 @@ > > / { > gic: interrupt-controller@2c001000 { > - compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic"; > + compatible = "arm,cortex-a15-gic"; > #interrupt-cells = <3>; > #address-cells = <2>; > interrupt-controller; > diff --git a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts > b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts > index c5d15cbd8cf6..f86f6451411f 100644 > --- a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts > +++ b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts > @@ -95,7 +95,7 @@ > }; > > gic: interrupt-controller@2c001000 { > - compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic"; > + compatible = "arm,cortex-a15-gic"; > #interrupt-cells = <3>; > #address-cells = <0>; > interrupt-controller; Since you've now allowed "arm,gic-400", why don't you use that in combination with "arm,cortex-a15-gic"? Having GIC-400 makes a lot more sense, as these two models are ARMv8 models. Thanks, M.
diff --git a/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi b/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi index 15fe81738e94..61a1750fcdd6 100644 --- a/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi +++ b/arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi @@ -6,7 +6,7 @@ / { gic: interrupt-controller@2c001000 { - compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic"; + compatible = "arm,cortex-a15-gic"; #interrupt-cells = <3>; #address-cells = <2>; interrupt-controller; diff --git a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts index c5d15cbd8cf6..f86f6451411f 100644 --- a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts +++ b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts @@ -95,7 +95,7 @@ }; gic: interrupt-controller@2c001000 { - compatible = "arm,cortex-a15-gic", "arm,cortex-a9-gic"; + compatible = "arm,cortex-a15-gic"; #interrupt-cells = <3>; #address-cells = <0>; interrupt-controller;
The GIC DT binding only allows a certain combination of DT compatible strings, mostly just consisting of one name. Drop the somewhat awkward combination of multiple names using different "cortex" based strings, in the fastmodel .dts files. Signed-off-by: Andre Przywara <andre.przywara@arm.com> --- arch/arm64/boot/dts/arm/foundation-v8-gicv2.dtsi | 2 +- arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)