[PATCH v2] dt-bindings: gpio: cavium,thunder-8890: Convert to DT schema

Shi Hao posted 1 patch 2 days, 22 hours ago
.../gpio/cavium,thunder-8890-gpio.yaml        | 45 +++++++++++++++++++
.../bindings/gpio/gpio-thunderx.txt           | 27 -----------
2 files changed, 45 insertions(+), 27 deletions(-)
create mode 100644 Documentation/devicetree/bindings/gpio/cavium,thunder-8890-gpio.yaml
delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-thunderx.txt
[PATCH v2] dt-bindings: gpio: cavium,thunder-8890: Convert to DT schema
Posted by Shi Hao 2 days, 22 hours ago
Convert Cavium ThunderX GPIO bindings to DT schema.

Signed-off-by: Shi Hao <i.shihao.999@gmail.com>
---

v2:
 - Rename schema file based on compatible string
 - Wrap commit message body as per kernel patch guidelines
 - Use appropriate maintainer name and email address in DT schema.
 - Change commit subject as per guidelines
 - Fix $id path
 - Rebased on linux-next

 Note:
* This patch is part of the GSoC2026 application process for device tree bindings conversions
* https://github.com/LinuxFoundationGSoC/ProjectIdeas/wiki/GSoC-2026-Device-Tree-Bindings
---
 .../gpio/cavium,thunder-8890-gpio.yaml        | 45 +++++++++++++++++++
 .../bindings/gpio/gpio-thunderx.txt           | 27 -----------
 2 files changed, 45 insertions(+), 27 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/gpio/cavium,thunder-8890-gpio.yaml
 delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-thunderx.txt

diff --git a/Documentation/devicetree/bindings/gpio/cavium,thunder-8890-gpio.yaml b/Documentation/devicetree/bindings/gpio/cavium,thunder-8890-gpio.yaml
new file mode 100644
index 000000000000..0d42ddaeb23a
--- /dev/null
+++ b/Documentation/devicetree/bindings/gpio/cavium,thunder-8890-gpio.yaml
@@ -0,0 +1,45 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/gpio/cavium,thunder-8890-gpio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Cavium ThunderX/OCTEON-TX GPIO Controller
+
+maintainers:
+  - Robert Richter <rric@kernel.org>
+
+properties:
+  compatible:
+    const: cavium,thunder-8890-gpio
+
+  reg:
+    maxItems: 1
+
+  gpio-controller: true
+
+  '#gpio-cells':
+    const: 2
+
+  interrupt-controller: true
+
+  '#interrupt-cells':
+    const: 2
+
+required:
+  - reg
+  - gpio-controller
+  - '#gpio-cells'
+
+additionalProperties: false
+
+examples:
+  - |
+    gpio_6_0: gpio@6,0 {
+      compatible = "cavium,thunder-8890-gpio";
+      reg = <0x3000 0x100>; /*  DEVFN = 0x30 (6:0) */
+      gpio-controller;
+      #gpio-cells = <2>;
+      interrupt-controller;
+      #interrupt-cells = <2>;
+    };
diff --git a/Documentation/devicetree/bindings/gpio/gpio-thunderx.txt b/Documentation/devicetree/bindings/gpio/gpio-thunderx.txt
deleted file mode 100644
index 3f883ae29d11..000000000000
--- a/Documentation/devicetree/bindings/gpio/gpio-thunderx.txt
+++ /dev/null
@@ -1,27 +0,0 @@
-Cavium ThunderX/OCTEON-TX GPIO controller bindings
-
-Required Properties:
-- reg: The controller bus address.
-- gpio-controller: Marks the device node as a GPIO controller.
-- #gpio-cells: Must be 2.
-  - First cell is the GPIO pin number relative to the controller.
-  - Second cell is a standard generic flag bitfield as described in gpio.txt.
-
-Optional Properties:
-- compatible: "cavium,thunder-8890-gpio", unused as PCI driver binding is used.
-- interrupt-controller: Marks the device node as an interrupt controller.
-- #interrupt-cells: Must be present and have value of 2 if
-                    "interrupt-controller" is present.
-  - First cell is the GPIO pin number relative to the controller.
-  - Second cell is triggering flags as defined in interrupts.txt.
-
-Example:
-
-gpio_6_0: gpio@6,0 {
-	compatible = "cavium,thunder-8890-gpio";
-	reg = <0x3000 0 0 0 0>; /*  DEVFN = 0x30 (6:0) */
-	gpio-controller;
-	#gpio-cells = <2>;
-	interrupt-controller;
-	#interrupt-cells = <2>;
-};
-- 
2.53.0
Re: [PATCH v2] dt-bindings: gpio: cavium,thunder-8890: Convert to DT schema
Posted by Krzysztof Kozlowski 1 day, 22 hours ago
On Mon, Mar 30, 2026 at 01:31:08PM +0530, Shi Hao wrote:
> Convert Cavium ThunderX GPIO bindings to DT schema.
> 
> Signed-off-by: Shi Hao <i.shihao.999@gmail.com>
> ---
> 
> v2:
>  - Rename schema file based on compatible string
>  - Wrap commit message body as per kernel patch guidelines
>  - Use appropriate maintainer name and email address in DT schema.
>  - Change commit subject as per guidelines
>  - Fix $id path
>  - Rebased on linux-next
> 
>  Note:
> * This patch is part of the GSoC2026 application process for device tree bindings conversions
> * https://github.com/LinuxFoundationGSoC/ProjectIdeas/wiki/GSoC-2026-Device-Tree-Bindings
> ---
>  .../gpio/cavium,thunder-8890-gpio.yaml        | 45 +++++++++++++++++++
>  .../bindings/gpio/gpio-thunderx.txt           | 27 -----------

This binding is odd and not used, so you should follow guidelines I gave
more than a year ago and updated recently and NOT convert it.

Or rather answer - why this cannot be removed instead? What is the
benefit of this binding? Is any other project using it?


Best regards,
Krzysztof