diff mbox series

[v2,1/1] dt-bindings: can: fsl,flexcan: add common 'can-transceiver' for fsl,flexcan

Message ID 20240629021754.3583641-1-Frank.Li@nxp.com (mailing list archive)
State Awaiting Upstream
Delegated to: Netdev Maintainers
Headers show
Series [v2,1/1] dt-bindings: can: fsl,flexcan: add common 'can-transceiver' for fsl,flexcan | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Frank Li June 29, 2024, 2:17 a.m. UTC
Add common 'can-transceiver' children node for fsl,flexcan.

Fix below warning:
arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dtb: can@2180000: 'can-transceiver' does not match any of the regexes: 'pinctrl-[0-9]+'
        from schema $id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml#

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
Change from v1 to v2
- rework commit message and add fix CHECK_DTBS warning
- Add unevaluatedProperties: false
---
 Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Rob Herring (Arm) July 1, 2024, 5:54 p.m. UTC | #1
On Fri, 28 Jun 2024 22:17:54 -0400, Frank Li wrote:
> Add common 'can-transceiver' children node for fsl,flexcan.
> 
> Fix below warning:
> arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dtb: can@2180000: 'can-transceiver' does not match any of the regexes: 'pinctrl-[0-9]+'
>         from schema $id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml#
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
> Change from v1 to v2
> - rework commit message and add fix CHECK_DTBS warning
> - Add unevaluatedProperties: false
> ---
>  Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml | 4 ++++
>  1 file changed, 4 insertions(+)
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Marc Kleine-Budde July 4, 2024, 10:07 a.m. UTC | #2
On 28.06.2024 22:17:54, Frank Li wrote:
> Add common 'can-transceiver' children node for fsl,flexcan.
> 
> Fix below warning:
> arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dtb: can@2180000: 'can-transceiver' does not match any of the regexes: 'pinctrl-[0-9]+'
>         from schema $id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml#
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>

Applied to linux-can-next.

Thanks,
Marc
Frank Li Aug. 1, 2024, 9:11 p.m. UTC | #3
On Thu, Jul 04, 2024 at 12:07:02PM +0200, Marc Kleine-Budde wrote:
> On 28.06.2024 22:17:54, Frank Li wrote:
> > Add common 'can-transceiver' children node for fsl,flexcan.
> >
> > Fix below warning:
> > arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dtb: can@2180000: 'can-transceiver' does not match any of the regexes: 'pinctrl-[0-9]+'
> >         from schema $id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml#
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
>
> Applied to linux-can-next.

I have not seen it at v6.11-rc1. Anything wrong?

Frank

>
> Thanks,
> Marc
>
> --
> Pengutronix e.K.                 | Marc Kleine-Budde          |
> Embedded Linux                   | https://www.pengutronix.de |
> Vertretung Nürnberg              | Phone: +49-5121-206917-129 |
> Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-9   |
Marc Kleine-Budde Aug. 5, 2024, 2:44 p.m. UTC | #4
On 01.08.2024 17:11:21, Frank Li wrote:
> On Thu, Jul 04, 2024 at 12:07:02PM +0200, Marc Kleine-Budde wrote:
> > On 28.06.2024 22:17:54, Frank Li wrote:
> > > Add common 'can-transceiver' children node for fsl,flexcan.
> > >
> > > Fix below warning:
> > > arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dtb: can@2180000: 'can-transceiver' does not match any of the regexes: 'pinctrl-[0-9]+'
> > >         from schema $id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml#
> > >
> > > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> >
> > Applied to linux-can-next.
> 
> I have not seen it at v6.11-rc1. Anything wrong?

No, it go mainline by v6.12.

regards,
Marc
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
index f197d9b516bb2..a4261a201fdb6 100644
--- a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
+++ b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml
@@ -80,6 +80,10 @@  properties:
       node then controller is assumed to be little endian. If this property is
       present then controller is assumed to be big endian.
 
+  can-transceiver:
+    $ref: can-transceiver.yaml#
+    unevaluatedProperties: false
+
   fsl,stop-mode:
     description: |
       Register bits of stop mode control.