[RFC 0/2] exynos5250/hdmi: replace dummy hdmiphy clock with pmu reg control

[Inki Dae]

2013/6/12 Inki Dae <inki.dae at samsung.com>

> Hi Rahul,
>
> 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.
> I konw that you had already posted hdmiphy relevant patch set, [PATCH 0/4]
> drm/exynos: hdmi: move hdmiphy related code to hdmiphy driver.
>
> I think we can couple pmu register controlling codes with that patch set
> without RFC. Could you update and post them again? like below,
> [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
>
> And then let's start review :)
>

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.


>
> Thanks,
> Inki Dae
>
>
>
> 2013/6/11 Rahul Sharma <rahul.sharma at samsung.com>
>
>> Previously, hdmiphy is added as a dummy clock in clock file for
>> exynos SoCs. Enable/Disable to this clock, actually toggles the power
>> control bit in PMU, instead of controlling the clock gate.
>>
>> This RFC adds the support to parse hdmiphy control node which is a child
>> node to hdmi, and map the pmu register to toggle the power control bit.
>>
>> This is based on drm-next branch in Inki Dae's tree.
>>
>> Rahul Sharma (2):
>>   drm/exynos: replace dummy hdmiphy clock with pmu register control
>>   ARM/dts: add hdmiphy power control pmu register to hdmi dt node
>>
>>  arch/arm/boot/dts/exynos5250.dtsi    |    6 +++
>>  drivers/gpu/drm/exynos/exynos_hdmi.c |   69
>> ++++++++++++++++++++++++++++++----
>>  drivers/gpu/drm/exynos/regs-hdmi.h   |    4 ++
>>  3 files changed, 71 insertions(+), 8 deletions(-)
>>
>> --
>> 1.7.10.4
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe
>> linux-samsung-soc" in
>> the body of a message to majordomo at vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130612/b557afea/attachment.html>