cron job: media_tree daily build: OK

2018-06-11 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 Jun 12 05:00:17 CEST 2018 media-tree git hash:f2809d20b9250c675fca8268a0f6274277cca7ff media_build

Re: [PATCH] media: tm6000: add error handling for dvb_register_adapter

2018-06-11 Thread Alexandre-Xavier Labonté-Lamoureux
"adater" should be "adapter". Have a nice day, Alexandre-Xavier On Mon, Jun 11, 2018 at 12:39 AM, Zhouyang Jia wrote: > When dvb_register_adapter fails, the lack of error-handling code may > cause unexpected results. > > This patch adds error-handling code after calling dvb_register_adapter. >

Re: [PATCH] gpu: ipu-v3: Allow negative offsets for interlaced scanning

2018-06-11 Thread Steve Longerbeam
On 06/11/2018 02:19 AM, Philipp Zabel wrote: Hi Steve, On Sun, 2018-06-10 at 17:08 -0700, Steve Longerbeam wrote: Hi Philipp, On 06/01/2018 06:13 AM, Philipp Zabel wrote: The IPU also supports interlaced buffers that start with the bottom field. To achieve this, the the base address EBA

Re: [Xen-devel] [PATCH v2 5/9] xen/gntdev: Allow mappings for DMA buffers

2018-06-11 Thread Stefano Stabellini
On Mon, 11 Jun 2018, Oleksandr Andrushchenko wrote: > On 06/08/2018 10:21 PM, Boris Ostrovsky wrote: > > On 06/08/2018 01:59 PM, Stefano Stabellini wrote: > > > > > > > > > >    @@ -325,6 +401,14 @@ static int map_grant_pages(struct > > > > > > > > > > grant_map > > > > > > > > > > *map) > > > >

Re: [PATCH 2/2] media: i2c: Add driver for Aptina MT9V111

2018-06-11 Thread kbuild test robot
/linux/commits/Jacopo-Mondi/media-i2c-mt9v111-sensor-driver/20180611-233038 base: git://linuxtv.org/media_tree.git master config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests

Re: [PATCH] media: soc_camera: ov772x: correct setting of banding filter

2018-06-11 Thread jacopo mondi
Hi Mita-san, On Mon, Jun 11, 2018 at 12:42:26AM +0900, Akinobu Mita wrote: > The banding filter ON/OFF is controlled via bit 5 of COM8 register. It > is attempted to be enabled in ov772x_set_params() by the following line. > > ret = ov772x_mask_set(client, COM8, BNDF_ON_OFF, 1); > > But

[PATCH 0/2] media: i2c: mt9v111 sensor driver

2018-06-11 Thread Jacopo Mondi
Hello, this is a sensor level driver for Aptina MT9V111. The driver supports YUYV_2X8 output formats and VGA,QVGA,QQVGA,CIF and QCIF resolution. The driver allows control of frame rate through s_frame_interval or V4L2_CID_H/VBLANK control. In order to allow manual frame control, the chip is

[PATCH 1/2] dt-bindings: media: i2c: Document MT9V111 bindings

2018-06-11 Thread Jacopo Mondi
Add documentation for Aptina MT9V111 image sensor. Signed-off-by: Jacopo Mondi --- .../bindings/media/i2c/aptina,mt9v111.txt | 46 ++ 1 file changed, 46 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/i2c/aptina,mt9v111.txt diff --git

[PATCH 2/2] media: i2c: Add driver for Aptina MT9V111

2018-06-11 Thread Jacopo Mondi
Add V4L2 sensor driver for Aptina MT9V111 CMOS image sensor. The MT9V111 is a 1/4-Inch CMOS image sensor based on MT9V011 with an integrated Image Flow Processor. Signed-off-by: Jacopo Mondi --- MAINTAINERS |8 + drivers/media/i2c/Kconfig | 12 +

[PATCH] [media] cx18: remove redundant zero check on retval

2018-06-11 Thread Colin King
From: Colin Ian King The check for a zero retval is redundant as all paths that lead to this point have set retval to an error return value that is non-zero. Remove the redundant check. Detected by CoverityScan, CID#102589 ("Logically dead code") Signed-off-by: Colin Ian King ---

Re: [PATCH] media: stm32-dcmi: add power saving support

2018-06-11 Thread kbuild test robot
/commits/Hugues-Fruchet/media-stm32-dcmi-add-power-saving-support/20180611-174016 base: git://linuxtv.org/media_tree.git master config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel

Re: [Xen-devel] [PATCH v2 5/9] xen/gntdev: Allow mappings for DMA buffers

2018-06-11 Thread Oleksandr Andrushchenko
On 06/08/2018 10:21 PM, Boris Ostrovsky wrote: On 06/08/2018 01:59 PM, Stefano Stabellini wrote:    @@ -325,6 +401,14 @@ static int map_grant_pages(struct grant_map *map)    map->unmap_ops[i].handle = map->map_ops[i].handle;    if (use_ptemod)   

[GIT PULL for 4.19] Sensor, lens and ISP driver patches

2018-06-11 Thread Sakari Ailus
Hi Mauro, Here's the usual bunch of sensor, lens and ISP driver patches. In particular, there are new drivers for ov2680 sensor and dw9807 VCM lens controller part. Please pull. The following changes since commit f2809d20b9250c675fca8268a0f6274277cca7ff: media: omap2: fix compile-testing

[GIT PULL for 4.19] Add "rotation" property for sensors, use it

2018-06-11 Thread Sakari Ailus
Hi Mauro, This pull request adds the "rotation" property already used for display panels for sensors. Support for the property is added to the smiapp and ov5640 drivers. Please pull. The following changes since commit f2809d20b9250c675fca8268a0f6274277cca7ff: media: omap2: fix

Re: [PATCH v4 1/2] dt-bindings: Add bindings for AKM ak7375 voice coil lens

2018-06-11 Thread Sakari Ailus
On Thu, Jun 07, 2018 at 11:50:32AM +0800, bingbu@intel.com wrote: > From: Bingbu Cao > > Add device tree bindings for AKM ak7375 voice coil lens > driver. This chip is used to drive a lens in a camera module. > > Signed-off-by: Tianshu Qiu > Signed-off-by: Bingbu Cao Please remember to

Re: [PATCH] media: stm32-dcmi: add power saving support

2018-06-11 Thread kbuild test robot
/commits/Hugues-Fruchet/media-stm32-dcmi-add-power-saving-support/20180611-174016 base: git://linuxtv.org/media_tree.git master config: i386-allmodconfig (attached as .config) compiler: gcc-7 (Debian 7.3.0-16) 7.3.0 reproduce: # save the attached .config to linux build tree make ARCH

Re: Fwd: Re: [PATCHv5 0/3] drm/i915: add DisplayPort CEC-Tunneling-over-AUX support

2018-06-11 Thread Hans Verkuil
Hi Ville, I finally found some time to dig deeper into when a CEC device should be registered. Since it's been a long while since we discussed this let me just recap the situation and then propose three solutions: CEC is implemented for DP-to-HDMI branch devices through DPCD CEC registers.

[PATCH v4 7/8] media: imx274: fix typo

2018-06-11 Thread Luca Ceresoli
pd -> pad Signed-off-by: Luca Ceresoli Cc: Sakari Ailus --- Changed v3 -> v4: nothing --- drivers/media/i2c/imx274.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/imx274.c b/drivers/media/i2c/imx274.c index e5ba19b97083..2c13961e9764 100644 ---

[PATCH v4 0/8] media: imx274: cleanups, improvements and SELECTION API support

2018-06-11 Thread Luca Ceresoli
Hi, this patchset introduces cropping support for the Sony IMX274 sensor using the SELECTION API. With respect to v3, this version uses the SELECTION API with taget V4L2_SEL_TGT_COMPOSE to change the output resolution. This is the recommended API for cropping + downscaling. However for backward

[PATCH v4 3/8] media: imx274: get rid of mode_index

