Re: [PATCH v4 1/9] brcmsmac: make some local variables 'static const' to reduce stack size

2017-09-24 Thread Kalle Valo
Arnd Bergmann writes: > With KASAN and a couple of other patches applied, this driver is one > of the few remaining ones that actually use more than 2048 bytes of > kernel stack: > > broadcom/brcm80211/brcmsmac/phy/phy_n.c: In function > 'wlc_phy_workarounds_nphy_gainctrl': >

cron job: media_tree daily build: ERRORS

2017-09-24 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 Sep 25 05:00:16 CEST 2017 media-tree git hash:d5426f4c2ebac8cf05de43988c3fccddbee13d28 media_build

Re: f26: dvb_usb_rtl28xxu not tuning "Leadtek Winfast DTV2000 DS PLUS TV"

2017-09-24 Thread Eyal Lebedinsky
On 24/09/17 19:04, Vincent McIntyre wrote: On Sat, Sep 23, 2017 at 10:48:34PM +1000, Eyal Lebedinsky wrote: On 18/09/17 14:26, Eyal Lebedinsky wrote: I have just upgraded to f24. I am now using the standard dvb_usb_rtl28xxu fe I have upgraded to f26 and this driver still fails to tune the

Re: [GIT PULL] SAA716x DVB driver

2017-09-24 Thread Soeren Moch
On 16.09.2017 19:49, Mauro Carvalho Chehab wrote: > Em Sat, 16 Sep 2017 14:54:15 +0200 > Soeren Moch escreveu: > >> On 09.09.2017 23:20, Mauro Carvalho Chehab wrote: >>> Em Sat, 9 Sep 2017 14:52:18 +0200 >>> Soeren Moch escreveu: >>> You explicitly want to

Re: [PATCH v6 1/2] media:imx274 device tree binding file

2017-09-24 Thread Rob Herring
On Sun, Sep 24, 2017 at 12:53:28AM -0700, Leon Luo wrote: > The binding file for imx274 CMOS sensor V4l2 driver > > Signed-off-by: Leon Luo > --- > v6: > - no changes > v5: > - add 'port' and 'endpoint' information > v4: > - no changes > v3: > - remove redundant

[PATCH 4/4] [media] omap3isp: Delete an unnecessary variable initialisation in isp_video_open()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 19:43:06 +0200 The variable "ret" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring ---

[PATCH 3/4] [media] omap3isp: Use common error handling code in isp_video_open()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 19:30:52 +0200 * Adjust jump targets so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. * Delete a repeated check (for the

[PATCH 2/4] [media] omap3isp: Adjust 53 checks for null pointers

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 18:56:33 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus

[PATCH 1/4] [media] omap3isp: Delete an error message for a failed memory allocation in three functions

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 18:25:44 +0200 Omit an extra message for a memory allocation failure in these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/4] [media] OMAP3 ISP: Adjustments for some function implementations

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 19:57:34 +0200 Some update suggestions were taken into account from static source code analysis. Markus Elfring (4): Delete an error message for a failed memory allocation in three functions Adjust 53 checks for null

Re: [PATCH v2] scripts: kernel-doc: fix nexted handling

2017-09-24 Thread Mauro Carvalho Chehab
Em Sun, 24 Sep 2017 17:13:13 +0200 Markus Heiser escreveu: > > Am 24.09.2017 um 12:23 schrieb Mauro Carvalho Che > > > > v2: handle embedded structs/unions from inner to outer > > > > When we have multiple levels of embedded structs, > > > > we need a smarter rule

Re: [PATCH v2] scripts: kernel-doc: fix nexted handling

2017-09-24 Thread Markus Heiser
> Am 24.09.2017 um 12:23 schrieb Mauro Carvalho Che > > v2: handle embedded structs/unions from inner to outer > > When we have multiple levels of embedded structs, > > we need a smarter rule that will be removing nested structs > from the inner to the outer ones. So, changed the parsing rule

