Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates
On Fri, 01 Sep 2023, Maxime Ripard wrote: > On Thu, Aug 31, 2023 at 09:51:24PM +0300, Jani Nikula wrote: >> On Thu, 24 Aug 2023, Jani Nikula wrote: >> > Avoid accessing the raw edid directly. Pre-parse the source physical >> > address during normal EDID parsing and use that for CEC. >> > >> > Jani Nikula (6): >> > drm/edid: add drm_edid_is_digital() >> > drm/i915/display: use drm_edid_is_digital() >> > drm/edid: parse source physical address >> > drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid >> > drm/i915/cec: switch to setting physical address directly >> >> Maarten, Maxime, Thomas, ack for merging patches 1, 3 and 4 via via >> drm-intel? > > Acked-by: Maxime Ripard Thanks for all the reviews and acks, pushed the series to drm-intel-next. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center
Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates
On Thu, Aug 31, 2023 at 09:51:24PM +0300, Jani Nikula wrote: > On Thu, 24 Aug 2023, Jani Nikula wrote: > > Avoid accessing the raw edid directly. Pre-parse the source physical > > address during normal EDID parsing and use that for CEC. > > > > Jani Nikula (6): > > drm/edid: add drm_edid_is_digital() > > drm/i915/display: use drm_edid_is_digital() > > drm/edid: parse source physical address > > drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid > > drm/i915/cec: switch to setting physical address directly > > Maarten, Maxime, Thomas, ack for merging patches 1, 3 and 4 via via > drm-intel? Acked-by: Maxime Ripard Maxime signature.asc Description: PGP signature
Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates
On 31/08/2023 20:51, Jani Nikula wrote: > On Thu, 24 Aug 2023, Jani Nikula wrote: >> Avoid accessing the raw edid directly. Pre-parse the source physical >> address during normal EDID parsing and use that for CEC. >> >> Jani Nikula (6): >> drm/edid: add drm_edid_is_digital() >> drm/i915/display: use drm_edid_is_digital() >> drm/edid: parse source physical address >> drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid >> drm/i915/cec: switch to setting physical address directly > > Maarten, Maxime, Thomas, ack for merging patches 1, 3 and 4 via via > drm-intel? > >> media: cec: core: add note about *_from_edid() function usage in drm > > Hans, while there's no build dependency here, I think it would make > sense to merge this together with patches 3 and 4. Ack for merging via > drm-intel? That's fine, it makes sense to do that. If you need it, for this series: Acked-by: Hans Verkuil Regards, Hans > > Thanks, > Jani. > > >> >> drivers/gpu/drm/display/drm_dp_cec.c | 22 +++--- >> drivers/gpu/drm/drm_edid.c| 22 -- >> drivers/gpu/drm/i915/display/intel_crt.c | 11 --- >> drivers/gpu/drm/i915/display/intel_dp.c | 7 ++- >> drivers/gpu/drm/i915/display/intel_hdmi.c | 8 +++- >> drivers/gpu/drm/i915/display/intel_sdvo.c | 7 ++- >> drivers/media/cec/core/cec-adap.c | 4 >> drivers/media/cec/core/cec-notifier.c | 4 >> include/drm/display/drm_dp_helper.h | 6 ++ >> include/drm/drm_connector.h | 8 >> include/drm/drm_edid.h| 1 + >> 11 files changed, 73 insertions(+), 27 deletions(-) >
Re: [Intel-gfx] [PATCH 0/6] drm, cec and edid updates
On Thu, 24 Aug 2023, Jani Nikula wrote: > Avoid accessing the raw edid directly. Pre-parse the source physical > address during normal EDID parsing and use that for CEC. > > Jani Nikula (6): > drm/edid: add drm_edid_is_digital() > drm/i915/display: use drm_edid_is_digital() > drm/edid: parse source physical address > drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid > drm/i915/cec: switch to setting physical address directly Maarten, Maxime, Thomas, ack for merging patches 1, 3 and 4 via via drm-intel? > media: cec: core: add note about *_from_edid() function usage in drm Hans, while there's no build dependency here, I think it would make sense to merge this together with patches 3 and 4. Ack for merging via drm-intel? Thanks, Jani. > > drivers/gpu/drm/display/drm_dp_cec.c | 22 +++--- > drivers/gpu/drm/drm_edid.c| 22 -- > drivers/gpu/drm/i915/display/intel_crt.c | 11 --- > drivers/gpu/drm/i915/display/intel_dp.c | 7 ++- > drivers/gpu/drm/i915/display/intel_hdmi.c | 8 +++- > drivers/gpu/drm/i915/display/intel_sdvo.c | 7 ++- > drivers/media/cec/core/cec-adap.c | 4 > drivers/media/cec/core/cec-notifier.c | 4 > include/drm/display/drm_dp_helper.h | 6 ++ > include/drm/drm_connector.h | 8 > include/drm/drm_edid.h| 1 + > 11 files changed, 73 insertions(+), 27 deletions(-) -- Jani Nikula, Intel Open Source Graphics Center
[Intel-gfx] [PATCH 0/6] drm, cec and edid updates
Avoid accessing the raw edid directly. Pre-parse the source physical address during normal EDID parsing and use that for CEC. Jani Nikula (6): drm/edid: add drm_edid_is_digital() drm/i915/display: use drm_edid_is_digital() drm/edid: parse source physical address drm/cec: add drm_dp_cec_attach() as the non-edid version of set edid drm/i915/cec: switch to setting physical address directly media: cec: core: add note about *_from_edid() function usage in drm drivers/gpu/drm/display/drm_dp_cec.c | 22 +++--- drivers/gpu/drm/drm_edid.c| 22 -- drivers/gpu/drm/i915/display/intel_crt.c | 11 --- drivers/gpu/drm/i915/display/intel_dp.c | 7 ++- drivers/gpu/drm/i915/display/intel_hdmi.c | 8 +++- drivers/gpu/drm/i915/display/intel_sdvo.c | 7 ++- drivers/media/cec/core/cec-adap.c | 4 drivers/media/cec/core/cec-notifier.c | 4 include/drm/display/drm_dp_helper.h | 6 ++ include/drm/drm_connector.h | 8 include/drm/drm_edid.h| 1 + 11 files changed, 73 insertions(+), 27 deletions(-) -- 2.39.2