| Message ID | 20210107205321.166698-1-Rodrigo.Siqueira@amd.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Revert "drm/amd/display: Expose new CRC window property" and changes associated with this commit | expand |
On Thu, Jan 7, 2021 at 3:53 PM Rodrigo Siqueira <Rodrigo.Siqueira@amd.com> wrote: > > Hi, > > A couple of weeks ago, Daniel highlighted [1] some issue related to a > patch entitle "drm/amd/display: Expose new CRC window property". After > discussion, we realize that we can revert that patch because we will > need to create a debugfs or full UAPI for CRC soon, which will make this > code obsolete. We got two other patches related to this same code; for > this reason, this patchset reverts all changes associated with that > specific commit. > > Cc: Wayne Lin <Wayne.Lin@amd.com> > Cc: Alexander Deucher <Alexander.Deucher@amd.com> > Cc: Harry Wentland <Harry.Wentland@amd.com> > Cc: Roman Li <Roman.Li@amd.com> > Cc: Bindu R <Bindu.R@amd.com> > Cc: Daniel Vetter <daniel@ffwll.ch> Series is: Acked-by: Alex Deucher <alexander.deucher@amd.com> > > 1. https://www.spinics.net/lists/dri-devel/msg283767.html > > Thanks > > Rodrigo Siqueira (3): > Revert "drm/amd/display: Fix unused variable warning" > Revert "drm/amdgpu/disply: fix documentation warnings in display > manager" > Revert "drm/amd/display: Expose new CRC window property" > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 142 ++---------------- > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 38 ----- > .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 56 +------ > .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.h | 5 +- > drivers/gpu/drm/amd/display/dc/core/dc_link.c | 2 + > 5 files changed, 14 insertions(+), 229 deletions(-) > > -- > 2.25.1 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
On Thu, Jan 07, 2021 at 03:53:18PM -0500, Rodrigo Siqueira wrote: > Hi, > > A couple of weeks ago, Daniel highlighted [1] some issue related to a > patch entitle "drm/amd/display: Expose new CRC window property". After > discussion, we realize that we can revert that patch because we will > need to create a debugfs or full UAPI for CRC soon, which will make this > code obsolete. We got two other patches related to this same code; for > this reason, this patchset reverts all changes associated with that > specific commit. > > Cc: Wayne Lin <Wayne.Lin@amd.com> > Cc: Alexander Deucher <Alexander.Deucher@amd.com> > Cc: Harry Wentland <Harry.Wentland@amd.com> > Cc: Roman Li <Roman.Li@amd.com> > Cc: Bindu R <Bindu.R@amd.com> > Cc: Daniel Vetter <daniel@ffwll.ch> > > 1. https://www.spinics.net/lists/dri-devel/msg283767.html Thanks for taking care of this. Since it's not super pressing I'll leave it to Alex for merging with the usual -fixes pull request train next week. Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> Cheers, Daniel > > Thanks > > Rodrigo Siqueira (3): > Revert "drm/amd/display: Fix unused variable warning" > Revert "drm/amdgpu/disply: fix documentation warnings in display > manager" > Revert "drm/amd/display: Expose new CRC window property" > > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 142 ++---------------- > .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 38 ----- > .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 56 +------ > .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.h | 5 +- > drivers/gpu/drm/amd/display/dc/core/dc_link.c | 2 + > 5 files changed, 14 insertions(+), 229 deletions(-) > > -- > 2.25.1 >
Hi, A couple of weeks ago, Daniel highlighted [1] some issue related to a patch entitle "drm/amd/display: Expose new CRC window property". After discussion, we realize that we can revert that patch because we will need to create a debugfs or full UAPI for CRC soon, which will make this code obsolete. We got two other patches related to this same code; for this reason, this patchset reverts all changes associated with that specific commit. Cc: Wayne Lin <Wayne.Lin@amd.com> Cc: Alexander Deucher <Alexander.Deucher@amd.com> Cc: Harry Wentland <Harry.Wentland@amd.com> Cc: Roman Li <Roman.Li@amd.com> Cc: Bindu R <Bindu.R@amd.com> Cc: Daniel Vetter <daniel@ffwll.ch> 1. https://www.spinics.net/lists/dri-devel/msg283767.html Thanks Rodrigo Siqueira (3): Revert "drm/amd/display: Fix unused variable warning" Revert "drm/amdgpu/disply: fix documentation warnings in display manager" Revert "drm/amd/display: Expose new CRC window property" .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 142 ++---------------- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 38 ----- .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 56 +------ .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.h | 5 +- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 2 + 5 files changed, 14 insertions(+), 229 deletions(-)