On Thu, Jan 10, 2019 at 8:18 PM Richter, Robert
<Robert.Richter@xxxxxxxxxx> wrote:
>
> Rob,
>
> On 09.01.19 14:29:34, Rob Herring wrote:
> > External Email
> >
> > 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.
>
> > diff --git a/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi b/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi
> > index ff5c4c47b22b..c616132f7d6d 100644
> > --- a/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi
> > +++ b/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi
> > @@ -27,28 +27,28 @@
> >
> > cpu@0 {
> > device_type = "cpu";
> > - compatible = "cavium,thunder2", "brcm,vulcan", "arm,armv8";
> > + compatible = "cavium,thunder2";
>
> We are just checking if removing the "brcm,vulcan" string has any
> impact on early silicon revisions we want to support (see also
> BRCM_CPU_PART_VULCAN).
>
> Maybe you make this a separate change?
Dropping brcm,vulcan was unintentional (regex fail). Though keeping it
means we need a schema change as it was not documented to be a
fallback.
Rob
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-rockchip
