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@xxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html