From 7d9f99eccc8f94ace31030a2a7ff73cf5f8c12a0 Mon Sep 17 00:00:00 2001 From: David Brownell Date: Tue, 8 May 2007 00:33:38 -0700 Subject: [PATCH] rtc: simplified /proc/driver/rtc handling This simplifies the RTC procfs support by removing the class_interface that hooks it into the rtc core. If it's configured, then sysfs support is now part of the RTC core, and is never a separate module. It also removes the class_interface hook, now that its last remaining user is gone. (That API is usable only with a "struct class_device".) It's another step towards being able to remove "struct class_device". Signed-off-by: David Brownell Acked-by: Greg Kroah-Hartman Acked-By: Alessandro Zummo Signed-off-by: Andrew Morton Signed-off-by: Linus Torvalds --- drivers/rtc/Kconfig | 2 +- drivers/rtc/Makefile | 3 +-- drivers/rtc/class.c | 9 ++------ drivers/rtc/rtc-core.h | 14 ++++++++++-- drivers/rtc/rtc-proc.c | 51 ++++-------------------------------------- 5 files changed, 20 insertions(+), 59 deletions(-) diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig index d738725d4a..ec33ee8773 100644 --- a/drivers/rtc/Kconfig +++ b/drivers/rtc/Kconfig @@ -59,7 +59,7 @@ config RTC_INTF_SYSFS will be called rtc-sysfs. config RTC_INTF_PROC - tristate "proc" + boolean "proc" depends on RTC_CLASS && PROC_FS default RTC_CLASS help diff --git a/drivers/rtc/Makefile b/drivers/rtc/Makefile index 37bb9ceba2..a1afbc2360 100644 --- a/drivers/rtc/Makefile +++ b/drivers/rtc/Makefile @@ -12,10 +12,9 @@ obj-$(CONFIG_RTC_CLASS) += rtc-core.o rtc-core-y := class.o interface.o rtc-core-$(CONFIG_RTC_INTF_DEV) += rtc-dev.o +rtc-core-$(CONFIG_RTC_INTF_PROC) += rtc-proc.o rtc-core-$(CONFIG_RTC_INTF_SYSFS) += rtc-sysfs.o -obj-$(CONFIG_RTC_INTF_PROC) += rtc-proc.o - obj-$(CONFIG_RTC_DRV_CMOS) += rtc-cmos.o obj-$(CONFIG_RTC_DRV_X1205) += rtc-x1205.o obj-$(CONFIG_RTC_DRV_ISL1208) += rtc-isl1208.o diff --git a/drivers/rtc/class.c b/drivers/rtc/class.c index 5845e6e9b5..9230001bd5 100644 --- a/drivers/rtc/class.c +++ b/drivers/rtc/class.c @@ -90,6 +90,7 @@ struct rtc_device *rtc_device_register(const char *name, struct device *dev, rtc_dev_add_device(rtc); rtc_sysfs_add_device(rtc); + rtc_proc_add_device(rtc); dev_info(dev, "rtc core: registered %s as %s\n", rtc->name, rtc->class_dev.class_id); @@ -126,6 +127,7 @@ void rtc_device_unregister(struct rtc_device *rtc) */ rtc_sysfs_del_device(rtc); rtc_dev_del_device(rtc); + rtc_proc_del_device(rtc); class_device_unregister(&rtc->class_dev); rtc->ops = NULL; mutex_unlock(&rtc->ops_lock); @@ -134,13 +136,6 @@ void rtc_device_unregister(struct rtc_device *rtc) } EXPORT_SYMBOL_GPL(rtc_device_unregister); -int rtc_interface_register(struct class_interface *intf) -{ - intf->class = rtc_class; - return class_interface_register(intf); -} -EXPORT_SYMBOL_GPL(rtc_interface_register); - static int __init rtc_init(void) { rtc_class = class_create(THIS_MODULE, "rtc"); diff --git a/drivers/rtc/rtc-core.h b/drivers/rtc/rtc-core.h index 30bcd14377..4675a9fd5b 100644 --- a/drivers/rtc/rtc-core.h +++ b/drivers/rtc/rtc-core.h @@ -1,5 +1,3 @@ -extern int rtc_interface_register(struct class_interface *intf); - #ifdef CONFIG_RTC_INTF_DEV extern void __init rtc_dev_init(void); @@ -16,6 +14,18 @@ extern void rtc_dev_del_device(struct rtc_device *rtc); #endif +#ifdef CONFIG_RTC_INTF_PROC + +void rtc_proc_add_device(struct rtc_device *rtc); +void rtc_proc_del_device(struct rtc_device *rtc); + +#else + +#define rtc_proc_add_device(r) do{}while(0) +#define rtc_proc_del_device(r) do{}while(0) + +#endif + #ifdef CONFIG_RTC_INTF_SYSFS extern void __init rtc_sysfs_init(struct class *); diff --git a/drivers/rtc/rtc-proc.c b/drivers/rtc/rtc-proc.c index 195be6587b..3d7f4547c7 100644 --- a/drivers/rtc/rtc-proc.c +++ b/drivers/rtc/rtc-proc.c @@ -19,9 +19,6 @@ #include "rtc-core.h" -static struct class_device *rtc_dev = NULL; -static DEFINE_MUTEX(rtc_lock); - static int rtc_proc_show(struct seq_file *seq, void *offset) { int err; @@ -106,62 +103,22 @@ static const struct file_operations rtc_proc_fops = { .release = rtc_proc_release, }; -static int rtc_proc_add_device(struct class_device *class_dev, - struct class_interface *class_intf) +void rtc_proc_add_device(struct rtc_device *rtc) { - mutex_lock(&rtc_lock); - if (rtc_dev == NULL) { + if (rtc->id == 0) { struct proc_dir_entry *ent; - rtc_dev = class_dev; - ent = create_proc_entry("driver/rtc", 0, NULL); if (ent) { - struct rtc_device *rtc = to_rtc_device(class_dev); - ent->proc_fops = &rtc_proc_fops; ent->owner = rtc->owner; ent->data = rtc; - - dev_dbg(class_dev->dev, "rtc intf: proc\n"); } - else - rtc_dev = NULL; } - mutex_unlock(&rtc_lock); - - return 0; } -static void rtc_proc_remove_device(struct class_device *class_dev, - struct class_interface *class_intf) +void rtc_proc_del_device(struct rtc_device *rtc) { - mutex_lock(&rtc_lock); - if (rtc_dev == class_dev) { + if (rtc->id == 0) remove_proc_entry("driver/rtc", NULL); - rtc_dev = NULL; - } - mutex_unlock(&rtc_lock); -} - -static struct class_interface rtc_proc_interface = { - .add = &rtc_proc_add_device, - .remove = &rtc_proc_remove_device, -}; - -static int __init rtc_proc_init(void) -{ - return rtc_interface_register(&rtc_proc_interface); } - -static void __exit rtc_proc_exit(void) -{ - class_interface_unregister(&rtc_proc_interface); -} - -subsys_initcall(rtc_proc_init); -module_exit(rtc_proc_exit); - -MODULE_AUTHOR("Alessandro Zummo "); -MODULE_DESCRIPTION("RTC class proc interface"); -MODULE_LICENSE("GPL"); -- 2.39.5