GitHub/LineageOS/android_kernel_motorola_exynos9610.git
12 years ago[media] drivers/media/dvb/siano/smscoreapi.c: use list_for_each_entry
Julia Lawall [Sun, 15 Jul 2012 08:25:22 +0000 (05:25 -0300)]
[media] drivers/media/dvb/siano/smscoreapi.c: use list_for_each_entry

Use list_for_each_entry and perform some other induced simplifications.

The semantic match that finds the opportunity for this reorganization is as
follows: (http://coccinelle.lip6.fr/)

// <smpl>
@@
struct list_head *pos;
struct list_head *head;
statement S;
@@

*for (pos = (head)->next; pos != (head); pos = pos->next)
S
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] Use a named union in struct v4l2_ioctl_info
Hans Verkuil [Thu, 12 Jul 2012 15:06:24 +0000 (12:06 -0300)]
[media] Use a named union in struct v4l2_ioctl_info

Hi Mauro,

struct v4l2_ioctl_info uses an anonymous union, which is initialized
in the v4l2_ioctls table.

Unfortunately gcc < 4.6 uses a non-standard syntax for that, so trying to
compile v4l2-ioctl.c with an older gcc will fail.

It is possible to work around this by testing the gcc version, but in this
case it is easier to make the union named since it is used in only a few
places.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Reported-by: Randy Dunlap <rdunlap@xenotime.net>
Acked-by: Randy Dunlap <rdunlap@xenotime.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] mceusb: Add Twisted Melon USB IDs
Mark Lord [Wed, 11 Jul 2012 21:53:28 +0000 (18:53 -0300)]
[media] mceusb: Add Twisted Melon USB IDs

Add USB identifiers for MCE compatible I/R transceivers from Twisted Melon.

Signed-off-by: Mark Lord <mlord@pobox.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] staging/media/solo6x10: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 05:45:26 +0000 (02:45 -0300)]
[media] staging/media/solo6x10: use module_pci_driver macro

the driver duplicates the module_pci_driver code,
how?
module_pci_driver is used for those drivers whose
init and exit paths does only register and unregister
to pci API and nothing else.

so use the module_pci_driver macro instead

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Acked-by: Ismael Luceno <ismael.luceno@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] staging/media/dt3155v4l: use module_pci_driver macro
Devendra Naga [Tue, 10 Jul 2012 05:43:48 +0000 (02:43 -0300)]
[media] staging/media/dt3155v4l: use module_pci_driver macro

the driver duplicates the module_pci_driver code,
remove the duplicate code and use the module_pci_driver macro.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] lirc: fix non-ANSI function declaration warning
Emil Goode [Sat, 7 Jul 2012 17:53:25 +0000 (14:53 -0300)]
[media] lirc: fix non-ANSI function declaration warning

Sparse is warning about non-ANSI function declaration.
Add void to the parameterless function.

drivers/staging/media/lirc/lirc_bt829.c:174:22: warning:
non-ANSI function declaration of function 'poll_main'

Signed-off-by: Emil Goode <emilgoode@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] rc: ati_remote.c: code style fixing
Du, Changbin [Sat, 7 Jul 2012 06:53:28 +0000 (03:53 -0300)]
[media] rc: ati_remote.c: code style fixing

changes:
1. wrap some lines that are longer than 80 characters.
2. remove local function prototype declarations which do not
   need.
3. replace TAB character with a space character in function
   comments.

