#
3610f58b |
| 25-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: make sure the packet size is >= 4 before checking for headers in em28xx_urb_data_copy_vbi() Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mau
[media] em28xx: make sure the packet size is >= 4 before checking for headers in em28xx_urb_data_copy_vbi() Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
b77e0c08 |
| 25-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: fix video data start position calculation in em28xx_urb_data_copy_vbi() The header check/removal code at the end of function em28xx_urb_data_copy_vbi() is obsolete, becau
[media] em28xx: fix video data start position calculation in em28xx_urb_data_copy_vbi() The header check/removal code at the end of function em28xx_urb_data_copy_vbi() is obsolete, because this is already done earlier in this function. In fact it is incomplete (doesn't check for vbi header) and causes trouble when the first data bytes are the same as header bytes (which is fortunately very unlikely). Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
c647a91a |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: improve USB endpoint logic, also use bulk transfers The current enpoint logic ignores all bulk endpoints and uses a fixed mapping between endpint addresses and the suppor
[media] em28xx: improve USB endpoint logic, also use bulk transfers The current enpoint logic ignores all bulk endpoints and uses a fixed mapping between endpint addresses and the supported data stream types (analog/audio/DVB): Ep 0x82, isoc => analog Ep 0x83, isoc => audio Ep 0x84, isoc => DVB Now that the code can also do bulk transfers, the endpoint logic has to be extended to also consider bulk endpoints. The new logic preserves backwards compatibility and reflects the endpoint configurations we have seen so far: Ep 0x82, isoc => analog Ep 0x82, bulk => analog Ep 0x83, isoc* => audio Ep 0x84, isoc => digital Ep 0x84, bulk => analog or digital** (*: audio should always be isoc) (**: analog, if ep 0x82 is isoc, otherwise digital) [mchehab@redhat.com: Fix a CodingStyle issue: don't break strings into separate lines] Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
0cf544a6 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename some USB parameter fields in struct em28xx to clarify their role Also improve the comments. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Si
[media] em28xx: rename some USB parameter fields in struct em28xx to clarify their role Also improve the comments. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
4601cc39 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename function em28xx_isoc_copy_vbi and extend for USB bulk transfers The URB data processing for bulk transfers is very similar to what is done with isoc transfers, so
[media] em28xx: rename function em28xx_isoc_copy_vbi and extend for USB bulk transfers The URB data processing for bulk transfers is very similar to what is done with isoc transfers, so create a common function that works with both transfer types based on the existing isoc function. [mchehab@redhat.com: Fix a CodingStyle issue: don't break strings into separate lines] Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
0fa4a402 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename function em28xx_isoc_copy and extend for USB bulk transfers The URB data processing for bulk transfers is very similar to what is done with isoc transfers, so crea
[media] em28xx: rename function em28xx_isoc_copy and extend for USB bulk transfers The URB data processing for bulk transfers is very similar to what is done with isoc transfers, so create a common function that works with both transfer types based on the existing isoc function. [mchehab@redhat.com: Fix a CodingStyle issue: don't break strings into separate lines] Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
1653cb0c |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: remove double checks for urb->status == -ENOENT in urb_data_copy functions This check is already done in the URB handler em28xx_irq_callback before calling these function
[media] em28xx: remove double checks for urb->status == -ENOENT in urb_data_copy functions This check is already done in the URB handler em28xx_irq_callback before calling these functions. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
057ca0da |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: create a common function for isoc and bulk USB transfer initialization - rename em28xx_init_isoc to em28xx_init_usb_xfer - add parameter for isoc/bulk transfer selection
[media] em28xx: create a common function for isoc and bulk USB transfer initialization - rename em28xx_init_isoc to em28xx_init_usb_xfer - add parameter for isoc/bulk transfer selection which is passed to em28xx_alloc_urbs - rename local variable isoc_buf to usb_bufs Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
afb177e0 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename function em28xx_uninit_isoc to em28xx_uninit_usb_xfer This function will be used to uninitialize USB bulk transfers, too. Also rename the local variable isoc_bufs
[media] em28xx: rename function em28xx_uninit_isoc to em28xx_uninit_usb_xfer This function will be used to uninitialize USB bulk transfers, too. Also rename the local variable isoc_bufs to usb_bufs. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
74209dc0 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename struct em28xx_usb_isoc_ctl to em28xx_usb_ctl Also rename the corresponding field isoc_ctl in struct em28xx to usb_ctl. We will use this struct for USB bulk tra
[media] em28xx: rename struct em28xx_usb_isoc_ctl to em28xx_usb_ctl Also rename the corresponding field isoc_ctl in struct em28xx to usb_ctl. We will use this struct for USB bulk transfers, too. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
515688a8 |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: rename isoc packet number constants and parameters Rename EM28XX_NUM_PACKETS to EM28XX_NUM_ISOC_PACKETS and EM28XX_DVB_MAX_PACKETS to EM28XX_DVB_NUM_ISOC_PACKETS to c
[media] em28xx: rename isoc packet number constants and parameters Rename EM28XX_NUM_PACKETS to EM28XX_NUM_ISOC_PACKETS and EM28XX_DVB_MAX_PACKETS to EM28XX_DVB_NUM_ISOC_PACKETS to clarify that these values are used only for isoc usb transfers. Also use the term num_packets instead of max_packets, as this is how these values are used and called in struct urb. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
c02ec71b |
| 08-Nov-2012 |
Frank Schaefer <fschaefer.oss@googlemail.com> |
[media] em28xx: fix wrong data offset for non-interlaced mode in em28xx_copy_video em28xx_copy_video uses a wrong offset for the target buffer when copying the data from an USB isoc pack
[media] em28xx: fix wrong data offset for non-interlaced mode in em28xx_copy_video em28xx_copy_video uses a wrong offset for the target buffer when copying the data from an USB isoc packet. This happens only for the second and all following lines in the packet. The reason why this bug doesn't cause image corruption with my test device (SilverCrest Webcam 1.3 MPix) is, that this device never sends any packets that cross the end of a line. I don't know if all devices behave like this, so this patch should be considered for stable. With the upcoming patches to add support for USB bulk transfers, em28xx_copy_video will be called once per URB, which will always trigger this bug. Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
8e6057b5 |
| 15-Sep-2012 |
Sakari Ailus <sakari.ailus@iki.fi> |
[media] v4l: Convert drivers to use monotonic timestamps Convert drivers using wall clock time (CLOCK_REALTIME) to timestamp from the monotonic timer (CLOCK_MONOTONIC). Signed-o
[media] v4l: Convert drivers to use monotonic timestamps Convert drivers using wall clock time (CLOCK_REALTIME) to timestamp from the monotonic timer (CLOCK_MONOTONIC). Signed-off-by: Sakari Ailus <sakari.ailus@iki.fi> Acked-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
0e8025b9 |
| 04-Sep-2012 |
Hans Verkuil <hans.verkuil@cisco.com> |
[media] v4l2: make vidioc_s_audio const Write-only ioctls should have a const argument in the ioctl op. Do this conversion for vidioc_s_audio. Adding const for write-only ioctls was
[media] v4l2: make vidioc_s_audio const Write-only ioctls should have a const argument in the ioctl op. Do this conversion for vidioc_s_audio. Adding const for write-only ioctls was decided during the 2012 Media Workshop. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
Revision tags: v3.6-rc4, v3.6-rc3, v3.6-rc2 |
|
#
e36c92fd |
| 14-Aug-2012 |
Dan Carpenter <dan.carpenter@oracle.com> |
[media] em28xx: use after free in em28xx_v4l2_close() We need to move the unlock before the kfree(dev); Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Ma
[media] em28xx: use after free in em28xx_v4l2_close() We need to move the unlock before the kfree(dev); Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|
#
0c0d06ca |
| 13-Aug-2012 |
Mauro Carvalho Chehab <mchehab@redhat.com> |
[media] rename most media/video usb drivers to media/usb Rename all USB drivers with their own directory under drivers/media/video into drivers/media/usb and update the building syst
[media] rename most media/video usb drivers to media/usb Rename all USB drivers with their own directory under drivers/media/video into drivers/media/usb and update the building system. Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
show more ...
|