From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS, URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BE2A8C43381 for ; Sun, 31 Mar 2019 10:04:17 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8C8DB20700 for ; Sun, 31 Mar 2019 10:04:17 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="p3Oz4eJT"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="mE0aMILM" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8C8DB20700 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=xbcZm1Rd5dAkccekB6v0RqtVM8DI7VUsEQF4RGGR+xw=; b=p3Oz4eJTXR2G60 hKG2SJc4/lQAnRuzkgLUNr9s5GAyHAPE0sTTVqE7UvUAqphoaB5AjS9TBq1Wi14FUyRZu6LhLvL2O MiDXMd1erTWIVXvZWG0//6FQ0T/gob+RFpXEF9yQBXfANrNoB1KOQIiM7nv5hXYreGt7krtlUiGBn lEA51WxCfMPj4xDrlv0VqzpSOskFRG0yqtE7EF6ojH+Cn4VeLs9sHZWpBXCE1OxP5c4ZYfkH2Qm43 jUtnjccJ4yG8YFvoaIuh/hTC342bHkKJrkKQUrPLf95BxWj5P6W13qF6umb3ArQM6lTfGjre0B8ec AJScGM00qB3DkInUy8nA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hAXJy-0004mK-0F; Sun, 31 Mar 2019 10:04:10 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hAXJt-0004lr-VN for linux-arm-kernel@lists.infradead.org; Sun, 31 Mar 2019 10:04:07 +0000 Received: from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net [81.96.234.148]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5EA0D206BA; Sun, 31 Mar 2019 10:04:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1554026644; bh=v1rlFX1bD8p6R7krMflQEWhK3ImKzZkOt7GbuWsGxEQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=mE0aMILM8dGf/+/nSGdVicPYADjYkcGDKasAszUh7k3dzTJSOcGd+byDrytChJ2NF k1m19XMEeFT+XNdG5NCIybHwpr9aR2Ie9XnffCEEhkvFaYVdTWr9aASWiaYCOQb8rf TFm9vEHbrYZjcZ5OBz00XUkx4M2Cvaw5MnvaaPVs= Date: Sun, 31 Mar 2019 11:03:58 +0100 From: Jonathan Cameron To: Fabrice Gasnier Subject: Re: [PATCH] iio: adc: stm32-dfsdm: add PM support Message-ID: <20190331110358.04b59b09@archlinux> In-Reply-To: <1553525368-23419-1-git-send-email-fabrice.gasnier@st.com> References: <1553525368-23419-1-git-send-email-fabrice.gasnier@st.com> X-Mailer: Claws Mail 3.17.3 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190331_030406_051967_0474A39E X-CRM114-Status: GOOD ( 22.77 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: lars@metafoo.de, olivier.moysan@st.com, alexandre.torgue@st.com, linux-iio@vger.kernel.org, pmeerw@pmeerw.net, arnaud.pouliquen@st.com, linux-kernel@vger.kernel.org, mcoquelin.stm32@gmail.com, knaack.h@gmx.de, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 25 Mar 2019 15:49:28 +0100 Fabrice Gasnier wrote: > Add PM and runtime PM support to STM32 DFSDM drivers: > - stm32-dfsdm-core: manage clocks. > - stm32-dfsdm-adc: restore channels configuration upon resume. Also stop > restart everything in case of buffer mode. > > Signed-off-by: Fabrice Gasnier I'll confess that power management code always gives me a headache, but I 'think' this looks right. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- > drivers/iio/adc/stm32-dfsdm-adc.c | 75 +++++++++++++++-- > drivers/iio/adc/stm32-dfsdm-core.c | 163 +++++++++++++++++++++++++++++++------ > 2 files changed, 206 insertions(+), 32 deletions(-) > > diff --git a/drivers/iio/adc/stm32-dfsdm-adc.c b/drivers/iio/adc/stm32-dfsdm-adc.c > index 531ca7e..a599097 100644 > --- a/drivers/iio/adc/stm32-dfsdm-adc.c > +++ b/drivers/iio/adc/stm32-dfsdm-adc.c > @@ -851,24 +851,18 @@ static int stm32_dfsdm_update_scan_mode(struct iio_dev *indio_dev, > return 0; > } > > -static int stm32_dfsdm_postenable(struct iio_dev *indio_dev) > +static int __stm32_dfsdm_postenable(struct iio_dev *indio_dev) > { > struct stm32_dfsdm_adc *adc = iio_priv(indio_dev); > int ret; > > - if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED) { > - ret = iio_triggered_buffer_postenable(indio_dev); > - if (ret < 0) > - return ret; > - } > - > /* Reset adc buffer index */ > adc->bufi = 0; > > if (adc->hwc) { > ret = iio_hw_consumer_enable(adc->hwc); > if (ret < 0) > - goto err_predisable; > + return ret; > } > > ret = stm32_dfsdm_start_dfsdm(adc->dfsdm); > @@ -896,6 +890,26 @@ static int stm32_dfsdm_postenable(struct iio_dev *indio_dev) > err_stop_hwc: > if (adc->hwc) > iio_hw_consumer_disable(adc->hwc); > + > + return ret; > +} > + > +static int stm32_dfsdm_postenable(struct iio_dev *indio_dev) > +{ > + int ret; > + > + if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED) { > + ret = iio_triggered_buffer_postenable(indio_dev); > + if (ret < 0) > + return ret; > + } > + > + ret = __stm32_dfsdm_postenable(indio_dev); > + if (ret < 0) > + goto err_predisable; > + > + return 0; > + > err_predisable: > if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED) > iio_triggered_buffer_predisable(indio_dev); > @@ -903,7 +917,7 @@ static int stm32_dfsdm_postenable(struct iio_dev *indio_dev) > return ret; > } > > -static int stm32_dfsdm_predisable(struct iio_dev *indio_dev) > +static void __stm32_dfsdm_predisable(struct iio_dev *indio_dev) > { > struct stm32_dfsdm_adc *adc = iio_priv(indio_dev); > > @@ -915,6 +929,11 @@ static int stm32_dfsdm_predisable(struct iio_dev *indio_dev) > > if (adc->hwc) > iio_hw_consumer_disable(adc->hwc); > +} > + > +static int stm32_dfsdm_predisable(struct iio_dev *indio_dev) > +{ > + __stm32_dfsdm_predisable(indio_dev); > > if (indio_dev->currentmode == INDIO_BUFFER_TRIGGERED) > iio_triggered_buffer_predisable(indio_dev); > @@ -1492,10 +1511,48 @@ static int stm32_dfsdm_adc_remove(struct platform_device *pdev) > return 0; > } > > +static int __maybe_unused stm32_dfsdm_adc_suspend(struct device *dev) > +{ > + struct stm32_dfsdm_adc *adc = dev_get_drvdata(dev); > + struct iio_dev *indio_dev = iio_priv_to_dev(adc); > + > + if (iio_buffer_enabled(indio_dev)) > + __stm32_dfsdm_predisable(indio_dev); > + > + return 0; > +} > + > +static int __maybe_unused stm32_dfsdm_adc_resume(struct device *dev) > +{ > + struct stm32_dfsdm_adc *adc = dev_get_drvdata(dev); > + struct iio_dev *indio_dev = iio_priv_to_dev(adc); > + const struct iio_chan_spec *chan; > + struct stm32_dfsdm_channel *ch; > + int i, ret; > + > + /* restore channels configuration */ > + for (i = 0; i < indio_dev->num_channels; i++) { > + chan = indio_dev->channels + i; > + ch = &adc->dfsdm->ch_list[chan->channel]; > + ret = stm32_dfsdm_chan_configure(adc->dfsdm, ch); > + if (ret) > + return ret; > + } > + > + if (iio_buffer_enabled(indio_dev)) > + __stm32_dfsdm_postenable(indio_dev); > + > + return 0; > +} > + > +static SIMPLE_DEV_PM_OPS(stm32_dfsdm_adc_pm_ops, > + stm32_dfsdm_adc_suspend, stm32_dfsdm_adc_resume); > + > static struct platform_driver stm32_dfsdm_adc_driver = { > .driver = { > .name = "stm32-dfsdm-adc", > .of_match_table = stm32_dfsdm_adc_match, > + .pm = &stm32_dfsdm_adc_pm_ops, > }, > .probe = stm32_dfsdm_adc_probe, > .remove = stm32_dfsdm_adc_remove, > diff --git a/drivers/iio/adc/stm32-dfsdm-core.c b/drivers/iio/adc/stm32-dfsdm-core.c > index 472b809..0a4d374 100644 > --- a/drivers/iio/adc/stm32-dfsdm-core.c > +++ b/drivers/iio/adc/stm32-dfsdm-core.c > @@ -12,6 +12,8 @@ > #include > #include > #include > +#include > +#include > #include > #include > > @@ -90,6 +92,36 @@ struct dfsdm_priv { > struct clk *aclk; /* audio clock */ > }; > > +static inline struct dfsdm_priv *to_stm32_dfsdm_priv(struct stm32_dfsdm *dfsdm) > +{ > + return container_of(dfsdm, struct dfsdm_priv, dfsdm); > +} > + > +static int stm32_dfsdm_clk_prepare_enable(struct stm32_dfsdm *dfsdm) > +{ > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > + int ret; > + > + ret = clk_prepare_enable(priv->clk); > + if (ret || !priv->aclk) > + return ret; > + > + ret = clk_prepare_enable(priv->aclk); > + if (ret) > + clk_disable_unprepare(priv->clk); > + > + return ret; > +} > + > +static void stm32_dfsdm_clk_disable_unprepare(struct stm32_dfsdm *dfsdm) > +{ > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > + > + if (priv->aclk) > + clk_disable_unprepare(priv->aclk); > + clk_disable_unprepare(priv->clk); > +} > + > /** > * stm32_dfsdm_start_dfsdm - start global dfsdm interface. > * > @@ -98,24 +130,17 @@ struct dfsdm_priv { > */ > int stm32_dfsdm_start_dfsdm(struct stm32_dfsdm *dfsdm) > { > - struct dfsdm_priv *priv = container_of(dfsdm, struct dfsdm_priv, dfsdm); > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > struct device *dev = &priv->pdev->dev; > unsigned int clk_div = priv->spi_clk_out_div, clk_src; > int ret; > > if (atomic_inc_return(&priv->n_active_ch) == 1) { > - ret = clk_prepare_enable(priv->clk); > + ret = pm_runtime_get_sync(dev); > if (ret < 0) { > - dev_err(dev, "Failed to start clock\n"); > + pm_runtime_put_noidle(dev); > goto error_ret; > } > - if (priv->aclk) { > - ret = clk_prepare_enable(priv->aclk); > - if (ret < 0) { > - dev_err(dev, "Failed to start audio clock\n"); > - goto disable_clk; > - } > - } > > /* select clock source, e.g. 0 for "dfsdm" or 1 for "audio" */ > clk_src = priv->aclk ? 1 : 0; > @@ -123,21 +148,21 @@ int stm32_dfsdm_start_dfsdm(struct stm32_dfsdm *dfsdm) > DFSDM_CHCFGR1_CKOUTSRC_MASK, > DFSDM_CHCFGR1_CKOUTSRC(clk_src)); > if (ret < 0) > - goto disable_aclk; > + goto pm_put; > > /* Output the SPI CLKOUT (if clk_div == 0 clock if OFF) */ > ret = regmap_update_bits(dfsdm->regmap, DFSDM_CHCFGR1(0), > DFSDM_CHCFGR1_CKOUTDIV_MASK, > DFSDM_CHCFGR1_CKOUTDIV(clk_div)); > if (ret < 0) > - goto disable_aclk; > + goto pm_put; > > /* Global enable of DFSDM interface */ > ret = regmap_update_bits(dfsdm->regmap, DFSDM_CHCFGR1(0), > DFSDM_CHCFGR1_DFSDMEN_MASK, > DFSDM_CHCFGR1_DFSDMEN(1)); > if (ret < 0) > - goto disable_aclk; > + goto pm_put; > } > > dev_dbg(dev, "%s: n_active_ch %d\n", __func__, > @@ -145,11 +170,8 @@ int stm32_dfsdm_start_dfsdm(struct stm32_dfsdm *dfsdm) > > return 0; > > -disable_aclk: > - clk_disable_unprepare(priv->aclk); > -disable_clk: > - clk_disable_unprepare(priv->clk); > - > +pm_put: > + pm_runtime_put_sync(dev); > error_ret: > atomic_dec(&priv->n_active_ch); > > @@ -165,7 +187,7 @@ EXPORT_SYMBOL_GPL(stm32_dfsdm_start_dfsdm); > */ > int stm32_dfsdm_stop_dfsdm(struct stm32_dfsdm *dfsdm) > { > - struct dfsdm_priv *priv = container_of(dfsdm, struct dfsdm_priv, dfsdm); > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > int ret; > > if (atomic_dec_and_test(&priv->n_active_ch)) { > @@ -183,9 +205,7 @@ int stm32_dfsdm_stop_dfsdm(struct stm32_dfsdm *dfsdm) > if (ret < 0) > return ret; > > - clk_disable_unprepare(priv->clk); > - if (priv->aclk) > - clk_disable_unprepare(priv->aclk); > + pm_runtime_put_sync(&priv->pdev->dev); > } > dev_dbg(&priv->pdev->dev, "%s: n_active_ch %d\n", __func__, > atomic_read(&priv->n_active_ch)); > @@ -325,14 +345,111 @@ static int stm32_dfsdm_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, dfsdm); > > - return devm_of_platform_populate(&pdev->dev); > + ret = stm32_dfsdm_clk_prepare_enable(dfsdm); > + if (ret) { > + dev_err(&pdev->dev, "Failed to start clock\n"); > + return ret; > + } > + > + pm_runtime_get_noresume(&pdev->dev); > + pm_runtime_set_active(&pdev->dev); > + pm_runtime_enable(&pdev->dev); > + > + ret = of_platform_populate(pdev->dev.of_node, NULL, NULL, &pdev->dev); > + if (ret) > + goto pm_put; > + > + pm_runtime_put(&pdev->dev); > + > + return 0; > + > +pm_put: > + pm_runtime_disable(&pdev->dev); > + pm_runtime_set_suspended(&pdev->dev); > + pm_runtime_put_noidle(&pdev->dev); > + stm32_dfsdm_clk_disable_unprepare(dfsdm); > + > + return ret; > +} > + > +static int stm32_dfsdm_core_remove(struct platform_device *pdev) > +{ > + struct stm32_dfsdm *dfsdm = platform_get_drvdata(pdev); > + > + pm_runtime_get_sync(&pdev->dev); > + of_platform_depopulate(&pdev->dev); > + pm_runtime_disable(&pdev->dev); > + pm_runtime_set_suspended(&pdev->dev); > + pm_runtime_put_noidle(&pdev->dev); > + stm32_dfsdm_clk_disable_unprepare(dfsdm); > + > + return 0; > +} > + > +static int __maybe_unused stm32_dfsdm_core_suspend(struct device *dev) > +{ > + struct stm32_dfsdm *dfsdm = dev_get_drvdata(dev); > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > + int ret; > + > + ret = pm_runtime_force_suspend(dev); > + if (ret) > + return ret; > + > + /* Balance devm_regmap_init_mmio_clk() clk_prepare() */ > + clk_unprepare(priv->clk); > + > + return pinctrl_pm_select_sleep_state(dev); > +} > + > +static int __maybe_unused stm32_dfsdm_core_resume(struct device *dev) > +{ > + struct stm32_dfsdm *dfsdm = dev_get_drvdata(dev); > + struct dfsdm_priv *priv = to_stm32_dfsdm_priv(dfsdm); > + int ret; > + > + ret = pinctrl_pm_select_default_state(dev); > + if (ret) > + return ret; > + > + ret = clk_prepare(priv->clk); > + if (ret) > + return ret; > + > + return pm_runtime_force_resume(dev); > } > > +static int __maybe_unused stm32_dfsdm_core_runtime_suspend(struct device *dev) > +{ > + struct stm32_dfsdm *dfsdm = dev_get_drvdata(dev); > + > + stm32_dfsdm_clk_disable_unprepare(dfsdm); > + > + return 0; > +} > + > +static int __maybe_unused stm32_dfsdm_core_runtime_resume(struct device *dev) > +{ > + struct stm32_dfsdm *dfsdm = dev_get_drvdata(dev); > + > + return stm32_dfsdm_clk_prepare_enable(dfsdm); > +} > + > +static const struct dev_pm_ops stm32_dfsdm_core_pm_ops = { > + SET_SYSTEM_SLEEP_PM_OPS(stm32_dfsdm_core_suspend, > + stm32_dfsdm_core_resume) > + SET_RUNTIME_PM_OPS(stm32_dfsdm_core_runtime_suspend, > + stm32_dfsdm_core_runtime_resume, > + NULL) > +}; > + > static struct platform_driver stm32_dfsdm_driver = { > .probe = stm32_dfsdm_probe, > + .remove = stm32_dfsdm_core_remove, > .driver = { > .name = "stm32-dfsdm", > .of_match_table = stm32_dfsdm_of_match, > + .pm = &stm32_dfsdm_core_pm_ops, > }, > }; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel