Re: [PATCH 1/3] time/sched_clock: Export sched_clock_register()

[Daniel Lezcano]

On 08/02/2023 20:45, Krzysztof Kozlowski wrote:
> On 08/02/2023 20:41, Matthias Brugger wrote:
> > On 08/02/2023 15:24, Krzysztof Kozlowski wrote:
> > > On 08/02/2023 10:48, walter.chang@xxxxxxxxxxxx wrote:
> > > > From: Chun-Hung Wu <chun-hung.wu@xxxxxxxxxxxx>
> > > >
> > > > clocksource driver may use sched_clock_register()
> > > > to resigter itself as a sched_clock source.
> > > > Export it to support building such driver
> > > > as module, like timer-mediatek.c
> > > >
> > > > Signed-off-by: Chun-Hung Wu <chun-hung.wu@xxxxxxxxxxxx>
> > > > ---

[ ... ]

> ... and actually test if the system works fine when booted from such
> clocksource as a module. I have doubts that and unfortunately folks
> working on GKI like to put whatever stuff from mainline into modules
> even if it does not make sense for us (see long time ago discussion
> about pinctrl drivers).

+1

It is not the first time there is a proposal to convert the timers to 
modules. After asking, nobody came with a real study regarding the 
impact of the modularization of these drivers vs the time core framework 
and the benefit.

My gut feeling is that is not that simple.


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog