<br><div class="gmail_quote">On Tue, May 10, 2011 at 9:34 PM, Todd Poynor <span dir="ltr">&lt;<a href="mailto:toddpoynor@google.com">toddpoynor@google.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="gmail_quote">On Tue, May 10, 2011 at 3:30 AM, Govindraj.R <span dir="ltr">&lt;<a href="mailto:govindraj.raja@ti.com" target="_blank">govindraj.raja@ti.com</a>&gt;</span> wrote:</div><div class="gmail_quote">...<div class="im">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+/**<br>
+ * omap_device_enable_ioring_wakeup - Set wakeup bit for iopad ring.<br>
+ * @pdev: platform_device for which wakeup needs to be set.<br>
+ *<br>
+ * Caller should ensure this is called if device_may_wakeup(dev) is true<br>
+ * traverse through each hwmod and check each available pads<br>
+ * if pad is enabled then set wakeup enable flag for the mux pin.<br>
+ * Return error if pads are not enabled or not available.<br>
+ * Wakeup enable flag will be we used during hwmod idle transistion.<br>
+ */<br>
+int omap_device_enable_ioring_wakeup(struct platform_device *pdev)<br>
+{<br>
+       int ret = -EINVAL, i;<br>
+       struct omap_device *od;<br>
+       struct omap_hwmod *oh;<br>
+<br>
+       od = _find_by_pdev(pdev);<br>
+       for (i = 0; i &lt; od-&gt;hwmods_cnt; i++) {<br>
+               oh = od-&gt;hwmods[i];<br>
+               ret = omap_hwmod_enable_ioring_wakeup(oh);<br>
+       }<br>
+<br>
+       return ret;<br></blockquote><div><br></div></div><div>Error returns from all but the last hwmod are discarded.  If the error returns are meaningful and need corrective action (or warning messages) by callers then these should be preserved.</div>

</div></blockquote><div><br></div><div>Actually here we either consider whether dynamic_io_pad was available for enabling wakeup</div><div>or whether its never available for given hwmod. So thats the reason we are not preserving ret, </div>

<div>also wakeup capability will be dependent on wakeup enable flag and remux flag which will set</div><div>during hwmod_iopad initialisation which takes static structures for all pads.</div><div><br></div><div>--</div><div>

Govindraj.R </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote">
<div><br></div><div>...</div><div><br></div><div>Todd</div><div> </div></div>
</blockquote></div><br>