[PATCH v2] rtc: make rtc-omap driver ioremap its register space

[PATCH v2] rtc: make rtc-omap driver ioremap its register space

[Mark A. Greer]

From: Mark A. Greer <mgreer@mvista.com>

The rtc-omap driver currently assumes that the rtc's
registers are at a fixed address and already mapped
into virtual memory space.  Remove those assumptions
so the same driver can be used for similar devices
that reside at different physical addresses (e.g.,
TI's DA8xx/OMAP-L13x SoC's).

Also allow the possibility for the timer and alarm
interrupts to use the same IRQ.

Signed-off-by: Mark A. Greer <mgreer@mvista.com>
CC: David Brownell <david-b@pacbell.net>
---
Sorry for the cross-posting but this is relevant to the different lists.

This patch was tested on a da830 and a da850.  It was only compiled
for an omap1 platform (don't have the hardware) but it should be low
risk.  Note that the platform_device data is already set up in
arch/arm/mach-omap1/devices.c so it those platforms should
"Just Work(tm)".

 drivers/rtc/Kconfig    |    7 ++++---
 drivers/rtc/rtc-omap.c |   47 ++++++++++++++++++++++++++---------------------
 2 files changed, 30 insertions(+), 24 deletions(-)

diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index 81adbdb..f06e885 100644
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -539,10 +539,11 @@ comment "on-CPU RTC drivers"
 
 config RTC_DRV_OMAP
 	tristate "TI OMAP1"
-	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730
+	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX
 	help
-	  Say "yes" here to support the real time clock on TI OMAP1 chips.
-	  This driver can also be built as a module called rtc-omap.
+	  Say "yes" here to support the real time clock on TI OMAP1 and
+	  DA8xx/OMAP-L13x chips.  This driver can also be built as a
+	  module called rtc-omap.
 
 config RTC_DRV_S3C
 	tristate "Samsung S3C series SoC RTC"
diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c
index bd1ce8e..aa418d5 100644
--- a/drivers/rtc/rtc-omap.c
+++ b/drivers/rtc/rtc-omap.c
@@ -87,9 +87,10 @@
 #define OMAP_RTC_INTERRUPTS_IT_ALARM    (1<<3)
 #define OMAP_RTC_INTERRUPTS_IT_TIMER    (1<<2)
 
+static void __iomem	*rtc_base;
 
-#define rtc_read(addr)		omap_readb(OMAP_RTC_BASE + (addr))
-#define rtc_write(val, addr)	omap_writeb(val, OMAP_RTC_BASE + (addr))
+#define rtc_read(addr)		__raw_readb(rtc_base + (addr))
+#define rtc_write(val, addr)	__raw_writeb(val, rtc_base + (addr))
 
 
 /* we rely on the rtc framework to handle locking (rtc->ops_lock),
@@ -330,32 +331,31 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
 		return -ENOENT;
 	}
 
-	/* NOTE:  using static mapping for RTC registers */
 	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	if (res && res->start != OMAP_RTC_BASE) {
-		pr_debug("%s: RTC registers at %08x, expected %08x\n",
-			pdev->name, (unsigned) res->start, OMAP_RTC_BASE);
+	if (!res) {
+		pr_debug("%s: RTC resource data missing\n", pdev->name);
 		return -ENOENT;
 	}
 
-	if (res)
-		mem = request_mem_region(res->start,
-				res->end - res->start + 1,
-				pdev->name);
-	else
-		mem = NULL;
+	mem = request_mem_region(res->start, resource_size(res), pdev->name);
 	if (!mem) {
 		pr_debug("%s: RTC registers at %08x are not free\n",
-			pdev->name, OMAP_RTC_BASE);
+			pdev->name, res->start);
 		return -EBUSY;
 	}
 
+	rtc_base = ioremap(res->start, resource_size(res));
+	if (!rtc_base) {
+		pr_debug("%s: RTC registers can't be mapped\n", pdev->name);
+		goto fail;
+	}
+
 	rtc = rtc_device_register(pdev->name, &pdev->dev,
 			&omap_rtc_ops, THIS_MODULE);
 	if (IS_ERR(rtc)) {
 		pr_debug("%s: can't register RTC device, err %ld\n",
 			pdev->name, PTR_ERR(rtc));
-		goto fail;
+		goto fail0;
 	}
 	platform_set_drvdata(pdev, rtc);
 	dev_set_drvdata(&rtc->dev, mem);
@@ -380,13 +380,14 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
 			dev_name(&rtc->dev), rtc)) {
 		pr_debug("%s: RTC timer interrupt IRQ%d already claimed\n",
 			pdev->name, omap_rtc_timer);
-		goto fail0;
+		goto fail1;
 	}
