Re: [2.6 patch] small input cleanup

[Vojtech Pavlik]

On Sun, Nov 07, 2004 at 06:29:29PM +0100, Adrian Bunk wrote:
> On Sat, Nov 06, 2004 at 10:49:54PM -0500, Dmitry Torokhov wrote:
> 
> > Hi,
> 
> Hi Dmitry,
> 
> > On Saturday 06 November 2004 10:12 pm, Adrian Bunk wrote:
> > > The patch below does the following cleanups under drivers/input/ :
> > > - make some needlessly global code static
> > > - remove the completely unused EXPORT_SYMBOL'ed function gameport_rescan
> > 
> > It will be used (but in some transformed) once I finish gameport sysfs
> > support, but it probably need not be exported.
> >  
> > > - make the EXPORT_SYMBOL'ed function ps2_sendbyte static since it isn't
> > >   used outside the file where it's defined
> > 
> > libps2 is a library for communicating with standard PS/2 device and while
> > the function is not currently used it is part of the interface. I would
> > like to leave the function as is.
> 
> my personal opinions:
> - if gameport_rescan will not be needed in it's current form, there's
>   no need for it (you can always add the "real" function when it's 
>   required

Well, it works in its current form, and drivers should call it when
their reinit logic fails to reinitialize the device. They don't, which
is a bug, and should be fixed. I don't think removing gameport_rescan()
will help fixing them.

> - could ps2_sendbyte be #ifdef 0'ed until it's required?
>   this way, it wouldn't make the kernel bigger today
 
It is used, just not outside libps2. Does the EXPORT_SYMBOL() make the
kernel so much bigger?

-- 
Vojtech Pavlik
SuSE Labs, SuSE CR
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/