From: Thierry Reding <tred...@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

Reply via email to