On Thu, 02 Sep 2010 21:09:42 -0400 Andy Walls <awalls@xxxxxxxxxxxxxxxx> wrote: [snip] > Add a v4l2 control to get the lamp control code working for the Intel > Play QX3 microscope. My daughter in middle school thought it was > cool, and is now examining the grossest specimens she can find. [snip] > - u8 toplight; /* top light lit , R/W */ > - u8 bottomlight; /* bottom light lit, R/W */ > + u8 toplamp; /* top lamp lit , R/W */ > + u8 bottomlamp; /* bottom lamp lit, R/W */ [snip] > +#define V4L2_CID_LAMPS (V4L2_CID_PRIVATE_BASE+1) [snip] Hi Andy, First, I do not see why you changed the name 'light' to 'lamp' while 'light' is used in the other cpia driver (cpia2). Then, you used a private control ID, and linux-media people don't like that. As many gspca users are waiting for a light/LED/illuminator/lamp control, I tried to define a standard one in March 2009: http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/3095 A second, but more restrictive, attempt was done by Németh Márton in February 2010: http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/16705 The main objection to that proposals was that the sysfs LED interface should be used instead: http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/3114 A patch in this way was done by Németh Márton in February 2010: http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/16670 but it was rather complex, and there was no consensus http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/17111 So, I don't think that your patch could be accepted... Best regards. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
