Re: [PATCH v1 1/1] media: atmel-isc: Add safety checks for NULL isc->raw_fmt struct

2018-11-23 Thread Eugen.Hristev
On 21.11.2018 16:50, Ken Sloat wrote: >>> From: Ken Sloat >>> >>> In some usages isc->raw_fmt will not be initialized. If this is the >>> case, it is very possible that a NULL struct de-reference will occur, >>> as this member is referenced many times. > >> Hello Ken, > >> Do you have any

[PATCH] v4l2-ioctl: Zero v4l2_pix_format_mplane reserved fields

2018-11-23 Thread Ezequiel Garcia
Make the core set the reserved fields to zero in v4l2_pix_format_mplane and v4l2_plane_pix_format structs, for _MPLANE queue types. Moving this to the core avoids having to do so in each and every driver. Suggested-by: Tomasz Figa Signed-off-by: Ezequiel Garcia ---

[PATCH 2/3] media: stkwebcam: Bugfix for not correctly initialized camera

2018-11-23 Thread Andreas Pape
stk_start_stream can only be called successfully if stk_initialise and stk_setup_format are called before. When using e.g. cheese it was observed that stk_initialise and stk_setup_format have not been called before which leads to no picture in that software whereas other tools like guvcview worked

[PATCH 1/3] media: stkwebcam: Support for ASUS A6VM notebook added.

2018-11-23 Thread Andreas Pape
The ASUS A6VM notebook has a built in stk11xx webcam which is mounted in a way that the video is vertically and horizontally flipped. Therefore this notebook is added to the special handling in the driver to automatically flip the video into the correct orientation. Signed-off-by: Andreas Pape

[PATCH 0/3] Fix for webcam issues with ASUS A6VM

2018-11-23 Thread Andreas Pape
This set of patches makes the Syntek USB webcam integrated into Asus A6VM notebooks accessible again. Andreas Pape (3): media: stkwebcam: Support for ASUS A6VM notebook added. media: stkwebcam: Bugfix for not correctly initialized camera media: stkwebcam: Bugfix for wrong return values

[PATCH 3/3] media: stkwebcam: Bugfix for wrong return values

2018-11-23 Thread Andreas Pape
usb_control_msg returns in case of a successfully sent message the number of sent bytes as a positive number. Don't use this value as a return value for stk_camera_read_reg, as a non-zero return value is used as an error condition in some cases when stk_camera_read_reg is called. Signed-off-by:

Re: 'bad remote port parent' warnings

2018-11-23 Thread Sakari Ailus
Hi Fabio, Philipp, On Thu, Nov 22, 2018 at 05:17:44PM -0200, Fabio Estevam wrote: > Hi Philipp, > > On Thu, Nov 22, 2018 at 2:27 PM Philipp Zabel wrote: > > > There are empty endpoint nodes (without remote-endpoint property) > > labeled ipu1_csi[01]_mux_from_parallel_sensor in the i.MX6 device

[PATCH] media: dvb-pll: don't re-validate tuner frequencies

2018-11-23 Thread Mauro Carvalho Chehab
The dvb_frontend core already checks for the frequencies. No need for any additional check inside the driver. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/dvb-pll.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/media/dvb-frontends/dvb-pll.c

Re: 'bad remote port parent' warnings

2018-11-23 Thread Fabio Estevam
Hi Sakari, On Fri, Nov 23, 2018 at 10:35 AM Sakari Ailus wrote: > Makes sense. This is not necessarily a fatal error. Could you send a patch? Yes, I have just sent it. Thanks

[PATCH] media: v4l2-fwnode: Demote warning to debug level

2018-11-23 Thread Fabio Estevam
On a imx6q-wandboard the following warnings are observed: [4.327794] video-mux 20e.iomuxc-gpr:ipu1_csi0_mux: bad remote port parent [4.336118] video-mux 20e.iomuxc-gpr:ipu2_csi1_mux: bad remote port parent As explained by Philipp Zabel: "There are empty endpoint nodes (without

[PATCH] seco-cec: fix Makefile

2018-11-23 Thread Hans Verkuil
Fix the incorrect obj-y. Signed-off-by: Hans Verkuil --- diff --git a/drivers/media/platform/seco-cec/Makefile b/drivers/media/platform/seco-cec/Makefile index 09900b087d02..a3f2c6bd3ac0 100644 --- a/drivers/media/platform/seco-cec/Makefile +++ b/drivers/media/platform/seco-cec/Makefile @@ -1

[PATCH v4l-utils] keytable: do not install bpf protocols decoders with execute permission

2018-11-23 Thread Sean Young
The rpm packaging system will try to extract debug information, which fails since there is no build id. This can be avoided by removing the execute permission. BPF relocatable files are executable anyway so this is the right thing to do. See:

[PATCH 1/2] media: sum6i-csi: Fix a few coding style issues

2018-11-23 Thread Mauro Carvalho Chehab
Make checkpatch.pl happier by running it on strict mode and using the --fix-inline to solve some issues. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c | 9 - drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.h | 2 +-

[PATCH 2/2] media: sun6i-csi: manually fix other coding style issues

2018-11-23 Thread Mauro Carvalho Chehab
There are a few other coding style issues reported by checkpatch while in --strict mode. Fix the ones that make sense. Signed-off-by: Mauro Carvalho Chehab --- .../platform/sunxi/sun6i-csi/sun6i_csi.c | 9 +++ .../platform/sunxi/sun6i-csi/sun6i_csi_reg.h | 24 +--

[GIT PULL FOR v4.21 v2] Various fixes

2018-11-23 Thread Hans Verkuil
(fixed up my SoB mess) Just one note: the "cec: keep track of outstanding transmits" is CC-ed to stable for v4.18 and up, but I prefer to wait until v4.21 before merging it to give it more test time. It is not something that happens in normal usage, so delaying this isn't a problem. Regards,

[GIT PULL FOR v4.21 v2] Various fixes/improvements

2018-11-23 Thread Hans Verkuil
The following changes since commit 5200ab6a32d6055428896a49ec9e3b1652c1a100: media: vidioc_cropcap -> vidioc_g_pixelaspect (2018-11-20 13:57:21 -0500) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-v4.20d3 for you to fetch changes up to

[GIT PULL FOR v4.21 v2] Various fixes (coda, rcar, pxp, others)

2018-11-23 Thread Hans Verkuil
Fixed my SoB mess. Regards, Hans The following changes since commit fbe57dde7126d1b2712ab5ea93fb9d15f89de708: media: ov7740: constify structures stored in fields of v4l2_subdev_ops structure (2018-11-06 07:17:02 -0500) are available in the Git repository at:

[PATCH] vivid: fix smatch warnings

2018-11-23 Thread Hans Verkuil
Reorganize code to fix two smatch warnings: drivers/media/platform/vivid/vivid-core.c: drivers/media/platform/vivid/vivid-core.c:889 vivid_create_instance() warn: potentially one past the end of array 'dev->query_dv_timings_qmenu[dev->query_dv_timings_size]'

[PATCH] media: vb2: be sure to free on errors

2018-11-23 Thread Mauro Carvalho Chehab
As reported by smatch: drivers/media/common/videobuf2/videobuf2-core.c: drivers/media/common/videobuf2/videobuf2-core.c:2159 vb2_mmap() warn: inconsistent returns 'mutex:>mmap_lock'. Locked on: line 2148 Unlocked on: line 2100 line 2108 line 2113

[PATCH] media: seco-cec: declare ops as static const

2018-11-23 Thread Mauro Carvalho Chehab
As warned by smatch: drivers/media/platform/seco-cec/seco-cec.c:338:21: warning: symbol 'secocec_cec_adap_ops' was not declared. Should it be static? This struct should be static. Also, it is const, so declare it as such. Signed-off-by: Mauro Carvalho Chehab ---

[GIT PULL FOR v4.21 v2] Various fixes

2018-11-23 Thread Hans Verkuil
(fixed my SoB mess in this v2) Various fixes, mostly related to issues found by syzbot. The following changes since commit fbe57dde7126d1b2712ab5ea93fb9d15f89de708: media: ov7740: constify structures stored in fields of v4l2_subdev_ops structure (2018-11-06 07:17:02 -0500) are available in

[GIT PULL FOR v4.21 v2] mem2mem, venus and vb2 fixes/improvements

2018-11-23 Thread Hans Verkuil
(Fixed my SoB mess) The following changes since commit fbe57dde7126d1b2712ab5ea93fb9d15f89de708: media: ov7740: constify structures stored in fields of v4l2_subdev_ops structure (2018-11-06 07:17:02 -0500) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git

Re: [PATCH] media: vb2: be sure to free on errors

2018-11-23 Thread Hans Verkuil
On 11/23/2018 01:07 PM, Mauro Carvalho Chehab wrote: > As reported by smatch: > drivers/media/common/videobuf2/videobuf2-core.c: > drivers/media/common/videobuf2/videobuf2-core.c:2159 vb2_mmap() warn: > inconsistent returns 'mutex:>mmap_lock'. > Locked on: line 2148 > Unlocked on: line

[PATCH 6/6] media: pixfmt-meta-d4xx.rst: Add a license to it

2018-11-23 Thread Mauro Carvalho Chehab
This file is a recent addition to media docs, and it is now the only one without any license. While the best is to dual-license with GFDL and GPL, it is, at least, compatible with GFDL, as this is a requirement to be part of the media uAPI docs. So, add such license to it. Signed-off-by: Mauro

[PATCH 4/6] media: svg files: dual-licence some files with GPL and GFDL

2018-11-23 Thread Mauro Carvalho Chehab
Along the time, several image files got replaced by me by new ones with similar contents. As those were not simple conversions, dual-license them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dvbstb.svg | 33 ++-

[PATCH 0/6] Add license tags/texts to all media documentation files

2018-11-23 Thread Mauro Carvalho Chehab
Right now, almost every file under Documentation/media don't contain any licensing text. That's bad, specially uAPI documents are licensed under GFDL[1]. [1] This is due to historical reasons: by the time they were written, it was believed that the best license for documents would be GFDL.

Wine List

2018-11-23 Thread Carolyn Williams
Hi, Would you be interested in acquiring an email list of "Wine Enthusiasts" from USA? Our Databases:- Beer Enthusiasts List, Alcohol Enthusiasts, Travelers, Liquor Enthusiasts, Chocolate Lovers, Camping and Outdoor Enthusiasts List and many more.., Each record in the list contains

Re: 'bad remote port parent' warnings

2018-11-23 Thread Steve Longerbeam
Hi Fabio, On 11/22/18 11:17 AM, Fabio Estevam wrote: Hi Philipp, On Thu, Nov 22, 2018 at 2:27 PM Philipp Zabel wrote: There are empty endpoint nodes (without remote-endpoint property) labeled ipu1_csi[01]_mux_from_parallel_sensor in the i.MX6 device trees for board DT implementers'

[PATCH 1/6] Documentation/media: uapi: Explicitly say there are no Invariant Sections

2018-11-23 Thread Mauro Carvalho Chehab
From: Ben Hutchings The GNU Free Documentation License allows for a work to specify Invariant Sections that are not allowed to be modified. (Debian considers that this makes such works non-free.) The Linux Media Infrastructure userspace API documentation does not specify any such sections, but

Re: [PATCH] media: v4l2-fwnode: Demote warning to debug level

2018-11-23 Thread Steve Longerbeam
Reviewed-by: Steve Longerbeam On 11/23/18 4:50 AM, Fabio Estevam wrote: On a imx6q-wandboard the following warnings are observed: [4.327794] video-mux 20e.iomuxc-gpr:ipu1_csi0_mux: bad remote port parent [4.336118] video-mux 20e.iomuxc-gpr:ipu2_csi1_mux: bad remote port

Re: [PATCH for v4.9] Revert "media: videobuf2-core: don't call memop 'finish' when queueing"

2018-11-23 Thread Sasha Levin
On Thu, Nov 22, 2018 at 12:43:56PM +0100, Hans Verkuil wrote: This reverts commit 9ac47200b51cb09d2f15dbefa67e0412741d98aa. This commit fixes a bug in upstream commit a136f59c0a1f ("vb2: Move buffer cache synchronisation to prepare from queue") which isn't present in 4.9. So as a result you

cron job: media_tree daily build: ERRORS

2018-11-23 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: Sat Nov 24 05:00:11 CET 2018 media-tree git hash:708d75fe1c7c6e9abc5381b6fcc32b49830383d0 media_build