On Fri, Feb 21, 2025 at 08:40:04AM +0100, Dimitri Fedrau via B4 Relay wrote: > From: Dimitri Fedrau <dimitri.fedrau@xxxxxxxxxxxx> > > Currently the flexcan driver does only support adding PHYs by using the > "old" regulator bindings. Add support for CAN transceivers as a PHY. > > Signed-off-by: Dimitri Fedrau <dimitri.fedrau@xxxxxxxxxxxx> > --- > .../devicetree/bindings/net/can/fsl,flexcan.yaml | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > > diff --git a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > index 73252fe56fe6c8e9fd19142208bb655dc86d47cd..81125883cf86b9d19616bde378f74bdb6a32f1b2 100644 > --- a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > +++ b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > @@ -77,6 +77,9 @@ properties: > xceiver-supply: > description: Regulator that powers the CAN transceiver. > > + phys: > + maxItems: 1 > + > big-endian: > $ref: /schemas/types.yaml#/definitions/flag > description: | > @@ -171,6 +174,18 @@ allOf: > interrupts: > maxItems: 1 > interrupt-names: false > + - if: > + required: > + - xceiver-supply > + then: > + properties: > + phys: false > + - if: > + required: > + - phys > + then: > + properties: > + xceiver-supply: false The duplication here is not needed, they both will cause errors in the same situation. With one dropped, Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
Attachment:
signature.asc
Description: PGP signature