| Date | Fri, 5 Apr 2019 22:44:31 +0200 (CEST) | From | Thomas Gleixner <> | Subject | Re: [PATCH v11 04/27] clocksource: Add a new timer-ingenic driver |
| |
On Thu, 28 Mar 2019, Paul Cercueil wrote:
> This driver handles the TCU (Timer Counter Unit) present on the Ingenic > JZ47xx SoCs, and provides the kernel with a system timer, and optionally > with a clocksource and a sched_clock. > > It also provides clocks and interrupt handling to client drivers.
Why on earth has a timer driver clock and interrupt chip drivers in it?
We have different subsystems for good reasons. Please split this apart and add the support to the appropriate subsystems, clocksource, irqchip, clk
Thanks,
tglx
|