2018-06-11 Thread Luca Ceresoli
After restructuring struct imx274_frmfmt, the mode_index field is still in use only for two dev_dbg() calls in imx274_s_stream(). Let's remove it and avoid duplicated information. Replacing the first usage requires some rather annoying but trivial pointer math. The other one can be removed

[PATCH v4 1/8] media: imx274: initialize format before v4l2 controls

2018-06-11 Thread Luca Ceresoli
The current probe function calls v4l2_ctrl_handler_setup() before initializing the format info. This triggers call paths such as: imx274_probe -> v4l2_ctrl_handler_setup -> imx274_s_ctrl -> imx274_set_exposure, where priv->mode_index is accessed before being assigned. This is wrong but does not

[PATCH v4 5/8] media: imx274: simplify imx274_write_table()

2018-06-11 Thread Luca Ceresoli
imx274_write_table() is a mere wrapper (and the only user) to imx274_regmap_util_write_table_8(). Remove this useless indirection by merging the two functions into one. Also get rid of the wait_ms_addr and end_addr parameters since it does not make any sense to give them any values other than

[PATCH v4 6/8] media: imx274: add helper function to fill a reg_8 table chunk

2018-06-11 Thread Luca Ceresoli
Tables of struct reg_8 are used to simplify multi-byte register assignment. However filling these snippets with values computed at runtime is currently implemented by very similar functions doing the needed shift & mask manipulation. Replace all those functions with a unique helper function to

[PATCH v4 8/8] media: imx274: add SELECTION support for cropping

2018-06-11 Thread Luca Ceresoli
Currently this driver does not support cropping. The supported modes are the following, all capturing the entire area: - 3840x2160, 1:1 binning (native sensor resolution) - 1920x1080, 2:1 binning - 1280x720, 3:1 binning The VIDIOC_SUBDEV_S_FMT ioctl chooses among these 3 configurations the

[PATCH v4 2/8] media: imx274: consolidate per-mode data in imx274_frmfmt

2018-06-11 Thread Luca Ceresoli
Data about the implemented readout modes is partially stored in imx274_formats[], the rest is scattered in several arrays. The latter are then accessed using the mode index, e.g.: min_frame_len[priv->mode_index] Consolidate all these data in imx274_formats[], and store a pointer to the

[PATCH v4 4/8] media: imx274: actually use IMX274_DEFAULT_MODE

2018-06-11 Thread Luca Ceresoli
IMX274_DEFAULT_MODE is defined but not used. Start using it, so the default can be more easily changed without digging into the code. Signed-off-by: Luca Ceresoli Cc: Sakari Ailus --- Changed v3 -> v4: nothing Changed v2 -> v3: nothing Changed v1 -> v2: - add "media: " prefix to commit

Re: [PATCH 2/2] media: ov5640: add support of module orientation

2018-06-11 Thread Hugues FRUCHET
Hi Sakari, I'm fine with the change to dev_fwnode(>dev). Many thanks Sakari, Hugues. On 06/11/2018 12:10 PM, Sakari Ailus wrote: > On Mon, Jun 11, 2018 at 11:29:17AM +0200, Hugues Fruchet wrote: >> Add support of module being physically mounted upside down. >> In this case, mirror and flip are

Re: [PATCH v7 0/6] Add ChromeOS EC CEC Support

2018-06-11 Thread Hans Verkuil
On 11/06/18 10:56, Neil Armstrong wrote: > Hi Lee, > > On 11/06/2018 08:03, Lee Jones wrote: >> On Fri, 08 Jun 2018, Hans Verkuil wrote: >>> On 08/06/18 10:17, Neil Armstrong wrote: On 08/06/2018 09:53, Hans Verkuil wrote: > On 06/01/2018 10:19 AM, Neil Armstrong wrote: >> Hi All,

Re: [PATCH 2/2] media: ov5640: add support of module orientation

2018-06-11 Thread Sakari Ailus
On Mon, Jun 11, 2018 at 11:29:17AM +0200, Hugues Fruchet wrote: > Add support of module being physically mounted upside down. > In this case, mirror and flip are enabled to fix captured images > orientation. > > Signed-off-by: Hugues Fruchet > --- > .../devicetree/bindings/media/i2c/ov5640.txt

