Re: [PATCHv2 0/6] Idle status patches revisited

2010-01-12 Thread Kevin Hilman
Tero Kristo tero.kri...@nokia.com writes:

 From: Tero Kristo tero.kri...@nokia.com

 Improvements compared to previous set:

 - Fixed wrong pwrdm state check on IO chain arming on patch 2
 - Improved changelog on patch 2  4
 - Moved FCLK checks inside clockdomain code from powerdomain code in patch 5
 - Some style changes on patch 6

 Still, I believe there is no simple way of implementing the idle status
 check by using the usecounts inside clock framework. It would probably be
 possible by adding a new usecount variable that would be updated according
 to slightly different rules, but this also looks bit complicated compared
 to the need.

Tero,

Thanks for this series and the descriptive changelogs and comments.
Very nice.

Along with the comments on individual patches, this should probably
also be rebased on current PM branch.

Also, Paul has lots of other clockdomain/powerdomain changes queued up
for 2.6.34, so it might tbe helpful to Paul if those were based on his
upcoming changes.

Thanks,

Kevin

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCHv2 0/6] Idle status patches revisited

2009-12-04 Thread Tero Kristo
From: Tero Kristo tero.kri...@nokia.com

Improvements compared to previous set:

- Fixed wrong pwrdm state check on IO chain arming on patch 2
- Improved changelog on patch 2  4
- Moved FCLK checks inside clockdomain code from powerdomain code in patch 5
- Some style changes on patch 6

Still, I believe there is no simple way of implementing the idle status
check by using the usecounts inside clock framework. It would probably be
possible by adding a new usecount variable that would be updated according
to slightly different rules, but this also looks bit complicated compared
to the need.

-Tero


--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html