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

["Christian Gmeiner" <christian@visual-page.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.
Ok, then it is an API, bu who says that is not well designed? In my patch,
which
i will release at the weekend - i hope - there will be only basic API's for
the kernel-filesystems, because nobody has answerd my previous mails with
questions
about exactly this stuff.
But now i have somebody, who can help me to design the API's :)

The cause why i am writing this patch, is that i want/need sysfs support.

> That's why I asked who will use it and for what purpose. You
> cannot design /proc/dvb if you cannot answer these questions.
As i wrote before, i have asked guys about the /proc API and
they told me, that is a good thing to get some basic infos about
an adapter quickly.

Christian




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