On Tue, 25 Jun 2019, Paul Cercueil wrote: > + > +struct ingenic_soc_info { > + unsigned int num_channels; > +}; > + > +struct ingenic_tcu { > + struct regmap *map; > + struct clk *timer_clk, *cs_clk; > + > + unsigned int timer_channel, cs_channel; > + struct clock_event_device cevt; > + struct clocksource cs; > + char name[4]; > + > + unsigned long pwm_channels_mask; > +}; As in the irq driver. Aside of that: Reviewed-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>