summaryrefslogtreecommitdiff
path: root/lib/drmtest.c
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2019-07-11 19:07:24 +0200
committerDaniel Vetter <daniel.vetter@ffwll.ch>2019-07-16 10:12:14 +0200
commitb9cac102542f2fbbbed9e5aa2b1ac04ec3795bc6 (patch)
tree554674cf8ed6c1f94f2a7f984e83213c16147372 /lib/drmtest.c
parent41ff022b62b45a5b84504daa3537fa1b295b97c9 (diff)
lib: Ditch "DRIVER_* for kms tests" cargo cult
KMS is meant to be a (at least somewhat) generic userspace API. We do not want nor need to add a special match function for every driver ever written, that doesn't make sense, defeats the point of having generic tests for a generic testsuite, and really doesn't scale. Also add a comment so people don't try to add ever more of these. Also, this means no autoloading for you, but really igt should reinvent udev, and mostly this was needed for 2 reasons: - CI configuration falling to pieces because tests unloaded a driver, and then died before cleaning up. - vgem, which is fake, and needs to be requested explictily. We might want to throw that all out again, except for vgem. Cc: Liviu Dudau <liviu.dudau@arm.com> Cc: Deepak Rawat <drawat@vmware.com> Cc: Harry Wentland <harry.wentland@amd.com> Cc: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> Cc: Leo Li <sunpeng.li@amd.com> Cc: MichaƂ Winiarski <michal.winiarski@intel.com> Cc: "Ser, Simon" <simon.ser@intel.com> Cc: Oleg Vasilev <oleg.vasilev@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> Reviewed-by: Simon Ser <simon.ser@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'lib/drmtest.c')
-rw-r--r--lib/drmtest.c9
1 files changed, 0 insertions, 9 deletions
diff --git a/lib/drmtest.c b/lib/drmtest.c
index 25f20353..c379a7b7 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -187,12 +187,6 @@ static int modprobe(const char *driver)
return igt_kmod_load(driver, "");
}
-/* virtio's driver name is virtio_gpu but the module is virtio-gpu.ko */
-static void modprobe_virtio(const char *name)
-{
- igt_kmod_load("virtio-gpu", "");
-}
-
static void modprobe_i915(const char *name)
{
/* When loading i915, we also want to load snd-hda et al */
@@ -210,7 +204,6 @@ static const struct module {
{ DRIVER_V3D, "v3d" },
{ DRIVER_VC4, "vc4" },
{ DRIVER_VGEM, "vgem" },
- { DRIVER_VIRTIO, "virtio_gpu", modprobe_virtio },
{}
};
@@ -363,8 +356,6 @@ static const char *chipset_to_str(int chipset)
return "vc4";
case DRIVER_VGEM:
return "vgem";
- case DRIVER_VIRTIO:
- return "virtio";
case DRIVER_AMDGPU:
return "amdgpu";
case DRIVER_PANFROST: