+Sean<br><br>On Thursday, August 6, 2015, Yakir Yang <<a href="mailto:ykk@rock-chips.com">ykk@rock-chips.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Jingoo,<br>
<br>
在 2015/8/6 22:41, Jingoo Han 写道:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thursday, August 06, 2015 10:49 PM, Yakir Yang wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
    Samsung exynos and Rockchip rk3288 almost share same dp controller,<br>
so I split the common code out, then rk3288 and exynos can re-used the<br>
same dp core driver. Cause I can't find the exact IP name of exynos dp<br>
controller, so I decide to name dp core driver with "analogix" which I<br>
find in rk3288 eDP TRM ;)<br>
</blockquote>
<br>
OK, I see.<br>
The Samsung Exynos eDP contoller and Rockchip rk3288 eDP contoller share<br>
the same IP. So, a lot of parts can be re-used. I agree with this.<br>
However, we have to review the code carefully, as others did.<br>
</blockquote>
Yeah, feel happy to be reviewed  ;)<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I also cannot find the exact IP name. The "analogix" may be the vendor name<br>
of this IP.<br>
</blockquote>
Okay, so "analogix" is okay for now<br>
<br>
Thanks,<br>
- Yakir<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Best regards,<br>
Jingoo Han<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Beyond that, there are three light registers setting differents bewteen<br>
exynos and rk3288.<br>
1. RK3288 have five special pll resigters which not indicata in exynos<br>
    dp controller.<br>
2. The address of DP_PHY_PD(dp phy power manager register) are different<br>
    between rk3288 and exynos.<br>
3. Rk3288 and exynos have different setting with AUX_HW_RETRY_CTL(dp debug<br>
    register).<br>
<br>
My series patches can be divider into two parts: One for spliting the<br>
analogix_dp code from exynos dp driver. Another are trying to add rk3288<br>
dp driver support.<br>
<br>
Best regards,<br>
- Yakir<br>
<br>
<br>
--<br>
2.1.2<br>
</blockquote>
<br>
<br>
<br>
<br>
</blockquote>
<br>
<br>
--<br>
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in<br>
the body of a message to <a>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>
</blockquote>