[PATCH 2/2] ARM: Switch to using CLKDEV_INIT macro for creating clk_lookup instance
H Hartley Sweeten
hartleys at visionengravers.com
Fri Sep 30 13:08:58 EDT 2011
On Friday, September 30, 2011 5:35 AM, Ryan Mallon wrote:
> On 30/09/11 21:11, Padmavathi Venna wrote:
>> Replace platform specific macros that creates struct clk_lookup
>> instance with the new common CLKDEV_INIT macro
>>
>> Suggested by: Russell King <rmk+kernel at arm.linux.org.uk>
>> Signed-off-by: Padmavathi Venna <padma.v at samsung.com>
>> ---
[...]
>> static struct clk_lookup cdce_clks[] = {
>> - CLK(NULL, "xin", &cdce_clk_in),
>> - CLK(NULL, NULL, NULL),
>> + CLKDEV_INIT(NULL, "xin", cdce_clk_in)
>> + {.dev_id = NULL, .con_id = NULL, .clk = NULL,},
>
> Why not CLKDEV_INIT(NULL, NULL, NULL)? Or create another macro
> CLKDEV_NULL (or similar) for the last entry in the list rather than
> duplicating this everywhere.
The CLKDEV_INIT macro does this with the third parameter:
.clk = &c,
I'm not sure &NULL is valid.
Is the sentinel entry actually needed? ep93xx does not have one.
Regards,
Hartley
More information about the linux-arm-kernel
mailing list