Messages in this thread | | | From | Dmitry Torokhov <> | Subject | Re: [PATCH 5/8] RTC subsystem, dev interface | Date | Mon, 9 Jan 2006 01:39:29 -0500 |
| |
On Sunday 08 January 2006 22:12, Alessandro Zummo wrote: > > > +/* interface registration */ > > > + > > > +struct class_interface rtc_dev_interface = { > > > + .add = &rtc_dev_add_device, > > > + .remove = &rtc_dev_remove_device, > > > +}; > > > + > > > > I wonder if doing rtc dev as a class device interface is a good idea. > > It may be cleaner to fold it into the core. > > What the code implements is actually an interface, so this should > be the riht place. It is also fully optional, everything could work > without it. Probably the interface implementation hasn't all the primitives > to handle this kind of work, but I'm not willing to go into that right now ;) >
Yes, it is an interface. What I am trying to say - is it a main interface? What is the preferred, most efficient way to interface with RTC? If it is through this interface it may make sence to fold it into the core. Otherwise do what input layer does and have interface create another class device which reprsesents your /dev node.
-- Dmitry - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |