Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-12-04 Thread Hans Verkuil
On 12/05/2017 04:22 AM, Tomasz Figa wrote: > Hi Raj, > > On Tue, Dec 5, 2017 at 9:13 AM, Mani, Rajmohan > wrote: >> Hi Hans, >> >> Thanks for your patience and sharing your thoughts on this. >> >>> Subject: Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset >>> >>> Hi Rajmohan, >>> >>> On 11/17/2017

Re: Donation

2017-12-04 Thread Mayrhofer Family
Good Day, My wife and I have awarded you with a donation of $ 1,000,000.00 Dollars from part of our Jackpot Lottery of 50 Million Dollars, respond with your details for claims to our private email. We await your earliest response and God Bless you. Best Regards, Friedrich And Annand Mayrhofer.

cron job: media_tree daily build: ERRORS

2017-12-04 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: Tue Dec 5 05:00:23 CET 2017 media-tree git hash:781b045baefdabf7e0bc9f33672ca830d3db9f27 media_build git

Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-12-04 Thread Tomasz Figa
Hi Raj, On Tue, Dec 5, 2017 at 9:13 AM, Mani, Rajmohan wrote: > Hi Hans, > > Thanks for your patience and sharing your thoughts on this. > >> Subject: Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset >> >> Hi Rajmohan, >> >> On 11/17/2017 03:58 AM, Mani, Rajmohan wrote: >> > Hi Sakari and all, >> >

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-04 Thread Laurent Pinchart
Hi Guennadi, On Tuesday, 5 December 2017 02:24:30 EET Laurent Pinchart wrote: > On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: [snip] > > +static void uvc_video_decode_meta(struct uvc_streaming *stream, > > + struct uvc_buffer *buf, struct uvc_buffer *m

Re: [PATCH 2/8] [media] uvc_video: use ktime_t for timestamps

2017-12-04 Thread Laurent Pinchart
Hi Arnd, On Tuesday, 5 December 2017 02:37:27 EET Laurent Pinchart wrote: > On Monday, 27 November 2017 15:19:54 EET Arnd Bergmann wrote: > > uvc_video_get_ts() returns a 'struct timespec', but all its users > > really want a nanoseconds variable anyway. > > > > Changing the deprecated ktime_get_

Re: [PATCH 1/8] [media] uvc_video: use ktime_t for stats

2017-12-04 Thread Laurent Pinchart
Hi Arnd, Thank you for the patch. On Monday, 27 November 2017 15:19:53 EET Arnd Bergmann wrote: > 'struct timespec' works fine here, but we try to migrate > away from it in favor of ktime_t or timespec64. In this > case, using ktime_t produces the simplest code. > > Signed-off-by: Arnd Bergmann

Re: [PATCH 5/8] [media] omap3isp: support 64-bit version of omap3isp_stat_data

2017-12-04 Thread Laurent Pinchart
Hi Arnd, Thank you for the patch. I'll try to review this without too much delay. In the meantime, I'm CC'ing Sakari Ailus who might be faster than me :-) On Monday, 27 November 2017 15:19:57 EET Arnd Bergmann wrote: > C libraries with 64-bit time_t use an incompatible format for > struct omap3

Re: [PATCH 2/8] [media] uvc_video: use ktime_t for timestamps

2017-12-04 Thread Laurent Pinchart
Hi Arnd, Thank you for the patch. On Monday, 27 November 2017 15:19:54 EET Arnd Bergmann wrote: > uvc_video_get_ts() returns a 'struct timespec', but all its users > really want a nanoseconds variable anyway. > > Changing the deprecated ktime_get_ts/ktime_get_real_ts to ktime_get > and ktime_get

Re: [PATCH 3/3 v7] uvcvideo: add a metadata device node

2017-12-04 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. We're getting very close, I only have small comments, please see below. On Wednesday, 8 November 2017 18:00:14 EET Guennadi Liakhovetski wrote: > Some UVC video cameras contain metadata in their payload headers. This > patch extracts that data, adding more c

RE: [PATCH v4 00/12] Intel IPU3 ImgU patchset

2017-12-04 Thread Mani, Rajmohan
Hi Hans, Thanks for your patience and sharing your thoughts on this. > Subject: Re: [PATCH v4 00/12] Intel IPU3 ImgU patchset > > Hi Rajmohan, > > On 11/17/2017 03:58 AM, Mani, Rajmohan wrote: > > Hi Sakari and all, > > > >> -Original Message- > >> From: Zhi, Yong > >> Sent: Tuesday, Oc

[PATCH 2/2] uvcvideo: Report V4L2 device caps through the video_device structure

2017-12-04 Thread Laurent Pinchart
The V4L2 core populates the struct v4l2_capability device_caps field from the same field in video_device. There's no need to handle that manually in the driver. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_driver.c | 11 +++ drivers/media/usb/uvc/uvc_v4l2.c | 4

[PATCH 0/2] uvcvideo: Refactor code to ease metadata implementation

2017-12-04 Thread Laurent Pinchart
Hi Guennadi, This small patch series refactors the uvc_video_register() function to extract the code that you need into a new uvc_video_register_device() function. Please let me know if it can help. Laurent Pinchart (2): uvcvideo: Factor out video device registration to a function uvcvideo: R

[PATCH 1/2] uvcvideo: Factor out video device registration to a function

2017-12-04 Thread Laurent Pinchart
The function will then be used to register the video device for metadata capture. Signed-off-by: Laurent Pinchart --- drivers/media/usb/uvc/uvc_driver.c | 66 +++--- drivers/media/usb/uvc/uvcvideo.h | 8 + 2 files changed, 49 insertions(+), 25 deletions(-)

[GIT PULL FOR v4.16] uvcvideo

2017-12-04 Thread Laurent Pinchart
Hi Mauro, The following changes since commit 781b045baefdabf7e0bc9f33672ca830d3db9f27: media: imx274: Fix error handling, add MAINTAINERS entry (2017-11-30 04:45:12 -0500) are available in the git repository at: git://linuxtv.org/pinchartl/media.git uvc/next for you to fetch changes up to

Re: [PATCH v6 0/9] i2c: document DMA handling and add helpers for it

2017-12-04 Thread Mark Brown
On Sun, Dec 03, 2017 at 08:43:47PM +0100, Wolfram Sang wrote: > > It's a bit different in that it's much more likely that a SPI controller > > will actually do DMA than an I2C one since the speeds are higher and > > there's frequent applications that do large transfers so it's more > > likely that

[PATCH 1/1] v4l: fwnode: Add debug prints for V4L2 endpoint property parsing

2017-12-04 Thread Sakari Ailus
Print debug info as standard V4L2 endpoint are parsed. Signed-off-by: Sakari Ailus --- drivers/media/v4l2-core/v4l2-fwnode.c | 101 +++--- 1 file changed, 80 insertions(+), 21 deletions(-) diff --git a/drivers/media/v4l2-core/v4l2-fwnode.c b/drivers/media/v4l2-core/

Re: [PATCH v6 0/9] i2c: document DMA handling and add helpers for it

2017-12-04 Thread Wolfram Sang
On Sat, Nov 04, 2017 at 09:20:00PM +0100, Wolfram Sang wrote: > So, after revisiting old mail threads, taking part in a similar discussion on > the USB list, and implementing a not-convincing solution before, here is what > I > cooked up to document and ease DMA handling for I2C within Linux. Plea

Re: [PATCH v12 2/2] rcar-csi2: add Renesas R-Car MIPI CSI-2 receiver driver

2017-12-04 Thread Sakari Ailus
Hejssan!! On Sat, Dec 02, 2017 at 03:50:21PM +0100, Niklas Söderlund wrote: > Hej Sakari, > > Thanks for your feedback. > > On 2017-12-02 16:05:08 +0200, Sakari Ailus wrote: > > Hejssan, > > > > On Sat, Dec 02, 2017 at 12:08:21PM +0100, Niklas Söderlund wrote: > > ... > > > > > +static int rcar

[RFC 1/1] v4l: async: Use endpoint node, not device node, for fwnode match

2017-12-04 Thread Sakari Ailus
V4L2 async framework can use both device's fwnode and endpoints's fwnode for matching the async sub-device with the sub-device. In order to proceed moving towards endpoint matching assign the endpoint to the async sub-device. As most async sub-device drivers (and the related hardware) only support

Re: [PATCH v6 1/2] media: ov7740: Document device tree bindings

2017-12-04 Thread Rob Herring
On Mon, Dec 04, 2017 at 02:58:57PM +0800, Wenyou Yang wrote: > Add the device tree binding documentation for the ov7740 sensor driver. > > Signed-off-by: Wenyou Yang > --- > > Changes in v6: None > Changes in v5: None > Changes in v4: None > Changes in v3: > - Explicitly document the "remote-en

Re: [PATCH] v4l: vsp1: Fix function declaration/definition mismatch

2017-12-04 Thread Laurent Pinchart
Hi Eugeniu, On Monday, 4 December 2017 20:10:34 EET Eugeniu Rosca wrote: > On Mon, Dec 04, 2017 at 12:52:17PM +0200, Laurent Pinchart wrote: > > On Friday, 24 November 2017 20:40:57 EET Kieran Bingham wrote: > >> Hi Eugeniu, > >> > >> Thankyou for the patch, > >> > >> Laurent - Any comments on t

Re: [PATCH 3/3] media: dvb_frontend: Add commands implementation for compat ioct

2017-12-04 Thread kbuild test robot
Hi Jaedon, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.15-rc2 next-20171204] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

