[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_NtTEMGka7vJFZLvOi7UFQkwKex_SL5Hunt61HJW1KFpw@mail.gmail.com>
Date: Wed, 1 Apr 2020 14:32:38 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: Lyude Paul <lyude@...hat.com>
Cc: amd-gfx list <amd-gfx@...ts.freedesktop.org>,
Maling list - DRI developers
<dri-devel@...ts.freedesktop.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
Leo Li <sunpeng.li@....com>,
Pankaj Bharadiya <pankaj.laxminarayan.bharadiya@...el.com>,
David Francis <david.francis@....com>,
Wenjing Liu <wenjing.liu@....com>,
LKML <linux-kernel@...r.kernel.org>,
Nicholas Kazlauskas <nicholas.kazlauskas@....com>,
David Airlie <airlied@...ux.ie>,
Alex Deucher <alexander.deucher@....com>,
Rodrigo Siqueira <rodrigo.siqueira@....com>,
Mikita Lipski <mikita.lipski@....com>,
Bhawanpreet Lakha <bhawanpreet.lakha@....com>,
Christian König <christian.koenig@....com>
Subject: Re: [PATCH 0/4] drm/dp_mst: Remove ->destroy_connector() callback
On Tue, Mar 31, 2020 at 4:58 PM Lyude Paul <lyude@...hat.com> wrote:
>
> This finishes up the work that Pankaj Bharadiya started in:
>
> https://patchwork.freedesktop.org/series/74412/
>
> And allows us to entirely remove ->destroy_connector()
>
> Lyude Paul (4):
> drm/amd/amdgpu_dm/mst: Remove unneeded edid assignment when destroying
> connectors
> drm/amd/amdgpu_dm/mst: Remove ->destroy_connector() callback
> drm/amd/amdgpu_dm/mst: Stop printing extra messages in
> dm_dp_add_mst_connector()
> drm/dp_mst: Remove drm_dp_mst_topology_cbs.destroy_connector
I noticed this as well when I was sorting out the load and unload
callback removal. Thanks for finishing this up. This series looks
good to me, assuming none of the display folks have any concerns:
Reviewed-by: Alex Deucher <alexander.deucher@....com>
>
> .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 45 +++++--------------
> drivers/gpu/drm/drm_dp_mst_topology.c | 16 ++-----
> include/drm/drm_dp_mst_helper.h | 2 -
> 3 files changed, 15 insertions(+), 48 deletions(-)
>
> --
> 2.25.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
Powered by blists - more mailing lists