lkml.org 
[lkml]   [2013]   [Aug]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH RFC v2 11/16] ARM: socfpga: remove call to of_clk_init
    Date
    On Thursday 29 August 2013, Dinh Nguyen wrote:
    > On Tue, 2013-08-27 at 23:28 +0200, Sebastian Hesselbarth wrote:

    > > @@ -107,7 +106,6 @@ static void __init socfpga_cyclone5_init(void)
    > > {
    > > l2x0_of_init(0, ~0UL);
    > > of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
    > > - of_clk_init(NULL);
    > > socfpga_init_clocks();
    > > }
    > >
    >
    > Acked-by: Dinh Nguyen <dinguyen@altera.com>

    I saw that your socfpga_init_clocks() function only has these contents:

    void __init socfpga_init_clocks(void)
    {
    struct clk *clk;
    int ret;

    clk = clk_register_fixed_factor(NULL, "smp_twd", "mpuclk", 0, 1, 4);
    ret = clk_register_clkdev(clk, NULL, "smp_twd");
    if (ret)
    pr_err("smp_twd alias not registered\n");
    }


    I think that can easily be expressed with DT syntax now (correct me if I'm wrong), so
    you should be able to kill this off as well. Once somebody gets around to automate
    the l2x0 setup, the entire socfpga_cyclone5_init() function can go away too.

    Arnd


    \
     
     \ /
      Last update: 2013-08-29 20:02    [W:4.176 / U:0.132 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site