Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 31 Dec 2020 23:04:48 +0100
From:   Jonathan =?utf-8?Q?Neusch=C3=A4fer?= <j.neuschaefer@gmx.net>
To:     Andy Whitcroft <apw@canonical.com>, Joe Perches <joe@perches.com>
Cc:     Rob Herring <robh@kernel.org>, linux-kernel@vger.kernel.org
Subject: checkpatch.pl: Bogus case of DT_SPLIT_BINDING_PATCH
Message-ID: <X+5LAFw7AHvAXJaK@latitude>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="2nvqc0kNWltf5Qkb"
Content-Disposition: inline
Precedence: bulk


--2nvqc0kNWltf5Qkb
Content-Type: multipart/mixed; boundary="ESYGnOEREAsA8eQo"
Content-Disposition: inline


--ESYGnOEREAsA8eQo
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

I've encountered a case where the DT_SPLIT_BINDING_PATCH warning was
emitted even though I didn't change anything outside of Documentation/
devicetree/bindings. I just converted a binding from plain text to YAML.

Here's a transcript of checkpatch (from Linux 5.11-rc1)'s output:

  $ scripts/checkpatch.pl --strict patches-wpcm/0001-dt-bindings-arm-Conver=
t-nuvoton-npcm750-binding-to-Y.patch
  WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
  #31:
  deleted file mode 100644
 =20
  WARNING: DT binding docs and includes should be a separate patch. See: Do=
cumentation/devicetree/bindings/submitting-patches.rst
 =20
  WARNING: DT binding docs and includes should be a separate patch. See: Do=
cumentation/devicetree/bindings/submitting-patches.rst
 =20
  total: 0 errors, 3 warnings, 0 checks, 21 lines checked
 =20
  NOTE: For some of the reported defects, checkpatch may be able to
        mechanically convert to the typical style using --fix or --fix-inpl=
ace.
 =20
  patches-wpcm/0001-dt-bindings-arm-Convert-nuvoton-npcm750-binding-to-Y.pa=
tch has style problems, please review.
 =20
  NOTE: If any of the errors are false positives, please report
        them to the maintainer, see CHECKPATCH in MAINTAINERS.


I attached the patch, for reference.


Best regards,
Jonathan Neusch=C3=A4fer

--ESYGnOEREAsA8eQo
Content-Type: text/plain; charset=utf-8
Content-Disposition: attachment; filename=patch
Content-Transfer-Encoding: quoted-printable

=46rom 3aaf23345633070909e23a05a9f92748e445b3b3 Mon Sep 17 00:00:00 2001
=46rom: =3D?UTF-8?q?Jonathan=3D20Neusch=3DC3=3DA4fer?=3D <j.neuschaefer@gmx=
=2Enet>
Date: Mon, 28 Dec 2020 01:09:31 +0100
Subject: [PATCH 01/11] dt-bindings: arm: Convert nuvoton,npcm750 binding to
 YAML
MIME-Version: 1.0
Content-Type: text/plain; charset=3DUTF-8
Content-Transfer-Encoding: 8bit

The general trend is to have devicetree bindings in YAML format, to
allow automatic validation of bindings and devicetrees.

Convert the NPCM SoC family's binding to YAML before extending it.

The nuvoton,npcm750-evb compatible string is introduced to keep the
structure of the binding a little simpler.

Signed-off-by: Jonathan Neusch=C3=A4fer <j.neuschaefer@gmx.net>
---

If someone else (e.g. Brendan Higgins) wants to be listed as the
maintainer, please let me know.
---
 .../devicetree/bindings/arm/npcm/npcm.txt     |  6 ------
 .../devicetree/bindings/arm/npcm/npcm.yaml    | 21 +++++++++++++++++++
 2 files changed, 21 insertions(+), 6 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/arm/npcm/npcm.txt
 create mode 100644 Documentation/devicetree/bindings/arm/npcm/npcm.yaml

diff --git a/Documentation/devicetree/bindings/arm/npcm/npcm.txt b/Document=
ation/devicetree/bindings/arm/npcm/npcm.txt
deleted file mode 100644
index 2d87d9ecea85b..0000000000000
--- a/Documentation/devicetree/bindings/arm/npcm/npcm.txt
+++ /dev/null
@@ -1,6 +0,0 @@
-NPCM Platforms Device Tree Bindings
------------------------------------
-NPCM750 SoC
-Required root node properties:
-	- compatible =3D "nuvoton,npcm750";
-
diff --git a/Documentation/devicetree/bindings/arm/npcm/npcm.yaml b/Documen=
tation/devicetree/bindings/arm/npcm/npcm.yaml
new file mode 100644
index 0000000000000..3ecd00803c003
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/npcm/npcm.yaml
@@ -0,0 +1,21 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/arm/npcm.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NPCM Platforms Device Tree Bindings
+
+maintainers:
+  - Jonathan Neusch=C3=A4fer <j.neuschaefer@gmx.net>
+
+properties:
+  $nodename:
+    const: '/'
+  compatible:
+    oneOf:
+      - description: NPCM750 based boards
+        items:
+          - enum:
+            - nuvoton,npcm750-evb           # NPCM750 evaluation board
+          - const: nuvoton,npcm750
--=20
2.29.2


--ESYGnOEREAsA8eQo--

--2nvqc0kNWltf5Qkb
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEvHAHGBBjQPVy+qvDCDBEmo7zX9sFAl/uSvkACgkQCDBEmo7z
X9vgsg//Wcpmagto1fiQX6J6uVVXuffSZXSEfHlKd/QSTSfJpx1mDOlEQrvUht7w
Xvk9jmHhm3nM2S6B1afrVR+ygW4g0TBwXcc3JmPeKK2/80Z8BWzW8wUPdusxOpoX
bgOQVdt7X3BGZCR/5gbIydTJ7uBMl/sdBsmIykPP6m8nCorD5aIc1UAa/49KnnHU
K5NnX1m9f2Ye6tJSn47mBajNP+lCeKyn6rXzJFvbD0b8EMOm40/I7CQdNWnzJRRq
GfkbCJD0C8uoqTY0bvGQQSwv69DJc5DcU7dgz2uByRvIg7XFfucUvknwOlmKM1DL
Bwn1e2+QqjTZhy5T68s3Xs6bDbo2uJ9di6UahF+0Mvrj230pXtJsLG6MH0esQHfS
j9av7rAi+ZDjnUjX9gQu1t7uoEBXp9VI59pwf2HukFOftCp/V0iXsvodl3arARbn
+oZN4tmduz+9WgoJWoau4vUwF5l4Pz4C7y3uPj5w8x2lEi9Ds2aVgNiF6xoVweqQ
8kgPPBmaURBEVAcCtjZ+lCbg2pXkEqblXUgIbzYMA1+4hI1cOXC58eqUG11WiliW
QEn7O43F6fm/Z+g6z4abV19Zj6S4rsJ+c03pHvzUF/tCneaO4ymYcDxjLUfgI46M
IFv79L16aH2OJ33gFW9Dt4ehVL6SXiznTrZ/O3SFxiGE0yfu9nQ=
=snBh
-----END PGP SIGNATURE-----

--2nvqc0kNWltf5Qkb--