linux-toradex.git/drivers/media/usb/usbvision, branch v3.11

[media] usbvision-video: fix memory leak of alt_max_pkt_size

2013-06-21T14:29:24+00:00

1. usbvision->alt_max_pkt_size is not deallocated anywhere.
2. if allocation of usbvision->alt_max_pkt_size fails,
there is no proper deallocation of already acquired resources.
The patch adds kfree(usbvision->alt_max_pkt_size) to
usbvision_release() as soon as other deallocations happen there.
It calls usbvision_release() if allocation of
usbvision->alt_max_pkt_size fails as soon as usbvision_release()
is safe to work with incompletely initialized usbvision structure.
Found by Linux Driver Verification project (linuxtesting.org).

Signed-off-by: Alexey Khoroshilov 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: remove obsolete v4l2_chip_match_host()

2013-06-21T13:30:54+00:00

This function is no longer needed since it is now the responsibility of the
v4l2 core to check if the DBG_G/S_REGISTER and DBG_G_CHIP_INFO ioctls are
called for the bridge driver or not.

Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] usbvision: replace current_norm by g_std

2013-06-17T12:37:23+00:00

current_norm use is deprecated because it is per-devicenode and if you
have more device nodes all dependent on the same video source, then this
no longer works. Just implement g_std instead.

Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: add const to argument of write-only s_register ioctl

2013-03-24T11:47:01+00:00

This ioctl is defined as IOW, so pass the argument as const.

Signed-off-by: Hans Verkuil 
Acked-by: Guennadi Liakhovetski 
Acked-by: Lad, Prabhakar 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: pass std by value to the write-only s_std ioctl

2013-03-24T09:47:00+00:00

This ioctl is defined as IOW, so pass the argument by value instead of by
reference. I could have chosen to add const instead, but this is 1) easier
to handle in drivers and 2) consistent with the s_std subdev operation.

Signed-off-by: Hans Verkuil 
Acked-by: Laurent Pinchart 
Acked-by: Jonathan Corbet 
Acked-by: Guennadi Liakhovetski 
Acked-by: Lad, Prabhakar 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: add const to argument of write-only s_tuner ioctl

2013-03-24T09:31:23+00:00

This ioctl is defined as IOW, so pass the argument as const.

Signed-off-by: Hans Verkuil 
Acked-by: Laurent Pinchart 
Acked-by: Alexey Klimov 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2: add const to argument of write-only s_frequency ioctl

2013-03-24T09:25:34+00:00

This ioctl is defined as IOW, so pass the argument as const.

Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

Merge branch 'v4l_for_linus' into staging/for_v3.9

2013-01-24T20:49:18+00:00

* v4l_for_linus: (464 commits)
  [media] uvcvideo: Set error_idx properly for S_EXT_CTRLS failures
  [media] uvcvideo: Cleanup leftovers of partial revert
  [media] uvcvideo: Return -EACCES when trying to set a read-only control
  Linux 3.8-rc3
  mm: reinstante dropped pmd_trans_splitting() check
  cred: Remove tgcred pointer from struct cred
  drm/ttm: fix fence locking in ttm_buffer_object_transfer
  ARM: clps711x: Fix bad merge of clockevents setup
  ARM: highbank: save and restore L2 cache and GIC on suspend
  ARM: highbank: add a power request clear
  ARM: highbank: fix secondary boot and hotplug
  ARM: highbank: fix typos with hignbank in power request functions
  ARM: dts: fix highbank cpu mpidr values
  ARM: dts: add device_type prop to cpu nodes on Calxeda platforms
  drm/prime: drop reference on imported dma-buf come from gem
  xen/netfront: improve truesize tracking
  ARM: mx5: Fix MX53 flexcan2 clock
  ARM: OMAP2+: am33xx-hwmod: Fix wrongly terminated am33xx_usbss_mpu_irqs array
  sctp: fix Kconfig bug in default cookie hmac selection
  EDAC: Cleanup device deregistering path
  ...

Conflicts:
	drivers/media/pci/dm1105/dm1105.c
	drivers/media/platform/soc_camera/mx2_camera.c

Drivers: media: remove __dev* attributes.

2013-01-03T23:57:02+00:00

CONFIG_HOTPLUG is going away as an option.  As a result, the __dev*
markings need to be removed.

This change removes the use of __devinit, __devexit_p, __devinitdata,
__devinitconst, and __devexit from these drivers.

Based on patches originally written by Bill Pemberton, but redone by me
in order to handle some of the coding style issues better, by hand.

Cc: Bill Pemberton 
Cc: Mauro Carvalho Chehab 
Signed-off-by: Greg Kroah-Hartman

[media] usbvision: Replace memcpy with struct assignment

2012-12-27T20:44:30+00:00

This kind of memcpy() is error-prone. Its replacement with a struct
assignment is prefered because it's type-safe and much easier to read.
Found by coccinelle. Hand patched and reviewed.
Tested by compilation only.
A simplified version of the semantic match that finds this problem is as
follows: (http://coccinelle.lip6.fr/)
// 
@@
identifier struct_name;
struct struct_name to;
struct struct_name from;
expression E;
@@
-memcpy(&(to), &(from), E);
+to = from;
// 

Signed-off-by: Peter Senna Tschudin 
Signed-off-by: Ezequiel Garcia 
Signed-off-by: Mauro Carvalho Chehab