From patchwork Fri Mar 23 01:30:53 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kunihiko Hayashi X-Patchwork-Id: 132316 Delivered-To: patch@linaro.org Received: by 10.46.84.29 with SMTP id i29csp1551074ljb; Thu, 22 Mar 2018 18:31:10 -0700 (PDT) X-Google-Smtp-Source: AG47ELtitMi674iPAKe2qSv7tmio1lkKavXzW4Y10mvYkmZr5IN6gqlRuHgj61OzK6UfWuTbYUbh X-Received: by 2002:a17:902:7502:: with SMTP id i2-v6mr10327251pll.220.1521768670080; Thu, 22 Mar 2018 18:31:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521768670; cv=none; d=google.com; s=arc-20160816; b=Y6tVcZIv0R7CbdzbXVJMztAOfggZ6WbvNASdFL5soUF+HHgKiqUB996BeJciFEZMMK Dt6uwBuUS2+VU1tejJ0fIDqI6ZO/LbXt6zW6WQg9RMV8kyuAp4xG+0DYp+nrT+0T4Wev xYx5I8SwZPrSHwJF4woz+xF31CY8i7QBNiEe/IzLzt6JMWlznlgFLddQzEto3Cwrm81S DmjS4jf6cpb/2qhZsyw5k1pz0RIVomrTdwczXKKaA1SKjr0JAFQmTHKvwKXFg3cOC2xz 5Ar+16arzrPYdk42A0QGeaPD8k+yK8X2oDpAammK7Zsl7+COnqVGHrHQDEmFtUgGNIbq ghfQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=SVOP3PIDBSO5QweS395UpHqPSaAAjCYTRu0TGQm+SAs=; b=HaYdpmSAQrgubKWpLVCQsmGSwk+eGfky1y1QmZ2MA460L2epjE/cLE2gMrVx7Ziz7u qzDrzynPavE/vGEXNm7DVBOC9rEQ1gtJkncoITxf0Rqj1Tm+EGSzm0M4L6a8tTdj5n2e 8vJ32dg9yRHcyxpQN3j7HJeCQm05QEdwvT3s7eDvplWyYgQ/b4iyurZjSXvylTeDZAlv lP8zgLjgtkD+zCgmtYHilxVzpkYfdvapD/Ui3FpfQ2r7bn4zg1xw7z5/oTpU0GQ5rH8G o7+QacW9vue2KshstCsSpw5krKl5vDqyPFJPOdQ5eO3OrlBEUHLF+BcHEAQbhhZqsXmP kSPg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of devicetree-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x6si5550648pfb.105.2018.03.22.18.31.09; Thu, 22 Mar 2018 18:31:10 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of devicetree-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of devicetree-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=devicetree-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751686AbeCWBbI (ORCPT + 6 others); Thu, 22 Mar 2018 21:31:08 -0400 Received: from mx.socionext.com ([202.248.49.38]:57567 "EHLO mx.socionext.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751657AbeCWBbH (ORCPT ); Thu, 22 Mar 2018 21:31:07 -0400 Received: from unknown (HELO kinkan-ex.css.socionext.com) ([172.31.9.52]) by mx.socionext.com with ESMTP; 23 Mar 2018 10:31:06 +0900 Received: from mail.mfilter.local (m-filter-1 [10.213.24.61]) by kinkan-ex.css.socionext.com (Postfix) with ESMTP id 78850180B68; Fri, 23 Mar 2018 10:31:06 +0900 (JST) Received: from 172.31.9.51 (172.31.9.51) by m-FILTER with ESMTP; Fri, 23 Mar 2018 10:31:06 +0900 Received: from plum.e01.socionext.com (unknown [10.213.132.32]) by kinkan.css.socionext.com (Postfix) with ESMTP id A53BD1A124F; Fri, 23 Mar 2018 10:31:05 +0900 (JST) From: Kunihiko Hayashi To: Tejun Heo , Hans de Goede , Rob Herring , Mark Rutland Cc: linux-ide@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Kunihiko Hayashi Subject: [PATCH] ata: ahci-platform: add reset control support Date: Fri, 23 Mar 2018 10:30:53 +0900 Message-Id: <1521768653-11824-1-git-send-email-hayashi.kunihiko@socionext.com> X-Mailer: git-send-email 2.7.4 Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Add support to get and control a list of resets for the device as optional and shared. These resets must be kept de-asserted until the device is enabled. This is specified as shared because some SoCs like UniPhier series have common reset controls with all ahci controller instances. Signed-off-by: Kunihiko Hayashi --- .../devicetree/bindings/ata/ahci-platform.txt | 1 + drivers/ata/ahci.h | 1 + drivers/ata/libahci_platform.c | 24 +++++++++++++++++++--- 3 files changed, 23 insertions(+), 3 deletions(-) -- 2.7.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Reviewed-by: Hans de Goede diff --git a/Documentation/devicetree/bindings/ata/ahci-platform.txt b/Documentation/devicetree/bindings/ata/ahci-platform.txt index c760ecb..f4006d3 100644 --- a/Documentation/devicetree/bindings/ata/ahci-platform.txt +++ b/Documentation/devicetree/bindings/ata/ahci-platform.txt @@ -30,6 +30,7 @@ compatible: Optional properties: - dma-coherent : Present if dma operations are coherent - clocks : a list of phandle + clock specifier pairs +- resets : a list of phandle + reset specifier pairs - target-supply : regulator for SATA target power - phys : reference to the SATA PHY node - phy-names : must be "sata-phy" diff --git a/drivers/ata/ahci.h b/drivers/ata/ahci.h index 749fd94..47ec77a2 100644 --- a/drivers/ata/ahci.h +++ b/drivers/ata/ahci.h @@ -347,6 +347,7 @@ struct ahci_host_priv { u32 em_msg_type; /* EM message type */ bool got_runtime_pm; /* Did we do pm_runtime_get? */ struct clk *clks[AHCI_MAX_CLKS]; /* Optional */ + struct reset_control *rsts; /* Optional */ struct regulator **target_pwrs; /* Optional */ /* * If platform uses PHYs. There is a 1:1 relation between the port number and diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c index 341d0ef..3130db9 100644 --- a/drivers/ata/libahci_platform.c +++ b/drivers/ata/libahci_platform.c @@ -25,6 +25,7 @@ #include #include #include +#include #include "ahci.h" static void ahci_host_stop(struct ata_host *host); @@ -195,7 +196,8 @@ EXPORT_SYMBOL_GPL(ahci_platform_disable_regulators); * following order: * 1) Regulator * 2) Clocks (through ahci_platform_enable_clks) - * 3) Phys + * 3) Resets + * 4) Phys * * If resource enabling fails at any point the previous enabled resources * are disabled in reverse order. @@ -215,12 +217,19 @@ int ahci_platform_enable_resources(struct ahci_host_priv *hpriv) if (rc) goto disable_regulator; - rc = ahci_platform_enable_phys(hpriv); + rc = reset_control_deassert(hpriv->rsts); if (rc) goto disable_clks; + rc = ahci_platform_enable_phys(hpriv); + if (rc) + goto disable_resets; + return 0; +disable_resets: + reset_control_assert(hpriv->rsts); + disable_clks: ahci_platform_disable_clks(hpriv); @@ -239,12 +248,15 @@ EXPORT_SYMBOL_GPL(ahci_platform_enable_resources); * following order: * 1) Phys * 2) Clocks (through ahci_platform_disable_clks) - * 3) Regulator + * 3) Resets + * 4) Regulator */ void ahci_platform_disable_resources(struct ahci_host_priv *hpriv) { ahci_platform_disable_phys(hpriv); + reset_control_assert(hpriv->rsts); + ahci_platform_disable_clks(hpriv); ahci_platform_disable_regulators(hpriv); @@ -393,6 +405,12 @@ struct ahci_host_priv *ahci_platform_get_resources(struct platform_device *pdev) hpriv->clks[i] = clk; } + hpriv->rsts = devm_reset_control_array_get_optional_shared(dev); + if (IS_ERR(hpriv->rsts)) { + rc = PTR_ERR(hpriv->rsts); + goto err_out; + } + hpriv->nports = child_nodes = of_get_child_count(dev->of_node); /*