[PATCH] media: drxd: make const array fastIncrDecLUT static

From: Colin Ian King Don't populate array fastIncrDecLUT on the stack but instead make it static. Makes the object code smaller by over 360 bytes: textdata bss dec hex filename 32680 944 64 336888398 drxd_hard.o textdata

[PATCH] [media] cx88: make const arrays default_addr_list and pvr2000_addr_list static

From: Colin Ian King Don't populate arrays default_addr_list and pvr2000_addr_list on the stack but instead make them static. Makes the object code smaller by over 340 bytes: Before: textdata bss dec hex filename 125202800 64 15384

[PATCH] [media] au0828: make const array addr_list static

From: Colin Ian King Don't populate array addr_list on the stack but instead make it static. Makes the object code smaller by over 360 bytes: Before: textdata bss dec hex filename 80361488 192971625f4 au0828-input.o After: text

Re: [RESEND PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent

Hi Sakari, Thank you for the patch. On Thursday, 2 November 2017 11:59:18 EET Sakari Ailus wrote: > (Fixed Mauro's e-mail.) > > to_of_node() macro checks whether the fwnode_handle passed to it is not an > OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it > returns the

Re: [PATCH 1/1] media: i2c: as3645a: Remove driver

Hi Sakari, Thank you for the patch. On Friday, 8 September 2017 16:51:40 EET Sakari Ailus wrote: > Remove the V4L2 AS3645A sub-device driver in favour of the LED flash class > driver for the same hardware, drivers/leds/leds-as3645a.c. The latter uses > the V4L2 flash LED class framework to

Re: 32bit userland cannot work with DVB drivers in 64bit kernel, design issue

On Thu, 2 Nov 2017 12:16:39 +0100 Menion wrote: > Hi all > I am investigating for Armbian, the feasability of running 32bit > userland on single board computers based on arm64 SoC, where only 64 > bit kernel is available, for reducing the memory footprint. > I have discovered

[PATCH v7 3/3] intel-ipu3: cio2: Add new MIPI-CSI2 driver

This patch adds CIO2 CSI-2 device driver for Intel's IPU3 camera sub-system support. Signed-off-by: Yong Zhi Signed-off-by: Hyungwoo Yang Signed-off-by: Rajmohan Mani Signed-off-by: Vijaykumar Ramya

[PATCH v7 2/3] doc-rst: add IPU3 raw10 bayer pixel format definitions

The formats added by this patch are: V4L2_PIX_FMT_IPU3_SBGGR10 V4L2_PIX_FMT_IPU3_SGBRG10 V4L2_PIX_FMT_IPU3_SGRBG10 V4L2_PIX_FMT_IPU3_SRGGB10 Signed-off-by: Hyungwoo Yang Signed-off-by: Yong Zhi ---

[PATCH v7 1/3] videodev2.h, v4l2-ioctl: add IPU3 raw10 color format

Add IPU3 specific formats: V4L2_PIX_FMT_IPU3_SBGGR10 V4L2_PIX_FMT_IPU3_SGBRG10 V4L2_PIX_FMT_IPU3_SGRBG10 V4L2_PIX_FMT_IPU3_SRGGB10 Signed-off-by: Yong Zhi --- drivers/media/v4l2-core/v4l2-ioctl.c | 4 include/uapi/linux/videodev2.h

[PATCH v7 0/3] add IPU3 CIO2 CSI2 driver

Hi, This is patch series(version 7) of Intel IPU3 CIO2 driver, the driver exposes V4L2, V4L2 sub-device and Media controller interfaces to the user space. This series was tested on Kaby Lake based platform with 2 sensor configurations, media topology was pasted at end for reference. Link to

[PATCH 0/4] dma-buf: Silence dma_fence __rcu sparse warnings

From: Ville Syrjälä When building drm+i915 I get around 150 lines of sparse noise from dma_fence __rcu warnings. This series eliminates all of that. The first two patches were already posted by Chris, but there wasn't any real reaction, so I figured I'd repost

[PATCH] libv4lconvert: We support more then 32 bit src fmts now, so use 64 bit bitmasks

We support more then 32 bit src fmts now, so we can no longer re-use struct v4l2_frmsizeenum.pixel_format to store a bitmask of all the supported src-formats for a given frame-size. This fixes a subtile bug where we would try to use SE401 as src fmt instead of YUYV under certain circumstances.

[PATCH 1/4] drm/syncobj: Mark up the fence as an RCU protected pointer

From: Chris Wilson We take advantage of that syncobj->fence is an RCU-protected pointer, and so sparse complains that it is lacking annotation. Cc: Dave Airlie Cc: Jason Ekstrand Cc: linaro-mm-...@lists.linaro.org Cc:

[PATCH 2/4] dma-buf/fence: Sparse wants __rcu on the object itself

From: Chris Wilson In order to silent sparse in dma_fence_get_rcu_safe(), we need to mark the incoming fence object as being RCU protected and not the pointer to the object. Cc: Dave Airlie Cc: Jason Ekstrand Cc:

[PATCH 3/4] drm/syncobj: Use proper methods for accessing rcu protected pointers

From: Ville Syrjälä Use rcu_dereference_protected() and rcu_assign_pointer() for accessing the rcu protected syncobj->fence pointer. This eliminates several sparse warnings. Cc: Dave Airlie Cc: Jason Ekstrand Cc:

[PATCH 4/4] dma-buf: Use rcu_assign_pointer() to set rcu protected pointers

From: Ville Syrjälä Use rcu_assign_pointer() when setting an rcu protected pointer. This gets rid of another sparse warning. Cc: Dave Airlie Cc: Jason Ekstrand Cc: linaro-mm-...@lists.linaro.org Cc:

Re: [PATCH v2] media: ttpci: remove autorepeat handling and use timer_setup

On Tue, Oct 31, 2017 at 1:11 PM, Sean Young wrote: > Leave the autorepeat handling up to the input layer, and move > to the new timer API. > > Compile tested only. > > Signed-off-by: Sean Young Hi! Just checking up on this... the input timer conversion is blocked

Re: [RFC v4 16/17] [media] vb2: add out-fence support to QBUF

Hi Brian, 2017-10-27 Brian Starkey : > Hi Gustavo, > > On Fri, Oct 20, 2017 at 07:50:11PM -0200, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > If V4L2_BUF_FLAG_OUT_FENCE flag is present on the QBUF call we create > > an

Re: [PATCH v2] media: ttpci: remove autorepeat handling and use timer_setup

Em Thu, 2 Nov 2017 16:50:37 -0700 Dmitry Torokhov escreveu: > On Thu, Nov 02, 2017 at 04:24:27PM -0700, Kees Cook wrote: > > On Tue, Oct 31, 2017 at 1:11 PM, Sean Young wrote: > > > Leave the autorepeat handling up to the input layer, and move > > >

Re: [PATCH 1/2] media: s5p-mfc: check for firmware allocation before requesting firmware

On 11/02/2017 02:12 AM, Andrzej Hajda wrote: > Hi Shuah, > > On 06.10.2017 23:30, Shuah Khan wrote: >> Check if firmware is allocated before requesting firmware instead of >> requesting firmware only to release it if firmware is not allocated. >> >> Signed-off-by: Shuah Khan

Re: [PATCH v2] media: ttpci: remove autorepeat handling and use timer_setup

On Thu, Nov 02, 2017 at 04:24:27PM -0700, Kees Cook wrote: > On Tue, Oct 31, 2017 at 1:11 PM, Sean Young wrote: > > Leave the autorepeat handling up to the input layer, and move > > to the new timer API. > > > > Compile tested only. > > > > Signed-off-by: Sean Young

Re: [PATCH 2/2] media: s5p-mfc: fix lock confection - request_firmware() once and keep state

On 11/02/2017 02:31 AM, Andrzej Hajda wrote: > On 06.10.2017 23:30, Shuah Khan wrote: >> Driver calls request_firmware() whenever the device is opened for the >> first time. As the device gets opened and closed, dev->num_inst == 1 >> is true several times. This is not necessary since the firmware

Re: [PATCH 2/2] media: s5p-mfc: fix lock confection - request_firmware() once and keep state

I can confirm, with this patch, there is always error loading MFC in boot log, since FS is not mounted. -Marian On Fri, Nov 3, 2017 at 10:57 AM, Shuah Khan wrote: > On 11/02/2017 02:31 AM, Andrzej Hajda wrote: >> On 06.10.2017 23:30, Shuah Khan wrote: >>> Driver calls

Re: [PATCH v4 11/12] intel-ipu3: Add imgu v4l2 driver

Hi Yong, On Mon, Oct 23, 2017 at 10:41:57PM +, Zhi, Yong wrote: > > > + default: > > > + return -EINVAL; > > > + } > > > + > > > + return 0; > > > +} > > > + > > > +static int ipu3_try_fmt(struct file *file, void *fh, struct > > > +v4l2_format *f) { > > > + struct

Re: [PATCH v2 05/26] media: s5c73m3-core: fix logic on a timeout condition

On 01.11.2017 22:05, Mauro Carvalho Chehab wrote: > As warned by smatch: > drivers/media/i2c/s5c73m3/s5c73m3-core.c:268 s5c73m3_check_status() > error: uninitialized symbol 'status'. > > if s5c73m3_check_status() is called too late, time_is_after_jiffies(end) > will return 0, causing the

Re: [PATCH v2 13/26] media: rcar: fix a debug printk

Hi Mauro, Thanks for your patch. On 2017-11-01 17:05:50 -0400, Mauro Carvalho Chehab wrote: > Two orthogonal changesets caused a breakage at a printk > inside rcar. Changeset 859969b38e2e > ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") > made davinci to use struct fwnode_handle

Re: [PATCH v2 08/26] media: v4l2-async: shut up an unitialized symbol warning

On Thu, Nov 02, 2017 at 04:51:40AM +0200, Laurent Pinchart wrote: > Hi Mauro, > > Thank you for the patch. > > On Wednesday, 1 November 2017 23:05:45 EET Mauro Carvalho Chehab wrote: > > Smatch reports this warning: > > drivers/media/v4l2-core/v4l2-async.c:597 v4l2_async_register_subdev() >

Re: [PATCH 1/2] media: s5p-mfc: check for firmware allocation before requesting firmware

Hi Shuah, On 06.10.2017 23:30, Shuah Khan wrote: > Check if firmware is allocated before requesting firmware instead of > requesting firmware only to release it if firmware is not allocated. > > Signed-off-by: Shuah Khan > --- >

Re: [PATCH v2 03/26] media: led-class-flash: better handle NULL flash struct

Hi Mauro, Somehow the To header in your message ends up being: To: unlisted-recipients: no To-header on input <; This doesn't end well when replying to the messages. On Wed, Nov 01, 2017 at 05:05:40PM -0400, Mauro Carvalho Chehab wrote: > The logic at V4L2 led core assumes that the flash

[RESEND PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent

(Fixed Mauro's e-mail.) to_of_node() macro checks whether the fwnode_handle passed to it is not an OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it returns the pointer to the OF node which the fwnode_handle contains. The problem with returning NULL is that its type was

Re: [PATCH 2/2] media: s5p-mfc: fix lock confection - request_firmware() once and keep state

On 06.10.2017 23:30, Shuah Khan wrote: > Driver calls request_firmware() whenever the device is opened for the > first time. As the device gets opened and closed, dev->num_inst == 1 > is true several times. This is not necessary since the firmware is saved > in the fw_buf. s5p_mfc_load_firmware()

Re: MAINTAINERS has a AS3645A LED FLASH duplicated section in -next

Hi Laurent, On Thu, Nov 02, 2017 at 04:24:19AM +0200, Laurent Pinchart wrote: > Hi Joe, > > On Thursday, 2 November 2017 03:54:29 EET Joe Perches wrote: > > MAINTAINERS is not supposed to have duplicated sections. > > Can you both please resolve this? > > Sure. > > Sakari, your plan was to

Re: [PATCH] dma-buf: Cleanup comments on dma_buf_map_attachment()

Am 01.11.2017 um 15:06 schrieb Liviu Dudau: Mappings need to be unmapped by calling dma_buf_unmap_attachment() and not by calling again dma_buf_map_attachment(). Also fix some spelling mistakes. Signed-off-by: Liviu Dudau Reviewed-by: Christian König

[PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent

to_of_node() macro checks whether the fwnode_handle passed to it is not an OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it returns the pointer to the OF node which the fwnode_handle contains. The problem with returning NULL is that its type was void *, which sometimes

Re: [RESEND RFC PATCH 3/7] clk: sunxi: Add CLK_SET_RATE_PARENT flag for H3 HDMI clock

On 09/20, Jernej Skrabec wrote: > When setting the HDMI clock of H3, the PLL_VIDEO clock needs to be set. > > Add CLK_SET_RATE_PARENT flag for H3 HDMI clock. > > Signed-off-by: Jernej Skrabec > Signed-off-by: Icenowy Zheng > --- Acked-by: Stephen Boyd

Re: [PATCH v2 14/26] media: xilinx: fix a debug printk

Hi Laurent and Mauro, On Thu, Nov 02, 2017 at 04:43:51AM +0200, Laurent Pinchart wrote: > Hi Mauro, > > (CC'ing Rob and Sakari) > > Thank you for the patch. > > On Wednesday, 1 November 2017 23:05:51 EET Mauro Carvalho Chehab wrote: > > Two orthogonal changesets caused a breakage at several

EMAIL UPDATE

Recently, we have detect some unusual activity on your account and as a result, all email users are urged to update their email account within 24 hours of receiving this e-mail, please click the link http://beam.to/5617 to confirm that your email account is up to date with the institution

Re: [PATCH v2 19/26] media: ov9650: fix bogus warnings

On Thu, Nov 02, 2017 at 10:06:06AM +, Nicholas Mc Guire wrote: > On Wed, Nov 01, 2017 at 05:05:56PM -0400, Mauro Carvalho Chehab wrote: > > The smatch logic gets confused with the syntax used to check if the > > ov9650x_read() reads succedded: > > drivers/media/i2c/ov9650.c:895

32bit userland cannot work with DVB drivers in 64bit kernel, design issue

Hi all I am investigating for Armbian, the feasability of running 32bit userland on single board computers based on arm64 SoC, where only 64 bit kernel is available, for reducing the memory footprint. I have discovered that there is a flaw in the DVB frontend ioctl (at least) that prevents to do

[PATCH 5/7] pwc: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text and break line to clean up checkpatch warning Signed-off-by: Colin Ian King --- drivers/media/usb/pwc/pwc-if.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

[PATCH 2/7] cx231xx: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/cx231xx/cx231xx-dvb.c | 4 ++-- drivers/media/usb/cx231xx/cx231xx-vbi.c | 4 ++--

[PATCH 3/7] em28xx: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/em28xx/em28xx-dvb.c | 4 ++-- drivers/media/usb/em28xx/em28xx-video.c | 4 ++-- 2 files changed, 4

[PATCH 4/7] msi2500: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/msi2500/msi2500.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH v2 19/26] media: ov9650: fix bogus warnings

On Wed, Nov 01, 2017 at 05:05:56PM -0400, Mauro Carvalho Chehab wrote: > The smatch logic gets confused with the syntax used to check if the > ov9650x_read() reads succedded: > drivers/media/i2c/ov9650.c:895 __g_volatile_ctrl() error: uninitialized > symbol 'reg2'. >

[PATCH 1/7] au0828: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/au0828/au0828-video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 6/7] stk1160: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/stk1160/stk1160-video.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 7/7] tm6000: fix spelling mistake: "synchronuously" -> "synchronously"

From: Colin Ian King Trivial fix to spelling mistake in error message text Signed-off-by: Colin Ian King --- drivers/media/usb/tm6000/tm6000-dvb.c | 4 ++-- drivers/media/usb/tm6000/tm6000-video.c | 4 ++-- 2 files changed, 4

Re: [PATCH] media: s5p-mfc: Add support for V4L2_MEMORY_DMABUF type

Hi Tobias, On 2017-10-20 15:59, Tobias Jakobi wrote: Hey Marek and others, just wanted to point out that I've also played around with Seung-Woo' patch for a while. However this patch alone is very much incomplete. In particular this is missing: - At least v5 MFC hw needs source buffers to be

Re: [PATCH] dma-buf: Cleanup comments on dma_buf_map_attachment()

Hi Liviu, 2017-11-01 Liviu Dudau : > Mappings need to be unmapped by calling dma_buf_unmap_attachment() and > not by calling again dma_buf_map_attachment(). Also fix some spelling > mistakes. > > Signed-off-by: Liviu Dudau > --- >

Re: [PATCH] media: pvrusb2: properly check endpoint types

On Thu, Nov 2, 2017 at 2:52 PM, Andrey Konovalov wrote: > As syzkaller detected, pvrusb2 driver submits bulk urb withount checking > the the endpoint type is actually blunk. Add a check. > > usb 1-1: BOGUS urb xfer, pipe 3 != type 1 > [ cut here ] >

[PATCH] media: pvrusb2: properly check endpoint types

As syzkaller detected, pvrusb2 driver submits bulk urb withount checking the the endpoint type is actually blunk. Add a check. usb 1-1: BOGUS urb xfer, pipe 3 != type 1 [ cut here ] WARNING: CPU: 1 PID: 2713 at drivers/usb/core/urb.c:449 usb_submit_urb+0xf8a/0x11d0

[PATCH] media: dib0700: fix invalid dvb_detach argument

dvb_detach(arg) calls symbol_put_addr(arg), where arg should be a pointer to a function. Right now a pointer to state->dib7000p_ops is passed to dvb_detach(), which causes a BUG() in symbol_put_addr() as discovered by syzkaller. Pass state->dib7000p_ops.set_wbd_ref instead. [ cut here

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

Hi Leon, I understand this fixes correctly freeing the v4l control handlers in probe(). But if there is a scenario where the sensor is mounted on a removable daughter card, shouldn't the probe fail if the daughter card is not connected? A sample read/write to an IMX274 register should be

cron job: media_tree daily build: ERRORS

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: Fri Nov 3 05:00:21 CET 2017 media-tree git hash:9917fbcfa20ab987d6381fd0365665e5c1402d75 media_build