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

Ping! I really like to get this in for 4.16 so I can move forward with hooking this up for nouveau/amd. Regards, Hans On 11/20/2017 12:42 PM, Hans Verkuil wrote: > This patch series adds support for the DisplayPort CEC-Tunneling-over-AUX > feature. This patch series is based on the

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 1 05:00:17 CET 2017 media-tree git hash:781b045baefdabf7e0bc9f33672ca830d3db9f27 media_build

Re: [PATCH v8 28/28] rcar-vin: enable support for r8a77970

On Wed, Nov 29, 2017 at 08:43:42PM +0100, Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a77970. > > Signed-off-by: Niklas Söderlund > --- > .../devicetree/bindings/media/rcar_vin.txt | 1 + Acked-by: Rob Herring

Re: [PATCH v8 27/28] rcar-vin: enable support for r8a7796

On Wed, Nov 29, 2017 at 08:43:41PM +0100, Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a7796. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Hans Verkuil > --- >

[PATCH 1/1] media: uvcvideo: Add quirk to support light switch on Dino-Lite cameras

The Dino-Lite cameras are equipped with LED lights that can be switched on and off by setting a proprietary control. For this control, the camera reports a length of 1 byte, but actually the value set by the original Windows driver is 3 byte long. This makes it impossible to toggle the camera

[PATCH 0/1] Add quirk to support light switch on some cameras

The Dino-Lite cameras are equipped with LED lights that can be switched on and off by setting a proprietary control. For this control, the camera reports a length of 1 byte, but actually the value set by the original Windows driver is 3 byte long. This makes it impossible to toggle the camera

[PATCH 2/4] dma-buf: make reservation_object_copy_fences rcu save

From: Christian König Stop requiring that the src reservation object is locked for this operation. commit 39e16ba16c147e662bf9fbcee9a99d70d420382f upstream Acked-by: Chunming Zhou Signed-off-by: Christian König

[PATCH 0/4] Backported amdgpu ttm deadlock fixes for 4.14

I haven't gone to see where it started, but as of late a good number of pretty nasty deadlock issues have appeared with the kernel. Easy reproduction recipe on a laptop with i915/amdgpu prime with lockdep enabled: DRI_PRIME=1 glxinfo Additionally, some more race conditions exist that I've

Re: [BUG] ir-ctl: error sending file with multiple scancodes

Hi Matthias, On Thu, Nov 30, 2017 at 04:34:33PM +0100, Matthias Reichl wrote: > Hi Sean, > > On Wed, Nov 29, 2017 at 08:05:21PM +, Sean Young wrote: > > On Wed, Nov 29, 2017 at 03:44:00PM +0100, Matthias Reichl wrote: > > > The goal I'm trying to achieve is to send a repeated signal with

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

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 compound-literals and designated initializers to make them more comprehensible and less fragile. The compound-literals can also be used in

[PATCH 0/3] Clean up of data-structure initialization in the CSS API

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 compound-literals and designated initializers to make them more comprehensible and less fragile. The compound-literals can also be used in

[PATCH 3/3] media: atomisp: delete empty default struct values.

Removing zero-valued struct-members left a number of the default struct-values empty. These values have now been removed. Signed-off-by: Jeremy Sowden --- .../atomisp/pci/atomisp2/css2400/ia_css_pipe.h | 1 - .../atomisp/pci/atomisp2/css2400/ia_css_types.h| 1 -

[PATCH 2/3] media: atomisp: delete zero-valued struct members.

A lot of the members of the default struct values used by the CSS API were explicitly initialized to zero values. Designated initializers have allowed these members to be removed. Signed-off-by: Jeremy Sowden --- .../hive_isp_css_common/input_formatter_global.h | 17

Re: DVB-S2 and S2X API extensions

2017-11-30 20:26 GMT+02:00 Mauro Carvalho Chehab : > Em Mon, 20 Nov 2017 02:05:35 +0100 > Ralph Metzler escreveu: > >> Hi, > > (C/C Athanasios, as he also rised concerns about S2X). >> >> Mauro Carvalho Chehab writes: >> > Em Thu, 9 Nov 2017

Re: [PATCH 4/7] media: davinci: fix kernel-doc warnings

HI Mauro, Thanks for the patch. On Wed, Nov 29, 2017 at 12:08 PM, Mauro Carvalho Chehab wrote: > There are several of kernel-doc warnings: > > drivers/media/platform/davinci/vpif_display.c:114: warning: No > description found for parameter 'sizes' >

Re: [PATCH 06/12] media: vpif: don't generate a kernel-doc warning on a constant

On Wed, Nov 29, 2017 at 10:46 AM, Mauro Carvalho Chehab wrote: > Constants documentation is not supported by kernel-doc markups. > So, change the comment label to avoid this warning: > drivers/media/platform/davinci/vpif.c:54: warning: cannot understand >

Unknown symbol errors with latest RH7 kernel using meda_build

I'm new to mailing lists, so I'm sorry if this message is poorly formatted. I have been compiling v4l on RH7 using very minor changes (made it into an rpm, have to remove 'v3.16_wait_on_bit.patch' from the backports) for a while now. I had been using an older set of firmware and git hash from the

Re: [PATCH 07/22] media: s5k6aa: describe some function parameters

On Wed, Nov 29, 2017 at 02:08:25PM -0500, Mauro Carvalho Chehab wrote: > as warned: > drivers/media/i2c/s5k6aa.c:429: warning: No description found for parameter > 's5k6aa' > drivers/media/i2c/s5k6aa.c:679: warning: No description found for parameter > 's5k6aa' >

Re: [PATCH v2 3/4] media: ov5640: add support of DVP parallel interface

Hi Hugues, On Wed, Nov 29, 2017 at 3:11 PM, Hugues Fruchet wrote: > Add support of DVP parallel mode in addition of > existing MIPI CSI mode. The choice between two modes > and configuration is made through device tree. What about explaining how to select between the two

Re: [PATCH v2 2/4] media: ov5640: check chip id

Hi Hugues, On Wed, Nov 29, 2017 at 3:11 PM, Hugues Fruchet wrote: > /* read exposure, in number of line periods */ > static int ov5640_get_exposure(struct ov5640_dev *sensor) > { > @@ -1562,6 +1586,10 @@ static int ov5640_set_power(struct ov5640_dev *sensor, > bool

Re: [RFC v5 00/11] V4L2 Explicit Synchronization

Hi Smitha, 2017-11-20 Smitha T Murthy : > Hi Gustavo, > > I am currently referring to your implementation for explicit > synchronisation. For the same I needed your testapp, but I am unable > to download the same at the link provided >

Re: DVB-S2 and S2X API extensions

Em Mon, 20 Nov 2017 02:05:35 +0100 Ralph Metzler escreveu: > Hi, (C/C Athanasios, as he also rised concerns about S2X). > > Mauro Carvalho Chehab writes: > > Em Thu, 9 Nov 2017 16:28:18 +0100 > > Ralph Metzler escreveu: > > > > > Hi, > > > >

Re: DVB-S2X - S2 Extensions Support

Em Thu, 30 Nov 2017 15:24:47 +0200 Αθανάσιος Οικονόμου escreveu: > Hi All, > > I am not aware if there was a discussion regarding S2X before, so I > would like to hear what are your plans regarding S2X. Ralph started a discussion about that sometime ago. > There are

Re: [PATCH v8 28/28] rcar-vin: enable support for r8a77970

Hi Niklas, On 29/11/17 19:43, Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a77970. > > Signed-off-by: Niklas Söderlund Not going through the details on this one, as I don't know where to start yet other than the cursory chip,

Re: [PATCH v8 03/28] rcar-vin: unregister video device on driver removal

Hi Niklas, Thankyou for the patch. Only the grammar police here again :) On 29/11/17 19:43, Niklas Söderlund wrote: > If the video device was registered by the complete() callback it should > be unregistered when the driver is removed. Protect from printing a /printing a/printing an/ >

Re: [PATCH v8 02/28] rcar-vin: rename poorly named initialize and cleanup functions

Hi Niklas, If relevant I believe you could have a Tested-by: tag from me on this series now that I have capture working on the Eagle-V3M. I'll let you decide on that though. On 29/11/17 19:43, Niklas Söderlund wrote: > The functions to initialize and cleanup the hardware and video device >

[PATCH v2] dvb-frontends: fix i2c access helpers for KASAN

A typical code fragment was copied across many dvb-frontend drivers and causes large stack frames when built with with CONFIG_KASAN on gcc-5/6/7: drivers/media/dvb-frontends/cxd2841er.c:3225:1: error: the frame size of 3992 bytes is larger than 3072 bytes [-Werror=frame-larger-than=]

Re: [PATCH v8 23/28] rcar-vin: parse Gen3 OF and setup media graph

Hi, There is one error in this patch. A left over from moving the video device registration from probe() to the async callbacks, see bellow. I will await further feedback before resending but will include this fix in the next version. On 2017-11-29 20:43:37 +0100, Niklas Söderlund wrote: >

Re: [BUG] ir-ctl: error sending file with multiple scancodes

Hi Sean, On Wed, Nov 29, 2017 at 08:05:21PM +, Sean Young wrote: > On Wed, Nov 29, 2017 at 03:44:00PM +0100, Matthias Reichl wrote: > > The goal I'm trying to achieve is to send a repeated signal with ir-ctl > > (a user reported his sony receiver needs this to actually power up). > > That's

Re: [linux-sunxi] Cedrus driver

Hi Thomas, On Wed, Nov 29, 2017 at 04:36:01PM +0100, Thomas van Kleef wrote: > > C) I'm not sure what you tried to do with the application of the > >request API patches (such as e1ca861c168f) but we want to have the > >whole commits in there, and not a patch adding all of them. This > >

