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

[Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>]

> Gesendet: Montag, 11. November 2024 um 21:36
&gt; Von: "Rob Herring" <robh@xxxxxxxxxx>
&gt; An: "Frank Wunderlich" <linux@xxxxxxxxx>
&gt; 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
&gt; Betreff: Re: [PATCH v1 1/3] arm64: dts: marvell: Fix anyOf conditional failed
&gt;
&gt; On Sat, Nov 09, 2024 at 10:46:19AM +0100, Frank Wunderlich wrote:
&gt; &gt; From: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
&gt;
&gt; Thanks for reviving these.
&gt;
&gt; &gt; after converting the ahci-platform binding to yaml the following files
&gt; &gt; reporting "'anyOf' conditional failed" on
&gt;
&gt; Here and the subject, "fixing anyOf" isn't very specific and is just an
&gt; implementation detail of the schema. "Add missing required 'phys'
&gt; 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.

&gt; Rob
&gt; </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>