Subject: [PATCH 3/9] SPEAr13xx: Add common clock framework support

From: [email-masked] (Viresh Kumar)
Date: Tue, 24 Apr 2012 09:27:45 +0530

On 4/24/2012 3:13 AM, Turquette, Mike wrote:
>> > +void __init spear1310_clk_init(void)
>> > +{
>> > +       struct clk *clk, *clk1;
>> > +
>> > +       clk = clk_register_fixed_rate(NULL, "apb_pclk", NULL, CLK_IS_ROOT, 0);
>> > +       clk_register_clkdev(clk, "apb_pclk", NULL);
> There is no exception handling at all in this function.  Do you just
> expect to fail at some later point?

This is what Russell suggested in his clkdev patch. We don't have to check clk *
returned from clk_register_*(). This will be done in clk_register_clkdev(), clk
lookup wouldn't be registered for failing clocks.

-- 
viresh