Re: [PATCH v6 1/6] dt-bindings: ata: ahci-platform: Convert DT bindings to yaml

From: Rob Herring
Date: Fri Mar 11 2022 - 22:45:43 EST


On Fri, 11 Mar 2022 22:03:52 +0100, Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
>
> Create a yaml file for dtbs_check from the old txt binding.
>
> Signed-off-by: Frank Wunderlich <frank-w@xxxxxxxxxxxxxxx>
> ---
>
> v6:
> - fix indentation of examples
> - add compatible marvell,berlin2-ahci
> - change maximum of ports-implemented
> - add select to exclude qcom compatibles
> - drop marvell,armada-380-ahci
> it is not handled in the ahci-platform.c but ahci_mvebu.c
> and incompatible due to missing phys/target-supply
>
> v5:
> - change subject
> - drop brcm,iproc-ahci from standalone enum
> - fix reg address in example 2
> - move clocknames next to clocks, regnames to reg
> - drop interrupts description
> - drop newline from dma-coherent
> - drop max-items from ports-implemented
> - min2max in child phys
> - fix identation for compatible and sata-common
> - add additionalProperties=false for subnodes
> - pipe for paragraphs and newline after title
> - add maximum for ports-implemented (found only 0x1 as its value)
> - add phy-names to sata-ports
>
> v4:
> - fix min vs. max
> - fix indention of examples
> - move up sata-common.yaml
> - reorder compatible
> - add descriptions/maxitems
> - fix compatible-structure
> - fix typo in example achi vs. ahci
> - add clock-names and reg-names
> - fix ns2 errors in separate patch
> v3:
> - add conversion to sata-series
> - fix some errors in dt_binding_check and dtbs_check
> - move to unevaluated properties = false
>
> arch/arm/boot/dts/qcom-apq8064.dtsi had caused errors for clock-count
> ---
>
> have not added reviewed-by from v5 because i have changed patch too much
>
> ---
> .../devicetree/bindings/ata/ahci-platform.txt | 79 --------
> .../bindings/ata/ahci-platform.yaml | 182 ++++++++++++++++++
> 2 files changed, 182 insertions(+), 79 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/ata/ahci-platform.txt
> create mode 100644 Documentation/devicetree/bindings/ata/ahci-platform.yaml
>

Applied, thanks!