[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200703231718.28658.mws@linuxtv.org>
Date: Fri, 23 Mar 2007 17:18:27 +0100
From: Marcel Siegert <mws@...uxtv.org>
To: linux-kernel@...r.kernel.org
Cc: Arjan van de Ven <arjan@...radead.org>,
v4l-dvb-maintainer@...uxtv.org,
"Robert P. J. Day" <rpjday@...dspring.com>,
Andrew Morton <akpm@...l.org>
Subject: Re: [v4l-dvb-maintainer] [PATCH] DVB: Delete unused header file linux/dvb/version.h.
On Friday 23 March 2007, Arjan van de Ven wrote:
> On Fri, 2007-03-23 at 16:47 +0100, Marcel Siegert wrote:
> > On Friday 23 March 2007, Robert P. J. Day wrote:
> > >
> > > Delete the unreferenced header file include/linux/dvb/version.h.
> > >
> > > Signed-off-by: Robert P. J. Day <rpjday@...dspring.com>
> > >
> > NACK
> > this header is unreferenced in kernel, but used for applications that compile for userspace
> > to determine the api version.
>
>
> btw that is something extremely evil to do. Either you have varying apis
> between kernels (which is naughty, but I suppose you can go up in
> capabilities) or it's fully static.
>
> If it's dynamic, doing it compiletime is BAD. The same binaries should
> run on multiple kernels after all. An ioctl or a sysfs thing or
> something is a lot better than a compile time constant.
>
> If it's entirely static exposing a "version" doesn't mean anything and
> it might as well be killed...
>
> Greetings,
> Arjan van de Ven
hi arjan,
we discussed that topic - an IOCTL to determine the api version - on dvb ml in nov.2005 and
got no result. so this is still there and will stay imho, until we will find a solution.
regards
marcel
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists