On Thu, May 12, 2022 at 02:17:50AM +0300, Serge Semin wrote: > Indeed in accordance with what is imeplemtned in the AHCI paltform driver > and the way the AHCI DT nodes are defined in the DT files we can add the > next AHCI DT properties constraints: AHCI CSR ID is fixed to 'ahci', PHY > name is fixed to 'sata-phy', AHCI controller can't have more than 32 ports > by design. > > Signed-off-by: Serge Semin <Sergey.Semin@xxxxxxxxxxxxxxxxxxxx> > > Changelog v2: > - This is a new patch created after rebasing v1 onto the 5.18-rc3 kernel. > --- > .../devicetree/bindings/ata/ahci-common.yaml | 15 ++++++++++----- > 1 file changed, 10 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/ata/ahci-common.yaml b/Documentation/devicetree/bindings/ata/ahci-common.yaml > index 620042ca12e7..a7d1a8353de3 100644 > --- a/Documentation/devicetree/bindings/ata/ahci-common.yaml > +++ b/Documentation/devicetree/bindings/ata/ahci-common.yaml > @@ -31,6 +31,8 @@ properties: > > reg-names: > description: CSR space IDs > + contains: > + const: ahci Okay, with this it makes sense to keep. The others still should go. > > interrupts: > description: > @@ -71,14 +73,13 @@ properties: > maxItems: 1 > > phy-names: > - maxItems: 1 > + const: sata-phy > > ports-implemented: > $ref: '/schemas/types.yaml#/definitions/uint32' > description: > Mask that indicates which ports the HBA supports. Useful if PI is not > programmed by the BIOS, which is true for some embedded SoC's. > - maximum: 0x1f > > patternProperties: > "^sata-port@[0-9a-f]+$": > @@ -89,8 +90,12 @@ patternProperties: > > properties: > reg: > - description: AHCI SATA port identifier > - maxItems: 1 > + description: > + AHCI SATA port identifier. By design AHCI controller can't have > + more than 32 ports due to the CAP.NP fields and PI register size > + constraints. > + minimum: 0 > + maximum: 31 > > phys: > description: Individual AHCI SATA port PHY > @@ -98,7 +103,7 @@ patternProperties: > > phy-names: > description: AHCI SATA port PHY ID > - maxItems: 1 > + const: sata-phy > > target-supply: > description: Power regulator for SATA port target device > -- > 2.35.1 > >
