Re: [PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-10-08 Thread Rob Herring
On Tue, Oct 04, 2016 at 07:23:29PM +0300, Laurent Pinchart wrote: > LVDS is a physical layer specification defined in ANSI/TIA/EIA-644-A. > Multiple incompatible data link layers have been used over time to > transmit image data to LVDS panels. This binding supports display panels > compatible

cron job: media_tree daily build: ERRORS

2016-10-08 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: Sun Oct 9 05:00:19 CEST 2016 media-tree git hash:9fce0c226536fc36c7fb0a8ca38a995be43e media_build

[PATCH 0/2] [media] cx88-dsp: Fine-tuning for five function implementations

2016-10-08 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 8 Oct 2016 13:41:12 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (2): Use kmalloc_array() Add some spaces for better code readability drivers/media/pci/cx88/cx88-dsp.c

Re: [PATCH 2/2] [media] dvb-tc90522: Rename a jump label in tc90522_probe()

2016-10-08 Thread walter harms
Am 07.10.2016 21:46, schrieb SF Markus Elfring: > From: Markus Elfring > Date: Fri, 7 Oct 2016 21:13:57 +0200 > > Adjust a jump label according to the Linux coding style convention. > > Signed-off-by: Markus Elfring > --- >

[PATCH RFC] [media] dib0700: remove redundant else

2016-10-08 Thread Nicholas Mc Guire
The if and else are identical and can be consolidated here. Fixes: commit 91be260faaf8 ("[media] dib8000: Add support for Mygica/Geniatech S2870") Signed-off-by: Nicholas Mc Guire --- Problem found by coccinelle script Based only on reviewing this driver it seems that the

[PATCH v2 21/26] pctv452e: don't call BUG_ON() on non-fatal error

2016-10-08 Thread Mauro Carvalho Chehab
There are some conditions on this driver that are tested with BUG_ON() with are not serious enough to hang a machine. So, just return an error if this happens. Signed-off-by: Mauro Carvalho Chehab --- v2: simplify the logic and use its own error message.

[PATCH 1/2] [media] cx88-dsp: Use kmalloc_array() in read_rds_samples()

2016-10-08 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 7 Oct 2016 22:07:27 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was

[PATCH 2/2] [media] cx88-dsp: Add some spaces for better code readability

2016-10-08 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 7 Oct 2016 22:30:40 +0200 Use space characters at some source code places according to the Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/media/pci/cx88/cx88-dsp.c | 40

RFC - unclear change in "[media] DiBxxxx: Codingstype updates"

2016-10-08 Thread Nicholas Mc Guire
Hi Olivier ! in your commit 28fafca78797b ("[media] DiB0090: misc improvements") with commit message: This patch adds several performance improvements and prepares the usage of firmware-based devices. it seems you changed the logic of an if/else in dib0090_tune() in a way that