> Gesendet: Montag, 11. November 2024 um 18:15
> Von: "Andrew Lunn" <andrew@xxxxxxx>
> An: "Rob Herring" <robh@xxxxxxxxxx>
> CC: frank-w@xxxxxxxxxxxxxxx, "Frank Wunderlich" <linux@xxxxxxxxx>, "Damien Le Moal" <dlemoal@xxxxxxxxxx>, "Niklas Cassel" <cassel@xxxxxxxxxx>, "Krzysztof Kozlowski" <krzk+dt@xxxxxxxxxx>, "Conor Dooley" <conor+dt@xxxxxxxxxx>, "Gregory Clement" <gregory.clement@xxxxxxxxxxx>, "Sebastian Hesselbarth" <sebastian.hesselbarth@xxxxxxxxx>, "Russell King" <linux@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 Mon, Nov 11, 2024 at 10:25:12AM -0600, Rob Herring wrote:
> > On Sun, Nov 10, 2024 at 3:25 AM Frank Wunderlich
> > <frank-w@xxxxxxxxxxxxxxx> wrote:
> > >
> > > Am 9. November 2024 18:29:44 MEZ schrieb Andrew Lunn <andrew@xxxxxxx>:
> > > >On Sat, Nov 09, 2024 at 10:46:19AM +0100, Frank Wunderlich wrote:
> > > >> From: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
> > > >>
> > > >> after converting the ahci-platform binding to yaml the following files
> > > >> reporting "'anyOf' conditional failed" on
> > > >>
> > > >> sata@540000: sata-port@0
> > > >> diff --git a/arch/arm64/boot/dts/marvell/armada-7040-db.dts b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> > > >> index 1e0ab35cc686..2b5e45d2c5a6 100644
> > > >> --- a/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> > > >> +++ b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
> > > >> @@ -214,6 +214,7 @@ &cp0_sata0 {
> > > >>
> > > >> sata-port@1 {
> > > >> phys = <&cp0_comphy3 1>;
> > > >> + status = "okay";
> > > >> };
> > > >> };
> > > >
> > > >>
> > > >> diff --git a/arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts b/arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts
> > > >> index 7af949092b91..6bdc4f1e6939 100644
> > > >> --- a/arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts
> > > >> +++ b/arch/arm64/boot/dts/marvell/armada-7040-mochabin.dts
> > > >> @@ -433,11 +433,13 @@ &cp0_sata0 {
> > > >> /* 7 + 12 SATA connector (J24) */
> > > >> sata-port@0 {
> > > >> phys = <&cp0_comphy2 0>;
> > > >> + status = "okay";
> > > >> };
> > > >>
> > > >> /* M.2-2250 B-key (J39) */
> > > >> sata-port@1 {
> > > >> phys = <&cp0_comphy3 1>;
> > > >> + status = "okay";
> > > >> };
> > > >> };
> > > >> diff --git a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> > > >> index 7e595ac80043..161beec0b6b0 100644
> > > >> --- a/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> > > >> +++ b/arch/arm64/boot/dts/marvell/armada-cp11x.dtsi
> > > >> @@ -347,10 +347,12 @@ CP11X_LABEL(sata0): sata@540000 {
> > > >>
> > > >> sata-port@0 {
> > > >> reg = <0>;
> > > >> + status = "disabled";
> > > >> };
> > > >
> > > >I don't know the yaml too well, but it is not obvious how adding a few
> > > >status = "disabled"; status = "okay"; fixes a "'anyOf' conditional failed".
> > > >
> > > >Maybe you can expand the explanation a bit?
> > > >
> > > > Andrew
> > >
> > > Hi angelo,
> > >
> > > I guess the dtbs_check only checks required properties from yaml if the node is enabled.
> >
> > Yes, that is exactly how it works.
>
> So from this, can i imply that phys is a required property?
>
> Looking at the above patch, it appears that for armada-*.dts,
> sata-port@0 always uses phys = <&cp0_comphy2 0> and sata-port@1 uses
> phys = <&cp0_comphy3 1>. Is this an actual SoC property? Could it be
> moved up into the .dtsi file? Or is it really a board property?
as i said the phy may operate in different modes (not know marvell here, but on other vendors phys are defined at board level), maybe the boards where the phy is missing the phy is used in another mode. Without knowing the SoC and boards disable it at SoC-level and enable only the nodes containing a phys property is all i can do here to fix the issue. Imho it is always a good idea to enable only the conrollers a board will use.
> Andrew
> </frank-w@xxxxxxxxxxxxxxx></andrew@xxxxxxx></frank-w@xxxxxxxxxxxxxxx></axboe@xxxxxxxxx></hdegoede@xxxxxxxxxx></linux@xxxxxxxxxxxxxxx></sebastian.hesselbarth@xxxxxxxxx></gregory.clement@xxxxxxxxxxx></conor+dt@xxxxxxxxxx></krzk+dt@xxxxxxxxxx></cassel@xxxxxxxxxx></dlemoal@xxxxxxxxxx></linux@xxxxxxxxx></robh@xxxxxxxxxx></andrew@xxxxxxx>