Signed-off-by: Du, Changbin <changbin.du@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] Avoid sysfs oops when an rc_dev's raw device is absent
Douglas Bagnall [Sat, 7 Jul 2012 02:27:57 +0000 (23:27 -0300)]
[media] Avoid sysfs oops when an rc_dev's raw device is absent

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>
12 years ago[media] v4l2-dev.c: Move video_put() after debug printk
Ezequiel García [Thu, 26 Jul 2012 10:59:04 +0000 (07:59 -0300)]
[media] v4l2-dev.c: Move video_put() after debug printk

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] vivi: fix a few format-related compliance issues
Hans Verkuil [Wed, 25 Jul 2012 14:48:53 +0000 (11:48 -0300)]
[media] vivi: fix a few format-related compliance issues

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] drivers/staging/media/easycap/easycap_main.c: add missing usb_free_urb
Julia Lawall [Tue, 24 Jul 2012 14:06:09 +0000 (11:06 -0300)]
[media] drivers/staging/media/easycap/easycap_main.c: add missing usb_free_urb

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] az6007: fix incorrect memcpy
Alan Cox [Tue, 24 Jul 2012 15:02:46 +0000 (12:02 -0300)]
[media] az6007: fix incorrect memcpy

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx25821,medusa: incorrect check on decoder type
Alan Cox [Tue, 24 Jul 2012 15:00:24 +0000 (12:00 -0300)]
[media] cx25821,medusa: incorrect check on decoder type

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 <alan@linux.intel.com>
Cc: Palash Bandyopadhyay <Palash.Bandyopadhyay@conexant.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] ov9640: fix missing break
Alan Cox [Tue, 24 Jul 2012 14:50:49 +0000 (11:50 -0300)]
[media] ov9640: fix missing break

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: Alan Cox <alan@linux.intel.com>
Acked-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpbe: fix build error when CONFIG_VIDEO_ADV_DEBUG is enabled
Prabhakar Lad [Fri, 20 Jul 2012 12:56:48 +0000 (09:56 -0300)]
[media] davinci: vpbe: fix build error when CONFIG_VIDEO_ADV_DEBUG is enabled

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l: fix copy/paste typo in vb2_reqbufs comment
Nicolas THERY [Fri, 20 Jul 2012 12:25:37 +0000 (09:25 -0300)]
[media] v4l: fix copy/paste typo in vb2_reqbufs comment

Signed-off-by: Nicolas Thery <nicolas.thery@st.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] tuner-xc2028: unlock on error in xc2028_get_afc()
Dan Carpenter [Sat, 21 Jul 2012 07:32:59 +0000 (04:32 -0300)]
[media] tuner-xc2028: unlock on error in xc2028_get_afc()

We need to do a mutex_unlock(&priv->lock) before returning.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] Documentation: Add newline at end-of-file to files lacking one
Jesper Juhl [Fri, 20 Jul 2012 19:35:17 +0000 (16:35 -0300)]
[media] Documentation: Add newline at end-of-file to files lacking one

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpbe: fix check for s_dv_preset function pointer
Prabhakar Lad [Fri, 20 Jul 2012 13:00:57 +0000 (10:00 -0300)]
[media] davinci: vpbe: fix check for s_dv_preset function pointer

fix check for s_dv_preset function pointer to be NULL.
return -EINVAL if function pointer is NULL.

Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] tuner-xc2028: fix "=" vs "==" typo
Dan Carpenter [Sat, 21 Jul 2012 07:32:38 +0000 (04:32 -0300)]
[media] tuner-xc2028: fix "=" vs "==" typo

We intended to do a compare here, not an assignment.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] vivi: remove pointless video_nr++
Hans Verkuil [Thu, 19 Jul 2012 19:45:49 +0000 (16:45 -0300)]
[media] vivi: remove pointless video_nr++

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: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l: DocBook: VIDIOC_CREATE_BUFS: add hyperlink
Nicolas THERY [Wed, 18 Jul 2012 14:41:36 +0000 (11:41 -0300)]
[media] v4l: DocBook: VIDIOC_CREATE_BUFS: add hyperlink

Signed-off-by: Nicolas Thery <nicolas.thery@st.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-si470x: Add support for the new band APIs
Hans de Goede [Thu, 12 Jul 2012 19:55:48 +0000 (16:55 -0300)]
[media] radio-si470x: Add support for the new band APIs

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-si470x: Fix band selection
Hans de Goede [Thu, 12 Jul 2012 19:55:47 +0000 (16:55 -0300)]
[media] radio-si470x: Fix band selection

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 <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-si470x: restore ctrl settings after suspend/resume
Hans de Goede [Thu, 12 Jul 2012 19:55:46 +0000 (16:55 -0300)]
[media] radio-si470x: restore ctrl settings after suspend/resume

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2: Add rangelow and rangehigh fields to the v4l2_hw_freq_seek struct
Hans de Goede [Thu, 12 Jul 2012 19:55:45 +0000 (16:55 -0300)]
[media] v4l2: Add rangelow and rangehigh fields to the v4l2_hw_freq_seek struct

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 de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-cadet: implement frequency band enumeration
Hans Verkuil [Tue, 10 Jul 2012 11:26:04 +0000 (08:26 -0300)]
[media] radio-cadet: implement frequency band enumeration

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-cadet: fix RDS handling
Hans Verkuil [Mon, 2 Jul 2012 12:46:46 +0000 (09:46 -0300)]
[media] radio-cadet: fix RDS handling

