diff mbox series

[2/6] dt-bindings: can: flexcan: fix fsl,clk-source property

Message ID 20201016134320.20321-3-qiangqing.zhang@nxp.com
State Changes Requested
Headers show
Series can: flexcan: add stop mode support for i.MX8QM | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 2 warnings, 6 lines checked

Commit Message

Joakim Zhang Oct. 16, 2020, 1:43 p.m. UTC
Correct fsl,clk-source example since flexcan driver uses "of_property_read_u8"
to get this property.

Fixes: 9d733992772d ("dt-bindings: can: flexcan: add PE clock source property to device tree")
Signed-off-by: Joakim Zhang <qiangqing.zhang@nxp.com>
---
 Documentation/devicetree/bindings/net/can/fsl-flexcan.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marc Kleine-Budde Oct. 16, 2020, 6:21 a.m. UTC | #1
On 10/16/20 3:43 PM, Joakim Zhang wrote:
> Correct fsl,clk-source example since flexcan driver uses "of_property_read_u8"
> to get this property.

Hopefully today Oleksij will post the next round of the yaml bindings conversion
patch. Please resping when Oleksij's patch is applied.

Marc
Joakim Zhang Oct. 16, 2020, 6:52 a.m. UTC | #2
Hi Marc,

> -----Original Message-----
> From: Marc Kleine-Budde <mkl@pengutronix.de>
> Sent: 2020年10月16日 14:22
> To: Joakim Zhang <qiangqing.zhang@nxp.com>; robh+dt@kernel.org;
> shawnguo@kernel.org; s.hauer@pengutronix.de
> Cc: devicetree@vger.kernel.org; Peng Fan <peng.fan@nxp.com>; Ying Liu
> <victor.liu@nxp.com>; netdev@vger.kernel.org; Pankaj Bansal
> <pankaj.bansal@nxp.com>; linux-kernel@vger.kernel.org;
> linux-can@vger.kernel.org; dl-linux-imx <linux-imx@nxp.com>;
> kernel@pengutronix.de; Oleksij Rempel <ore@pengutronix.de>
> Subject: Re: [PATCH 2/6] dt-bindings: can: flexcan: fix fsl,clk-source property
> 
> On 10/16/20 3:43 PM, Joakim Zhang wrote:
> > Correct fsl,clk-source example since flexcan driver uses
> "of_property_read_u8"
> > to get this property.
> 
> Hopefully today Oleksij will post the next round of the yaml bindings conversion
> patch. Please resping when Oleksij's patch is applied.

Of course.

Best Regards,
Joakim Zhang
> Marc
> 
> --
> Pengutronix e.K.                 | Marc Kleine-Budde           |
> Embedded Linux                   | https://www.pengutronix.de  |
> Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
> Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt b/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt
index e10b6eb955e1..6af67f5e581c 100644
--- a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt
+++ b/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt
@@ -53,5 +53,5 @@  Example:
 		interrupts = <48 0x2>;
 		interrupt-parent = <&mpic>;
 		clock-frequency = <200000000>; // filled in by bootloader
-		fsl,clk-source = <0>; // select clock source 0 for PE
+		fsl,clk-source = /bits/ 8 <0>; // select clock source 0 for PE
 	};