| Message ID | 47d7bdbb86a241c8b61efba8cad97de4dca40eb5.1578579516.git.michal.simek@xilinx.com |
|---|---|
| State | Accepted |
| Commit | 8e3501ec900f8cab490345003b9bb9ed425fe2f8 |
| Headers | show |
| Series | xilinx: Various DT changes and syncups | expand |
On Thu, Jan 9, 2020 at 11:19 PM Michal Simek <michal.simek at xilinx.com> wrote: > > From: Rob Herring <robh at kernel.org> > > The 'arm,armv8' compatible string is only for software models. It adds > little value otherwise and is inconsistently used as a fallback on some > platforms. Remove it from those platforms. > > This fixes warnings generated by the DT schema. > > Reported-by: Michal Simek <michal.simek at xilinx.com> > Cc: Mark Rutland <mark.rutland at arm.com> > Cc: Will Deacon <will.deacon at arm.com> > Acked-by: Antoine Tenart <antoine.tenart at bootlin.com> > Acked-by: Nishanth Menon <nm at ti.com> > Acked-by: Maxime Ripard <maxime.ripard at bootlin.com> > Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org> > Acked-by: Chanho Min <chanho.min at lge.com> > Acked-by: Krzysztof Kozlowski <krzk at kernel.org> > Acked-by: Masahiro Yamada <yamada.masahiro at socionext.com> > Acked-by: Gregory CLEMENT <gregory.clement at bootlin.com> > Acked-by: Thierry Reding <treding at nvidia.com> > Acked-by: Heiko Stuebner <heiko at sntech.de> > Acked-by: Simon Horman <horms+renesas at verge.net.au> > Acked-by: Tero Kristo <t-kristo at ti.com> > Acked-by: Wei Xu <xuwei5 at hisilicon.com> > Acked-by: Liviu Dudau <liviu.dudau at arm.com> > Acked-by: Matthias Brugger <matthias.bgg at gmail.com> > Acked-by: Michal Simek <michal.simek at xilinx.com> > Acked-by: Scott Branden <scott.branden at broadcom.com> > Acked-by: Kevin Hilman <khilman at baylibre.com> > Acked-by: Chunyan Zhang <zhang.lyra at gmail.com> > Acked-by: Robert Richter <rrichter at cavium.com> > Acked-by: Jisheng Zhang <Jisheng.Zhang at synaptics.com> > Acked-by: Dinh Nguyen <dinguyen at kernel.org> I know you are importing this commit from Linux, but these Ack lines look weird to me... > Signed-off-by: Rob Herring <robh at kernel.org> > Signed-off-by: Arnd Bergmann <arnd at arndb.de> > Signed-off-by: Michal Simek <michal.simek at xilinx.com> > --- > > arch/arm/dts/zynqmp.dtsi | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi > index b453941baf05..8646c4c3a340 100644 > --- a/arch/arm/dts/zynqmp.dtsi > +++ b/arch/arm/dts/zynqmp.dtsi > @@ -25,7 +25,7 @@ > #size-cells = <0>; > > cpu0: cpu at 0 { > - compatible = "arm,cortex-a53", "arm,armv8"; > + compatible = "arm,cortex-a53"; > device_type = "cpu"; > enable-method = "psci"; > operating-points-v2 = <&cpu_opp_table>; > @@ -34,7 +34,7 @@ > }; > > cpu1: cpu at 1 { > - compatible = "arm,cortex-a53", "arm,armv8"; > + compatible = "arm,cortex-a53"; > device_type = "cpu"; > enable-method = "psci"; > reg = <0x1>; > @@ -43,7 +43,7 @@ > }; > > cpu2: cpu at 2 { > - compatible = "arm,cortex-a53", "arm,armv8"; > + compatible = "arm,cortex-a53"; > device_type = "cpu"; > enable-method = "psci"; > reg = <0x2>; > @@ -52,7 +52,7 @@ > }; > > cpu3: cpu at 3 { > - compatible = "arm,cortex-a53", "arm,armv8"; > + compatible = "arm,cortex-a53"; > device_type = "cpu"; > enable-method = "psci"; > reg = <0x3>; > -- > 2.24.0 >
On 10. 01. 20 16:36, Masahiro Yamada wrote: > On Thu, Jan 9, 2020 at 11:19 PM Michal Simek <michal.simek at xilinx.com> wrote: >> >> From: Rob Herring <robh at kernel.org> >> >> The 'arm,armv8' compatible string is only for software models. It adds >> little value otherwise and is inconsistently used as a fallback on some >> platforms. Remove it from those platforms. >> >> This fixes warnings generated by the DT schema. >> >> Reported-by: Michal Simek <michal.simek at xilinx.com> >> Cc: Mark Rutland <mark.rutland at arm.com> >> Cc: Will Deacon <will.deacon at arm.com> >> Acked-by: Antoine Tenart <antoine.tenart at bootlin.com> >> Acked-by: Nishanth Menon <nm at ti.com> >> Acked-by: Maxime Ripard <maxime.ripard at bootlin.com> >> Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam at linaro.org> >> Acked-by: Chanho Min <chanho.min at lge.com> >> Acked-by: Krzysztof Kozlowski <krzk at kernel.org> >> Acked-by: Masahiro Yamada <yamada.masahiro at socionext.com> >> Acked-by: Gregory CLEMENT <gregory.clement at bootlin.com> >> Acked-by: Thierry Reding <treding at nvidia.com> >> Acked-by: Heiko Stuebner <heiko at sntech.de> >> Acked-by: Simon Horman <horms+renesas at verge.net.au> >> Acked-by: Tero Kristo <t-kristo at ti.com> >> Acked-by: Wei Xu <xuwei5 at hisilicon.com> >> Acked-by: Liviu Dudau <liviu.dudau at arm.com> >> Acked-by: Matthias Brugger <matthias.bgg at gmail.com> >> Acked-by: Michal Simek <michal.simek at xilinx.com> >> Acked-by: Scott Branden <scott.branden at broadcom.com> >> Acked-by: Kevin Hilman <khilman at baylibre.com> >> Acked-by: Chunyan Zhang <zhang.lyra at gmail.com> >> Acked-by: Robert Richter <rrichter at cavium.com> >> Acked-by: Jisheng Zhang <Jisheng.Zhang at synaptics.com> >> Acked-by: Dinh Nguyen <dinguyen at kernel.org> > > > I know you are importing this commit from Linux, > but these Ack lines look weird to me... I have not a problem to remove them but not sure if this is the right thing to do. Thanks, Michal
diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi index b453941baf05..8646c4c3a340 100644 --- a/arch/arm/dts/zynqmp.dtsi +++ b/arch/arm/dts/zynqmp.dtsi @@ -25,7 +25,7 @@ #size-cells = <0>; cpu0: cpu at 0 { - compatible = "arm,cortex-a53", "arm,armv8"; + compatible = "arm,cortex-a53"; device_type = "cpu"; enable-method = "psci"; operating-points-v2 = <&cpu_opp_table>; @@ -34,7 +34,7 @@ }; cpu1: cpu at 1 { - compatible = "arm,cortex-a53", "arm,armv8"; + compatible = "arm,cortex-a53"; device_type = "cpu"; enable-method = "psci"; reg = <0x1>; @@ -43,7 +43,7 @@ }; cpu2: cpu at 2 { - compatible = "arm,cortex-a53", "arm,armv8"; + compatible = "arm,cortex-a53"; device_type = "cpu"; enable-method = "psci"; reg = <0x2>; @@ -52,7 +52,7 @@ }; cpu3: cpu at 3 { - compatible = "arm,cortex-a53", "arm,armv8"; + compatible = "arm,cortex-a53"; device_type = "cpu"; enable-method = "psci"; reg = <0x3>;