Re: [PATCH v6 0/2] Add initial support for RPM clocks

From: Georgi Djakov
Date: Mon Feb 22 2016 - 10:29:50 EST


On 02/12/2016 02:37 AM, Stephen Boyd wrote:
> On 12/15, Georgi Djakov wrote:
>>
>> .../devicetree/bindings/clock/qcom,rpmcc.txt | 37 ++
>> drivers/clk/qcom/Kconfig | 29 ++
>> drivers/clk/qcom/Makefile | 2 +
>> drivers/clk/qcom/clk-rpm.c | 290 +++++++++++++
>> drivers/clk/qcom/clk-rpm.h | 71 ++++
>
> Do we need this header file?
>
>> drivers/clk/qcom/clk-smd-rpm.c | 433 ++++++++++++++++++++
>> drivers/clk/qcom/clk-smd-rpm.h | 142 +++++++
>
> And this one too?
>
> From what I can tell they're only going to be included by the
> clk-rpm and clk-smd-rpm C files. We can also avoid exporting the
> RPM ops then if we're going to make the C files into full blown
> platform drivers.
>

Of course, good idea. Fixed it and will resubmit shortly.

Thanks,
Georgi