Re: [PATCH 0/7] drm/edid and drivers: ELD refactoring

From: Thierry Reding <thierry.reding@gmail.com>
To: Jani Nikula <jani.nikula@intel.com>
Cc: intel-gfx@lists.freedesktop.org,
	"Russell King" <linux@armlinux.org.uk>,
	dri-devel@lists.freedesktop.org,
	"Ben Skeggs" <bskeggs@redhat.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Vincent Abriou" <vincent.abriou@st.com>,
	"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH 0/7] drm/edid and drivers: ELD refactoring
Date: Mon, 6 Nov 2017 15:54:52 +0100	[thread overview]
Message-ID: <20171106145452.GA15647@ulmo> (raw)
In-Reply-To: <cover.1509545641.git.jani.nikula@intel.com>

[-- Attachment #1.1: Type: text/plain, Size: 2708 bytes --]

On Wed, Nov 01, 2017 at 04:20:56PM +0200, Jani Nikula wrote:
> We were recently bitten by drm_edid_to_eld() clearing the connector
> type, and us failing to set it back for DP. Here's a few ELD related
> patches to try to unify ELD handling and make it a bit simpler for
> drivers to get it right.
> 
> Apologies for the massive Cc list; it's the maintainers of all drivers
> that call drm_edid_to_eld().
> 
> I'm open to splitting up patch 6 to driver specific patches as needed,
> but I'd think it would be just fine to merge via drm-misc as-is too.
> 
> BR,
> Jani.
> 
> Cc: Alex Deucher <alexander.deucher@amd.com>
> Cc: Christian König <christian.koenig@amd.com>
> Cc: Archit Taneja <architt@codeaurora.org>
> Cc: Andrzej Hajda <a.hajda@samsung.com>
> Cc: Russell King <linux@armlinux.org.uk>
> Cc: CK Hu <ck.hu@mediatek.com>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Ben Skeggs <bskeggs@redhat.com>
> Cc: Mark Yao <mark.yao@rock-chips.com>
> Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Cc: Vincent Abriou <vincent.abriou@st.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Eric Anholt <eric@anholt.net>
> 
> 
> Jani Nikula (7):
>   drm/edid: use macros for ELD offsets and values
>   drm/edid: set ELD connector type in drm_edid_to_eld()
>   drm/i915: remove redundant ELD connector type update
>   drm/edid: abstract connector ELD clearing
>   drm/edid: build ELD in drm_add_edid_modes()
>   drm/drivers: drop redundant drm_edid_to_eld() calls
>   drm/edid: make drm_edid_to_eld() static
> 
>  drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c |  1 -
>  drivers/gpu/drm/bridge/analogix-anx78xx.c      |  2 -
>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.c      |  2 -
>  drivers/gpu/drm/drm_edid.c                     | 70 +++++++++++++++-----------
>  drivers/gpu/drm/i2c/tda998x_drv.c              |  1 -
>  drivers/gpu/drm/i915/intel_dp.c                |  1 -
>  drivers/gpu/drm/i915/intel_modes.c             | 18 -------
>  drivers/gpu/drm/mediatek/mtk_hdmi.c            |  1 -
>  drivers/gpu/drm/nouveau/nv50_display.c         |  5 +-
>  drivers/gpu/drm/radeon/radeon_connectors.c     |  1 -
>  drivers/gpu/drm/radeon/radeon_dp_mst.c         |  1 -
>  drivers/gpu/drm/rockchip/cdn-dp-core.c         |  4 +-
>  drivers/gpu/drm/sti/sti_hdmi.c                 |  1 -
>  drivers/gpu/drm/tegra/output.c                 |  1 -
>  drivers/gpu/drm/vc4/vc4_hdmi.c                 |  1 -
>  include/drm/drm_edid.h                         |  1 -
>  include/drm/drm_modeset_helper_vtables.h       |  3 --
>  17 files changed, 44 insertions(+), 70 deletions(-)

The series:

Acked-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx