cron job: media_tree daily build: WARNINGS

2014-11-30 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: Mon Dec 1 04:00:14 CET 2014 git branch: test git hash: 504febc3f98c87a8bebd8f2f274f32c0724131e4 gcc versio

Re: [PATCH] [media] gspca_touptek: Add support for ToupTek UCMOS series USB cameras

2014-11-30 Thread John McMaster
On 10/23/2014 06:09 AM, Hans de Goede wrote: > Hi, > > On 10/05/2014 08:43 AM, John McMaster wrote: >> Adds support for AmScope MU800 / ToupTek UCMOS08000KPB USB microscope camera. > First of all many thanks for writing this driver, and also for submitting it > upstream. We always appreciate it a l

[PATCH] [media] gspca_touptek: Add support for ToupTek UCMOS series USB cameras

2014-11-30 Thread John McMaster
Adds support for AmScope MU800 / ToupTek UCMOS08000KPB USB microscope camera. Signed-off-by: John McMaster --- drivers/media/usb/gspca/Kconfig | 10 + drivers/media/usb/gspca/Makefile |2 + drivers/media/usb/gspca/touptek.c | 729 + 3 files changed,

Re: [PATCH] linux-firmware: Add firmware files for Siano DTV devices

2014-11-30 Thread Kyle McMartin
On Sun, Nov 09, 2014 at 09:58:19PM -0200, Mauro Carvalho Chehab wrote: > From: Mauro Carvalho Chehab > > Acked-by: Rainer Miethling > Signed-off-by: Mauro Carvalho Chehab > --- > LICENCE.siano | 31 +++ > WHENCE| 18

Random memory corruption of fe[1]->dvb pointer

2014-11-30 Thread Benjamin Larsson
While working on a driver I noticed that I had trouble unloading the module after testing, it crashed while running dvb_usbv2_adapter_frontend_exit. So I added a print out of some pointers and got this: Init: usb 1-1: dvb_usbv2_adapter_frontend_init: adap=fe[0] 88006afa6818 usb 1-1: dvb_usb

[PATCH 1/1] [media] V4L2: Deletion of an unnecessary check before the function call "vb2_put_vma"

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 30 Nov 2014 23:10:51 +0100 The vb2_put_vma() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drive

[PATCH 1/1] [media] ddbridge: Deletion of an unnecessary check before the function call "dvb_unregister_device"

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 30 Nov 2014 22:50:20 +0100 The dvb_unregister_device() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

Re: [PATCH] media: v4l2-subdev.h: drop the guard CONFIG_VIDEO_V4L2_SUBDEV_API for v4l2_subdev_get_try_*()

2014-11-30 Thread Prabhakar Lad
Hi Laurent, On Sun, Nov 30, 2014 at 9:16 PM, Laurent Pinchart wrote: > On Sunday 30 November 2014 21:05:50 Prabhakar Lad wrote: >> On Sat, Nov 29, 2014 at 7:12 PM, Laurent Pinchart wrote: >> > Hi Prabhakar, >> >> [Snip] >> >> >> > Sure. That's a better choice than removing the config option depen

Re: [PATCH] media: v4l2-subdev.h: drop the guard CONFIG_VIDEO_V4L2_SUBDEV_API for v4l2_subdev_get_try_*()

2014-11-30 Thread Laurent Pinchart
On Sunday 30 November 2014 21:05:50 Prabhakar Lad wrote: > On Sat, Nov 29, 2014 at 7:12 PM, Laurent Pinchart wrote: > > Hi Prabhakar, > > [Snip] > > >> > Sure. That's a better choice than removing the config option dependency > >> > of the fields struct v4l2_subdev. > > > > Decoupling CONFIG_VID

Re: [PATCH] media: v4l2-subdev.h: drop the guard CONFIG_VIDEO_V4L2_SUBDEV_API for v4l2_subdev_get_try_*()

2014-11-30 Thread Prabhakar Lad
Hi Laurent, On Sat, Nov 29, 2014 at 7:12 PM, Laurent Pinchart wrote: > Hi Prabhakar, > [Snip] >> > Sure. That's a better choice than removing the config option dependency of >> > the fields struct v4l2_subdev. > > Decoupling CONFIG_VIDEO_V4L2_SUBDEV_API from the availability of the in-kernel > pa

