Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230615213154.1753313-1-robh@kernel.org>
In-Reply-To: <20230615213154.1753313-1-robh@kernel.org>
From:   Mathieu Poirier <mathieu.poirier@linaro.org>
Date:   Fri, 16 Jun 2023 09:20:45 -0600
Message-ID: <CANLsYkx_cd7QxMpE8OKe+jJT-tSt23Ub0p8ceAuKbchLtQmKcg@mail.gmail.com>
Subject: Re: [PATCH] dt-bindings: Remove last usage of "binding" or "schema"
 in titles
To:     Rob Herring <robh@kernel.org>
Cc:     Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Abel Vesa <abelvesa@kernel.org>, Peng Fan <peng.fan@nxp.com>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        NXP Linux Team <linux-imx@nxp.com>,
        Sebastian Reichel <sre@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        =?UTF-8?Q?Fern=C3=A1ndez_Rojas?= <noltari@gmail.com>,
        Marek Vasut <marex@denx.de>, Suman Anna <s-anna@ti.com>,
        - <devicetree-spec@vger.kernel.org>, linux-clk@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org,
        linux-remoteproc@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Thu, 15 Jun 2023 at 15:32, Rob Herring <robh@kernel.org> 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 <robh@kernel.org>
> ---
>  .../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-c=
locks.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:
>    - =C3=81lvaro Fern=C3=A1ndez Rojas <noltari@gmail.com>
> 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 <marex@denx.de>
> diff --git a/Documentation/devicetree/bindings/power/reset/restart-handle=
r.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 <sre@kernel.org>
> 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 <mathieu.poirier@linaro.org>

>  maintainers:
>    - Suman Anna <s-anna@ti.com>
> diff --git a/Documentation/devicetree/bindings/reserved-memory/framebuffe=
r.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:
>    - devicetree-spec@vger.kernel.org
> --
> 2.39.2
>