<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">2013/4/27 Tomasz Figa <span dir="ltr"><<a href="mailto:tomasz.figa@gmail.com" target="_blank">tomasz.figa@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 Friday 26 of April 2013 11:48:50 Sylwester Nawrocki wrote:<br>
> On 04/26/2013 10:20 AM, Inki Dae wrote:<br>
> > 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>
> 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>
<br>
</div>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></blockquote><div> </div><div>This way had already been used a long time ago and there was some probing issues so we made exynos drm framework to be one module.</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">
This way you would be able to load dynamically any submodule you want,<br>
without recompiling the modules.<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">
Best regards,<br>
Tomasz<br>
<div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div></div>