| Message ID | 20181202220847.24364-3-martin.blumenstingl@googlemail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Amlogic Meson6/8/8b/8m2 SoC RTC driver | expand |
Hi Martin,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v4.20-rc5 next-20181204]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Martin-Blumenstingl/Amlogic-Meson6-8-8b-8m2-SoC-RTC-driver/20181204-002803
base: https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: sparc64-allyesconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=7.2.0 make.cross ARCH=sparc64
All errors (new ones prefixed by >>):
drivers/rtc/rtc-meson.c: In function 'meson_rtc_probe':
>> drivers/rtc/rtc-meson.c:288:11: error: 'SZ_16' undeclared (first use in this function); did you mean 'SF_L6'?
.size = SZ_16,
^~~~~
SF_L6
drivers/rtc/rtc-meson.c:288:11: note: each undeclared identifier is reported only once for each function it appears in
vim +288 drivers/rtc/rtc-meson.c
281
282 static int meson_rtc_probe(struct platform_device *pdev)
283 {
284 struct nvmem_config meson_rtc_nvmem_config = {
285 .name = "meson-rtc-regmem",
286 .word_size = 4,
287 .stride = 4,
> 288 .size = SZ_16,
289 .reg_read = meson_rtc_regmem_read,
290 .reg_write = meson_rtc_regmem_write,
291 };
292 struct device *dev = &pdev->dev;
293 struct meson_rtc *rtc;
294 struct resource *res;
295 void __iomem *base;
296 int ret;
297 u32 tm;
298
299 rtc = devm_kzalloc(dev, sizeof(struct meson_rtc), GFP_KERNEL);
300 if (!rtc)
301 return -ENOMEM;
302
303 rtc->rtc = devm_rtc_allocate_device(dev);
304 if (IS_ERR(rtc->rtc))
305 return PTR_ERR(rtc->rtc);
306
307 platform_set_drvdata(pdev, rtc);
308
309 rtc->dev = dev;
310
311 rtc->rtc->ops = &meson_rtc_ops;
312 rtc->rtc->range_max = U32_MAX;
313
314 res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
315 base = devm_ioremap_resource(dev, res);
316 if (IS_ERR(base))
317 return PTR_ERR(base);
318
319 rtc->peripheral = devm_regmap_init_mmio(dev, base,
320 &meson_rtc_peripheral_regmap_config);
321 if (IS_ERR(rtc->peripheral)) {
322 dev_err(dev, "failed to create peripheral regmap\n");
323 return PTR_ERR(rtc->peripheral);
324 }
325
326 rtc->reset = devm_reset_control_get(dev, NULL);
327 if (IS_ERR(rtc->reset)) {
328 dev_err(dev, "missing reset line\n");
329 return PTR_ERR(rtc->reset);
330 }
331
332 rtc->vdd = devm_regulator_get(dev, "vdd");
333 if (IS_ERR(rtc->vdd)) {
334 dev_err(dev, "failed to get the vdd-supply\n");
335 return PTR_ERR(rtc->vdd);
336 }
337
338 ret = regulator_enable(rtc->vdd);
339 if (ret) {
340 dev_err(dev, "failed to enable vdd-supply\n");
341 return ret;
342 }
343
344 ret = meson_rtc_write_static(rtc, MESON_STATIC_DEFAULT);
345 if (ret) {
346 dev_err(dev, "failed to set static values\n");
347 goto out_disable_vdd;
348 }
349
350 rtc->serial = devm_regmap_init(dev, &meson_rtc_serial_bus, rtc,
351 &meson_rtc_serial_regmap_config);
352 if (IS_ERR(rtc->serial)) {
353 dev_err(dev, "failed to create serial regmap\n");
354 ret = PTR_ERR(rtc->serial);
355 goto out_disable_vdd;
356 }
357
358 /*
359 * check if we can read RTC counter, if not then the RTC is probably
360 * not functional. If it isn't probably best to not bind.
361 */
362 ret = regmap_read(rtc->serial, RTC_COUNTER, &tm);
363 if (ret) {
364 dev_err(dev, "cannot read RTC counter, RTC not functional\n");
365 goto out_disable_vdd;
366 }
367
368 meson_rtc_nvmem_config.priv = rtc;
369 ret = rtc_nvmem_register(rtc->rtc, &meson_rtc_nvmem_config);
370 if (ret)
371 goto out_disable_vdd;
372
373 ret = rtc_register_device(rtc->rtc);
374 if (ret)
375 goto out_unregister_nvmem;
376
377 return 0;
378
379 out_unregister_nvmem:
380 rtc_nvmem_unregister(rtc->rtc);
381
382 out_disable_vdd:
383 regulator_disable(rtc->vdd);
384 return ret;
385 }
386
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
On 02/12/2018 22:08, Martin Blumenstingl wrote: > Add support for the RTC block on the 32-bit Amlogic Meson6, Meson8, > Meson8b and Meson8m2 SoCs. > > The RTC is split in to two parts, which are both managed by this driver: > - the AHB front end > - and a simple serial connection to the actual registers > > The RTC_COUNTER register which holds the time is 32-bits wide. > > There are four 32-bit wide (in total: 16 bytes) "regmem" registers which > are exposed using nvmem. On Amlogic's 3.10 kernel this is used to store > data which needs to survive a suspend / resume cycle. > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > [resurrected Ben's patches after 2 years] > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Just checking if the change of author is deliberate? not sure how to do >1 author on patches like this. > --- > drivers/rtc/Kconfig | 11 ++ > drivers/rtc/Makefile | 1 + > drivers/rtc/rtc-meson.c | 409 ++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 421 insertions(+) > create mode 100644 drivers/rtc/rtc-meson.c > > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig > index a819ef07b7ec..d5d0e3affdc6 100644 > --- a/drivers/rtc/Kconfig > +++ b/drivers/rtc/Kconfig > @@ -1285,6 +1285,17 @@ config RTC_DRV_IMXDI > This driver can also be built as a module, if so, the module > will be called "rtc-imxdi". > > +config RTC_DRV_MESON > + tristate "Amlogic Meson RTC" > + depends on (ARM && ARCH_MESON) || COMPILE_TEST > + select REGMAP_MMIO > + help > + Support for the RTC block on the Amlogic Meson6, Meson8, Meson8b > + and Meson8m2 SoCs. > + > + This driver can also be built as a module, if so, the module > + will be called "rtc-meson". > + > config RTC_DRV_OMAP > tristate "TI OMAP Real Time Clock" > depends on ARCH_OMAP || ARCH_DAVINCI || COMPILE_TEST > diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile > index 290c1730fb0a..3b088e75149d 100644 > --- a/drivers/rtc/Makefile > +++ b/drivers/rtc/Makefile > @@ -99,6 +99,7 @@ obj-$(CONFIG_RTC_DRV_MAX8997) += rtc-max8997.o > obj-$(CONFIG_RTC_DRV_MAX8998) += rtc-max8998.o > obj-$(CONFIG_RTC_DRV_MC13XXX) += rtc-mc13xxx.o > obj-$(CONFIG_RTC_DRV_MCP795) += rtc-mcp795.o > +obj-$(CONFIG_RTC_DRV_MESON) += rtc-meson.o > obj-$(CONFIG_RTC_DRV_MOXART) += rtc-moxart.o > obj-$(CONFIG_RTC_DRV_MPC5121) += rtc-mpc5121.o > obj-$(CONFIG_RTC_DRV_MSM6242) += rtc-msm6242.o > diff --git a/drivers/rtc/rtc-meson.c b/drivers/rtc/rtc-meson.c > new file mode 100644 > index 000000000000..09d6849b804a > --- /dev/null > +++ b/drivers/rtc/rtc-meson.c > @@ -0,0 +1,409 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * RTC driver for the interal RTC block in the Amlogic Meson6, Meson8, > + * Meson8b and Meson8m2 SoCs. > + * > + * The RTC is split in to two parts, the AHB front end and a simple serial > + * connection to the actual registers. This driver manages both parts. > + * > + * Copyright (c) 2018 Martin Blumenstingl <martin.blumenstingl@googlemail.com> > + * Copyright (c) 2015 Ben Dooks <ben.dooks@codethink.co.uk> for Codethink Ltd > + * Based on origin by Carlo Caione <carlo@endlessm.com> > + */ > + > +#include <linux/bitfield.h> > +#include <linux/delay.h> > +#include <linux/io.h> > +#include <linux/kernel.h> > +#include <linux/module.h> > +#include <linux/nvmem-provider.h> > +#include <linux/of.h> > +#include <linux/platform_device.h> > +#include <linux/regmap.h> > +#include <linux/regulator/consumer.h> > +#include <linux/reset.h> > +#include <linux/rtc.h> > + > +/* registers accessed from cpu bus */ > +#define RTC_ADDR0 0x00 > + #define RTC_ADDR0_LINE_SCLK BIT(0) > + #define RTC_ADDR0_LINE_SEN BIT(1) > + #define RTC_ADDR0_LINE_SDI BIT(2) > + #define RTC_ADDR0_START_SER BIT(17) > + #define RTC_ADDR0_WAIT_SER BIT(22) > + #define RTC_ADDR0_DATA GENMASK(31, 24) > + > +#define RTC_ADDR1 0x04 > + #define RTC_ADDR1_SDO BIT(0) > + #define RTC_ADDR1_S_READY BIT(1) > + > +#define RTC_ADDR2 0x08 > +#define RTC_ADDR3 0x0c > + > +#define RTC_REG4 0x10 > + #define RTC_REG4_STATIC_VALUE GENMASK(7, 0) > + > +/* rtc registers accessed via rtc-serial interface */ > +#define RTC_COUNTER (0) > +#define RTC_SEC_ADJ (2) > +#define RTC_REGMEM_0 (4) > +#define RTC_REGMEM_1 (5) > +#define RTC_REGMEM_2 (6) > +#define RTC_REGMEM_3 (7) > + > +#define RTC_ADDR_BITS (3) /* number of address bits to send */ > +#define RTC_DATA_BITS (32) /* number of data bits to tx/rx */ > + > +#define MESON_STATIC_BIAS_CUR (0x5 << 1) > +#define MESON_STATIC_VOLTAGE (0x3 << 11) > +#define MESON_STATIC_DEFAULT (MESON_STATIC_BIAS_CUR | MESON_STATIC_VOLTAGE) > + > +struct meson_rtc { > + struct rtc_device *rtc; /* rtc device we created */ > + struct device *dev; /* device we bound from */ > + struct reset_control *reset; /* reset source */ > + struct regulator *vdd; /* voltage input */ > + struct regmap *peripheral; /* peripheral registers */ > + struct regmap *serial; /* serial registers */ > +}; > + > +static const struct regmap_config meson_rtc_peripheral_regmap_config = { > + .name = "peripheral-registers", > + .reg_bits = 8, > + .val_bits = 32, > + .reg_stride = 4, > + .max_register = RTC_REG4, > + .fast_io = true, > +}; > + > +/* RTC front-end serialiser controls */ > + > +static void meson_rtc_sclk_pulse(struct meson_rtc *rtc) > +{ > + udelay(5); > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, 0); > + udelay(5); > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, > + RTC_ADDR0_LINE_SCLK); > +} > + > +static void meson_rtc_send_bit(struct meson_rtc *rtc, unsigned int bit) > +{ > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, > + bit ? RTC_ADDR0_LINE_SDI : 0); > + meson_rtc_sclk_pulse(rtc); > +} > + > +static void meson_rtc_send_bits(struct meson_rtc *rtc, u32 data, > + unsigned int nr) > +{ > + u32 bit = 1 << (nr - 1); > + > + while (bit) { > + meson_rtc_send_bit(rtc, data & bit); > + bit >>= 1; > + } > +} > + > +static void meson_rtc_set_dir(struct meson_rtc *rtc, u32 mode) > +{ > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, 0); > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); > + meson_rtc_send_bit(rtc, mode); > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); > +} > + > +static u32 meson_rtc_get_data(struct meson_rtc *rtc) > +{ > + u32 tmp, val = 0; > + int bit; > + > + for (bit = 0; bit < RTC_DATA_BITS; bit++) { > + meson_rtc_sclk_pulse(rtc); > + val <<= 1; > + > + regmap_read(rtc->peripheral, RTC_ADDR1, &tmp); > + val |= tmp & RTC_ADDR1_SDO; > + } > + > + return val; > +} > + > +static int meson_rtc_get_bus(struct meson_rtc *rtc) > +{ > + int ret, retries = 3; > + u32 val; > + > + /* prepare bus for transfers, set all lines low */ > + val = RTC_ADDR0_LINE_SDI | RTC_ADDR0_LINE_SEN | RTC_ADDR0_LINE_SCLK; > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, val, 0); > + > + for (retries = 0; retries < 3; retries++) { > + /* wait for the bus to be ready */ > + if (!regmap_read_poll_timeout(rtc->peripheral, RTC_ADDR1, val, > + val & RTC_ADDR1_S_READY, 10, > + 10000)) > + return 0; > + > + dev_warn(rtc->dev, "failed to get bus, resetting RTC\n"); > + > + ret = reset_control_reset(rtc->reset); > + if (ret) > + return ret; > + } > + > + dev_err(rtc->dev, "bus is not ready\n"); > + return -ETIMEDOUT; > +} > + > +static int meson_rtc_serial_bus_reg_read(void *context, unsigned int reg, > + unsigned int *data) > +{ > + struct meson_rtc *rtc = context; > + int ret; > + > + ret = meson_rtc_get_bus(rtc); > + if (ret) > + return ret; > + > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, > + RTC_ADDR0_LINE_SEN); > + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); > + meson_rtc_set_dir(rtc, 0); > + *data = meson_rtc_get_data(rtc); > + > + return 0; > +} > + > +static int meson_rtc_serial_bus_reg_write(void *context, unsigned int reg, > + unsigned int data) > +{ > + struct meson_rtc *rtc = context; > + int ret; > + > + ret = meson_rtc_get_bus(rtc); > + if (ret) > + return ret; > + > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, > + RTC_ADDR0_LINE_SEN); > + meson_rtc_send_bits(rtc, data, RTC_DATA_BITS); > + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); > + meson_rtc_set_dir(rtc, 1); > + > + return 0; > +} > + > +static const struct regmap_bus meson_rtc_serial_bus = { > + .reg_read = meson_rtc_serial_bus_reg_read, > + .reg_write = meson_rtc_serial_bus_reg_write, > +}; > + > +static const struct regmap_config meson_rtc_serial_regmap_config = { > + .name = "serial-registers", > + .reg_bits = 4, > + .reg_stride = 1, > + .val_bits = 32, > + .max_register = RTC_REGMEM_3, > + .fast_io = false, > +}; > + > +static int meson_rtc_write_static(struct meson_rtc *rtc, u32 data) > +{ > + u32 tmp; > + > + regmap_write(rtc->peripheral, RTC_REG4, > + FIELD_PREP(RTC_REG4_STATIC_VALUE, (data >> 8))); > + > + /* write the static value and start the auto serializer */ > + tmp = FIELD_PREP(RTC_ADDR0_DATA, (data & 0xff)) | RTC_ADDR0_START_SER; > + regmap_update_bits(rtc->peripheral, RTC_ADDR0, > + RTC_ADDR0_DATA | RTC_ADDR0_START_SER, tmp); > + > + /* wait for the auto serializer to complete */ > + return regmap_read_poll_timeout(rtc->peripheral, RTC_REG4, tmp, > + !(tmp & RTC_ADDR0_WAIT_SER), 10, > + 10000); > +} > + > +/* RTC interface layer functions */ > + > +static int meson_rtc_gettime(struct device *dev, struct rtc_time *tm) > +{ > + struct meson_rtc *rtc = dev_get_drvdata(dev); > + u32 time; > + int ret; > + > + ret = regmap_read(rtc->serial, RTC_COUNTER, &time); > + if (!ret) > + rtc_time64_to_tm(time, tm); > + > + return ret; > +} > + > +static int meson_rtc_settime(struct device *dev, struct rtc_time *tm) > +{ > + struct meson_rtc *rtc = dev_get_drvdata(dev); > + > + return regmap_write(rtc->serial, RTC_COUNTER, rtc_tm_to_time64(tm)); > +} > + > +static const struct rtc_class_ops meson_rtc_ops = { > + .read_time = meson_rtc_gettime, > + .set_time = meson_rtc_settime, > +}; > + > +/* NVMEM interface layer functions */ > + > +static int meson_rtc_regmem_read(void *context, unsigned int offset, > + void *buf, size_t bytes) > +{ > + struct meson_rtc *rtc = context; > + unsigned int read_offset, read_size; > + > + read_offset = RTC_REGMEM_0 + (offset / 4); > + read_size = bytes / 4; > + > + return regmap_bulk_read(rtc->serial, read_offset, buf, read_size); > +} > + > +static int meson_rtc_regmem_write(void *context, unsigned int offset, > + void *buf, size_t bytes) > +{ > + struct meson_rtc *rtc = context; > + unsigned int write_offset, write_size; > + > + write_offset = RTC_REGMEM_0 + (offset / 4); > + write_size = bytes / 4; > + > + return regmap_bulk_write(rtc->serial, write_offset, buf, write_size); > +} > + > +static int meson_rtc_probe(struct platform_device *pdev) > +{ > + struct nvmem_config meson_rtc_nvmem_config = { > + .name = "meson-rtc-regmem", > + .word_size = 4, > + .stride = 4, > + .size = SZ_16, > + .reg_read = meson_rtc_regmem_read, > + .reg_write = meson_rtc_regmem_write, > + }; > + struct device *dev = &pdev->dev; > + struct meson_rtc *rtc; > + struct resource *res; > + void __iomem *base; > + int ret; > + u32 tm; > + > + rtc = devm_kzalloc(dev, sizeof(struct meson_rtc), GFP_KERNEL); > + if (!rtc) > + return -ENOMEM; > + > + rtc->rtc = devm_rtc_allocate_device(dev); > + if (IS_ERR(rtc->rtc)) > + return PTR_ERR(rtc->rtc); > + > + platform_set_drvdata(pdev, rtc); > + > + rtc->dev = dev; > + > + rtc->rtc->ops = &meson_rtc_ops; > + rtc->rtc->range_max = U32_MAX; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > + base = devm_ioremap_resource(dev, res); > + if (IS_ERR(base)) > + return PTR_ERR(base); > + > + rtc->peripheral = devm_regmap_init_mmio(dev, base, > + &meson_rtc_peripheral_regmap_config); > + if (IS_ERR(rtc->peripheral)) { > + dev_err(dev, "failed to create peripheral regmap\n"); > + return PTR_ERR(rtc->peripheral); > + } > + > + rtc->reset = devm_reset_control_get(dev, NULL); > + if (IS_ERR(rtc->reset)) { > + dev_err(dev, "missing reset line\n"); > + return PTR_ERR(rtc->reset); > + } > + > + rtc->vdd = devm_regulator_get(dev, "vdd"); > + if (IS_ERR(rtc->vdd)) { > + dev_err(dev, "failed to get the vdd-supply\n"); > + return PTR_ERR(rtc->vdd); > + } > + > + ret = regulator_enable(rtc->vdd); > + if (ret) { > + dev_err(dev, "failed to enable vdd-supply\n"); > + return ret; > + } > + > + ret = meson_rtc_write_static(rtc, MESON_STATIC_DEFAULT); > + if (ret) { > + dev_err(dev, "failed to set static values\n"); > + goto out_disable_vdd; > + } > + > + rtc->serial = devm_regmap_init(dev, &meson_rtc_serial_bus, rtc, > + &meson_rtc_serial_regmap_config); > + if (IS_ERR(rtc->serial)) { > + dev_err(dev, "failed to create serial regmap\n"); > + ret = PTR_ERR(rtc->serial); > + goto out_disable_vdd; > + } > + > + /* > + * check if we can read RTC counter, if not then the RTC is probably > + * not functional. If it isn't probably best to not bind. > + */ > + ret = regmap_read(rtc->serial, RTC_COUNTER, &tm); > + if (ret) { > + dev_err(dev, "cannot read RTC counter, RTC not functional\n"); > + goto out_disable_vdd; > + } > + > + meson_rtc_nvmem_config.priv = rtc; > + ret = rtc_nvmem_register(rtc->rtc, &meson_rtc_nvmem_config); > + if (ret) > + goto out_disable_vdd; > + > + ret = rtc_register_device(rtc->rtc); > + if (ret) > + goto out_unregister_nvmem; > + > + return 0; > + > +out_unregister_nvmem: > + rtc_nvmem_unregister(rtc->rtc); > + > +out_disable_vdd: > + regulator_disable(rtc->vdd); > + return ret; > +} > + > +static const struct of_device_id meson_rtc_dt_match[] = { > + { .compatible = "amlogic,meson6-rtc", }, > + { .compatible = "amlogic,meson8-rtc", }, > + { .compatible = "amlogic,meson8b-rtc", }, > + { .compatible = "amlogic,meson8m2-rtc", }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, meson_rtc_dt_match); > + > +static struct platform_driver meson_rtc_driver = { > + .probe = meson_rtc_probe, > + .driver = { > + .name = "meson-rtc", > + .of_match_table = of_match_ptr(meson_rtc_dt_match), > + }, > +}; > +module_platform_driver(meson_rtc_driver); > + > +MODULE_DESCRIPTION("Amlogic Meson RTC Driver"); > +MODULE_AUTHOR("Ben Dooks <ben.doosk@codethink.co.uk>"); > +MODULE_AUTHOR("Martin Blumenstingl <martin.blumenstingl@googlemail.com>"); > +MODULE_LICENSE("GPL v2"); > +MODULE_ALIAS("platform:meson-rtc"); >
Hi Ben, On 05/12/2018 14:36, Ben Dooks wrote: > On 02/12/2018 22:08, Martin Blumenstingl wrote: >> Add support for the RTC block on the 32-bit Amlogic Meson6, Meson8, >> Meson8b and Meson8m2 SoCs. >> >> The RTC is split in to two parts, which are both managed by this driver: >> - the AHB front end >> - and a simple serial connection to the actual registers >> >> The RTC_COUNTER register which holds the time is 32-bits wide. >> >> There are four 32-bit wide (in total: 16 bytes) "regmem" registers which >> are exposed using nvmem. On Amlogic's 3.10 kernel this is used to store >> data which needs to survive a suspend / resume cycle. >> >> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> >> [resurrected Ben's patches after 2 years] >> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > Just checking if the change of author is deliberate? > not sure how to do >1 author on patches like this. Martin has been very explicit about that in the cover letter : https://patchwork.kernel.org/cover/10708427/ Neil > >> --- >> drivers/rtc/Kconfig | 11 ++ >> drivers/rtc/Makefile | 1 + >> drivers/rtc/rtc-meson.c | 409 ++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 421 insertions(+) >> create mode 100644 drivers/rtc/rtc-meson.c >> >> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig >> index a819ef07b7ec..d5d0e3affdc6 100644 >> --- a/drivers/rtc/Kconfig >> +++ b/drivers/rtc/Kconfig >> @@ -1285,6 +1285,17 @@ config RTC_DRV_IMXDI >> This driver can also be built as a module, if so, the module >> will be called "rtc-imxdi". >> +config RTC_DRV_MESON >> + tristate "Amlogic Meson RTC" >> + depends on (ARM && ARCH_MESON) || COMPILE_TEST >> + select REGMAP_MMIO >> + help >> + Support for the RTC block on the Amlogic Meson6, Meson8, Meson8b >> + and Meson8m2 SoCs. >> + >> + This driver can also be built as a module, if so, the module >> + will be called "rtc-meson". >> + >> config RTC_DRV_OMAP >> tristate "TI OMAP Real Time Clock" >> depends on ARCH_OMAP || ARCH_DAVINCI || COMPILE_TEST >> diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile >> index 290c1730fb0a..3b088e75149d 100644 >> --- a/drivers/rtc/Makefile >> +++ b/drivers/rtc/Makefile >> @@ -99,6 +99,7 @@ obj-$(CONFIG_RTC_DRV_MAX8997) += rtc-max8997.o >> obj-$(CONFIG_RTC_DRV_MAX8998) += rtc-max8998.o >> obj-$(CONFIG_RTC_DRV_MC13XXX) += rtc-mc13xxx.o >> obj-$(CONFIG_RTC_DRV_MCP795) += rtc-mcp795.o >> +obj-$(CONFIG_RTC_DRV_MESON) += rtc-meson.o >> obj-$(CONFIG_RTC_DRV_MOXART) += rtc-moxart.o >> obj-$(CONFIG_RTC_DRV_MPC5121) += rtc-mpc5121.o >> obj-$(CONFIG_RTC_DRV_MSM6242) += rtc-msm6242.o >> diff --git a/drivers/rtc/rtc-meson.c b/drivers/rtc/rtc-meson.c >> new file mode 100644 >> index 000000000000..09d6849b804a >> --- /dev/null >> +++ b/drivers/rtc/rtc-meson.c >> @@ -0,0 +1,409 @@ >> +// SPDX-License-Identifier: GPL-2.0 >> +/* >> + * RTC driver for the interal RTC block in the Amlogic Meson6, Meson8, >> + * Meson8b and Meson8m2 SoCs. >> + * >> + * The RTC is split in to two parts, the AHB front end and a simple serial >> + * connection to the actual registers. This driver manages both parts. >> + * >> + * Copyright (c) 2018 Martin Blumenstingl <martin.blumenstingl@googlemail.com> >> + * Copyright (c) 2015 Ben Dooks <ben.dooks@codethink.co.uk> for Codethink Ltd >> + * Based on origin by Carlo Caione <carlo@endlessm.com> >> + */ >> + >> +#include <linux/bitfield.h> >> +#include <linux/delay.h> >> +#include <linux/io.h> >> +#include <linux/kernel.h> >> +#include <linux/module.h> >> +#include <linux/nvmem-provider.h> >> +#include <linux/of.h> >> +#include <linux/platform_device.h> >> +#include <linux/regmap.h> >> +#include <linux/regulator/consumer.h> >> +#include <linux/reset.h> >> +#include <linux/rtc.h> >> + >> +/* registers accessed from cpu bus */ >> +#define RTC_ADDR0 0x00 >> + #define RTC_ADDR0_LINE_SCLK BIT(0) >> + #define RTC_ADDR0_LINE_SEN BIT(1) >> + #define RTC_ADDR0_LINE_SDI BIT(2) >> + #define RTC_ADDR0_START_SER BIT(17) >> + #define RTC_ADDR0_WAIT_SER BIT(22) >> + #define RTC_ADDR0_DATA GENMASK(31, 24) >> + >> +#define RTC_ADDR1 0x04 >> + #define RTC_ADDR1_SDO BIT(0) >> + #define RTC_ADDR1_S_READY BIT(1) >> + >> +#define RTC_ADDR2 0x08 >> +#define RTC_ADDR3 0x0c >> + >> +#define RTC_REG4 0x10 >> + #define RTC_REG4_STATIC_VALUE GENMASK(7, 0) >> + >> +/* rtc registers accessed via rtc-serial interface */ >> +#define RTC_COUNTER (0) >> +#define RTC_SEC_ADJ (2) >> +#define RTC_REGMEM_0 (4) >> +#define RTC_REGMEM_1 (5) >> +#define RTC_REGMEM_2 (6) >> +#define RTC_REGMEM_3 (7) >> + >> +#define RTC_ADDR_BITS (3) /* number of address bits to send */ >> +#define RTC_DATA_BITS (32) /* number of data bits to tx/rx */ >> + >> +#define MESON_STATIC_BIAS_CUR (0x5 << 1) >> +#define MESON_STATIC_VOLTAGE (0x3 << 11) >> +#define MESON_STATIC_DEFAULT (MESON_STATIC_BIAS_CUR | MESON_STATIC_VOLTAGE) >> + >> +struct meson_rtc { >> + struct rtc_device *rtc; /* rtc device we created */ >> + struct device *dev; /* device we bound from */ >> + struct reset_control *reset; /* reset source */ >> + struct regulator *vdd; /* voltage input */ >> + struct regmap *peripheral; /* peripheral registers */ >> + struct regmap *serial; /* serial registers */ >> +}; >> + >> +static const struct regmap_config meson_rtc_peripheral_regmap_config = { >> + .name = "peripheral-registers", >> + .reg_bits = 8, >> + .val_bits = 32, >> + .reg_stride = 4, >> + .max_register = RTC_REG4, >> + .fast_io = true, >> +}; >> + >> +/* RTC front-end serialiser controls */ >> + >> +static void meson_rtc_sclk_pulse(struct meson_rtc *rtc) >> +{ >> + udelay(5); >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, 0); >> + udelay(5); >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, >> + RTC_ADDR0_LINE_SCLK); >> +} >> + >> +static void meson_rtc_send_bit(struct meson_rtc *rtc, unsigned int bit) >> +{ >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, >> + bit ? RTC_ADDR0_LINE_SDI : 0); >> + meson_rtc_sclk_pulse(rtc); >> +} >> + >> +static void meson_rtc_send_bits(struct meson_rtc *rtc, u32 data, >> + unsigned int nr) >> +{ >> + u32 bit = 1 << (nr - 1); >> + >> + while (bit) { >> + meson_rtc_send_bit(rtc, data & bit); >> + bit >>= 1; >> + } >> +} >> + >> +static void meson_rtc_set_dir(struct meson_rtc *rtc, u32 mode) >> +{ >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, 0); >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); >> + meson_rtc_send_bit(rtc, mode); >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); >> +} >> + >> +static u32 meson_rtc_get_data(struct meson_rtc *rtc) >> +{ >> + u32 tmp, val = 0; >> + int bit; >> + >> + for (bit = 0; bit < RTC_DATA_BITS; bit++) { >> + meson_rtc_sclk_pulse(rtc); >> + val <<= 1; >> + >> + regmap_read(rtc->peripheral, RTC_ADDR1, &tmp); >> + val |= tmp & RTC_ADDR1_SDO; >> + } >> + >> + return val; >> +} >> + >> +static int meson_rtc_get_bus(struct meson_rtc *rtc) >> +{ >> + int ret, retries = 3; >> + u32 val; >> + >> + /* prepare bus for transfers, set all lines low */ >> + val = RTC_ADDR0_LINE_SDI | RTC_ADDR0_LINE_SEN | RTC_ADDR0_LINE_SCLK; >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, val, 0); >> + >> + for (retries = 0; retries < 3; retries++) { >> + /* wait for the bus to be ready */ >> + if (!regmap_read_poll_timeout(rtc->peripheral, RTC_ADDR1, val, >> + val & RTC_ADDR1_S_READY, 10, >> + 10000)) >> + return 0; >> + >> + dev_warn(rtc->dev, "failed to get bus, resetting RTC\n"); >> + >> + ret = reset_control_reset(rtc->reset); >> + if (ret) >> + return ret; >> + } >> + >> + dev_err(rtc->dev, "bus is not ready\n"); >> + return -ETIMEDOUT; >> +} >> + >> +static int meson_rtc_serial_bus_reg_read(void *context, unsigned int reg, >> + unsigned int *data) >> +{ >> + struct meson_rtc *rtc = context; >> + int ret; >> + >> + ret = meson_rtc_get_bus(rtc); >> + if (ret) >> + return ret; >> + >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, >> + RTC_ADDR0_LINE_SEN); >> + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); >> + meson_rtc_set_dir(rtc, 0); >> + *data = meson_rtc_get_data(rtc); >> + >> + return 0; >> +} >> + >> +static int meson_rtc_serial_bus_reg_write(void *context, unsigned int reg, >> + unsigned int data) >> +{ >> + struct meson_rtc *rtc = context; >> + int ret; >> + >> + ret = meson_rtc_get_bus(rtc); >> + if (ret) >> + return ret; >> + >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, >> + RTC_ADDR0_LINE_SEN); >> + meson_rtc_send_bits(rtc, data, RTC_DATA_BITS); >> + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); >> + meson_rtc_set_dir(rtc, 1); >> + >> + return 0; >> +} >> + >> +static const struct regmap_bus meson_rtc_serial_bus = { >> + .reg_read = meson_rtc_serial_bus_reg_read, >> + .reg_write = meson_rtc_serial_bus_reg_write, >> +}; >> + >> +static const struct regmap_config meson_rtc_serial_regmap_config = { >> + .name = "serial-registers", >> + .reg_bits = 4, >> + .reg_stride = 1, >> + .val_bits = 32, >> + .max_register = RTC_REGMEM_3, >> + .fast_io = false, >> +}; >> + >> +static int meson_rtc_write_static(struct meson_rtc *rtc, u32 data) >> +{ >> + u32 tmp; >> + >> + regmap_write(rtc->peripheral, RTC_REG4, >> + FIELD_PREP(RTC_REG4_STATIC_VALUE, (data >> 8))); >> + >> + /* write the static value and start the auto serializer */ >> + tmp = FIELD_PREP(RTC_ADDR0_DATA, (data & 0xff)) | RTC_ADDR0_START_SER; >> + regmap_update_bits(rtc->peripheral, RTC_ADDR0, >> + RTC_ADDR0_DATA | RTC_ADDR0_START_SER, tmp); >> + >> + /* wait for the auto serializer to complete */ >> + return regmap_read_poll_timeout(rtc->peripheral, RTC_REG4, tmp, >> + !(tmp & RTC_ADDR0_WAIT_SER), 10, >> + 10000); >> +} >> + >> +/* RTC interface layer functions */ >> + >> +static int meson_rtc_gettime(struct device *dev, struct rtc_time *tm) >> +{ >> + struct meson_rtc *rtc = dev_get_drvdata(dev); >> + u32 time; >> + int ret; >> + >> + ret = regmap_read(rtc->serial, RTC_COUNTER, &time); >> + if (!ret) >> + rtc_time64_to_tm(time, tm); >> + >> + return ret; >> +} >> + >> +static int meson_rtc_settime(struct device *dev, struct rtc_time *tm) >> +{ >> + struct meson_rtc *rtc = dev_get_drvdata(dev); >> + >> + return regmap_write(rtc->serial, RTC_COUNTER, rtc_tm_to_time64(tm)); >> +} >> + >> +static const struct rtc_class_ops meson_rtc_ops = { >> + .read_time = meson_rtc_gettime, >> + .set_time = meson_rtc_settime, >> +}; >> + >> +/* NVMEM interface layer functions */ >> + >> +static int meson_rtc_regmem_read(void *context, unsigned int offset, >> + void *buf, size_t bytes) >> +{ >> + struct meson_rtc *rtc = context; >> + unsigned int read_offset, read_size; >> + >> + read_offset = RTC_REGMEM_0 + (offset / 4); >> + read_size = bytes / 4; >> + >> + return regmap_bulk_read(rtc->serial, read_offset, buf, read_size); >> +} >> + >> +static int meson_rtc_regmem_write(void *context, unsigned int offset, >> + void *buf, size_t bytes) >> +{ >> + struct meson_rtc *rtc = context; >> + unsigned int write_offset, write_size; >> + >> + write_offset = RTC_REGMEM_0 + (offset / 4); >> + write_size = bytes / 4; >> + >> + return regmap_bulk_write(rtc->serial, write_offset, buf, write_size); >> +} >> + >> +static int meson_rtc_probe(struct platform_device *pdev) >> +{ >> + struct nvmem_config meson_rtc_nvmem_config = { >> + .name = "meson-rtc-regmem", >> + .word_size = 4, >> + .stride = 4, >> + .size = SZ_16, >> + .reg_read = meson_rtc_regmem_read, >> + .reg_write = meson_rtc_regmem_write, >> + }; >> + struct device *dev = &pdev->dev; >> + struct meson_rtc *rtc; >> + struct resource *res; >> + void __iomem *base; >> + int ret; >> + u32 tm; >> + >> + rtc = devm_kzalloc(dev, sizeof(struct meson_rtc), GFP_KERNEL); >> + if (!rtc) >> + return -ENOMEM; >> + >> + rtc->rtc = devm_rtc_allocate_device(dev); >> + if (IS_ERR(rtc->rtc)) >> + return PTR_ERR(rtc->rtc); >> + >> + platform_set_drvdata(pdev, rtc); >> + >> + rtc->dev = dev; >> + >> + rtc->rtc->ops = &meson_rtc_ops; >> + rtc->rtc->range_max = U32_MAX; >> + >> + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); >> + base = devm_ioremap_resource(dev, res); >> + if (IS_ERR(base)) >> + return PTR_ERR(base); >> + >> + rtc->peripheral = devm_regmap_init_mmio(dev, base, >> + &meson_rtc_peripheral_regmap_config); >> + if (IS_ERR(rtc->peripheral)) { >> + dev_err(dev, "failed to create peripheral regmap\n"); >> + return PTR_ERR(rtc->peripheral); >> + } >> + >> + rtc->reset = devm_reset_control_get(dev, NULL); >> + if (IS_ERR(rtc->reset)) { >> + dev_err(dev, "missing reset line\n"); >> + return PTR_ERR(rtc->reset); >> + } >> + >> + rtc->vdd = devm_regulator_get(dev, "vdd"); >> + if (IS_ERR(rtc->vdd)) { >> + dev_err(dev, "failed to get the vdd-supply\n"); >> + return PTR_ERR(rtc->vdd); >> + } >> + >> + ret = regulator_enable(rtc->vdd); >> + if (ret) { >> + dev_err(dev, "failed to enable vdd-supply\n"); >> + return ret; >> + } >> + >> + ret = meson_rtc_write_static(rtc, MESON_STATIC_DEFAULT); >> + if (ret) { >> + dev_err(dev, "failed to set static values\n"); >> + goto out_disable_vdd; >> + } >> + >> + rtc->serial = devm_regmap_init(dev, &meson_rtc_serial_bus, rtc, >> + &meson_rtc_serial_regmap_config); >> + if (IS_ERR(rtc->serial)) { >> + dev_err(dev, "failed to create serial regmap\n"); >> + ret = PTR_ERR(rtc->serial); >> + goto out_disable_vdd; >> + } >> + >> + /* >> + * check if we can read RTC counter, if not then the RTC is probably >> + * not functional. If it isn't probably best to not bind. >> + */ >> + ret = regmap_read(rtc->serial, RTC_COUNTER, &tm); >> + if (ret) { >> + dev_err(dev, "cannot read RTC counter, RTC not functional\n"); >> + goto out_disable_vdd; >> + } >> + >> + meson_rtc_nvmem_config.priv = rtc; >> + ret = rtc_nvmem_register(rtc->rtc, &meson_rtc_nvmem_config); >> + if (ret) >> + goto out_disable_vdd; >> + >> + ret = rtc_register_device(rtc->rtc); >> + if (ret) >> + goto out_unregister_nvmem; >> + >> + return 0; >> + >> +out_unregister_nvmem: >> + rtc_nvmem_unregister(rtc->rtc); >> + >> +out_disable_vdd: >> + regulator_disable(rtc->vdd); >> + return ret; >> +} >> + >> +static const struct of_device_id meson_rtc_dt_match[] = { >> + { .compatible = "amlogic,meson6-rtc", }, >> + { .compatible = "amlogic,meson8-rtc", }, >> + { .compatible = "amlogic,meson8b-rtc", }, >> + { .compatible = "amlogic,meson8m2-rtc", }, >> + { }, >> +}; >> +MODULE_DEVICE_TABLE(of, meson_rtc_dt_match); >> + >> +static struct platform_driver meson_rtc_driver = { >> + .probe = meson_rtc_probe, >> + .driver = { >> + .name = "meson-rtc", >> + .of_match_table = of_match_ptr(meson_rtc_dt_match), >> + }, >> +}; >> +module_platform_driver(meson_rtc_driver); >> + >> +MODULE_DESCRIPTION("Amlogic Meson RTC Driver"); >> +MODULE_AUTHOR("Ben Dooks <ben.doosk@codethink.co.uk>"); >> +MODULE_AUTHOR("Martin Blumenstingl <martin.blumenstingl@googlemail.com>"); >> +MODULE_LICENSE("GPL v2"); >> +MODULE_ALIAS("platform:meson-rtc"); >> > >
Hi Ben, On Wed, Dec 5, 2018 at 2:36 PM Ben Dooks <ben.dooks@codethink.co.uk> wrote: > > On 02/12/2018 22:08, Martin Blumenstingl wrote: > > Add support for the RTC block on the 32-bit Amlogic Meson6, Meson8, > > Meson8b and Meson8m2 SoCs. > > > > The RTC is split in to two parts, which are both managed by this driver: > > - the AHB front end > > - and a simple serial connection to the actual registers > > > > The RTC_COUNTER register which holds the time is 32-bits wide. > > > > There are four 32-bit wide (in total: 16 bytes) "regmem" registers which > > are exposed using nvmem. On Amlogic's 3.10 kernel this is used to store > > data which needs to survive a suspend / resume cycle. > > > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > [resurrected Ben's patches after 2 years] > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > Just checking if the change of author is deliberate? > not sure how to do >1 author on patches like this. I changed the author on purpose after seeing how much I changed (compared to your v4). this is also why I added you to the CC list however, I left in your copyright (just like you kept Carlo's copyright note) and MODULE_AUTHOR if you're uncomfortable with this author change then please let me know so I can undo it for the next iteration (which I need to fix the issue found by the kbuild test robot). in any case: thank you for your work on the previous iterations of this series - that made life much easier for me :) Regards Martin
Hi Ben, On Wed, Dec 5, 2018 at 11:15 PM Martin Blumenstingl <martin.blumenstingl@googlemail.com> wrote: [...] > > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > > [resurrected Ben's patches after 2 years] > > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > > > Just checking if the change of author is deliberate? > > not sure how to do >1 author on patches like this. > I changed the author on purpose after seeing how much I changed > (compared to your v4). this is also why I added you to the CC list > however, I left in your copyright (just like you kept Carlo's > copyright note) and MODULE_AUTHOR please let me know: 1. whether you want to be the author of the patch or if you're fine with my name in there 2. if you are fine with both of us being listed as Signed-off-by or if you want me to switch to Co-authored-by 3. about any other concerns you have I would like to update this patch next weekend so it can be reviewed (and merged once all is fine) for v4.22 Kind Regards Martin
Hi Ben, On Tue, Dec 18, 2018 at 12:48 AM Martin Blumenstingl <martin.blumenstingl@googlemail.com> wrote: > > Hi Ben, > > On Wed, Dec 5, 2018 at 11:15 PM Martin Blumenstingl > <martin.blumenstingl@googlemail.com> wrote: > [...] > > > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > > > [resurrected Ben's patches after 2 years] > > > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > > > > > Just checking if the change of author is deliberate? > > > not sure how to do >1 author on patches like this. > > I changed the author on purpose after seeing how much I changed > > (compared to your v4). this is also why I added you to the CC list > > however, I left in your copyright (just like you kept Carlo's > > copyright note) and MODULE_AUTHOR > please let me know: > 1. whether you want to be the author of the patch or if you're fine > with my name in there > 2. if you are fine with both of us being listed as Signed-off-by or if > you want me to switch to Co-authored-by > 3. about any other concerns you have > > I would like to update this patch next weekend so it can be reviewed > (and merged once all is fine) for v4.22 I've been busy last weekend so I didn't manage to update this series please let me know how you want me to proceed with this Regards Martin
Hi Ben, On Tue, Dec 18, 2018 at 12:48 AM Martin Blumenstingl <martin.blumenstingl@googlemail.com> wrote: > > Hi Ben, > > On Wed, Dec 5, 2018 at 11:15 PM Martin Blumenstingl > <martin.blumenstingl@googlemail.com> wrote: > [...] > > > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > > > [resurrected Ben's patches after 2 years] > > > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > > > > > Just checking if the change of author is deliberate? > > > not sure how to do >1 author on patches like this. > > I changed the author on purpose after seeing how much I changed > > (compared to your v4). this is also why I added you to the CC list > > however, I left in your copyright (just like you kept Carlo's > > copyright note) and MODULE_AUTHOR > please let me know: > 1. whether you want to be the author of the patch or if you're fine > with my name in there > 2. if you are fine with both of us being listed as Signed-off-by or if > you want me to switch to Co-authored-by > 3. about any other concerns you have > > I would like to update this patch next weekend so it can be reviewed > (and merged once all is fine) for v4.22 gentle ping on this one now that the holiday season is over. Regards Martin
On 12/01/2019 13:52:00+0100, Martin Blumenstingl wrote: > Hi Ben, > > On Tue, Dec 18, 2018 at 12:48 AM Martin Blumenstingl > <martin.blumenstingl@googlemail.com> wrote: > > > > Hi Ben, > > > > On Wed, Dec 5, 2018 at 11:15 PM Martin Blumenstingl > > <martin.blumenstingl@googlemail.com> wrote: > > [...] > > > > > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > > > > [resurrected Ben's patches after 2 years] > > > > > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > > > > > > > Just checking if the change of author is deliberate? > > > > not sure how to do >1 author on patches like this. > > > I changed the author on purpose after seeing how much I changed > > > (compared to your v4). this is also why I added you to the CC list > > > however, I left in your copyright (just like you kept Carlo's > > > copyright note) and MODULE_AUTHOR > > please let me know: > > 1. whether you want to be the author of the patch or if you're fine > > with my name in there > > 2. if you are fine with both of us being listed as Signed-off-by or if > > you want me to switch to Co-authored-by > > 3. about any other concerns you have > > > > I would like to update this patch next weekend so it can be reviewed > > (and merged once all is fine) for v4.22 > gentle ping on this one now that the holiday season is over. > I would think you can resend as they are. Or, you could resend v4 from Ben and then a patch with your modifications.
diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index a819ef07b7ec..d5d0e3affdc6 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -1285,6 +1285,17 @@ config RTC_DRV_IMXDI This driver can also be built as a module, if so, the module will be called "rtc-imxdi". +config RTC_DRV_MESON + tristate "Amlogic Meson RTC" + depends on (ARM && ARCH_MESON) || COMPILE_TEST + select REGMAP_MMIO + help + Support for the RTC block on the Amlogic Meson6, Meson8, Meson8b + and Meson8m2 SoCs. + + This driver can also be built as a module, if so, the module + will be called "rtc-meson". + config RTC_DRV_OMAP tristate "TI OMAP Real Time Clock" depends on ARCH_OMAP || ARCH_DAVINCI || COMPILE_TEST diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile index 290c1730fb0a..3b088e75149d 100644 --- a/drivers/rtc/Makefile +++ b/drivers/rtc/Makefile @@ -99,6 +99,7 @@ obj-$(CONFIG_RTC_DRV_MAX8997) += rtc-max8997.o obj-$(CONFIG_RTC_DRV_MAX8998) += rtc-max8998.o obj-$(CONFIG_RTC_DRV_MC13XXX) += rtc-mc13xxx.o obj-$(CONFIG_RTC_DRV_MCP795) += rtc-mcp795.o +obj-$(CONFIG_RTC_DRV_MESON) += rtc-meson.o obj-$(CONFIG_RTC_DRV_MOXART) += rtc-moxart.o obj-$(CONFIG_RTC_DRV_MPC5121) += rtc-mpc5121.o obj-$(CONFIG_RTC_DRV_MSM6242) += rtc-msm6242.o diff --git a/drivers/rtc/rtc-meson.c b/drivers/rtc/rtc-meson.c new file mode 100644 index 000000000000..09d6849b804a --- /dev/null +++ b/drivers/rtc/rtc-meson.c @@ -0,0 +1,409 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * RTC driver for the interal RTC block in the Amlogic Meson6, Meson8, + * Meson8b and Meson8m2 SoCs. + * + * The RTC is split in to two parts, the AHB front end and a simple serial + * connection to the actual registers. This driver manages both parts. + * + * Copyright (c) 2018 Martin Blumenstingl <martin.blumenstingl@googlemail.com> + * Copyright (c) 2015 Ben Dooks <ben.dooks@codethink.co.uk> for Codethink Ltd + * Based on origin by Carlo Caione <carlo@endlessm.com> + */ + +#include <linux/bitfield.h> +#include <linux/delay.h> +#include <linux/io.h> +#include <linux/kernel.h> +#include <linux/module.h> +#include <linux/nvmem-provider.h> +#include <linux/of.h> +#include <linux/platform_device.h> +#include <linux/regmap.h> +#include <linux/regulator/consumer.h> +#include <linux/reset.h> +#include <linux/rtc.h> + +/* registers accessed from cpu bus */ +#define RTC_ADDR0 0x00 + #define RTC_ADDR0_LINE_SCLK BIT(0) + #define RTC_ADDR0_LINE_SEN BIT(1) + #define RTC_ADDR0_LINE_SDI BIT(2) + #define RTC_ADDR0_START_SER BIT(17) + #define RTC_ADDR0_WAIT_SER BIT(22) + #define RTC_ADDR0_DATA GENMASK(31, 24) + +#define RTC_ADDR1 0x04 + #define RTC_ADDR1_SDO BIT(0) + #define RTC_ADDR1_S_READY BIT(1) + +#define RTC_ADDR2 0x08 +#define RTC_ADDR3 0x0c + +#define RTC_REG4 0x10 + #define RTC_REG4_STATIC_VALUE GENMASK(7, 0) + +/* rtc registers accessed via rtc-serial interface */ +#define RTC_COUNTER (0) +#define RTC_SEC_ADJ (2) +#define RTC_REGMEM_0 (4) +#define RTC_REGMEM_1 (5) +#define RTC_REGMEM_2 (6) +#define RTC_REGMEM_3 (7) + +#define RTC_ADDR_BITS (3) /* number of address bits to send */ +#define RTC_DATA_BITS (32) /* number of data bits to tx/rx */ + +#define MESON_STATIC_BIAS_CUR (0x5 << 1) +#define MESON_STATIC_VOLTAGE (0x3 << 11) +#define MESON_STATIC_DEFAULT (MESON_STATIC_BIAS_CUR | MESON_STATIC_VOLTAGE) + +struct meson_rtc { + struct rtc_device *rtc; /* rtc device we created */ + struct device *dev; /* device we bound from */ + struct reset_control *reset; /* reset source */ + struct regulator *vdd; /* voltage input */ + struct regmap *peripheral; /* peripheral registers */ + struct regmap *serial; /* serial registers */ +}; + +static const struct regmap_config meson_rtc_peripheral_regmap_config = { + .name = "peripheral-registers", + .reg_bits = 8, + .val_bits = 32, + .reg_stride = 4, + .max_register = RTC_REG4, + .fast_io = true, +}; + +/* RTC front-end serialiser controls */ + +static void meson_rtc_sclk_pulse(struct meson_rtc *rtc) +{ + udelay(5); + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, 0); + udelay(5); + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SCLK, + RTC_ADDR0_LINE_SCLK); +} + +static void meson_rtc_send_bit(struct meson_rtc *rtc, unsigned int bit) +{ + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, + bit ? RTC_ADDR0_LINE_SDI : 0); + meson_rtc_sclk_pulse(rtc); +} + +static void meson_rtc_send_bits(struct meson_rtc *rtc, u32 data, + unsigned int nr) +{ + u32 bit = 1 << (nr - 1); + + while (bit) { + meson_rtc_send_bit(rtc, data & bit); + bit >>= 1; + } +} + +static void meson_rtc_set_dir(struct meson_rtc *rtc, u32 mode) +{ + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, 0); + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); + meson_rtc_send_bit(rtc, mode); + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SDI, 0); +} + +static u32 meson_rtc_get_data(struct meson_rtc *rtc) +{ + u32 tmp, val = 0; + int bit; + + for (bit = 0; bit < RTC_DATA_BITS; bit++) { + meson_rtc_sclk_pulse(rtc); + val <<= 1; + + regmap_read(rtc->peripheral, RTC_ADDR1, &tmp); + val |= tmp & RTC_ADDR1_SDO; + } + + return val; +} + +static int meson_rtc_get_bus(struct meson_rtc *rtc) +{ + int ret, retries = 3; + u32 val; + + /* prepare bus for transfers, set all lines low */ + val = RTC_ADDR0_LINE_SDI | RTC_ADDR0_LINE_SEN | RTC_ADDR0_LINE_SCLK; + regmap_update_bits(rtc->peripheral, RTC_ADDR0, val, 0); + + for (retries = 0; retries < 3; retries++) { + /* wait for the bus to be ready */ + if (!regmap_read_poll_timeout(rtc->peripheral, RTC_ADDR1, val, + val & RTC_ADDR1_S_READY, 10, + 10000)) + return 0; + + dev_warn(rtc->dev, "failed to get bus, resetting RTC\n"); + + ret = reset_control_reset(rtc->reset); + if (ret) + return ret; + } + + dev_err(rtc->dev, "bus is not ready\n"); + return -ETIMEDOUT; +} + +static int meson_rtc_serial_bus_reg_read(void *context, unsigned int reg, + unsigned int *data) +{ + struct meson_rtc *rtc = context; + int ret; + + ret = meson_rtc_get_bus(rtc); + if (ret) + return ret; + + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, + RTC_ADDR0_LINE_SEN); + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); + meson_rtc_set_dir(rtc, 0); + *data = meson_rtc_get_data(rtc); + + return 0; +} + +static int meson_rtc_serial_bus_reg_write(void *context, unsigned int reg, + unsigned int data) +{ + struct meson_rtc *rtc = context; + int ret; + + ret = meson_rtc_get_bus(rtc); + if (ret) + return ret; + + regmap_update_bits(rtc->peripheral, RTC_ADDR0, RTC_ADDR0_LINE_SEN, + RTC_ADDR0_LINE_SEN); + meson_rtc_send_bits(rtc, data, RTC_DATA_BITS); + meson_rtc_send_bits(rtc, reg, RTC_ADDR_BITS); + meson_rtc_set_dir(rtc, 1); + + return 0; +} + +static const struct regmap_bus meson_rtc_serial_bus = { + .reg_read = meson_rtc_serial_bus_reg_read, + .reg_write = meson_rtc_serial_bus_reg_write, +}; + +static const struct regmap_config meson_rtc_serial_regmap_config = { + .name = "serial-registers", + .reg_bits = 4, + .reg_stride = 1, + .val_bits = 32, + .max_register = RTC_REGMEM_3, + .fast_io = false, +}; + +static int meson_rtc_write_static(struct meson_rtc *rtc, u32 data) +{ + u32 tmp; + + regmap_write(rtc->peripheral, RTC_REG4, + FIELD_PREP(RTC_REG4_STATIC_VALUE, (data >> 8))); + + /* write the static value and start the auto serializer */ + tmp = FIELD_PREP(RTC_ADDR0_DATA, (data & 0xff)) | RTC_ADDR0_START_SER; + regmap_update_bits(rtc->peripheral, RTC_ADDR0, + RTC_ADDR0_DATA | RTC_ADDR0_START_SER, tmp); + + /* wait for the auto serializer to complete */ + return regmap_read_poll_timeout(rtc->peripheral, RTC_REG4, tmp, + !(tmp & RTC_ADDR0_WAIT_SER), 10, + 10000); +} + +/* RTC interface layer functions */ + +static int meson_rtc_gettime(struct device *dev, struct rtc_time *tm) +{ + struct meson_rtc *rtc = dev_get_drvdata(dev); + u32 time; + int ret; + + ret = regmap_read(rtc->serial, RTC_COUNTER, &time); + if (!ret) + rtc_time64_to_tm(time, tm); + + return ret; +} + +static int meson_rtc_settime(struct device *dev, struct rtc_time *tm) +{ + struct meson_rtc *rtc = dev_get_drvdata(dev); + + return regmap_write(rtc->serial, RTC_COUNTER, rtc_tm_to_time64(tm)); +} + +static const struct rtc_class_ops meson_rtc_ops = { + .read_time = meson_rtc_gettime, + .set_time = meson_rtc_settime, +}; + +/* NVMEM interface layer functions */ + +static int meson_rtc_regmem_read(void *context, unsigned int offset, + void *buf, size_t bytes) +{ + struct meson_rtc *rtc = context; + unsigned int read_offset, read_size; + + read_offset = RTC_REGMEM_0 + (offset / 4); + read_size = bytes / 4; + + return regmap_bulk_read(rtc->serial, read_offset, buf, read_size); +} + +static int meson_rtc_regmem_write(void *context, unsigned int offset, + void *buf, size_t bytes) +{ + struct meson_rtc *rtc = context; + unsigned int write_offset, write_size; + + write_offset = RTC_REGMEM_0 + (offset / 4); + write_size = bytes / 4; + + return regmap_bulk_write(rtc->serial, write_offset, buf, write_size); +} + +static int meson_rtc_probe(struct platform_device *pdev) +{ + struct nvmem_config meson_rtc_nvmem_config = { + .name = "meson-rtc-regmem", + .word_size = 4, + .stride = 4, + .size = SZ_16, + .reg_read = meson_rtc_regmem_read, + .reg_write = meson_rtc_regmem_write, + }; + struct device *dev = &pdev->dev; + struct meson_rtc *rtc; + struct resource *res; + void __iomem *base; + int ret; + u32 tm; + + rtc = devm_kzalloc(dev, sizeof(struct meson_rtc), GFP_KERNEL); + if (!rtc) + return -ENOMEM; + + rtc->rtc = devm_rtc_allocate_device(dev); + if (IS_ERR(rtc->rtc)) + return PTR_ERR(rtc->rtc); + + platform_set_drvdata(pdev, rtc); + + rtc->dev = dev; + + rtc->rtc->ops = &meson_rtc_ops; + rtc->rtc->range_max = U32_MAX; + + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); + base = devm_ioremap_resource(dev, res); + if (IS_ERR(base)) + return PTR_ERR(base); + + rtc->peripheral = devm_regmap_init_mmio(dev, base, + &meson_rtc_peripheral_regmap_config); + if (IS_ERR(rtc->peripheral)) { + dev_err(dev, "failed to create peripheral regmap\n"); + return PTR_ERR(rtc->peripheral); + } + + rtc->reset = devm_reset_control_get(dev, NULL); + if (IS_ERR(rtc->reset)) { + dev_err(dev, "missing reset line\n"); + return PTR_ERR(rtc->reset); + } + + rtc->vdd = devm_regulator_get(dev, "vdd"); + if (IS_ERR(rtc->vdd)) { + dev_err(dev, "failed to get the vdd-supply\n"); + return PTR_ERR(rtc->vdd); + } + + ret = regulator_enable(rtc->vdd); + if (ret) { + dev_err(dev, "failed to enable vdd-supply\n"); + return ret; + } + + ret = meson_rtc_write_static(rtc, MESON_STATIC_DEFAULT); + if (ret) { + dev_err(dev, "failed to set static values\n"); + goto out_disable_vdd; + } + + rtc->serial = devm_regmap_init(dev, &meson_rtc_serial_bus, rtc, + &meson_rtc_serial_regmap_config); + if (IS_ERR(rtc->serial)) { + dev_err(dev, "failed to create serial regmap\n"); + ret = PTR_ERR(rtc->serial); + goto out_disable_vdd; + } + + /* + * check if we can read RTC counter, if not then the RTC is probably + * not functional. If it isn't probably best to not bind. + */ + ret = regmap_read(rtc->serial, RTC_COUNTER, &tm); + if (ret) { + dev_err(dev, "cannot read RTC counter, RTC not functional\n"); + goto out_disable_vdd; + } + + meson_rtc_nvmem_config.priv = rtc; + ret = rtc_nvmem_register(rtc->rtc, &meson_rtc_nvmem_config); + if (ret) + goto out_disable_vdd; + + ret = rtc_register_device(rtc->rtc); + if (ret) + goto out_unregister_nvmem; + + return 0; + +out_unregister_nvmem: + rtc_nvmem_unregister(rtc->rtc); + +out_disable_vdd: + regulator_disable(rtc->vdd); + return ret; +} + +static const struct of_device_id meson_rtc_dt_match[] = { + { .compatible = "amlogic,meson6-rtc", }, + { .compatible = "amlogic,meson8-rtc", }, + { .compatible = "amlogic,meson8b-rtc", }, + { .compatible = "amlogic,meson8m2-rtc", }, + { }, +}; +MODULE_DEVICE_TABLE(of, meson_rtc_dt_match); + +static struct platform_driver meson_rtc_driver = { + .probe = meson_rtc_probe, + .driver = { + .name = "meson-rtc", + .of_match_table = of_match_ptr(meson_rtc_dt_match), + }, +}; +module_platform_driver(meson_rtc_driver); + +MODULE_DESCRIPTION("Amlogic Meson RTC Driver"); +MODULE_AUTHOR("Ben Dooks <ben.doosk@codethink.co.uk>"); +MODULE_AUTHOR("Martin Blumenstingl <martin.blumenstingl@googlemail.com>"); +MODULE_LICENSE("GPL v2"); +MODULE_ALIAS("platform:meson-rtc");