Re: [coda6] Error while decoding second h.264 chunk

2014-09-29 Thread Gaëtan Carlier
Hi Philipp and Fabio, First of all, thanks a lot for your reply. On 09/29/2014 03:53 PM, Philipp Zabel wrote: Hi Gaëtan, Am Mittwoch, den 24.09.2014, 11:18 +0200 schrieb Gaëtan Carlier: Hello Dears, I am back with my Coda6 (i.MX27). I have ported decoding from libvpu code to kernel 3.6 but I h

cron job: media_tree daily build: WARNINGS

2014-09-29 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: Tue Sep 30 04:00:14 CEST 2014 git branch: test git hash: cf3167cf1e969b17671a4d3d956d22718a8ceb85 gcc versi

Re: [PATCH v5 1/6] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-09-29 Thread Greg Kroah-Hartman
On Mon, Sep 29, 2014 at 08:03:34PM +0200, Philipp Zabel wrote: > Decrementing the reference count of the previous endpoint node allows to > use the of_graph_get_next_endpoint function in a for_each_... style macro. > All current users of this function that pass a non-NULL prev parameter > (that is,

Regression: in v4l2 converter does not set the buffer.length anymore

2014-09-29 Thread Nicolas Dufresne
This was initially reported to GStreamer project: https://bugzilla.gnome.org/show_bug.cgi?id=737521 We track this down to be a regression introduced in v4l2-utils from version 1.4.0. In recent GStreamer we make sure the buffer.length field (retreived with QUERYBUF) is bigger or equal to the exp

Re: [PATCH] dvb_usb_af9005: fix kernel panic on init if the driver is compiled in without IR symbols

2014-09-29 Thread Luca Olivetti
El 29/09/14 20:17, Frank Schäfer ha escrit: > This patches fixes an ancient bug in the dvb_usb_af9005 driver, which > has been reported at least in the following threads: > https://lkml.org/lkml/2009/2/4/350 > https://lkml.org/lkml/2014/9/18/558 > > If the driver is compiled in without any IR supp

Re: [PATCH v2 1/5] video: move mediabus format definition to a more standard place

2014-09-29 Thread Laurent Pinchart
Hi Boris, Thank you for the patch. On Monday 29 September 2014 16:02:39 Boris Brezillon wrote: > Rename mediabus formats and move the enum into a separate header file so > that it can be used by DRM/KMS subsystem without any reference to the V4L2 > subsystem. > > Old V4L2_MBUS_FMT_ definitions a

[PATCH 09/11] v4l: xilinx: Add Xilinx Video IP core

2014-09-29 Thread Laurent Pinchart
Xilinx platforms have no hardwired video capture or video processing interface. Users create capture and memory to memory processing pipelines in the FPGA fabric to suit their particular needs, by instantiating video IP cores from a large library. The Xilinx Video IP core is a framework that model

[PATCH 11/11] v4l: xilinx: Add Test Pattern Generator driver

2014-09-29 Thread Laurent Pinchart
The TPG generates multiple static or dynamic test patterns. The driver currently hardcodes the pattern to the moving box pattern. Signed-off-by: Christian Kohn Signed-off-by: Hyun Kwon Signed-off-by: Laurent Pinchart Signed-off-by: Michal Simek --- .../bindings/media/xilinx/xlnx,v-tpg.txt

[PATCH 06/11] dma: xilinx: vdma: Check if the segment list is empty in a descriptor

2014-09-29 Thread Laurent Pinchart
From: Srikanth Thokala The segment list in a descriptor should be checked for empty, else it will try to access invalid address for the first call. This patch fixes this issue. Signed-off-by: Srikanth Thokala Signed-off-by: Michal Simek --- drivers/dma/xilinx/xilinx_vdma.c | 8 +--- 1 fi

[PATCH 00/11] Xilinx Video IP Core support

2014-09-29 Thread Laurent Pinchart
Hello, Here's (finally) a set of kernel drivers to support the Xilinx FPGA Video IP Cores architecture. I won't detail in great lengths the Xilinx Video IP architecture here, as that would result in dozens of pages of documentation. The interested reader can refer to the Zynq ZC702 Base TRD (Targ

[PATCH 03/11] v4l: Sort YUV formats of v4l2_mbus_pixelcode

2014-09-29 Thread Laurent Pinchart
From: Hyun Kwon Keep the formats sorted by type, bus_width, bits per component, samples per pixel and order of subsamples, in that order. Signed-off-by: Hyun Kwon Signed-off-by: Laurent Pinchart --- Documentation/DocBook/media/v4l/subdev-formats.xml | 600 ++--- include/uapi/l

[PATCH 08/11] dma: xilinx: vdma: icg should be difference of stride and hsize

2014-09-29 Thread Laurent Pinchart
From: Srikanth Thokala This patch modifies the icg field to match the description as mentioned in the DMA Linux framework. Signed-off-by: Srikanth Thokala Signed-off-by: Michal Simek --- drivers/dma/xilinx/xilinx_vdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Cc: dmaeng...@vger

[PATCH 07/11] dma: xilinx: vdma: Allow only one chunk in a line

2014-09-29 Thread Laurent Pinchart
From: Srikanth Thokala This patch adds a sanity check to see if frame_size is 1. Signed-off-by: Srikanth Thokala Signed-off-by: Michal Simek --- drivers/dma/xilinx/xilinx_vdma.c | 3 +++ 1 file changed, 3 insertions(+) Cc: dmaeng...@vger.kernel.org diff --git a/drivers/dma/xilinx/xilinx_vdm

[PATCH 05/11] v4l: of: Add v4l2_of_parse_link() function

2014-09-29 Thread Laurent Pinchart
The function fills a link data structure with the device node and port number at both the local and remote ends of a link defined by one of its endpoint nodes. Signed-off-by: Laurent Pinchart --- drivers/media/v4l2-core/v4l2-of.c | 61 +++ include/media/v4l2-o

[PATCH 10/11] v4l: xilinx: Add Video Timing Controller driver

2014-09-29 Thread Laurent Pinchart
The Video Timing Controller (VTC) includes a timing detector and/or a timing generator. Only the generator is currently supported. Signed-off-by: Laurent Pinchart Signed-off-by: Michal Simek --- .../devicetree/bindings/media/xilinx/xlnx,v-tc.txt | 33 ++ drivers/media/platform/xilinx/Kconfig

[PATCH 04/11] v4l: Add VUY8 24 bits bus format

2014-09-29 Thread Laurent Pinchart
From: Hyun Kwon Add VUY8 24 bits bus format, V4L2_MBUS_FMT_VUY8_1X24. Signed-off-by: Hyun Kwon Signed-off-by: Laurent Pinchart --- Documentation/DocBook/media/v4l/subdev-formats.xml | 30 ++ include/uapi/linux/v4l2-mediabus.h | 3 ++- 2 files changed, 32 i

[PATCH 02/11] v4l: Add RBG and RGB 8:8:8 media bus formats on 24 and 32 bit busses

2014-09-29 Thread Laurent Pinchart
Signed-off-by: Laurent Pinchart --- Documentation/DocBook/media/v4l/subdev-formats.xml | 67 ++ include/uapi/linux/v4l2-mediabus.h | 4 +- 2 files changed, 70 insertions(+), 1 deletion(-) diff --git a/Documentation/DocBook/media/v4l/subdev-formats.xml b/Docu

[PATCH 01/11] media: entity: Document the media_entity_ops structure

2014-09-29 Thread Laurent Pinchart
Signed-off-by: Laurent Pinchart --- include/media/media-entity.h | 9 + 1 file changed, 9 insertions(+) diff --git a/include/media/media-entity.h b/include/media/media-entity.h index e004591..786906b 100644 --- a/include/media/media-entity.h +++ b/include/media/media-entity.h @@ -44,6 +4

Re: [PATCH v2] [media] uvcvideo: Add quirk to force the Oculus DK2 IR tracker to grayscale

2014-09-29 Thread Philipp Zabel
Hi Laurent, On Wed, Aug 6, 2014 at 10:50 PM, Philipp Zabel wrote: > This patch adds a quirk to force Y8 pixel format even if the camera reports > half-width YUYV. > > Signed-off-by: Philipp Zabel do you have any further comments on this patch? regards Philipp > --- > drivers/media/usb/uvc/uv

[PATCH] [media] cx231xx: cx231xx_uninit_bulk attempts to reference and free isoc_ctl instead of bulk_ctl

2014-09-29 Thread Luke Suchocki
cx231xx_uninit_bulk() checks that dev->video_mode.bulk_ctl.transfer_buffer[i] is not NULL, but then calls usb_free_cohert() with dev->video_mode.isoc_ctl.transfer_buffer[i] resulting in "BUG: unable to handle NULL pointer dereference" when closing stream; most likely a cut-and-paste slip from prev

Re: [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C

2014-09-29 Thread Johannes Stezenbach
On Mon, Sep 29, 2014 at 03:30:18PM -0300, Mauro Carvalho Chehab wrote: > Em Mon, 29 Sep 2014 19:44:30 +0200 > Johannes Stezenbach escreveu: > > > Disregarding your mails from the "em28xx breaks after hibernate" > > that hibernate doesn't work for you, I decided to give these > > changes a try on

Re: [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C

2014-09-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Sep 2014 19:44:30 +0200 Johannes Stezenbach escreveu: > Hi Mauro, > > On Sun, Sep 28, 2014 at 11:23:17PM -0300, Mauro Carvalho Chehab wrote: > > This patch series addresses some issues with suspend2ram of devices > > based on DRX-K. > > > > With this patch series, it is now possible

[PATCH] dvb_usb_af9005: fix kernel panic on init if the driver is compiled in without IR symbols

2014-09-29 Thread Frank Schäfer
This patches fixes an ancient bug in the dvb_usb_af9005 driver, which has been reported at least in the following threads: https://lkml.org/lkml/2009/2/4/350 https://lkml.org/lkml/2014/9/18/558 If the driver is compiled in without any IR support (neither DVB_USB_AF9005_REMOTE nor custom symbols),

[PATCH v5 5/6] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3 inser

[PATCH v5 3/6] of: Add of_graph_get_port_by_id function

2014-09-29 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/of/base.c| 26 ++ include/linux/of_graph.h | 7 +++ 2 files changed, 33 insertions(+) diff --git

[PATCH v5 2/6] of: Add for_each_endpoint_of_node helper macro

2014-09-29 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- include/linux/of_graph.h | 11 +++ 1 fi

[PATCH v5 6/6] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Since we can break out of the loop, we keep the call to of_node_put after the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 11 --- 1 file changed, 4 insertions(+), 7 delet

[PATCH v5 1/6] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-09-29 Thread Philipp Zabel
Decrementing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. All current users of this function that pass a non-NULL prev parameter (that is, soc_camera and imx-drm) are changed to not decrement the passed prev a

[PATCH v5 4/6] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.

[PATCH v5 0/6] Add of-graph helpers to loop over endpoints and find ports by id

2014-09-29 Thread Philipp Zabel
Hi, now the first three patches of the previous series are combined into one. This patch touches of, media, and staging. Since it is rebased onto Grant's tree, it now trivially conflicts with 30e94a564d07 (staging: imx-drm: Lines over 80 characters fixed.) This series converts all existing users

Re: [PATCH 0/6] some fixes and cleanups for the em28xx-based HVR-930C

2014-09-29 Thread Johannes Stezenbach
Hi Mauro, On Sun, Sep 28, 2014 at 11:23:17PM -0300, Mauro Carvalho Chehab wrote: > This patch series addresses some issues with suspend2ram of devices > based on DRX-K. > > With this patch series, it is now possible to suspend to ram while > streaming. At resume, the stream will continue to play.

Re: [PATCH v4 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Greg Kroah-Hartman
On Mon, Sep 29, 2014 at 11:45:23AM +0200, Philipp Zabel wrote: > Am Montag, den 29.09.2014, 12:13 +0300 schrieb Dan Carpenter: > > On Mon, Sep 29, 2014 at 10:15:44AM +0200, Philipp Zabel wrote: > > > In preparation for a following patch, stop decrementing the endpoint node > > > refcount in the loo

[PATCH] [media] vivid: add CONFIG_FB dependency

2014-09-29 Thread Arnd Bergmann
>From 6699184d4b791e8a10380d3b75be837607d3 Mon Sep 17 00:00:00 2001 From: Arnd Bergmann Date: Mon, 29 Sep 2014 17:33:25 +0200 Subject: [PATCH] [media] vivid: add CONFIG_FB dependency The vivid test driver creates a framebuffer, which fails if the the framebuffer layer is not enabled: drivers

Re: [PATCH] [media] pt3: remove bogus module_is_live() check

2014-09-29 Thread Mauro Carvalho Chehab
Hi Arnd, Em Mon, 29 Sep 2014 16:28:55 +0200 Arnd Bergmann escreveu: > The new pt3 driver checks the module reference for presence > before dropping it, which fails to compile when modules > are disabled: > > media/pci/pt3/pt3.c: In function 'pt3_attach_fe': > media/pci/pt3/pt3.c:433:6: error: i

Re: [PATCH v2 1/5] video: move mediabus format definition to a more standard place

2014-09-29 Thread Thierry Reding
On Mon, Sep 29, 2014 at 04:02:39PM +0200, Boris Brezillon wrote: > Rename mediabus formats and move the enum into a separate header file so > that it can be used by DRM/KMS subsystem without any reference to the V4L2 > subsystem. > > Old V4L2_MBUS_FMT_ definitions are now macros that points to VID

[PATCH] [media] pt3: remove bogus module_is_live() check

2014-09-29 Thread Arnd Bergmann
The new pt3 driver checks the module reference for presence before dropping it, which fails to compile when modules are disabled: media/pci/pt3/pt3.c: In function 'pt3_attach_fe': media/pci/pt3/pt3.c:433:6: error: implicit declaration of function 'module_is_live' [-Werror=implicit-function-declar

[PATCH] Add IR support for DVBSky T9580 Dual DVB-S2/T2/C PCIe card

2014-09-29 Thread Nibble Max
DVBSky T9580 uses Integrated CX23885 IR controller to decode IR signal. The IR type of DVBSky remote control is RC5. Signed-off-by: Nibble Max --- drivers/media/pci/cx23885/cx23885-cards.c | 3 +++ drivers/media/pci/cx23885/cx23885-input.c | 8 2 files changed, 11 insertions(+) diff --

Re: [PATCH 1/2] af9033: fix signal strength value not correct issue

2014-09-29 Thread Antti Palosaari
On 09/29/2014 03:02 PM, Antti Palosaari wrote: On 09/29/2014 11:43 AM, Bimow Chen wrote: Register 0x800048 is not dB measure but relative scale. Fix it and conform to NorDig specifications. eh, 0x800048 register returned strength normalized to 0-100 %. But that was earlier when older firmware

[PATCH v2 1/5] video: move mediabus format definition to a more standard place

2014-09-29 Thread Boris Brezillon
Rename mediabus formats and move the enum into a separate header file so that it can be used by DRM/KMS subsystem without any reference to the V4L2 subsystem. Old V4L2_MBUS_FMT_ definitions are now macros that points to VIDEO_BUS_FMT_ definitions. Signed-off-by: Boris BREZILLON Acked-by: Guennad

Re: [PATCH v2 1/5] video: move mediabus format definition to a more standard place

2014-09-29 Thread Boris BREZILLON
On Mon, 29 Sep 2014 16:02:39 +0200 Boris Brezillon wrote: > Rename mediabus formats and move the enum into a separate header file so > that it can be used by DRM/KMS subsystem without any reference to the V4L2 > subsystem. > > Old V4L2_MBUS_FMT_ definitions are now macros that points to VIDEO_BU

[PATCH v2 0/5] video: describe data bus formats

2014-09-29 Thread Boris Brezillon
Hello, This patch series is a proposal to describe the different data formats used by HW components to connect with each other. This is just a copy of the existing V4L2_MBUS_FMT defintions with a neutral name so that it can be used by V4L2 and DRM/KMS subsystem. This series also makes use of thi

[PATCH v2 3/5] drm: add bus_formats and nbus_formats fields to drm_display_info

2014-09-29 Thread Boris Brezillon
From: Boris BREZILLON Add bus_formats and nbus_formats fields and drm_display_info_set_bus_formats helper function to specify the bus formats supported by a given display. This information can be used by display controller drivers to configure the output interface appropriately (i.e. RGB565, RGB

[PATCH v2 5/5] drm: panel: simple-panel: add bus format information for foxlink panel

2014-09-29 Thread Boris Brezillon
From: Boris BREZILLON Foxlink's fl500wvr00-a0t supports RGB888 format. Signed-off-by: Boris BREZILLON --- drivers/gpu/drm/panel/panel-simple.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index eb3a17e..11bff3f

[PATCH v2 4/5] drm: panel: simple-panel: add support for bus_format retrieval

2014-09-29 Thread Boris Brezillon
From: Boris BREZILLON Provide a way to specify panel requirement in terms of supported media bus format (particularly useful for panels connected to an RGB or LVDS bus). Signed-off-by: Boris BREZILLON --- drivers/gpu/drm/panel/panel-simple.c | 5 + 1 file changed, 5 insertions(+) diff --g

[PATCH v2 2/5] video: add RGB444_1X12 and RGB565_1X16 bus formats

2014-09-29 Thread Boris Brezillon
Add RGB444 format using a 12 bits bus and RGB565 using a 16 bits bus. These formats will later be used by atmel-hlcdc driver. Signed-off-by: Boris BREZILLON --- include/uapi/linux/v4l2-mediabus.h| 2 ++ include/uapi/linux/video-bus-format.h | 4 +++- 2 files changed, 5 insertions(+), 1 dele

[PATCH 2/6] [media] coda: remove superfluous error message on devm_kzalloc failure

2014-09-29 Thread Philipp Zabel
When devm_kzalloc causes an OOM condition, this is already reported by the MM subsystem. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-common.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/p

[PATCH 3/6] [media] coda: add coda_write_base helper

2014-09-29 Thread Philipp Zabel
Add a helper function that writes a vb2_buffer's Y, Cb, and Cr plane base addresses of into three consecutive registers. This moves common code out of coda-bit.c. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c| 68 +++ drivers/media/platfo

[PATCH 4/6] [media] coda: disable rotator if not needed

2014-09-29 Thread Philipp Zabel
This will still do a 1:1 copy into the internal buffers, but stop producing visual artifacts in chroma interleaved (NV12) mode. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform

[PATCH 5/6] [media] coda: simplify frame memory control register handling

2014-09-29 Thread Philipp Zabel
Since the firmware newer writes to FRAME_MEM_CTRL, we can initialize it once per context (incidentally, we already do write it in coda_hw_init) and never have to read it back. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c| 8 drivers/media/platform/coda/cod

[PATCH 1/6] [media] coda: clear aborting flag in stop_streaming

2014-09-29 Thread Philipp Zabel
Clearing the aborting flag in stop_streaming is necessary if we want to start streaming again without having to closing and reopening the device. Also, do not explicitly set it in default_params; the context is zeroed by kzalloc anyway. Signed-off-by: Philipp Zabel --- drivers/media/platform/cod

[PATCH 0/6] CODA fixes and NV12 support

2014-09-29 Thread Philipp Zabel
Hi, this series fixes restarting of streams, cleans up a superfluous error message, unifies the YUV buffer base pointer setup, disables the encoder rotator unit when copying input buffers into the internal frame memory if not needed, simplifies the frame memory control register handling (no need t

[PATCH 6/6] [media] coda: add support for partial interleaved YCbCr 4:2:0 (NV12) format

2014-09-29 Thread Philipp Zabel
This patch adds support for the two-plane NV12 format with one luma plane and one interleaved chroma plane. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c| 26 +- drivers/media/platform/coda/coda-common.c | 8 2 files changed, 29 ins

Re: [PATCH 5/5] cx23855: add CI support for DVBSky T980C

2014-09-29 Thread Olli Salonen
Hi Max, Thanks for the comments. I looked through the existing CIMaX driver and came to the conclusion that it can be implemented either with the IRQs or then just to rely on the polling like the USB devices have to do. I don't think it's that important, as the user probably will not add/remove th

Re: [PATCH 2/2] af9033: fix snr value not correct issue

2014-09-29 Thread Antti Palosaari
On 09/29/2014 11:47 AM, Bimow Chen wrote: Snr returns value not correct. Fix it. And same here. You change SNR, which is reported by dB to relative 0-0x. If you like to do it for some reason, then do it for DVBv3 API only. Leave new DVBv5 statistic API reports to dB. I am pretty sure

Re: [PATCH 5/5] cx23855: add CI support for DVBSky T980C

2014-09-29 Thread Nibble Max
Hello, In hardware design, the CI host controller is wired with GPIO_0 of CX23885. The GPIO_0 can be configed as the interrupt source. Interrupt mode in PCIe driver is more faster than poll mode to detect CAM insert/remove operations etc. >Add CI support for DVBSky T980C. > >I used the new host d

Re: [PATCH 1/2] af9033: fix signal strength value not correct issue

2014-09-29 Thread Antti Palosaari
On 09/29/2014 11:43 AM, Bimow Chen wrote: Register 0x800048 is not dB measure but relative scale. Fix it and conform to NorDig specifications. eh, 0x800048 register returned strength normalized to 0-100 %. But that was earlier when older firmwares used. I have seen it does not return anythin

[PATCH] em28xx: fix uninitialized variable warning

2014-09-29 Thread Hans Verkuil
Fix this daily build warning: In file included from build/media_build/v4l/em28xx-core.c:35:0: build/media_build/v4l/em28xx-core.c: In function 'em28xx_audio_setup': build/media_build/v4l/em28xx.h:798:2: warning: 'vid' may be used uninitialized in this function [-Wmaybe-uninitialized] printk(KER

Re: [PATCH v4 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Philipp Zabel
Am Montag, den 29.09.2014, 12:13 +0300 schrieb Dan Carpenter: > On Mon, Sep 29, 2014 at 10:15:44AM +0200, Philipp Zabel wrote: > > In preparation for a following patch, stop decrementing the endpoint node > > refcount in the loop. This temporarily leaks a reference to the endpoint > > node, > > wh

Re: [PATCH v4 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Dan Carpenter
On Mon, Sep 29, 2014 at 10:15:44AM +0200, Philipp Zabel wrote: > In preparation for a following patch, stop decrementing the endpoint node > refcount in the loop. This temporarily leaks a reference to the endpoint node, > which will be fixed by having of_graph_get_next_endpoint decrement the > ref

cron job: media_tree daily build: ERRORS

2014-09-29 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 29 Sep 10:30:33 CEST 2014 git branch: test git hash: cf3167cf1e969b17671a4d3d956d22718a8ceb85 gcc versi

RE: buffer delivery stops with cx23885

2014-09-29 Thread James Harper
> > 041ad449683bb2d54a7f082d78ec15bbc958a175 introduced stats gathering > into the dib7000p code which seems to generate a lot more i2c traffic which > would exacerbate the problem, if one existed. The timing (May/June) very > roughly matches what I remember too. When my recording stops, so do the

[PATCH 2/2] af9033: fix snr value not correct issue

2014-09-29 Thread Bimow Chen
Snr returns value not correct. Fix it. >From 427a5c6ef49e3235ac35a0464c375f2a2706619e Mon Sep 17 00:00:00 2001 From: Bimow Chen Date: Mon, 29 Sep 2014 16:30:52 +0800 Subject: [PATCH 2/2] af9033: fix snr value not correct issue Snr returns value not correct. Fix it. Signed-off-by: Bimow Chen ---

[PATCH 1/2] af9033: fix signal strength value not correct issue

2014-09-29 Thread Bimow Chen
Register 0x800048 is not dB measure but relative scale. Fix it and conform to NorDig specifications. >From b85ad9df69884b80cce62877039aa9130243ef3a Mon Sep 17 00:00:00 2001 From: Bimow Chen Date: Mon, 29 Sep 2014 13:57:07 +0800 Subject: [PATCH 1/2] af9033: fix signal strength value not correct is

[PATCH v4 7/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_get_mux_id

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. This patch also properly decrements the endpoint node reference count before returning out of the loop. Signed-off-by: Philipp Zabel --- drivers/staging/imx-drm/imx-drm-core.c | 9 +++-- 1 file changed, 3 inser

[PATCH v4 1/8] [media] soc_camera: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Philipp Zabel
In preparation for a following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel --- C

[PATCH v4 4/8] of: Add for_each_endpoint_of_node helper macro

2014-09-29 Thread Philipp Zabel
Note that while of_graph_get_next_endpoint decrements the reference count of the child node passed to it, of_node_put(child) still has to be called manually when breaking out of the loop. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- include/linux/of_graph.h | 11 +++ 1 fi

[PATCH v4 0/8] Add of-graph helpers to loop over endpoints and find ports by id

2014-09-29 Thread Philipp Zabel
Hi, in theory, I'd like the first five patches to go in together through Grant's tree, but in practice that might not be necessary; changes in of_node reference counting are only relevant if CONFIG_OF_DYNAMIC is enabled. Since patch 2 depends on (or would trivially conflict with) 30e94a564d07 in t

[PATCH v4 6/8] drm: use for_each_endpoint_of_node macro in drm_of_find_possible_crtcs

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code a bit shorter and easier to read. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/gpu/drm/drm_of.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.

[PATCH v4 2/8] imx-drm: Do not decrement endpoint node refcount in the loop

2014-09-29 Thread Philipp Zabel
In preparation for the following patch, stop decrementing the endpoint node refcount in the loop. This temporarily leaks a reference to the endpoint node, which will be fixed by having of_graph_get_next_endpoint decrement the refcount of its prev argument instead. Signed-off-by: Philipp Zabel ---

[PATCH v4 5/8] of: Add of_graph_get_port_by_id function

2014-09-29 Thread Philipp Zabel
This patch adds a function to get a port device tree node by port id, or reg property value. Signed-off-by: Philipp Zabel Acked-by: Laurent Pinchart --- drivers/of/base.c| 26 ++ include/linux/of_graph.h | 7 +++ 2 files changed, 33 insertions(+) diff --git

[PATCH v4 3/8] of: Decrement refcount of previous endpoint in of_graph_get_next_endpoint

2014-09-29 Thread Philipp Zabel
Decrementing the reference count of the previous endpoint node allows to use the of_graph_get_next_endpoint function in a for_each_... style macro. Prior to this patch, all current users of this function that actually pass a non-NULL prev parameter should be changed to not decrement the passed prev

[PATCH v4 8/8] imx-drm: use for_each_endpoint_of_node macro in imx_drm_encoder_parse_of

2014-09-29 Thread Philipp Zabel
Using the for_each_... macro should make the code bit shorter and easier to read. Since we can break out of the loop, we keep the call to of_node_put after the loop. Signed-off-by: Philipp Zabel --- Changes since v3: - Kept of_node_put after the loop, as in the earlier soc_camera patch. This

[PATCH 4/5] cx23885: add I2C client for CI into state and handle unregistering

2014-09-29 Thread Olli Salonen
If the CI chip has an I2C driver, we need to store I2C client into state. Signed-off-by: Olli Salonen --- drivers/media/pci/cx23885/cx23885-dvb.c | 7 +++ drivers/media/pci/cx23885/cx23885.h | 1 + 2 files changed, 8 insertions(+) diff --git a/drivers/media/pci/cx23885/cx23885-dvb.c b/

[PATCH 5/5] cx23855: add CI support for DVBSky T980C

2014-09-29 Thread Olli Salonen
Add CI support for DVBSky T980C. I used the new host device independent CIMaX SP2 I2C driver to implement it. cx23885_sp2_ci_ctrl function is borrowed entirely from cimax2.c. Signed-off-by: Olli Salonen --- drivers/media/pci/cx23885/cx23885-dvb.c | 105 +++- 1 file

[PATCH 3/5] sp2: improve debug logging

2014-09-29 Thread Olli Salonen
Improve debugging output. Signed-off-by: Olli Salonen --- drivers/media/dvb-frontends/sp2.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/drivers/media/dvb-frontends/sp2.c b/drivers/media/dvb-frontends/sp2.c index 1f4f250..320cbe9 100644 --- a/drivers/

[PATCH 1/5] cx23855: add support for DVBSky T980C (no CI support)

2014-09-29 Thread Olli Salonen
This patch adds basic support for DVBSky T980C card. CI interface is not supported. DVBSky T980C is a PCIe card with the following components: - CX23885 PCIe bridge - Si2168-A20 demodulator - Si2158-A20 tuner - CIMaX SP2 CI chip The demodulator and tuner need firmware. They're the same as used w

[PATCH 2/5] sp2: fix incorrect struct

2014-09-29 Thread Olli Salonen
Incorrect struct used in the SP2 driver. Reported-by: Max Nibble Signed-off-by: Olli Salonen --- drivers/media/dvb-frontends/sp2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/dvb-frontends/sp2.c b/drivers/media/dvb-frontends/sp2.c index 9b684d5..1f4f250 10