[PATCH v2 3/3] ARM: omap: clk: Remove all direct dereferencing of struct clk

Paul Walmsley paul at pwsan.com
Fri Jun 29 16:23:34 EDT 2012


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.


- Paul



More information about the linux-arm-kernel mailing list