On Tue, Aug 21, 2018 at 07:16:23PM +0200, Paul Cercueil wrote: > The ingenic-timer "TCU" driver provides us with a regmap, that we can > use to safely access the TCU registers. > > It also provides us with clocks, that can be (un)gated, reparented or > reclocked from devicetree, instead of having these settings hardcoded in > this driver. > > While this driver is devicetree-compatible, it is never (as of now) > probed from devicetree, so this change does not introduce a ABI problem > with current devicetree files. > > Signed-off-by: Paul Cercueil > --- > > Notes: > v5: New patch > > v6: Drop requirement of probing from devicetree > > v7: No change > > drivers/pwm/Kconfig | 2 + > drivers/pwm/pwm-jz4740.c | 124 +++++++++++++++++++++++++++++++---------------- > 2 files changed, 83 insertions(+), 43 deletions(-) > > diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig > index a4d262db9945..58359bf22b96 100644 > --- a/drivers/pwm/Kconfig > +++ b/drivers/pwm/Kconfig > @@ -202,6 +202,8 @@ config PWM_IMX > config PWM_JZ4740 > tristate "Ingenic JZ47xx PWM support" > depends on MACH_INGENIC > + depends on COMMON_CLK > + select INGENIC_TIMER > help > Generic PWM framework driver for Ingenic JZ47xx based > machines. > diff --git a/drivers/pwm/pwm-jz4740.c b/drivers/pwm/pwm-jz4740.c > index a7b134af5e04..1bda8d8e9865 100644 > --- a/drivers/pwm/pwm-jz4740.c > +++ b/drivers/pwm/pwm-jz4740.c > @@ -14,21 +14,21 @@ > */ > > #include > +#include Why do you need this? Thierry