From: Gustavo Padovan gustavo.padovan@collabora.co.uk
Signed-off-by: Gustavo Padovan gustavo.padovan@collabora.co.uk --- .gitignore | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/.gitignore b/.gitignore index 1b853a2..a44566f 100644 --- a/.gitignore +++ b/.gitignore @@ -93,6 +93,8 @@ tests/modeprint/modeprint tests/modetest/modetest tests/name_from_fd tests/proptest/proptest +tests/kms/kms-steal-crtc +tests/kms/kms-universal-planes tests/kmstest/kmstest tests/vbltest/vbltest tests/radeon/radeon_ttm
From: Gustavo Padovan gustavo.padovan@collabora.co.uk
Signed-off-by: Gustavo Padovan gustavo.padovan@collabora.co.uk --- tests/modetest/modetest.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index a5ac5bd..21d5438 100644 --- a/tests/modetest/modetest.c +++ b/tests/modetest/modetest.c @@ -314,6 +314,8 @@ static void dump_prop(struct device *dev, drmModePropertyPtr prop, printf("\t\tvalue:"); if (drm_property_type_is(prop, DRM_MODE_PROP_BLOB)) dump_blob(dev, value); + else if (drm_property_type_is(prop, DRM_MODE_PROP_SIGNED_RANGE)) + printf(" %"PRId64"\n", value); else printf(" %"PRIu64"\n", value); }
Hi Gustavo,
On 22 March 2016 at 21:42, Gustavo Padovan gustavo@padovan.org wrote:
From: Gustavo Padovan gustavo.padovan@collabora.co.uk
Signed-off-by: Gustavo Padovan gustavo.padovan@collabora.co.uk
tests/modetest/modetest.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index a5ac5bd..21d5438 100644 --- a/tests/modetest/modetest.c +++ b/tests/modetest/modetest.c @@ -314,6 +314,8 @@ static void dump_prop(struct device *dev, drmModePropertyPtr prop, printf("\t\tvalue:"); if (drm_property_type_is(prop, DRM_MODE_PROP_BLOB)) dump_blob(dev, value);
else if (drm_property_type_is(prop, DRM_MODE_PROP_SIGNED_RANGE))printf(" %"PRId64"\n", value); else printf(" %"PRIu64"\n", value);
Looks like proptest needs a similar fix. I'll push that alongside this patch.
A few suggestions for follow up work (in case you get bored) - factor dump_prop/blob into tests/util/kms.c and use it in proptest + modetest - prepend "current" or alike to "value". or any other way to make "value" stand out from "values". - handle DRM_MODE_PROP_OBJECT :-)
Thanks Emil
dri-devel@lists.freedesktop.org
-
Emil Velikov -
Gustavo Padovan