<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 14, 2014 at 9:37 PM, Andy Yan <span dir="ltr"><<a href="mailto:andy.yan@rock-chips.com" target="_blank">andy.yan@rock-chips.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><br>
We found Freescale imx6 and Rockchip rk3288 and Ingenic JZ4780 (Xburst/MIPS)<br>
use the interface compatible Designware HDMI IP, but they also have some<br>
lightly differences, such as phy pll configuration, register width(imx hdmi<br>
register is one byte, but rk3288 is 4 bytes width and can only be accessed<br>
by word), 4K support(imx6 doesn't support 4k, but rk3288 does), and HDMI2.0<br>
support.<br>
<br>
To reuse the imx-hdmi driver, we make this patch set:<br>
(1): fix some CodingStyle warning to make checkpatch happy<br>
(2): split out imx-soc code from imx-hdmi to dw_hdmi-imx.c<br>
(3): move imx-hdmi to bridge/dw-hdmi, and convert it to a drm_bridge driver<br>
(4): add rockchip rk3288 platform specific code dw_hdmi-rockchip.c<br>
<br>
</span>Changes in v11:<br>
<span class="">- rearrange the patch order<br>
</span>- squash patch <split some phy configuration to platform driver><br>
- split some modification form patch <dw_hdmi: add rk3288 support><br>
<br>
Changes in v10:<br>
- split generic dw_hdmi.c improvements from patch#11 (add rk3288 support)<br>
<span class="">- add more display mode support mpll configuration for rk3288<br>
<br>
Changes in v9:<br>
- move some phy configuration to platform driver<br>
<br>
Changes in v8:<br>
- correct some spelling mistake<br>
- modify ddc-i2c-bus and interrupt description<br>
- Add documentation for rockchip dw hdmi<br>
<br>
Changes in v7:<br>
- remove unused variables from structure dw_hdmi<br>
- remove a wrong modification<br>
- add copyrights for dw_hdmi-imx.c<br>
<br>
Changes in v6:<br>
- rearrange the patch order<br>
</span>- move some modification from patch#5<br>
<span class="">- move some modification to patch#6<br>
- refactor register access without reg_shift<br>
<br>
</span><span class="">Changes in v5:<br>
- refactor reg-io-width<br>
<br>
Changes in v4:<br>
- fix checkpatch CHECK<br>
- defer probe ddc i2c adapter<br>
<br>
Changes in v3:<br>
</span>- split multi-register access to one indepent patch<br>
<span class=""><br>
Changes in v2:<br>
- use git format -M to generate these patch<br>
<br>
</span>Andy Yan (11):<br>
<span class=""> staging: imx-drm: imx-hdmi: make checkpatch happy<br>
staging: imx-drm: imx-hdmi: return defer if can't get ddc i2c adapter<br>
staging: imx-drm: imx-hdmi: split imx soc specific code from imx-hdmi<br>
staging: imx-drm: imx-hdmi: move imx-hdmi to bridge/dw_hdmi<br>
</span><span class=""> dt-bindings: add document for dw_hdmi<br>
</span> drm: bridge/dw_hdmi: add support for multi-byte register width access<br>
drm: bridge/dw_hdmi: add mode_valid support<br>
<span class=""> drm: bridge/dw_hdmi: clear i2cmphy_stat0 reg in hdmi_phy_wait_i2c_done<br>
</span> drm: bridge/dw_hdmi: add function dw_hdmi_phy_enable_spare<br>
<div class=""><div class="h5"> dt-bindings: Add documentation for rockchip dw hdmi<br>
drm: bridge/dw_hdmi: add rockchip rk3288 support<br></div></div></blockquote><div><br></div><div><div>Patch 1 & 2 are:</div><div><br></div><div>Reviewed-by: Daniel Kurtz <<a href="mailto:djkurtz@chromium.org">djkurtz@chromium.org</a>></div><div><br></div><div>I think these first two are ready (or will be, after a small nit fix to #2).</div><div><br></div><div>Since they are independent of the imx/dw_hdmi split, can we get them accepted first, to help shrink Andy's patch set?</div><div><br></div><div>Thanks,</div><div>-djk</div></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">
<br>
Yakir Yang (1):<br>
drm: bridge/dw_hdmi: convert dw-hdmi to drm_bridge mode<br>
<br>
.../devicetree/bindings/drm/bridge/dw_hdmi.txt | 40 ++<br>
.../devicetree/bindings/video/dw_hdmi-rockchip.txt | 43 ++<br>
drivers/gpu/drm/bridge/Kconfig | 5 +<br>
drivers/gpu/drm/bridge/Makefile | 1 +<br>
.../imx-hdmi.c => gpu/drm/bridge/dw_hdmi.c} | 762 +++++++++------------<br>
.../imx-hdmi.h => gpu/drm/bridge/dw_hdmi.h} | 8 +-<br>
drivers/gpu/drm/rockchip/Kconfig | 10 +<br>
drivers/gpu/drm/rockchip/Makefile | 2 +-<br>
drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 355 ++++++++++<br>
drivers/staging/imx-drm/Kconfig | 1 +<br>
drivers/staging/imx-drm/Makefile | 2 +-<br>
drivers/staging/imx-drm/dw_hdmi-imx.c | 274 ++++++++<br>
include/drm/bridge/dw_hdmi.h | 60 ++<br>
13 files changed, 1138 insertions(+), 425 deletions(-)<br>
create mode 100644 Documentation/devicetree/bindings/drm/bridge/dw_hdmi.txt<br>
create mode 100644 Documentation/devicetree/bindings/video/dw_hdmi-rockchip.txt<br>
rename drivers/{staging/imx-drm/imx-hdmi.c => gpu/drm/bridge/dw_hdmi.c} (70%)<br>
rename drivers/{staging/imx-drm/imx-hdmi.h => gpu/drm/bridge/dw_hdmi.h} (99%)<br>
create mode 100644 drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c<br>
create mode 100644 drivers/staging/imx-drm/dw_hdmi-imx.c<br>
create mode 100644 include/drm/bridge/dw_hdmi.h<br>
<br>
--<br>
1.9.1<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:'Times New Roman';font-size:medium"><div style="line-height:1.5em;padding-top:10px;margin-top:10px;color:rgb(85,85,85);font-family:sans-serif;font-size:small"><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(213,15,37);padding-top:2px;margin-top:2px">Daniel Kurtz |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(51,105,232);padding-top:2px;margin-top:2px"> Software Engineer |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(0,153,57);padding-top:2px;margin-top:2px"> <a href="mailto:djkurtz@google.com" target="_blank">djkurtz@google.com</a> |</span><span style="border-width:2px 0px 0px;border-style:solid;border-color:rgb(238,178,17);padding-top:2px;margin-top:2px"> <span id="gc-number-1" class="" title="Call with Google Voice">650.204.0722</span></span></div></span></div>
</div></div>