Re: [PATCH 1/5] drm/exynos: remove dummy hdmiphy clock from hdmi driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rahul,

On 02.04.2014 19:13, Rahul Sharma wrote:
From: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx>

Exynos drm hdmi driver used to get dummy hdmiphy clock to
control the PMU bit for hdmiphy. This clock is removed
during CCF migration. This should also be cleaned from
hdmi driver.

Signed-off-by: Rahul Sharma <Rahul.Sharma@xxxxxxxxxxx>
---
  drivers/gpu/drm/exynos/exynos_hdmi.c |    8 --------
  1 file changed, 8 deletions(-)

diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c
index 25bf65a..23abfa0 100644
--- a/drivers/gpu/drm/exynos/exynos_hdmi.c
+++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
@@ -74,7 +74,6 @@ struct hdmi_resources {
  	struct clk			*sclk_hdmi;
  	struct clk			*sclk_pixel;
  	struct clk			*sclk_hdmiphy;
-	struct clk			*hdmiphy;
  	struct clk			*mout_hdmi;
  	struct regulator_bulk_data	*regul_bulk;
  	int				regul_count;
@@ -1854,7 +1853,6 @@ static void hdmi_poweron(struct exynos_drm_display *display)
  	if (regulator_bulk_enable(res->regul_count, res->regul_bulk))
  		DRM_DEBUG_KMS("failed to enable regulator bulk\n");

-	clk_prepare_enable(res->hdmiphy);
  	clk_prepare_enable(res->hdmi);
  	clk_prepare_enable(res->sclk_hdmi);

@@ -1881,7 +1879,6 @@ static void hdmi_poweroff(struct exynos_drm_display *display)

  	clk_disable_unprepare(res->sclk_hdmi);
  	clk_disable_unprepare(res->hdmi);
-	clk_disable_unprepare(res->hdmiphy);
  	regulator_bulk_disable(res->regul_count, res->regul_bulk);

  	pm_runtime_put_sync(hdata->dev);
@@ -1977,11 +1974,6 @@ static int hdmi_resources_init(struct hdmi_context *hdata)
  		DRM_ERROR("failed to get clock 'sclk_hdmiphy'\n");
  		goto fail;
  	}
-	res->hdmiphy = devm_clk_get(dev, "hdmiphy");
-	if (IS_ERR(res->hdmiphy)) {
-		DRM_ERROR("failed to get clock 'hdmiphy'\n");
-		goto fail;
-	}
  	res->mout_hdmi = devm_clk_get(dev, "mout_hdmi");
  	if (IS_ERR(res->mout_hdmi)) {
  		DRM_ERROR("failed to get clock 'mout_hdmi'\n");


This patch makes the series non-bisectable. If you remove handling of this dummy clock until you add proper support for PHY isolation setting, then at this point you end up with non-working code.

You should first provide new infrastructure in parallel to existing one, then move all users to new one and only then drop the old one.

Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux