Re: [PATCH] media: ov9650: support VIDIOC_DBG_G/S_REGISTER ioctls

Hi Hans, On Tue, Dec 19, 2017 at 04:50:19PM +0100, Hans Verkuil wrote: > On 19/12/17 16:39, Akinobu Mita wrote: > > Hi Sakari, > > > > 2017-12-19 19:35 GMT+09:00 Sakari Ailus : > >> Hi Akinobu, > >> > >> On Thu, Dec 14, 2017 at 01:00:49AM +0900, Akinobu Mita wrote: > >>>

Re: [PATCH] devicetree: Add video bus switch

On Wed, Dec 20, 2017 at 07:54:12PM +0200, Laurent Pinchart wrote: > Hi Pavel, > > On Saturday, 4 February 2017 23:56:10 EET Pavel Machek wrote: > > Hi! > > > > +Required properties > > +=== > > + > > +compatible: must contain "video-bus-switch" > > >>> >

Re: [PATCH v1] media: videobuf2: Add new uAPI for DVB streaming I/O

Em Tue, 19 Dec 2017 09:05:53 +0530 Satendra Singh Thakur escreveu: > -Ported below mentioned patch to latest kernel: > commit ace52288edf0 ("Merge tag 'for-linus-20171218' of > git://git.infradead.org/linux-mtd") > > -Fixed few bugs in the patch, enhanced it and added

Re: [PATCH v4 16/18] scripts: kernel-doc: improve nested logic to handle multiple identifiers

Hi Mauro, I love your patch! Perhaps something to improve: [auto build test WARNING on lwn/docs-next] [also build test WARNING on v4.15-rc4 next-20171221] [cannot apply to linus/master] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url

[PATCH V2 1/3] media: dvb-core: Store device structure in dvb_register_device

From: Jasmin Jessich The device created by device_create in dvb_register_device was not available for DVB device drivers. Added "struct device *dev" to "struct dvb_device" and store the created device. Signed-off-by: Jasmin Jessich Acked-by: Ralph Metzler

[PATCH V2 2/3] media: dvb-core: Added timers for dvb_ca_en50221_write_data

From: Jasmin Jessich Some (older) CAMs are really slow in accepting data. The CI interface specification doesn't define a handshake for accepted data. Thus, the en50221 protocol driver can't control if a data byte has been correctly written to the CAM. The current implementation

[PATCH V2 0/3] Add timers to en50221 protocol driver

From: Jasmin Jessich Some (older) CAMs are really slow in accepting data. I got sometimes the already known error "CAM tried to send a buffer larger than the ecount size". I could track it down to the dvb_ca_en50221_write_data function not waiting between sending the data

[PATCH V2 3/3] media: dvb-core: Added documentation for ca sysfs timer nodes

From: Jasmin Jessich Added the documentation for the new ca? sysfs nodes in /sys/class/dvb/dvb?/ca?/tim_wr_. Signed-off-by: Jasmin Jessich --- Documentation/ABI/testing/sysfs-class-ca| 63 ++

[PATCH 1/1] v4l: fwnode: Use fwnode_graph_for_each_endpoint

Use fwnode_graph_for_each_endpoint iterator for better readability. Signed-off-by: Sakari Ailus --- drivers/media/v4l2-core/v4l2-fwnode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c

Re: [BUG] atomisp_ov2680 not initializing correctly

On Tue, Dec 19, 2017 at 10:37:01PM +0200, Andy Shevchenko wrote: > On Tue, 2017-12-19 at 14:00 +0200, Sakari Ailus wrote: > > Cc Alan and Andy. > > > > On Sat, Dec 16, 2017 at 04:50:04PM +0100, Kristian Beilke wrote: > > > Dear all, > > > > > > I am trying to get the cameras in a Lenovo IdeaPad

Re: [PATCH 1/2] [media] Add Rockchip RK1608 driver

Hi leo, 2017-12-12 14:28 GMT+08:00 Leo Wen : > Rk1608 is used as a PreISP to link on Soc, which mainly has two functions. > One is to download the firmware of RK1608, and the other is to match the > extra sensor such as camera and enable sensor by calling sensor's

cron job: media_tree daily build: ERRORS

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 Dec 22 05:00:15 CET 2017 media-tree git hash:ae49432810c5cca2143afc1445edad6582c9f270 media_build

Re: [PATCH v1 05/10] staging: atomisp: Remove non-ACPI leftovers