Re: [PATCH] v4l: vsp1: Fix function declaration/definition mismatch

2017-12-04 Thread Eugeniu Rosca
Hello Laurent, Kieran, On Mon, Dec 04, 2017 at 12:52:17PM +0200, Laurent Pinchart wrote: > Hello, > > On Friday, 24 November 2017 20:40:57 EET Kieran Bingham wrote: > > Hi Eugeniu, > > > > Thankyou for the patch, > > > > Laurent - Any comments on this? Otherwise I'll bundle this in with my > >

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-04 Thread Tim Harvey
On Mon, Dec 4, 2017 at 4:50 AM, Hans Verkuil wrote: > Hi Tim, > > Found a few more small issues. After that's fixed and you have the Ack for the > bindings this can be merged I think. Hans, Thanks. Can you weigh in on the bindings? Rob was hoping for some discussion on making some generic bus fo

Re: [PATCH 2/3] media: dvb_frontend: Add compat_ioctl callback

2017-12-04 Thread kbuild test robot
Hi Jaedon, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.15-rc2 next-20171204] [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

Re: [PATCH v4 3/5] staging: Introduce NVIDIA Tegra video decoder driver

2017-12-04 Thread Hans Verkuil
Hi Dmitry, As you already mention in the TODO, this should become a v4l2 codec driver. Good existing examples are the coda, qcom/venus and mtk-vcodec drivers. One thing that is not clear from this code is if the tegra hardware is a stateful or stateless codec, i.e. does it keep track of the deco

Re: [PATCH 3/3] media: dvb_frontend: Add commands implementation for compat ioct

2017-12-04 Thread Menion
Hello Sorry if I say something completely wrong here, I was thinking to implement the same on my own As far as I understand from the patch, you have added two "compact" specific ioctl commands As far as I know, the compact_ioctl is called automatically when the userland is 32 bit and kernel is 64 b

Re: [PATCH 0/9] media: imx: Add better OF graph support

2017-12-04 Thread Hans Verkuil
Hi Steve, On 10/28/2017 10:36 PM, Steve Longerbeam wrote: > This is a set of patches that improve support for more complex OF > graphs. Currently the imx-media driver only supports a single device > with a single port connected directly to either the CSI muxes or the > MIPI CSI-2 receiver input po

Re: [PATCH] v4l: rcar-vin: Implement V4L2 video node release handler

2017-12-04 Thread Hans Verkuil
On 12/04/2017 02:34 PM, Niklas Söderlund wrote: > Hi Hans, > > On 2017-12-04 10:05:35 +0100, Hans Verkuil wrote: >> Hi Niklas, >> >> On 11/16/2017 01:27 AM, Laurent Pinchart wrote: >>> Hi Niklas, >>> >>> On Thursday, 16 November 2017 01:34:33 EET Niklas Söderlund wrote: On 2017-11-16 00:49:07

Re: [PATCH] v4l: rcar-vin: Implement V4L2 video node release handler

2017-12-04 Thread Niklas Söderlund
Hi Hans, On 2017-12-04 10:05:35 +0100, Hans Verkuil wrote: > Hi Niklas, > > On 11/16/2017 01:27 AM, Laurent Pinchart wrote: > > Hi Niklas, > > > > On Thursday, 16 November 2017 01:34:33 EET Niklas Söderlund wrote: > >> On 2017-11-16 00:49:07 +0200, Laurent Pinchart wrote: > >>> The rvin_dev data

[PATCH for 4.15] omapdrm/dss/hdmi4_cec: fix interrupt handling

2017-12-04 Thread Hans Verkuil
The omap4 CEC hardware cannot tell a Nack from a Low Drive from an Arbitration Lost error, so just report a Nack, which is almost certainly the reason for the error anyway. This also simplifies the implementation. The only three interrupts that need to be enabled are: Transmit Buffer Full/Empty C

[GIT PULL FOR v4.16 v2] Various fixes all over the place

2017-12-04 Thread Hans Verkuil
This is the same as the previous version of this pull request (see https://patchwork.linuxtv.org/patch/45733/), except that the patch "media: staging/imx: do not return error in link_notify for unknown sources" from Steve Longerbeam has been dropped. When going through newer imx patches I noticed

Re: [PATCH 3/3] media: dvb_frontend: Add commands implementation for compat ioct

2017-12-04 Thread kbuild test robot
Hi Jaedon, Thank you for the patch! Yet something to improve: [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.15-rc2 next-20171204] [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

Re: [PATCH v4 3/5] media: i2c: Add TDA1997x HDMI receiver driver

2017-12-04 Thread Hans Verkuil
Hi Tim, Found a few more small issues. After that's fixed and you have the Ack for the bindings this can be merged I think. On 11/29/2017 10:19 PM, Tim Harvey wrote: > Add support for the TDA1997x HDMI receivers. > > Cc: Hans Verkuil > Signed-off-by: Tim Harvey > --- > v4: > - move include/dt

[GIT PULL FOR v4.16] Various fixes

2017-12-04 Thread Hans Verkuil
The usual set of fixes, cleanups and small enhancements. Regards, Hans The following changes since commit 781b045baefdabf7e0bc9f33672ca830d3db9f27: media: imx274: Fix error handling, add MAINTAINERS entry (2017-11-30 04:45:12 -0500) are available in the git repository at: git://l

Re: [PATCH] Support HVR-1200 analog video as a clone of HVR-1500. Tested, composite and s-video inputs.

2017-12-04 Thread Hans Verkuil
Hi Nigel, Can you repost this as a proper patch? It doesn't apply (issues with tabs and whitespace: please use tabs!), and I am missing a "Signed-off-by" line (see https://elinux.org/Developer_Certificate_Of_Origin). Thanks! Hans On 09/19/2017 09:16 PM, Nigel Kettlewell wrote: > [adding

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

2017-12-04 Thread Greg KH
On Thu, Nov 30, 2017 at 07:23:02PM -0500, Lyude Paul wrote: > 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 gl

Re: [PATCH 5/9] v4l: vsp1: Document the vsp1_du_atomic_config structure

2017-12-04 Thread Laurent Pinchart
Hi Sergei, On Monday, 4 December 2017 11:31:49 EET Sergei Shtylyov wrote: > On 12/3/2017 1:57 PM, Laurent Pinchart wrote: > > The structure is used in the API that the VSP1 driver exposes to the DU > > driver. Documenting it is thus important. > > > > Signed-off-by: Laurent Pinchart > > > > ---

[PATCH v3] media: vsp1: Prevent suspending and resuming DRM pipelines

2017-12-04 Thread Kieran Bingham
When used as part of a display pipeline, the VSP is stopped and restarted explicitly by the DU from its suspend and resume handlers. There is thus no need to stop or restart pipelines in the VSP suspend and resume handlers, and doing so would cause the hardware to be left in a misconfigured state.

Re: [PATCH] v4l: vsp1: Fix function declaration/definition mismatch

2017-12-04 Thread Laurent Pinchart
Hello, On Friday, 24 November 2017 20:40:57 EET Kieran Bingham wrote: > Hi Eugeniu, > > Thankyou for the patch, > > Laurent - Any comments on this? Otherwise I'll bundle this in with my > suspend/resume patch for a pull request. > > On 20/08/17 13:40, Eugeniu Rosca wrote: > > From: Eugeniu Rosc

Re: [GIT PULL] SAA716x DVB driver

2017-12-04 Thread Mauro Carvalho Chehab
Em Sun, 3 Dec 2017 15:04:31 + Jemma Denson escreveu: > On 03/12/17 14:11, Soeren Moch wrote: > > On 03.12.2017 11:57, Jemma Denson wrote: > >> On 02/12/17 23:59, Soeren Moch wrote: > > The whole purpose of driver development is bringing support for existing > > hardware to available user

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

2017-12-04 Thread Hans Verkuil
On 11/29/2017 08:43 PM, Niklas Söderlund wrote: > Parse the VIN Gen3 OF graph and register all CSI-2 devices in the VIN > group common media device. Once all CSI-2 subdevices are added to the > common media device create links between them. > > The parsing and registering CSI-2 subdevices with the

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

2017-12-04 Thread Hans Verkuil
On 11/29/2017 08:43 PM, Niklas Söderlund wrote: > Add the SoC specific information for Renesas r8a77970. > > Signed-off-by: Niklas Söderlund Reviewed-by: Hans Verkuil Regards, Hans > --- > .../devicetree/bindings/media/rcar_vin.txt | 1 + > drivers/media/platform/rcar-vin/r

Re: [PATCH v8 04/28] rcar-vin: move subdevice handling to async callbacks

2017-12-04 Thread Hans Verkuil
On 11/29/2017 08:43 PM, Niklas Söderlund wrote: > In preparation for Gen3 support move the subdevice initialization and > clean up from rvin_v4l2_{register,unregister}() directly to the async > callbacks. This simplifies the addition of Gen3 support as the > rvin_v4l2_register() can be shared for b

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

2017-12-04 Thread Hans Verkuil
On 11/29/2017 08:43 PM, 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 > uninitialized video device node name by adding a checking in > rvin_v4l2_unregister() by checking that th

Re: [PATCH v2 1/3] media: V3s: Add support for Allwinner CSI.

2017-12-04 Thread Yong
Hi Maxime, I just noticed that you are using the second iteration? Have you received my third iteration? On Sat, 25 Nov 2017 17:02:33 +0100 Maxime Ripard wrote: > On Thu, Nov 23, 2017 at 09:14:44AM +0800, Yong wrote: > > > On Wed, Nov 22, 2017 at 09:33:06AM +0800, Yong wrote: > > > > > On Thu,

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

2017-12-04 Thread Hans Verkuil
On 11/29/2017 08:43 PM, Niklas Söderlund wrote: > The functions to initialize and cleanup the hardware and video device > where poorly named from the start. Rename them to better describe there > intended function. > > Signed-off-by: Niklas Söderlund Reviewed-by: Hans Verkuil Much better, inde

Re: [PATCH 5/9] v4l: vsp1: Document the vsp1_du_atomic_config structure

2017-12-04 Thread Sergei Shtylyov
Hello! On 12/3/2017 1:57 PM, Laurent Pinchart wrote: The structure is used in the API that the VSP1 driver exposes to the DU driver. Documenting it is thus important. Signed-off-by: Laurent Pinchart --- include/media/vsp1.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/in

Re: [PATCH] v4l: rcar-vin: Implement V4L2 video node release handler

2017-12-04 Thread Hans Verkuil
Hi Niklas, On 11/16/2017 01:27 AM, Laurent Pinchart wrote: > Hi Niklas, > > On Thursday, 16 November 2017 01:34:33 EET Niklas Söderlund wrote: >> On 2017-11-16 00:49:07 +0200, Laurent Pinchart wrote: >>> The rvin_dev data structure contains driver private data for an instance >>> of the VIN. It i

[PATCH] cec-adap: add '0x' prefix when printing status

2017-12-04 Thread Hans Verkuil
It's not clear if the transmit status that is printed when debugging is enabled is hex or decimal. Add 0x prefix to make this explicit. Signed-off-by: Hans Verkuil --- drivers/media/cec/cec-adap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/cec/cec-adap.c b/