<br><br><div class="gmail_quote">On Mon, Jan 23, 2012 at 11:24 AM, Cousson, Benoit <span dir="ltr">&lt;<a href="mailto:b-cousson@ti.com">b-cousson@ti.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
+       if (oh) {<br>
+               dmm_platdata.base = omap_hwmod_get_mpu_rt_va(oh);<br>
+               dmm_platdata.irq = oh-&gt;mpu_irqs[0].irq;<br>
</blockquote>
<br></div>
These are internal hwmod attributes that should not be retrieved here. They are included in the device resources and this is up to the driver to get them using the platform_get_resource.</blockquote><div><br></div><div>Yeah this can be removed and I can switch to platform_get_resource.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
+               if (dmm_platdata.base)<br>
+                       omapdrm_platdata.dmm_pdata =&amp;dmm_platdata;<br>
</blockquote>
<br></div>
pdata is supposed to be used for storing board level information, and we are in the process of removing all of them for device tree adaptation. So you should not use that at all in this case if this is not strictly needed.</blockquote>
<div><br></div><div>Noted.  I&#39;ll just remove it.  I was planning ahead when I added this, but I can cross that bridge when I get there. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+       }<br>
+<br>
+       return platform_device_register(&amp;<u></u>omap_drm_device);<br>
</blockquote>
<br></div>
This is not the proper way to init a device nowadays.<br>
<br>
If you want to take advantage of the pm functionality, you should create an omap_device.<br>
Please have a look at the other OMAP devices creation code (GPIO, UART...).</blockquote><div><br></div><div>It was my understanding that you needed a hwmod entry that corresponded to the device if you wanted to use omap_device_build().  In the case of omap_drm, we don&#39;t have a hwmod entry.  We do have an entry for DMM.</div>
<div><br></div><div><br></div><div>Regards,</div><div><br></div><div>Andy Gross</div></div>