-	if (request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
-			dev_name(&rtc->dev), rtc)) {
+	if ((omap_rtc_timer != omap_rtc_alarm) &&
+		(request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
+			dev_name(&rtc->dev), rtc))) {
 		pr_debug("%s: RTC alarm interrupt IRQ%d already claimed\n",
 			pdev->name, omap_rtc_alarm);
-		goto fail1;
+		goto fail2;
 	}
 
 	/* On boards with split power, RTC_ON_NOFF won't reset the RTC */
@@ -419,10 +420,12 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
 
 	return 0;
 
-fail1:
+fail2:
 	free_irq(omap_rtc_timer, NULL);
-fail0:
+fail1:
 	rtc_device_unregister(rtc);
+fail0:
+	iounmap(rtc_base);
 fail:
 	release_resource(mem);
 	return -EIO;
@@ -438,7 +441,9 @@ static int __exit omap_rtc_remove(struct platform_device *pdev)
 	rtc_write(0, OMAP_RTC_INTERRUPTS_REG);
 
 	free_irq(omap_rtc_timer, rtc);
-	free_irq(omap_rtc_alarm, rtc);
+
+	if (omap_rtc_timer != omap_rtc_alarm)
+		free_irq(omap_rtc_alarm, rtc);
 
 	release_resource(dev_get_drvdata(&rtc->dev));
 	rtc_device_unregister(rtc);
-- 
1.6.2.5.182.ga808d

Re: [PATCH v2] rtc: make rtc-omap driver ioremap its register space

[Kevin Hilman]

"Mark A. Greer" <mgreer@mvista.com> writes:

> From: Mark A. Greer <mgreer@mvista.com>
>
> The rtc-omap driver currently assumes that the rtc's
> registers are at a fixed address and already mapped
> into virtual memory space.  Remove those assumptions
> so the same driver can be used for similar devices
> that reside at different physical addresses (e.g.,
> TI's DA8xx/OMAP-L13x SoC's).
>
> Also allow the possibility for the timer and alarm
> interrupts to use the same IRQ.
>
> Signed-off-by: Mark A. Greer <mgreer@mvista.com>
> CC: David Brownell <david-b@pacbell.net>

Acked-by: Kevin Hilman <khilman@deeprootsystems.com>

for relevance to davinci platforms.

Kevin


