From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: char: legacy RTC cleanups Date: Wed, 27 Apr 2016 10:33:04 +0200 Message-ID: <6883809.SRgmlLMdyd@wuerfel> References: <1461707052-1337718-1-git-send-email-arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: linux-alpha-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , Tony Luck , "James E.J. Bottomley" , Helge Deller , Benjamin Herrenschmidt , Michael Ellerman , Rich Felker , David Howells , Koichi Yasutake , Richard Henderson , Ivan Kokshaysky , alpha , the arch/x86 maintainers , Alessandro Zummo , Alexandre Belloni , "linux-kernel@vger.kernel.org" , "linux-ia64@vger.kernel.org" , Parisc List , "linuxppc-dev@lists.ozlabs.org" Linux-sh list List-Id: linux-arch.vger.kernel.org On Wednesday 27 April 2016 09:54:41 Geert Uytterhoeven wrote: > Hi Arnd, > > On Tue, Apr 26, 2016 at 11:44 PM, Arnd Bergmann wrote: > > For the genrtc driver, rearranging the headers makes it simpler > > to use and reduces duplication. In case of alpha and mn10300, > > I've shown that the genrtc and rtc drivers are doing the same > > thing, so we don't need them both. The remaining three > > architectures (m68k, parisc, powerpc) actually all support > > the newer rtc-generic driver, so we could remove genrtc completely > > if we want to. > > CONFIG_GEN_RTC is not enabled in any of the m68k defconfigs, so I think genrtc > has been unused for a while. > All defconfigs either use CONFIG_RTC_DRV_GENERIC, or enable a more specific > RTC driver. Ok, good to know. I'm guessing the same is true for parisc, but there are also very few users. Regarding the Q40 specific ioctls, what do you think this means, is it a) nobody uses Q40 with modern kernels, b) nobody calls RTC_PLL_GET/RTC_PLL_SET on q40, or c) Q40 users have their own configurations and enable GEN_RTC? On powerpc, a quarter of the defconfigs (mostly for really old hardware) still use GEN_RTC, but I guess we can either bulk-convert them to RTC_GENERIC, or convert the five implementations of .get_rtc_time/.set_rtc_time (8xx, rtas, chrp, powermac, maple) into five regular RTC class drivers. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mout.kundenserver.de ([217.72.192.73]:53390 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752383AbcD0IeS (ORCPT ); Wed, 27 Apr 2016 04:34:18 -0400 From: Arnd Bergmann Subject: Re: char: legacy RTC cleanups Date: Wed, 27 Apr 2016 10:33:04 +0200 Message-ID: <6883809.SRgmlLMdyd@wuerfel> In-Reply-To: References: <1461707052-1337718-1-git-send-email-arnd@arndb.de> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-arch-owner@vger.kernel.org List-ID: To: Geert Uytterhoeven Cc: Greg Kroah-Hartman , Tony Luck , "James E.J. Bottomley" , Helge Deller , Benjamin Herrenschmidt , Michael Ellerman , Rich Felker , David Howells , Koichi Yasutake , Richard Henderson , Ivan Kokshaysky , alpha , the arch/x86 maintainers , Alessandro Zummo , Alexandre Belloni , "linux-kernel@vger.kernel.org" , "linux-ia64@vger.kernel.org" , Parisc List , "linuxppc-dev@lists.ozlabs.org" , Linux-sh list , RTCLINUX , Linux-Arch Message-ID: <20160427083304.jE-jQMie1xoPDq9hAIIviwCiOcVf607bJ3oYvQ3SQaY@z> On Wednesday 27 April 2016 09:54:41 Geert Uytterhoeven wrote: > Hi Arnd, > > On Tue, Apr 26, 2016 at 11:44 PM, Arnd Bergmann wrote: > > For the genrtc driver, rearranging the headers makes it simpler > > to use and reduces duplication. In case of alpha and mn10300, > > I've shown that the genrtc and rtc drivers are doing the same > > thing, so we don't need them both. The remaining three > > architectures (m68k, parisc, powerpc) actually all support > > the newer rtc-generic driver, so we could remove genrtc completely > > if we want to. > > CONFIG_GEN_RTC is not enabled in any of the m68k defconfigs, so I think genrtc > has been unused for a while. > All defconfigs either use CONFIG_RTC_DRV_GENERIC, or enable a more specific > RTC driver. Ok, good to know. I'm guessing the same is true for parisc, but there are also very few users. Regarding the Q40 specific ioctls, what do you think this means, is it a) nobody uses Q40 with modern kernels, b) nobody calls RTC_PLL_GET/RTC_PLL_SET on q40, or c) Q40 users have their own configurations and enable GEN_RTC? On powerpc, a quarter of the defconfigs (mostly for really old hardware) still use GEN_RTC, but I guess we can either bulk-convert them to RTC_GENERIC, or convert the five implementations of .get_rtc_time/.set_rtc_time (8xx, rtas, chrp, powermac, maple) into five regular RTC class drivers. Arnd