[PATCH 1/3] staging: android: ion: Remove unused members from ion_buffer

A few members in ion_buffer struct are unused after features like page faulting, ion_handle and ion_client were removed. Remove these members and the leftover references to them. Signed-off-by: Archit Taneja --- drivers/staging/android/ion/ion.c | 2 -- drivers/staging/android/ion/ion.h | 14

[PATCH 3/3] staging: android: ion: Avoid calling free_duped_table() twice

twice. Don't call free_duped_table() in ion_map_dma_buf() to avoid this. Signed-off-by: Archit Taneja --- drivers/staging/android/ion/ion.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index

[PATCH 2/3] staging: android: ion: Remove ION_FLAG_CACHED_NEEDS_SYNC

The flag ION_FLAG_CACHED_NEEDS_SYNC isn't used anymore. Remove it. Signed-off-by: Archit Taneja <arch...@codeaurora.org> --- drivers/staging/android/uapi/ion.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/android/uapi/ion.h b/drivers/staging/android/uapi/ion.h

[PATCH 0/3] staging: android: ion: Minor clean ups and fixes

The recent ION clean ups introduced some leftover code that can be removed, and a bug that comes up if the call to dma_buf_map_attachment() from an importer fails. Fix these. Archit Taneja (3): staging: android: ion: Remove unused members from ion_buffer staging: android: ion: Remove

[PATCH 2/3] staging: android: ion: Remove ION_FLAG_CACHED_NEEDS_SYNC

The flag ION_FLAG_CACHED_NEEDS_SYNC isn't used anymore. Remove it. Signed-off-by: Archit Taneja --- drivers/staging/android/uapi/ion.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/android/uapi/ion.h b/drivers/staging/android/uapi/ion.h index b76db1b2e197

[PATCH 0/3] staging: android: ion: Minor clean ups and fixes

The recent ION clean ups introduced some leftover code that can be removed, and a bug that comes up if the call to dma_buf_map_attachment() from an importer fails. Fix these. Archit Taneja (3): staging: android: ion: Remove unused members from ion_buffer staging: android: ion: Remove

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

On 05/12/2017 04:31 PM, Laurent Pinchart wrote: Hi Archit, On Friday 12 May 2017 16:20:07 Archit Taneja wrote: On 05/12/2017 03:08 PM, Laurent Pinchart wrote: On Wednesday 10 May 2017 17:14:33 Daniel Vetter wrote: On Wed, May 10, 2017 at 04:41:09PM +0300, Ville Syrjälä wrote: On Tue, May

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

On 05/12/2017 04:31 PM, Laurent Pinchart wrote: Hi Archit, On Friday 12 May 2017 16:20:07 Archit Taneja wrote: On 05/12/2017 03:08 PM, Laurent Pinchart wrote: On Wednesday 10 May 2017 17:14:33 Daniel Vetter wrote: On Wed, May 10, 2017 at 04:41:09PM +0300, Ville Syrjälä wrote: On Tue, May

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

.@ffwll.ch> Cc: Dave Airlie <airl...@linux.ie> Cc: Andrzej Hajda <a.ha...@samsung.com> Cc: Archit Taneja <arch...@codeaurora.org> --- drivers/gpu/drm/drm_bridge.c | 33 + include/drm/drm_bridge.h | 2 ++ 2 files changed, 35 insertions(+) dif

Re: [PATCH v2 6/8] drm: Introduce drm_bridge_mode_valid()

mode_valid() callback for all bridges in an encoder chain. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- drivers/gpu/drm/drm_bridge.c | 33 + include/drm

Re: [PATCH v2 1/7] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

G_DRM_PANEL +struct drm_bridge *drm_panel_bridge_add(struct drm_panel *panel, + u32 connector_type); +void drm_panel_bridge_remove(struct drm_bridge *bridge); +#endif Shouldn't this be CONFIG_DRM_PANEL_BRIDGE? As I mentioned on irc, I was facing some build issues with this patch applied. Once we sort that out: Acked-by: Archit Taneja <arch...@codeaurora.org> Thanks, Archit -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH v2 1/7] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

u32 connector_type); +void drm_panel_bridge_remove(struct drm_bridge *bridge); +#endif Shouldn't this be CONFIG_DRM_PANEL_BRIDGE? As I mentioned on irc, I was facing some build issues with this patch applied. Once we sort that out: Acked-by: Archit Taneja Thanks, Archit -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

Re: [PATCH v2 7/8] drm: Use mode_valid() in atomic modeset

