cron job: media_tree daily build: ERRORS

2018-09-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: Wed Sep 12 04:00:17 CEST 2018 media-tree git hash:cc1e6315e83db0e517dd9279050b88adc83a7eba media_build

Inquiry

2018-09-11 Thread Sinara Group
Hello, This is Daniel Murray and i am from Sinara Group Co.Ltd Group Co.,LTD in Russia. We are glad to know about your company from the web and we are interested in your products. Could you kindly send us your Latest catalog and price list for our trial order. Best Regards, Daniel Murray

Re: Fund

2018-09-11 Thread Reem Al-Hashimy
Hello, My name is ms. Reem Al-Hashimi. The UAE minister of state for international cooparation. I got your contact from a certain email database from your country while i was looking for someone to handle a huge financial transaction for me in confidence. Can you receive and invest on behalf

[PATCH v3 1/2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
add V4L2_FRMIVAL_TYPE_CONTINUOUS and V4L2_FRMIVAL_TYPE_STEPWISE for subdev's frame intervals in addition to implicit existing V4L2_FRMIVAL_TYPE_DISCRETE type. This needs three new fields in the v4l2_subdev_frame_interval_enum struct : - type - max_interval - step_interval A helper function

[PATCH v3 2/2] media: imx: capture: use 'v4l2_fill_frmivalenum_from_subdev'

2018-09-11 Thread Philippe De Muyter
--- drivers/staging/media/imx/imx-media-capture.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/staging/media/imx/imx-media-capture.c b/drivers/staging/media/imx/imx-media-capture.c index 256039c..688dd7a 100644 ---

Editing 3

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

[PATCH v3 0/2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
add V4L2_FRMIVAL_TYPE_CONTINUOUS and V4L2_FRMIVAL_TYPE_STEPWISE for subdev's frame intervals in addition to implicit existing V4L2_FRMIVAL_TYPE_DISCRETE type. -- v2: Add a 'type' field and a helper function, as asked by Hans v3: Fix documentation (as asked by Hans) Convert

Editing 5

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

Editing 3

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

Editing 4

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

Editing 4

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

Re: [PATCH 1/3] media: use strscpy() instead of strlcpy()

2018-09-11 Thread Hans Verkuil
On 09/10/2018 02:19 PM, Mauro Carvalho Chehab wrote: > The implementation of strscpy() is more robust and safer. > > That's now the recommended way to copy NUL terminated strings. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Hans Verkuil

Re: [PATCH 2/3 v2] media: replace strcpy() by strscpy()

2018-09-11 Thread Hans Verkuil
On 09/10/2018 10:20 PM, Mauro Carvalho Chehab wrote: > The strcpy() function is being deprecated upstream. Replace > it by the safer strscpy(). > > Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Hans Verkuil Regards, Hans

Re: [PATCH 5/6] media: isp: fix a warning about a wrong struct initializer

2018-09-11 Thread Laurent Pinchart
Hi Mauro, On Friday, 7 September 2018 14:46:34 EEST Laurent Pinchart wrote: > Hi Mauro, > > As maintainers should be held to the same level of obligations as > developers, and to avoid demotivating reviewers, could you handle comments > you receive before pushing your own patches to your tree ?

[PATCH v4] vb2: check for sane values from queue_setup

2018-09-11 Thread Johan Fjeldtvedt
Warn and return error from the reqbufs ioctl when driver sets 0 number of planes or 0 as plane sizes, as these values don't make any sense. Checking this here stops obviously wrong values from propagating further and causing various problems that are hard to trace back to either of these values

Editing 1

2018-09-11 Thread Aaron
Hi, If you have photos for editing, please send email to: hansre...@outlook.com We have 12 in house image editors and we can help you for cutting out your photos, or path the photos. Includes retouching if needed. Used for products photos or portrait photos, catalog photos. You may drop us

[PATCH v3 4/5] media: ov5640: fix auto controls values when switching to manual mode

2018-09-11 Thread Hugues Fruchet
When switching from auto to manual mode, V4L2 core is calling g_volatile_ctrl() in manual mode in order to get the manual initial value. Remove the manual mode check/return to not break this behaviour. Signed-off-by: Hugues Fruchet --- drivers/media/i2c/ov5640.c | 4 1 file changed, 4

[PATCH v3 0/5] Fix OV5640 exposure & gain

2018-09-11 Thread Hugues Fruchet
This patch serie fixes some problems around exposure & gain in OV5640 driver. The 4th patch about autocontrols requires also a fix in v4l2-ctrls.c: https://www.mail-archive.com/linux-media@vger.kernel.org/msg133164.html Here is the test procedure used for exposure & gain controls check: 1)

[PATCH v3 2/5] media: ov5640: fix auto gain & exposure when changing mode

2018-09-11 Thread Hugues Fruchet
Ensure that auto gain and auto exposure are well restored when changing mode. Signed-off-by: Hugues Fruchet --- drivers/media/i2c/ov5640.c | 96 ++ 1 file changed, 54 insertions(+), 42 deletions(-) diff --git a/drivers/media/i2c/ov5640.c

[PATCH v3 3/5] media: ov5640: fix wrong binning value in exposure calculation

2018-09-11 Thread Hugues Fruchet
ov5640_set_mode_exposure_calc() is checking binning value but binning value read is buggy, fix this. Rename ov5640_binning_on() to ov5640_get_binning() as per other similar functions. Signed-off-by: Hugues Fruchet Reviewed-by: Laurent Pinchart --- drivers/media/i2c/ov5640.c | 8 1

[PATCH v3 1/5] media: ov5640: fix exposure regression

2018-09-11 Thread Hugues Fruchet
fixes: bf4a4b518c20 ("media: ov5640: Don't force the auto exposure state at start time"). Symptom was black image when capturing HD or 5Mp picture due to manual exposure set to 1 while it was intended to set autoexposure to "manual", fix this. Signed-off-by: Hugues Fruchet Reviewed-by: Laurent

[PATCH v3 5/5] media: ov5640: fix restore of last mode set

2018-09-11 Thread Hugues Fruchet
Mode setting depends on last mode set, in particular because of exposure calculation when downscale mode change between subsampling and scaling. At stream on the last mode was wrongly set to current mode, so no change was detected and exposure calculation was not made, fix this. Signed-off-by:

Re: [PATCH v2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Hans Verkuil
On 09/11/18 10:36, Philippe De Muyter wrote: > add V4L2_FRMIVAL_TYPE_CONTINUOUS and V4L2_FRMIVAL_TYPE_STEPWISE for > subdev's frame intervals in addition to implicit existing > V4L2_FRMIVAL_TYPE_DISCRETE type. This needs three new fields in the > v4l2_subdev_frame_interval_enum struct : > - type

[PATCH v3] vb2: check for sane values from queue_setup

2018-09-11 Thread Johan Fjeldtvedt
Warn and return error from the reqbufs ioctl when driver sets 0 number of planes or 0 as plane sizes, as these values don't make any sense. Checking this here stops obviously wrong values from propagating further and causing various problems that are hard to trace back to either of these values

Re: [PATCH v2] vb2: check for sane values from queue_setup

2018-09-11 Thread Hans Verkuil
A few more comments: On 09/11/18 13:58, Johan Fjeldtvedt wrote: > Warn when driver sets 0 number of planes or 0 as plane sizes. It should also return an error, since there is no point continuing with garbage values. Also add *why* this is useful to the commit. I know why, since I suggested it

[PATCH v2] vb2: check for sane values from queue_setup

2018-09-11 Thread Johan Fjeldtvedt
Warn when driver sets 0 number of planes or 0 as plane sizes. Signed-off-by: Johan Fjeldtvedt --- drivers/media/common/videobuf2/videobuf2-core.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/media/common/videobuf2/videobuf2-core.c

Re: [PATCH] vb2: check for sane values from queue_setup

2018-09-11 Thread Hans Verkuil
Hi Johan, Thank you for the patch, but I have some small comments: On 09/11/18 13:28, Johan Fjeldtvedt wrote: > Warn when driver sets 0 number of planes or 0 as plane sizes. > --- > drivers/media/common/videobuf2/videobuf2-core.c | 8 > 1 file changed, 8 insertions(+) > > diff --git

[PATCH] vb2: check for sane values from queue_setup

2018-09-11 Thread Johan Fjeldtvedt
Warn when driver sets 0 number of planes or 0 as plane sizes. --- drivers/media/common/videobuf2/videobuf2-core.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/media/common/videobuf2/videobuf2-core.c b/drivers/media/common/videobuf2/videobuf2-core.c index

Re: [PATCH v6 0/3] Add support for MPEG-2 in DELTA video decoder

2018-09-11 Thread Hugues FRUCHET
Hi Hans, On 09/09/2018 11:38 AM, Hans Verkuil wrote: > Hi Hugues, > > On 04/28/2017 03:25 PM, Hugues Fruchet wrote: >> The patchset implements the MPEG-2 part of V4L2 unified low-level decoder >> API RFC [0] needed by stateless video decoders, ie decoders which requires >> specific parsing

[PATCH v2] media: v4l2-subdev.h: allow V4L2_FRMIVAL_TYPE_CONTINUOUS & _STEPWISE

2018-09-11 Thread Philippe De Muyter
add V4L2_FRMIVAL_TYPE_CONTINUOUS and V4L2_FRMIVAL_TYPE_STEPWISE for subdev's frame intervals in addition to implicit existing V4L2_FRMIVAL_TYPE_DISCRETE type. This needs three new fields in the v4l2_subdev_frame_interval_enum struct : - type - max_interval - step_interval A helper function

Re: [PATCH v2 5/5] media: ov5640: fix restore of last mode set

2018-09-11 Thread Hugues FRUCHET
Hi Laurent, On 09/10/2018 10:56 PM, Laurent Pinchart wrote: > Hi Hugues, > > On Monday, 10 September 2018 18:14:45 EEST Hugues FRUCHET wrote: >> On 09/07/2018 04:18 PM, Laurent Pinchart wrote: >>> On Thursday, 16 August 2018 18:07:54 EEST Hugues FRUCHET wrote: On 08/16/2018 12:10 PM, jacopo

[GIT PULL FOR v4.20 (request_api branch)] Add Allwinner cedrus decoder driver

2018-09-11 Thread Hans Verkuil
This supersedes my previous pull request for this since I inadvertently left in the dts patches, but those go through a separate subsystem. Hi Mauro, This is the cedrus Allwinner decoder driver. It is for the request_api topic branch, but it assumes that this pull request is applied first:

Re: [GIT PULL FOR v4.20 (request_api branch)] Add Allwinner cedrus decoder driver

2018-09-11 Thread Hans Verkuil
On 09/11/18 09:47, Maxime Ripard wrote: > Hi Hans, > > On Mon, Sep 10, 2018 at 10:34:53AM +0200, Hans Verkuil wrote: >> This is the cedrus Allwinner decoder driver. It is for the request_api topic >> branch, but it assumes that this pull request is applied first: >>

Re: [GIT PULL FOR v4.20 (request_api branch)] Add Allwinner cedrus decoder driver

2018-09-11 Thread Maxime Ripard
Hi Hans, On Mon, Sep 10, 2018 at 10:34:53AM +0200, Hans Verkuil wrote: > This is the cedrus Allwinner decoder driver. It is for the request_api topic > branch, but it assumes that this pull request is applied first: > https://patchwork.linuxtv.org/patch/51889/ > > The last two patches could

Re: [PATCH v2 5/5] media: ov5640: fix restore of last mode set

2018-09-11 Thread Hugues FRUCHET
Hi Jacopo, On 08/25/2018 04:53 PM, jacopo mondi wrote: > Hi Hugues, > one more comment on this patch.. > > On Mon, Aug 13, 2018 at 12:19:46PM +0200, Hugues Fruchet wrote: >> Mode setting depends on last mode set, in particular >> because of exposure calculation when downscale mode >> change

hi

2018-09-11 Thread Sherri Gallagher
Please reply me back I have something to tell you.I am Sgt.Sherri.

Re: [PATCH 2/2] vicodec: set state->info before calling the encode/decode funcs

2018-09-11 Thread Hans Verkuil
On 09/10/2018 05:37 PM, Ezequiel Garcia wrote: > On Mon, 2018-09-10 at 17:00 +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> state->info was NULL since I completely forgot to set state->info. >> Oops. >> >> Reported-by: Ezequiel Garcia >> Signed-off-by: Hans Verkuil > > For both