[PATCH 2/2] [media] tuners-si2157: One function call less in si2157_init() after error detection

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 30 Nov 2014 20:48:24 +0100 The release_firmware() function was called in some cases by the si2157_init() function during error handling even if the passed variable contained still a null pointer. This implementation detail could be improved by the introduction of a

[PATCH 1/2] [media] tuners: Deletion of unnecessary checks before the function call "release_firmware"

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 30 Nov 2014 19:05:48 +0100 The release_firmware() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/2] [media] tuners: Deletion of two unnecessary checks

2014-11-30 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 30 Nov 2014 20:50:15 +0100 Another update suggestion was taken into account after a patch was applied from static source code analysis. Markus Elfring (2): Deletion of unnecessary checks before the function call "release_firmware" One function call less in si2

Re: Kernel 3.17.0 broke xc4000-based DTV1800h

2014-11-30 Thread István , Varga
By the way, from the xc4000_firmware.tar.gz package, the only files that are actually needed are: build_fw.c (source code of simple program to write the firmware file) xc4000_firmwares.h (header file from Xceive) xc4000_scodes.h (also from Xceive) Everything else is related to extracting the

Re: Kernel 3.17.0 broke xc4000-based DTV1800h

2014-11-30 Thread István , Varga
> On 16 Oct 2014, at 17:33, Rodney Baker iinet.net.au> wrote: > > Since installing kernel 3.17.0-1.gc467423-desktop (on openSuSE 13.1) my > xc4000/zl10353/cx88 based DTV card has failed to initialise on boot. Apparently, the default firmware file name has been changed to dvb-fe-xc4000-1.4.1.fw, a

Re: [PATCH] drivers:media: Add proper sanity checking for register setting to variable in lg2160.c

2014-11-30 Thread Michael Ira Krufky
On Sat, Nov 29, 2014 at 10:04 PM, Nicholas Krause wrote: > Fixs issue with setting the variable value of val in the > functionm,lg2161_set_output_interface > and not sanity checking if the value has been correctly set with the correct > value of the struct > state of type lg216x_state passed to

[REVIEW PATCH v2.1 09/11] smiapp: Obtain device information from the Device Tree if OF node exists

2014-11-30 Thread Sakari Ailus
Platform data support is retained. of_property_read_u64_array() isn't used yet as it's not in yet. Signed-off-by: Sakari Ailus --- since v2: - Don't use of_property_read_u64_array() yet. drivers/media/i2c/smiapp/smiapp-core.c | 114 +++- 1 file changed, 112 inserti

[REVIEW PATCH v2.1 08/11] of: smiapp: Add documentation

2014-11-30 Thread Sakari Ailus
Document the smiapp device tree properties. Signed-off-by: Sakari Ailus --- since v2: - Cleanups - Removed clock-names property documentation - Port node documentation was really endpoint node documentation - Added remote-endpoint as mandatory endpoint node properties .../devicetree/bindings/me

Re: ISDB caption support

2014-11-30 Thread Devin Heitmueller
> With regards to CC decoding, IMHO, the best would be to add a parser > for ISDB CC at libdvbv5. It probably makes more sense to extend one of the existing libraries that supports captions/subtitles to include support for ISDB (such as libzvbi or ccextractor). The libdvbv5 library has no infrast

Re: [PATCH v3 0/3] drm: describe display bus format

2014-11-30 Thread Boris Brezillon
Hi Laurent, On Sat, 29 Nov 2014 00:29:10 +0200 Laurent Pinchart wrote: > Hi Boris, > > On Thursday 27 November 2014 14:37:50 Boris Brezillon wrote: > > On Tue, 18 Nov 2014 14:46:17 +0100 Boris Brezillon wrote: > > > Hello, > > > > > > This series makes use of the MEDIA_BUS_FMT definition to de

Re: [PATCH v3 1/3] drm: add bus_formats and nbus_formats fields to drm_display_info

2014-11-30 Thread Boris Brezillon
Hi Laurent, On Sat, 29 Nov 2014 00:13:47 +0200 Laurent Pinchart wrote: > Hi Boris, > > Thank you for the patch. I just have two small comments. > > On Tuesday 18 November 2014 14:46:18 Boris Brezillon wrote: > > Add bus_formats and nbus_formats fields and > > drm_display_info_set_bus_formats h