[PATCH 1/2] ARM: CLKDEV: Add Common Macro for clk_lookup

padma venkat padma.kvr at gmail.com
Tue Oct 4 01:20:30 EDT 2011


Thank you all for your comments.
Will resend the patches after removing "&" from the clk parameter and
trailing "," at the end of Macro.

Thanks and Regards,
Padmavathi Venna

On Sat, Oct 1, 2011 at 8:59 PM, Russell King - ARM Linux
<linux at arm.linux.org.uk> wrote:
> On Fri, Sep 30, 2011 at 04:41:36PM +0530, Padmavathi Venna wrote:
>> +#define CLKDEV_INIT(d, n, c) \
>> +     { \
>> +             .dev_id = d, \
>> +             .con_id = n, \
>> +             .clk = &c, \
>> +     },
>> +
>
> As Heartley says, the trailing , and the end should not be there - it
> should be with each use of CLKDEV_INIT().  The reasoning is that it
> makes the use of macros much more conventional and consistent throughout
> the kernel:
>
> struct cl_lookup foo_lookups[] = {
>        CLKDEV_INIT(...),
>        CLKDEV_INIT(...),
> };
>
> struct cl_lookup bar_lookup = CLKDEV_INIT(...);
>
> Could you re-spin with that change please?
>
> Thanks.
>



More information about the linux-arm-kernel mailing list