<div dir="ltr"><div><div>Hi Sylwester,<br><br></div><div>DRM FIMC driver could be more cleaned up with this patch series. And your third patch <br></div><div>And just minor issue. The second patch has build warnings like below,<br>
</div><div><br>WARNING: static const char * array should probably be static const char * const<br>
#111: FILE: drivers/gpu/drm/exynos/exynos_drm_fimc.c:89:<br>+static const char * fimc_clock_names[] = {<br><br>ERROR: "foo * bar" should be "foo *bar"<br>#111: FILE: drivers/gpu/drm/exynos/exynos_drm_fimc.c:89:<br>

<br><br></div><div>This is a minor issue so I can fix them. And as you already know, now drm fimc driver should be more cleaned up. Your patch set looks good to me but I'd like to take more opinions from others.</div>
<br></div>And you can find my comments at the third patch.<br><div><br><div>Thanks,<br>Inki Dae<br></div><div><br></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/17 Sylwester Nawrocki <span dir="ltr"><<a href="mailto:s.nawrocki@samsung.com" target="_blank">s.nawrocki@samsung.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Inki,<br>
<br>
This small patch series adds device tree support for the DRM FIMC driver.<br>
The binding documentation can be found in -next at Documentation/devicetree/<br>
bindings/media/samsung-fimc.txt.<br>
It will make the driver dependent on OF. This patch series is needed in<br>
3.10 to ensure simultaneous operation of the DRM FIMC and the camera ISP<br>
on Exynos4x12.<br>
<br>
Changes since v1:<br>
 - removed devm_kfree() that got erroneously re-added in patch 2/3 during<br>
   rebase.<br>
<br>
Thanks,<br>
Sylwester<br>
<br>
Sylwester Nawrocki (3):<br>
  drm/exynos: Remove redundant devm_kfree()<br>
  drm/exynos: Rework fimc clocks handling<br>
  drm/exynos: Add device tree support for fimc ipp driver<br>
<br>
 drivers/gpu/drm/exynos/Kconfig           |    2 +-<br>
 drivers/gpu/drm/exynos/exynos_drm_fimc.c |  264 ++++++++++++++++--------------<br>
 drivers/gpu/drm/exynos/regs-fimc.h       |    7 +-<br>
 3 files changed, 147 insertions(+), 126 deletions(-)<br>
<br>
--<br>
1.7.9.5<br>
<br>
_______________________________________________<br>
dri-devel mailing list<br>
<a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
</blockquote></div><br></div>