Re: [PATCH v3 0/6] drm/radeon: remove load callback & drm_dev_alloc

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

 



Am 04.07.24 um 06:58 schrieb Hoi Pok Wu:
Thanks a lot for your help Thomas.

On Wed, Jul 3, 2024 at 4:52 AM Thomas Zimmermann <tzimmermann@xxxxxxx> wrote:
Hi

Am 30.06.24 um 18:59 schrieb Wu Hoi Pok:
.load and drm_dev_alloc are deprecated. These patch series aims to
remove them.

v3: Both v1 and v2 sucks. v3 improves greatly on readability.
Very nice, thank you. I tested the patches with Radeon hardware and did
not find issues. AFAICT this can be merged as-is. I think it should go
into the amd trees. Maybe Alex or Christian can pick it up.

Thanks for sticking with it.

Nice work guys. Skimming over it it looks ok to me, but maybe Alex has some more notes.

Regards,
Christian.


Best regards
Thomas

Wu Hoi Pok (6):
    drm/radeon: change variable name "dev" to "ddev" for consistency
    drm/radeon: remove load callback from kms_driver
    drm/radeon: use variable flags as parameter
    drm/radeon: add helper rdev_to_drm(rdev)
    drm/radeon: change rdev->ddev to rdev_to_drm(rdev)
    drm/radeon: change drm_dev_alloc to devm_drm_dev_alloc

   drivers/gpu/drm/radeon/atombios_encoders.c |  2 +-
   drivers/gpu/drm/radeon/cik.c               | 14 ++--
   drivers/gpu/drm/radeon/dce6_afmt.c         |  2 +-
   drivers/gpu/drm/radeon/evergreen.c         | 12 ++--
   drivers/gpu/drm/radeon/ni.c                |  2 +-
   drivers/gpu/drm/radeon/r100.c              | 24 +++----
   drivers/gpu/drm/radeon/r300.c              |  6 +-
   drivers/gpu/drm/radeon/r420.c              |  6 +-
   drivers/gpu/drm/radeon/r520.c              |  2 +-
   drivers/gpu/drm/radeon/r600.c              | 12 ++--
   drivers/gpu/drm/radeon/r600_cs.c           |  2 +-
   drivers/gpu/drm/radeon/r600_dpm.c          |  4 +-
   drivers/gpu/drm/radeon/r600_hdmi.c         |  2 +-
   drivers/gpu/drm/radeon/radeon.h            |  7 +-
   drivers/gpu/drm/radeon/radeon_acpi.c       | 10 +--
   drivers/gpu/drm/radeon/radeon_agp.c        |  2 +-
   drivers/gpu/drm/radeon/radeon_atombios.c   |  2 +-
   drivers/gpu/drm/radeon/radeon_audio.c      |  4 +-
   drivers/gpu/drm/radeon/radeon_combios.c    | 12 ++--
   drivers/gpu/drm/radeon/radeon_device.c     | 13 ++--
   drivers/gpu/drm/radeon/radeon_display.c    | 74 +++++++++++-----------
   drivers/gpu/drm/radeon/radeon_drv.c        | 27 +++++---
   drivers/gpu/drm/radeon/radeon_fbdev.c      | 26 ++++----
   drivers/gpu/drm/radeon/radeon_fence.c      |  8 +--
   drivers/gpu/drm/radeon/radeon_gem.c        |  2 +-
   drivers/gpu/drm/radeon/radeon_i2c.c        |  2 +-
   drivers/gpu/drm/radeon/radeon_ib.c         |  2 +-
   drivers/gpu/drm/radeon/radeon_irq_kms.c    | 12 ++--
   drivers/gpu/drm/radeon/radeon_kms.c        |  8 +--
   drivers/gpu/drm/radeon/radeon_object.c     |  2 +-
   drivers/gpu/drm/radeon/radeon_pm.c         | 20 +++---
   drivers/gpu/drm/radeon/radeon_ring.c       |  2 +-
   drivers/gpu/drm/radeon/radeon_ttm.c        |  6 +-
   drivers/gpu/drm/radeon/rs400.c             |  6 +-
   drivers/gpu/drm/radeon/rs600.c             | 14 ++--
   drivers/gpu/drm/radeon/rs690.c             |  2 +-
   drivers/gpu/drm/radeon/rv515.c             |  4 +-
   drivers/gpu/drm/radeon/rv770.c             |  2 +-
   drivers/gpu/drm/radeon/si.c                |  4 +-
   39 files changed, 184 insertions(+), 179 deletions(-)

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)





[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