The current RDS code suffered from bit rot. Clean it up and make it work again.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-cadet: upgrade to latest frameworks
Hans Verkuil [Mon, 2 Jul 2012 12:36:39 +0000 (09:36 -0300)]
[media] radio-cadet: upgrade to latest frameworks

- 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 <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2 spec: add VIDIOC_ENUM_FREQ_BANDS documentation
Hans Verkuil [Thu, 5 Jul 2012 09:54:38 +0000 (06:54 -0300)]
[media] v4l2 spec: add VIDIOC_ENUM_FREQ_BANDS documentation

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2: add core support for the new VIDIOC_ENUM_FREQ_BANDS ioctl
Hans Verkuil [Thu, 5 Jul 2012 09:37:08 +0000 (06:37 -0300)]
[media] v4l2: add core support for the new VIDIOC_ENUM_FREQ_BANDS ioctl

This adds the usual core support code for this new ioctl.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] videodev2.h: add VIDIOC_ENUM_FREQ_BANDS
Hans Verkuil [Thu, 5 Jul 2012 09:04:04 +0000 (06:04 -0300)]
[media] videodev2.h: add VIDIOC_ENUM_FREQ_BANDS

Add a new ioctl to enumerate the supported frequency bands of a tuner.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l: DocBook: fix version number typo
Nicolas THERY [Thu, 12 Jul 2012 15:12:12 +0000 (12:12 -0300)]
[media] v4l: DocBook: fix version number typo

Signed-off-by: Nicolas Thery <nicolas.thery@st.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] tvp5150: signedness bug in tvp5150_selmux()
Dan Carpenter [Thu, 12 Jul 2012 13:47:28 +0000 (10:47 -0300)]
[media] tvp5150: signedness bug in tvp5150_selmux()

tvp5150_read() returns negative error codes so this needs to be an int
for the error handling to work.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] videobuf-dma-contig: Use NULL instead of plain integer
Sachin Kamat [Thu, 12 Jul 2012 10:39:50 +0000 (07:39 -0300)]
[media] videobuf-dma-contig: Use NULL instead of plain integer

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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] radio-si470x: Lower firmware version requirements
Hans de Goede [Wed, 11 Jul 2012 14:47:38 +0000 (11:47 -0300)]
[media] radio-si470x: Lower firmware version requirements

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: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] adv7180.c: convert to v4l2 control framework
Federico Vaga [Wed, 11 Jul 2012 14:29:33 +0000 (11:29 -0300)]
[media] adv7180.c: convert to v4l2 control framework

[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: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] Fix DV_TIMINGS_CAP documentation
Hans Verkuil [Wed, 11 Jul 2012 10:48:52 +0000 (07:48 -0300)]
[media] Fix DV_TIMINGS_CAP documentation

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: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] V4L: Use NULL pointer instead of plain integer in v4l2-ctrls.c file
Sachin Kamat [Tue, 10 Jul 2012 10:14:46 +0000 (07:14 -0300)]
[media] V4L: Use NULL pointer instead of plain integer in v4l2-ctrls.c file

Fixes the following sparse warning:
drivers/media/video/v4l2-ctrls.c:2123:43: warning: Using plain integer as NULL pointer

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] media/video: vpif: fixed vpfe->vpif typo
Dror Cohen [Tue, 10 Jul 2012 08:43:22 +0000 (05:43 -0300)]
[media] media/video: vpif: fixed vpfe->vpif typo

Signed-off-by: Dror Cohen <dror@liveu.tv>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx25821: Replace struct memcpy with struct assignment
Ezequiel García [Wed, 27 Jun 2012 15:52:54 +0000 (12:52 -0300)]
[media] cx25821: Replace struct memcpy with struct assignment

Copying structs by assignment is type safe.
Plus, is shorter and easier to read.

Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx231xx: Replace struct memcpy with struct assignment
Ezequiel García [Wed, 27 Jun 2012 15:52:53 +0000 (12:52 -0300)]
[media] cx231xx: Replace struct memcpy with struct assignment

Copying structs by assignment is type safe.
Plus, is shorter and easier to read.

Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx23885: Replace struct memcpy with struct assignment
Ezequiel García [Wed, 27 Jun 2012 15:52:52 +0000 (12:52 -0300)]
[media] cx23885: Replace struct memcpy with struct assignment