> ---
> Sorry for the cross-posting but this is relevant to the different lists.
>
> This patch was tested on a da830 and a da850.  It was only compiled
> for an omap1 platform (don't have the hardware) but it should be low
> risk.  Note that the platform_device data is already set up in
> arch/arm/mach-omap1/devices.c so it those platforms should
> "Just Work(tm)".
>
>  drivers/rtc/Kconfig    |    7 ++++---
>  drivers/rtc/rtc-omap.c |   47 ++++++++++++++++++++++++++---------------------
>  2 files changed, 30 insertions(+), 24 deletions(-)
>
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index 81adbdb..f06e885 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -539,10 +539,11 @@ comment "on-CPU RTC drivers"
>  
>  config RTC_DRV_OMAP
>  	tristate "TI OMAP1"
> -	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730
> +	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX
>  	help
> -	  Say "yes" here to support the real time clock on TI OMAP1 chips.
> -	  This driver can also be built as a module called rtc-omap.
> +	  Say "yes" here to support the real time clock on TI OMAP1 and
> +	  DA8xx/OMAP-L13x chips.  This driver can also be built as a
> +	  module called rtc-omap.
>  
>  config RTC_DRV_S3C
>  	tristate "Samsung S3C series SoC RTC"
> diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c
> index bd1ce8e..aa418d5 100644
> --- a/drivers/rtc/rtc-omap.c
> +++ b/drivers/rtc/rtc-omap.c
> @@ -87,9 +87,10 @@
>  #define OMAP_RTC_INTERRUPTS_IT_ALARM    (1<<3)
>  #define OMAP_RTC_INTERRUPTS_IT_TIMER    (1<<2)
>  
> +static void __iomem	*rtc_base;
>  
> -#define rtc_read(addr)		omap_readb(OMAP_RTC_BASE + (addr))
> -#define rtc_write(val, addr)	omap_writeb(val, OMAP_RTC_BASE + (addr))
> +#define rtc_read(addr)		__raw_readb(rtc_base + (addr))
> +#define rtc_write(val, addr)	__raw_writeb(val, rtc_base + (addr))
>  
>  
>  /* we rely on the rtc framework to handle locking (rtc->ops_lock),
> @@ -330,32 +331,31 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  		return -ENOENT;
>  	}
>  
> -	/* NOTE:  using static mapping for RTC registers */
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	if (res && res->start != OMAP_RTC_BASE) {
> -		pr_debug("%s: RTC registers at %08x, expected %08x\n",
> -			pdev->name, (unsigned) res->start, OMAP_RTC_BASE);
> +	if (!res) {
> +		pr_debug("%s: RTC resource data missing\n", pdev->name);
>  		return -ENOENT;
>  	}
>  
> -	if (res)
> -		mem = request_mem_region(res->start,
> -				res->end - res->start + 1,
> -				pdev->name);
> -	else
> -		mem = NULL;
> +	mem = request_mem_region(res->start, resource_size(res), pdev->name);
>  	if (!mem) {
>  		pr_debug("%s: RTC registers at %08x are not free\n",
> -			pdev->name, OMAP_RTC_BASE);
> +			pdev->name, res->start);
>  		return -EBUSY;
>  	}
>  
> +	rtc_base = ioremap(res->start, resource_size(res));
> +	if (!rtc_base) {
> +		pr_debug("%s: RTC registers can't be mapped\n", pdev->name);
> +		goto fail;
> +	}
> +
>  	rtc = rtc_device_register(pdev->name, &pdev->dev,
>  			&omap_rtc_ops, THIS_MODULE);
>  	if (IS_ERR(rtc)) {
>  		pr_debug("%s: can't register RTC device, err %ld\n",
>  			pdev->name, PTR_ERR(rtc));
> -		goto fail;
> +		goto fail0;
>  	}
>  	platform_set_drvdata(pdev, rtc);
>  	dev_set_drvdata(&rtc->dev, mem);
> @@ -380,13 +380,14 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  			dev_name(&rtc->dev), rtc)) {
>  		pr_debug("%s: RTC timer interrupt IRQ%d already claimed\n",
>  			pdev->name, omap_rtc_timer);
> -		goto fail0;
> +		goto fail1;
>  	}
> -	if (request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> -			dev_name(&rtc->dev), rtc)) {
> +	if ((omap_rtc_timer != omap_rtc_alarm) &&
> +		(request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> +			dev_name(&rtc->dev), rtc))) {
>  		pr_debug("%s: RTC alarm interrupt IRQ%d already claimed\n",
>  			pdev->name, omap_rtc_alarm);
> -		goto fail1;
> +		goto fail2;
>  	}
>  
>  	/* On boards with split power, RTC_ON_NOFF won't reset the RTC */
> @@ -419,10 +420,12 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  
>  	return 0;
>  
> -fail1:
> +fail2:
>  	free_irq(omap_rtc_timer, NULL);
> -fail0:
> +fail1:
>  	rtc_device_unregister(rtc);
> +fail0:
> +	iounmap(rtc_base);
>  fail:
>  	release_resource(mem);
>  	return -EIO;
> @@ -438,7 +441,9 @@ static int __exit omap_rtc_remove(struct platform_device *pdev)
>  	rtc_write(0, OMAP_RTC_INTERRUPTS_REG);
>  
>  	free_irq(omap_rtc_timer, rtc);
> -	free_irq(omap_rtc_alarm, rtc);
> +
> +	if (omap_rtc_timer != omap_rtc_alarm)
> +		free_irq(omap_rtc_alarm, rtc);
>  
>  	release_resource(dev_get_drvdata(&rtc->dev));
>  	rtc_device_unregister(rtc);
> -- 
> 1.6.2.5.182.ga808d
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2] rtc: make rtc-omap driver ioremap its register space

