Re: [PATCH 0/4] Fix S3 and S4 issues related to fbdev

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

 



On 8/9/2022 10:50, Alex Deucher wrote:
On Tue, Aug 9, 2022 at 11:10 AM Mario Limonciello
<mario.limonciello@xxxxxxx> wrote:

Moving to the generic fbdev helper introduced a regression in S4 support
on Hawaii PRO graphics cards.  A workaround commit was introduced to
avoid this S4 regression until the root cause could be found, but that
workaround commit introduced a completely separate regression on S3 on
a WX3200.

This series reverts all the code to move to generic fbdev helpers to
fix both issues.

I'm not opposed to reverting this, but I fear we'll never get rid of
this old fbdev code.

Yeah, makes sense to me.

What about just applying the workaround for
hawaii as per the attached patch?  There is always the concern that
this will lead to more workarounds, but I'd rather avoid touching the
older asic support as much as possible.

Yup that fixes S3 for me on the WX3200 (tested on top of 040370564b68) and I would expect keeps the W/A for S4 on the Hawaii one working effectively.

Reviewed-and-Tested-by: Mario Limonciello <mario.limonciello@xxxxxxx>


Alex


Mario Limonciello (4):
   Revert "drm/amdgpu/display: disable prefer_shadow for generic fb
     helpers"
   Revert "drm/amdgpu: keep fbdev buffers pinned during suspend"
   Revert "drm/amdgpu: drop amdgpu_display_gem_fb_init()"
   Revert "drm/amdgpu: use generic fb helpers instead of setting up AMD
     own's."

  drivers/gpu/drm/amd/amdgpu/Makefile           |   2 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c    |  12 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c   |  43 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c       |  13 -
  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c        | 388 ++++++++++++++++++
  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c       |  30 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h      |  25 ++
  drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c      |   3 +-
  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |   3 +-
  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |   3 +-
  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |   3 +-
  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |   3 +-
  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |   3 +-
  13 files changed, 457 insertions(+), 74 deletions(-)
  create mode 100644 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c

--
2.34.1





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux