[PATCH u-boot-marvell 1/4] rtc: add armada38x driver

Stefan Roese sr at denx.de
Fri Feb 26 10:19:55 CET 2021


On 09.02.21 23:55, Marek Behún wrote:
> Add RTC driver for Armada 38x, based on Linux' driver.
> For now implement only `marvell,armada-380-rtc` compatible.
> 
> Signed-off-by: Marek Behún <marek.behun at nic.cz>
> Cc: Pali Rohár <pali at kernel.org>
> Cc: Stefan Roese <sr at denx.de>
> Cc: Baruch Siach <baruch at tkos.co.il>
> Cc: Chris Packham <judge.packham at gmail.com>
> Cc: Simon Glass <sjg at chromium.org>
> ---
>   drivers/rtc/Kconfig     |   7 ++
>   drivers/rtc/Makefile    |   1 +
>   drivers/rtc/armada38x.c | 185 ++++++++++++++++++++++++++++++++++++++++
>   3 files changed, 193 insertions(+)
>   create mode 100644 drivers/rtc/armada38x.c
> 
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index aa6d90158c..dafba35279 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -38,6 +38,13 @@ config RTC_ENABLE_32KHZ_OUTPUT
>   	   Some real-time clocks support the output of 32kHz square waves (such as ds3231),
>   	   the config symbol choose Real Time Clock device 32Khz output feature.
>   
> +config RTC_ARMADA38X
> +	bool "Enable Armada 38x Marvell SoC RTC"
> +	depends on DM_RTC && ARCH_MVEBU
> +	help
> +	  This adds support for the in-chip RTC that can be found in the
> +	  Armada 38x Marvell's SoC devices.
> +
>   config RTC_PCF2127
>   	bool "Enable PCF2127 driver"
>   	depends on DM_RTC
> diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile
> index 6a45a9c874..15609e7b18 100644
> --- a/drivers/rtc/Makefile
> +++ b/drivers/rtc/Makefile
> @@ -8,6 +8,7 @@ obj-$(CONFIG_$(SPL_TPL_)DM_RTC) += rtc-uclass.o
>   
>   obj-$(CONFIG_RTC_AT91SAM9_RTT) += at91sam9_rtt.o
>   obj-y += rtc-lib.o
> +obj-$(CONFIG_RTC_ARMADA38X) += armada38x.o
>   obj-$(CONFIG_RTC_DAVINCI) += davinci.o
>   obj-$(CONFIG_RTC_DS1302) += ds1302.o
>   obj-$(CONFIG_RTC_DS1306) += ds1306.o
> diff --git a/drivers/rtc/armada38x.c b/drivers/rtc/armada38x.c
> new file mode 100644
> index 0000000000..88dc9977c6
> --- /dev/null
> +++ b/drivers/rtc/armada38x.c
> @@ -0,0 +1,185 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * RTC driver for the Armada 38x Marvell SoCs
> + *
> + * Copyright (C) 2021 Marek Behun <marek.behun at nic.cz>
> + *
> + * Based on Linux' driver by Gregory Clement and Marvell
> + */
> +
> +#include <common.h>

Please don't include "common.h" any more - this is deprecated. Other
than that looks good:

Reviewed-by: Stefan Roese <sr at denx.de>

for the complete series. Please resend v2 once it's ready.

Thanks,
Stefan

> +#include <asm/io.h>
> +#include <dm.h>
> +#include <linux/delay.h>
> +#include <rtc.h>
> +
> +#define RTC_STATUS			0x0
> +#define RTC_TIME			0xC
> +#define RTC_CONF_TEST			0x1C
> +
> +/* Armada38x SoC registers  */
> +#define RTC_38X_BRIDGE_TIMING_CTL	0x0
> +#define RTC_38X_PERIOD_OFFS		0
> +#define RTC_38X_PERIOD_MASK		(0x3FF << RTC_38X_PERIOD_OFFS)
> +#define RTC_38X_READ_DELAY_OFFS		26
> +#define RTC_38X_READ_DELAY_MASK		(0x1F << RTC_38X_READ_DELAY_OFFS)
> +
> +#define SAMPLE_NR			100
> +
> +struct armada38x_rtc {
> +	void __iomem *regs;
> +	void __iomem *regs_soc;
> +};
> +
> +/*
> + * According to Erratum RES-3124064 we have to do some configuration in MBUS.
> + * To read an RTC register we need to read it 100 times and return the most
> + * frequent value.
> + * To write an RTC register we need to write 2x zero into STATUS register,
> + * followed by the proper write. Linux adds an 5 us delay after this, so we do
> + * it here as well.
> + */
> +static void update_38x_mbus_timing_params(struct armada38x_rtc *rtc)
> +{
> +	u32 reg;
> +
> +	reg = readl(rtc->regs_soc + RTC_38X_BRIDGE_TIMING_CTL);
> +	reg &= ~RTC_38X_PERIOD_MASK;
> +	reg |= 0x3FF << RTC_38X_PERIOD_OFFS; /* Maximum value */
> +	reg &= ~RTC_38X_READ_DELAY_MASK;
> +	reg |= 0x1F << RTC_38X_READ_DELAY_OFFS; /* Maximum value */
> +	writel(reg, rtc->regs_soc + RTC_38X_BRIDGE_TIMING_CTL);
> +}
> +
> +static void armada38x_rtc_write(u32 val, struct armada38x_rtc *rtc, u8 reg)
> +{
> +	writel(0, rtc->regs + RTC_STATUS);
> +	writel(0, rtc->regs + RTC_STATUS);
> +	writel(val, rtc->regs + reg);
> +	udelay(5);
> +}
> +
> +static u32 armada38x_rtc_read(struct armada38x_rtc *rtc, u8 reg)
> +{
> +	u8 counts[SAMPLE_NR], max_idx;
> +	u32 samples[SAMPLE_NR], max;
> +	int i, j, last;
> +
> +	for (i = 0, last = 0; i < SAMPLE_NR; ++i) {
> +		u32 sample = readl(rtc->regs + reg);
> +
> +		/* find if this value was already read */
> +		for (j = 0; j < last; ++j) {
> +			if (samples[j] == sample)
> +				break;
> +		}
> +
> +		if (j < last) {
> +			/* if yes, increment count */
> +			++counts[j];
> +		} else {
> +			/* if not, add */
> +			samples[last] = sample;
> +			counts[last] = 1;
> +			++last;
> +		}
> +	}
> +
> +	/* finally find the sample that was read the most */
> +	max = 0;
> +	max_idx = 0;
> +
> +	for (i = 0; i < last; ++i) {
> +		if (counts[i] > max) {
> +			max = counts[i];
> +			max_idx = i;
> +		}
> +	}
> +
> +	return samples[max_idx];
> +}
> +
> +static int armada38x_rtc_get(struct udevice *dev, struct rtc_time *tm)
> +{
> +	struct armada38x_rtc *rtc = dev_get_priv(dev);
> +	u32 time;
> +
> +	time = armada38x_rtc_read(rtc, RTC_TIME);
> +
> +	rtc_to_tm(time, tm);
> +
> +	return 0;
> +}
> +
> +static int armada38x_rtc_reset(struct udevice *dev)
> +{
> +	struct armada38x_rtc *rtc = dev_get_priv(dev);
> +	u32 reg;
> +
> +	reg = armada38x_rtc_read(rtc, RTC_CONF_TEST);
> +
> +	if (reg & 0xff) {
> +		armada38x_rtc_write(0, rtc, RTC_CONF_TEST);
> +		mdelay(500);
> +		armada38x_rtc_write(0, rtc, RTC_TIME);
> +		armada38x_rtc_write(BIT(0) | BIT(1), 0, RTC_STATUS);
> +	}
> +
> +	return 0;
> +}
> +
> +static int armada38x_rtc_set(struct udevice *dev, const struct rtc_time *tm)
> +{
> +	struct armada38x_rtc *rtc = dev_get_priv(dev);
> +	unsigned long time;
> +
> +	time = rtc_mktime(tm);
> +
> +	if (time > U32_MAX)
> +		printf("%s: requested time to set will overflow\n", dev->name);
> +
> +	armada38x_rtc_reset(dev);
> +	armada38x_rtc_write(time, rtc, RTC_TIME);
> +
> +	return 0;
> +}
> +
> +static int armada38x_probe(struct udevice *dev)
> +{
> +	struct armada38x_rtc *rtc = dev_get_priv(dev);
> +
> +	rtc->regs = dev_remap_addr_name(dev, "rtc");
> +	if (!rtc->regs)
> +		goto err;
> +
> +	rtc->regs_soc = dev_remap_addr_name(dev, "rtc-soc");
> +	if (!rtc->regs_soc)
> +		goto err;
> +
> +	update_38x_mbus_timing_params(rtc);
> +
> +	return 0;
> +err:
> +	printf("%s: io address missing\n", dev->name);
> +	return -ENODEV;
> +}
> +
> +static const struct rtc_ops armada38x_rtc_ops = {
> +	.get = armada38x_rtc_get,
> +	.set = armada38x_rtc_set,
> +	.reset = armada38x_rtc_reset,
> +};
> +
> +static const struct udevice_id armada38x_rtc_ids[] = {
> +	{ .compatible = "marvell,armada-380-rtc", .data = 0 },
> +	{ }
> +};
> +
> +U_BOOT_DRIVER(rtc_armada38x) = {
> +	.name		= "rtc-armada38x",
> +	.id		= UCLASS_RTC,
> +	.of_match	= armada38x_rtc_ids,
> +	.probe		= armada38x_probe,
> +	.priv_auto	= sizeof(struct armada38x_rtc),
> +	.ops		= &armada38x_rtc_ops,
> +};
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de


More information about the U-Boot mailing list