mbox series

[v2,00/10] dt-bindings: arm: freescale: Switch fsl,scu from txt to yaml

Message ID 20220608164928.2706697-1-abel.vesa@nxp.com
Headers show
Series dt-bindings: arm: freescale: Switch fsl,scu from txt to yaml | expand

Message

Abel Vesa June 8, 2022, 4:49 p.m. UTC
This patchset is splitting the fsl,scu.txt file from
bindings/arm/freescale into different yaml files throughout multiple
bindings directories. Last patch actually removes the file entirely.

Here is the v1:
https://lore.kernel.org/lkml/20220607105951.1821519-1-abel.vesa@nxp.com/

Changes since v1:
 * added the yaml files to their proper subsystems
 * cleaned up according with Krzysztof's and Rob's comments
 * changed commit messages to reflect that the fsl,scu.txt needs to go away

Abel Vesa (10):
  dt-bindings: firmware: Add fsl,scu yaml file
  dt-bindings: clk: imx: Add fsl,scu-clk yaml file
  dt-bindings: pinctrl: imx: Add fsl,scu-iomux yaml file
  dt-bindings: input: Add fsl,scu-key yaml file
  dt-bindings: nvmem: Add fsl,scu-ocotp yaml file
  dt-bindings: power: Add fsl,scu-pd yaml file
  dt-bindings: rtc: Add fsl,scu-rtc yaml file
  dt-bindings: thermal: Add fsl,scu-thermal yaml file
  dt-bindings: watchdog: Add fsl,scu-wdt yaml file
  dt-bindings: arm: freescale: Remove fsl,scu txt file

 .../bindings/arm/freescale/fsl,scu.txt        | 271 ------------------
 .../bindings/clock/fsl,scu-clk.yaml           |  52 ++++
 .../devicetree/bindings/firmware/fsl,scu.yaml | 162 +++++++++++
 .../bindings/input/fsl,scu-key.yaml           |  30 ++
 .../bindings/nvmem/fsl,scu-ocotp.yaml         |  37 +++
 .../bindings/pinctrl/fsl,scu-pinctrl.yaml     |  32 +++
 .../devicetree/bindings/power/fsl,scu-pd.yaml |  34 +++
 .../devicetree/bindings/rtc/fsl,scu-rtc.yaml  |  22 ++
 .../bindings/thermal/fsl,scu-thermal.yaml     |  32 +++
 .../bindings/watchdog/fsl,scu-wdt.yaml        |  29 ++
 10 files changed, 430 insertions(+), 271 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
 create mode 100644 Documentation/devicetree/bindings/clock/fsl,scu-clk.yaml
 create mode 100644 Documentation/devicetree/bindings/firmware/fsl,scu.yaml
 create mode 100644 Documentation/devicetree/bindings/input/fsl,scu-key.yaml
 create mode 100644 Documentation/devicetree/bindings/nvmem/fsl,scu-ocotp.yaml
 create mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,scu-pinctrl.yaml
 create mode 100644 Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
 create mode 100644 Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
 create mode 100644 Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
 create mode 100644 Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml

--
2.34.3

Comments

Alexandre Belloni June 8, 2022, 5:04 p.m. UTC | #1
On 08/06/2022 19:49:25+0300, Abel Vesa wrote:
> In order to replace the fsl,scu txt file from bindings/arm/freescale,
> we need to split it between the right subsystems. This patch documents
> separately the 'rtc' child node of the SCU main node.
> 
> Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
> ---
>  .../devicetree/bindings/rtc/fsl,scu-rtc.yaml  | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> 
> diff --git a/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml b/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> new file mode 100644
> index 000000000000..bd8c3cf365ff
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> @@ -0,0 +1,22 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/rtc/fsl,scu-rtc.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: i.MX SCU Client Device Node - RTC bindings based on SCU Message Protocol
> +
> +maintainers:
> +  - Alexandre Belloni <alexandre.belloni@bootlin.com>

Isn't NXP going to maintain their binding?

> +
> +description: i.MX SCU Client Device Node
> +  Client nodes are maintained as children of the relevant IMX-SCU device node.
> +
> +properties:
> +  compatible:
> +    const: fsl,imx8qxp-sc-rtc
> +
> +required:
> +  - compatible
> +
> +additionalProperties: false
> -- 
> 2.34.3
>
Abel Vesa June 9, 2022, 1:55 p.m. UTC | #2
On 22-06-08 19:04:55, Alexandre Belloni wrote:
> On 08/06/2022 19:49:25+0300, Abel Vesa wrote:
> > In order to replace the fsl,scu txt file from bindings/arm/freescale,
> > we need to split it between the right subsystems. This patch documents
> > separately the 'rtc' child node of the SCU main node.
> >
> > Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
> > ---
> >  .../devicetree/bindings/rtc/fsl,scu-rtc.yaml  | 22 +++++++++++++++++++
> >  1 file changed, 22 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml b/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> > new file mode 100644
> > index 000000000000..bd8c3cf365ff
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
> > @@ -0,0 +1,22 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/rtc/fsl,scu-rtc.yaml
> > +$schema: http://devicetree.org/meta-schemas/core.yaml
> > +
> > +title: i.MX SCU Client Device Node - RTC bindings based on SCU Message Protocol
> > +
> > +maintainers:
> > +  - Alexandre Belloni <alexandre.belloni@bootlin.com>
>
> Isn't NXP going to maintain their binding?
>

Since there is no NXP maintainer for the i.MX rtc drivers,
I took whoever was in the MAINTAINERS file.

I had a chat with Aisheng Dong and he is OK being mentioned
as maintainer for all the binding docs related to SCU, not just this
one.

I'll send a new version.

> > +
> > +description: i.MX SCU Client Device Node
> > +  Client nodes are maintained as children of the relevant IMX-SCU device node.
> > +
> > +properties:
> > +  compatible:
> > +    const: fsl,imx8qxp-sc-rtc
> > +
> > +required:
> > +  - compatible
> > +
> > +additionalProperties: false
> > --
> > 2.34.3
> >
>
> --
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com/
Abel Vesa June 9, 2022, 2:43 p.m. UTC | #3
On 22-06-08 19:49:18, Abel Vesa wrote:
> This patchset is splitting the fsl,scu.txt file from
> bindings/arm/freescale into different yaml files throughout multiple
> bindings directories. Last patch actually removes the file entirely.

Everyone, please ignore this version. Here is the new (3rd) version:
https://lore.kernel.org/lkml/20220609143423.2839186-1-abel.vesa@nxp.com/

>
> Here is the v1:
> https://lore.kernel.org/lkml/20220607105951.1821519-1-abel.vesa@nxp.com/
>
> Changes since v1:
>  * added the yaml files to their proper subsystems
>  * cleaned up according with Krzysztof's and Rob's comments
>  * changed commit messages to reflect that the fsl,scu.txt needs to go away
>
> Abel Vesa (10):
>   dt-bindings: firmware: Add fsl,scu yaml file
>   dt-bindings: clk: imx: Add fsl,scu-clk yaml file
>   dt-bindings: pinctrl: imx: Add fsl,scu-iomux yaml file
>   dt-bindings: input: Add fsl,scu-key yaml file
>   dt-bindings: nvmem: Add fsl,scu-ocotp yaml file
>   dt-bindings: power: Add fsl,scu-pd yaml file
>   dt-bindings: rtc: Add fsl,scu-rtc yaml file
>   dt-bindings: thermal: Add fsl,scu-thermal yaml file
>   dt-bindings: watchdog: Add fsl,scu-wdt yaml file
>   dt-bindings: arm: freescale: Remove fsl,scu txt file
>
>  .../bindings/arm/freescale/fsl,scu.txt        | 271 ------------------
>  .../bindings/clock/fsl,scu-clk.yaml           |  52 ++++
>  .../devicetree/bindings/firmware/fsl,scu.yaml | 162 +++++++++++
>  .../bindings/input/fsl,scu-key.yaml           |  30 ++
>  .../bindings/nvmem/fsl,scu-ocotp.yaml         |  37 +++
>  .../bindings/pinctrl/fsl,scu-pinctrl.yaml     |  32 +++
>  .../devicetree/bindings/power/fsl,scu-pd.yaml |  34 +++
>  .../devicetree/bindings/rtc/fsl,scu-rtc.yaml  |  22 ++
>  .../bindings/thermal/fsl,scu-thermal.yaml     |  32 +++
>  .../bindings/watchdog/fsl,scu-wdt.yaml        |  29 ++
>  10 files changed, 430 insertions(+), 271 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
>  create mode 100644 Documentation/devicetree/bindings/clock/fsl,scu-clk.yaml
>  create mode 100644 Documentation/devicetree/bindings/firmware/fsl,scu.yaml
>  create mode 100644 Documentation/devicetree/bindings/input/fsl,scu-key.yaml
>  create mode 100644 Documentation/devicetree/bindings/nvmem/fsl,scu-ocotp.yaml
>  create mode 100644 Documentation/devicetree/bindings/pinctrl/fsl,scu-pinctrl.yaml
>  create mode 100644 Documentation/devicetree/bindings/power/fsl,scu-pd.yaml
>  create mode 100644 Documentation/devicetree/bindings/rtc/fsl,scu-rtc.yaml
>  create mode 100644 Documentation/devicetree/bindings/thermal/fsl,scu-thermal.yaml
>  create mode 100644 Documentation/devicetree/bindings/watchdog/fsl,scu-wdt.yaml
>
> --
> 2.34.3
>