dt-bindings: ata: ahci-platform: Drop unnecessary select schema

[PATCH] dt-bindings: ata: ahci-platform: Drop unnecessary select schema

Posted by Rob Herring (Arm) 1 month ago

The "select" schema is not necessary. It looks like it is there to prevent
matching on "generic-ahci" compatible, but that's not necessary because
this is the only place "generic-ahci" compatible is present.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 .../bindings/ata/ahci-platform.yaml           | 20 -------------------
 1 file changed, 20 deletions(-)

diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.yaml b/Documentation/devicetree/bindings/ata/ahci-platform.yaml
index cc35cdc02840..cd67926aae41 100644
--- a/Documentation/devicetree/bindings/ata/ahci-platform.yaml
+++ b/Documentation/devicetree/bindings/ata/ahci-platform.yaml
@@ -18,26 +18,6 @@ maintainers:
   - Hans de Goede <hdegoede@redhat.com>
   - Jens Axboe <axboe@kernel.dk>
 
-select:
-  properties:
-    compatible:
-      contains:
-        enum:
-          - brcm,iproc-ahci
-          - cavium,octeon-7130-ahci
-          - hisilicon,hisi-ahci
-          - ibm,476gtr-ahci
-          - marvell,armada-3700-ahci
-          - marvell,armada-8k-ahci
-          - marvell,berlin2q-ahci
-          - qcom,apq8064-ahci
-          - qcom,ipq806x-ahci
-          - socionext,uniphier-pro4-ahci
-          - socionext,uniphier-pxs2-ahci
-          - socionext,uniphier-pxs3-ahci
-  required:
-    - compatible
-
 properties:
   compatible:
     oneOf:
-- 
2.51.0

Re: [PATCH] dt-bindings: ata: ahci-platform: Drop unnecessary select schema

Posted by Damien Le Moal 4 weeks, 1 day ago

On 1/5/26 22:29, Rob Herring (Arm) wrote:
> The "select" schema is not necessary. It looks like it is there to prevent
> matching on "generic-ahci" compatible, but that's not necessary because
> this is the only place "generic-ahci" compatible is present.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Pushed to for-6.20. Thanks !


-- 
Damien Le Moal
Western Digital Research