Copying structs by assignment is type safe.
Plus, is shorter and easier to read.

Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] saa7164: Replace struct memcpy with struct assignment
Ezequiel García [Wed, 27 Jun 2012 15:52:51 +0000 (12:52 -0300)]
[media] saa7164: Replace struct memcpy with struct assignment

Copying structs by assignment is type safe.
Plus, is shorter and easier to read.

Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] saa7164: Remove unused saa7164_call_i2c_clients()
Ezequiel García [Wed, 27 Jun 2012 15:52:50 +0000 (12:52 -0300)]
[media] saa7164: Remove unused saa7164_call_i2c_clients()

This function has no users, so it's safe to remove it.
Tested by compilation only.

Signed-off-by: Ezequiel Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx25821: Remove useless struct i2c_algo_bit_data
Ezequiel García [Wed, 27 Jun 2012 15:52:49 +0000 (12:52 -0300)]
[media] cx25821: Remove useless struct i2c_algo_bit_data

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 Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx231xx: Remove useless struct i2c_algo_bit_data
Ezequiel García [Wed, 27 Jun 2012 15:52:48 +0000 (12:52 -0300)]
[media] cx231xx: Remove useless struct i2c_algo_bit_data

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 Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] cx23885: Remove useless struct i2c_algo_bit_data
Ezequiel García [Wed, 27 Jun 2012 15:52:47 +0000 (12:52 -0300)]
[media] cx23885: Remove useless struct i2c_algo_bit_data

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 Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] saa7164: Remove useless struct i2c_algo_bit_data
Ezequiel García [Wed, 27 Jun 2012 15:52:46 +0000 (12:52 -0300)]
[media] saa7164: Remove useless struct i2c_algo_bit_data

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 Garcia <elezegarcia@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] Feature removal: using capture and output capabilities for m2m devices
Sylwester Nawrocki [Thu, 26 Jul 2012 12:30:50 +0000 (09:30 -0300)]
[media] Feature removal: using capture and output capabilities for m2m devices

Identifying a memory-to-memory video device through an ORed output and
capture capability flags is not reliable. Schedule this for removal.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] V4L: Add capability flags for memory-to-memory devices
Sylwester Nawrocki [Thu, 26 Jul 2012 12:30:00 +0000 (09:30 -0300)]
[media] V4L: Add capability flags for memory-to-memory devices

This patch adds new V4L2_CAP_VIDEO_M2M and V4L2_CAP_VIDEO_M2M_MPLANE
capability flags that are intended to be used for memory-to-memory (M2M)
devices, instead of ORed V4L2_CAP_VIDEO_CAPTURE and V4L2_CAP_VIDEO_OUTPUT.

V4L2_CAP_VIDEO_M2M flag is added at the drivers, CAPTURE and OUTPUT
capability flags are left untouched and will be removed in future,
after a transition period required for existing applications to be
adapted to check only for V4L2_CAP_VIDEO_M2M.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Acked-by: Kamil Debski <k.debski@samsung.com>
Acked-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] m5mols: Correct reported ISO values
Sylwester Nawrocki [Tue, 24 Jul 2012 15:12:07 +0000 (12:12 -0300)]
[media] m5mols: Correct reported ISO values

The V4L2_CID_ISO_SENSITIVITY control menu values should be
standard ISO values multiplied by 1000. Multiply all menu
items by 1000 so ISO is properly reported as 50...3200 range.

This applies to kernels 3.5+.

Cc: stable@vger.kernel.org
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] s5p-fimc: Use switch statement for better readability
Sylwester Nawrocki [Tue, 24 Jul 2012 15:06:26 +0000 (12:06 -0300)]
[media] s5p-fimc: Use switch statement for better readability

Use switch statement rather than multiple 'else if'.

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] s5p-fimc: Replace custom err() macro with v4l2_err() macro
Sachin Kamat [Tue, 12 Jun 2012 06:12:26 +0000 (03:12 -0300)]
[media] s5p-fimc: Replace custom err() macro with v4l2_err() macro

Replace custom err() macro with v4l2_err() macro.

[s.nawrocki: added missing end-of-line at the log print]

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] s5p-fimc: Remove V4L2_FL_LOCK_ALL_FOPS flag
Sylwester Nawrocki [Wed, 27 Jun 2012 12:56:17 +0000 (09:56 -0300)]
[media] s5p-fimc: Remove V4L2_FL_LOCK_ALL_FOPS flag

