Re: [PATCH 0/2] gspca sn9c20x subdriver rev2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
shouldn't there be a check when allocating jpg_hdr? or i am missing something?


 sd->jpeg_hdr = kmalloc(JPEG_HDR_SZ, GFP_KERNEL);
 jpeg_define(sd->jpeg_hdr, height, width,  0x21);
 jpeg_set_qual(sd->jpeg_hdr, sd->quality);

Best Regards.
Nice to see microdia project here :)


On Sat, Jul 18, 2009 at 11:19 AM, Brian Johnson<brijohn@xxxxxxxxx> wrote:
>> - in the 1st patch, in gspca.c, why is the get_chip_ident check needed
>>   in vidioc_g_register / vidioc_s_register)?
>>
> According the the current v4l2 spec regarding the set and get
> register ioctls drivers that support them must also support the get
> chip ident ioctl. As such in the vidioc_s/g_register ioctls I do the
> check to ensure that the driver also defines the get_chip_ident ioctl.
>
>
> Brian Johnson
> --
> 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

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux