From patchwork Thu Mar 15 09:29:48 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Benjamin Gaignard X-Patchwork-Id: 10284143 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 5943F60386 for ; Thu, 15 Mar 2018 09:31:01 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5D84E28972 for ; Thu, 15 Mar 2018 09:31:01 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 5230628977; Thu, 15 Mar 2018 09:31:01 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CE7C528972 for ; Thu, 15 Mar 2018 09:31:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751588AbeCOJam (ORCPT ); Thu, 15 Mar 2018 05:30:42 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:52662 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751481AbeCOJak (ORCPT ); Thu, 15 Mar 2018 05:30:40 -0400 Received: by mail-wm0-f68.google.com with SMTP id t3so9059550wmc.2 for ; Thu, 15 Mar 2018 02:30:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=gfV0c872ULAHYwlNt2ZtFduTpDn5/sTi8IyqFyrjMvM=; b=LbbQl2kZse737gz5gc3kSxxkKXYJDtd/+1s3EMOzrjZGereaWnGm1IbR1WWgaCirE6 PZelymBIFzgeODZUvOGj+ZKzodulcmyZgmSazvfM+ALfytlX+zYJqiKHSNfPalG1G+bW 0to1NYc2EAMBtzDjIUJXuFWmFNXwzqyoMy37w= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=gfV0c872ULAHYwlNt2ZtFduTpDn5/sTi8IyqFyrjMvM=; b=R+jmkDWdJAY3bB0RTSnYHgNuuGaIV0FAMr6RnMhNnV81eTGmGsLASEeNVJdwjhbxeH s145oReOuG7PpchGmHjy3B5YSCI5DFAmRy/RceHZh0IgSQyiCWNWKBKLnwweYdTnhd95 Sc6s2E5a1jfRsBjXNOPfxrQCaFZ4YCafpK+OdXzaE73IrwTDJ6YVVz/fGbgMCf5msBjD qqdNw7DwY8KN065hU67sYkzNbDvFbaOCookY+UCeYSylvTZe4N81eKkvHg5KaqoNxAzW ZfsJQN2GHtfv7bu2DxZT9L6B0XpPlf4++lSUjX+rnGoS8OCs1lD7EFQhEKv1AYnHPTXW pCKA== X-Gm-Message-State: AElRT7FFwJ0G7KOaJXWPAcTjt1Da6zMAiWs3bQlQZEYq02LNiL12d+5b NOML30PvdtIcV9b9oW1mwmkjUg== X-Google-Smtp-Source: AG47ELukeXaLuEjgHvViGnOSmh5IGM1JTRImoSoNOqrEaNrtJQRLJPN115G8Ie1BQ17JnoPHD17/4A== X-Received: by 10.28.144.5 with SMTP id s5mr3971333wmd.98.1521106239147; Thu, 15 Mar 2018 02:30:39 -0700 (PDT) Received: from lmecxl0911.lme.st.com ([80.214.78.222]) by smtp.gmail.com with ESMTPSA id m71sm3603844wmd.6.2018.03.15.02.30.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 15 Mar 2018 02:30:38 -0700 (PDT) From: Benjamin Gaignard X-Google-Original-From: Benjamin Gaignard To: mchehab@kernel.org, mcoquelin.stm32@gmail.com, alexandre.torgue@st.com, hans.verkuil@cisco.com Cc: linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Benjamin Gaignard Subject: [PATCH 1/2] cec: stm32: simplify clock management Date: Thu, 15 Mar 2018 10:29:48 +0100 Message-Id: <20180315092949.9895-2-benjamin.gaignard@st.com> X-Mailer: git-send-email 2.15.0 In-Reply-To: <20180315092949.9895-1-benjamin.gaignard@st.com> References: <20180315092949.9895-1-benjamin.gaignard@st.com> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Benjamin Gaignard Since CEC framework enable and disable the adapter when it is needed just follow it orders to enable/disable the clocks. Call stm32_cec_hw_init() when the adapter is enabled and do not let regmap manage registers clock help to simplify clocking scheme. While reworking cec clock start using pm_runtime. Signed-off-by: Benjamin Gaignard --- drivers/media/platform/stm32/stm32-cec.c | 70 ++++++++++++++++++-------------- 1 file changed, 40 insertions(+), 30 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-cec.c b/drivers/media/platform/stm32/stm32-cec.c index 7c496bc1cf38..35cc2ffd6b96 100644 --- a/drivers/media/platform/stm32/stm32-cec.c +++ b/drivers/media/platform/stm32/stm32-cec.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include @@ -70,7 +71,7 @@ struct stm32_cec { int tx_cnt; }; -static void cec_hw_init(struct stm32_cec *cec) +static void stm32_cec_hw_init(struct stm32_cec *cec) { regmap_update_bits(cec->regmap, CEC_CR, TXEOM | TXSOM | CECEN, 0); @@ -166,22 +167,17 @@ static irqreturn_t stm32_cec_irq_handler(int irq, void *arg) static int stm32_cec_adap_enable(struct cec_adapter *adap, bool enable) { struct stm32_cec *cec = adap->priv; - int ret = 0; if (enable) { - ret = clk_enable(cec->clk_cec); - if (ret) - dev_err(cec->dev, "fail to enable cec clock\n"); - - clk_enable(cec->clk_hdmi_cec); + pm_runtime_get_sync(cec->dev); + stm32_cec_hw_init(cec); regmap_update_bits(cec->regmap, CEC_CR, CECEN, CECEN); } else { - clk_disable(cec->clk_cec); - clk_disable(cec->clk_hdmi_cec); regmap_update_bits(cec->regmap, CEC_CR, CECEN, 0); + pm_runtime_disable(cec->dev); } - return ret; + return 0; } static int stm32_cec_adap_log_addr(struct cec_adapter *adap, u8 logical_addr) @@ -260,8 +256,8 @@ static int stm32_cec_probe(struct platform_device *pdev) if (IS_ERR(mmio)) return PTR_ERR(mmio); - cec->regmap = devm_regmap_init_mmio_clk(&pdev->dev, "cec", mmio, - &stm32_cec_regmap_cfg); + cec->regmap = devm_regmap_init_mmio(&pdev->dev, mmio, + &stm32_cec_regmap_cfg); if (IS_ERR(cec->regmap)) return PTR_ERR(cec->regmap); @@ -284,19 +280,10 @@ static int stm32_cec_probe(struct platform_device *pdev) return PTR_ERR(cec->clk_cec); } - ret = clk_prepare(cec->clk_cec); - if (ret) { - dev_err(&pdev->dev, "Unable to prepare cec clock\n"); - return ret; - } - cec->clk_hdmi_cec = devm_clk_get(&pdev->dev, "hdmi-cec"); - if (!IS_ERR(cec->clk_hdmi_cec)) { - ret = clk_prepare(cec->clk_hdmi_cec); - if (ret) { - dev_err(&pdev->dev, "Unable to prepare hdmi-cec clock\n"); - return ret; - } + if (IS_ERR(cec->clk_hdmi_cec)) { + dev_err(&pdev->dev, "Cannot get cec clock\n"); + return PTR_ERR(cec->clk_hdmi_cec); } /* @@ -315,24 +302,46 @@ static int stm32_cec_probe(struct platform_device *pdev) return ret; } - cec_hw_init(cec); - platform_set_drvdata(pdev, cec); + pm_runtime_enable(&pdev->dev); - return 0; + return ret; } static int stm32_cec_remove(struct platform_device *pdev) { struct stm32_cec *cec = platform_get_drvdata(pdev); - clk_unprepare(cec->clk_cec); - clk_unprepare(cec->clk_hdmi_cec); - cec_unregister_adapter(cec->adap); + pm_runtime_disable(&pdev->dev); + return 0; } +static int __maybe_unused stm32_cec_runtime_suspend(struct device *dev) +{ + struct stm32_cec *cec = dev_get_drvdata(dev); + + clk_disable_unprepare(cec->clk_cec); + clk_disable_unprepare(cec->clk_hdmi_cec); + + return 0; +} + +static int __maybe_unused stm32_cec_runtime_resume(struct device *dev) +{ + struct stm32_cec *cec = dev_get_drvdata(dev); + + clk_prepare_enable(cec->clk_cec); + clk_prepare_enable(cec->clk_hdmi_cec); + + return 0; +} + +static const struct dev_pm_ops stm32_cec_pm_ops = { + SET_RUNTIME_PM_OPS(stm32_cec_runtime_suspend, stm32_cec_runtime_resume, + NULL) +}; static const struct of_device_id stm32_cec_of_match[] = { { .compatible = "st,stm32-cec" }, @@ -346,6 +355,7 @@ static struct platform_driver stm32_cec_driver = { .driver = { .name = CEC_NAME, .of_match_table = stm32_cec_of_match, + .pm = &stm32_cec_pm_ops, }, };