This patch adds locking for open(), close(), poll() and mmap()
file operations in the driver as a follow up to the changes
done in commit 5126f2590bee412e3053de851cb07f531
"v4l2-dev: add flag to have the core lock all file operations".

Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-dev: G_PARM was incorrectly enabled for all video nodes
Hans Verkuil [Thu, 19 Jul 2012 11:49:25 +0000 (08:49 -0300)]
[media] v4l2-dev: G_PARM was incorrectly enabled for all video nodes

G_PARM should only be enabled if:

- vidioc_g_parm is present
- or: it is a video node and vidioc_g_std or tvnorms are set.

Without this additional check v4l2-compliance would complain about
being able to use g_parm when it didn't expect it.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] mem2mem_testdev: set default size and fix colorspace
Hans Verkuil [Wed, 18 Jul 2012 14:33:22 +0000 (11:33 -0300)]
[media] mem2mem_testdev: set default size and fix colorspace

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] mem2mem_testdev: add control events support
Hans Verkuil [Wed, 18 Jul 2012 13:54:59 +0000 (10:54 -0300)]
[media] mem2mem_testdev: add control events support

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-mem2mem: support events in v4l2_m2m_poll
Hans Verkuil [Wed, 18 Jul 2012 13:53:04 +0000 (10:53 -0300)]
[media] v4l2-mem2mem: support events in v4l2_m2m_poll

v4l2_m2m_poll didn't support events, but that's essential if you want to
be able to use control events for example.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] mem2mem_testdev: set bus_info and device_caps
Hans Verkuil [Wed, 18 Jul 2012 13:46:01 +0000 (10:46 -0300)]
[media] mem2mem_testdev: set bus_info and device_caps

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] mem2mem_testdev: convert to the control framework and v4l2_fh
Hans Verkuil [Wed, 18 Jul 2012 13:35:37 +0000 (10:35 -0300)]
[media] mem2mem_testdev: convert to the control framework and v4l2_fh

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] tw9910: Don't access the device in the g_mbus_fmt operation
Laurent Pinchart [Wed, 18 Jul 2012 13:54:00 +0000 (10:54 -0300)]
[media] tw9910: Don't access the device in the g_mbus_fmt operation

The g_mbus_fmt operation only needs to return the current mbus frame
format and doesn't need to configure the hardware to do so. Fix it to
avoid requiring the chip to be powered on when calling the operation.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] ov772x: Don't access the device in the g_mbus_fmt operation
Laurent Pinchart [Wed, 18 Jul 2012 13:53:59 +0000 (10:53 -0300)]
[media] ov772x: Don't access the device in the g_mbus_fmt operation

The g_mbus_fmt operation only needs to return the current mbus frame
format and doesn't need to configure the hardware to do so. Fix it to
avoid requiring the chip to be powered on when calling the operation.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] ov2640: Don't access the device in the g_mbus_fmt operation
Laurent Pinchart [Wed, 18 Jul 2012 13:53:58 +0000 (10:53 -0300)]
[media] ov2640: Don't access the device in the g_mbus_fmt operation

The g_mbus_fmt operation only needs to return the current mbus frame
format and doesn't need to configure the hardware to do so. Fix it to
avoid requiring the chip to be powered on when calling the operation.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] soc-camera: Pass the physical device to the power operation
Laurent Pinchart [Wed, 18 Jul 2012 13:53:57 +0000 (10:53 -0300)]
[media] soc-camera: Pass the physical device to the power operation

There will be no soc_camera_device instance with a soc-camera device is
used with a non soc-camera host, so we won't be able to pass the
soc_camera_device fake platform device to board code. Pass the physical
device instead.

The argument is currently not used by any board file so this is safe.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] soc-camera: Don't fail at module init time if no device is present
Laurent Pinchart [Wed, 18 Jul 2012 13:53:56 +0000 (10:53 -0300)]
[media] soc-camera: Don't fail at module init time if no device is present

The soc-camera module exports functions that are needed by soc-camera
client drivers even when not running in soc-camera mode. Replace the
platform_driver_probe() with a platform_driver_register() call to avoid
module load failures if no soc-camera device is present.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-ioctl.c: zero the v4l2_dv_timings_cap struct
Hans Verkuil [Wed, 11 Jul 2012 12:15:06 +0000 (09:15 -0300)]
[media] v4l2-ioctl.c: zero the v4l2_dv_timings_cap struct

