cron job: media_tree daily build: OK

2018-08-09 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: Fri Aug 10 05:00:09 CEST 2018 media-tree git hash:8179de98cd35242446f4c7a6ecdd73c6c41f0cfb media_build

want to edit

2018-08-09 Thread Tony
We just found that you need image editing. We are an image studio, we do all kinds of image editing such as for e-commerce photos, jewelry images and portrait mages. Our service includes cutting out and clipping path etc , we also do retouching. You may send us a photo, testing will be

Re: [PATCHv17 13/34] v4l2-ctrls: use ref in helper instead of ctrl

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:05 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > The next patch needs the reference to a control instead of the > control itself, so change struct v4l2_ctrl_helper accordingly. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- >

Re: [PATCHv17 12/34] v4l2-ctrls: alloc memory for p_req

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:04 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > To store request data the handler_new_ref() allocates memory > for it if needed. > > Signed-off-by: Hans Verkuil > --- > drivers/media/v4l2-core/v4l2-ctrls.c | 20 > 1 file changed, 16

Re: [PATCHv17 11/34] v4l2-ctrls: prepare internal structs for request API

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:03 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Embed and initialize a media_request_object in struct v4l2_ctrl_handler. > > Add a p_req field to struct v4l2_ctrl_ref that will store the > request value. > > Signed-off-by: Hans Verkuil > Signed-off-by:

Re: [PATCHv17 10/34] v4l2-ctrls: v4l2_ctrl_add_handler: add from_other_dev

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:02 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add a 'bool from_other_dev' argument: set to true if the two > handlers refer to different devices (e.g. it is true when > inheriting controls from a subdev into a main v4l2 bridge > driver). > > This will be

Re: [PATCHv17 09/34] videodev2.h: add request_fd field to v4l2_ext_controls

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:01 +0200 Hans Verkuil escreveu: > From: Alexandre Courbot > > If 'which' is V4L2_CTRL_WHICH_REQUEST_VAL, then the 'request_fd' field > can be used to specify a request for the G/S/TRY_EXT_CTRLS ioctls. > > Signed-off-by: Alexandre Courbot > Signed-off-by: Hans

Re: [PATCHv17 08/34] v4l2-dev: lock req_queue_mutex

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:45:00 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > We need to serialize streamon/off with queueing new requests. > These ioctls may trigger the cancellation of a streaming > operation, and that should not be mixed with queuing a new > request at the same time. >

Re: [PATCHv17 07/34] v4l2-device.h: add v4l2_device_supports_requests() helper

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:59 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add a simple helper function that tests if the driver supports > the request API. > > Signed-off-by: Hans Verkuil Reviewed-by: Mauro Carvalho Chehab > --- > include/media/v4l2-device.h | 11 +++ >

Re: [PATCHv17 06/34] media-request: add media_request_object_find

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:58 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add media_request_object_find to find a request object inside a > request based on ops and priv values. > > Objects of the same type (vb2 buffer, control handler) will have > the same ops value. And objects

Re: [PATCHv17 05/34] media-request: add media_request_get_by_fd

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:57 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add media_request_get_by_fd() to find a request based on the file > descriptor. > > The caller has to call media_request_put() for the returned > request since this function increments the refcount. > >

Re: [PATCHv17 04/34] media: doc: Add media-request.h header to documentation build

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:56 +0200 Hans Verkuil escreveu: > From: Sakari Ailus > > media-request.h has been recently added; add it to the documentation build > as well. Looks good. Please add my reviewed-by: Reviewed-by: Mauro Carvalho Chehab (from now on, if I find a patch without

Re: [PATCHv17 03/34] media-request: implement media requests

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:55 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Add initial media request support: > > 1) Add MEDIA_IOC_REQUEST_ALLOC ioctl support to media-device.c > 2) Add struct media_request to store request objects. > 3) Add struct media_request_object to represent a

Re: [PATCHv17 02/34] uapi/linux/media.h: add request API

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:54 +0200 Hans Verkuil escreveu: > From: Hans Verkuil > > Define the public request API. > > This adds the new MEDIA_IOC_REQUEST_ALLOC ioctl to allocate a request > and two ioctls that operate on a request in order to queue the > contents of the request to the driver

Re: [PATCHv17 01/34] Documentation: v4l: document request API

2018-08-09 Thread Mauro Carvalho Chehab
Em Sat, 4 Aug 2018 14:44:53 +0200 Hans Verkuil escreveu: > From: Alexandre Courbot > > Document the request API for V4L2 devices, and amend the documentation > of system calls influenced by it. It follows some comments. Most are nitpicks. There are just two ones that aren't: - a

