From: Thierry Reding <treding@xxxxxxxxxx> 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@xxxxxxxxxx> --- 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 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
