On Tue, May 17, 2022 at 02:14:09PM -0500, Rob Herring wrote: > 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. Ok. -Sergey > > > > > 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 > > > >
