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
