Re: [RFC PATCH 01/15] PM / devfreq: exynos: Add generic exynos bus frequency driver

[Anand Moon <linux.amoon@gmail.com>]

Hi Chanwoo,

On 9 December 2015 at 09:41, Chanwoo Choi <cw00.choi@samsung.com> wrote:
> Hi Anand,
>
> On 2015년 11월 27일 09:34, Chanwoo Choi wrote:
>> Hi Anand,
>>
>> On 2015년 11월 27일 02:17, Anand Moon wrote:
>>> Hi Chanwoo,
>>>
>>> On 26 November 2015 at 21:42, Chanwoo Choi <cwchoi00@gmail.com> wrote:
>>>> On Thu, Nov 26, 2015 at 11:00 PM, MyungJoo Ham <myungjoo.ham@samsung.com> wrote:
>>>>> On Thu, Nov 26, 2015 at 10:47 PM, Chanwoo Choi <cw00.choi@samsung.com> wrote:
>>>>>> This patch adds the generic exynos bus frequency driver for AMBA AXI bus
>>>>>> of sub-blocks in exynos SoC with DEVFREQ framework. The Samsung Exynos SoC
>>>>>> have the common architecture for bus between DRAM and sub-blocks in SoC.
>>>>>> This driver can support the generic bus frequency driver for Exynos SoCs.
>>>>>>
>>>>>> In devicetree, Each bus block has a bus clock, regulator, operation-point
>>>>>> and devfreq-event devices which measure the utilization of each bus block.
>>>>>>
>>>>>> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
>>>>>> ---
>>>>>>  drivers/devfreq/Kconfig             |  15 ++
>>>>>>  drivers/devfreq/Makefile            |   1 +
>>>>>>  drivers/devfreq/exynos/Makefile     |   1 +
>>>>>>  drivers/devfreq/exynos/exynos-bus.c | 443 ++++++++++++++++++++++++++++++++++++
>>>>>>  4 files changed, 460 insertions(+)
>>>>>>  create mode 100644 drivers/devfreq/exynos/exynos-bus.c
>>>>>>
>>>>>
>>>>> Are we finally getting a common Exynos bus driver with full DT support?
>>>>> (can this replace both Exynos4/5 drivers and support Exynos7 series?)
>>>>
>>>> Yes.
>>>> This patch-set would support all Exynos SoCs for bus frequency driver.
>>>> To make sure the support for Exynos7 series, I need to check the TRM
>>>> document of Exynos7.  I think it is possible for support Exynos7.
>>>>
>>>> I'm going to test this driver on various Exynos-based board.
>>>>
>>>> Regards,
>>>> Chanwoo Choi
>>>
>>> Please do consider Exynos 542x series as well.
>>
>> Sure. I'll to test it on Exynos5422-based Odroid-XU3.
>
> I send the v2 patchset but this patchset don't include
> the support of Odroid-XU3 because of only Exynos542x has the
> special addtional sequence to change the source clock
> of DRAM. So, I'm going to support the bus frequency on Exynos542x.
> After completing it, I'll send the separate patches.
>
> Thanks,
> Chanwoo Choi
>

Not an issue. Thanks for the update.

-Anand Moon