2023-06-15 22:01:28

by Rob Herring (Arm)

[permalink] [raw]
Subject: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

The Devicetree bindings document does not have to say in the title that
it is a "Devicetree binding", but instead just describe the hardware.

Most of these have been fixed already, so fix the handful that snuck in.
With this, a meta-schema check can be enabled to catch these
automatically.

Signed-off-by: Rob Herring <[email protected]>
---
.../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
.../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
.../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
.../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
index 199818b2fb6d..cd0d763ce2f3 100644
--- a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
+++ b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/clock/brcm,bcm63268-timer-clocks.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Broadcom BCM63268 Timer Clock and Reset Device Tree Bindings
+title: Broadcom BCM63268 Timer Clock and Reset

maintainers:
- Álvaro Fernández Rojas <[email protected]>
diff --git a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
index ff9600474df2..0a6dc1a6e122 100644
--- a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
+++ b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/clock/imx8mp-audiomix.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: NXP i.MX8MP AudioMIX Block Control Binding
+title: NXP i.MX8MP AudioMIX Block Control

maintainers:
- Marek Vasut <[email protected]>
diff --git a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
index 1f9a2aac53c0..f2ffdd29d52a 100644
--- a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
+++ b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/power/reset/restart-handler.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Restart and shutdown handler generic binding
+title: Restart and shutdown handler Common Properties

maintainers:
- Sebastian Reichel <[email protected]>
diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
index c6d86964b72a..35f0bb38f7b2 100644
--- a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/remoteproc/ti,pru-consumer.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: Common TI PRU Consumer Binding
+title: TI PRU Consumer Common Properties

maintainers:
- Suman Anna <[email protected]>
diff --git a/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml b/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
index 05b6648b3458..851ec24d6142 100644
--- a/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
+++ b/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
@@ -4,7 +4,7 @@
$id: http://devicetree.org/schemas/reserved-memory/framebuffer.yaml#
$schema: http://devicetree.org/meta-schemas/core.yaml#

-title: /reserved-memory framebuffer node bindings
+title: /reserved-memory framebuffer node

maintainers:
- [email protected]
--
2.39.2



2023-06-15 22:55:28

by Marek Vasut

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

On 6/15/23 23:31, Rob Herring wrote:
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
> .../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
> .../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> index 199818b2fb6d..cd0d763ce2f3 100644
> --- a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> +++ b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/clock/brcm,bcm63268-timer-clocks.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Broadcom BCM63268 Timer Clock and Reset Device Tree Bindings
> +title: Broadcom BCM63268 Timer Clock and Reset
>
> maintainers:
> - Álvaro Fernández Rojas <[email protected]>
> diff --git a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> index ff9600474df2..0a6dc1a6e122 100644
> --- a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/clock/imx8mp-audiomix.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: NXP i.MX8MP AudioMIX Block Control Binding
> +title: NXP i.MX8MP AudioMIX Block Control

For AudioMIX:
Reviewed-by: Marek Vasut <[email protected]>

2023-06-15 23:22:25

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

On Thu, Jun 15, 2023 at 03:31:54PM -0600, Rob Herring wrote:

> -title: Restart and shutdown handler generic binding
> +title: Restart and shutdown handler Common Properties

With the new caps for Common Properties, the title here looks a bit odd.
Should probably be s/shutdown handler/Shutdown Handler/ for title case,
but w/e...
Reviewed-by: Conor Dooley <[email protected]>

Cheers,
Conor.

Attachments:
(No filename) (416.00 B)
 signature.asc (235.00 B)
 Download all attachments

2023-06-15 23:32:16

by Suman Anna

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

On 6/15/23 16:31, Rob Herring wrote:
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
> .../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
> .../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>

...

> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> index c6d86964b72a..35f0bb38f7b2 100644
> --- a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/remoteproc/ti,pru-consumer.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Common TI PRU Consumer Binding
> +title: TI PRU Consumer Common Properties
>
> maintainers:
> - Suman Anna <[email protected]>

For PRU Consumer binding,
Reviewed-by: Suman Anna <[email protected]>

