cron job: media_tree daily build: WARNINGS

2019-09-13 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 Sep 14 05:00:11 CEST 2019 media-tree git hash:6f51fdfd8229d5358c2d6e272cf73478866e8ddc media_build gi

Re: [PATCH 3/3] media: imx: fix media bus format enumeration

2019-09-13 Thread kbuild test robot
Hi Philipp, I love your patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [cannot apply to v5.3-rc8 next-20190904] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/co

Re: [PATCH 2/3] media: imx: fix and simplify pixel format enumeration

2019-09-13 Thread kbuild test robot
Hi Philipp, I love your patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [cannot apply to v5.3-rc8 next-20190904] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/co

Re: [PATCH] uvc/vsp1/ipu3: use VFL_TYPE_METADATA for metadata device.

2019-09-13 Thread kbuild test robot
Hi Vandana, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [cannot apply to v5.3-rc8 next-20190904] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/co

Re: [RFC PATCH V3 1/5] dt-bindings: mt8183: Added DIP dt-bindings

2019-09-13 Thread Rob Herring
On Tue, 10 Sep 2019 03:22:40 +0800, wrote: > From: Frederic Chen > > This patch adds DT binding documentation for the Digital Image > Processing (DIP) unit of camera ISP system on Mediatek's SoCs. > > It depends on the SCP and MDP 3 patch as following: > > 1. dt-bindings: Add a binding for Med

Re: [PATCH v3 06/11] media: v4l2-fwnode: Add helper to parse device properties

2019-09-13 Thread Jacopo Mondi
Hi Hans, On Fri, Sep 13, 2019 at 04:08:10PM +0200, Hans Verkuil wrote: > On 9/12/19 10:10 PM, Jacopo Mondi wrote: > > Add an helper function to parse common device properties in the same > > way as v4l2_fwnode_endpoint_parse() parses common endpoint properties. > > > > Initially parse the 'rotatio

Re: [PATCH v3 04/11] media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_ROTATION

2019-09-13 Thread Jacopo Mondi
Hi Hans, On Fri, Sep 13, 2019 at 04:02:45PM +0200, Hans Verkuil wrote: > On 9/12/19 10:10 PM, Jacopo Mondi wrote: > > Add documentation for the V4L2_CID_CAMERA_SENSOR_ROTATION camera > > control. The newly added read-only control reports the camera device > > mounting rotation. > > > > Signed-off-

[PATCH -next v2 0/2] media: aspeed: refine mode detection flow

2019-09-13 Thread Jae Hyun Yoo
This patch series refines mode detection flow by fixing and refining of hsync/vsync polarity setting register handling. Please review this change. Thanks, -Jae Changes since v1: * Updated commit message. Jae Hyun Yoo (2): media: aspeed: refine hsync/vsync polarity setting logic media: aspee

[PATCH -next v2 2/2] media: aspeed: set hsync and vsync polarities to normal before starting mode detection

2019-09-13 Thread Jae Hyun Yoo
Sometimes it detects a weird resolution such as 1024x287 when the actual resolution is 1024x768. To resolve such an issue, this commit adds clearing for hsync and vsync polarity register bits at the beginning of the first mode detection. This is recommended in the datasheet. Signed-off-by: Jae Hyu

[PATCH -next v2 1/2] media: aspeed: refine hsync/vsync polarity setting logic

2019-09-13 Thread Jae Hyun Yoo
To prevent inaccurate detections of resolution, this commit enables clearing of hsync/vsync polarity bits based on probed sync state. Signed-off-by: Jae Hyun Yoo --- Changes since v1: * Updated commit message. drivers/media/platform/aspeed-video.c | 43 +-- 1 file chang

[PATCH] media: add a subsystem profile documentation

2019-09-13 Thread Mauro Carvalho Chehab
Document the basic policies of the media subsystem profile. Signed-off-by: Mauro Carvalho Chehab --- That's basically a modified version of: https://patchwork.linuxtv.org/patch/52999/ Applied to the new template Documentation/media/index.rst | 1 + .../media/maintainer-e

Re: [PATCH -next 2/2] media: aspeed: set hsync and vsync polarities to normal before starting mode detection

2019-09-13 Thread Jae Hyun Yoo
On 9/12/2019 5:18 PM, Andrew Jeffery wrote: On Fri, 13 Sep 2019, at 02:36, Jae Hyun Yoo wrote: On 9/11/2019 10:39 PM, Andrew Jeffery wrote: On Wed, 11 Sep 2019, at 04:37, Jae Hyun Yoo wrote: Sometimes it detects a weird resolution such as 1024x287 when the actual resolution is 1024x768. To

Re: Proposal for a v4l2_ctrl_new_std_compound() function

2019-09-13 Thread kbuild test robot
/commits/Hans-Verkuil/Proposal-for-a-v4l2_ctrl_new_std_compound-function/20190913-222917 base: git://linuxtv.org/media_tree.git master reproduce: make htmldocs If you fix the issue, kindly add following tag Reported-by: kbuild test robot All warnings (new ones prefixed by >>): Warnin

coda9 jpeg support?

2019-09-13 Thread Tim Harvey
Greetings, What would need to be done to support JPEG enc/dec for coda9? Best Regards, Tim

Re: [PATCH v3 09/11] media: v4l2-ctrls: Add helper to register properties

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Add an helper function to v4l2-ctrls to register controls associated > with a device property. Add an UNSET flag to the device properties to > distinguish uninitialized properties from properties with an actual > value at control registration time. > > Si

Re: [PATCH v3 06/11] media: v4l2-fwnode: Add helper to parse device properties

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Add an helper function to parse common device properties in the same > way as v4l2_fwnode_endpoint_parse() parses common endpoint properties. > > Initially parse the 'rotation' and 'location' properties from the Just drop 'Initially'. It's redundant. >

Re: [PATCH v3 04/11] media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_ROTATION

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Add documentation for the V4L2_CID_CAMERA_SENSOR_ROTATION camera > control. The newly added read-only control reports the camera device > mounting rotation. > > Signed-off-by: Jacopo Mondi > --- > .../media/uapi/v4l/ext-ctrls-camera.rst | 117

Re: [PATCH v3 03/11] dt-bindings: video-interfaces: Expand rotation description

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Expand the 'rotation' property description to define the direction and > orientation of the axis around which the device mounting rotation is > expressed. > > Signed-off-by: Jacopo Mondi > --- > .../devicetree/bindings/media/video-interfaces.txt

Re: [PATCH v3 02/11] media: v4l2-ctrl: Document V4L2_CID_CAMERA_SENSOR_LOCATION

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Add documentation for the V4L2_CID_CAMERA_SENSOR_LOCATION camera > control. The newly added read-only control reports the camera device > mounting position. > > Signed-off-by: Jacopo Mondi > --- > .../media/uapi/v4l/ext-ctrls-camera.rst | 32 +

Re: [PATCH v3 01/11] dt-bindings: video-interfaces: Document 'location' property

2019-09-13 Thread Hans Verkuil
On 9/12/19 10:10 PM, Jacopo Mondi wrote: > Add the 'location' device property, used to specify a device mounting > position. The property is particularly meaningful for mobile devices > with a well defined usage orientation. > > Signed-off-by: Jacopo Mondi > --- > .../devicetree/bindings/media/v

Re: [PATCH v3 5/6] rcar-vin: Add support for V4L2_FIELD_ALTERNATE

2019-09-13 Thread Kieran Bingham
On 04/09/2019 22:54, Niklas Söderlund wrote: > The hardware is capable to passing V4L2_FIELD_ALTERNATE to user-space. s/to/of/ > Allow users to request this field format but still default to using the > hardware interlacer if alternating is not explicitly requested. > > Before this change a sens

Re: [PATCH v3 2/6] rcar-vin: Make use of V4L2_FIELD_IS_INTERLACED() macro

2019-09-13 Thread Kieran Bingham
On 04/09/2019 22:54, Niklas Söderlund wrote: > The V4L2_FIELD_IS_INTERLACED() can be used to make the code more > readable, use it. Definitely! > > Signed-off-by: Niklas Söderlund > Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham > --- > drivers/media/platform/rcar-vin/rcar-dma.c

Re: [PATCH v3 1/6] rcar-vin: Fix incorrect return statement in rvin_try_format()

2019-09-13 Thread Kieran Bingham
Hi Niklas, On 04/09/2019 22:54, Niklas Söderlund wrote: > While refactoring code the return statement became corrupted, fix it by > returning the correct return code. > > Reported-by: Kieran Bingham > Fixes: 897e371389e77514 ("media: rcar-vin: simplify how formats are set and > reset" > Signed-

Re: [ANN] Topics for a media summit in Lyon in October

2019-09-13 Thread Daniel Gomez
Hi, On Fri, 13 Sep 2019 at 13:27, Hans Verkuil wrote: > > On 9/3/19 9:21 AM, Hans Verkuil wrote: > > Hi all, > > > > I've decided to hold the meeting during the ELCE and not on Thursday. > > Two key people can't be there on Thursday and some travel back Thursday > > evening and so would have to l

Re: [PATCH v4] vivid: Add metadata capture support

2019-09-13 Thread Hans Verkuil
Hi Vandana, Some checkpatch warnings: Applying: vivid: Add metadata capture support WARNING: line over 80 characters #43: FILE: drivers/media/platform/vivid/vivid-core.c:103: +/* Default: video + vbi-cap (raw and sliced) + radio rx + radio tx + sdr + vbi-out + vid-out + meta-cap */ This can eas

Re: [ANN] Topics for a media summit in Lyon in October

2019-09-13 Thread Hans Verkuil
On 9/3/19 9:21 AM, Hans Verkuil wrote: > Hi all, > > I've decided to hold the meeting during the ELCE and not on Thursday. > Two key people can't be there on Thursday and some travel back Thursday > evening and so would have to leave early. > > Looking at the schedule I propose instead to have on

Re: [V2, 2/2] media: i2c: Add more sensor modes for ov8856 camera sensor

2019-09-13 Thread Sakari Ailus
Hi Dongchun, On Tue, Sep 10, 2019 at 09:04:46PM +0800, dongchun@mediatek.com wrote: > From: Dongchun Zhu > > This patch mainly adds two more sensor modes for OV8856 CMOS image sensor. > That is, the resolution of 1632*1224 and 3264*2448, corresponding to the > bayer order of BGGR. > The sen