Zero all fields after the first type field.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Cc: stable@kernel.org
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-dev: forgot to add VIDIOC_DV_TIMINGS_CAP
Hans Verkuil [Wed, 11 Jul 2012 12:12:45 +0000 (09:12 -0300)]
[media] v4l2-dev: forgot to add VIDIOC_DV_TIMINGS_CAP

The VIDIOC_DV_TIMINGS_CAP ioctl check wasn't added to determine_valid_ioctls().
This caused this ioctl to always return -ENOTTY.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Cc: stable@kernel.org
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: Enable selection of the ADV7343 and THS7303
Manjunath Hadli [Fri, 29 Jun 2012 06:24:29 +0000 (03:24 -0300)]
[media] davinci: vpif: Enable selection of the ADV7343 and THS7303

The auto selection of pertinent helper chips (VIDEO_HELPER_CHIPS_AUTO)
should select the ADV7343 and THS7303 driver, which is used by VPIF
display driver.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: add build configuration for vpif drivers
Manjunath Hadli [Fri, 29 Jun 2012 06:23:37 +0000 (03:23 -0300)]
[media] davinci: vpif: add build configuration for vpif drivers

add generic build configuration for vpif capture
and display drivers as it is common for DM6467/DA850/OMAP-L138.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: Add suspend/resume callbacks to vpif driver
Manjunath Hadli [Fri, 13 Apr 2012 07:51:13 +0000 (04:51 -0300)]
[media] davinci: vpif: Add suspend/resume callbacks to vpif driver

add clock enable and disable in probe and remove functions.
Probe will succeed only if the device clock is provided instead of
assuming that the clock is always enabled. VPIF clock has to be
dealt with during suspend and resume. Implement power management
callbacks to VPIF driver to disable/enable clock on suspend/resume
respectively.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif capture:Add power management support
Manjunath Hadli [Fri, 13 Apr 2012 07:50:55 +0000 (04:50 -0300)]
[media] davinci: vpif capture:Add power management support

Implement power management operations - suspend and resume as part of
dev_pm_ops for VPIF capture driver.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif display: Add power management support
Manjunath Hadli [Fri, 13 Apr 2012 07:50:35 +0000 (04:50 -0300)]
[media] davinci: vpif display: Add power management support

Implement power management operations - suspend and resume as part of
dev_pm_ops for VPIF display driver.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: add support for clipping on output data
Manjunath Hadli [Fri, 29 Jun 2012 06:20:12 +0000 (03:20 -0300)]
[media] davinci: vpif: add support for clipping on output data

add hardware clipping support for VPIF output data. This
is needed as it is possible that the external encoder
might get confused between the FF or 00 which are a part
of the data and that of the SAV or EAV codes.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif display: migrate driver to videobuf2
Lad, Prabhakar [Thu, 28 Jun 2012 12:28:36 +0000 (09:28 -0300)]
[media] davinci: vpif display: migrate driver to videobuf2

This patch migrates VPIF display driver to videobuf2 framework.

Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif capture: migrate driver to videobuf2
Lad, Prabhakar [Thu, 28 Jun 2012 12:28:05 +0000 (09:28 -0300)]
[media] davinci: vpif capture: migrate driver to videobuf2

This patch migrates VPIF capture driver to videobuf2 framework.

Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif capture: size up the memory for the buffers from the buffer...
Manjunath Hadli [Fri, 13 Apr 2012 07:49:34 +0000 (04:49 -0300)]
[media] davinci: vpif capture: size up the memory for the buffers from the buffer pool

Size up the memory for the buffers from the buffer pool allocated in board
file.  Then adjust the reqbuf count depending the available memory.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif display: size up the memory for the buffers from the buffer...
Manjunath Hadli [Fri, 13 Apr 2012 07:49:10 +0000 (04:49 -0300)]
[media] davinci: vpif display: size up the memory for the buffers from the buffer pool

Size up the memory for the buffers from the buffer pool allocated in board
file.  Then adjust the reqbuf count depending the available memory.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: fix setting of data width in config_vpif_params() function
Manjunath Hadli [Fri, 13 Apr 2012 07:47:17 +0000 (04:47 -0300)]
[media] davinci: vpif: fix setting of data width in config_vpif_params() function

fix setting of data width in config_vpif_params() function,
which was wrongly set.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: make request_irq flags as shared
Manjunath Hadli [Fri, 13 Apr 2012 07:44:31 +0000 (04:44 -0300)]
[media] davinci: vpif: make request_irq flags as shared

