<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/27 Sylwester Nawrocki <span dir="ltr"><<a href="mailto:sylvester.nawrocki@gmail.com" target="_blank">sylvester.nawrocki@gmail.com</a>></span><br>
<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote"><div class="im">On 04/26/2013 09:42 PM, Tomasz Figa wrote:<br>

<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
On Friday 26 of April 2013 11:48:50 Sylwester Nawrocki wrote:<br>
<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
On 04/26/2013 10:20 AM, Inki Dae wrote:<br>
<blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
Exactly right. it's my mistake. But now it seems that<br>
__mode_of_device_table is multi defined at fimd and g2d side so there<br>
still is module build error. :(<br>
</blockquote>
Since all drivers seem to be linked into single a single module, you<br>
likely need to create a separate table of struct of_device_id just for<br>
the purpose of MODULE_DEVICE_TABLE(of, ...). This table would contain<br>
'compatible' strings for all devices. Or choose of_device_id for just<br>
one device and define MODULE_DEVICE_TABLE() for it in some common place,<br>
e.g. exynos_drm_drv.c. I believe all devices should be listed though.<br>
</blockquote>
<br>
IMHO, the most proper solution would be to split the module into parent<br>
exynos_drm module and per-device submodules, which would depend on the<br>
parent module.<br>
<br>
This way you would be able to load dynamically any submodule you want,<br>
without recompiling the modules.<br>
</blockquote>
<br></div>
Yes, this is how it would have been in a perfect world. Probably something<br>
worth to consider for the future, it likely implies a lot of work.<br>
<br></blockquote><div> </div><div><div>For now, exynos drm framework has a problem to module support with device tree. And  for this, going back to previous style might be a right way but this also still has the probing issue. Frankly spreaking, I don't really want back to previous style because that way not only still has a probing issue but also makes exynos drm framework so complicated(there are many things to consider every time framework updating). So I'd like to resolve this issue as is first. </div>
</div><div> </div><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote"> </blockquote><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">

OTOH if there is one device for which the driver will always be present<br>
in the main module it should be enough to make a single entry<br>
MODULE_DEVICE_TABLE including its compatible string to ensure the driver<br>
is properly loaded, shouldn't it ?<br>
<br> </blockquote><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">
--<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/<u></u>majordomo-info.html</a><br>
</blockquote></div><br></div></div>