Re: [PATCH v2 0/8] PM / devfreq: Update the devfreq and devfreq-event device

[Chanwoo Choi <cwchoi00@gmail.com>]

Dear Myungjoo,

Thanks for your review for patch1/2/3/4/5.

But, patch6/7/8 are not yet reviewed. The patch4 should be resent
according to your comment.
So, Could you please review these patch6/7/8?

2016-12-28 20:56 GMT+09:00 Chanwoo Choi <cw00.choi@samsung.com>:
> This patches update the devfreq and devfreq-event device. I add the summary
> of each patch as following.
>
> - Patch1 is documented for bus frequency scaling of Exynos5433's VDD_INT.
>   The related-to patches[1] were merged by Exynos SoC maintainer.
> - Patch2 uses the regmap interface instead of raw_read/write function.
> - Patch3/4 print the useful log for exynos-bus.c and exynos-ppmu.c driver.
> - Patch5 fixes the checkpatch warning of devfreq.c.
> - Patch6/7 modify the name of sysfs entry for devfreq/devfreq-event device
> - Patch8 fixes the wrong description of govenro_userspace.c.
>
> Changes from v1:
> - Rebase these patches on v4.10-rc1.
> - Include the separate patch[2] in these patches.
>
> [1] https://lkml.org/lkml/2016/12/7/846
> - [PATCH v2 0/5] arm64: dts: Enable bus frequency scaling on Exynos5433-based TM2 board
> [2] https://lkml.org/lkml/2016/12/16/7
> - [PATCH] PM / devfreq: Fix the wrong description for userspace governor
>
>
> Depends on:
> - These patches depend on both v4.10-rc1 and patches[3].
>
> [3] https://lkml.org/lkml/2016/12/28/91
> - [PATCH v2 0/3] PM / devfreq: Fix the bug and add reviewer for devfreq support
>
> Chanwoo Choi (8):
>   PM / devfreq: exynos-bus: Add the detailed correlation for Exynos5433
>   PM / devfreq: exynos-ppmu: Use the regmap interface to handle the registers
>   PM / devfreq: exynos-bus: Print the real clock rate of bus
>   PM / devfreq: exynos-ppmu: Show the registred device for ppmu device
>   PM / devfreq: Fix the checkpatch warnings
>   PM / devfreq: Modify the device name as devfreq[X] for sysfs
>   PM / devfreq: Simplify the sysfs name of devfreq-event device
>   PM / devfreq: Fix the wrong description for userspace governor
>
>  .../devicetree/bindings/devfreq/exynos-bus.txt     |  14 +
>  drivers/devfreq/devfreq-event.c                    |   2 +-
>  drivers/devfreq/devfreq.c                          |  17 +-
>  drivers/devfreq/event/exynos-ppmu.c                | 331 +++++++++++++++------
>  drivers/devfreq/exynos-bus.c                       |   8 +-
>  drivers/devfreq/governor_userspace.c               |   3 +-
>  6 files changed, 271 insertions(+), 104 deletions(-)
>
> --
> 1.9.1
>



-- 
Best Regards,
Chanwoo Choi