Re: [GIT PULL FOR v3.10] v4l2: make arg of write-only ioctls const

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

 



On Fri March 22 2013 11:35:17 Hans Verkuil wrote:
> Hi Mauro,
> 
> This is the second and last phase of ensuring that the arguments of write-only
> ioctls in V4L2 are const. The first phase was 4-5 months ago and added const
> to s_crop, s_modulator, s_audio, s_audout, (un)subscribe_event, s_freq_hw_seek,
> s_jpegcomp and s_fbuf.
> 
> This second phase adds const to s_frequency, s_tuner, s_std and s_register.
> Actually, for s_std it doesn't add const but changes it to pass the std by
> value which is more consistent in that particular case.
> 
> As a result drivers will be aware when they are implementing write-only ioctls
> (and I saw a few drivers attempting to return data back to the user), and the
> v4l2 core will know that drivers won't change the argument of a write-only
> ioctls which simplifies the core debug code.
> 
> The changes have been compile-tested with the linux-media daily build but
> I may have missed some more exotic architectures.
> 
> Ideally I would like to have this merged fairly early on so we have enough
> time to shake out any remaining compile problems.
> 
> This pull request is identical to:
> 
> http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg59774.html and
> http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg59886.html with
> this final change for radio-keene.c:
> 
> http://www.spinics.net/lists/linux-media/msg61339.html
> 
> Note that I will be making more pull requests based on top of this pull
> request since you will get merge conflicts otherwise.

Nacked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx>

I just discovered that after the recent merged the s_register patch causes
a compile warning in ths7303. Nacking this and posting a new one.

Regards,

	Hans

