* [PATCH 10/10] RTC S3C: Fix type of s3c_rtc_setaie()
@ 2010-11-28 18:31 Yauhen Kharuzhy
2010-11-29 10:36 ` Kukjin Kim
0 siblings, 1 reply; 3+ messages in thread
From: Yauhen Kharuzhy @ 2010-11-28 18:31 UTC (permalink / raw)
To: linux-arm-kernel
Change s3c_rtc_setaie() declaration in accordance with rtc_class_ops
structure declaration.
Signed-off-by: Yauhen Kharuzhy <jekhor@gmail.com>
---
arch/arm/mach-s3c2412/Kconfig | 7 +++++++
arch/arm/mach-s3c2412/Makefile | 3 ++-
arch/arm/mach-s3c2416/Kconfig | 1 +
drivers/rtc/rtc-s3c.c | 8 +++++---
4 files changed, 15 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-s3c2412/Kconfig b/arch/arm/mach-s3c2412/Kconfig
index fa2e5bf..6983cb4 100644
--- a/arch/arm/mach-s3c2412/Kconfig
+++ b/arch/arm/mach-s3c2412/Kconfig
@@ -28,9 +28,16 @@ config S3C2412_DMA
config S3C2412_PM
bool
+ select S3C2412_PM_SLEEP
help
Internal config node to apply S3C2412 power management
+config S3C2412_PM_SLEEP
+ bool
+ help
+ Internal config node to apply sleep for S3C2412 power management.
+ Can be selected by another SoCs with similar sleep procedure.
+
# Note, the S3C2412 IOtiming support is in plat-s3c24xx
config S3C2412_CPUFREQ
diff --git a/arch/arm/mach-s3c2412/Makefile b/arch/arm/mach-s3c2412/Makefile
index 530ec46..6c48a91 100644
--- a/arch/arm/mach-s3c2412/Makefile
+++ b/arch/arm/mach-s3c2412/Makefile
@@ -14,7 +14,8 @@ obj-$(CONFIG_CPU_S3C2412) += irq.o
obj-$(CONFIG_CPU_S3C2412) += clock.o
obj-$(CONFIG_CPU_S3C2412) += gpio.o
obj-$(CONFIG_S3C2412_DMA) += dma.o
-obj-$(CONFIG_S3C2412_PM) += pm.o sleep.o
+obj-$(CONFIG_S3C2412_PM) += pm.o
+obj-$(CONFIG_S3C2412_PM_SLEEP) += sleep.o
obj-$(CONFIG_S3C2412_CPUFREQ) += cpu-freq.o
# Machine support
diff --git a/arch/arm/mach-s3c2416/Kconfig b/arch/arm/mach-s3c2416/Kconfig
index 2eaadcf..69b48a7 100644
--- a/arch/arm/mach-s3c2416/Kconfig
+++ b/arch/arm/mach-s3c2416/Kconfig
@@ -27,6 +27,7 @@ config S3C2416_DMA
config S3C2416_PM
bool
+ select S3C2412_PM_SLEEP
help
Internal config node to apply S3C2416 power management
diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c
index cf953ec..4101522 100644
--- a/drivers/rtc/rtc-s3c.c
+++ b/drivers/rtc/rtc-s3c.c
@@ -77,7 +77,7 @@ static irqreturn_t s3c_rtc_tickirq(int irq, void *id)
}
/* Update control registers */
-static void s3c_rtc_setaie(int to)
+static int s3c_rtc_setaie(struct device *dev, unsigned int to)
{
unsigned int tmp;
@@ -89,6 +89,8 @@ static void s3c_rtc_setaie(int to)
tmp |= S3C2410_RTCALM_ALMEN;
writeb(tmp, s3c_rtc_base + S3C2410_RTCALM);
+
+ return 0;
}
static int s3c_rtc_setpie(struct device *dev, int enabled)
@@ -308,7 +310,7 @@ static int s3c_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm)
writeb(alrm_en, base + S3C2410_RTCALM);
- s3c_rtc_setaie(alrm->enabled);
+ s3c_rtc_setaie(dev, alrm->enabled);
return 0;
}
@@ -440,7 +442,7 @@ static int __devexit s3c_rtc_remove(struct platform_device *dev)
rtc_device_unregister(rtc);
s3c_rtc_setpie(&dev->dev, 0);
- s3c_rtc_setaie(0);
+ s3c_rtc_setaie(&dev->dev, 0);
clk_disable(rtc_clk);
clk_put(rtc_clk);
--
1.7.2.3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 10/10] RTC S3C: Fix type of s3c_rtc_setaie()
2010-11-28 18:31 [PATCH 10/10] RTC S3C: Fix type of s3c_rtc_setaie() Yauhen Kharuzhy
@ 2010-11-29 10:36 ` Kukjin Kim
2010-11-29 11:02 ` Yauhen Kharuzhy
0 siblings, 1 reply; 3+ messages in thread
From: Kukjin Kim @ 2010-11-29 10:36 UTC (permalink / raw)
To: linux-arm-kernel
Yauhen Kharuzhy wrote:
>
> Change s3c_rtc_setaie() declaration in accordance with rtc_class_ops
> structure declaration.
>
Hi,
According to your patch description, this should be regarding fix
s3c_rtc_setaie().
But includes others :-(
Please make sure your patch has no problem before submitting.
As a note, already has been submitted about fix s3c_rtc_setaie().
> Signed-off-by: Yauhen Kharuzhy <jekhor@gmail.com>
> ---
> arch/arm/mach-s3c2412/Kconfig | 7 +++++++
> arch/arm/mach-s3c2412/Makefile | 3 ++-
> arch/arm/mach-s3c2416/Kconfig | 1 +
> drivers/rtc/rtc-s3c.c | 8 +++++---
> 4 files changed, 15 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/mach-s3c2412/Kconfig b/arch/arm/mach-s3c2412/Kconfig
> index fa2e5bf..6983cb4 100644
> --- a/arch/arm/mach-s3c2412/Kconfig
> +++ b/arch/arm/mach-s3c2412/Kconfig
> @@ -28,9 +28,16 @@ config S3C2412_DMA
>
> config S3C2412_PM
> bool
> + select S3C2412_PM_SLEEP
> help
> Internal config node to apply S3C2412 power management
>
> +config S3C2412_PM_SLEEP
> + bool
> + help
> + Internal config node to apply sleep for S3C2412 power management.
> + Can be selected by another SoCs with similar sleep procedure.
> +
> # Note, the S3C2412 IOtiming support is in plat-s3c24xx
>
> config S3C2412_CPUFREQ
> diff --git a/arch/arm/mach-s3c2412/Makefile
b/arch/arm/mach-s3c2412/Makefile
> index 530ec46..6c48a91 100644
> --- a/arch/arm/mach-s3c2412/Makefile
> +++ b/arch/arm/mach-s3c2412/Makefile
> @@ -14,7 +14,8 @@ obj-$(CONFIG_CPU_S3C2412) += irq.o
> obj-$(CONFIG_CPU_S3C2412) += clock.o
> obj-$(CONFIG_CPU_S3C2412) += gpio.o
> obj-$(CONFIG_S3C2412_DMA) += dma.o
> -obj-$(CONFIG_S3C2412_PM) += pm.o sleep.o
> +obj-$(CONFIG_S3C2412_PM) += pm.o
> +obj-$(CONFIG_S3C2412_PM_SLEEP) += sleep.o
> obj-$(CONFIG_S3C2412_CPUFREQ) += cpu-freq.o
>
> # Machine support
> diff --git a/arch/arm/mach-s3c2416/Kconfig b/arch/arm/mach-s3c2416/Kconfig
> index 2eaadcf..69b48a7 100644
> --- a/arch/arm/mach-s3c2416/Kconfig
> +++ b/arch/arm/mach-s3c2416/Kconfig
> @@ -27,6 +27,7 @@ config S3C2416_DMA
>
> config S3C2416_PM
> bool
> + select S3C2412_PM_SLEEP
> help
> Internal config node to apply S3C2416 power management
>
> diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c
> index cf953ec..4101522 100644
> --- a/drivers/rtc/rtc-s3c.c
> +++ b/drivers/rtc/rtc-s3c.c
> @@ -77,7 +77,7 @@ static irqreturn_t s3c_rtc_tickirq(int irq, void *id)
> }
>
> /* Update control registers */
> -static void s3c_rtc_setaie(int to)
> +static int s3c_rtc_setaie(struct device *dev, unsigned int to)
> {
> unsigned int tmp;
>
> @@ -89,6 +89,8 @@ static void s3c_rtc_setaie(int to)
> tmp |= S3C2410_RTCALM_ALMEN;
>
> writeb(tmp, s3c_rtc_base + S3C2410_RTCALM);
> +
> + return 0;
> }
>
> static int s3c_rtc_setpie(struct device *dev, int enabled)
> @@ -308,7 +310,7 @@ static int s3c_rtc_setalarm(struct device *dev, struct
> rtc_wkalrm *alrm)
>
> writeb(alrm_en, base + S3C2410_RTCALM);
>
> - s3c_rtc_setaie(alrm->enabled);
> + s3c_rtc_setaie(dev, alrm->enabled);
>
> return 0;
> }
> @@ -440,7 +442,7 @@ static int __devexit s3c_rtc_remove(struct
> platform_device *dev)
> rtc_device_unregister(rtc);
>
> s3c_rtc_setpie(&dev->dev, 0);
> - s3c_rtc_setaie(0);
> + s3c_rtc_setaie(&dev->dev, 0);
>
> clk_disable(rtc_clk);
> clk_put(rtc_clk);
> --
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 10/10] RTC S3C: Fix type of s3c_rtc_setaie()
2010-11-29 10:36 ` Kukjin Kim
@ 2010-11-29 11:02 ` Yauhen Kharuzhy
0 siblings, 0 replies; 3+ messages in thread
From: Yauhen Kharuzhy @ 2010-11-29 11:02 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Nov 29, 2010 at 07:36:31PM +0900, Kukjin Kim wrote:
> Yauhen Kharuzhy wrote:
> >
> > Change s3c_rtc_setaie() declaration in accordance with rtc_class_ops
> > structure declaration.
> >
> Hi,
>
> According to your patch description, this should be regarding fix
> s3c_rtc_setaie().
> But includes others :-(
I am sorry, my mistake after reorder of my patches. OK, I will re-sent
patch for S3C2412/S3C2416 PM separately.
>
> Please make sure your patch has no problem before submitting.
> As a note, already has been submitted about fix s3c_rtc_setaie().
>
> > Signed-off-by: Yauhen Kharuzhy <jekhor@gmail.com>
> > ---
> > arch/arm/mach-s3c2412/Kconfig | 7 +++++++
> > arch/arm/mach-s3c2412/Makefile | 3 ++-
> > arch/arm/mach-s3c2416/Kconfig | 1 +
> > drivers/rtc/rtc-s3c.c | 8 +++++---
> > 4 files changed, 15 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm/mach-s3c2412/Kconfig b/arch/arm/mach-s3c2412/Kconfig
> > index fa2e5bf..6983cb4 100644
> > --- a/arch/arm/mach-s3c2412/Kconfig
> > +++ b/arch/arm/mach-s3c2412/Kconfig
> > @@ -28,9 +28,16 @@ config S3C2412_DMA
> >
> > config S3C2412_PM
> > bool
> > + select S3C2412_PM_SLEEP
> > help
> > Internal config node to apply S3C2412 power management
> >
> > +config S3C2412_PM_SLEEP
> > + bool
> > + help
> > + Internal config node to apply sleep for S3C2412 power management.
> > + Can be selected by another SoCs with similar sleep procedure.
> > +
> > # Note, the S3C2412 IOtiming support is in plat-s3c24xx
> >
> > config S3C2412_CPUFREQ
> > diff --git a/arch/arm/mach-s3c2412/Makefile
> b/arch/arm/mach-s3c2412/Makefile
> > index 530ec46..6c48a91 100644
> > --- a/arch/arm/mach-s3c2412/Makefile
> > +++ b/arch/arm/mach-s3c2412/Makefile
> > @@ -14,7 +14,8 @@ obj-$(CONFIG_CPU_S3C2412) += irq.o
> > obj-$(CONFIG_CPU_S3C2412) += clock.o
> > obj-$(CONFIG_CPU_S3C2412) += gpio.o
> > obj-$(CONFIG_S3C2412_DMA) += dma.o
> > -obj-$(CONFIG_S3C2412_PM) += pm.o sleep.o
> > +obj-$(CONFIG_S3C2412_PM) += pm.o
> > +obj-$(CONFIG_S3C2412_PM_SLEEP) += sleep.o
> > obj-$(CONFIG_S3C2412_CPUFREQ) += cpu-freq.o
> >
> > # Machine support
> > diff --git a/arch/arm/mach-s3c2416/Kconfig b/arch/arm/mach-s3c2416/Kconfig
> > index 2eaadcf..69b48a7 100644
> > --- a/arch/arm/mach-s3c2416/Kconfig
> > +++ b/arch/arm/mach-s3c2416/Kconfig
> > @@ -27,6 +27,7 @@ config S3C2416_DMA
> >
> > config S3C2416_PM
> > bool
> > + select S3C2412_PM_SLEEP
> > help
> > Internal config node to apply S3C2416 power management
> >
> > diff --git a/drivers/rtc/rtc-s3c.c b/drivers/rtc/rtc-s3c.c
> > index cf953ec..4101522 100644
> > --- a/drivers/rtc/rtc-s3c.c
> > +++ b/drivers/rtc/rtc-s3c.c
> > @@ -77,7 +77,7 @@ static irqreturn_t s3c_rtc_tickirq(int irq, void *id)
> > }
> >
> > /* Update control registers */
> > -static void s3c_rtc_setaie(int to)
> > +static int s3c_rtc_setaie(struct device *dev, unsigned int to)
> > {
> > unsigned int tmp;
> >
> > @@ -89,6 +89,8 @@ static void s3c_rtc_setaie(int to)
> > tmp |= S3C2410_RTCALM_ALMEN;
> >
> > writeb(tmp, s3c_rtc_base + S3C2410_RTCALM);
> > +
> > + return 0;
> > }
> >
> > static int s3c_rtc_setpie(struct device *dev, int enabled)
> > @@ -308,7 +310,7 @@ static int s3c_rtc_setalarm(struct device *dev, struct
> > rtc_wkalrm *alrm)
> >
> > writeb(alrm_en, base + S3C2410_RTCALM);
> >
> > - s3c_rtc_setaie(alrm->enabled);
> > + s3c_rtc_setaie(dev, alrm->enabled);
> >
> > return 0;
> > }
> > @@ -440,7 +442,7 @@ static int __devexit s3c_rtc_remove(struct
> > platform_device *dev)
> > rtc_device_unregister(rtc);
> >
> > s3c_rtc_setpie(&dev->dev, 0);
> > - s3c_rtc_setaie(0);
> > + s3c_rtc_setaie(&dev->dev, 0);
> >
> > clk_disable(rtc_clk);
> > clk_put(rtc_clk);
> > --
>
>
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
--
Yauhen Kharuzhy jekhor _at_ gmail.com
JID: jek at jabber.ru
A: No
Q: Should I quote below my post?
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-11-29 11:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-28 18:31 [PATCH 10/10] RTC S3C: Fix type of s3c_rtc_setaie() Yauhen Kharuzhy
2010-11-29 10:36 ` Kukjin Kim
2010-11-29 11:02 ` Yauhen Kharuzhy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).