Aw: Re: [PATCH v1 1/3] arm64: dts: marvell: Fix anyOf conditional failed

[Frank Wunderlich]

> Gesendet: Montag, 11. November 2024 um 21:36
> Von: "Rob Herring" <robh at kernel.org>
> An: "Frank Wunderlich" <linux at fw-web.de>
> CC: "Damien Le Moal" <dlemoal at kernel.org>, "Niklas Cassel" <cassel at kernel.org>, "Krzysztof Kozlowski" <krzk+dt at kernel.org>, "Conor Dooley" <conor+dt at kernel.org>, "Andrew Lunn" <andrew at lunn.ch>, "Gregory Clement" <gregory.clement at bootlin.com>, "Sebastian Hesselbarth" <sebastian.hesselbarth at gmail.com>, "Russell King" <linux at armlinux.org.uk>, "Frank Wunderlich" <frank-w at public-files.de>, "Hans de Goede" <hdegoede at redhat.com>, "Jens Axboe" <axboe at kernel.dk>, linux-ide at vger.kernel.org, devicetree at vger.kernel.org, linux-kernel at vger.kernel.org, linux-arm-kernel at lists.infradead.org
> 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 at public-files.de>
>
> 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 at public-files.de></axboe at kernel.dk></hdegoede at redhat.com></frank-w at public-files.de></linux at armlinux.org.uk></sebastian.hesselbarth at gmail.com></gregory.clement at bootlin.com></andrew at lunn.ch></conor+dt at kernel.org></krzk+dt at kernel.org></cassel at kernel.org></dlemoal at kernel.org></linux at fw-web.de></robh at kernel.org>