Re: [PATCH v2 08/20] drm: msm: Rely on the default ->best_encoder() behavior where appropriate

2016-06-08 Thread Archit Taneja
us. Works fine with msm. Tested-by: Archit Taneja <arch...@codeaurora.org> Thanks, Archit Signed-off-by: Boris Brezillon <boris.brezil...@free-electrons.com> --- drivers/gpu/drm/msm/edp/edp_connector.c| 10 -- drivers/gpu/drm/msm/hdmi/hdmi_connector.

Re: [PATCH] drm/bridge: adv7511: add support for the 2nd chip

2016-09-02 Thread Archit Taneja
Hi, On 9/2/2016 2:13 AM, Sergei Shtylyov wrote: The Renesas Wheat board has 2 ADV7513 chips on the same I2C bus, however the ADV751x driver only supports 1 chip as it tries to assign the packet/ EDID/CEC memory I2C devices to the fixed I2C addresses. Assign these I2C addresses at the fixed

Re: [PATCH v2] drm/bridge: adv7511: add support for the 2nd chip

2016-09-12 Thread Archit Taneja
On 09/06/2016 01:13 AM, Sergei Shtylyov wrote: The Renesas Wheat board has 2 ADV7513 chips on the same I2C bus, however the ADV751x driver only supports 1 chip as it tries to assign the packet/ EDID/CEC memory I2C devices to the fixed I2C addresses. Assign these I2C addresses at the fixed

Re: [PATCH v4] drm: bridge: add DesignWare HDMI I2S audio support

2016-11-24 Thread Archit Taneja
Hi, On 11/24/2016 10:43 AM, Kuninori Morimoto wrote: Hi Archit, David, and DRM ML I had heared that Archit is the maintainer of dw-hdmi driver, but am I wrong ?? I'm posting this patch series since half year ago, but no response from him, and nothing happen (I got review from Russell though).

Re: [PATCH 2/8] drm: bridge: vga-dac: Add adi,adv7123 compatible string

2016-10-21 Thread Archit Taneja
On 10/21/2016 03:52 PM, Laurent Pinchart wrote: Hi Archit, On Friday 21 Oct 2016 10:43:34 Archit Taneja wrote: On 10/19/2016 07:55 PM, Laurent Pinchart wrote: The ADV7123 is a transparent VGA DAC. Unlike dumb VGA DACs it can be controlled through a power save pin, and requires a power

