Aw: Re: [PATCH v1 1/3] arm64: dts: marvell: Fix anyOf conditional failed
From: Frank Wunderlich
Date: Mon Nov 11 2024 - 16:39:33 EST
> Gesendet: Montag, 11. November 2024 um 21:36
> Von: "Rob Herring" <robh@xxxxxxxxxx>
> An: "Frank Wunderlich" <linux@xxxxxxxxx>
> CC: "Damien Le Moal" <dlemoal@xxxxxxxxxx>, "Niklas Cassel" <cassel@xxxxxxxxxx>, "Krzysztof Kozlowski" <krzk+dt@xxxxxxxxxx>, "Conor Dooley" <conor+dt@xxxxxxxxxx>, "Andrew Lunn" <andrew@xxxxxxx>, "Gregory Clement" <gregory.clement@xxxxxxxxxxx>, "Sebastian Hesselbarth" <sebastian.hesselbarth@xxxxxxxxx>, "Russell King" <linux@xxxxxxxxxxxxxxx>, "Frank Wunderlich" <frank-w@xxxxxxxxxxxxxxx>, "Hans de Goede" <hdegoede@xxxxxxxxxx>, "Jens Axboe" <axboe@xxxxxxxxx>, linux-ide@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Betreff: Re: [PATCH v1 1/3] arm64: dts: marvell: Fix anyOf conditional failed
>
> On Sat, Nov 09, 2024 at 10:46:19AM +0100, Frank Wunderlich wrote:
> > From: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
>
> Thanks for reviving these.
>
> > after converting the ahci-platform binding to yaml the following files
> > reporting "'anyOf' conditional failed" on
>
> Here and the subject, "fixing anyOf" isn't very specific and is just an
> implementation detail of the schema. "Add missing required 'phys'
> property" would be more exact.
imho it does not match what patch does...i do not add required phys...i just disable the nodes and enable them only where phys is set.
> Rob
> </frank-w@xxxxxxxxxxxxxxx></axboe@xxxxxxxxx></hdegoede@xxxxxxxxxx></frank-w@xxxxxxxxxxxxxxx></linux@xxxxxxxxxxxxxxx></sebastian.hesselbarth@xxxxxxxxx></gregory.clement@xxxxxxxxxxx></andrew@xxxxxxx></conor+dt@xxxxxxxxxx></krzk+dt@xxxxxxxxxx></cassel@xxxxxxxxxx></dlemoal@xxxxxxxxxx></linux@xxxxxxxxx></robh@xxxxxxxxxx>