Re: [PATCH 1/1] V4L: stk11xx, add a new webcam driver

From: Ismail Dönmez
Date: Thu May 24 2007 - 14:00:06 EST


On Thursday 24 May 2007 20:38:05 Diego Calleja wrote:
> El Thu, 24 May 2007 16:01:33 +0200 (CEST), Jiri Slaby <jirislaby@xxxxxxxxx>
escribió:
> > +config USB_STK11XX
> > + tristate "STK11XX based webcams"
> > + depends on VIDEO_V4L2
> > + ---help---
> > + This will add support for Syntek webcams such as dc1125 and stk1135.
> > +
> > + If you choose to build it as module, it will be called stk11xx.
>
> Maybe this is a too picky requeriment, but IMO it would be nice if the
> module would be called "camera_stk11xx", or had any other prefix that gives
> some meaning about what is doing. It's hard to decipher from the name when
> you run lsmod.

That would be too picky indeed since other camera modules have no camera_
prefix like pwc,spca5xx, etc.

Regards,
ismail

--
Perfect is the enemy of good
-
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/