Re: [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Archit Taneja
On 11/29/2016 02:34 PM, Laurent Pinchart wrote: Instead of linking encoders and bridges in every driver (and getting it wrong half of the time, as many drivers forget to set the drm_bridge encoder pointer), do so in core code. The drm_bridge_attach() function needs the encoder and optional

Re: [PATCH v3 04/13] drm: bridge: Detach bridge from encoder at encoder cleanup time

2016-11-29 Thread Archit Taneja
On 11/29/2016 02:34 PM, Laurent Pinchart wrote: Most drivers that use bridges forgot to detach them at cleanup time. Instead of fixing them one by one, detach the bridge in the core drm_encoder_cleanup() function. Signed-off-by: Laurent Pinchart ---

Re: [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Archit Taneja
On 11/29/2016 11:27 PM, Laurent Pinchart wrote: Hi Archit, On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: On 11/29/2016 02:34 PM, Laurent Pinchart wrote: Instead of linking encoders and bridges in every driver (and getting it wrong half of the time, as many drivers forget to set

Re: [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 03:53 PM, Laurent Pinchart wrote: Hi Archit, On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: On 11/29/2016 11:27 PM, Laurent Pinchart wrote: On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: On 11/29/2016 02:34 PM, Laurent Pinchart wrote: Instead of linking

Re: [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 4:35 PM, Laurent Pinchart wrote: Hi Archit, On Wednesday 30 Nov 2016 16:30:53 Archit Taneja wrote: On 11/30/2016 03:53 PM, Laurent Pinchart wrote: On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: On 11/29/2016 11:27 PM, Laurent Pinchart wrote: On Tuesday 29 Nov 2016 15

Re: [PATCH v5 00/10] drm: bridge: dw-hdmi: Refactor PHY support

2017-03-10 Thread Archit Taneja
On 3/3/2017 10:49 PM, Laurent Pinchart wrote: Hello, This patch series refactors all the PHY handling code in order to allow support of vendor PHYs and Synopsys DWC HDMI 2.0 TX PHYs. The series starts with a few cleanups and small fixes. Patch 01/10 just removes unused code, patch 02/10

Re: [PATCH v4 2/4] drm: bridge: Add LVDS encoder driver

2017-03-06 Thread Archit Taneja
Hi, On 3/2/2017 4:17 PM, Laurent Pinchart wrote: The LVDS encoder driver is a DRM bridge driver that supports the parallel to LVDS encoders that don't require any configuration. The driver thus doesn't interact with the device, but creates an LVDS connector for the panel and exposes its size

Re: [PATCH 4/4] drm: adv7511/33: add HDMI CEC support

2017-08-10 Thread Archit Taneja
On 07/30/2017 06:37 PM, Hans Verkuil wrote: From: Hans Verkuil Add support for HDMI CEC to the drm adv7511/adv7533 drivers. The CEC registers that we need to use are identical for both drivers, but they appear at different offsets in the register map. Thanks for

Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support

2017-08-10 Thread Archit Taneja
Hi Hans, On 07/30/2017 06:37 PM, Hans Verkuil wrote: From: Hans Verkuil This patch series adds CEC support to the drm adv7511/adv7533 drivers. I have tested this with the Qualcomm Dragonboard C410 (adv7533 based) and the Renesas R-Car Koelsch board (adv7511 based).

Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support

2017-08-10 Thread Archit Taneja
On 08/10/2017 02:26 PM, Hans Verkuil wrote: On 10/08/17 10:49, Archit Taneja wrote: Hi Hans, On 07/30/2017 06:37 PM, Hans Verkuil wrote: From: Hans Verkuil <hans.verk...@cisco.com> This patch series adds CEC support to the drm adv7511/adv7533 drivers. I have tested this with the Qu

Re: [PATCH 5/8] drm: Nuke drm_atomic_helper_plane_set_property

2017-07-25 Thread Archit Taneja
On 07/25/2017 01:31 PM, Daniel Vetter wrote: It's dead code, the core handles all this directly now. This also allows us to unexport drm_atomic_helper_plane_set_property. Reviewed-by: Archit Taneja <arch...@codeaurora.org> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com&g

Re: [PATCH 0/4] drm/bridge/adv7511: add CEC support

2017-08-09 Thread Archit Taneja
On 07/30/2017 06:37 PM, Hans Verkuil wrote: From: Hans Verkuil This patch series adds CEC support to the drm adv7511/adv7533 drivers. I have tested this with the Qualcomm Dragonboard C410 (adv7533 based) and the Renesas R-Car Koelsch board (adv7511 based). Note: the

Re: [PATCH][resend] drm: dw-hdmi-i2s: add missing company name on Copyright

2017-08-06 Thread Archit Taneja
On 08/07/2017 07:41 AM, Kuninori Morimoto wrote: From: Kuninori Morimoto This driver's Copyright is under Renesas Solutions Corp Can we update the year to 2017 while we're at it? Archit Signed-off-by: Kuninori Morimoto

Re: [PATCH v2] drm: dw-hdmi-i2s: add missing company name on Copyright

2017-08-06 Thread Archit Taneja
On 08/07/2017 09:39 AM, Kuninori Morimoto wrote: From: Kuninori Morimoto This driver's Copyright is under Renesas Solutions Corp. This patch updates the year, because this driver was moved into synopsys folder in 2017. Thanks. Queued to drm-misc-next.

Re: [PATCH][resend] drm: dw-hdmi-i2s: add missing company name on Copyright

2017-08-06 Thread Archit Taneja
On 08/07/2017 09:25 AM, Kuninori Morimoto wrote: Hi Archit Thank you for your feedback On 08/07/2017 07:41 AM, Kuninori Morimoto wrote: From: Kuninori Morimoto This driver's Copyright is under Renesas Solutions Corp Can we update the year to 2017

Re: [PATCH 5/5] drm: dw-hdmi-i2s: add .get_dai_id callback for ALSA SoC

2017-05-25 Thread Archit Taneja
On 05/26/2017 09:46 AM, Archit Taneja wrote: Hi, On 05/25/2017 05:04 AM, Kuninori Morimoto wrote: Hi Mark Cc: DRM maintainer ALSA SoC needs to know connected DAI ID for probing. It is not a big problem if device/driver was only for sound, but getting DAI ID will be difficult if device

Re: [PATCH 5/5] drm: dw-hdmi-i2s: add .get_dai_id callback for ALSA SoC

2017-05-25 Thread Archit Taneja
Hi, On 05/25/2017 05:04 AM, Kuninori Morimoto wrote: Hi Mark Cc: DRM maintainer ALSA SoC needs to know connected DAI ID for probing. It is not a big problem if device/driver was only for sound, but getting DAI ID will be difficult if device includes both Video/Sound, like HDMI. As far as I

Re: [PATCHv3 0/2] drm/bridge/adv7511: add CEC support

2017-10-09 Thread Archit Taneja
On 10/07/2017 04:16 PM, Hans Verkuil wrote: From: Hans Verkuil This patch series adds CEC support to the drm adv7511/adv7533 drivers. I have tested this with the Qualcomm Dragonboard C410 (adv7533 based) and the Renesas R-Car Koelsch board (adv7511 based). I only

Re: [PATCH 2/2] drm: adv7511: Add support for i2c_new_secondary_device

2018-01-28 Thread Archit Taneja
Hi, On 01/22/2018 06:20 PM, Kieran Bingham wrote: The ADV7511 has four 256-byte maps that can be accessed via the main I²C ports. Each map has it own I²C address and acts as a standard slave device on the I²C bus. Allow a device tree node to override the default addresses so that address

Re: [PATCH v2 0/3] drm: Add LVDS decoder bridge

2018-03-09 Thread Archit Taneja
Hi, On Friday 09 March 2018 07:21 PM, Jacopo Mondi wrote: Hello, after some discussion on the proposed bindings for generic lvds decoder and Thine THC63LVD1024, I decided to drop the THC63 specific part and just live with a transparent decoder that does not support any configuration from

Re: [PATCH v4 2/5] dt-bindings: adv7511: Extend bindings to allow specifying slave map addresses

2018-04-25 Thread Archit Taneja
On Tuesday 13 February 2018 11:18 PM, Kieran Bingham wrote: From: Kieran Bingham The ADV7511 has four 256-byte maps that can be accessed via the main I2C ports. Each map has it own I2C address and acts as a standard slave device on the I2C bus.

Re: [PATCH v4 5/5] drm: adv7511: Add support for i2c_new_secondary_device

2018-04-25 Thread Archit Taneja
On Tuesday 13 February 2018 11:18 PM, Kieran Bingham wrote: From: Kieran Bingham The ADV7511 has four 256-byte maps that can be accessed via the main I2C ports. Each map has it own I2C address and acts as a standard slave device on the I2C bus. Allow

Re: [RFC][PATCH 04/11] drm: Split the display info into static and dynamic parts

2018-02-27 Thread Archit Taneja
(>connector, tc->bridge.encoder); The sii902x driver sets the bus_formats in get_modes, but it's a fixed value and we may as well do it in bridge's attach op. For the bridge drivers: Reviewed-by: Archit Taneja <arch...@codeaurora.org> Thanks, Archit