On Fri, Oct 09, 2020 at 03:29:12PM +0200, Oleksij Rempel wrote: > In order to automate the verification of DT nodes convert > fsl-flexcan.txt to fsl,flexcan.yaml > > Signed-off-by: Oleksij Rempel <o.rempel@xxxxxxxxxxxxxx> > --- > .../bindings/net/can/fsl,flexcan.yaml | 137 ++++++++++++++++++ > .../bindings/net/can/fsl-flexcan.txt | 57 -------- > 2 files changed, 137 insertions(+), 57 deletions(-) > create mode 100644 Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > delete mode 100644 Documentation/devicetree/bindings/net/can/fsl-flexcan.txt > > diff --git a/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > new file mode 100644 > index 000000000000..6486047f48b8 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/can/fsl,flexcan.yaml > @@ -0,0 +1,137 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/can/fsl,flexcan.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: > + Flexcan CAN controller on Freescale's ARM and PowerPC system-on-a-chip (SOC). > + > +maintainers: > + - Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> > + > +properties: > + compatible: > + oneOf: > + - enum: > + - fsl,imx8qm-flexcan > + - fsl,imx8mp-flexcan > + - fsl,imx6q-flexcan > + - fsl,imx53-flexcan > + - fsl,imx35-flexcan > + - fsl,imx28-flexcan > + - fsl,imx25-flexcan > + - fsl,p1010-flexcan > + - fsl,vf610-flexcan > + - fsl,ls1021ar2-flexcan > + - fsl,lx2160ar1-flexcan > + - items: > + - enum: > + - fsl,imx7d-flexcan > + - fsl,imx6ul-flexcan > + - fsl,imx6sx-flexcan > + - const: fsl,imx6q-flexcan > + - items: > + - enum: > + - fsl,ls1028ar1-flexcan > + - const: fsl,lx2160ar1-flexcan > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + maxItems: 2 > + > + clock-names: > + items: > + - const: ipg > + - const: per > + > + clock-frequency: > + description: | > + The oscillator frequency driving the flexcan device, filled in by the > + boot loader. This property should only be used the used operating system > + doesn't support the clocks and clock-names property. > + $ref: /schemas/types.yaml#/definitions/uint32 > + > + xceiver-supply: > + description: Regulator that powers the CAN transceiver. > + maxItems: 1 > + > + big-endian: > + $ref: /schemas/types.yaml#/definitions/flag > + description: | > + This means the registers of FlexCAN controller are big endian. This is > + optional property.i.e. if this property is not present in device tree > + node then controller is assumed to be little endian. If this property is > + present then controller is assumed to be big endian. > + > + fsl,stop-mode: > + description: | > + Register bits of stop mode control. > + > + The format should be as follows: > + <gpr req_gpr req_bit> > + gpr is the phandle to general purpose register node. > + req_gpr is the gpr register offset of CAN stop request. > + req_bit is the bit offset of CAN stop request. > + $ref: /schemas/types.yaml#/definitions/uint32-array Should be a phandle-array. > + items: > + items: > + - description: The 'gpr' is the phandle to general purpose register node. > + - description: The 'req_gpr' is the gpr register offset of CAN stop request. > + maximum: 0xff > + - description: The 'req_bit' is the bit offset of CAN stop request. > + maximum: 0x1f > + minItems: 1 > + maxItems: 1 Drop the outer 'items' and these 2. > + > + fsl,clk-source: > + description: | > + Select the clock source to the CAN Protocol Engine (PE). It's SoC > + implementation dependent. Refer to RM for detailed definition. If this > + property is not set in device tree node then driver selects clock source 1 > + by default. > + 0: clock source 0 (oscillator clock) > + 1: clock source 1 (peripheral clock) > + $ref: /schemas/types.yaml#/definitions/uint32 > + default: 1 > + minimum: 0 > + maximum: 1 > + > + wakeup-source: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + Enable CAN remote wakeup. > + > +required: > + - compatible > + - reg > + - interrupts > + > +additionalProperties: false > + > +examples: > + - | > + can@1c000 { > + compatible = "fsl,p1010-flexcan"; > + reg = <0x1c000 0x1000>; > + interrupts = <48 0x2>; > + interrupt-parent = <&mpic>; > + clock-frequency = <200000000>; > + fsl,clk-source = <0>; > + }; > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + > + flexcan@2090000 { can@... > + compatible = "fsl,imx6q-flexcan"; > + reg = <0x02090000 0x4000>; > + interrupts = <0 110 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&clks 1>, <&clks 2>; > + clock-names = "ipg", "per"; > + fsl,stop-mode = <&gpr 0x34 28>; > + }; > diff --git a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt b/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt > deleted file mode 100644 > index e10b6eb955e1..000000000000 > --- a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt > +++ /dev/null > @@ -1,57 +0,0 @@ > -Flexcan CAN controller on Freescale's ARM and PowerPC system-on-a-chip (SOC). > - > -Required properties: > - > -- compatible : Should be "fsl,<processor>-flexcan" > - > - where <processor> is imx8qm, imx6q, imx28, imx53, imx35, imx25, p1010, > - vf610, ls1021ar2, lx2160ar1, ls1028ar1. > - > - The ls1028ar1 must be followed by lx2160ar1, e.g. > - - "fsl,ls1028ar1-flexcan", "fsl,lx2160ar1-flexcan" > - > - An implementation should also claim any of the following compatibles > - that it is fully backwards compatible with: > - > - - fsl,p1010-flexcan > - > -- reg : Offset and length of the register set for this device > -- interrupts : Interrupt tuple for this device > - > -Optional properties: > - > -- clock-frequency : The oscillator frequency driving the flexcan device > - > -- xceiver-supply: Regulator that powers the CAN transceiver > - > -- big-endian: This means the registers of FlexCAN controller are big endian. > - This is optional property.i.e. if this property is not present in > - device tree node then controller is assumed to be little endian. > - if this property is present then controller is assumed to be big > - endian. > - > -- fsl,stop-mode: register bits of stop mode control, the format is > - <&gpr req_gpr req_bit>. > - gpr is the phandle to general purpose register node. > - req_gpr is the gpr register offset of CAN stop request. > - req_bit is the bit offset of CAN stop request. > - > -- fsl,clk-source: Select the clock source to the CAN Protocol Engine (PE). > - It's SoC Implementation dependent. Refer to RM for detailed > - definition. If this property is not set in device tree node > - then driver selects clock source 1 by default. > - 0: clock source 0 (oscillator clock) > - 1: clock source 1 (peripheral clock) > - > -- wakeup-source: enable CAN remote wakeup > - > -Example: > - > - can@1c000 { > - compatible = "fsl,p1010-flexcan"; > - reg = <0x1c000 0x1000>; > - interrupts = <48 0x2>; > - interrupt-parent = <&mpic>; > - clock-frequency = <200000000>; // filled in by bootloader > - fsl,clk-source = <0>; // select clock source 0 for PE > - }; > -- > 2.28.0 >
