2022-12-03 10:25:27

by Cai Huoqing

[permalink] [raw]
Subject: [PATCH linux-next v2 00/10] drm: Remove some obsolete drivers(tdfx, mga, i810, savage, r128, sis, via)

Commit 399516ab0fee ("MAINTAINERS: Add a bunch of legacy (UMS) DRM drivers")
marked these drivers obsolete 7 years ago.
And the mesa UMD of these drm drivers already in deprecated list
in the link: https://docs.mesa3d.org/systems.html

3dfx Glide-->driver/gpu/drm/tdfx
Matrox-->driver/gpu/drm/mga
Intel i810-->driver/gpu/drm/i810
S3 Savage-->drivers/gpu/drm/savage
ATI Rage 128->drivers/gpu/drm/r128
Silicon Integrated Systems->drivers/gpu/drm/sis
VIA Unichrome->drivers/gpu/drm/via

v1->v2:
1.Add drm via driver to the patchset.
2.Remove related drm_pciids.
3.Remove related drm uapi header files.
4.split to series avoid large patch email.

Cai Huoqing (10):
drm: Remove the obsolete driver-i810
drm: Remove the obsolete driver-mga
drm: Remove the obsolete driver-r128
drm: Remove the obsolete driver-savage
drm: Remove the obsolete driver-sis
drm: Remove the obsolete driver-tdfx
drm: Remove the obsolete driver-via
drm: Add comments to Kconfig
drm: Remove some obsolete drm pciids(tdfx, mga, i810, savage, r128,
sis, via)
MAINTAINERS: Remove some obsolete drivers info(tdfx, mga, i810,
savage, r128, sis)

