- 31 Jul, 2012 8 commits
-
-
Douglas Bagnall authored
For some reason, when the lirc daemon learns that a usb remote control has been unplugged, it wants to read the sysfs attributes of the disappearing device. This is useful for uncovering transient inconsistencies, but less so for keeping the system running when such inconsistencies exist. Under some circumstances (like every time I unplug my dvb stick from my laptop), lirc catches an rc_dev whose raw event handler has been removed (presumably by ir_raw_event_unregister), and proceeds to interrogate the raw protocols supported by the NULL pointer. This patch avoids the NULL dereference, and ignores the issue of how this state of affairs came about in the first place. Version 2 incorporates changes recommended by Mauro Carvalho Chehab (-ENODEV instead of -EINVAL, and a signed-off-by). Signed-off-by:
Douglas Bagnall <douglas@paradise.net.nz> Signed-off-by:
Mauro Carvalho Chehab <mchehab@redhat.com>
-
Ezequiel García authored
It is possible that video_put() releases video_device struct, provoking a panic when debug printk wants to get video_device node name. Signed-off-by:
Ezequiel Garcia <elezegarcia@gmail.com> Acked-by:
Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by:
-
Hans Verkuil authored
This patch will always set the field to INTERLACED (this fixes a bug were a driver should never return FIELD_ANY), and will default to YUYV pixelformat if an unknown pixelformat was specified. This way S/TRY_FMT will always return a valid format struct. Regards, Hans Signed-off-by: -
Julia Lawall authored
Add missing usb_free_urb on failure path after usb_alloc_urb. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @km exists@ local idexpression e; expression e1,e2,e3; type T,T1; identifier f; @@ * e = usb_alloc_urb(...) ... when any when != e = e1 when != e1 = (T)e when != e1(...,(T)e,...) when != &e->f if(...) { ... when != e2(...,(T1)e,...) when != e3 = e when forall ( return <+...e...+>; | * return ...; ) } // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Alan Cox authored
Some parts of the C language are subtle and evil. This is one example. Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44041 Reported-by: dcb314@hotmail.com Signed-off-by:
Alan Cox <alan@linux.intel.com> Signed-off-by:
-
Alan Cox authored
Unsupported requests should be ignored but in fact affected VDEC_A Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44051 Reported-by: dcb314@hotmail.com Signed-off-by:
-
Alan Cox authored
Without this rev2 ends up behaving as rev3 Resolves-bug: https://bugzilla.kernel.org/show_bug.cgi?id=44081 Reported-by: dcb314@hotmail.com Signed-off-by:
Guennadi Liakhovetski <g.liakhovetski@gmx.de> Signed-off-by:
-
Prabhakar Lad authored
Fix build error when CONFIG_VIDEO_ADV_DEBUG is enabled, declare the vpbe_dev variable. Signed-off-by:
Lad, Prabhakar <prabhakar.lad@ti.com> Signed-off-by:
Manjunath Hadli <manjunath.hadli@ti.com> Signed-off-by:
-
- 30 Jul, 2012 32 commits
-
-
Nicolas THERY authored
Signed-off-by:
Nicolas Thery <nicolas.thery@st.com> Signed-off-by:
-
Dan Carpenter authored
We need to do a mutex_unlock(&priv->lock) before returning. Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Jesper Juhl authored
This patch simply adds a newline character at end-of-file to those files in Documentation/ that currently lack one. This is done for a few different reasons: A) It's rather annoying when you do "cat some_file.txt" that your prompt/cursor ends up at the end of the last line of output rather than on a new line. B) Some tools that process files line-by-line may get confused by the lack of a newline on the last line. C) The "\ No newline at end of file" line in diffs annoys me for some reason. So, let's just add the missing newline once and for all. Signed-off-by:
Jesper Juhl <jj@chaosbits.net> Signed-off-by:
-
Prabhakar Lad authored
fix check for s_dv_preset function pointer to be NULL. return -EINVAL if function pointer is NULL. Signed-off-by:
-
Dan Carpenter authored
We intended to do a compare here, not an assignment. Signed-off-by:
-
Hans Verkuil authored
Remove the pointless video_nr++. It doesn't do anything useful and it has the unexpected side-effect of changing the video_nr module option, so cat /sys/module/vivi/parameters/video_nr gives a different value back then what was specified with modprobe. Signed-off-by:
-
Nicolas THERY authored
Signed-off-by:
-
Hans de Goede authored
Signed-off-by:
Hans de Goede <hdegoede@redhat.com> Signed-off-by:
-
Hans de Goede authored
The mask was wrong resulting in band 0 and 1 always ending up as band 0 in the register. Signed-off-by:
-
Hans de Goede authored
Signed-off-by:
-
Hans de Goede authored
To allow apps to limit a hw-freq-seek to a specific band, for further info see the documentation this patch adds for these new fields. Signed-off-by:
-
Hans Verkuil authored
Signed-off-by:
-
Hans Verkuil authored
The current RDS code suffered from bit rot. Clean it up and make it work again. Signed-off-by:
-
Hans Verkuil authored
- add control framework - use core locking - use V4L2_TUNER_CAP_LOW - remove volume support: there is no hardware volume control Signed-off-by:
-
Hans Verkuil authored
Signed-off-by:
-
Hans Verkuil authored
This adds the usual core support code for this new ioctl. Signed-off-by:
-
Hans Verkuil authored
Add a new ioctl to enumerate the supported frequency bands of a tuner. Signed-off-by:
-
Nicolas THERY authored
Signed-off-by:
-
Dan Carpenter authored
tvp5150_read() returns negative error codes so this needs to be an int for the error handling to work. Signed-off-by:
-
Sachin Kamat authored
Fixes the following sparse warning: drivers/media/video/videobuf-dma-contig.c:59:46: warning: Using plain integer as NULL pointer Signed-off-by:
Sachin Kamat <sachin.kamat@linaro.org> Signed-off-by:
-
Hans de Goede authored
Testing with a firmware version 12 usb radio stick has shown version 12 to work fine too. Reported-by:
Antti Palosaari <crope@iki.fi> Signed-off-by:
-
Federico Vaga authored
[mchehab@redhat.com: fix checkpatch.pl ERROR: Macros with complex values should be enclosed in parenthesis] Signed-off-by:
Federico Vaga <federico.vaga@gmail.com> Signed-off-by:
-
Hans Verkuil authored
This patch fixes the DV_TIMINGS_CAP documentation: part of it was copy-and-paste from the ENUM_DV_TIMINGS documentation. Regards, Hans Signed-off-by:
-
Sachin Kamat authored
Fixes the following sparse warning: drivers/media/video/v4l2-ctrls.c:2123:43: warning: Using plain integer as NULL pointer Signed-off-by:
-
Dror Cohen authored
Signed-off-by:
Dror Cohen <dror@liveu.tv> Signed-off-by:
-
Ezequiel García authored
Copying structs by assignment is type safe. Plus, is shorter and easier to read. Signed-off-by:
-
Ezequiel García authored
Copying structs by assignment is type safe. Plus, is shorter and easier to read. Signed-off-by:
-
Ezequiel García authored
Copying structs by assignment is type safe. Plus, is shorter and easier to read. Signed-off-by:
-
Ezequiel García authored
Copying structs by assignment is type safe. Plus, is shorter and easier to read. Signed-off-by:
-
Ezequiel García authored
This function has no users, so it's safe to remove it. Tested by compilation only. Signed-off-by:
-
Ezequiel García authored
The field 'struct i2c_algo_bit_data i2c_algo' is wrongly confused with struct i2c_algorithm. Moreover, i2c_algo field is not used since i2c is registered using i2c_add_adpater() and not i2c_bit_add_bus(). Therefore, it's safe to remove it. Tested by compilation only. Signed-off-by:
-
Ezequiel García authored
The field 'struct i2c_algo_bit_data i2c_algo' is wrongly confused with struct i2c_algorithm. Moreover, i2c_algo field is not used since i2c is registered using i2c_add_adpater() and not i2c_bit_add_bus(). Therefore, it's safe to remove it. Tested by compilation only. Signed-off-by:
-