omap-l138 shares the interrupt between capture and display.
Make sure we are able to request for the same irq number
by making a shared irq request.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: make generic changes to re-use the vpif drivers on da850/omap...
Manjunath Hadli [Fri, 13 Apr 2012 07:44:00 +0000 (04:44 -0300)]
[media] davinci: vpif: make generic changes to re-use the vpif drivers on da850/omap-l138 soc

change the dm646x specific strings in the driver to make
them generic across platforms. In this case change all the
strings which have a dm646x connotation to vpif which is a
platform independent ip.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] davinci: vpif: add check for genuine interrupts in the isr
Manjunath Hadli [Fri, 13 Apr 2012 07:43:10 +0000 (04:43 -0300)]
[media] davinci: vpif: add check for genuine interrupts in the isr

As the same interrupt is shared between capture and display devices,
sometimes we get isr calls where the interrupt might not genuinely belong
to capture or display. Hence, add a condition in the isr to check for
interrupt ownership and channel number to make sure we do not
service wrong interrupts.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] videobuf-dma-contig: restore buffer mapping for uncached bufers
Lad, Prabhakar [Fri, 22 Jun 2012 09:19:28 +0000 (06:19 -0300)]
[media] videobuf-dma-contig: restore buffer mapping for uncached bufers

from commit a8f3c203e19b702fa5e8e83a9b6fb3c5a6d1cce4
restore the mapping scheme for uncached buffers,
which was changed in a common scheme for cached and uncached.
This apparently was wrong, and was probably intended only for cached buffers.
the fix fixes the crash observed while mapping uncached buffers.

Signed-off-by: Lad, Prabhakar <prabhakar.lad@ti.com>
Signed-off-by: Hadli, Manjunath <manjunath.hadli@ti.com>
Acked-by: Federico Vaga <federico.vaga@gmail.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Cc: stable@kernel.org
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] shark2: New driver for the Griffin radioSHARK v2 USB radio receiver
Hans de Goede [Sat, 23 Jun 2012 07:39:58 +0000 (04:39 -0300)]
[media] shark2: New driver for the Griffin radioSHARK v2 USB radio receiver

This driver consists of 2 parts, a generic tea5777 driver and a driver
for the Griffin radioSHARK v2 USB radio receiver, which is the only driver
using the generic tea5777 for now.

This first version only implements FM support, once the the new
VIDIOC_ENUM_FREQ_BANDS API is upstream I'll also add AM support.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-ioctl: Don't assume file->private_data always points to a v4l2_fh
Hans Verkuil [Sat, 7 Jul 2012 19:11:11 +0000 (16:11 -0300)]
[media] v4l2-ioctl: Don't assume file->private_data always points to a v4l2_fh

Commit efbceecd4522a41b8442c6b4f68b4508d57d1ccf, adds a number of helper
functions for ctrl related ioctls to v4l2-ioctl.c, these helpers assume that
if file->private_data != NULL, it points to a v4l2_fh, which is only the case
for drivers which actually use v4l2_fh.

This breaks for example bttv which use the "filedata" pointer for its own uses,
and now all the ctrl ioctls try to use whatever its filedata points to as
v4l2_fh and think it has a ctrl_handler, leading to:

[  142.499214] BUG: unable to handle kernel NULL pointer dereference at 0000000000000021
[  142.499270] IP: [<ffffffffa01cb959>] v4l2_queryctrl+0x29/0x230 [videodev]
[  142.514649]  [<ffffffffa01c7a77>] v4l_queryctrl+0x47/0x90 [videodev]
[  142.517417]  [<ffffffffa01c58b1>] __video_do_ioctl+0x2c1/0x420 [videodev]
[  142.520116]  [<ffffffffa01c7ee6>] video_usercopy+0x1a6/0x470 [videodev]
...

This patch adds the missing test_bit(V4L2_FL_USES_V4L2_FH, &vfd->flags) tests
to the ctrl ioctl helpers v4l2_fh paths, fixing the issues with for example
the bttv driver.

Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] v4l2-ctrls: Teach v4l2-ctrls that V4L2_CID_AUTOBRIGHTNESS is a boolean
Hans de Goede [Sun, 1 Jul 2012 14:26:13 +0000 (11:26 -0300)]
[media] v4l2-ctrls: Teach v4l2-ctrls that V4L2_CID_AUTOBRIGHTNESS is a boolean

And document V4L2_CID_AUTOBRIGHTNESS.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca: Don't use video_device_node_name in v4l2_device release handler
Hans de Goede [Sat, 7 Jul 2012 15:11:57 +0000 (12:11 -0300)]
[media] gspca: Don't use video_device_node_name in v4l2_device release handler

When the v4l2_device release handler gets called the kobject under
vdev->dev has already been released, so we cannot use kobject_name on
it (which video_device_node_name does).

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_xirlink_cit: Grab backlight compensation control while streaming
Hans de Goede [Sat, 7 Jul 2012 19:47:35 +0000 (16:47 -0300)]
[media] gspca_xirlink_cit: Grab backlight compensation control while streaming

As it cannot be changed while streaming.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_t613: Disable CIF resolutions
Hans de Goede [Sat, 7 Jul 2012 11:55:06 +0000 (08:55 -0300)]
[media] gspca_t613: Disable CIF resolutions

These are broken with my test cam and I've been unable to fix them.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_stv0680: Remove empty ctrls array
Hans de Goede [Sat, 7 Jul 2012 20:12:14 +0000 (17:12 -0300)]
[media] gspca_stv0680: Remove empty ctrls array

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gscpa_spca561: Add brightness control for rev12a cams
Hans de Goede [Mon, 2 Jul 2012 18:29:56 +0000 (15:29 -0300)]
[media] gscpa_spca561: Add brightness control for rev12a cams

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_sn9c2028: Remove empty ctrls array
Hans de Goede [Sun, 1 Jul 2012 18:19:33 +0000 (15:19 -0300)]
[media] gspca_sn9c2028: Remove empty ctrls array

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_konica: Fix init sequence
Hans de Goede [Sat, 30 Jun 2012 12:03:21 +0000 (09:03 -0300)]
[media] gspca_konica: Fix init sequence

The konica needs a freaking large time (circa 6.5 seconds) to "boot", and
does not want to be bothered while doing so, so sleep for 6 seconds, and
then query its status register at 100ms intervals until it becomes ready.

This removes the "reg_w err: -32" messages shown in dmesg whenever a
konica cam gets initialized, and also fixes the camera not working when
an app tries to use it directly after it has been plugged in and after
a suspend/resume cycle.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca: Add reset_resume callback to all sub-drivers
Hans de Goede [Sat, 30 Jun 2012 09:44:47 +0000 (06:44 -0300)]
[media] gspca: Add reset_resume callback to all sub-drivers

1) The gspca-core's suspend/resume code is such that resume being called after
a reset is safe / ok.

2) All devices tested sofar seem to need the reset_resume callback to work
properly over a suspend

3) The USB-core won't call the reset_resume callback for devices which don't
need it

Thus it seems the simplest and the best to just add the callback to all
sub-drivers, rather then adding the callbacks one-by-one as each driver gets
tested with suspend/resume.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca_benq: Remove empty ctrls array
Hans de Goede [Sat, 30 Jun 2012 08:53:36 +0000 (05:53 -0300)]
[media] gspca_benq: Remove empty ctrls array

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca: Remove bogus JPEG quality controls from various sub-drivers
Hans de Goede [Wed, 27 Jun 2012 19:48:33 +0000 (16:48 -0300)]
[media] gspca: Remove bogus JPEG quality controls from various sub-drivers

Various gspca-subdrivers have a JPEG quality control which only changes
the quantization tables in the JPEG headers send to user-space without
making any changes to the settings of the bridge. Remove these bogus / wrong
controls.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca: always call v4l2_ctrl_handler_setup after start
Hans Verkuil [Fri, 15 Jun 2012 08:24:26 +0000 (05:24 -0300)]
[media] gspca: always call v4l2_ctrl_handler_setup after start

This ensures the controls are setup correctly.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca: clear priv field and disable relevant ioctls
Hans Verkuil [Wed, 23 May 2012 20:17:12 +0000 (17:17 -0300)]
[media] gspca: clear priv field and disable relevant ioctls

The v4l2_pix_format priv field must be 0, so zero it.

Also disable ioctls that are not implemented by a subdriver.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] gspca-topro: convert to the control framework
Hans Verkuil [Mon, 21 May 2012 08:37:39 +0000 (05:37 -0300)]
[media] gspca-topro: convert to the control framework

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
12 years ago[media] vc032x: convert to the control framework
Hans Verkuil [Fri, 18 May 2012 09:45:30 +0000 (06:45 -0300)]
[media] vc032x: convert to the control framework

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>