From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.free-electrons.com (down.free-electrons.com. [37.187.137.238]) by gmr-mx.google.com with ESMTP id t6si88391wiz.0.2015.06.12.01.42.43 for ; Fri, 12 Jun 2015 01:42:43 -0700 (PDT) Date: Fri, 12 Jun 2015 10:42:38 +0200 From: Alexandre Belloni To: Xunlei Pang , "David S. Miller" Cc: linux-kernel@vger.kernel.org, rtc-linux@googlegroups.com, Alessandro Zummo , John Stultz , Arnd Bergmann , sparclinux@vger.kernel.org, Xunlei Pang Subject: [rtc-linux] Re: [PATCH v3 2/3] sparc: time: Replace update_persistent_clock() with CONFIG_RTC_SYSTOHC Message-ID: <20150612084238.GE3890@piout.net> References: <1434078618-16373-1-git-send-email-xlpang@126.com> <1434078618-16373-2-git-send-email-xlpang@126.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 In-Reply-To: <1434078618-16373-2-git-send-email-xlpang@126.com> Reply-To: rtc-linux@googlegroups.com List-ID: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , Dave, I would like to take that patch as part of the series once you have acked it. On 12/06/2015 at 11:10:17 +0800, Xunlei Pang wrote : > From: Xunlei Pang > > On Sparc systems, update_persistent_clock() uses RTC drivers to do > the job, it makes more sense to hand it over to CONFIG_RTC_SYSTOHC. > > In the long run, all the update_persistent_clock() should migrate to > proper class RTC drivers if any and use CONFIG_RTC_SYSTOHC instead. > > Signed-off-by: Xunlei Pang > --- > arch/sparc/Kconfig | 2 +- > arch/sparc/kernel/time_32.c | 21 --------------------- > arch/sparc/kernel/time_64.c | 14 -------------- > 3 files changed, 1 insertion(+), 36 deletions(-) > > diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig > index e49502a..56442d2 100644 > --- a/arch/sparc/Kconfig > +++ b/arch/sparc/Kconfig > @@ -25,6 +25,7 @@ config SPARC > select ARCH_HAS_ATOMIC64_DEC_IF_POSITIVE > select RTC_CLASS > select RTC_DRV_M48T59 > + select RTC_SYSTOHC > select HAVE_DMA_ATTRS > select HAVE_DMA_API_DEBUG > select HAVE_ARCH_JUMP_LABEL if SPARC64 > @@ -35,7 +36,6 @@ config SPARC > select HAVE_BPF_JIT > select HAVE_DEBUG_BUGVERBOSE > select GENERIC_SMP_IDLE_THREAD > - select GENERIC_CMOS_UPDATE > select GENERIC_CLOCKEVENTS > select GENERIC_STRNCPY_FROM_USER > select GENERIC_STRNLEN_USER > diff --git a/arch/sparc/kernel/time_32.c b/arch/sparc/kernel/time_32.c > index 8caf45e..c9692f3 100644 > --- a/arch/sparc/kernel/time_32.c > +++ b/arch/sparc/kernel/time_32.c > @@ -23,7 +23,6 @@ > #include > #include > #include > -#include > #include > #include > #include > @@ -65,8 +64,6 @@ DEFINE_PER_CPU(struct clock_event_device, sparc32_clockevent); > DEFINE_SPINLOCK(rtc_lock); > EXPORT_SYMBOL(rtc_lock); > > -static int set_rtc_mmss(unsigned long); > - > unsigned long profile_pc(struct pt_regs *regs) > { > extern char __copy_user_begin[], __copy_user_end[]; > @@ -87,11 +84,6 @@ EXPORT_SYMBOL(profile_pc); > > volatile u32 __iomem *master_l10_counter; > > -int update_persistent_clock(struct timespec now) > -{ > - return set_rtc_mmss(now.tv_sec); > -} > - > irqreturn_t notrace timer_interrupt(int dummy, void *dev_id) > { > if (timer_cs_enabled) { > @@ -362,16 +354,3 @@ void __init time_init(void) > sbus_time_init(); > } > > - > -static int set_rtc_mmss(unsigned long secs) > -{ > - struct rtc_device *rtc = rtc_class_open("rtc0"); > - int err = -1; > - > - if (rtc) { > - err = rtc_set_mmss(rtc, secs); > - rtc_class_close(rtc); > - } > - > - return err; > -} > diff --git a/arch/sparc/kernel/time_64.c b/arch/sparc/kernel/time_64.c > index edbbeb1..2e6035c 100644 > --- a/arch/sparc/kernel/time_64.c > +++ b/arch/sparc/kernel/time_64.c > @@ -28,7 +28,6 @@ > #include > #include > #include > -#include > #include > #include > #include > @@ -394,19 +393,6 @@ static struct sparc64_tick_ops hbtick_operations __read_mostly = { > > static unsigned long timer_ticks_per_nsec_quotient __read_mostly; > > -int update_persistent_clock(struct timespec now) > -{ > - struct rtc_device *rtc = rtc_class_open("rtc0"); > - int err = -1; > - > - if (rtc) { > - err = rtc_set_mmss(rtc, now.tv_sec); > - rtc_class_close(rtc); > - } > - > - return err; > -} > - > unsigned long cmos_regs; > EXPORT_SYMBOL(cmos_regs); > > -- > 1.9.1 > > -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- -- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/d/optout.