On Sun, 2010-09-12 at 17:28 -0400, Andy Walls wrote:
> On Sun, 2010-09-12 at 17:12 -0400, Andy Walls wrote:
> > On Sun, 2010-09-12 at 22:26 +0200, Hans Verkuil wrote:
> > 
> > > And other news on the V4L1 front:
> > 
> > > I'm waiting for test results on the cpia2 driver. If it works, then the 
> > > V4L1
> > > support can be removed from that driver as well.
> > 


> In the cpia2 driver:
> 
>        /* CPIA2 extension to Video4Linux API */
>         case CPIA2_IOC_SET_GPIO:
>                 retval = ioctl_set_gpio(arg, cam);
>                 break;
> 
> Yuck.

And another gem in the cpia2 driver, this V4L2 control:


        {
                .id            = CPIA2_CID_GPIO,
                .type          = V4L2_CTRL_TYPE_INTEGER,
                .name          = "GPIO",
                .minimum       = 0,
                .maximum       = 255,
                .step          = 1,
                .default_value = 0,
        },

Give me a GUI with a slider for that control, and I'm sure I can fry a
camera.

That should be removed.

Regards,
Andy


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to