> 
> Regards,
> 
>         Hans
> 
> The following changes since commit d7104bffcfb7a1a7f1dbb1274443e339588c2cb3:
> 
>   [media] MAINTAINERS: add drivers/media/tuners/it913x* (2013-03-21 19:06:43 -0300)
> 
> are available in the git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git const2
> 
> for you to fetch changes up to 0b03adbbaef3e6ec164ba7647dd1f53f48ef76c5:
> 
>   v4l2-ioctl: add precision when printing names. (2013-03-22 09:31:28 +0100)
> 
> ----------------------------------------------------------------
> Hans Verkuil (6):
>       v4l2: add const to argument of write-only s_frequency ioctl.
>       v4l2: add const to argument of write-only s_tuner ioctl.
>       v4l2: pass std by value to the write-only s_std ioctl.
>       v4l2: add const to argument of write-only s_register ioctl.
>       v4l2-ioctl: simplify debug code.
>       v4l2-ioctl: add precision when printing names.
> 
>  drivers/media/common/saa7146/saa7146_video.c     |    4 +--
>  drivers/media/dvb-frontends/au8522_decoder.c     |    2 +-
>  drivers/media/i2c/ad9389b.c                      |    2 +-
>  drivers/media/i2c/adv7183.c                      |    2 +-
>  drivers/media/i2c/adv7604.c                      |    2 +-
>  drivers/media/i2c/ak881x.c                       |    2 +-
>  drivers/media/i2c/cs5345.c                       |    2 +-
>  drivers/media/i2c/cx25840/cx25840-core.c         |    6 ++--
>  drivers/media/i2c/m52790.c                       |    2 +-
>  drivers/media/i2c/msp3400-driver.c               |    4 +--
>  drivers/media/i2c/mt9m032.c                      |    2 +-
>  drivers/media/i2c/mt9v011.c                      |    2 +-
>  drivers/media/i2c/ov7670.c                       |    2 +-
>  drivers/media/i2c/saa6588.c                      |    2 +-
>  drivers/media/i2c/saa7115.c                      |    2 +-
>  drivers/media/i2c/saa7127.c                      |    2 +-
>  drivers/media/i2c/saa717x.c                      |    4 +--
>  drivers/media/i2c/soc_camera/mt9m001.c           |    2 +-
>  drivers/media/i2c/soc_camera/mt9m111.c           |    2 +-
>  drivers/media/i2c/soc_camera/mt9t031.c           |    2 +-
>  drivers/media/i2c/soc_camera/mt9t112.c           |    2 +-
>  drivers/media/i2c/soc_camera/mt9v022.c           |    2 +-
>  drivers/media/i2c/soc_camera/ov2640.c            |    2 +-
>  drivers/media/i2c/soc_camera/ov5642.c            |    2 +-
>  drivers/media/i2c/soc_camera/ov6650.c            |    2 +-
>  drivers/media/i2c/soc_camera/ov772x.c            |    2 +-
>  drivers/media/i2c/soc_camera/ov9640.c            |    2 +-
>  drivers/media/i2c/soc_camera/ov9740.c            |    2 +-
>  drivers/media/i2c/soc_camera/rj54n1cb0c.c        |    2 +-
>  drivers/media/i2c/soc_camera/tw9910.c            |    2 +-
>  drivers/media/i2c/tda9840.c                      |    2 +-
>  drivers/media/i2c/tvaudio.c                      |    4 +--
>  drivers/media/i2c/tvp5150.c                      |    2 +-
>  drivers/media/i2c/tvp7002.c                      |    2 +-
>  drivers/media/i2c/upd64031a.c                    |    4 +--
>  drivers/media/i2c/upd64083.c                     |    2 +-
>  drivers/media/i2c/vp27smpx.c                     |    2 +-
>  drivers/media/i2c/vs6624.c                       |    2 +-
>  drivers/media/i2c/wm8775.c                       |    2 +-
>  drivers/media/parport/pms.c                      |    4 +--
>  drivers/media/pci/bt8xx/bttv-driver.c            |   36 +++++++++++++-----------
>  drivers/media/pci/cx18/cx18-av-core.c            |    6 ++--
>  drivers/media/pci/cx18/cx18-driver.c             |    2 +-
>  drivers/media/pci/cx18/cx18-ioctl.c              |   50 ++++++++++++++-------------------
>  drivers/media/pci/cx18/cx18-ioctl.h              |    4 +--
>  drivers/media/pci/cx23885/cx23885-417.c          |   10 +++----
>  drivers/media/pci/cx23885/cx23885-ioctl.c        |    9 ++----
>  drivers/media/pci/cx23885/cx23885-ioctl.h        |    2 +-
>  drivers/media/pci/cx23885/cx23885-video.c        |   14 +++++-----
>  drivers/media/pci/cx23885/cx23885.h              |    2 +-
>  drivers/media/pci/cx23885/cx23888-ir.c           |    2 +-
>  drivers/media/pci/cx25821/cx25821-video.c        |   14 +++++-----
>  drivers/media/pci/cx25821/cx25821-video.h        |   10 +++----
>  drivers/media/pci/cx88/cx88-blackbird.c          |    8 +++---
>  drivers/media/pci/cx88/cx88-video.c              |   25 ++++++++---------
>  drivers/media/pci/cx88/cx88.h                    |    2 +-
>  drivers/media/pci/ivtv/ivtv-driver.c             |    4 +--
>  drivers/media/pci/ivtv/ivtv-firmware.c           |    4 +--
>  drivers/media/pci/ivtv/ivtv-gpio.c               |    2 +-
>  drivers/media/pci/ivtv/ivtv-ioctl.c              |   73 ++++++++++++++++++++++++------------------------
>  drivers/media/pci/ivtv/ivtv-ioctl.h              |    6 ++--
>  drivers/media/pci/saa7134/saa7134-empress.c      |    2 +-
>  drivers/media/pci/saa7134/saa7134-video.c        |   22 +++++++--------
>  drivers/media/pci/saa7134/saa7134.h              |    2 +-
>  drivers/media/pci/saa7146/mxb.c                  |    9 +++---
>  drivers/media/pci/saa7164/saa7164-encoder.c      |   14 +++++-----
>  drivers/media/pci/saa7164/saa7164-vbi.c          |   12 ++++----
>  drivers/media/pci/sta2x11/sta2x11_vip.c          |   18 ++++++------
>  drivers/media/pci/ttpci/av7110_v4l.c             |    4 +--
>  drivers/media/pci/zoran/zoran_driver.c           |    4 +--
>  drivers/media/platform/blackfin/bfin_capture.c   |    8 +++---
>  drivers/media/platform/davinci/vpbe.c            |    8 +++---
>  drivers/media/platform/davinci/vpbe_display.c    |    4 +--
>  drivers/media/platform/davinci/vpfe_capture.c    |   12 ++++----
>  drivers/media/platform/davinci/vpif_capture.c    |    9 +++---
>  drivers/media/platform/davinci/vpif_display.c    |   13 +++++----
>  drivers/media/platform/fsl-viu.c                 |    6 ++--
>  drivers/media/platform/marvell-ccic/mcam-core.c  |    4 +--
>  drivers/media/platform/s5p-tv/mixer_video.c      |    4 +--
>  drivers/media/platform/sh_vou.c                  |   14 +++++-----
>  drivers/media/platform/soc_camera/soc_camera.c   |    6 ++--
>  drivers/media/platform/timblogiw.c               |    6 ++--
>  drivers/media/platform/via-camera.c              |    2 +-
>  drivers/media/platform/vino.c                    |   10 +++----
>  drivers/media/radio/dsbr100.c                    |    4 +--
>  drivers/media/radio/radio-cadet.c                |   48 ++++++++++++++++----------------
>  drivers/media/radio/radio-isa.c                  |   11 ++++----
>  drivers/media/radio/radio-keene.c                |    8 +++---
>  drivers/media/radio/radio-ma901.c                |    4 +--
>  drivers/media/radio/radio-miropcm20.c            |   12 ++++----
>  drivers/media/radio/radio-mr800.c                |    4 +--
>  drivers/media/radio/radio-sf16fmi.c              |    4 +--
>  drivers/media/radio/radio-si4713.c               |    2 +-
>  drivers/media/radio/radio-tea5764.c              |    4 +--
>  drivers/media/radio/radio-tea5777.c              |    9 +++---
>  drivers/media/radio/radio-timb.c                 |    4 +--
>  drivers/media/radio/radio-wl1273.c               |    4 +--
>  drivers/media/radio/si470x/radio-si470x-common.c |    4 +--
>  drivers/media/radio/si4713-i2c.c                 |    5 ++--
>  drivers/media/radio/tef6862.c                    |    4 +--
>  drivers/media/radio/wl128x/fmdrv_v4l2.c          |    8 ++----
>  drivers/media/usb/au0828/au0828-video.c          |   12 ++++----
>  drivers/media/usb/cx231xx/cx231xx-417.c          |    4 +--
>  drivers/media/usb/cx231xx/cx231xx-video.c        |   19 +++++++------
>  drivers/media/usb/cx231xx/cx231xx.h              |    6 ++--
>  drivers/media/usb/em28xx/em28xx-video.c          |   21 +++++++-------
>  drivers/media/usb/gspca/gspca.c                  |    2 +-
>  drivers/media/usb/gspca/gspca.h                  |    8 ++++--
>  drivers/media/usb/gspca/pac7302.c                |    2 +-
>  drivers/media/usb/gspca/sn9c20x.c                |    2 +-
>  drivers/media/usb/hdpvr/hdpvr-video.c            |    4 +--
>  drivers/media/usb/pvrusb2/pvrusb2-hdw.c          |    2 +-
>  drivers/media/usb/pvrusb2/pvrusb2-hdw.h          |    2 +-
>  drivers/media/usb/pvrusb2/pvrusb2-v4l2.c         |   10 +++----
>  drivers/media/usb/s2255/s2255drv.c               |    8 +++---
>  drivers/media/usb/stk1160/stk1160-v4l.c          |    6 ++--
>  drivers/media/usb/tlg2300/pd-radio.c             |    4 +--
>  drivers/media/usb/tlg2300/pd-video.c             |   15 +++++-----
>  drivers/media/usb/tm6000/tm6000-video.c          |   16 ++++-------
>  drivers/media/usb/usbvision/usbvision-video.c    |   10 +++----
>  drivers/media/v4l2-core/tuner-core.c             |    4 +--
>  drivers/media/v4l2-core/v4l2-ioctl.c             |   85 +++++++++++++++++++++++++++-----------------------------
>  drivers/staging/media/davinci_vpfe/vpfe_video.c  |    6 ++--
>  drivers/staging/media/dt3155v4l/dt3155v4l.c      |    4 +--
>  drivers/staging/media/go7007/go7007-v4l2.c       |   20 ++++++-------
>  drivers/staging/media/solo6x10/v4l2-enc.c        |    2 +-
>  drivers/staging/media/solo6x10/v4l2.c            |    2 +-
>  include/media/davinci/vpbe.h                     |    2 +-
>  include/media/v4l2-ioctl.h                       |    8 +++---
>  include/media/v4l2-subdev.h                      |    6 ++--
>  sound/i2c/other/tea575x-tuner.c                  |    6 ++--
>  131 files changed, 499 insertions(+), 512 deletions(-)
> --
> 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