Re: [PATCH v4 0/3] devfreq: Add support for devices which can idle

[MyungJoo Ham <myungjoo.ham@samsung.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=euc-kr, Size: 2351 bytes --]

> This patchset updates devfreq core to add support for devices
> which can idle. When device idleness is detected perhaps
> through runtime-pm, need some mechanism to suspend devfreq
> load monitoring and resume when device is back online.
> 
> patch 1 introduce core design changes - per device work, decouple
> delayed work from core and event based interaction.
> patch 2 add devfreq suspend and resume apis.
> patch 3 add new sysfs attribute for governor predicted next target
> frequency and callback for current device frequency.
> 
> The existing devfreq apis are kept intact. Two new apis
> devfreq_suspend_device() and devfreq_resume_device() are
> added to support suspend/resume of device devfreq.

Could you please tell me which version you have rebased?

It seems that the patchset has issues on applying over 3.6.
(tried at a0d271cbfed1dd50278c6b06bead3d00ba0a88f9)


> 
> Changes since v1:
> - revised locking mechanism
> - added kerneldoc comments for load monitoring helper functions
> - fixed minor review comments
> 
> Changes since v2:
> - added new helper function for polling interval update
> - handled work suspend/resume contention between devfreq driver
> and sysfs
> 
> Changes since v3:
> - added additonal checks in suspend/resume to avoid invalid usage of apis
> - added check in devfreq_monitor_start, not to start monitoring when
> polling_ms is set to zero.
> 
> --
> Rajagopal Venkat (3):
>   devfreq: Core updates to support devices which can idle
>   devfreq: Add suspend and resume apis
>   devfreq: Add current freq callback in device profile
> 
>  Documentation/ABI/testing/sysfs-class-devfreq |  15 +-
>  drivers/devfreq/devfreq.c                     | 481 ++++++++++++--------------
>  drivers/devfreq/governor.h                    |  13 +
>  drivers/devfreq/governor_performance.c        |  16 +-
>  drivers/devfreq/governor_powersave.c          |  16 +-
>  drivers/devfreq/governor_simpleondemand.c     |  33 ++
>  drivers/devfreq/governor_userspace.c          |  23 +-
>  include/linux/devfreq.h                       |  49 +--
>  8 files changed, 353 insertions(+), 293 deletions(-)
> 
> -- 
> 1.7.11.3
> 
> 
> 
> 
>        
>   
>          
> 
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