[Tony Lindgren]

* Kevin Hilman <khilman@deeprootsystems.com> [090922 10:27]:
> "Mark A. Greer" <mgreer@mvista.com> writes:
> 
> > From: Mark A. Greer <mgreer@mvista.com>
> >
> > The rtc-omap driver currently assumes that the rtc's
> > registers are at a fixed address and already mapped
> > into virtual memory space.  Remove those assumptions
> > so the same driver can be used for similar devices
> > that reside at different physical addresses (e.g.,
> > TI's DA8xx/OMAP-L13x SoC's).
> >
> > Also allow the possibility for the timer and alarm
> > interrupts to use the same IRQ.
> >
> > Signed-off-by: Mark A. Greer <mgreer@mvista.com>
> > CC: David Brownell <david-b@pacbell.net>
> 
> Acked-by: Kevin Hilman <khilman@deeprootsystems.com>
> 
> for relevance to davinci platforms.

Acked-by: Tony Lindgren <tony@atomide.com>



> 
> Kevin
> 
> 
> > ---
> > Sorry for the cross-posting but this is relevant to the different lists.
> >
> > This patch was tested on a da830 and a da850.  It was only compiled
> > for an omap1 platform (don't have the hardware) but it should be low
> > risk.  Note that the platform_device data is already set up in
> > arch/arm/mach-omap1/devices.c so it those platforms should
> > "Just Work(tm)".
> >
> >  drivers/rtc/Kconfig    |    7 ++++---
> >  drivers/rtc/rtc-omap.c |   47 ++++++++++++++++++++++++++---------------------
> >  2 files changed, 30 insertions(+), 24 deletions(-)
> >
> > diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> > index 81adbdb..f06e885 100644
> > --- a/drivers/rtc/Kconfig
> > +++ b/drivers/rtc/Kconfig
> > @@ -539,10 +539,11 @@ comment "on-CPU RTC drivers"
> >  
> >  config RTC_DRV_OMAP
> >  	tristate "TI OMAP1"
> > -	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730
> > +	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX
> >  	help
> > -	  Say "yes" here to support the real time clock on TI OMAP1 chips.
> > -	  This driver can also be built as a module called rtc-omap.
> > +	  Say "yes" here to support the real time clock on TI OMAP1 and
> > +	  DA8xx/OMAP-L13x chips.  This driver can also be built as a
> > +	  module called rtc-omap.
> >  
> >  config RTC_DRV_S3C
> >  	tristate "Samsung S3C series SoC RTC"
> > diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c
> > index bd1ce8e..aa418d5 100644
> > --- a/drivers/rtc/rtc-omap.c
> > +++ b/drivers/rtc/rtc-omap.c
> > @@ -87,9 +87,10 @@
> >  #define OMAP_RTC_INTERRUPTS_IT_ALARM    (1<<3)
> >  #define OMAP_RTC_INTERRUPTS_IT_TIMER    (1<<2)
> >  
> > +static void __iomem	*rtc_base;
> >  
> > -#define rtc_read(addr)		omap_readb(OMAP_RTC_BASE + (addr))
> > -#define rtc_write(val, addr)	omap_writeb(val, OMAP_RTC_BASE + (addr))
> > +#define rtc_read(addr)		__raw_readb(rtc_base + (addr))
> > +#define rtc_write(val, addr)	__raw_writeb(val, rtc_base + (addr))
> >  
> >  
> >  /* we rely on the rtc framework to handle locking (rtc->ops_lock),
> > @@ -330,32 +331,31 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
> >  		return -ENOENT;
> >  	}
> >  
> > -	/* NOTE:  using static mapping for RTC registers */
> >  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> > -	if (res && res->start != OMAP_RTC_BASE) {
> > -		pr_debug("%s: RTC registers at %08x, expected %08x\n",
> > -			pdev->name, (unsigned) res->start, OMAP_RTC_BASE);
> > +	if (!res) {
> > +		pr_debug("%s: RTC resource data missing\n", pdev->name);
> >  		return -ENOENT;
> >  	}
> >  
> > -	if (res)
> > -		mem = request_mem_region(res->start,
> > -				res->end - res->start + 1,
> > -				pdev->name);
> > -	else
> > -		mem = NULL;
> > +	mem = request_mem_region(res->start, resource_size(res), pdev->name);
> >  	if (!mem) {
> >  		pr_debug("%s: RTC registers at %08x are not free\n",
> > -			pdev->name, OMAP_RTC_BASE);
> > +			pdev->name, res->start);
> >  		return -EBUSY;
> >  	}
> >  
> > +	rtc_base = ioremap(res->start, resource_size(res));
> > +	if (!rtc_base) {
> > +		pr_debug("%s: RTC registers can't be mapped\n", pdev->name);
> > +		goto fail;
> > +	}
> > +
> >  	rtc = rtc_device_register(pdev->name, &pdev->dev,
> >  			&omap_rtc_ops, THIS_MODULE);
> >  	if (IS_ERR(rtc)) {
> >  		pr_debug("%s: can't register RTC device, err %ld\n",
> >  			pdev->name, PTR_ERR(rtc));
> > -		goto fail;
> > +		goto fail0;
> >  	}
> >  	platform_set_drvdata(pdev, rtc);
> >  	dev_set_drvdata(&rtc->dev, mem);
> > @@ -380,13 +380,14 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
> >  			dev_name(&rtc->dev), rtc)) {
> >  		pr_debug("%s: RTC timer interrupt IRQ%d already claimed\n",
> >  			pdev->name, omap_rtc_timer);
> > -		goto fail0;
> > +		goto fail1;
> >  	}
> > -	if (request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> > -			dev_name(&rtc->dev), rtc)) {
> > +	if ((omap_rtc_timer != omap_rtc_alarm) &&
> > +		(request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> > +			dev_name(&rtc->dev), rtc))) {
> >  		pr_debug("%s: RTC alarm interrupt IRQ%d already claimed\n",
> >  			pdev->name, omap_rtc_alarm);
> > -		goto fail1;
> > +		goto fail2;
> >  	}
> >  
> >  	/* On boards with split power, RTC_ON_NOFF won't reset the RTC */
> > @@ -419,10 +420,12 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
> >  
> >  	return 0;
> >  
> > -fail1:
> > +fail2:
> >  	free_irq(omap_rtc_timer, NULL);
> > -fail0:
> > +fail1:
> >  	rtc_device_unregister(rtc);
> > +fail0:
> > +	iounmap(rtc_base);
> >  fail:
> >  	release_resource(mem);
> >  	return -EIO;
> > @@ -438,7 +441,9 @@ static int __exit omap_rtc_remove(struct platform_device *pdev)
> >  	rtc_write(0, OMAP_RTC_INTERRUPTS_REG);
> >  
> >  	free_irq(omap_rtc_timer, rtc);
> > -	free_irq(omap_rtc_alarm, rtc);
> > +
> > +	if (omap_rtc_timer != omap_rtc_alarm)
> > +		free_irq(omap_rtc_alarm, rtc);
> >  
> >  	release_resource(dev_get_drvdata(&rtc->dev));
> >  	rtc_device_unregister(rtc);
> > -- 
> > 1.6.2.5.182.ga808d
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2] rtc: make rtc-omap driver ioremap its register space

