[PATCH v2 3/3] ARM: omap: clk: Remove all direct dereferencing of struct clk
Rajendra Nayak
rnayak at ti.com
Mon Jul 2 05:56:43 EDT 2012
On Saturday 30 June 2012 01:53 AM, Paul Walmsley wrote:
> Hi
>
> On Wed, 27 Jun 2012, Rajendra Nayak wrote:
>
>> While we move to Common Clk Framework (CCF), direct deferencing of struct
>> clk wouldn't be possible anymore. Hence get rid of all such instances
>> in the current clock code and use macros/helpers similar to the ones that
>> are provided by CCF.
>>
>> Signed-off-by: Rajendra Nayak<rnayak at ti.com>
>
> This patch generates checkpatch warnings. Here's a sample:
>
> WARNING: quoted string split across lines
> #479: FILE: arch/arm/mach-omap2/clock.c:109:
> pr_debug("clock: could not associate clk %s to "
> + "clkdm %s\n", clk_name, clk->clkdm_name);
>
> CHECK: Alignment should match open parenthesis
> #594: FILE: arch/arm/mach-omap2/dpll3xxx.c:455:
> + if (__clk_get_rate(dd->clk_bypass) == rate&&
> (clk->dpll_data->modes& (1<< DPLL_LOW_POWER_BYPASS))) {
>
> ERROR: Macros with complex values should be enclosed in parenthesis
> #706: FILE: arch/arm/plat-omap/include/plat/clock.h:22:
> +#define __clk_get_name(clk) clk->name
>
> In the case of the quoted string warnings, please go ahead and
> concatenate those strings while you are there. That needs to be done
> anyway.
Ok, will do. I was able to fix all but one hard to fix CHECK for this
patch..
---
CHECK: Alignment should match open parenthesis
#608: FILE: arch/arm/mach-omap2/dpll3xxx.c:455:
+ if (__clk_get_rate(dd->clk_bypass) == rate &&
(clk->dpll_data->modes & (1 << DPLL_LOW_POWER_BYPASS))) {
total: 0 errors, 0 warnings, 1 checks, 608 lines checked
0003-ARM-omap-clk-Remove-all-direct-dereferencing-of-stru.patch has
style problems, please review.
If any of these errors are false positives, please report
---
I could not find anything wrong with the alignment, but checkpatch
keeps complaining. It complains for the original code too.
Might be a checkpatch bug? Or do you really see anything wrong in
the alignment?
>
>
> - Paul
More information about the linux-arm-kernel
mailing list