[PATCH] dt-bindings: ata: Convert ti,dm816-ahci to DT schema

[PATCH] dt-bindings: ata: Convert ti,dm816-ahci to DT schema

[Rob Herring (Arm)]

Convert the TI DM816 AHCI SATA Controller to DT schema format. It's a
straight-forward conversion.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../devicetree/bindings/ata/ahci-dm816.txt    | 21 ---------
 .../bindings/ata/ti,dm816-ahci.yaml           | 43 +++++++++++++++++++
 2 files changed, 43 insertions(+), 21 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/ata/ahci-dm816.txt
 create mode 100644 Documentation/devicetree/bindings/ata/ti,dm816-ahci.yaml

diff --git a/Documentation/devicetree/bindings/ata/ahci-dm816.txt b/Documentation/devicetree/bindings/ata/ahci-dm816.txt
deleted file mode 100644
index f8c535f3541f..000000000000
--- a/Documentation/devicetree/bindings/ata/ahci-dm816.txt
+++ /dev/null
@@ -1,21 +0,0 @@
-Device tree binding for the TI DM816 AHCI SATA Controller
----------------------------------------------------------
-
-Required properties:
-  - compatible: must be "ti,dm816-ahci"
-  - reg: physical base address and size of the register region used by
-         the controller (as defined by the AHCI 1.1 standard)
-  - interrupts: interrupt specifier (refer to the interrupt binding)
-  - clocks: list of phandle and clock specifier pairs (or only
-            phandles for clock providers with '0' defined for
-            #clock-cells); two clocks must be specified: the functional
-            clock and an external reference clock
-
-Example:
-
-	sata: sata@4a140000 {
-		compatible = "ti,dm816-ahci";
-		reg = <0x4a140000 0x10000>;
-		interrupts = <16>;
-		clocks = <&sysclk5_ck>, <&sata_refclk>;
-	};
diff --git a/Documentation/devicetree/bindings/ata/ti,dm816-ahci.yaml b/Documentation/devicetree/bindings/ata/ti,dm816-ahci.yaml
new file mode 100644
index 000000000000..d0ff9e78afe6
--- /dev/null
+++ b/Documentation/devicetree/bindings/ata/ti,dm816-ahci.yaml
@@ -0,0 +1,43 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/ata/ti,dm816-ahci.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: TI DM816 AHCI SATA Controller
+
+maintainers:
+  - Bartosz Golaszewski <brgl@bgdev.pl>
+
+allOf:
+  - $ref: ahci-common.yaml#
+
+properties:
+  compatible:
+    const: ti,dm816-ahci
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    items:
+      - description: functional clock
+      - description: external reference clock
+
+  ti,hwmods:
+    const: sata
+
+required:
+  - compatible
+  - clocks
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    sata@4a140000 {
+        compatible = "ti,dm816-ahci";
+        reg = <0x4a140000 0x10000>;
+        interrupts = <16>;
+        clocks = <&sysclk5_ck>, <&sata_refclk>;
+    };
-- 
2.47.2

Re: [PATCH] dt-bindings: ata: Convert ti,dm816-ahci to DT schema

[Bartosz Golaszewski]

On Mon, May 12, 2025 at 11:57 PM Rob Herring (Arm) <robh@kernel.org> wrote:
>
> Convert the TI DM816 AHCI SATA Controller to DT schema format. It's a
> straight-forward conversion.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---

Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

Re: [PATCH] dt-bindings: ata: Convert ti,dm816-ahci to DT schema

[Damien Le Moal]

On 5/13/25 06:57, Rob Herring (Arm) wrote:
> Convert the TI DM816 AHCI SATA Controller to DT schema format. It's a
> straight-forward conversion.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Applied to for-6.16. Thanks !

Note: I will wait a little more for the other patches to see if reviews come in.

-- 
Damien Le Moal
Western Digital Research

Re: [PATCH] dt-bindings: ata: Convert ti,dm816-ahci to DT schema

[Rob Herring]

On Wed, May 14, 2025 at 04:24:00PM +0900, Damien Le Moal wrote:
> On 5/13/25 06:57, Rob Herring (Arm) wrote:
> > Convert the TI DM816 AHCI SATA Controller to DT schema format. It's a
> > straight-forward conversion.
> > 
> > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> 
> Applied to for-6.16. Thanks !
> 
> Note: I will wait a little more for the other patches to see if reviews come in.

Some of this stuff is ancient, so you might not. xgene is pretty much 
abandoned, but there's still a handful of machines around with people 
using occasionally.

Rob