From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonyoung Shim Subject: Re: [RFC] drm/exynos: move hdmi clk disable out of pm ops Date: Fri, 30 Jan 2015 11:02:50 +0900 Message-ID: <54CAE64A.2040906@samsung.com> References: <1422538273-3097-1-git-send-email-gustavo@padovan.org> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: Received: from mailout1.samsung.com ([203.254.224.24]:40704 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758757AbbA3CCu (ORCPT ); Thu, 29 Jan 2015 21:02:50 -0500 Received: from epcpsbgr1.samsung.com (u141.gpu120.samsung.co.kr [203.254.230.141]) by mailout1.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0NIY00K7NWCOCZE0@mailout1.samsung.com> for linux-samsung-soc@vger.kernel.org; Fri, 30 Jan 2015 11:02:48 +0900 (KST) In-reply-to: <1422538273-3097-1-git-send-email-gustavo@padovan.org> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Gustavo Padovan , linux-samsung-soc@vger.kernel.org Cc: Marek Szyprowski , Andrzej Hajda , Sylwester Nawrocki , Tobias Jakobi , inki.dae@samsung.com, Prathyush K , kgene@kernel.org +Cc Kukjin, Hi, On 01/29/2015 10:31 PM, Gustavo Padovan wrote: > From: Prathyush K > > When VPLL clock of less than 140 MHz was used and all the three > clocks - hdmiphy, hdmi, sclk_hdmi are disabled, the system hangs > during S2R when HDMI is connected. Since we want to use a vpll > clock of 70.5 MHz, we cannot disable these 3 clocks before suspending. > This patch moves the clk enable/disable of hdmi and sclk_hdmi > outside of the pm ops. Now system suspends and resumes with HDMI > connected with VPLL set at 70.5 MHz. > > Signed-off-by: Prathyush K > Signed-off-by: Andrew Bresticker > Signed-off-by: Gustavo Padovan > > --- > This work depends on the HDMI support patches from Javier: > https://lkml.org/lkml/2015/1/20/235 > > This patch comes from a downstream tree (Google chormeOS) and it is > authored by a Samsung employee, but we still think this may not fix > the real cause of the bug, there might be something else that we > haven't found that could be the cause of this issue. Anyone has some > comment to add here? Hmm, do you test from which exynos SoC? Kukjin, if it is hw issue, do you know hw experts of exynos hdmi and could we get any advice? > --- > drivers/gpu/drm/exynos/exynos_hdmi.c | 17 +++++++++++------ > 1 file changed, 11 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c > index 6aa0d65..7a473cb 100644 > --- a/drivers/gpu/drm/exynos/exynos_hdmi.c > +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c > @@ -2064,9 +2064,6 @@ static void hdmi_poweron(struct exynos_drm_display *display) > regmap_update_bits(hdata->pmureg, PMU_HDMI_PHY_CONTROL, > PMU_HDMI_PHY_ENABLE_BIT, 1); > > - clk_prepare_enable(res->hdmi); > - clk_prepare_enable(res->sclk_hdmi); > - > hdmiphy_poweron(hdata); > hdmi_commit(display); > } > @@ -2088,9 +2085,6 @@ static void hdmi_poweroff(struct exynos_drm_display *display) > > cancel_delayed_work(&hdata->hotplug_work); > > - clk_disable_unprepare(res->sclk_hdmi); > - clk_disable_unprepare(res->hdmi); > - > /* reset pmu hdmiphy control bit to disable hdmiphy */ > regmap_update_bits(hdata->pmureg, PMU_HDMI_PHY_CONTROL, > PMU_HDMI_PHY_ENABLE_BIT, 0); > @@ -2254,6 +2248,14 @@ static int hdmi_resources_init(struct hdmi_context *hdata) > } else > res->reg_hdmi_en = NULL; > > + /* > + * These two clocks are not moved into hdmi_poweron/off since system > + * fails to suspend if VPLL clock of 70.5 MHz is used and these > + * clocks are disabled before suspend. So enable them here. > + */ > + clk_prepare_enable(res->sclk_hdmi); > + clk_prepare_enable(res->hdmi); > + Then twe clocks are turned on always. I don't think it's reasonable. Thanks. > return ret; > fail: > DRM_ERROR("HDMI resource init - failed\n"); > @@ -2516,6 +2518,9 @@ static int hdmi_remove(struct platform_device *pdev) > if (hdata->res.reg_hdmi_en) > regulator_disable(hdata->res.reg_hdmi_en); > > + clk_disable_unprepare(hdata->res.sclk_hdmi); > + clk_disable_unprepare(hdata->res.hdmi); > + > if (hdata->hdmiphy_port) > put_device(&hdata->hdmiphy_port->dev); > put_device(&hdata->ddc_adpt->dev); >