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

Rob Herring
2019-01-10T14:16:24Z

On Wed, Jan 9, 2019 at 8:27 PM Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> wrote:
>
> On 14:29-20190109, Rob Herring wrote:
> > 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-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org>
> > Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
> > Cc: Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>
> > Signed-off-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > ---
> [...]
>
> ARM SoC maintainers, At least on a quick look (I could be wrong), I
> think we don't have any explicit processor bindings documentation.

bindings/arm/cpus.yaml

> However, most of SoC folks seem to key off matches else where I
> guess..
>
> Not sure, while at this, you if you are planning on fixing up
> Documentation/devicetree/bindings/arm/arm-boards:
> compatible = "arm,cortex-a53","arm,armv8";
> Documentation/devicetree/bindings/arm/cpu-capacity.txt:
> compatible = "arm,cortex-a53","arm,armv8";

I'll handle documentation separately.

Rob