| Message ID | cover.1692884619.git.jani.nikula@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | drm, cec and edid updates | expand |
On Thu, 24 Aug 2023, Jani Nikula <jani.nikula@intel.com> 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(-)
On 31/08/2023 20:51, Jani Nikula wrote: > On Thu, 24 Aug 2023, Jani Nikula <jani.nikula@intel.com> 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 <hverkuil-cisco@xs4all.nl> 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(-) >
On Thu, Aug 31, 2023 at 09:51:24PM +0300, Jani Nikula wrote: > On Thu, 24 Aug 2023, Jani Nikula <jani.nikula@intel.com> 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 <mripard@kernel.org> Maxime
On Fri, 01 Sep 2023, Maxime Ripard <mripard@kernel.org> wrote: > On Thu, Aug 31, 2023 at 09:51:24PM +0300, Jani Nikula wrote: >> On Thu, 24 Aug 2023, Jani Nikula <jani.nikula@intel.com> 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 <mripard@kernel.org> Thanks for all the reviews and acks, pushed the series to drm-intel-next. BR, Jani.