<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/12 Inki Dae <span dir="ltr"><<a href="mailto:inki.dae@samsung.com" target="_blank">inki.dae@samsung.com</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr"><div><div>Hi Rahul,<br><br></div>This patch is important to us. Actually, previous hdmi driver had controlled hdmiphy HDMI_PHY_CONTROL as if that were a clock but now that doesn't exist anymore. So we need to discuss how hdmiphy should be handled.<br>


I konw that you had already posted hdmiphy relevant patch set, [PATCH 0/4] drm/exynos: hdmi: move hdmiphy related code to hdmiphy driver.<br><br></div><div>I think we can couple pmu register controlling codes with that patch set without RFC. Could you update and post them again? like below,<br>

[PATCH 0/4] drm/exynos: hdmi: move hdmiphy related code to hdmiphy driver + [RFC 0/2] exynos5250/hdmi: replace dummy hdmiphy clock with pmu reg control<br><br></div><div>And then let's start review :)<br></div></div>
</blockquote><div><br></div><div>And I think It would be better to move the pmu register controlling codes into hdmiphy driver like drivers/usb/phy/phy-samsung-usb2.c driver does.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div dir="ltr"><div></div><div><div>
<br></div><div>Thanks,<br>Inki Dae<br></div><div><br></div></div></div><div class=""><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/11 Rahul Sharma <span dir="ltr"><<a href="mailto:rahul.sharma@samsung.com" target="_blank">rahul.sharma@samsung.com</a>></span><br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Previously, hdmiphy is added as a dummy clock in clock file for<br>
exynos SoCs. Enable/Disable to this clock, actually toggles the power<br>
control bit in PMU, instead of controlling the clock gate.<br>
<br>
This RFC adds the support to parse hdmiphy control node which is a child<br>
node to hdmi, and map the pmu register to toggle the power control bit.<br>
<br>
This is based on drm-next branch in Inki Dae's tree.<br>
<br>
Rahul Sharma (2):<br>
  drm/exynos: replace dummy hdmiphy clock with pmu register control<br>
  ARM/dts: add hdmiphy power control pmu register to hdmi dt node<br>
<br>
 arch/arm/boot/dts/exynos5250.dtsi    |    6 +++<br>
 drivers/gpu/drm/exynos/exynos_hdmi.c |   69 ++++++++++++++++++++++++++++++----<br>
 drivers/gpu/drm/exynos/regs-hdmi.h   |    4 ++<br>
 3 files changed, 71 insertions(+), 8 deletions(-)<br>
<span><font color="#888888"><br>
--<br>
1.7.10.4<br>
<br>
--<br>
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in<br>
the body of a message to <a href="mailto:majordomo@vger.kernel.org" target="_blank">majordomo@vger.kernel.org</a><br>
More majordomo info at  <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div>