Messages in this thread | | | Subject | Re: [PATCH 2/3] cpu: expose pm_qos_resume_latency for each cpu | From | Alex Shi <> | Date | Thu, 12 Jan 2017 21:04:49 +0800 |
| |
On 01/12/2017 02:40 AM, Rafael J. Wysocki wrote: >> > >> > Hi Greg, >> > >> > Thanks for comments! >> > >> > The function dev_pm_qos_expose_latency_limit() is null if no CONFIG_PM. >> > So when CONFIG_PM enabled, may we could consider the cpu idle is also >> > wanted. In this assumption the #ifdef could be removed. If user want to >> > use this feature, she/he should understand the feature only work on menu >> > gov only currently. So consider this, I guess we could remove this >> > #ifdef. :) > But instead of putting the #ifdef into the function body, you can use > a wrapper function defined to be empty for CONFIG_PM unset.
Thanks Rafael!
The function dev_pm_qos_expose_latency_limit() is empty now when CONFIG_PM disabled. :) I will resend the patch without the #ifdef.
Thanks! Alex
| |