Re: [PATCH v2 0/4] Revert "drm/amd/display: Expose new CRC window property" and changes associated with this commit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Siqueira.

Series is:
Reviewed-by: Wayne Lin<Wayne.Lin@xxxxxxx>

On 1/12/21 10:55 PM, Rodrigo Siqueira wrote:
Hi,

In the V1, Wayne pointed out two problems:

1. The revert patch included one extra line that does not belong to it;
2. The original patch also had other fixes in the same commit;

I removed the extra line from the reverted patch for tackling this
issue, and I added one additional patch to this series that includes the
other fix requested by Wayne.

Thanks

Original cover letter:
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.

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"

Wayne Lin (1):
   drm/amd/display: Fix to be able to stop crc calculation

  .../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 |  54 +------
  .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.h |   5 +-
  4 files changed, 11 insertions(+), 228 deletions(-)

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux