Re: [PATCH v2 7/9] dt-bindings: arm: stm32: Switch st,stm32mp23/25-syscfg into simple-mfd
Krzysztof Kozlowski <[email protected]>
| Newsgroups | org.infradead.lists.linux-phy,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.linux-usb |
|---|---|
| Message-ID | <20260818-invisible-polar-cow-fb2a93@quoll> |
On Sun, Aug 16, 2026 at 11:37:09PM +0200, Marek Vasut wrote: > The st,stm32mp23-syscfg and st,stm32mp25-syscfg can have subnodes, which > represent the USB 2.0 FEMTO-PHY. Turn the syscfg into "simple-mfd" so the > PHYs would get populated by the OS. > > Signed-off-by: Marek Vasut <[email protected]> > --- > Cc: Alexandre Torgue <[email protected]> > Cc: Christian Bruel <[email protected]> > Cc: Conor Dooley <[email protected]> > Cc: Fabrice Gasnier <[email protected]> > Cc: Greg Kroah-Hartman <[email protected]> > Cc: Krzysztof Kozlowski <[email protected]> > Cc: Maxime Coquelin <[email protected]> > Cc: Neil Armstrong <[email protected]> > Cc: Pankaj Dev <[email protected]> > Cc: Rahul Kumar <[email protected]> > Cc: Rob Herring <[email protected]> > Cc: Rosen Penev <[email protected]> > Cc: Thinh Nguyen <[email protected]> > Cc: Vinod Koul <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > --- > V2: New patch > --- > .../devicetree/bindings/arm/stm32/st,stm32-syscon.yaml | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > index 95d2319afe235..ef640443dd239 100644 > --- a/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > +++ b/Documentation/devicetree/bindings/arm/stm32/st,stm32-syscon.yaml > @@ -22,11 +22,12 @@ properties: > - st,stm32mp151-pwr-mcu > - st,stm32mp157-syscfg > - st,stm32mp21-syscfg > - - st,stm32mp23-syscfg > - - st,stm32mp25-syscfg > - const: syscon > - items: > - - const: st,stm32-tamp > + - enum: > + - st,stm32-tamp > + - st,stm32mp23-syscfg > + - st,stm32mp25-syscfg You never tested your code - DTS in this case. This clearly generates warnings. Best regards, Krzysztof -- linux-phy mailing list [email protected] https://lists.infradead.org/mailman/listinfo/linux-phy