[PATCH] media: stm32-dcmi: revisit stop streaming ops

2018-06-11 Thread Hugues Fruchet
Do not wait for interrupt completion when stopping streaming, stopping sensor and disabling interruptions are enough. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 29 + 1 file changed, 1 insertion(+), 28 deletions(-) diff --git

[PATCH 3/4] media: stm32-dcmi: clarify state logic on buffer starvation

2018-06-11 Thread Hugues Fruchet
Introduce WAIT_FOR_BUFFER state instead of "active" field checking to manage buffer starvation case. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git

[PATCH] media: stm32-dcmi: code cleanup

2018-06-11 Thread Hugues Fruchet
Minor non-functional fixes around comments, variable namings and trace point enhancement. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-dcmi.c

[PATCH 0/4] Revisit and fix DCMI buffers handling

2018-06-11 Thread Hugues Fruchet
Revisit and fix DCMI buffers handling. Hugues Fruchet (4): media: stm32-dcmi: do not fall into error on buffer starvation media: stm32-dcmi: return buffer in error state on dma error media: stm32-dcmi: clarify state logic on buffer starvation media: stm32-dcmi: revisit buffer list

[PATCH 1/4] media: stm32-dcmi: do not fall into error on buffer starvation

2018-06-11 Thread Hugues Fruchet
Return silently instead of falling into error when running out of available buffers when restarting capture. Capture will be restarted when new buffers will be provided by V4L2 client. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 7 ++- 1 file changed, 2

[PATCH 2/4] media: stm32-dcmi: return buffer in error state on dma error

2018-06-11 Thread Hugues Fruchet
Return buffer to V4L2 in error state if DMA error occurs. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/platform/stm32/stm32-dcmi.c b/drivers/media/platform/stm32/stm32-dcmi.c index

[PATCH 4/4] media: stm32-dcmi: revisit buffer list management

2018-06-11 Thread Hugues Fruchet
Cleanup "active" field usage and enhance list management to avoid exceptions when releasing buffers on error or stopping streaming. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 65 +++ 1 file changed, 31 insertions(+), 34 deletions(-)

[PATCH] media: stm32-dcmi: increase max width/height to 2592

2018-06-11 Thread Hugues Fruchet
DCMI can capture 5Mp raw frames, increase limit accordingly. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-dcmi.c

[PATCH] media: stm32-dcmi: add power saving support

2018-06-11 Thread Hugues Fruchet
Implements runtime & system sleep power management ops. Signed-off-by: Hugues Fruchet --- drivers/media/platform/stm32/stm32-dcmi.c | 80 --- 1 file changed, 64 insertions(+), 16 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-dcmi.c

[PATCH 2/2] media: ov5640: add support of module orientation

2018-06-11 Thread Hugues Fruchet
Add support of module being physically mounted upside down. In this case, mirror and flip are enabled to fix captured images orientation. Signed-off-by: Hugues Fruchet --- .../devicetree/bindings/media/i2c/ov5640.txt | 3 +++ drivers/media/i2c/ov5640.c | 28

[PATCH 0/2] OV5640 hflip, vflip and module orientation support

2018-06-11 Thread Hugues Fruchet
This patch serie relates to flip and mirror at sensor level through registers "TIMING TC REG20/REG21". The first commit implements HFLIP and VFLIP V4L2 controls allowing V4L2 client to change horizontal and vertical flip before or during streaming. The second commit allows to inform driver of

[PATCH 1/2] media: ov5640: add HFLIP/VFLIP controls support

2018-06-11 Thread Hugues Fruchet
Add HFLIP/VFLIP controls support by setting registers REG21/REG20. Useless values in hardcoded mode sequences are removed and remaining binning values are now set after mode sequence being set. Note that due to BSI (Back Side Illuminated) technology, image capture is physically mirrored, mirror

Re: [PATCH] gpu: ipu-v3: Allow negative offsets for interlaced scanning

2018-06-11 Thread Philipp Zabel
Hi Steve, On Sun, 2018-06-10 at 17:08 -0700, Steve Longerbeam wrote: > Hi Philipp, > > On 06/01/2018 06:13 AM, Philipp Zabel wrote: > > The IPU also supports interlaced buffers that start with the bottom field. > > To achieve this, the the base address EBA has to be increased by a stride > >

Re: [PATCH v7 0/6] Add ChromeOS EC CEC Support

2018-06-11 Thread Neil Armstrong
Hi Lee, On 11/06/2018 08:03, Lee Jones wrote: > On Fri, 08 Jun 2018, Hans Verkuil wrote: >> On 08/06/18 10:17, Neil Armstrong wrote: >>> On 08/06/2018 09:53, Hans Verkuil wrote: On 06/01/2018 10:19 AM, Neil Armstrong wrote: > Hi All, > > The new Google "Fizz" Intel-based ChromeOS

[PATCH] media: bt8xx: bttv: fix spelling mistake: "culpit" -> "culprit"

2018-06-11 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in pr_notice message text Signed-off-by: Colin Ian King --- drivers/media/pci/bt8xx/bttv-driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/pci/bt8xx/bttv-driver.c

Re: [RFC PATCH 2/2] media: docs-rst: Add encoder UAPI specification to Codec Interfaces

2018-06-11 Thread Tomasz Figa
Hi Hans, On Thu, Jun 7, 2018 at 6:21 PM Hans Verkuil wrote: > > On 06/05/2018 12:33 PM, Tomasz Figa wrote: [snip] > > +Initialization > > +-- > > + > > +1. (optional) Enumerate supported formats and resolutions. See > > + capability enumeration. > > + > > +2. Set a coded format on

[PATCH] media: cx88: add error handling for snd_ctl_add

2018-06-11 Thread Zhouyang Jia
When snd_ctl_add fails, the lack of error-handling code may cause unexpected results. This patch adds error-handling code after calling snd_ctl_add. Signed-off-by: Zhouyang Jia --- drivers/media/pci/cx88/cx88-alsa.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git

Re: [PATCH v2 1/2] media: v4l2-ctrl: Add control for VP9 profile

2018-06-11 Thread Hans Verkuil
On 11/06/18 08:44, Keiichi Watanabe wrote: > Hi, Hans. > Thank you for the review. > Your idea sounds good. > > However, I think that changing V4L2_CID_MPEG_VIDEO_VPX_PROFILE to an enum > breaks both of s5p-mfc and venus drivers. This is because they call > 'v4l2_ctrl_new_std' for it. For menu

Re: [PATCH -next] media/platform/cadence: add to fix build error

2018-06-11 Thread Maxime Ripard
On Fri, Jun 08, 2018 at 02:19:06PM -0700, Randy Dunlap wrote: > From: Randy Dunlap > > Add #include to fix build errors. > This driver uses kzalloc() and kfree() so it needs to #include > the appropriate header file for those interfaces. > > Fixes these build errors: > >

Re: [PATCH v2 1/2] media: v4l2-ctrl: Add control for VP9 profile

2018-06-11 Thread Keiichi Watanabe
Hi, Hans. Thank you for the review. Your idea sounds good. However, I think that changing V4L2_CID_MPEG_VIDEO_VPX_PROFILE to an enum breaks both of s5p-mfc and venus drivers. This is because they call 'v4l2_ctrl_new_std' for it. For menu controls, 'v4l2_ctrl_new_std_menu' must be used.

Re: [PATCH v7 0/6] Add ChromeOS EC CEC Support

2018-06-11 Thread Lee Jones
On Fri, 08 Jun 2018, Hans Verkuil wrote: > On 08/06/18 10:17, Neil Armstrong wrote: > > On 08/06/2018 09:53, Hans Verkuil wrote: > >> On 06/01/2018 10:19 AM, Neil Armstrong wrote: > >>> Hi All, > >>> > >>> The new Google "Fizz" Intel-based ChromeOS device is gaining CEC support > >>> through it's