Re: [PATCH 5/8] RTC subsystem, dev interface

[Dmitry Torokhov <dtor_core@ameritech.net>]

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