regards
Suman


2023-06-16 01:01:16

by Sebastian Reichel

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

Hi,

On Thu, Jun 15, 2023 at 03:31:54PM -0600, Rob Herring wrote:
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
> .../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
> .../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)

...

> diff --git a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> index 1f9a2aac53c0..f2ffdd29d52a 100644
> --- a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> +++ b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/power/reset/restart-handler.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Restart and shutdown handler generic binding
> +title: Restart and shutdown handler Common Properties
>
> maintainers:
> - Sebastian Reichel <[email protected]>

Acked-by: Sebastian Reichel <[email protected]>

-- Sebastian

Attachments:
(No filename) (1.58 kB)
 signature.asc (849.00 B)
 Download all attachments

2023-06-16 15:40:37

by Mathieu Poirier

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

On Thu, 15 Jun 2023 at 15:32, Rob Herring <[email protected]> wrote:
>
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
> .../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
> .../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> index 199818b2fb6d..cd0d763ce2f3 100644
> --- a/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> +++ b/Documentation/devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/clock/brcm,bcm63268-timer-clocks.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Broadcom BCM63268 Timer Clock and Reset Device Tree Bindings
> +title: Broadcom BCM63268 Timer Clock and Reset
>
> maintainers:
> - Álvaro Fernández Rojas <[email protected]>
> diff --git a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> index ff9600474df2..0a6dc1a6e122 100644
> --- a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/clock/imx8mp-audiomix.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: NXP i.MX8MP AudioMIX Block Control Binding
> +title: NXP i.MX8MP AudioMIX Block Control
>
> maintainers:
> - Marek Vasut <[email protected]>
> diff --git a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> index 1f9a2aac53c0..f2ffdd29d52a 100644
> --- a/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> +++ b/Documentation/devicetree/bindings/power/reset/restart-handler.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/power/reset/restart-handler.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Restart and shutdown handler generic binding
> +title: Restart and shutdown handler Common Properties
>
> maintainers:
> - Sebastian Reichel <[email protected]>
> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> index c6d86964b72a..35f0bb38f7b2 100644
> --- a/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/ti,pru-consumer.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/remoteproc/ti,pru-consumer.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: Common TI PRU Consumer Binding
> +title: TI PRU Consumer Common Properties
>

Reviewed-by: Mathieu Poirier <[email protected]>

> maintainers:
> - Suman Anna <[email protected]>
> diff --git a/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml b/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
> index 05b6648b3458..851ec24d6142 100644
> --- a/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
> +++ b/Documentation/devicetree/bindings/reserved-memory/framebuffer.yaml
> @@ -4,7 +4,7 @@
> $id: http://devicetree.org/schemas/reserved-memory/framebuffer.yaml#
> $schema: http://devicetree.org/meta-schemas/core.yaml#
>
> -title: /reserved-memory framebuffer node bindings
> +title: /reserved-memory framebuffer node
>
> maintainers:
> - [email protected]
> --
> 2.39.2
>

2023-06-16 19:01:38

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles

Quoting Rob Herring (2023-06-15 14:31:54)
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---

Acked-by: Stephen Boyd <[email protected]>

2023-06-16 20:12:43

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema" in titles


On Thu, 15 Jun 2023 15:31:54 -0600, Rob Herring wrote:
> The Devicetree bindings document does not have to say in the title that
> it is a "Devicetree binding", but instead just describe the hardware.
>
> Most of these have been fixed already, so fix the handful that snuck in.
> With this, a meta-schema check can be enabled to catch these
> automatically.
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> .../devicetree/bindings/clock/brcm,bcm63268-timer-clocks.yaml | 2 +-
> Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml | 2 +-
> .../devicetree/bindings/power/reset/restart-handler.yaml | 2 +-
> .../devicetree/bindings/remoteproc/ti,pru-consumer.yaml | 2 +-
> .../devicetree/bindings/reserved-memory/framebuffer.yaml | 2 +-
> 5 files changed, 5 insertions(+), 5 deletions(-)
>

Applied, thanks!