Messages in this thread | | | From | Noam Camus <> | Subject | Re: [PATCH v3 2/3] clocksource: Add NPS400 timers driver | Date | Tue, 9 Feb 2016 21:47:03 +0000 |
| |
>From: Daniel Lezcano <daniel.lezcano@linaro.org> >Sent: Tuesday, February 9, 2016 3:38 PM
>Actually I was referring to clk_prepare_enable, clocksource_register_hz. >Agree clk_get_rate is always valid. Thanks for making this clear. Any way as you can see I do call pr_err() in case of error just like most drivers around. By "hang" do you mean calling panic()? What if there is another clocksource in DT (even with worse rating)? I still prefer using it then having non workable machine.
>> >>> You can simplify the driver even more by using >>> clocksource_mmio_init. >> Since my base address depends on cluster number, which CPU is part >> of, this interface is not much of a use. On top of that it assumes >> that I am little endian by using readl family accessories.
>Why can't you use ?
>clocksource_mmio_init(nps_msu_reg_low_addr, "EZnps-tick", >nps_timer_rate, 32, nps_clksrc_read);
I believe that the simplification is meant for drivers that can actually use the clocksource_mmio..() accessories. Could you explain what is the advantage here, for my case, to use clocksource_mmio driver?
Thanks for your patience -Noam
| |