[David Brownell]

On Wednesday 16 September 2009, Mark A. Greer wrote:
> From: Mark A. Greer <mgreer-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
> 
> The rtc-omap driver currently assumes that the rtc's
> registers are at a fixed address and already mapped
> into virtual memory space.  Remove those assumptions
> so the same driver can be used for similar devices
> that reside at different physical addresses (e.g.,
> TI's DA8xx/OMAP-L13x SoC's).
> 
> Also allow the possibility for the timer and alarm
> interrupts to use the same IRQ.
> 
> Signed-off-by: Mark A. Greer <mgreer-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
> CC: David Brownell <david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>

Acked-by: David Brownell <dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>

... but not tested on OMAP1 (e.g. OSK5912, H2).

> ---
> Sorry for the cross-posting but this is relevant to the different lists.
> 
> This patch was tested on a da830 and a da850.  It was only compiled
> for an omap1 platform (don't have the hardware) but it should be low
> risk.  Note that the platform_device data is already set up in
> arch/arm/mach-omap1/devices.c so it those platforms should
> "Just Work(tm)".
> 
>  drivers/rtc/Kconfig    |    7 ++++---
>  drivers/rtc/rtc-omap.c |   47 ++++++++++++++++++++++++++---------------------
>  2 files changed, 30 insertions(+), 24 deletions(-)
> 
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index 81adbdb..f06e885 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -539,10 +539,11 @@ comment "on-CPU RTC drivers"
>  
>  config RTC_DRV_OMAP
>  	tristate "TI OMAP1"
> -	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730
> +	depends on ARCH_OMAP15XX || ARCH_OMAP16XX || ARCH_OMAP730 || ARCH_DAVINCI_DA8XX
>  	help
> -	  Say "yes" here to support the real time clock on TI OMAP1 chips.
> -	  This driver can also be built as a module called rtc-omap.
> +	  Say "yes" here to support the real time clock on TI OMAP1 and
> +	  DA8xx/OMAP-L13x chips.  This driver can also be built as a
> +	  module called rtc-omap.
>  
>  config RTC_DRV_S3C
>  	tristate "Samsung S3C series SoC RTC"
> diff --git a/drivers/rtc/rtc-omap.c b/drivers/rtc/rtc-omap.c
> index bd1ce8e..aa418d5 100644
> --- a/drivers/rtc/rtc-omap.c
> +++ b/drivers/rtc/rtc-omap.c
> @@ -87,9 +87,10 @@
>  #define OMAP_RTC_INTERRUPTS_IT_ALARM    (1<<3)
>  #define OMAP_RTC_INTERRUPTS_IT_TIMER    (1<<2)
>  
> +static void __iomem	*rtc_base;
>  
> -#define rtc_read(addr)		omap_readb(OMAP_RTC_BASE + (addr))
> -#define rtc_write(val, addr)	omap_writeb(val, OMAP_RTC_BASE + (addr))
> +#define rtc_read(addr)		__raw_readb(rtc_base + (addr))
> +#define rtc_write(val, addr)	__raw_writeb(val, rtc_base + (addr))
>  
>  
>  /* we rely on the rtc framework to handle locking (rtc->ops_lock),
> @@ -330,32 +331,31 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  		return -ENOENT;
>  	}
>  
> -	/* NOTE:  using static mapping for RTC registers */
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	if (res && res->start != OMAP_RTC_BASE) {
> -		pr_debug("%s: RTC registers at %08x, expected %08x\n",
> -			pdev->name, (unsigned) res->start, OMAP_RTC_BASE);
> +	if (!res) {
> +		pr_debug("%s: RTC resource data missing\n", pdev->name);
>  		return -ENOENT;
>  	}
>  
> -	if (res)
> -		mem = request_mem_region(res->start,
> -				res->end - res->start + 1,
> -				pdev->name);
> -	else
> -		mem = NULL;
> +	mem = request_mem_region(res->start, resource_size(res), pdev->name);
>  	if (!mem) {
>  		pr_debug("%s: RTC registers at %08x are not free\n",
> -			pdev->name, OMAP_RTC_BASE);
> +			pdev->name, res->start);
>  		return -EBUSY;
>  	}
>  
> +	rtc_base = ioremap(res->start, resource_size(res));
> +	if (!rtc_base) {
> +		pr_debug("%s: RTC registers can't be mapped\n", pdev->name);
> +		goto fail;
> +	}
> +
>  	rtc = rtc_device_register(pdev->name, &pdev->dev,
>  			&omap_rtc_ops, THIS_MODULE);
>  	if (IS_ERR(rtc)) {
>  		pr_debug("%s: can't register RTC device, err %ld\n",
>  			pdev->name, PTR_ERR(rtc));
> -		goto fail;
> +		goto fail0;
>  	}
>  	platform_set_drvdata(pdev, rtc);
>  	dev_set_drvdata(&rtc->dev, mem);
> @@ -380,13 +380,14 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  			dev_name(&rtc->dev), rtc)) {
>  		pr_debug("%s: RTC timer interrupt IRQ%d already claimed\n",
>  			pdev->name, omap_rtc_timer);
> -		goto fail0;
> +		goto fail1;
>  	}
> -	if (request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> -			dev_name(&rtc->dev), rtc)) {
> +	if ((omap_rtc_timer != omap_rtc_alarm) &&
> +		(request_irq(omap_rtc_alarm, rtc_irq, IRQF_DISABLED,
> +			dev_name(&rtc->dev), rtc))) {
>  		pr_debug("%s: RTC alarm interrupt IRQ%d already claimed\n",
>  			pdev->name, omap_rtc_alarm);
> -		goto fail1;
> +		goto fail2;
>  	}
>  
>  	/* On boards with split power, RTC_ON_NOFF won't reset the RTC */
> @@ -419,10 +420,12 @@ static int __init omap_rtc_probe(struct platform_device *pdev)
>  
>  	return 0;
>  
> -fail1:
> +fail2:
>  	free_irq(omap_rtc_timer, NULL);
> -fail0:
> +fail1:
>  	rtc_device_unregister(rtc);
> +fail0:
> +	iounmap(rtc_base);
>  fail:
>  	release_resource(mem);
>  	return -EIO;
> @@ -438,7 +441,9 @@ static int __exit omap_rtc_remove(struct platform_device *pdev)
>  	rtc_write(0, OMAP_RTC_INTERRUPTS_REG);
>  
>  	free_irq(omap_rtc_timer, rtc);
> -	free_irq(omap_rtc_alarm, rtc);
> +
> +	if (omap_rtc_timer != omap_rtc_alarm)
> +		free_irq(omap_rtc_alarm, rtc);
>  
>  	release_resource(dev_get_drvdata(&rtc->dev));
>  	rtc_device_unregister(rtc);
> -- 
> 1.6.2.5.182.ga808d
> 
> 

Re: [PATCH v2] rtc: make rtc-omap driver ioremap its register space

[Mark A. Greer]

On Wed, Sep 16, 2009 at 05:04:58PM -0700, Mark A. Greer wrote:
> From: Mark A. Greer <mgreer-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
> 
> The rtc-omap driver currently assumes that the rtc's
> registers are at a fixed address and already mapped
> into virtual memory space.  Remove those assumptions
> so the same driver can be used for similar devices
> that reside at different physical addresses (e.g.,
> TI's DA8xx/OMAP-L13x SoC's).
> 
> Also allow the possibility for the timer and alarm
> interrupts to use the same IRQ.
> 
> Signed-off-by: Mark A. Greer <mgreer-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
> CC: David Brownell <david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
> ---

Ping?

Just wondering if this patch was noticed.

FYI, the following were added by subsequent emails:

	Acked-by: David Brownell <dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
	Acked-by: Kevin Hilman <khilman-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
	Acked-by: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>

Thanks,

Mark
--

Re: [rtc-linux] Re: [PATCH v2] rtc: make rtc-omap driver ioremap its register space

[Alessandro Zummo]

On Tue, 13 Oct 2009 16:00:23 -0700
"Mark A. Greer" <mgreer@mvista.com> wrote:

> Ping?
> 
> Just wondering if this patch was noticed.

 noticed. it's in the queue for the next submission set,
 which means after upstream merges all of current pending patches 
 (which are now in -mm) .
 
-- 

 Best regards,

 Alessandro Zummo,
  Tower Technologies - Torino, Italy

  http://www.towertech.it