1. 04 11月, 2014 1 次提交
  2. 17 7月, 2014 1 次提交
    • L
      [media] v4l: Support extending the v4l2_pix_format structure · d52e2381
      Laurent Pinchart 提交于
      The v4l2_pix_format structure has no reserved field. It is embedded in
      the v4l2_framebuffer structure which has no reserved fields either, and
      in the v4l2_format structure which has reserved fields that were not
      previously required to be zeroed out by applications.
      
      To allow extending v4l2_pix_format, inline it in the v4l2_framebuffer
      structure, and use the priv field as a magic value to indicate that the
      application has set all v4l2_pix_format extended fields and zeroed all
      reserved fields following the v4l2_pix_format field in the v4l2_format
      structure.
      
      The availability of this API extension is reported to userspace through
      the new V4L2_CAP_EXT_PIX_FORMAT capability flag. Just checking that the
      priv field is still set to the magic value at [GS]_FMT return wouldn't
      be enough, as older kernels don't zero the priv field on return.
      
      To simplify the internal API towards drivers zero the extended fields
      and set the priv field to the magic value for applications not aware of
      the extensions.
      Signed-off-by: NLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
      Signed-off-by: NMauro Carvalho Chehab <m.chehab@samsung.com>
      d52e2381
  3. 05 7月, 2014 1 次提交
  4. 25 5月, 2014 1 次提交
  5. 17 6月, 2013 1 次提交
  6. 24 3月, 2013 1 次提交
  7. 06 3月, 2013 8 次提交
  8. 21 12月, 2012 1 次提交
  9. 16 8月, 2012 1 次提交
  10. 31 7月, 2012 1 次提交
  11. 15 5月, 2012 1 次提交
    • H
      [media] cx231xx: fix compiler warnings · da983503
      Hans Verkuil 提交于
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c: In function ‘snd_cx231xx_hw_capture_params’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:527:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:31: warning: variable ‘format’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:25: warning: variable ‘rate’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:526:15: warning: variable ‘channels’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c: In function ‘snd_cx231xx_capture_trigger’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-audio.c:589:6: warning: variable ‘retval’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_isoc_copy’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:331:17: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_bulk_copy’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:434:17: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c: In function ‘cx231xx_reset_video_buffer’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-video.c:704:7: warning: variable ‘outp’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_set_mode’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:699:6: warning: variable ‘errCode’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_ep5_bulkout’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:763:6: warning: variable ‘errCode’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_isoc_irq_callback’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:800:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_bulk_irq_callback’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:846:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_stop_TS1’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:1234:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c: In function ‘cx231xx_start_TS1’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-core.c:1254:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_enable656’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:937:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_disable656’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:955:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_dump_HH_reg’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1323:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_dump_SC_reg’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1358:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_Setup_AFE_for_LowIF’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1444:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_set_Colibri_For_LowIF’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1504:5: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_set_DIF_bandpass’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:1559:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c: In function ‘cx231xx_gpio_i2c_write’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-avcore.c:3093:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c: In function ‘cx231xx_initialize_codec’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c:1098:9: warning: variable ‘data’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c: In function ‘vidioc_streamon’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-417.c:1795:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c: In function ‘cx231xx_isoc_vbi_copy’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c:86:25: warning: variable ‘buf’ set but not used [-Wunused-but-set-variable]
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c: In function ‘cx231xx_irq_vbi_callback’:
      v4l-dvb-git/drivers/media/video/cx231xx/cx231xx-vbi.c:313:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
      Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      da983503
  12. 24 1月, 2012 1 次提交
    • H
      [media] V4L2: Add per-device-node capabilities · 583aa3a9
      Hans Verkuil 提交于
      If V4L2_CAP_DEVICE_CAPS is set, then the new device_caps field is filled with
      the capabilities of the opened device node.
      
      The capabilities field traditionally contains the capabilities of the physical
      device, being a superset of all capabilities available at the several device
      nodes. E.g., if you open /dev/video0, then if it contains VBI caps then that means
      that there is a corresponding vbi node as well. And the capabilities field of
      both the video and vbi nodes should contain identical caps.
      
      However, it would be very useful to also have a capabilities field that contains
      just the caps for the currently open device, hence the new CAP bit and field.
      Signed-off-by: NHans Verkuil <hans.verkuil@cisco.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      583aa3a9
  13. 22 3月, 2011 1 次提交
  14. 29 12月, 2010 1 次提交
  15. 09 11月, 2010 1 次提交
    • A
      [media] v4l: kill the BKL · 0edf2e5e
      Arnd Bergmann 提交于
      All of the hard problems for BKL removal appear to be solved in the
      v4l-dvb/master tree. This removes the BKL from the various open
      functions that do not need it, or only use it to protect an
      open count.
      
      The zoran driver is nontrivial in this regard, so I introduce
      a new mutex that locks both the open/release and the ioctl
      functions. Someone with access to the hardware can probably
      improve that by using the existing lock in all cases.
      
      Finally, all drivers that still use the locked version of the
      ioctl function now get called under a new mutex instead of
      the BKL.
      Signed-off-by: NArnd Bergmann <arnd@arndb.de>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      0edf2e5e
  16. 21 10月, 2010 8 次提交
    • S
      [media] v4l-dvb: using vmalloc needs vmalloc.h in cx231xx-417.c · 94399431
      Stephen Rothwell 提交于
      Fixes these build errors and warnings:
      
      drivers/media/video/cx231xx/cx231xx-417.c: In function 'cx231xx_load_firmware':
      drivers/media/video/cx231xx/cx231xx-417.c:943: error: implicit declaration of function 'vmalloc'
      drivers/media/video/cx231xx/cx231xx-417.c:943: warning: cast to pointer from integer of different size
      drivers/media/video/cx231xx/cx231xx-417.c:950: warning: cast to pointer from integer of different size
      drivers/media/video/cx231xx/cx231xx-417.c:1039: error: implicit declaration of function 'vfree'
      Signed-off-by: NStephen Rothwell <sfr@canb.auug.org.au>
      94399431
    • M
      [media] cx231xx: remove some unused functions · 1001f908
      Mauro Carvalho Chehab 提交于
      This file came originally from cx23885 driver. Some functions aren't
      used. Now that they are declared as static, we have those errors:
      
      drivers/media/video/cx231xx/cx231xx-417.c:615: warning: ‘mc417_gpio_set’ defined but not used
      drivers/media/video/cx231xx/cx231xx-417.c:625: warning: ‘mc417_gpio_clear’ defined but not used
      drivers/media/video/cx231xx/cx231xx-417.c:635: warning: ‘mc417_gpio_enable’ defined but not used
      
      As they're not used, just remove them. If needed, they can be restored from
      the git logs or from the cx23885 driver.
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      1001f908
    • M
      [media] cx231xx: declare static functions as such · 82c3ccaa
      Mauro Carvalho Chehab 提交于
      drivers/media/video/cx23885/built-in.o: In function `mc417_memory_write':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:482: multiple definition of `mc417_memory_write'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:477: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_gpio_set':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:636: multiple definition of `mc417_gpio_set'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:615: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_gpio_enable':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:656: multiple definition of `mc417_gpio_enable'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:635: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_memory_read':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:546: multiple definition of `mc417_memory_read'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:541: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_gpio_clear':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:646: multiple definition of `mc417_gpio_clear'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:625: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_register_read':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:388: multiple definition of `mc417_register_read'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:401: first defined here
      drivers/media/video/cx23885/built-in.o: In function `mc417_register_write':
      /home/v4l/v4l/patchwork/drivers/media/video/cx23885/cx23885-417.c:324: multiple definition of `mc417_register_write'
      drivers/media/video/cx231xx/built-in.o:/home/v4l/v4l/patchwork/drivers/media/video/cx231xx/cx231xx-417.c:343: first defined here
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      82c3ccaa
    • M
      [media] cx231xx-417: Fix a gcc warning · bae94dc3
      Mauro Carvalho Chehab 提交于
      gcc didn't like to have i++ inside a complex operation:
      
      drivers/media/video/cx231xx/cx231xx-417.c: In function ‘cx231xx_load_firmware’:
      drivers/media/video/cx231xx/cx231xx-417.c:1059: warning: operation on ‘i’ may be undefined
      drivers/media/video/cx231xx/cx231xx-417.c:1061: warning: operation on ‘i’ may be undefined
      drivers/media/video/cx231xx/cx231xx-417.c:1063: warning: operation on ‘i’ may be undefined
      
      Btw, I agree with gcc, as we're using i and i++ at the same operation and,
      depending on how optimization may occur, it may produce a wrong code.
      
      While here, fix CodingStyle issues on the changed code.
      Acked-by: NSri Deevi <Srinivasa.Deevi@conexant.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      bae94dc3
    • M
      [media] CodingStyle cleanup at s5h1432 and cx231xx · 955e6ed8
      Mauro Carvalho Chehab 提交于
      The patches received from the vendor contained a lot of CodingStyle
      issues. Cleans the style issues reported by checkpatch.pl on
      those drivers.
      
      It is better to do such style fixes when merging a big set of
      changes than latter. Of course, the better is to receive patches
      already cleaned ;)
      Acked-by: NSri Deevi <Srinivasa.Deevi@conexant.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      955e6ed8
    • M
      [media] cx231xx: remove a printk warning at -avcore and at -417 · 62c78c96
      Mauro Carvalho Chehab 提交于
      drivers/media/video/cx231xx/cx231xx-avcore.c:1608: warning: format ‘%d’ expects type ‘int’, but argument 3 has type ‘long unsigned int’
      drivers/media/video/cx231xx/cx231xx-417.c:1047: warning: format ‘%d’ expects type ‘int’, but argument 3 has type ‘size_t’
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      62c78c96
    • D
      [media] cx231xx: fix format string warning · dd067a8d
      Devin Heitmueller 提交于
      Change a %x to a %p since the variable is a pointer
      Signed-off-by: NDevin Heitmueller <dheitmueller@hauppauge.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      dd067a8d
    • P
      [media] cx231xx: Added support for Carraera, Shelby, RDx_253S and VIDEO_GRABBER · 64fbf444
      Palash Bandyopadhyay 提交于
      Added support for new cx231xx boards - Carraera, Shelby, RDx_253S and
      VIDEO_GRABBER.
      
      [mchehab@redhat.com: Fix a merge conflict with BKL removal patches]
      Signed-off-by: NPalash Bandyopadhyay <palash.bandyopadhyay@conexant.com>
      Signed-off-by: NDevin Heitmueller <dheitmueller@hauppauge.com>
      Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      64fbf444