On Wed, Sep 22, 2010 at 03:43:14PM -0500, Cousson, Benoit wrote:
Hi Hema,
On 9/23/2010 2:30 AM, Kalliguddi, Hema wrote:
OMAP USBOTG modules has a requirement to set the auto idle bit only after
setting smart idle bit. Modified the _sys_enable api to set the smart idle
first and then the autoidle bit. Setting this will not have any impact on the
other modules.
I think you should change the subject, because it does not reflect
accurately the patch.
Just a little bit of nitpicking...
For me, an API change is a change in the signature of the API, not a
change inside an API. Otherwise, since 99% of the code is inside a
function, we can call most patches like that...
Moreover I don't think _sysc_enable can be considered as an API since it
is a pure static helper function not exported to the outside.
Something like that seems more accurate for my point of view:
OMAP: hwmod: Set autoidle after smartidle during _sysc_enable
+1
--
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
