{"id":1641658,"url":"http://patchwork.ozlabs.org/api/patches/1641658/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-ide/patch/20220610081801.11854-4-Sergey.Semin@baikalelectronics.ru/","project":{"id":13,"url":"http://patchwork.ozlabs.org/api/projects/13/?format=json","name":"Linux IDE development","link_name":"linux-ide","list_id":"linux-ide.vger.kernel.org","list_email":"linux-ide@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20220610081801.11854-4-Sergey.Semin@baikalelectronics.ru>","list_archive_url":null,"date":"2022-06-10T08:17:41","name":"[v4,03/23] dt-bindings: ata: ahci-platform: Clarify common AHCI props constraints","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"dad54b68d958fa9818f92d94b49b1b747abaadad","submitter":{"id":78624,"url":"http://patchwork.ozlabs.org/api/people/78624/?format=json","name":"Serge Semin","email":"Sergey.Semin@baikalelectronics.ru"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-ide/patch/20220610081801.11854-4-Sergey.Semin@baikalelectronics.ru/mbox/","series":[{"id":304159,"url":"http://patchwork.ozlabs.org/api/series/304159/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-ide/list/?series=304159","date":"2022-06-10T08:17:42","name":"ata: ahci: Add DWC/Baikal-T1 AHCI SATA support","version":4,"mbox":"http://patchwork.ozlabs.org/series/304159/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/1641658/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/1641658/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["bilbo.ozlabs.org;\n\tdkim=pass (1024-bit key;\n unprotected) header.d=baikalelectronics.ru header.i=@baikalelectronics.ru\n header.a=rsa-sha256 header.s=mail header.b=Il7nEgMd;\n\tdkim-atps=neutral","ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2620:137:e000::1:20; helo=out1.vger.email;\n envelope-from=linux-ide-owner@vger.kernel.org; receiver=)"],"Received":["from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20])\n\tby bilbo.ozlabs.org (Postfix) with ESMTP id 4LKDPf5SHGz9s75\n\tfor ; Fri, 10 Jun 2022 18:18:46 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n id S1347324AbiFJISo (ORCPT );\n Fri, 10 Jun 2022 04:18:44 -0400","from lindbergh.monkeyblade.net ([23.128.96.19]:55844 \"EHLO\n lindbergh.monkeyblade.net\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n with ESMTP id S244704AbiFJISN (ORCPT\n ); Fri, 10 Jun 2022 04:18:13 -0400","from mail.baikalelectronics.com (mail.baikalelectronics.com\n [87.245.175.230])\n by lindbergh.monkeyblade.net (Postfix) with ESMTP id EF75A23428B;\n Fri, 10 Jun 2022 01:18:11 -0700 (PDT)","from mail (mail.baikal.int [192.168.51.25])\n by mail.baikalelectronics.com (Postfix) with ESMTP id 4789716A4;\n Fri, 10 Jun 2022 11:18:58 +0300 (MSK)","from localhost (192.168.53.207) by mail (192.168.51.25) with\n Microsoft SMTP Server (TLS) id 15.0.1395.4; Fri, 10 Jun 2022 11:18:05 +0300"],"DKIM-Filter":"OpenDKIM Filter v2.11.0 mail.baikalelectronics.com 4789716A4","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=baikalelectronics.ru; s=mail; t=1654849138;\n bh=lwyWyOK1nXgLe3QnPATCiNk6Mfk5wQ5Jb1rS0FxolVA=;\n h=From:To:CC:Subject:Date:In-Reply-To:References:From;\n b=Il7nEgMdm5C8vlcGQUDUpMRj10ccx7oOKEmRE5zwxOFt9D4pHV8IUDxQ15zNgb0Ar\n ZqOqlt65VoSQGX221r2L4jvGauxsv0NSEI1rbqMzb0k7aCea+Dj80CLThyopN/aYJg\n knB0Pk5cKazWxB6Zb6uATblGKPmoDl/hpKdTlkKo=","From":"Serge Semin ","To":"Damien Le Moal ,\n Hans de Goede ,\n Jens Axboe , Hannes Reinecke ,\n Rob Herring ,\n Krzysztof Kozlowski ","CC":"Serge Semin ,\n Serge Semin ,\n Alexey Malahov ,\n Pavel Parkhomenko ,\n , ,\n ","Subject":"[PATCH v4 03/23] dt-bindings: ata: ahci-platform: Clarify common AHCI\n props constraints","Date":"Fri, 10 Jun 2022 11:17:41 +0300","Message-ID":"<20220610081801.11854-4-Sergey.Semin@baikalelectronics.ru>","In-Reply-To":"<20220610081801.11854-1-Sergey.Semin@baikalelectronics.ru>","References":"<20220610081801.11854-1-Sergey.Semin@baikalelectronics.ru>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Content-Type":"text/plain","X-ClientProxiedBy":"MAIL.baikal.int (192.168.51.25) To mail (192.168.51.25)","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS,\n T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no\n version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n lindbergh.monkeyblade.net","Precedence":"bulk","List-ID":"","X-Mailing-List":"linux-ide@vger.kernel.org"},"content":"Indeed in accordance with what is implemented in the AHCI platform driver\nand the way the AHCI DT nodes are defined in the DT files we can add the\nnext AHCI DT properties constraints: AHCI CSR ID is fixed to 'ahci', PHY\nname is fixed to 'sata-phy', AHCI controller can't have more than 32 ports\nby design, AHCI controller can have up to 32 IRQ lines.\n\nSigned-off-by: Serge Semin \nReviewed-by: Hannes Reinecke \n\n---\n\nChangelog v2:\n- This is a new patch created after rebasing v1 onto the 5.18-rc3 kernel.\n\nChangelog v4:\n- Fix spelling: 'imeplemtned' and 'paltform' in the patch log. (@Hannes)\n- Add the interrupts property constraints. (@Rob)\n- Add forgotten '---' patchlog-changelog separator. (@Sergei)\n---\n .../devicetree/bindings/ata/ahci-common.yaml | 17 ++++++++++++-----\n 1 file changed, 12 insertions(+), 5 deletions(-)","diff":"diff --git a/Documentation/devicetree/bindings/ata/ahci-common.yaml b/Documentation/devicetree/bindings/ata/ahci-common.yaml\nindex e89bda3b62cc..12a97b56226f 100644\n--- a/Documentation/devicetree/bindings/ata/ahci-common.yaml\n+++ b/Documentation/devicetree/bindings/ata/ahci-common.yaml\n@@ -31,12 +31,16 @@ properties:\n \n reg-names:\n description: CSR space IDs\n+ contains:\n+ const: ahci\n \n interrupts:\n description:\n Generic AHCI state change interrupt. Can be implemented either as a\n single line attached to the controller or as a set of the signals\n indicating the particular port events.\n+ minItems: 1\n+ maxItems: 32\n \n ahci-supply:\n description: Power regulator for AHCI controller\n@@ -52,14 +56,13 @@ properties:\n maxItems: 1\n \n phy-names:\n- maxItems: 1\n+ const: sata-phy\n \n ports-implemented:\n $ref: '/schemas/types.yaml#/definitions/uint32'\n description:\n Mask that indicates which ports the HBA supports. Useful if PI is not\n programmed by the BIOS, which is true for some embedded SoC's.\n- maximum: 0x1f\n \n patternProperties:\n \"^sata-port@[0-9a-f]+$\":\n@@ -80,8 +83,12 @@ $defs:\n \n properties:\n reg:\n- description: AHCI SATA port identifier\n- maxItems: 1\n+ description:\n+ AHCI SATA port identifier. By design AHCI controller can't have\n+ more than 32 ports due to the CAP.NP fields and PI register size\n+ constraints.\n+ minimum: 0\n+ maximum: 31\n \n phys:\n description: Individual AHCI SATA port PHY\n@@ -89,7 +96,7 @@ $defs:\n \n phy-names:\n description: AHCI SATA port PHY ID\n- maxItems: 1\n+ const: sata-phy\n \n target-supply:\n description: Power regulator for SATA port target device\n","prefixes":["v4","03/23"]}