Hi Andy and Dan, On Wed, Dec 20, 2017 at 12:24:36PM +0200, Andy Shevchenko wrote: > On Wed, Dec 20, 2017 at 6:54 AM, Dan Carpenter > wrote: > > On Tue, Dec 19, 2017 at 10:59:52PM +0200, Andy Shevchenko wrote: > >> @@ -1147,10 +1145,8 @@ static int gc2235_probe(struct

Re: [PATCH v4 00/18] kernel-doc: add supported to document nested structs

Em Thu, 21 Dec 2017 14:08:43 -0700 Jonathan Corbet escreveu: > On Mon, 18 Dec 2017 10:30:01 -0200 > Mauro Carvalho Chehab wrote: > > > This is a rebased version of my patch series that add support for > > nested structs on kernel-doc. With this

[GIT PULL v2 for 4.16] An ordinary pile of atomisp cleanups and fixes

Hi Mauro, Here's the regular pile of atomisp cleanups and some fixes, too. since v1: - Add Andy's cleanups and fixes. Please pull. The following changes since commit ae49432810c5cca2143afc1445edad6582c9f270: media: ddbridge: improve ddb_ports_attach() failure handling (2017-12-19

Re: [BUG] atomisp_ov2680 not initializing correctly

On Thu, 2017-12-21 at 13:54 +0100, Kristian Beilke wrote: > On Tue, Dec 19, 2017 at 10:37:01PM +0200, Andy Shevchenko wrote: > > On Tue, 2017-12-19 at 14:00 +0200, Sakari Ailus wrote: > > > Cc Alan and Andy. > > > > > > On Sat, Dec 16, 2017 at 04:50:04PM +0100, Kristian Beilke wrote: > > > > Dear

Re: [PATCH v4 00/18] kernel-doc: add supported to document nested structs

On Mon, 18 Dec 2017 10:30:01 -0200 Mauro Carvalho Chehab wrote: > This is a rebased version of my patch series that add support for > nested structs on kernel-doc. With this version, it won't produce anymore > hundreds of identical warnings, as patch 17 removes the

[PATCH 02/11] media: videobuf2: Add new uAPI for DVB streaming I/O

From: Satendra Singh Thakur Adds a new uAPI for DVB to use streaming I/O which is implemented based on videobuf2, using those new ioctls: - DMX_REQBUFS: Request kernel to allocate buffers which count and size are dedicated by user. - DMX_QUERYBUF: Get

[PATCH 01/11] media: vb2-core: Fix a bug about unnecessary calls to queue cancel and free

From: Satendra Singh Thakur When the func vb2_core_reqbufs is called first time after vb2_core_queue_init(), the condition q->memory != memory always gets satisfied, since q->memory was set to 0 at vb2_core_queue_init(). After the condition is true, unnecessary calls to

[PATCH 05/11] media: dvb_vb2: fix a warning about streamoff logic

The streamoff logic is causing those warnings: WARNING: CPU: 3 PID: 3382 at drivers/media/v4l2-core/videobuf2-core.c:1652 __vb2_queue_cancel+0x177/0x250 [videobuf2_core] Modules linked in: bnep fuse xt_CHECKSUM iptable_mangle tun ebtable_filter ebtables ip6table_filter ip6_tables xt_physdev

[PATCH 04/11] media: vb2-core: add a new warning about pending buffers

There's a logic at the VB2 core that produces a WARN_ON if there are still buffers waiting to be filled. However, it doesn't indicate what buffers are still opened, with makes harder to identify issues inside caller drivers. So, add a new pr_warn() pointing to such buffers. That, together with

[PATCH 08/11] media: dvb uAPI docs: document mmap-related ioctls

5 new ioctls were added to the DVB demux API, in order to handle memory maped I/O. Add documentation for them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dmx-expbuf.rst | 90 +++

[PATCH 03/11] media: vb2-core: add pr_fmt() macro

Simplify the pr_foo() macros by adding a pr_fmt() macro. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/v4l2-core/videobuf2-core.c | 30 -- 1 file changed, 16 insertions(+), 14 deletions(-) diff --git

[PATCH 07/11] media: dvb uAPI docs: document demux mmap/munmap syscalls

With the new dmx mmap interface, those two syscalls are now handled by the subsystem. Document them. This patch is based on the V4L2 text for those ioctls. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dmx-mmap.rst | 116

[PATCH 10/11] fs: compat_ioctl: add new DVB demux ioctls

Use trivial handling for the new DVB demux ioctls, as none of them passes a pointer inside their structures. Signed-off-by: Mauro Carvalho Chehab --- fs/compat_ioctl.c | 5 + 1 file changed, 5 insertions(+) diff --git a/fs/compat_ioctl.c b/fs/compat_ioctl.c index

[PATCH 09/11] media: dvb-core: get rid of mmap reserved field

The "reserved" field was a way, used at V4L2 API, to add new data to existing structs without breaking userspace. However, there are now clever ways of doing that, without needing to add an uneeded overhead. So, get rid of them. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 06/11] media: move videobuf2 to drivers/media/common

Now that VB2 is used by both V4L2 and DVB core, move it to the common part of the subsystem. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/common/Kconfig | 1 + drivers/media/common/Makefile | 2 +-

[PATCH 11/11] media: dvb_vb2: add SPDX headers

This code is released under GPL. Add the corresponding SPDX headers. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-core/dvb_vb2.c | 1 + drivers/media/dvb-core/dvb_vb2.h | 2 ++ 2 files changed, 3 insertions(+) diff --git

Re: [PATCH v1 03/10] v4l: platform: Add Renesas CEU driver

Hi Laurent, On Mon, Dec 18, 2017 at 05:28:43PM +0200, Laurent Pinchart wrote: > Hi Jacopo, > > On Monday, 18 December 2017 14:25:12 EET jacopo mondi wrote: > > On Mon, Dec 11, 2017 at 06:15:23PM +0200, Laurent Pinchart wrote: [snip] > > >> +/** > > >> + * ceu_soft_reset() - Software reset the CEU

Re: [PATCH v6 2/6] [media] v4l: add 'unordered' flag to format description ioctl

Em Mon, 11 Dec 2017 16:27:37 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > For explicit synchronization it important for userspace to know if the > format being used by the driver can deliver the buffers back to userspace > in

Re: [PATCH v6 4/6] [media] vb2: add in-fence support to QBUF

Em Mon, 11 Dec 2017 16:27:39 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Receive in-fence from userspace and add support for waiting on them > before queueing the buffer to the driver. Buffers can't be queued to the > driver

Re: [PATCH v6 0/6] V4L2 Explicit Synchronization

Em Mon, 11 Dec 2017 16:27:35 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Hi, > > One more iteration of the explicit fences patches, please refer > to the previous version[1] for more details about the general > mechanism > >

Re: [PATCH v6 3/6] [media] vb2: add explicit fence user API

Em Mon, 11 Dec 2017 16:27:38 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Turn the reserved2 field into fence_fd that we will use to send > an in-fence to the kernel and return an out-fence from the kernel to > userspace. > >

Re: [PATCH v6 1/6] [media] vb2: add is_unordered callback for drivers

Em Mon, 11 Dec 2017 16:27:36 -0200 Gustavo Padovan escreveu: > From: Gustavo Padovan > > Explicit synchronization benefits a lot from ordered queues, they fit > better in a pipeline with DRM for example so create a opt-in way for > drivers

Re: [PATCH] devicetree: Add video bus switch

Hi, On 20.12.2017 19:54, Laurent Pinchart wrote: Hi Pavel, On Saturday, 4 February 2017 23:56:10 EET Pavel Machek wrote: Hi! +Required properties +=== + +compatible : must contain "video-bus-switch" How generic is this? Should we have e.g. nokia,video-bus-switch? And

[PATCH 2/2] media: dvb-frontends/stv0910: report FEC 1/4 and 1/3 in get_frontend()

From: Daniel Scheller The first two missing FECs in the modcod2fec fe_code_rate table were 1/4 and 1/3. Add them as they're now defined by the API. Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/stv0910.c | 2 +- 1 file changed, 1

[PATCH 1/2] media: dvb_frontend: add FEC modes, S2X modulations and 64K transmission

From: Daniel Scheller Add 1/4 and 1/3 FEC ratios, 64/128/256-APSK S2X modulations and 64K transmission mode. Update relevant doc items aswell. Signed-off-by: Daniel Scheller --- Documentation/media/frontend.h.rst.exceptions | 6 ++

[PATCH 0/2] Add FEC rates, S2X modulations and 64K transmission

From: Daniel Scheller As the DVB API is bumped to 5.11 for the next cycle. dddvb brings a few additional FEC rates (1/4 and 1/3), 64/128/256APSK modulations (DVB-S2X) and the 64K transmission mode. These two rather trivial patches bring them to mainline, and puts these

Re: [PATCH v6 0/6] V4L2 Explicit Synchronization

2017-12-21 Mauro Carvalho Chehab : > Em Mon, 11 Dec 2017 16:27:35 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Hi, > > > > One more iteration of the explicit fences patches, please refer > >

Re: [PATCH v6 1/6] [media] vb2: add is_unordered callback for drivers

2017-12-21 Mauro Carvalho Chehab : > Em Mon, 11 Dec 2017 16:27:36 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Explicit synchronization benefits a lot from ordered queues, they fit > > better

Re: [PATCH v6 3/6] [media] vb2: add explicit fence user API

2017-12-21 Mauro Carvalho Chehab : > Em Mon, 11 Dec 2017 16:27:38 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Turn the reserved2 field into fence_fd that we will use to send > > an in-fence

Re: [PATCH v6 4/6] [media] vb2: add in-fence support to QBUF

2017-12-21 Mauro Carvalho Chehab : > Em Mon, 11 Dec 2017 16:27:39 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Receive in-fence from userspace and add support for waiting on them > > before

Re: [PATCH v4 1/3] media: atomisp: convert default struct values to use compound-literals with designated initializers.

On 2017-12-19, at 14:07:49 +0200, Sakari Ailus wrote: > On Sat, Dec 02, 2017 at 10:11:59PM +, Jeremy Sowden wrote: > > The CSS API uses a lot of nested anonymous structs defined in object > > macros to assign default values to its data-structures. These have > > been changed to use

Re: [PATCH v6 0/6] V4L2 Explicit Synchronization

Em Thu, 21 Dec 2017 16:49:31 -0200 Mauro Carvalho Chehab escreveu: > Em Mon, 11 Dec 2017 16:27:35 -0200 > Gustavo Padovan escreveu: > > > From: Gustavo Padovan > > > > Hi, > > > > One more iteration of the explicit