[linux-dvb] Re: Question about dvb_frontend.c

[Johannes Stezenbach <js@convergence.de>]

Christian Gmeiner wrote:
> > Christian Gmeiner wrote:
> > >
> > > cat /proc/dvb/adapter0
> > > ----- Adapter Info -----
> > > DVB card: KNC1 DVB-S
> > > Frontend: STV0299/TSA5059/SL1935 based
> > >
> > > ----- Devices -----
> > > ~ frontend0
> > > ~ net0
> > > ~ dvr0
> > > ~ demux0
> >
> > I'm not sure that I like this.
> >
> > Who is going to use /proc/dvb and for what purpose?
> Maybe nobody is using this, but it was writen by me for testing. As you
> mabe know the whole patch adds sysfs support to the dvb-core. So
> i have coded the small dvb_procfs to check if my concepts are working or
> not.
> But i have asked some other guys and the told me that the proc is an nice
> idea
> to get very quickly some informations about adapters, so i wont kill procfs
> support.
> 
> > You are creating an additional API that people are
> > going to use in scripts/programs, so it better be
> > well thought out, so it won't change all the time.
> It is not an addinal API! I havent changed the interface of the dvb-core.

A /proc entry *is* an API. It should be well designed, and not just
created because it is easy to do.

That's why I asked who will use it and for what purpose. You
cannot design /proc/dvb if you cannot answer these questions.


Regards,
Johannes


-- 
Info:
To unsubscribe send a mail to ecartis@linuxtv.org with "unsubscribe linux-dvb" as subject.