Re: [PATCH 19/22] [media] tvp5150: add input source selection of_graph support

2018-08-09 Thread Marco Felsch
Hi Mauro, On 18-08-09 13:04, Mauro Carvalho Chehab wrote: > Em Thu, 9 Aug 2018 16:35:20 +0200 > Marco Felsch escreveu: > > > Hi Mauro, > > > > Thanks for your feedback. > > > > > > > + dev_dbg(sd->dev, "link setup '%s':%d->'%s':%d[%d]", > > > > > > + remote->entity->name,

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

2018-08-09 Thread Michael Ira Krufky
On Thu, Aug 9, 2018 at 11:19 AM Mauro Carvalho Chehab wrote: > > Em Fri, 3 Aug 2018 15:41:53 +0100 > Ben Hutchings escreveu: > > > 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

we found that

2018-08-09 Thread Tony
We just found that you need image editing. We are an image studio, we do all kinds of image editing such as for e-commerce photos, jewelry images and portrait mages. Our service includes cutting out and clipping path etc , we also do retouching. You may send us a photo, testing will be

Re: [PATCH 19/22] [media] tvp5150: add input source selection of_graph support

2018-08-09 Thread Mauro Carvalho Chehab
Em Thu, 9 Aug 2018 16:35:20 +0200 Marco Felsch escreveu: > Hi Mauro, > > Thanks for your feedback. > > > > > + dev_dbg(sd->dev, "link setup '%s':%d->'%s':%d[%d]", > > > > > + remote->entity->name, remote->index, > > > > > local->entity->name, > > > > > +

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

2018-08-09 Thread Mauro Carvalho Chehab
Em Thu, 9 Aug 2018 12:19:20 -0300 Mauro Carvalho Chehab escreveu: > Em Fri, 3 Aug 2018 15:41:53 +0100 > Ben Hutchings escreveu: > > > The GNU Free Documentation License allows for a work to specify > > Invariant Sections that are not allowed to be modified. (Debian > > considers that this

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

2018-08-09 Thread Mauro Carvalho Chehab
Em Fri, 3 Aug 2018 15:41:53 +0100 Ben Hutchings escreveu: > 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

in our studio

2018-08-09 Thread Tony
We just found that you need image editing. We are an image studio, we do all kinds of image editing such as for e-commerce photos, jewelry images and portrait mages. Our service includes cutting out and clipping path etc , we also do retouching. You may send us a photo, testing will be

Re: [PATCH 19/22] [media] tvp5150: add input source selection of_graph support

2018-08-09 Thread Marco Felsch
Hi Mauro, Thanks for your feedback. On 18-08-09 10:36, Mauro Carvalho Chehab wrote: > Em Thu, 9 Aug 2018 14:55:07 +0200 > Marco Felsch escreveu: > > > Hi Mauro, > > > > On 18-08-08 15:52, Mauro Carvalho Chehab wrote: > > > Em Wed, 8 Aug 2018 17:29:49 +0200 > > > Marco Felsch escreveu: > > >

Re: [PATCH 06/22] [media] tvp5150: add FORMAT_TRY support for get/set selection handlers

2018-08-09 Thread Marco Felsch
Hi Mauro, during my work for a v2 series, I prepared two patches which fix this in a common way. The first will fix the compiler break and the second will fix a missing dependency, since those helper functions require the media_entity which is only available if MEDIA_CONTROLLER is enabled. I

Re: [PATCH 19/22] [media] tvp5150: add input source selection of_graph support

2018-08-09 Thread Mauro Carvalho Chehab
Em Thu, 9 Aug 2018 14:55:07 +0200 Marco Felsch escreveu: > Hi Mauro, > > On 18-08-08 15:52, Mauro Carvalho Chehab wrote: > > Em Wed, 8 Aug 2018 17:29:49 +0200 > > Marco Felsch escreveu: > > > > > Hi Mauro, > > > > > > the discussion about the correct binding was spread around some patches.

Re: [PATCH 19/22] [media] tvp5150: add input source selection of_graph support

2018-08-09 Thread Marco Felsch
Hi Mauro, On 18-08-08 15:52, Mauro Carvalho Chehab wrote: > Em Wed, 8 Aug 2018 17:29:49 +0200 > Marco Felsch escreveu: > > > Hi Mauro, > > > > the discussion about the correct binding was spread around some patches. > > So I will try use the correct thread for that theme. I did my rework for >