[PATCH v2] staging: atomisp: add a driver for ov5648 camera sensor

2017-09-24 Thread Devid Antonio Floni
The ov5648 5-megapixel camera sensor from OmniVision supports up to 2592x1944 resolution and MIPI CSI-2 interface. Output format is raw sRGB/Bayer with 10 bits per colour (SGRBG10_1X10). This patch is a port of ov5648 driver after applying following 01org/ProductionKernelQuilts patches: -

Re: [PATCH v3 0/4] AS3645A fixes

2017-09-24 Thread Sakari Ailus
Jacek Anaszewski wrote: Patch set applied to the fixes-4.14-rc3 branch of linux-leds.git. Thanks, Jacek! -- Sakari Ailus sakari.ai...@linux.intel.com

Re: [GIT PULL FOR v4.15] RC cleanup fixes

2017-09-24 Thread Sean Young
On Sun, Sep 24, 2017 at 11:40:20AM +0100, Sean Young wrote: > On Sun, Sep 24, 2017 at 06:09:32AM -0300, Mauro Carvalho Chehab wrote: > > Ah, I see. Well, if none of the in-kernel drivers use it, we can > > drop it. > > Looks like our emails crossed each other -- I have already pushed out >

Re: [GIT PULL FOR v4.15] RC cleanup fixes

2017-09-24 Thread Sean Young
On Sun, Sep 24, 2017 at 06:09:32AM -0300, Mauro Carvalho Chehab wrote: > Em Sat, 23 Sep 2017 21:38:59 +0100 > Sean Young escreveu: > > > Hi Mauro, > > > > On Sat, Sep 23, 2017 at 04:35:31PM -0300, Mauro Carvalho Chehab wrote: > > > Hi Sean, > > > > > > Em Sat, 23 Sep 2017

Re: [PATCH 1/6] [media] omap_vout: Delete an error message for a failed memory allocation in omap_vout_create_video_devices()

2017-09-24 Thread Joe Perches
On Sun, 2017-09-24 at 12:22 +0200, SF Markus Elfring wrote: > Omit an extra message for a memory allocation failure in this function. [] > diff --git a/drivers/media/platform/omap/omap_vout.c > b/drivers/media/platform/omap/omap_vout.c [] > @@ -1948,7 +1948,5 @@ static int __init

Re: [PATCH] scripts: kernel-doc: fix nexted handling

2017-09-24 Thread Mauro Carvalho Chehab
Em Thu, 21 Sep 2017 14:44:34 +0200 Markus Heiser escreveu: > > Jon, > > > > While documenting some DVB demux headers, I noticed the above bug. > > > > scripts/kernel-doc | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git

[PATCH 6/6] [media] omap_vout: Delete two unnecessary variable initialisations in omap_vout_probe()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 11:33:39 +0200 The variables "dssdev" and "vid_dev" will eventually be set to appropriate pointers a bit later. Thus omit the explicit initialisations at the beginning. Signed-off-by: Markus Elfring

[PATCH 5/6] [media] omap_vout: Delete an unnecessary variable initialisation in omap_vout_open()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 11:20:11 +0200 The local variable "vout" is reassigned by a statement at the beginning. Thus omit the explicit initialisation. Signed-off-by: Markus Elfring ---

[PATCH 4/6] [media] omap_vout: Fix a possible null pointer dereference in omap_vout_open()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 11:00:57 +0200 Move a debug message so that a null pointer access can not happen for the variable "vout" in this function. Fixes: 5c7ab6348e7b3fcca2b8ee548306c774472971e2 ("V4L/DVB: V4L2: Add support for OMAP2/3 V4L2

[PATCH 3/6] [media] omap_vout: Adjust a null pointer check in two functions

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 10:30:29 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus

[PATCH 2/6] [media] omap_vout: Improve a size determination in two functions

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 10:18:26 +0200 Replace the specification of data structures by variable references as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 1/6] [media] omap_vout: Delete an error message for a failed memory allocation in omap_vout_create_video_devices()

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 10:08:22 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH v2] scripts: kernel-doc: fix nexted handling

2017-09-24 Thread Mauro Carvalho Chehab
At DVB, we have, at some structs, things like (see struct dvb_demux_feed, at dvb_demux.h): union { struct dmx_ts_feed ts; struct dmx_section_feed sec; } feed; union { dmx_ts_cb ts; dmx_section_cb sec;

[GIT PULL v2 FOR v4.15] RC cleanup fixes

2017-09-24 Thread Sean Young
Hi Mauro, Thank you for your comments on the first PR. You are right, removing the LIRC_GET_{MIN,MAX}_TIMEOUT ioctls from the non-rc-core lirc drivers had no reason to be merged. The changes to use file->private_data should not break imon since the imon driver uses it for the lcd chardev, not

[PATCH 0/6] [media] omap_vout: Adjustments for three function implementations

2017-09-24 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 24 Sep 2017 12:06:54 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (6): Delete an error message for a failed memory allocation in omap_vout_create_video_devices()

Re: [PATCH 2/2] media: rc-core.rst: add an introduction for RC core

2017-09-24 Thread Sean Young
On Sat, Sep 23, 2017 at 04:43:34PM -0300, Mauro Carvalho Chehab wrote: > The RC core does several assumptions, but those aren't documented > anywhere, with could make harder for ones that want to understand > what's there. > > So, add an introduction explaining the basic concepts of RC and > how

Re: f26: dvb_usb_rtl28xxu not tuning "Leadtek Winfast DTV2000 DS PLUS TV"

2017-09-24 Thread Eyal Lebedinsky
Thanks for writing Vince, On 24/09/17 19:04, Vincent McIntyre wrote: On Sat, Sep 23, 2017 at 10:48:34PM +1000, Eyal Lebedinsky wrote: On 18/09/17 14:26, Eyal Lebedinsky wrote: I have just upgraded to f24. I am now using the standard dvb_usb_rtl28xxu fe I have upgraded to f26 and this driver

Re: [GIT PULL FOR v4.15] RC cleanup fixes

2017-09-24 Thread Mauro Carvalho Chehab
Em Sat, 23 Sep 2017 21:38:59 +0100 Sean Young escreveu: > Hi Mauro, > > On Sat, Sep 23, 2017 at 04:35:31PM -0300, Mauro Carvalho Chehab wrote: > > Hi Sean, > > > > Em Sat, 23 Sep 2017 11:33:56 +0100 > > Sean Young escreveu: > > > > > Hi Mauro, > > > > > >

Re: f26: dvb_usb_rtl28xxu not tuning "Leadtek Winfast DTV2000 DS PLUS TV"

2017-09-24 Thread Vincent McIntyre
On Sat, Sep 23, 2017 at 10:48:34PM +1000, Eyal Lebedinsky wrote: > On 18/09/17 14:26, Eyal Lebedinsky wrote: > >I have just upgraded to f24. I am now using the standard dvb_usb_rtl28xxu fe > > I have upgraded to f26 and this driver still fails to tune the "Leadtek > Winfast DTV2000 DS PLUS TV".

[PATCH v6 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor

2017-09-24 Thread Leon Luo
The imx274 is a Sony CMOS image sensor that has 1/2.5 image size. It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface is 4-lane MIPI CSI-2 running at 1.44Gbps each. This driver has been tested on Xilinx ZCU102 platform with a Leopard LI-IMX274MIPI-FMC camera board. Support for

[PATCH v6 1/2] media:imx274 device tree binding file

2017-09-24 Thread Leon Luo
The binding file for imx274 CMOS sensor V4l2 driver Signed-off-by: Leon Luo --- v6: - no changes v5: - add 'port' and 'endpoint' information v4: - no changes v3: - remove redundant properties and references - document 'reg' property v2: - no changes ---