[PATCH 0/3] drm/amdgpu replace drm_detect_hdmi_monitor() with drm_display_info.is_hdmi

[Alex Deucher]

yeah, sorry about that.  I was mixing these up with another patch set.
I've applied them.  Sorry for the delay.

Alex

On Tue, Dec 7, 2021 at 8:45 AM Claudio Suarez <cssk at net-c.es> wrote:
>
> On Mon, Dec 06, 2021 at 05:15:11PM -0500, Alex Deucher wrote:
> > Do you have push rights to drm-misc?  IIRC, these patches depend on
> > the is_hdmi changes that recently went into drm-misc, so these patches
> > should probably go upstream via drm-misc rather than amdgpu.
>
> Sorry, I have missed to answer the second part.
> This patch depends on nothing. The only dependency is
> a92d083d08b0 (drm/edid: Add flag to drm_display_info to identify HDMI sinks, 2020-02-26)
> and it is in the kernel source since February 2020.
>
> These patches are not uploaded to misc:
> https://cgit.freedesktop.org/drm/drm-misc/tree/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c
> I don't think these patches will go upstream via drm-misc.
> The only post was to this list.
>
> I thought this is the list for pathches to amd drivers.
> Hope I am not wrong.
>
> Best regards,
> Claudio Suarez
>
>
> >
> > Alex
> >
> > On Mon, Dec 6, 2021 at 5:21 AM Claudio Suarez <cssk at net-c.es> wrote:
> > >
> > >
> > > Hello,
> > >
> > > These patches
> > >
> > > https://www.mail-archive.com/amd-gfx@lists.freedesktop.org/msg69247.html
> > >
> > > are not uploaded to the linux source. I suppose I have to ping here.
> > >
> > > Best regards,
> > > Claudio Suarez.
> > >
> > >
> > >
> > >
> > > On Mon, Oct 18, 2021 at 09:37:13AM -0400, Harry Wentland wrote:
> > > > On 2021-10-17 07:34, Claudio Suarez wrote:
> > > > >
> > > > > From the TODO list Documentation/gpu/todo.rst
> > > > > -----------------------
> > > > > Once EDID is parsed, the monitor HDMI support information is available through
> > > > > drm_display_info.is_hdmi. Many drivers still call drm_detect_hdmi_monitor() to
> > > > > retrieve the same information, which is less efficient.
> > > > >
> > > > > Audit each individual driver calling drm_detect_hdmi_monitor() and switch to
> > > > > drm_display_info.is_hdmi if applicable.
> > > > > -----------------------
> > > > > The task is divided in three small patches. The last patch depends on the
> > > > > first one.
> > > > >
> > > >
> > > > Thanks.
> > > >
> > > > This series is
> > > > Reviewed-by: Harry Wentland <harry.wentland at amd.com>
> > > >
> > > > Harry
> > > >
> > > > >
> > > > >
> > > > > Claudio Suarez (3):
> > > > >   drm/amdgpu: update drm_display_info correctly when the edid is read
> > > > >   drm/amdgpu: use drm_edid_get_monitor_name() instead of duplicating the
> > > > >     code
> > > > >   drm/amdgpu: replace drm_detect_hdmi_monitor() with
> > > > >     drm_display_info.is_hdmi
> > > > >
> > > > >  .../gpu/drm/amd/amdgpu/amdgpu_connectors.c    | 17 +++++----
> > > > >  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c   |  2 +-
> > > > >  drivers/gpu/drm/amd/amdgpu/amdgpu_encoders.c  |  4 +-
> > > > >  .../gpu/drm/amd/amdgpu/atombios_encoders.c    |  6 +--
> > > > >  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  3 +-
> > > > >  .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 37 +++++--------------
> > > > >  drivers/gpu/drm/amd/display/dc/core/dc.c      |  2 +-
> > > > >  drivers/gpu/drm/amd/display/dc/dm_helpers.h   |  2 +-
> > > > >  8 files changed, 29 insertions(+), 44 deletions(-)
> > > > >
> > > >
> > >
> > >
>
>