linux-stable.git/drivers/media/common, branch v4.12.2

[media] b2c2: constify nxt200x_config structure

2017-04-17T15:18:18+00:00

Declare nxt200x_config structure as const as it is only passed as
an argument to the function dvb_attach. dvb_attach calls its first
argument on the rest of its arguments. The first argument of
dvb_attach in the changed case is nxt200x_attach and the parameter of
this function to which the object reference is passed is of type
const. So, nxt200x_config structures having this property can be made
const.

File size before:
   text	   data	    bss	    dec	    hex	filename
   7566	    568	      0	   8134	   1fc6	common/b2c2/flexcop-fe-tuner.o

File size after:
   text	   data	    bss	    dec	    hex	filename
   7582	    536	      0	   8118	   1fb6	common/b2c2/flexcop-fe-tuner.o

Signed-off-by: Bhumika Goyal 
Signed-off-by: Mauro Carvalho Chehab

[media] v4l2-tpg: don't clamp XV601/709 to lim range

2017-04-10T17:58:06+00:00

The XV601/709 encodings are special: they signal limited range, but use the full range
to encode a larger gamut with R', G' and B' values outside the [0-1] range.

So don't clamp to limited range for these two encodings.

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

[media] saa7146: use setup_timer

2017-04-10T17:46:12+00:00

Use setup_timer() instead of init_timer() to simplify the code.

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

[media] tveeprom: get rid of unused arg on tveeprom_hauppauge_analog()

2017-03-03T10:35:02+00:00

tveeprom_hauppauge_analog() used to need the I2C adapter in
order to print debug messages. As it now uses pr_foo() facilities
since commit 6037b3ca28f4 ("[media] tveeprom: print log messages
using pr_foo()"), the first argument of the function is not
needed anymore.

So, get rid of it.

Signed-off-by: Mauro Carvalho Chehab

[media] b2c2: use IS_REACHABLE() instead of open-coding it

2017-02-03T09:12:59+00:00

The FE_SUPPORTED() macro is basically the same as IS_REACHABLE, except
that it causes a warning with gcc-7:

common/b2c2/flexcop-fe-tuner.c:30:1: error: this use of "defined" may not be portable [-Werror=expansion-to-defined]
common/b2c2/flexcop-fe-tuner.c:30:1: error: this use of "defined" may not be portable [-Werror=expansion-to-defined]
common/b2c2/flexcop-fe-tuner.c:30:1: error: this use of "defined" may not be portable [-Werror=expansion-to-defined]

Using IS_REACHABLE() to define it avoids the warning.

Fixes: 3785bc170f79 ("[media] b2c2: break it into common/pci/usb directories")

Signed-off-by: Arnd Bergmann 
Signed-off-by: Mauro Carvalho Chehab

[media] rc-main: assign driver type during allocation

2017-01-30T15:59:57+00:00

The driver type can be assigned immediately when an RC device
requests to the framework to allocate the device.

This is an 'enum rc_driver_type' data type and specifies whether
the device is a raw receiver or scancode receiver. The type will
be given as parameter to the rc_allocate_device device.

Change accordingly all the drivers calling rc_allocate_device()
so that the device type is specified during the rc device
allocation. Whenever the device type is not specified, it will be
set as RC_DRIVER_SCANCODE which was the default '0' value.

Suggested-by: Sean Young 
Signed-off-by: Andi Shyti 
Signed-off-by: Sean Young 
Signed-off-by: Mauro Carvalho Chehab

[media] rc: raw IR drivers cannot handle cec, unknown or other

2017-01-30T14:40:50+00:00

unknown and other are for IR protocols for which we have no decoder,
so the raw IR drivers have no chance of generating them. cec is not
an IR protocol.

Signed-off-by: Sean Young 
Cc: Jiri Kosina 
Cc: Benjamin Tissoires 
Cc: Bruno Prémont 
Signed-off-by: Mauro Carvalho Chehab

[media] media: Drop FSF's postal address from the source code files

2017-01-27T13:38:09+00:00

Drop the FSF's postal address from the source code files that typically
contain mostly the license text. Of the 628 removed instances, 578 are
outdated.

The patch has been created with the following command without manual edits:

git grep -l "675 Mass Ave\|59 Temple Place\|51 Franklin St" -- \
	drivers/media/ include/media|while read i; do i=$i perl -e '
open(F,"< $ENV{i}");
$a=join("", );
$a =~ s/[ \t]*\*\n.*You should.*\n.*along with.*\n.*(\n.*USA.*$)?\n//m
	&& $a =~ s/(^.*)Or, (point your browser to) /$1To obtain the license, $2\n$1/m;
close(F);
open(F, "> $ENV{i}");
print F $a;
close(F);'; done

Signed-off-by: Sakari Ailus

[media] v4l2-tpg: Init hv_enc field with a valid value

2016-11-29T14:12:13+00:00

Zero is not a valid value for hsv_enc. Set the field to a valid
initial value.

This is not a problem for vivid, because it sets the field to 180 via
tpg_s_hsv_enc() on the control initialization, but it might be a source
of errors for other drivers that use this code.

Signed-off-by: Ricardo Ribalda Delgado 
Signed-off-by: Hans Verkuil 
Signed-off-by: Mauro Carvalho Chehab

[media] saa7146: Fix for while releasing video buffers

2016-11-23T21:52:12+00:00

Fix this bug:
	"[BUG] process stuck when closing saa7146 [dvb_ttpci]"

Release queued DMA buffers when ending streaming, so that
videobuf_waiton() doesn't block forever.

As reported, this fixes avoids occasional lockup of process reading from
video device, which manifests in such log:

INFO: task ffmpeg:9864 blocked for more than 120 seconds.
      Tainted: P           O    4.6.7 #3
"echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
ffmpeg          D ffff880177cc7b00     0  9864      1 0x00000000
 ffff880177cc7b00 0000000000000202 0000000000000202 ffffffff8180b4c0
 ffff88019d79e4c0 ffffffff81064050 ffff880177cc7ae0 ffff880177cc8000
 ffff880177cc7b18 ffff8801fd41d648 ffff8802307acca0 ffff8802307acc70
Call Trace:
 [] ? preempt_count_add+0x89/0xab
 [] schedule+0x86/0x9e
 [] ? schedule+0x86/0x9e
 [] videobuf_waiton+0x131/0x15e [videobuf_core]
 [] ? wait_woken+0x6d/0x6d
 [] saa7146_dma_free+0x39/0x5b [saa7146_vv]
 [] buffer_release+0x2a/0x3e [saa7146_vv]
 [] videobuf_vm_close+0xd8/0x103 [videobuf_dma_sg]
 [] remove_vma+0x25/0x4d
 [] exit_mmap+0xce/0xf7
 [] mmput+0x4e/0xe2
 [] do_exit+0x372/0x920
 [] do_group_exit+0x3c/0x98
 [] get_signal+0x4e8/0x56e
 [] ? task_dead_fair+0xd/0xf
 [] do_signal+0x23/0x521
 [] ? _raw_spin_unlock_irqrestore+0x13/0x25
 [] ? hrtimer_try_to_cancel+0xd7/0x104
 [] ? ktime_get+0x4c/0xa1
 [] ? update_rmtp+0x46/0x5b
 [] ? hrtimer_nanosleep+0xe4/0x10e
 [] ? hrtimer_init+0xeb/0xeb
 [] exit_to_usermode_loop+0x4f/0x93
 [] syscall_return_slowpath+0x3b/0x46
 [] entry_SYSCALL_64_fastpath+0x8d/0x8f

Reported-by: Philipp Matthias Hahn 
Tested-by: Philipp Matthias Hahn 
Signed-off-by: Andrey Utkin 
Signed-off-by: Mauro Carvalho Chehab