[PATCH] ravb: remove erroneous comment

2018-03-03 Thread Niklas Söderlund
When addressing a review comment in a early version of the offending patch a comment where left in which should have been removed. Remove the comment to keep it consistent with the code. Fixes: 75efa06f457bbed3 ("ravb: add support for changing MTU") Reported-by: Sergei Shtylyov

[PATCH v3 2/2] arm64: dts: renesas: eagle: add I2C0 support

2018-03-03 Thread Sergei Shtylyov
Define the Eagle board dependent part of the I2C0 device node. The I2C0 bus is populated by ON Semiconductor PCA9653 I/O expander and Analog Devices ADV7511W HDMI transmitter (but we're only describing the former chip now). Based on the original (and large) patch by Vladimir Barinov.

[PATCH v3 1/2] arm64: dts: renesas: r8a77970: add I2C support

2018-03-03 Thread Sergei Shtylyov
Define the generic R8A77970 parts of the I2C[0-4] device node. Based on the original (and large) patch by Daisuke Matsushita . Signed-off-by: Vladimir Barinov Signed-off-by: Sergei Shtylyov

[PATCH v3 0/2] Add R8A77970/Eagle I2C support

2018-03-03 Thread Sergei Shtylyov
Hello! Here's the set of 2 patches against Simon Horman's 'renesas.git' repo's 'renesas-devel-20180301-v4.16-rc3' tag. We're adding the R8A77970 I2C nodes and then describing the PCA9654 I/O expander connected to the I2C0 bus. [1/2] arm64: dts: renesas: r8a77970: add I2C support [2/2] arm64:

Re: [PATCH v11 16/32] rcar-vin: read subdevice format for crop only when needed

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2018-03-02 13:06:27 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:35 EET Niklas Söderlund wrote: > > Instead of caching the subdevice format each time the video device > > format is set read

Re: [PATCH v11 12/32] rcar-vin: fix handling of single field frames (top, bottom and alternate fields)

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your comments. On 2018-03-02 12:48:55 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:31 EET Niklas Söderlund wrote: > > There was never proper support in the VIN driver to deliver ALTERNATING > > field format to

Re: [PATCH v11 15/32] rcar-vin: break out format alignment and checking

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2018-03-02 11:53:54 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:34 EET Niklas Söderlund wrote: > > Part of the format alignment and checking can be shared with the Gen3 > > format handling.

Re: [PATCH v11 28/32] rcar-vin: add link notify for Gen3

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2018-03-02 14:00:19 +0200, Laurent Pinchart wrote: [snip] > > + > > + /* If any entity is in use don't allow link changes. */ > > + media_device_for_each_entity(entity, >mdev) > > + if (entity->use_count) > > + return

Re: [PATCH v11 22/32] rcar-vin: force default colorspace for media centric mode

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2018-03-02 11:59:14 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:41 EET Niklas Söderlund wrote: > > When the VIN driver is running in media centric mode (on Gen3) the > > colorspace is not

Re: [PATCH v11 19/32] rcar-vin: add function to manipulate Gen3 chsel value

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thank you for your feedback. On 2018-03-02 13:31:47 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:38 EET Niklas Söderlund wrote: > > On Gen3 the CSI-2 routing is controlled by the VnCSI_IFMD register. One > > feature of

Re: [PATCH v11 17/32] rcar-vin: move media bus configuration to struct rvin_info

2018-03-03 Thread Niklas Söderlund
Hi Laurent, Thanks for your feedback. On 2018-03-02 13:26:58 +0200, Laurent Pinchart wrote: > Hi Niklas, > > Thank you for the patch. > > On Friday, 2 March 2018 03:57:36 EET Niklas Söderlund wrote: > > Bus configuration will once the driver is extended to support Gen3 > > contain information