[PATCH libdrm] tests: util: Fixup util_open() parameter order
Thierry Reding
thierry.reding at gmail.com
Tue Jan 5 06:21:23 PST 2016
From: Thierry Reding <treding at nvidia.com>
util_open() takes a device parameter, followed by a module parameter.
The existing tests used the drmOpen() function, which uses a different
ordering of the parameters, and the old ordering was accidentally kept
during the conversion.
Signed-off-by: Thierry Reding <treding at nvidia.com>
---
Evidently I didn't properly test this. Things do work in most cases
with the current code because of the auto-detection, but there will
be cases where it fails.
tests/modetest/modetest.c | 2 +-
tests/proptest/proptest.c | 2 +-
tests/vbltest/vbltest.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 22e3e81bd3cc..f665240a53c5 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetest/modetest.c
@@ -1603,7 +1603,7 @@ int main(int argc, char **argv)
if (!args)
encoders = connectors = crtcs = planes = framebuffers = 1;
- dev.fd = util_open(module, device);
+ dev.fd = util_open(device, module);
if (dev.fd < 0)
return -1;
diff --git a/tests/proptest/proptest.c b/tests/proptest/proptest.c
index 24c634569f2f..4bd08665461c 100644
--- a/tests/proptest/proptest.c
+++ b/tests/proptest/proptest.c
@@ -295,7 +295,7 @@ int main(int argc, char *argv[])
args = argc - optind;
- fd = util_open(module, device);
+ fd = util_open(device, module);
if (fd < 0)
return 1;
diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
index 183332122978..4475b49b7b9a 100644
--- a/tests/vbltest/vbltest.c
+++ b/tests/vbltest/vbltest.c
@@ -120,7 +120,7 @@ int main(int argc, char **argv)
}
}
- fd = util_open(module, device);
+ fd = util_open(device, module);
if (fd < 0)
return 1;
--
2.5.0
More information about the dri-devel
mailing list