On Mon, Jun 20, 2016 at 03:46:47PM +0100, Ramesh Shanmugasundaram wrote: > The controller can operate in one of the two global modes > - CAN FD only mode (default) > - Classical CAN (CAN2.0) only mode > > This patch adds support for Classical CAN only mode. It can be enabled > by defining the optional device tree property "renesas,can-nofd" of this > node. > > Note: R-Car Gen3 h/w manual v0.51E shows bit6 of RSCFDnCFDGCFG as > reserved, which is incorrect. This bit is same as RSCFDnGCFG. > > Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@xxxxxxxxxxxxxx> > --- > Hi Marc, All, > > This patch is based on top of can-next(tag:linux-can-next-for-4.8-20160617). > > Thanks, > Ramesh. > --- > .../devicetree/bindings/net/can/rcar_canfd.txt | 9 +- > drivers/net/can/rcar/rcar_canfd.c | 355 ++++++++++++++------- > 2 files changed, 249 insertions(+), 115 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > index d45182b..a343359 100644 > --- a/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > +++ b/Documentation/devicetree/bindings/net/can/rcar_canfd.txt > @@ -32,6 +32,12 @@ below properties. > - assigned-clocks: phandle of canfd clock. > - assigned-clock-rates: maximum frequency of this clock. > > +Optional property: > +The controller can operate in either CAN FD only mode (default) or > +Classical CAN only mode. The mode is global to both the channels. In order to > +enable the later, define the following optional property. > + - renesas,can-nofd: puts the controller in Classical CAN only mode. renesas,no-can-fd would be a bit more readable. > + > Example > ------- > > @@ -63,11 +69,12 @@ SoC common .dtsi file: > > Board specific .dts file: > > -E.g. below enables Channel 1 alone in the board. > +E.g. below enables Channel 1 alone in the board in Classical CAN only mode. > > &canfd { > pinctrl-0 = <&canfd1_pins>; > pinctrl-names = "default"; > + renesas,can-nofd; Now we have a mixture of tabs and spaces. > status = "okay"; > > channel1 { -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html