On 6/26/07, Atsushi Nemoto <anemo@xxxxxxxxxxxxx> wrote:
I just think having centralized clock_list[] array might cause
maintainance issue. Calling clk_register() in your platform's
arch_initcall (or so) seems enough for me.
It seems that clock configuration highly depends on the board, not the
arch. For example, a board can have only static clocks which won't be
unregistered later. In that case most of code provided by the patch is
useless.
So maybe the best thing is to let each board implements their own
generic clock API (exactly like the generic GPIO API you did
recently). Or make a default implementation (your patch) and allow
others boards to make their own implementation.
What do you think ?
--
Franck