Re: [PATCH 2/2] media: rc-core.rst: add the lirc_dev.h header

Em Thu, 30 Nov 2017 08:20:56 -0500 Mauro Carvalho Chehab escreveu: > There is a kAPI declaration there. Add it to the documentation. > > Signed-off-by: Mauro Carvalho Chehab Please ignore this one. Was sent by mistake. Thanks, Mauro

Re: [PATCH, RESEND 1/2] dvb-frontends: fix i2c access helpers for KASAN

Em Thu, 30 Nov 2017 15:06:15 +0100 Arnd Bergmann escreveu: > On Thu, Nov 30, 2017 at 1:49 PM, Mauro Carvalho Chehab > wrote: > >> Link: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81715 > >> Signed-off-by: Arnd Bergmann > >> --- > >> I'm

Re: [PATCH, RESEND 1/2] dvb-frontends: fix i2c access helpers for KASAN

On Thu, Nov 30, 2017 at 1:49 PM, Mauro Carvalho Chehab wrote: >> Link: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81715 >> Signed-off-by: Arnd Bergmann >> --- >> I'm undecided here whether there should be a comment pointing >> to PR81715 for each file that

[PATCH] kernel-doc: parse DECLARE_KFIFO_PTR()

On media, we now have an struct declared with: struct lirc_fh { struct list_head list; struct rc_dev *rc; int carrier_low; boolsend_timeout_reports; DECLARE_KFIFO_PTR(rawir, unsigned int);

DVB-S2X - S2 Extensions Support

Hi All, I am not aware if there was a discussion regarding S2X before, so I would like to hear what are your plans regarding S2X. There are already several manufacturers with S2X capable receivers and soon more will follow. The S2 extensions will bring more code rates and modulations, so I

[PATCH 1/2] media: RC docs: add enum rc_proto description at the docs

This is part of the uAPI. Add it to the documentation again, and fix cross-references. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/rc/lirc-dev-intro.rst | 19 +-- Documentation/media/uapi/rc/lirc-read.rst | 2 +-

[PATCH 2/2] media: rc-core.rst: add the lirc_dev.h header

There is a kAPI declaration there. Add it to the documentation. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/kapi/rc-core.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/media/kapi/rc-core.rst

Re: [PATCH v3 26/26] kfifo: DECLARE_KIFO_PTR(fifo, u64) does not work on arm 32 bit

On Thu, 2017-11-30 at 10:29 -0200, Mauro Carvalho Chehab wrote: > Em Tue, 10 Oct 2017 09:59:42 +0200 > Sean Young escreveu: > > > If you try to store u64 in a kfifo (or a struct with u64 members), > > then the buf member of __STRUCT_KFIFO_PTR will cause 4 bytes > > padding due to

Re: [PATCH, RESEND 1/2] dvb-frontends: fix i2c access helpers for KASAN

Hi Arnd, Em Thu, 30 Nov 2017 12:08:04 +0100 Arnd Bergmann escreveu: > A typical code fragment was copied across many dvb-frontend drivers and > causes large stack frames when built with with CONFIG_KASAN on gcc-5/6/7: > > drivers/media/dvb-frontends/cxd2841er.c:3225:1: error:

[GIT PULL for 4.16] Sensor driver patches, et8ek8 lens binding

Hi Mauro, Here are sensor driver patches, including error handling fixes, and DT binding documentation and DTS changes for Nokia N900. Please pull. The following changes since commit be9b53c83792e3898755dce90f8c632d40e7c83e: media: dvb-frontends: complete kernel-doc markups (2017-11-30

Re: [PATCH v3 26/26] kfifo: DECLARE_KIFO_PTR(fifo, u64) does not work on arm 32 bit

Em Tue, 10 Oct 2017 09:59:42 +0200 Sean Young escreveu: > If you try to store u64 in a kfifo (or a struct with u64 members), > then the buf member of __STRUCT_KFIFO_PTR will cause 4 bytes > padding due to alignment (note that struct __kfifo is 20 bytes > on 32 bit). > > That in

[GIT PULL for 4.16] Intel IPU3 CIO2 CSI-2 receiver driver

Hi Mauro, Here's the Intel IPU3 CIO2 CSI-2 receiver driver, with the accompanying format definitions. Please pull. The following changes since commit be9b53c83792e3898755dce90f8c632d40e7c83e: media: dvb-frontends: complete kernel-doc markups (2017-11-30 04:19:05 -0500) are available in the

[PATCH, RESEND 1/2] dvb-frontends: fix i2c access helpers for KASAN

A typical code fragment was copied across many dvb-frontend drivers and causes large stack frames when built with with CONFIG_KASAN on gcc-5/6/7: drivers/media/dvb-frontends/cxd2841er.c:3225:1: error: the frame size of 3992 bytes is larger than 3072 bytes [-Werror=frame-larger-than=]

[PATCH, RESEND 2/2] r820t: fix r820t_write_reg for KASAN

With CONFIG_KASAN, we get an overly long stack frame due to inlining the register access functions: drivers/media/tuners/r820t.c: In function 'generic_set_freq.isra.7': drivers/media/tuners/r820t.c:1334:1: error: the frame size of 2880 bytes is larger than 2048 bytes [-Werror=frame-larger-than=]

Re: [bug report] media: dvb_frontend: cleanup ioctl handling logic

Hello Is anyone working on adding compact_ioctl? So far it is impossible to use linux-media from 32bit userland on 64bit kernels Bye 2017-11-30 10:58 GMT+01:00 Dan Carpenter : > Hello Mauro Carvalho Chehab, > > The patch d73dcf0cdb95: "media: dvb_frontend: cleanup ioctl

Re: [PATCH 12/22] media: s3c-camif: add missing description at s3c_camif_find_format()

On 11/29/2017 08:08 PM, Mauro Carvalho Chehab wrote: > Fix this warning: > drivers/media/platform/s3c-camif/camif-core.c:112: warning: No > description found for parameter 'vp' > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Sylwester Nawrocki

[bug report] media: dvb_frontend: cleanup ioctl handling logic

Hello Mauro Carvalho Chehab, The patch d73dcf0cdb95: "media: dvb_frontend: cleanup ioctl handling logic" from Sep 18, 2017, leads to the following static checker warning: drivers/media/dvb-core/dvb_frontend.c:2469 dvb_frontend_handle_ioctl() error: uninitialized symbol 'err'.

Re: [PATCH 16/22] media: vsp1: add a missing kernel-doc parameter

Hi Mauro, Thanks for the patch. On 29/11/17 19:08, Mauro Carvalho Chehab wrote: > Fix this warning: > drivers/media/platform/vsp1/vsp1_dl.c:87: warning: No description found > for parameter 'has_chain' > > Signed-off-by: Mauro Carvalho Chehab Ah yes, I missed

Re: [PATCH 19/22] media: drivers: remove "/**" from non-kernel-doc comments

Hi Mauro, Thanks for the patch. On 29/11/17 19:08, Mauro Carvalho Chehab wrote: > Several comments are wrongly tagged as kernel-doc, causing > those warnings: > > drivers/media/rc/st_rc.c:98: warning: No description found for parameter > 'irq' > drivers/media/rc/st_rc.c:98: warning: No