RE: [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8' compatible string

From: yamada.masahiro
Date: Thu Jan 10 2019 - 03:02:10 EST




> -----Original Message-----
> From: Rob Herring [mailto:robh@xxxxxxxxxx]
> Sent: Thursday, January 10, 2019 5:30 AM
> To: arm@xxxxxxxxxx
> Cc: devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Andreas
> FÃrber <afaerber@xxxxxxx>; Manivannan Sadhasivam
> <manivannan.sadhasivam@xxxxxxxxxx>; Tsahee Zidenberg
> <tsahee@xxxxxxxxxxxxxxxxx>; Antoine Tenart <antoine.tenart@xxxxxxxxxxx>;
> Maxime Ripard <maxime.ripard@xxxxxxxxxxx>; Chen-Yu Tsai <wens@xxxxxxxx>;
> Dinh Nguyen <dinguyen@xxxxxxxxxx>; Kevin Hilman <khilman@xxxxxxxxxxxx>;
> Duc Dang <dhdang@xxxxxxx>; Liviu Dudau <liviu.dudau@xxxxxxx>; Sudeep Holla
> <sudeep.holla@xxxxxxx>; Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>;
> Ray Jui <rjui@xxxxxxxxxxxx>; Scott Branden <sbranden@xxxxxxxxxxxx>;
> bcm-kernel-feedback-list@xxxxxxxxxxxx; Robert Richter
> <rrichter@xxxxxxxxxx>; Jayachandran C <jnair@xxxxxxxxxxxxxxxxxx>; Kukjin
> Kim <kgene@xxxxxxxxxx>; Krzysztof Kozlowski <krzk@xxxxxxxxxx>; Wei Xu
> <xuwei5@xxxxxxxxxxxxx>; Chanho Min <chanho.min@xxxxxxx>; Jason Cooper
> <jason@xxxxxxxxxxxxxx>; Andrew Lunn <andrew@xxxxxxx>; Gregory Clement
> <gregory.clement@xxxxxxxxxxx>; Sebastian Hesselbarth
> <sebastian.hesselbarth@xxxxxxxxx>; Matthias Brugger
> <matthias.bgg@xxxxxxxxx>; Thierry Reding <thierry.reding@xxxxxxxxx>;
> Jonathan Hunter <jonathanh@xxxxxxxxxx>; Andy Gross
> <andy.gross@xxxxxxxxxx>; David Brown <david.brown@xxxxxxxxxx>; Simon
> Horman <horms@xxxxxxxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>; Heiko
> Stuebner <heiko@xxxxxxxxx>; Yamada, Masahiro/åç çå
> <yamada.masahiro@xxxxxxxxxxxxx>; Orson Zhai <orsonzhai@xxxxxxxxx>;
> Baolin Wang <baolin.wang@xxxxxxxxxx>; Chunyan Zhang
> <zhang.lyra@xxxxxxxxx>; Jisheng Zhang <Jisheng.Zhang@xxxxxxxxxxxxx>;
> Tero Kristo <t-kristo@xxxxxx>; Nishanth Menon <nm@xxxxxx>; Jun Nie
> <jun.nie@xxxxxxxxxx>; Shawn Guo <shawnguo@xxxxxxxxxx>;
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> linux-amlogic@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx;
> linux-mediatek@xxxxxxxxxxxxxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx;
> linux-arm-msm@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx;
> linux-rockchip@xxxxxxxxxxxxxxxxxxx; Mark Rutland <mark.rutland@xxxxxxx>;
> Will Deacon <will.deacon@xxxxxxx>; Michal Simek <michal.simek@xxxxxxxxxx>
> Subject: [PATCH] arm64: dts: Remove inconsistent use of 'arm,armv8'
> compatible string
>
> 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@xxxxxxxxxx>
> Cc: Mark Rutland <mark.rutland@xxxxxxx>
> Cc: Will Deacon <will.deacon@xxxxxxx>
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> armsoc folks, Please apply directly.
>
> Note that there's still a case left in msm8998 that needs a new
> compatible string.
>
> Rob

[> ]

> .../boot/dts/socionext/uniphier-ld11.dtsi | 4 +-
> .../boot/dts/socionext/uniphier-ld20.dtsi | 8 +-
> .../boot/dts/socionext/uniphier-pxs3.dtsi | 8 +-


For uniphier,

Acked-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>


BTW, do you have a plan to fix files under
Documentation/devicetree/bindings/ ?