Re: [PATCH] staging: Replaced BUG_ON with warnings

2016-12-02 Thread Allen
On Sat, Dec 3, 2016 at 12:32 PM, Shilpa Puttegowda wrote: > From: Shilpa P > > Don't crash the Kernel for driver errors > > Signed-off-by: Shilpa P > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 12 ++-- > 1 file

[PATCH] staging: Replaced BUG_ON with warnings

2016-12-02 Thread Shilpa Puttegowda
From: Shilpa P Don't crash the Kernel for driver errors Signed-off-by: Shilpa P --- drivers/staging/media/bcm2048/radio-bcm2048.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git

cron job: media_tree daily build: ERRORS

2016-12-02 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Dec 3 05:00:19 CET 2016 media-tree git hash:365fe4e0ce218dc5ad10df17b150a366b6015499 media_build

Re: [PATCH v4 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2016-12-02 Thread kbuild test robot
Hi Stanimir, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.9-rc7 next-20161202] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Stanimir-Varbanov/Qualcomm

Re: [PATCH v4 9/9] media: venus: enable building of Venus video driver

2016-12-02 Thread kbuild test robot
Hi Stanimir, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on next-20161202] [cannot apply to v4.9-rc7] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Stanimir-Varbanov

[PATCH v4l-utils 5/6] ir-keytable: "-p all" or "-p mce-kdb" does not work

2016-12-02 Thread Sean Young
When writing to the sysfs protocols file, use the underscore variant. The kernel does not accept "mce-kdb" and it never did. Signed-off-by: Sean Young --- utils/keytable/keytable.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/keytable/keytable.c

[PATCH v4l-utils 4/6] ir-ctl: improve scancode validation

2016-12-02 Thread Sean Young
Ensure rc6 mce is that just that and that nec32 is not necx or nec. Signed-off-by: Sean Young --- utils/ir-ctl/ir-ctl.c| 4 ++-- utils/ir-ctl/ir-encode.c | 18 ++ utils/ir-ctl/ir-encode.h | 1 + 3 files changed, 21 insertions(+), 2 deletions(-) diff --git

[PATCH v4l-utils 3/6] ir-ctl: 0 is valid scancode

2016-12-02 Thread Sean Young
Same for 0x. Signed-off-by: Sean Young --- utils/ir-ctl/ir-ctl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/ir-ctl/ir-ctl.c b/utils/ir-ctl/ir-ctl.c index f19bd05..768daad 100644 --- a/utils/ir-ctl/ir-ctl.c +++ b/utils/ir-ctl/ir-ctl.c @@

[PATCH v4l-utils 6/6] ir-ctl: rename rc5x to rc5x_20

2016-12-02 Thread Sean Young
There are many extended rc5 protocols and we can only generate the 20 bit variant. Signed-off-by: Sean Young --- utils/ir-ctl/ir-ctl.1.in | 7 --- utils/ir-ctl/ir-encode.c | 4 ++-- utils/ir-ctl/ir-encode.h | 2 +- 3 files changed, 7 insertions(+), 6 deletions(-) diff --git

[PATCH v4l-utils 2/6] ir-ctl: fix rc5x encoding

2016-12-02 Thread Sean Young
6th command bit was missing. Signed-off-by: Sean Young --- utils/ir-ctl/ir-encode.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/utils/ir-ctl/ir-encode.c b/utils/ir-ctl/ir-encode.c index 1bf0ac6..704ce95 100644 --- a/utils/ir-ctl/ir-encode.c +++

[PATCH v4l-utils 1/6] ir-ctl: uninitialised memory used

2016-12-02 Thread Sean Young
We might end up with a corrupt rc6. Signed-off-by: Sean Young --- utils/ir-ctl/ir-encode.c | 1 + 1 file changed, 1 insertion(+) diff --git a/utils/ir-ctl/ir-encode.c b/utils/ir-ctl/ir-encode.c index a0d2f4c..1bf0ac6 100644 --- a/utils/ir-ctl/ir-encode.c +++

[PATCH 7/8] [media] rc5x: 6th command bit is S2 bit

2016-12-02 Thread Sean Young
The 2nd stop bit in rc5 is reused as an inverted 6th command bit in 20 bits rc5x. Currently the rc5x decoder sets the 6th command bit as an inverted duplicate of the lowest system bit; as a result we do not have all the command bits. Note that there are no rc5x keymaps present. Signed-off-by:

[PATCH 3/8] [media] lirc: LIRC_{G,S}ET_SEND_MODE fail if device cannot transmit

2016-12-02 Thread Sean Young
These ioctls should not succeed if the device cannot send. Also make it clear that these ioctls should return the lirc mode, although the actual value does not change. Signed-off-by: Sean Young --- drivers/media/rc/ir-lirc-codec.c | 10 -- 1 file changed, 8 insertions(+),

[PATCH 6/8] [media] rc: allow software timeout to be set

2016-12-02 Thread Sean Young
Both the iguanair and the technotrend usb ir do not do any timeout handling in hardware, so timeout is entirely done in ir_raw_event_store_with_filter(). Any sensible timeout value will do, so allow it to be set using LIRC_SET_REC_TIMEOUT. Signed-off-by: Sean Young ---

[PATCH 2/8] [media] lirc_dev: LIRC_{G,S}ET_REC_MODE do not work

2016-12-02 Thread Sean Young
Since "273b902 [media] lirc_dev: use LIRC_CAN_REC() define" these ioctls no longer work. Signed-off-by: Sean Young Cc: Andi Shyti Cc: # v4.8+ --- drivers/media/rc/lirc_dev.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH 8/8] [media] rc5x: document that this is the 20 bit variant

2016-12-02 Thread Sean Young
There are many variants of extended rc5. This implements the 20 bit version. Signed-off-by: Sean Young Cc: David Härdeman --- drivers/media/rc/ir-rc5-decoder.c | 6 +++--- drivers/media/rc/rc-main.c| 2 +- include/media/rc-map.h| 8

[PATCH 4/8] [media] em28xx: IR protocol not reported correctly

2016-12-02 Thread Sean Young
Report the correct NEC variant. Signed-off-by: Sean Young --- drivers/media/usb/em28xx/em28xx-input.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/drivers/media/usb/em28xx/em28xx-input.c b/drivers/media/usb/em28xx/em28xx-input.c index

[PATCH 1/8] [media] mceusb: LIRC_SET_SEND_CARRIER returns 0 on success

2016-12-02 Thread Sean Young
LIRC_SET_SEND_CARRIER ioctl should not return the carrier used, it should return 0. Signed-off-by: Sean Young --- drivers/media/rc/mceusb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c index

[PATCH 5/8] [media] serial_ir: generate timeout

2016-12-02 Thread Sean Young
No timeout is generated by serial_ir since the port only generates interrupts on edges. Some IR protocols like rc6 and rc5 need a trailing space or timeout so they know there are no more bits coming. Without it, the current key will only be reported once some more IR occurs. Signed-off-by: Sean

Re: [RFC v4 19/21] omap3isp: Allocate the media device dynamically

2016-12-02 Thread Sakari Ailus
Hi Hans, On Tue, Nov 22, 2016 at 11:05:49AM +0100, Hans Verkuil wrote: ... > >@@ -2183,7 +2185,7 @@ static int isp_subdev_notifier_complete(struct > >v4l2_async_notifier *async) > > if (ret < 0) > > return ret; > > > >-return media_device_register(>media_dev); > >+return

Re: [PATCH v3 4/4] stk1160: Give the chip some time to retrieve data from AC97 codec.

2016-12-02 Thread Mauro Carvalho Chehab
Em Sun, 27 Nov 2016 12:12:36 +0100 Marcel Hasler escreveu: > The STK1160 needs some time to transfer data from the AC97 registers into its > own. On some > systems reading the chip's own registers to soon will return wrong values. > The "proper" way to > handle this would

Re: [PATCH v3 3/4] stk1160: Add module param for setting the record gain.

2016-12-02 Thread Mauro Carvalho Chehab
Em Sun, 27 Nov 2016 12:11:48 +0100 Marcel Hasler escreveu: > Allow setting a custom record gain for the internal AC97 codec (if > available). This can be > a value between 0 and 15, 8 is the default and should be suitable for most > users. The Windows > driver also sets

[PATCH v2 3/3] uvcvideo: add a metadata device node

2016-12-02 Thread Guennadi Liakhovetski
Some UVC video cameras contain metadata in their payload headers. This patch extracts that data, skipping the standard part of the header, on both bulk and isochronous endpoints and makes it available to the user space on a separate video node, using the V4L2_CAP_META_CAPTURE capability and the

Re: [PATCH] bdisp: Clean up file handle in open() error path.

2016-12-02 Thread Fabien DESSENNE
Hi Shailendra, Thank you for the patch, it's good for me. On 12/02/2016 05:48 AM, Shailendra Verma wrote: > The File handle is not yet added in the vdev list.So no need to call > v4l2_fh_del(>fh)if it fails to create control. > > Signed-off-by: Shailendra Verma

[RFC][PATCH] [media] atmel-isc: add the isc pipeline function

2016-12-02 Thread Songjun Wu
Image Sensor Controller has an internal image processor. It can convert raw format to the other formats, like RGB565, YUV420P. A module parameter 'sensor_preferred' is used to enable or disable the pipeline function. Some v4l2 controls are added to tuning the image when the pipeline function is