On Fri, 09 Dec 2016, Benjamin Gaignard wrote: > This hardware block could at used at same time for PWM generation > and IIO timers. > PWM and IIO timer configuration are mixed in the same registers > so we need a multi fonction driver to be able to share those registers. > > version 6: > - rename files to stm32-timers > - rename functions to stm32_timers_xxx > > version 5: > - fix Lee comments about detect function > - add missing dependency on REGMAP_MMIO > > version 4: > - add a function to detect Auto Reload Register (ARR) size > - rename the structure shared with other drivers > > version 2: > - rename driver "stm32-gptimer" to be align with SoC documentation > - only keep one compatible > - use of_platform_populate() instead of devm_mfd_add_devices() > > Signed-off-by: Benjamin Gaignard <benjamin.gaignard@xxxxxx> > --- > drivers/mfd/Kconfig | 11 ++++++ > drivers/mfd/Makefile | 2 + > drivers/mfd/stm32-timers.c | 80 ++++++++++++++++++++++++++++++++++++++++ > include/linux/mfd/stm32-timers.h | 71 +++++++++++++++++++++++++++++++++++ > 4 files changed, 164 insertions(+) > create mode 100644 drivers/mfd/stm32-timers.c > create mode 100644 include/linux/mfd/stm32-timers.h For my own reference: Acked-for-MFD-by: Lee Jones <lee.jones@xxxxxxxxxx> > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index c6df644..4ec1906 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -1607,6 +1607,17 @@ config MFD_STW481X > in various ST Microelectronics and ST-Ericsson embedded > Nomadik series. > > +config MFD_STM32_TIMERS > + tristate "Support for STM32 Timers" > + depends on (ARCH_STM32 && OF) || COMPILE_TEST > + select MFD_CORE > + select REGMAP > + select REGMAP_MMIO > + help > + Select this option to enable STM32 timers driver used > + for PWM and IIO Timer. This driver allow to share the > + registers between the others drivers. > + > menu "Multimedia Capabilities Port drivers" > depends on ARCH_SA1100 > > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 9834e66..11a52f8 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -211,3 +211,5 @@ obj-$(CONFIG_INTEL_SOC_PMIC) += intel-soc-pmic.o > obj-$(CONFIG_MFD_MT6397) += mt6397-core.o > > obj-$(CONFIG_MFD_ALTERA_A10SR) += altera-a10sr.o > + > +obj-$(CONFIG_MFD_STM32_TIMERS) += stm32-timers.o > diff --git a/drivers/mfd/stm32-timers.c b/drivers/mfd/stm32-timers.c > new file mode 100644 > index 0000000..68d115e > --- /dev/null > +++ b/drivers/mfd/stm32-timers.c > @@ -0,0 +1,80 @@ > +/* > + * Copyright (C) STMicroelectronics 2016 > + * > + * Author: Benjamin Gaignard <benjamin.gaignard@xxxxxx> > + * > + * License terms: GNU General Public License (GPL), version 2 > + */ > + > +#include <linux/mfd/stm32-timers.h> > +#include <linux/module.h> > +#include <linux/of_platform.h> > +#include <linux/reset.h> > + > +static const struct regmap_config stm32_timers_regmap_cfg = { > + .reg_bits = 32, > + .val_bits = 32, > + .reg_stride = sizeof(u32), > + .max_register = 0x400, > +}; > + > +static void stm32_timers_get_arr_size(struct stm32_timers *ddata) > +{ > + /* > + * Only the available bits will be written so when readback > + * we get the maximum value of auto reload register > + */ > + regmap_write(ddata->regmap, TIM_ARR, ~0L); > + regmap_read(ddata->regmap, TIM_ARR, &ddata->max_arr); > + regmap_write(ddata->regmap, TIM_ARR, 0x0); > +} > + > +static int stm32_timers_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct stm32_timers *ddata; > + struct resource *res; > + void __iomem *mmio; > + > + ddata = devm_kzalloc(dev, sizeof(*ddata), GFP_KERNEL); > + if (!ddata) > + return -ENOMEM; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + mmio = devm_ioremap_resource(dev, res); > + if (IS_ERR(mmio)) > + return PTR_ERR(mmio); > + > + ddata->regmap = devm_regmap_init_mmio_clk(dev, "clk_int", mmio, > + &stm32_timers_regmap_cfg); > + if (IS_ERR(ddata->regmap)) > + return PTR_ERR(ddata->regmap); > + > + ddata->clk = devm_clk_get(dev, NULL); > + if (IS_ERR(ddata->clk)) > + return PTR_ERR(ddata->clk); > + > + stm32_timers_get_arr_size(ddata); > + > + platform_set_drvdata(pdev, ddata); > + > + return of_platform_populate(pdev->dev.of_node, NULL, NULL, &pdev->dev); > +} > + > +static const struct of_device_id stm32_timers_of_match[] = { > + { .compatible = "st,stm32-timers", }, > + { /* end node */ }, > +}; > +MODULE_DEVICE_TABLE(of, stm32_timers_of_match); > + > +static struct platform_driver stm32_timers_driver = { > + .probe = stm32_timers_probe, > + .driver = { > + .name = "stm32-timers", > + .of_match_table = stm32_timers_of_match, > + }, > +}; > +module_platform_driver(stm32_timers_driver); > + > +MODULE_DESCRIPTION("STMicroelectronics STM32 Timers"); > +MODULE_LICENSE("GPL v2"); > diff --git a/include/linux/mfd/stm32-timers.h b/include/linux/mfd/stm32-timers.h > new file mode 100644 > index 0000000..d030004 > --- /dev/null > +++ b/include/linux/mfd/stm32-timers.h > @@ -0,0 +1,71 @@ > +/* > + * Copyright (C) STMicroelectronics 2016 > + * > + * Author: Benjamin Gaignard <benjamin.gaignard@xxxxxx> > + * > + * License terms: GNU General Public License (GPL), version 2 > + */ > + > +#ifndef _LINUX_STM32_GPTIMER_H_ > +#define _LINUX_STM32_GPTIMER_H_ > + > +#include <linux/clk.h> > +#include <linux/regmap.h> > + > +#define TIM_CR1 0x00 /* Control Register 1 */ > +#define TIM_CR2 0x04 /* Control Register 2 */ > +#define TIM_SMCR 0x08 /* Slave mode control reg */ > +#define TIM_DIER 0x0C /* DMA/interrupt register */ > +#define TIM_SR 0x10 /* Status register */ > +#define TIM_EGR 0x14 /* Event Generation Reg */ > +#define TIM_CCMR1 0x18 /* Capt/Comp 1 Mode Reg */ > +#define TIM_CCMR2 0x1C /* Capt/Comp 2 Mode Reg */ > +#define TIM_CCER 0x20 /* Capt/Comp Enable Reg */ > +#define TIM_PSC 0x28 /* Prescaler */ > +#define TIM_ARR 0x2c /* Auto-Reload Register */ > +#define TIM_CCR1 0x34 /* Capt/Comp Register 1 */ > +#define TIM_CCR2 0x38 /* Capt/Comp Register 2 */ > +#define TIM_CCR3 0x3C /* Capt/Comp Register 3 */ > +#define TIM_CCR4 0x40 /* Capt/Comp Register 4 */ > +#define TIM_BDTR 0x44 /* Break and Dead-Time Reg */ > + > +#define TIM_CR1_CEN BIT(0) /* Counter Enable */ > +#define TIM_CR1_ARPE BIT(7) /* Auto-reload Preload Ena */ > +#define TIM_CR2_MMS (BIT(4) | BIT(5) | BIT(6)) /* Master mode selection */ > +#define TIM_SMCR_SMS (BIT(0) | BIT(1) | BIT(2)) /* Slave mode selection */ > +#define TIM_SMCR_TS (BIT(4) | BIT(5) | BIT(6)) /* Trigger selection */ > +#define TIM_DIER_UIE BIT(0) /* Update interrupt */ > +#define TIM_SR_UIF BIT(0) /* Update interrupt flag */ > +#define TIM_EGR_UG BIT(0) /* Update Generation */ > +#define TIM_CCMR_PE BIT(3) /* Channel Preload Enable */ > +#define TIM_CCMR_M1 (BIT(6) | BIT(5)) /* Channel PWM Mode 1 */ > +#define TIM_CCER_CC1E BIT(0) /* Capt/Comp 1 out Ena */ > +#define TIM_CCER_CC1P BIT(1) /* Capt/Comp 1 Polarity */ > +#define TIM_CCER_CC1NE BIT(2) /* Capt/Comp 1N out Ena */ > +#define TIM_CCER_CC1NP BIT(3) /* Capt/Comp 1N Polarity */ > +#define TIM_CCER_CC2E BIT(4) /* Capt/Comp 2 out Ena */ > +#define TIM_CCER_CC3E BIT(8) /* Capt/Comp 3 out Ena */ > +#define TIM_CCER_CC4E BIT(12) /* Capt/Comp 4 out Ena */ > +#define TIM_CCER_CCXE (BIT(0) | BIT(4) | BIT(8) | BIT(12)) > +#define TIM_BDTR_BKE BIT(12) /* Break input enable */ > +#define TIM_BDTR_BKP BIT(13) /* Break input polarity */ > +#define TIM_BDTR_AOE BIT(14) /* Automatic Output Enable */ > +#define TIM_BDTR_MOE BIT(15) /* Main Output Enable */ > +#define TIM_BDTR_BKF (BIT(16) | BIT(17) | BIT(18) | BIT(19)) > +#define TIM_BDTR_BK2F (BIT(20) | BIT(21) | BIT(22) | BIT(23)) > +#define TIM_BDTR_BK2E BIT(24) /* Break 2 input enable */ > +#define TIM_BDTR_BK2P BIT(25) /* Break 2 input polarity */ > + > +#define MAX_TIM_PSC 0xFFFF > +#define TIM_CR2_MMS_SHIFT 4 > +#define TIM_SMCR_TS_SHIFT 4 > +#define TIM_BDTR_BKF_MASK 0xF > +#define TIM_BDTR_BKF_SHIFT 16 > +#define TIM_BDTR_BK2F_SHIFT 20 > + > +struct stm32_timers { > + struct clk *clk; > + struct regmap *regmap; > + u32 max_arr; > +}; > +#endif -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html