l Vetter <daniel.vet...@ffwll.ch> Cc: Dave Airlie <airl...@linux.ie> Cc: Andrzej Hajda <a.ha...@samsung.com> Cc: Archit Taneja <arch...@codeaurora.org> --- Changes v1->v2: - Removed call to connector->mode_valid (Ville, Daniel) - Change function

Re: [PATCH v2 7/8] drm: Use mode_valid() in atomic modeset

) so that we can make sure that the mode will be accepted in every components. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Alexey Brodkin Cc: Ville Syrjälä Cc: Daniel Vetter Cc: Dave Airlie Cc: Andrzej Hajda Cc: Archit Taneja --- Changes v1->v2: - Removed call to connec

Re: [PATCH 1/2] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

On 05/03/2017 10:00 PM, Eric Anholt wrote: Archit Taneja <arch...@codeaurora.org> writes: Hi, On 04/27/2017 10:06 PM, Eric Anholt wrote: Many DRM drivers have common code to make a stub connector implementation that wraps a drm_panel. By wrapping the panel in a DRM bridg

Re: [PATCH 1/2] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

On 05/03/2017 10:00 PM, Eric Anholt wrote: Archit Taneja writes: Hi, On 04/27/2017 10:06 PM, Eric Anholt wrote: Many DRM drivers have common code to make a stub connector implementation that wraps a drm_panel. By wrapping the panel in a DRM bridge, all of the connector code (including

Re: [PATCH 1/2] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

Hi, On 04/27/2017 10:06 PM, Eric Anholt wrote: Many DRM drivers have common code to make a stub connector implementation that wraps a drm_panel. By wrapping the panel in a DRM bridge, all of the connector code (including calls during encoder enable/disable) goes away. Signed-off-by: Eric

Re: [PATCH 1/2] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

Hi, On 04/27/2017 10:06 PM, Eric Anholt wrote: Many DRM drivers have common code to make a stub connector implementation that wraps a drm_panel. By wrapping the panel in a DRM bridge, all of the connector code (including calls during encoder enable/disable) goes away. Signed-off-by: Eric

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

On 04/19/2017 10:21 AM, Archit Taneja wrote: On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, the audio sampler clock is enabled from dw_hdmi_setup() at step E. and is kept enabled for later use. This clock should be enabled and disabled along with the actual audio stream

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

On 04/19/2017 10:21 AM, Archit Taneja wrote: On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, the audio sampler clock is enabled from dw_hdmi_setup() at step E. and is kept enabled for later use. This clock should be enabled and disabled along with the actual audio stream

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, the audio sampler clock is enabled from dw_hdmi_setup() at step E. and is kept enabled for later use. This clock should be enabled and disabled along with the actual audio stream and not always on (that is bad for PM). Futhermore, as

Re: [PATCH v2 2/2] drm: dw-hdmi: gate audio clock from the I2S enablement callbacks

On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, the audio sampler clock is enabled from dw_hdmi_setup() at step E. and is kept enabled for later use. This clock should be enabled and disabled along with the actual audio stream and not always on (that is bad for PM). Futhermore, as

Re: [PATCH v2 1/2] drm: dw-hdmi: add specific I2S and AHB functions for stream handling

On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, CTS+N is forced to zero as a workaround of the IP block for i.MX platforms. This is requested in the datasheet of the corresponding IP for AHB mode only. However, we have seen that it introduces glitches or delays when playing a sound on

Re: [PATCH v2 1/2] drm: dw-hdmi: add specific I2S and AHB functions for stream handling

On 04/14/2017 02:01 PM, Romain Perier wrote: Currently, CTS+N is forced to zero as a workaround of the IP block for i.MX platforms. This is requested in the datasheet of the corresponding IP for AHB mode only. However, we have seen that it introduces glitches or delays when playing a sound on

Re: [PATCHv3 07/22] staging: android: ion: Remove page faulting support

Hi, On 04/04/2017 12:27 AM, Laura Abbott wrote: The new method of syncing with dma_map means that the page faulting sync implementation is no longer applicable. Remove it. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 117

Re: [PATCHv3 07/22] staging: android: ion: Remove page faulting support

Hi, On 04/04/2017 12:27 AM, Laura Abbott wrote: The new method of syncing with dma_map means that the page faulting sync implementation is no longer applicable. Remove it. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 117 -- 1 file

Re: [PATCHv3 08/22] staging: android: ion: Remove crufty cache support

Hi, On 04/04/2017 12:27 AM, Laura Abbott wrote: Now that we call dma_map in the dma_buf API callbacks there is no need to use the existing cache APIs. Remove the sync ioctl and the existing bad dma_sync calls. Explicit caching can be handled with the dma_buf sync API. We could get rid of the

Re: [PATCHv3 08/22] staging: android: ion: Remove crufty cache support

Hi, On 04/04/2017 12:27 AM, Laura Abbott wrote: Now that we call dma_map in the dma_buf API callbacks there is no need to use the existing cache APIs. Remove the sync ioctl and the existing bad dma_sync calls. Explicit caching can be handled with the dma_buf sync API. We could get rid of the

Re: [PATCH] drm: dw-hdmi: Implement the mode_fixup drm helper

On 04/08/2017 12:03 AM, Daniel Vetter wrote: On Fri, Apr 07, 2017 at 02:17:43PM +0200, Romain Perier wrote: This helper is supposed to validate or reject the modeline before it applied by the mode setting. Currently this function has been dropped, it was previously set to a dummy function

Re: [PATCH] drm: dw-hdmi: Implement the mode_fixup drm helper

On 04/08/2017 12:03 AM, Daniel Vetter wrote: On Fri, Apr 07, 2017 at 02:17:43PM +0200, Romain Perier wrote: This helper is supposed to validate or reject the modeline before it applied by the mode setting. Currently this function has been dropped, it was previously set to a dummy function

Re: [PATCH 0/2] drm: dw-hdmi: various improvements

Hi, On 04/07/2017 07:49 PM, Romain Perier wrote: This set of patches split the stream handling functions in two parts. It introduces new callbacks that are specific to each variant, one for I2S and one for AHB. Then, as requested by the datasheet for the I2S variant, it adds support for gating

Re: [PATCH 0/2] drm: dw-hdmi: various improvements

Hi, On 04/07/2017 07:49 PM, Romain Perier wrote: This set of patches split the stream handling functions in two parts. It introduces new callbacks that are specific to each variant, one for I2S and one for AHB. Then, as requested by the datasheet for the I2S variant, it adds support for gating

Re: [PATCH] drm: dw-hdmi: Implement the mode_fixup drm helper

Hi, On 4/7/2017 5:47 PM, Romain Perier wrote: This helper is supposed to validate or reject the modeline before it applied by the mode setting. Currently this function has been dropped, it was previously set to a dummy function that always returned true. For both cases, this means that

Re: [PATCH] drm: dw-hdmi: Implement the mode_fixup drm helper

Hi, On 4/7/2017 5:47 PM, Romain Perier wrote: This helper is supposed to validate or reject the modeline before it applied by the mode setting. Currently this function has been dropped, it was previously set to a dummy function that always returned true. For both cases, this means that

Re: [PATCH] drm: bridge: dw-hdmi: fix input format/encoding from plat_data

plat_data->input_bus_encoding too? You might want to rephrase it as the "default color space", or something along those lines. This patch changes the if test to > 0. The change technically makes the if statement check for a non-zero value. Reviewed-by: Archit Taneja <arch..

Re: [PATCH] drm: bridge: dw-hdmi: fix input format/encoding from plat_data

plat_data->input_bus_encoding too? You might want to rephrase it as the "default color space", or something along those lines. This patch changes the if test to > 0. The change technically makes the if statement check for a non-zero value. Reviewed-by: Archit Taneja Feel free

Re: [PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

Hi Laurent, On 4/6/2017 2:40 PM, Laurent Pinchart wrote: Hi Archit, On Wednesday 05 Apr 2017 16:53:50 Archit Taneja wrote: Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They actively review and contribute to bridge drivers and the bridge API. How about adding me

Re: [PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

Hi Laurent, On 4/6/2017 2:40 PM, Laurent Pinchart wrote: Hi Archit, On Wednesday 05 Apr 2017 16:53:50 Archit Taneja wrote: Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They actively review and contribute to bridge drivers and the bridge API. How about adding me

[PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They actively review and contribute to bridge drivers and the bridge API. Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> Cc: Andrzej Hajda <a.ha...@samsung.com> Signed-off-by: Archit Taneja <arch...

[PATCH] MAINTAINERS: Add Laurent and Andrzej as maintainers for bridge drivers

Add Laurent and Andrzej as maintainers for DRM bridge chip drivers. They actively review and contribute to bridge drivers and the bridge API. Cc: Laurent Pinchart Cc: Andrzej Hajda Signed-off-by: Archit Taneja --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+) diff --git a/MAINTAINERS b

Re: [PATCH] megachips-stdpxxxx-ge-b850v3-fw: Fix drm bridge initialization

On 03/30/2017 01:49 PM, Peter Senna Tschudin wrote: Reordering of the device nodes based on unit address resulted in ge_b850v3_lvds_attach() being called before ge_b850v3_lvds_ptr->stdp4028_i2c was populated. This patch moves the drm bridge initialization from ge_b850v3_lvds_init() to

Re: [PATCH] megachips-stdpxxxx-ge-b850v3-fw: Fix drm bridge initialization

On 03/30/2017 01:49 PM, Peter Senna Tschudin wrote: Reordering of the device nodes based on unit address resulted in ge_b850v3_lvds_attach() being called before ge_b850v3_lvds_ptr->stdp4028_i2c was populated. This patch moves the drm bridge initialization from ge_b850v3_lvds_init() to

Re: [PATCH v5.1 0/6] drm: bridge: dw-hdmi: Add support for Custom PHYs

Hi, On 03/31/2017 07:55 PM, Neil Armstrong wrote: The Amlogic GX SoCs implements a Synopsys DesignWare HDMI TX Controller in combination with a very custom PHY. Thanks to Laurent Pinchart's changes, the HW report the following : Detected HDMI TX controller v2.01a with HDCP (meson_dw_hdmi_phy)

Re: [PATCH v5.1 0/6] drm: bridge: dw-hdmi: Add support for Custom PHYs

Hi, On 03/31/2017 07:55 PM, Neil Armstrong wrote: The Amlogic GX SoCs implements a Synopsys DesignWare HDMI TX Controller in combination with a very custom PHY. Thanks to Laurent Pinchart's changes, the HW report the following : Detected HDMI TX controller v2.01a with HDCP (meson_dw_hdmi_phy)

Re: [PATCH v2] drm: dw_hdmi: Don't rely on the status of the bridge for updating HPD

Hi, On 03/10/2017 03:01 PM, Romain Perier wrote: Currently, the irq handler that monitores changes for HPD anx RX_SENSE relies on the status of the bridge for updating the status of the HPD. The update is done only when the bridge is enabled. However, on Rockchip platforms we have found use

Re: [PATCH v2] drm: dw_hdmi: Don't rely on the status of the bridge for updating HPD

Hi, On 03/10/2017 03:01 PM, Romain Perier wrote: Currently, the irq handler that monitores changes for HPD anx RX_SENSE relies on the status of the bridge for updating the status of the HPD. The update is done only when the bridge is enabled. However, on Rockchip platforms we have found use

Re: [PATCH] drm: bridge: dw-hdmi: add HDMI vendor specific infoframe config

On 03/21/2017 01:17 PM, Andrzej Hajda wrote: On 20.03.2017 07:10, Nickey Yang wrote: Vendor specific infoframe is mandatory for 4K2K resolution. Without this, the HDMI protocol compliance fails. Signed-off-by: Nickey Yang --- drivers/gpu/drm/bridge/dw-hdmi.c |

Re: [PATCH] drm: bridge: dw-hdmi: add HDMI vendor specific infoframe config

On 03/21/2017 01:17 PM, Andrzej Hajda wrote: On 20.03.2017 07:10, Nickey Yang wrote: Vendor specific infoframe is mandatory for 4K2K resolution. Without this, the HDMI protocol compliance fails. Signed-off-by: Nickey Yang --- drivers/gpu/drm/bridge/dw-hdmi.c | 50

Re: [PATCH v2] drm/bridge: dw_hdmi: support i2c extended read mode

On 03/20/2017 08:27 AM, Nickey Yang wrote: "I2C Master Interface Extended Read Mode" implements a segment pointer-based read operation using the Special Register configuration. This patch fix https://patchwork.kernel.org/patch/7098101/ mentioned "The current implementation does not support

Re: [PATCH v2] drm/bridge: dw_hdmi: support i2c extended read mode

On 03/20/2017 08:27 AM, Nickey Yang wrote: "I2C Master Interface Extended Read Mode" implements a segment pointer-based read operation using the Special Register configuration. This patch fix https://patchwork.kernel.org/patch/7098101/ mentioned "The current implementation does not support

Re: [PATCH v3 2/6] media: uapi: Add RGB and YUV bus formats for Synopsys HDMI TX Controller

-by: Archit Taneja <arch...@codeaurora.org> Signed-off-by: Neil Armstrong <narmstr...@baylibre.com> --- include/uapi/linux/media-bus-format.h | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-

Re: [PATCH v3 2/6] media: uapi: Add RGB and YUV bus formats for Synopsys HDMI TX Controller

-by: Archit Taneja Signed-off-by: Neil Armstrong --- include/uapi/linux/media-bus-format.h | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h index 2168759..7cc820b 100644 --- a/include/uapi

Re: [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats

On 3/7/2017 10:12 PM, Neil Armstrong wrote: Add documentation for added Bus Formats to describe RGB and YUS formats used s/YUS/YUV as input to the Synopsys DesignWare HDMI TX Controller. Signed-off-by: Neil Armstrong ---

Re: [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats

On 3/7/2017 10:12 PM, Neil Armstrong wrote: Add documentation for added Bus Formats to describe RGB and YUS formats used s/YUS/YUV as input to the Synopsys DesignWare HDMI TX Controller. Signed-off-by: Neil Armstrong --- Documentation/media/uapi/v4l/subdev-formats.rst | 4992

Re: [PATCH v3 5/6] drm: bridge: dw-hdmi: Add Documentation on supported input formats

On 3/7/2017 10:12 PM, Neil Armstrong wrote: This patch adds a new DRM documentation entry and links to the input format table added in the dw_hdmi header. Signed-off-by: Neil Armstrong --- Documentation/gpu/dw-hdmi.rst | 15 +++

Re: [PATCH v3 5/6] drm: bridge: dw-hdmi: Add Documentation on supported input formats

On 3/7/2017 10:12 PM, Neil Armstrong wrote: This patch adds a new DRM documentation entry and links to the input format table added in the dw_hdmi header. Signed-off-by: Neil Armstrong --- Documentation/gpu/dw-hdmi.rst | 15 +++ Documentation/gpu/index.rst | 1 + Maybe we

Re: [PATCH V3 0/4] megachips-stdpxxxx-ge-b850v3-fw

Hi, On 3/3/2017 9:27 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output on the GE B850v3: Host -> LVDS|--(STDP4028)--|DP -> DP|--(STDP2690)--|DP++ -> Video output Each bridge has a dedicated flash containing firmware for supporting the custom design. The

Re: [PATCH V3 0/4] megachips-stdpxxxx-ge-b850v3-fw

Hi, On 3/3/2017 9:27 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output on the GE B850v3: Host -> LVDS|--(STDP4028)--|DP -> DP|--(STDP2690)--|DP++ -> Video output Each bridge has a dedicated flash containing firmware for supporting the custom design. The

Re: [PATCH V2 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

Hi Peter, On 3/1/2017 4:08 PM, Peter Senna Tschudin wrote: Hi Archit, Thank you for the review! On Wed, Mar 01, 2017 at 09:38:48AM +0530, Archit Taneja wrote: On 02/28/2017 07:58 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output on the GE B850v3: Host

Re: [PATCH V2 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

Hi Peter, On 3/1/2017 4:08 PM, Peter Senna Tschudin wrote: Hi Archit, Thank you for the review! On Wed, Mar 01, 2017 at 09:38:48AM +0530, Archit Taneja wrote: On 02/28/2017 07:58 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output on the GE B850v3: Host

Re: [PATCH V2 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

; Cc: Thierry Reding <tred...@nvidia.com> Cc: Thierry Reding <thierry.red...@gmail.com> Cc: Archit Taneja <arch...@codeaurora.org> Cc: Enric Balletbo <enric.balle...@collabora.com> Signed-off-by: Peter Senna Tschudin <peter.se...@collabora.com> --- Changes from

Re: [PATCH V2 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

c: David Airlie Cc: Thierry Reding Cc: Thierry Reding Cc: Archit Taneja Cc: Enric Balletbo Signed-off-by: Peter Senna Tschudin --- Changes from V1: - Updated copyright year - Fixed blank line issues - Updated ge_b850v3_lvds_remove() to not rely on ge_b850v3_lvds_ptr->edid and adde

Re: [PATCH 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

On 02/27/2017 07:10 PM, Peter Senna Tschudin wrote: On Wed, Feb 01, 2017 at 04:17:21PM +0530, Archit Taneja wrote: Hi Archit, Hi, Some minor comments: Thank you for the review! On 01/28/2017 07:51 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output

Re: [PATCH 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

On 02/27/2017 07:10 PM, Peter Senna Tschudin wrote: On Wed, Feb 01, 2017 at 04:17:21PM +0530, Archit Taneja wrote: Hi Archit, Hi, Some minor comments: Thank you for the review! On 01/28/2017 07:51 PM, Peter Senna Tschudin wrote: The video processing pipeline on the second output

Re: [PATCH -next] drm/bridge/tfp410: Make symbol tfp410_platform_driver static

On 2/9/2017 8:55 PM, Wei Yongjun wrote: From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/bridge/ti-tfp410.c:223:24: warning: symbol 'tfp410_platform_driver' was not declared. Should it be static? This was queued to drm-misc-next Thanks,

Re: [PATCH -next] drm/bridge/tfp410: Make symbol tfp410_platform_driver static

On 2/9/2017 8:55 PM, Wei Yongjun wrote: From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/bridge/ti-tfp410.c:223:24: warning: symbol 'tfp410_platform_driver' was not declared. Should it be static? This was queued to drm-misc-next Thanks, Archit Signed-off-by: Wei

Re: [PATCH v2 5/6] drm: convert drivers to use drm_of_find_panel_or_bridge

depending on the particular board, so we want to avoid spurious messages. Plus the kernel is not a DT validator. For the bridge drivers: Reviewed-by: Archit Taneja <arch...@codeaurora.org> Thanks, Archit Signed-off-by: Rob Herring <r...@kernel.org> --- v2: - fix wrong node ptr in

Re: [PATCH v2 5/6] drm: convert drivers to use drm_of_find_panel_or_bridge

depending on the particular board, so we want to avoid spurious messages. Plus the kernel is not a DT validator. For the bridge drivers: Reviewed-by: Archit Taneja Thanks, Archit Signed-off-by: Rob Herring --- v2: - fix wrong node ptr in imx-ldb - build fixes in kirin and imx drivers drive

Re: [PATCH v2 3/6] drm: of: introduce drm_of_find_panel_or_bridge

On 02/10/2017 12:35 AM, Rob Herring wrote: Many drivers have a common pattern of searching the OF graph for either an attached panel or bridge and then finding the DRM struct for the panel or bridge. Also, most drivers need to handle deferred probing when the DRM device is not yet

Re: [PATCH v2 3/6] drm: of: introduce drm_of_find_panel_or_bridge

On 02/10/2017 12:35 AM, Rob Herring wrote: Many drivers have a common pattern of searching the OF graph for either an attached panel or bridge and then finding the DRM struct for the panel or bridge. Also, most drivers need to handle deferred probing when the DRM device is not yet

Re: [PATCH v2 4/6] drm: convert drivers to use of_graph_get_remote_node

the kernel is not a DT validator. For the msm and adv75xx driver: Tested by: Archit Taneja <arch...@codeaurora.org> Signed-off-by: Rob Herring <r...@kernel.org> Acked-by: Neil Armstrong <narmstr...@baylibre.com> Acked-by: Philipp Zabel <p.za...@pengutronix.de> Tested-by

Re: [PATCH v2 4/6] drm: convert drivers to use of_graph_get_remote_node

the kernel is not a DT validator. For the msm and adv75xx driver: Tested by: Archit Taneja Signed-off-by: Rob Herring Acked-by: Neil Armstrong Acked-by: Philipp Zabel Tested-by: Liviu Dudau Tested-by: Eric Anholt --- v2: - tilcdc fix (Jyri Sarha) - Dropped an incomplete meson change. drive

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

On 02/01/2017 05:51 PM, Peter Senna Tschudin wrote: On 01 February, 2017 12:35 CET, Daniel Vetter <dan...@ffwll.ch> wrote: On Wed, Feb 01, 2017 at 10:58:43AM +, Peter Senna Tschudin wrote: Hi Archit, On 01 February, 2017 10:44 CET, Archit Taneja <arch...@codeaurora.o

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

On 02/01/2017 05:51 PM, Peter Senna Tschudin wrote: On 01 February, 2017 12:35 CET, Daniel Vetter wrote: On Wed, Feb 01, 2017 at 10:58:43AM +, Peter Senna Tschudin wrote: Hi Archit, On 01 February, 2017 10:44 CET, Archit Taneja wrote: On 01/30/2017 10:35 PM, Jani Nikula wrote

Re: [PATCH 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

;airl...@linux.ie> Cc: Thierry Reding <tred...@nvidia.com> Cc: Thierry Reding <thierry.red...@gmail.com> Cc: Archit Taneja <arch...@codeaurora.org> Cc: Enric Balletbo <enric.balle...@collabora.com> Signed-off-by: Peter Senna Tschudin <peter.se...@collabora.c

Re: [PATCH 3/4] drm/bridge: Drivers for megachips-stdpxxxx-ge-b850v3-fw (LVDS-DP++)

Herring Cc: Fabio Estevam Cc: David Airlie Cc: Thierry Reding Cc: Thierry Reding Cc: Archit Taneja Cc: Enric Balletbo Signed-off-by: Peter Senna Tschudin --- drivers/gpu/drm/bridge/Kconfig | 11 + drivers/gpu/drm/bridge/Makefile| 1 + .../drm/bridge/me

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

On 01/30/2017 10:35 PM, Jani Nikula wrote: On Sat, 28 Jan 2017, Peter Senna Tschudin <peter.se...@collabora.com> wrote: On Thu, Jan 05, 2017 at 01:18:47PM +0530, Archit Taneja wrote: Hi Archit, Thank you for the comments! [...] + total_size = (block[EDID_EXT_BLOCK_CN

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

On 01/30/2017 10:35 PM, Jani Nikula wrote: On Sat, 28 Jan 2017, Peter Senna Tschudin wrote: On Thu, Jan 05, 2017 at 01:18:47PM +0530, Archit Taneja wrote: Hi Archit, Thank you for the comments! [...] + total_size = (block[EDID_EXT_BLOCK_CNT] + 1) * EDID_LENGTH

Re: [PATCH] dt-bindings: display: move ANX7814 and SiI8620 bridge bindings

On 01/31/2017 11:24 PM, Rob Herring wrote: A few bindings snuck into bindings/video/bridge since consolidating everything under bindings/display/bridge/. Move them to the correct spot. Cc: Andrzej Hajda <a.ha...@samsung.com> Cc: Archit Taneja <arch...@codeaurora.org> Cc: Enri

Re: [PATCH] dt-bindings: display: move ANX7814 and SiI8620 bridge bindings

On 01/31/2017 11:24 PM, Rob Herring wrote: A few bindings snuck into bindings/video/bridge since consolidating everything under bindings/display/bridge/. Move them to the correct spot. Cc: Andrzej Hajda Cc: Archit Taneja Cc: Enric Balletbo i Serra Cc: Thierry Reding Signed-off-by: Rob

Re: [PATCH] drm: bridge: dw-hdmi: fix building without CONFIG_OF

On 01/23/2017 05:50 PM, Arnd Bergmann wrote: The of_node member in struct drm_bridge is hidden when CONFIG_OF is disabled, causing a build error: drivers/gpu/drm/bridge/dw-hdmi.c: In function '__dw_hdmi_probe': drivers/gpu/drm/bridge/dw-hdmi.c:2063:14: error: 'struct drm_bridge' has no

Re: [PATCH] drm: bridge: dw-hdmi: fix building without CONFIG_OF

On 01/23/2017 05:50 PM, Arnd Bergmann wrote: The of_node member in struct drm_bridge is hidden when CONFIG_OF is disabled, causing a build error: drivers/gpu/drm/bridge/dw-hdmi.c: In function '__dw_hdmi_probe': drivers/gpu/drm/bridge/dw-hdmi.c:2063:14: error: 'struct drm_bridge' has no

Re: [PATCH 0/6 v4] adv7511 EDID probing improvements

weaked connector.status assignment to avoid race, as suggested by Laurent * Reworked the __adv7511_power_on helpers to avoid calling regcache_sync in the EDID probe path * Added new patch to set EDID_I2C_ADDR register before doing EDID read. Cc: David Airlie <airl...@linux.ie> Cc: Archit

Re: [PATCH 0/6 v4] adv7511 EDID probing improvements

race, as suggested by Laurent * Reworked the __adv7511_power_on helpers to avoid calling regcache_sync in the EDID probe path * Added new patch to set EDID_I2C_ADDR register before doing EDID read. Cc: David Airlie Cc: Archit Taneja Cc: Wolfram Sang Cc: Lars-Peter Clausen Cc: Laurent

Re: [PATCH 0/5 v3] adv7511 EDID probing improvements

On 01/12/2017 05:36 AM, John Stultz wrote: On Wed, Jan 11, 2017 at 12:48 AM, Archit Taneja <arch...@codeaurora.org> wrote: Hi, On 01/04/2017 01:11 AM, John Stultz wrote: Hope everyone had a good newyears! Wanted to re-send out v3 of this patch set improving the EDID probing on the a

Re: [PATCH 0/5 v3] adv7511 EDID probing improvements

On 01/12/2017 05:36 AM, John Stultz wrote: On Wed, Jan 11, 2017 at 12:48 AM, Archit Taneja wrote: Hi, On 01/04/2017 01:11 AM, John Stultz wrote: Hope everyone had a good newyears! Wanted to re-send out v3 of this patch set improving the EDID probing on the adv7511 used on HiKey

Re: [PATCH 0/5 v3] adv7511 EDID probing improvements

-john New in v3: * Addressed naming improvements and drm_kms_helper_hotplug_event usage corrections as suggested by Laurent. Cc: David Airlie <airl...@linux.ie> Cc: Archit Taneja <arch...@codeaurora.org> Cc: Wolfram Sang <wsa+rene...@sang-engineering.com> Cc: Lars-Peter Clause

Re: [PATCH 0/5 v3] adv7511 EDID probing improvements

-john New in v3: * Addressed naming improvements and drm_kms_helper_hotplug_event usage corrections as suggested by Laurent. Cc: David Airlie Cc: Archit Taneja Cc: Wolfram Sang Cc: Lars-Peter Clausen Cc: Laurent Pinchart Cc: dri-de...@lists.freedesktop.org Archit Taneja (1): drm/bridge

Re: [PATCH 2/6] arm64: dts: apq8016-sbc: add support to hdmi audio via adv7533

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: This patch adds support to hdmi audio via adv7533. Tested-by: Archit Taneja <arch...@codeaurora.org> Signed-off-by: Srinivas Kandagatla <srinivas.kandaga...@linaro.org> --- arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 10 +

Re: [PATCH 2/6] arm64: dts: apq8016-sbc: add support to hdmi audio via adv7533

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: This patch adds support to hdmi audio via adv7533. Tested-by: Archit Taneja Signed-off-by: Srinivas Kandagatla --- arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm64

Re: [PATCH 2/4] ARM: dts: sd-600eval: add hdmi support

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: Signed-off-by: Srinivas Kandagatla <srinivas.kandaga...@linaro.org> Reviewed-by: Archit Taneja <arch...@codeaurora.org> --- .../arm/boot/dts/qcom-apq8064-arrow-sd-600eval.dts | 44 ++ 1 file changed, 4

Re: [PATCH 2/4] ARM: dts: sd-600eval: add hdmi support

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: Signed-off-by: Srinivas Kandagatla Reviewed-by: Archit Taneja --- .../arm/boot/dts/qcom-apq8064-arrow-sd-600eval.dts | 44 ++ 1 file changed, 44 insertions(+) diff --git a/arch/arm/boot/dts/qcom-apq8064-arrow-sd

Re: [PATCH 1/4] ARM: dts: move hdmi pinctrl out of board file.

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: This patch moves hdmi pinctrl defination from board file to soc level pinctrl file. If not this pinctrl setup will be duplicated across all the apq8064 based board files. Reviewed-by: Archit Taneja <arch...@codeaurora.org> Sign

Re: [PATCH 1/4] ARM: dts: move hdmi pinctrl out of board file.

On 01/04/2017 07:04 PM, Srinivas Kandagatla wrote: This patch moves hdmi pinctrl defination from board file to soc level pinctrl file. If not this pinctrl setup will be duplicated across all the apq8064 based board files. Reviewed-by: Archit Taneja Signed-off-by: Srinivas Kandagatla

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

gt; CC: David Airlie <airl...@linux.ie> CC: Thierry Reding <tred...@nvidia.com> CC: Thierry Reding <thierry.red...@gmail.com> CC: Archit Taneja <arch...@codeaurora.org> Reviewed-by: Enric Balletbo <enric.balle...@collabora.com> Signed-off-by: Peter Senna Tschudin

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

etter Cc: Enric Balletbo i Serra Cc: Philipp Zabel Cc: Rob Herring Cc: Fabio Estevam CC: David Airlie CC: Thierry Reding CC: Thierry Reding CC: Archit Taneja Reviewed-by: Enric Balletbo Signed-off-by: Peter Senna Tschudin --- drivers/gpu/drm/bridge/Kconfig | 11 + drivers/gpu/d

Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

Hi Sekhar, On 1/2/2017 4:38 PM, Sekhar Nori wrote: Hi Archit, On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote: On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote: THS8135 is a configurable video DAC, but no configuration is actually necessary to make it work. For now use

Re: [PATCH v7 3/5] drm: bridge: add support for TI ths8135

Hi Sekhar, On 1/2/2017 4:38 PM, Sekhar Nori wrote: Hi Archit, On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote: On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote: THS8135 is a configurable video DAC, but no configuration is actually necessary to make it work. For now use

Re: [PATCH] drm/bridge: analogix_dp: set the DPCD600 during disabling the psr

On 12/14/2016 03:30 PM, Sean Paul wrote: On Wed, Dec 14, 2016 at 12:03 AM, Archit Taneja <arch...@codeaurora.org> wrote: Hi, On 12/12/2016 08:28 PM, Sean Paul wrote: On Fri, Dec 9, 2016 at 9:49 PM, Caesar Wang <w...@rock-chips.com> wrote: Look likes, the BOE panel F

Re: [PATCH] drm/bridge: analogix_dp: set the DPCD600 during disabling the psr

On 12/14/2016 03:30 PM, Sean Paul wrote: On Wed, Dec 14, 2016 at 12:03 AM, Archit Taneja wrote: Hi, On 12/12/2016 08:28 PM, Sean Paul wrote: On Fri, Dec 9, 2016 at 9:49 PM, Caesar Wang wrote: Look likes, the BOE panel FW didn't ack the DPCD600 signal from the host device

<    1   2   3   4   5   6   7   8   9   >