MAINTAINERS | 29 -
drivers/gpu/drm/Kconfig | 59 +-
drivers/gpu/drm/Makefile | 7 -
drivers/gpu/drm/i810/Makefile | 8 -
drivers/gpu/drm/i810/i810_dma.c | 1266 ---------
drivers/gpu/drm/i810/i810_drv.c | 101 -
drivers/gpu/drm/i810/i810_drv.h | 246 --
drivers/gpu/drm/mga/Makefile | 11 -
drivers/gpu/drm/mga/mga_dma.c | 1168 --------
drivers/gpu/drm/mga/mga_drv.c | 104 -
drivers/gpu/drm/mga/mga_drv.h | 685 -----
drivers/gpu/drm/mga/mga_ioc32.c | 197 --
drivers/gpu/drm/mga/mga_irq.c | 169 --
drivers/gpu/drm/mga/mga_state.c | 1099 --------
drivers/gpu/drm/mga/mga_warp.c | 167 --
drivers/gpu/drm/r128/Makefile | 10 -
drivers/gpu/drm/r128/ati_pcigart.c | 228 --
drivers/gpu/drm/r128/ati_pcigart.h | 31 -
drivers/gpu/drm/r128/r128_cce.c | 944 -------
drivers/gpu/drm/r128/r128_drv.c | 116 -
drivers/gpu/drm/r128/r128_drv.h | 544 ----
drivers/gpu/drm/r128/r128_ioc32.c | 199 --
drivers/gpu/drm/r128/r128_irq.c | 118 -
drivers/gpu/drm/r128/r128_state.c | 1641 -----------
drivers/gpu/drm/savage/Makefile | 9 -
drivers/gpu/drm/savage/savage_bci.c | 1082 --------
drivers/gpu/drm/savage/savage_drv.c | 91 -
drivers/gpu/drm/savage/savage_drv.h | 580 ----
drivers/gpu/drm/savage/savage_state.c | 1169 --------
drivers/gpu/drm/sis/Makefile | 10 -
drivers/gpu/drm/sis/sis_drv.c | 143 -
drivers/gpu/drm/sis/sis_drv.h | 80 -
drivers/gpu/drm/sis/sis_mm.c | 363 ---
drivers/gpu/drm/tdfx/Makefile | 8 -
drivers/gpu/drm/tdfx/tdfx_drv.c | 90 -
drivers/gpu/drm/tdfx/tdfx_drv.h | 47 -
drivers/gpu/drm/via/Makefile | 8 -
drivers/gpu/drm/via/via_3d_reg.h | 1771 ------------
drivers/gpu/drm/via/via_dri1.c | 3630 -------------------------
include/drm/drm_pciids.h | 112 -
include/uapi/drm/i810_drm.h | 292 --
include/uapi/drm/mga_drm.h | 429 ---
include/uapi/drm/r128_drm.h | 336 ---
include/uapi/drm/savage_drm.h | 220 --
include/uapi/drm/sis_drm.h | 77 -
include/uapi/drm/via_drm.h | 282 --
46 files changed, 1 insertion(+), 19975 deletions(-)
delete mode 100644 drivers/gpu/drm/i810/Makefile
delete mode 100644 drivers/gpu/drm/i810/i810_dma.c
delete mode 100644 drivers/gpu/drm/i810/i810_drv.c
delete mode 100644 drivers/gpu/drm/i810/i810_drv.h
delete mode 100644 drivers/gpu/drm/mga/Makefile
delete mode 100644 drivers/gpu/drm/mga/mga_dma.c
delete mode 100644 drivers/gpu/drm/mga/mga_drv.c
delete mode 100644 drivers/gpu/drm/mga/mga_drv.h
delete mode 100644 drivers/gpu/drm/mga/mga_ioc32.c
delete mode 100644 drivers/gpu/drm/mga/mga_irq.c
delete mode 100644 drivers/gpu/drm/mga/mga_state.c
delete mode 100644 drivers/gpu/drm/mga/mga_warp.c
delete mode 100644 drivers/gpu/drm/r128/Makefile
delete mode 100644 drivers/gpu/drm/r128/ati_pcigart.c
delete mode 100644 drivers/gpu/drm/r128/ati_pcigart.h
delete mode 100644 drivers/gpu/drm/r128/r128_cce.c
delete mode 100644 drivers/gpu/drm/r128/r128_drv.c
delete mode 100644 drivers/gpu/drm/r128/r128_drv.h
delete mode 100644 drivers/gpu/drm/r128/r128_ioc32.c
delete mode 100644 drivers/gpu/drm/r128/r128_irq.c
delete mode 100644 drivers/gpu/drm/r128/r128_state.c
delete mode 100644 drivers/gpu/drm/savage/Makefile
delete mode 100644 drivers/gpu/drm/savage/savage_bci.c
delete mode 100644 drivers/gpu/drm/savage/savage_drv.c
delete mode 100644 drivers/gpu/drm/savage/savage_drv.h
delete mode 100644 drivers/gpu/drm/savage/savage_state.c
delete mode 100644 drivers/gpu/drm/sis/Makefile
delete mode 100644 drivers/gpu/drm/sis/sis_drv.c
delete mode 100644 drivers/gpu/drm/sis/sis_drv.h
delete mode 100644 drivers/gpu/drm/sis/sis_mm.c
delete mode 100644 drivers/gpu/drm/tdfx/Makefile
delete mode 100644 drivers/gpu/drm/tdfx/tdfx_drv.c
delete mode 100644 drivers/gpu/drm/tdfx/tdfx_drv.h
delete mode 100644 drivers/gpu/drm/via/Makefile
delete mode 100644 drivers/gpu/drm/via/via_3d_reg.h
delete mode 100644 drivers/gpu/drm/via/via_dri1.c
delete mode 100644 include/uapi/drm/i810_drm.h
delete mode 100644 include/uapi/drm/mga_drm.h
delete mode 100644 include/uapi/drm/r128_drm.h
delete mode 100644 include/uapi/drm/savage_drm.h
delete mode 100644 include/uapi/drm/sis_drm.h
delete mode 100644 include/uapi/drm/via_drm.h

--
2.25.1


2022-12-08 17:32:08

by Emil Velikov

[permalink] [raw]
Subject: Re: [PATCH linux-next v2 00/10] drm: Remove some obsolete drivers(tdfx, mga, i810, savage, r128, sis, via)

On Sat, 3 Dec 2022 at 09:45, Cai Huoqing <[email protected]> wrote:
>
> Commit 399516ab0fee ("MAINTAINERS: Add a bunch of legacy (UMS) DRM drivers")
> marked these drivers obsolete 7 years ago.

As the author of said patch - calling them obsolete was a bad decision
on my part.
They are effectively orphan with no maintainer and no other driver(s)
that provide the same, or equivalent functionality.

As a DRM newcomer I would recommend sticking to smaller and more
tangible tasks. The TODO list has plenty of varying complexity.

[1] https://www.kernel.org/doc/html/latest/gpu/todo.html