Re: [PATCH V6 0/4] clk: new APIs to handle all available clocks

Thor Thayer
2018-09-19T14:47:22Z

Hi,

On 09/16/2018 08:24 AM, A.s. Dong wrote:
> Ping again
>
>> -----Original Message-----
>> From: A.s. Dong
>> Sent: Thursday, September 6, 2018 11:23 AM
>> To: linux-clk@vger.kernel.org
>> Cc: linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
>> sboyd@kernel.org; mturquette@baylibre.com; shawnguo@kernel.org;
>> thor.thayer@linux.intel.com; dl-linux-imx <linux-imx@nxp.com>;
>> hdegoede@redhat.com
>> Subject: RE: [PATCH V6 0/4] clk: new APIs to handle all available clocks
>>
>> Gently ping...
>>
>>> -----Original Message-----
>>> From: A.s. Dong
>>> Sent: Friday, August 31, 2018 12:46 PM
>>> To: linux-clk@vger.kernel.org
>>> Cc: linux-kernel@vger.kernel.org;
>>> linux-arm-kernel@lists.infradead.org;
>>> sboyd@kernel.org; mturquette@baylibre.com; shawnguo@kernel.org;
>>> thor.thayer@linux.intel.com; dl-linux-imx <linux-imx@nxp.com>;
>>> hdegoede@redhat.com; A.s. Dong <aisheng.dong@nxp.com>
>>> Subject: [PATCH V6 0/4] clk: new APIs to handle all available clocks
>>>
>>> This patch series is a continue of discussion from here,
>>> https://patchwork.kernel.org/patch/9986293/
>>> that some users may want to handle all available clocks from device
>>> tree without need to know the detailed clock information likes clock
>>> numbers and names. This is useful in writing some generic drivers to handle
>> clock part.
>>>
>>> Note:
>>> This patch series is tested on MX6Q SDB cpufreq driver with a minor
>>> change to switch to use clk_bulk_get_all.
>>> But patch 4 only test compiling. Hopefully someone could help test the
>>> function.
>>>
>>> v3->v4:
>>> * improve 'devres->clks = *clks' according to Stephen's suggestion
>>> v2->v3:
>>> * address all comments from Stephen
>>> * fix build warnings on other architectures.
>>> v1->v2:
>>> * add clk_bulk_{get|put}_all() which only supports DT platform
>>> currently
>>> * remove _all variants and the wrapper struct clk_bulk
>>> * make of_clk_bulk_get and of_clk_bulk_get_all private until someone
>>> proves they need it because they don't have a struct device pointer.
>>>
>>> Dong Aisheng (4):
>>> clk: bulk: add of_clk_bulk_get()
>>> clk: add new APIs to operate on all available clocks
>>> clk: add managed version of clk_bulk_get_all
>>> video: simplefb: switch to use clk_bulk API to simplify clock
>>> operations
>>>
>>> drivers/clk/clk-bulk.c | 80
>>> ++++++++++++++++++++++++++++++++++++++++++
>>> drivers/clk/clk-devres.c | 24 +++++++++++++
>>> drivers/video/fbdev/simplefb.c | 72 ++++++++++---------------------------
>>> include/linux/clk.h | 65
>>> +++++++++++++++++++++++++++++++++-
>>> 4 files changed, 186 insertions(+), 55 deletions(-)
>>>
>>> --
>>> 2.7.4
>
Just checking on the status of this patch. The clock routines (patches
1-3) are useful for one of my drivers but if they aren't accepted or
will take a long time to be accepted, I'll need to refactor my driver.

Thanks,

Thor