On Monday, April 30, 2012 01:51:20 Andy Walls wrote: > On Sun, 2012-04-29 at 13:57 +0200, Hans Verkuil wrote: > > Hi all, > > > > I managed to get hold of a cpia2-based Hanse USB microscope almost a year > > ago after reports from Andrea that the driver didn't work properly. > > > > I finally had time to take a really good look at the driver and it was > > broken in many respects. This patch brings the driver up to speed with > > respect to the v4l2 framework and it now works as expected, including > > disconnect handling and suspend/resume handling. > > > > The only thing left to do some day is to convert it to the videobuf2 > > framework. > > Hans, > > cpia2_s_ctrl() doesn't seem to have a case for V4L2_CID_ILLUMINATORS_2. > Does control cluster magic take care of that? Correct, the two illuminator controls are clustered so only the 'master control' (ILLUMINATORS_1) shows up in the switch. Regards, Hans > > Regards, > Andy > > > Regards, > > > > Hans > > > > The following changes since commit bcb2cf6e0bf033d79821c89e5ccb328bfbd44907: > > > > [media] ngene: remove an unneeded condition (2012-04-26 15:29:23 -0300) > > > > are available in the git repository at: > > > > git://linuxtv.org/hverkuil/media_tree.git cpia2 > > > > for you to fetch changes up to 14a3d232eb5d25c768f40fbd6a87db48a249a391: > > > > cpia2: major overhaul to get it in a working state again. (2012-04-29 13:44:44 +0200) > > > > ---------------------------------------------------------------- > > Hans Verkuil (1): > > cpia2: major overhaul to get it in a working state again. > > > > drivers/media/video/cpia2/cpia2.h | 34 ++- > > drivers/media/video/cpia2/cpia2_core.c | 142 +++--------- > > drivers/media/video/cpia2/cpia2_usb.c | 78 +++++-- > > drivers/media/video/cpia2/cpia2_v4l.c | 846 +++++++++++++++++++++------------------------------------------------ > > drivers/media/video/cpia2/cpia2dev.h | 50 ----- > > 5 files changed, 363 insertions(+), 787 deletions(-) > > delete mode 100644 drivers/media/video/cpia2/cpia2dev.h > > -- > > 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 > > > -- > 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 > -- 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
