Hi Ville, Thank you for the patch.
On Tuesday, 20 November 2018 18:13:42 EET Ville Syrjala wrote: > From: Ville Syrjälä <[email protected]> > > Make life easier for drivers by simply passing the connector > to drm_hdmi_avi_infoframe_from_display_mode() and > drm_hdmi_avi_infoframe_quant_range(). That way drivers don't > need to worry about is_hdmi2_sink mess. While this is good for display controller drivers, the change isn't great for bridge drivers. Down the road we're looking at moving connector support out of the bridge drivers. Adding an additional dependency to connectors in the bridges will make that more difficult. Ideally bridges should retrieve the information from their sink, regardless of whether it is a connector or another bridge. Please see below for an additional comment. > Cc: Alex Deucher <[email protected]> > Cc: "Christian König" <[email protected]> > Cc: "David (ChunMing) Zhou" <[email protected]> > Cc: Archit Taneja <[email protected]> > Cc: Andrzej Hajda <[email protected]> > Cc: Laurent Pinchart <[email protected]> > Cc: Inki Dae <[email protected]> > Cc: Joonyoung Shim <[email protected]> > Cc: Seung-Woo Kim <[email protected]> > Cc: Kyungmin Park <[email protected]> > Cc: Russell King <[email protected]> > Cc: CK Hu <[email protected]> > Cc: Philipp Zabel <[email protected]> > Cc: Rob Clark <[email protected]> > Cc: Ben Skeggs <[email protected]> > Cc: Tomi Valkeinen <[email protected]> > Cc: Sandy Huang <[email protected]> > Cc: "Heiko Stübner" <[email protected]> > Cc: Benjamin Gaignard <[email protected]> > Cc: Vincent Abriou <[email protected]> > Cc: Thierry Reding <[email protected]> > Cc: Eric Anholt <[email protected]> > Cc: Shawn Guo <[email protected]> > Cc: Ilia Mirkin <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 2 +- > drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 2 +- > drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 3 ++- > drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 2 +- > drivers/gpu/drm/bridge/analogix-anx78xx.c | 5 ++-- > drivers/gpu/drm/bridge/sii902x.c | 3 ++- > drivers/gpu/drm/bridge/sil-sii8620.c | 3 +-- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 3 ++- > drivers/gpu/drm/drm_edid.c | 33 ++++++++++++++--------- > drivers/gpu/drm/exynos/exynos_hdmi.c | 3 ++- > drivers/gpu/drm/i2c/tda998x_drv.c | 3 ++- > drivers/gpu/drm/i915/intel_hdmi.c | 14 +++++----- > drivers/gpu/drm/i915/intel_lspcon.c | 15 ++++++----- > drivers/gpu/drm/i915/intel_sdvo.c | 10 ++++--- > drivers/gpu/drm/mediatek/mtk_hdmi.c | 3 ++- > drivers/gpu/drm/msm/hdmi/hdmi_bridge.c | 3 ++- > drivers/gpu/drm/nouveau/dispnv50/disp.c | 7 +++-- > drivers/gpu/drm/omapdrm/omap_encoder.c | 5 ++-- > drivers/gpu/drm/radeon/radeon_audio.c | 2 +- > drivers/gpu/drm/rockchip/inno_hdmi.c | 4 ++- > drivers/gpu/drm/sti/sti_hdmi.c | 3 ++- > drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c | 3 ++- > drivers/gpu/drm/tegra/hdmi.c | 3 ++- > drivers/gpu/drm/tegra/sor.c | 3 ++- > drivers/gpu/drm/vc4/vc4_hdmi.c | 11 +++++--- > drivers/gpu/drm/zte/zx_hdmi.c | 4 ++- > include/drm/drm_edid.h | 8 +++--- > 27 files changed, 94 insertions(+), 66 deletions(-) For dw-hdmi and omapdrm, Reviewed-by: Laurent Pinchart <[email protected]> -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
