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

[Claudio Suarez <cssk@net-c.es>]

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@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@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(-)
> > > >
> > >
> >
> >