[PATCH v4 00/16] genrtc removal

From: Arnd Bergmann
Date: Mon May 30 2016 - 15:00:28 EST


This is the promised resend of the removal of the genrtc driver and some
related code, now that we are at a good time to merge this for the coming
merge window.

The contents are the same as before, except fixing the build regressions
reported by the 0day builder: on both x86 and mn10300 the rearranging
of the header files caused some missing headers that were included
implicitly before.

Things regarding rtc are a bit intertwined, so I'm mixing patches for
drivers/rtc with drivers/char and arch. This series should make that
better in the end.

This now thoroughly removes the genrtc driver including the asm/rtc.h
headers it uses. For all architectures that still have a meaningful
asm/rtc.h, this goes through two stages:

1) make the rtc-generic implementation independent of asm/rtc.h
2) remove the asm/rtc.h header and disallow the gen_rtc driver

As the last step, the driver itself gets removed. There are a
couple of follow-up cleanup tasks that we could do once this is
all merged, including removing all the remaining nonstandard
RTC drivers from drivers/char:

- alpha most likely doesn't ever use CONFIG_RTC, as it has its
own replacement driver in arch/alpha/kernel/rtc.c, and the legacy
driver is slightly broken on at least one alpha machine

- loongson64 is the only MIPS machine that has the option of running
the legacy CONFIG_RTC driver, but it should also work with rtc-cmos,
and support for that was added in 2010 with 5df74352876 ("MIPS:
Loongson: Define rtc device on MC146818-equipped systems").

- with the above two gone, the only remaining user of CONFIG_RTC
is arch/mn10300, and that just needs to register the appropriate
platform device to use the RTC_CMOS driver instead. I'm sure the
only reason that hasn't happened is because there is very little
active development on this architecture.

- CONFIG_JS_RTC depends on SPARC32, but can not be selected there
because SPARC unconditionally selects RTC_CLASS since 2008, so
this is effectively dead code. Interestingly, this is not even
a separate driver, but actually uses the same code as CONFIG_RTC
with some #ifdef

- CONFIG_EFI_RTC is actively used on all six Itanium configurations,
though the CONFIG_RTC_DRV_EFI replacement driver is also available
there (and has been Itanium-only until ARM64 support for EFI was
merged). I suspect it's just a matter of getting an Ack from the
Itanium maintainers to remove it and change the defconfigs.

- the last remaining legacy RTC driver after those is CONFIG_DS1302,
which has a slightly nonstandard ioctl interface and is only used
on arch/m32r. I have experimented with this a bit and think it
can be converted in a backward-compatible way as well.

- Regarding the users of RTC_DRV_GENERIC, it would be nice to convert
them into hardware-specific drivers that look more like the other
drivers. parisc is particularly easy, it just needs to call
platform_create_bundle() instead of platform_device_register.
The three other architectures have their own abstraction layer
on top of rtc-generic and should kill that off. There are only
a few drivers in each case: m68k has one (q40), sh has two
(dreamcast and sh3) and powerpc has four (rtas, mpc8xx, chrp
and pmac).

- Once all RTC support uses the RTC class subsystem, we should
remove all implementations of read_persistent_clock{,64} that only
access the RTC or return a hardwired time, and we should
remove all update_persistent_clock{,64} users and
CONFIG_GENERIC_CMOS_UPDATE, as both are replaced with
generic rtc code.

Arnd

Cc: geert@xxxxxxxxxxxxxx
Cc: gregkh@xxxxxxxxxxxxxxxxxxx
Cc: deller@xxxxxx
Cc: benh@xxxxxxxxxxxxxxxxxxx
Cc: mpe@xxxxxxxxxxxxxx
Cc: dalias@xxxxxxxx
Cc: dhowells@xxxxxxxxxx
Cc: linux-alpha@xxxxxxxxxxxxxxx
Cc: a.zummo@xxxxxxxxxxxx
Cc: linux-kernel@xxxxxxxxxxxxxxx
Cc: linux-parisc@xxxxxxxxxxxxxxx
Cc: linuxppc-dev@xxxxxxxxxxxxxxxx
Cc: linux-sh@xxxxxxxxxxxxxxx
Cc: linux-m68k@xxxxxxxxxxxxxxxxxxxx
Cc: rtc-linux@xxxxxxxxxxxxxxxx
Cc: linux-arch@xxxxxxxxxxxxxxx

Arnd Bergmann (16):
rtc: cmos: remove empty asm/mc146818rtc.h files
rtc: cmos: move mc146818rtc code out of asm-generic/rtc.h
char/genrtc: x86: remove remnants of asm/rtc.h
rtc: sh: provide rtc_class_ops directly
char/genrtc: remove alpha support
char/genrtc: remove mn10300 support
rtc: parisc: provide rtc_class_ops directly
char/genrtc: remove parisc support
rtc: m68k: provide rtc_class_ops directly
rtc: m68k: provide ioctl for q40
char/genrtc: remove m68k support
rtc: powerpc: provide rtc_class_ops directly
char/genrtc: remove powerpc support
rtc: generic: remove get_rtc_time/set_rtc_time wrappers
char/genrtc: remove asm-generic/rtc.h from mips
char/genrtc: remove the rest of the driver

arch/alpha/include/asm/rtc.h | 1 -
arch/alpha/kernel/core_marvel.c | 1 -
arch/alpha/kernel/rtc.c | 6 +-
arch/frv/include/asm/mc146818rtc.h | 16 -
arch/h8300/include/asm/mc146818rtc.h | 9 -
arch/ia64/include/asm/mc146818rtc.h | 10 -
arch/m68k/amiga/config.c | 1 -
arch/m68k/apollo/config.c | 1 -
arch/m68k/bvme6000/config.c | 1 -
arch/m68k/hp300/config.c | 2 +-
arch/m68k/include/asm/rtc.h | 79 ----
arch/m68k/kernel/time.c | 48 ++-
arch/m68k/mac/config.c | 3 +-
arch/m68k/mac/misc.c | 1 -
arch/m68k/mvme147/config.c | 1 -
arch/m68k/mvme16x/config.c | 1 -
arch/m68k/q40/config.c | 2 +-
arch/m68k/sun3/config.c | 1 -
arch/m68k/sun3/intersil.c | 2 +-
arch/m68k/sun3x/time.c | 2 +-
arch/mips/sni/time.c | 1 -
arch/mn10300/include/asm/rtc.h | 2 -
arch/mn10300/kernel/rtc.c | 2 +-
arch/mn10300/proc-mn103e010/proc-init.c | 1 +
arch/mn10300/proc-mn2ws0050/proc-init.c | 1 +
arch/parisc/include/asm/mc146818rtc.h | 9 -
arch/parisc/include/asm/rtc.h | 131 -------
arch/parisc/kernel/time.c | 36 +-
arch/powerpc/include/asm/rtc.h | 78 ----
arch/powerpc/kernel/time.c | 29 +-
arch/powerpc/platforms/Kconfig | 11 +
arch/powerpc/platforms/ps3/time.c | 2 +-
arch/sh/include/asm/mc146818rtc.h | 7 -
arch/sh/include/asm/rtc.h | 11 -
arch/sh/kernel/time.c | 33 +-
arch/x86/include/asm/mc146818rtc.h | 1 -
arch/x86/include/asm/rtc.h | 1 -
arch/x86/kernel/hpet.c | 3 +-
arch/x86/kernel/rtc.c | 3 +-
arch/x86/platform/efi/efi.c | 1 -
arch/x86/platform/efi/efi_64.c | 1 +
arch/x86/platform/intel-mid/intel_mid_vrtc.c | 1 +
drivers/acpi/acpi_cmos_rtc.c | 2 +-
drivers/base/power/trace.c | 6 +-
drivers/char/Kconfig | 26 --
drivers/char/Makefile | 1 -
drivers/char/genrtc.c | 539 ---------------------------
drivers/rtc/rtc-cmos.c | 12 +-
drivers/rtc/rtc-generic.c | 36 +-
drivers/rtc/rtc-mrst.c | 2 +-
include/asm-generic/rtc.h | 247 ------------
include/linux/mc146818rtc.h | 194 ++++++++++
52 files changed, 362 insertions(+), 1256 deletions(-)
delete mode 100644 arch/alpha/include/asm/rtc.h
delete mode 100644 arch/frv/include/asm/mc146818rtc.h
delete mode 100644 arch/h8300/include/asm/mc146818rtc.h
delete mode 100644 arch/ia64/include/asm/mc146818rtc.h
delete mode 100644 arch/m68k/include/asm/rtc.h
delete mode 100644 arch/parisc/include/asm/mc146818rtc.h
delete mode 100644 arch/parisc/include/asm/rtc.h
delete mode 100644 arch/powerpc/include/asm/rtc.h
delete mode 100644 arch/sh/include/asm/mc146818rtc.h
delete mode 100644 arch/x86/include/asm/rtc.h
delete mode 100644 drivers/char/genrtc.c
delete mode 100644 include/asm-generic/rtc.h

--
2.7.0