From: Andrew Davis <afd@ti.com>
To: Denys Dmytriyenko <denys@konsulko.com>,
Ryan Eatmon <reatmon@ti.com>, Randolph Sapp <rs@ti.com>,
<meta-arago@lists.yoctoproject.org>
Cc: Andrew Davis <afd@ti.com>
Subject: [meta-arago][master][PATCH 2/2] libdrm: Remove ltp-ddt test harness workaround
Date: Thu, 5 Nov 2026 14:37:26 -0600 [thread overview]
Message-ID: <20261105203726.1306888-2-afd@ti.com> (raw)
In-Reply-To: <20261105203726.1306888-1-afd@ti.com>
Modetest exits on stdin input. ltp-ddt sends data to stdin when a test
is run, so a workaround was added to modetest. Instead the test harness
should be fixed. If it has not been or cannot be fixed, then this patch
should be worked upstream and carried in meta-ti-test until then, not
here in the distro layer.
Signed-off-by: Andrew Davis <afd@ti.com>
---
.../recipes-graphics/drm/libdrm-arago.inc | 7 --
...dd-option-to-run-a-test-indefinitely.patch | 77 -------------------
.../recipes-graphics/drm/libdrm_%.bbappend | 4 -
3 files changed, 88 deletions(-)
delete mode 100644 meta-arago-distro/recipes-graphics/drm/libdrm-arago.inc
delete mode 100644 meta-arago-distro/recipes-graphics/drm/libdrm/0001-Add-option-to-run-a-test-indefinitely.patch
delete mode 100644 meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm-arago.inc b/meta-arago-distro/recipes-graphics/drm/libdrm-arago.inc
deleted file mode 100644
index 6703fd1d..00000000
--- a/meta-arago-distro/recipes-graphics/drm/libdrm-arago.inc
+++ /dev/null
@@ -1,7 +0,0 @@
-FILESEXTRAPATHS:prepend := "${THISDIR}/libdrm:"
-
-SRC_URI += " \
-file://0001-Add-option-to-run-a-test-indefinitely.patch \
-"
-
-PR:append = ".arago4"
diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm/0001-Add-option-to-run-a-test-indefinitely.patch b/meta-arago-distro/recipes-graphics/drm/libdrm/0001-Add-option-to-run-a-test-indefinitely.patch
deleted file mode 100644
index 5b051361..00000000
--- a/meta-arago-distro/recipes-graphics/drm/libdrm/0001-Add-option-to-run-a-test-indefinitely.patch
+++ /dev/null
@@ -1,77 +0,0 @@
-From 115aa882abeb6db92c7588f3fb0953593fa2f634 Mon Sep 17 00:00:00 2001
-From: Alejandro Hernandez <ajhernandez@ti.com>
-Date: Tue, 3 Mar 2020 09:52:06 -0500
-Subject: [PATCH] Add option to run a test indefinitely
-
-This patch adds option -t to the modetest cli. When this option is
-speficied modetest will not stop on stdin input. This patch
-allows using modetest within ltp-ddt, before this patch modetest
-was not able to run in ltp-ddt because ltp-ddt sends data to stdin
-when a test is run
-
-Upstream-Status: Pending
-
-Signed-off-by: Alejandro Hernandez <ajhernandez@ti.com>
-Signed-off-by: Denys Dmytriyenko <denis@denix.org>
----
- tests/modetest/modetest.c | 12 ++++++++----
- 1 file changed, 8 insertions(+), 4 deletions(-)
-
-diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
-index d9e761e..687f7a4 100644
---- a/tests/modetest/modetest.c
-+++ b/tests/modetest/modetest.c
-@@ -1888,7 +1888,7 @@ static void clear_cursors(struct device *dev)
- bo_destroy(dev->mode.cursor_bo);
- }
-
--static void test_page_flip(struct device *dev, struct pipe_arg *pipes, unsigned int count)
-+static void test_page_flip(struct device *dev, struct pipe_arg *pipes, unsigned int count, int dont_stop)
- {
- unsigned int other_fb_id;
- struct bo *other_bo;
-@@ -1946,7 +1946,7 @@ static void test_page_flip(struct device *dev, struct pipe_arg *pipes, unsigned
- fd_set fds;
-
- FD_ZERO(&fds);
-- FD_SET(0, &fds);
-+ if (!dont_stop) FD_SET(0, &fds);
- FD_SET(dev->fd, &fds);
- ret = select(dev->fd + 1, &fds, NULL, NULL, &timeout);
-
-@@ -2194,7 +2194,7 @@ static void usage(char *name)
- exit(0);
- }
-
--static char optstr[] = "acdD:efF:M:P:ps:Cvrw:o:S:";
-+static char optstr[] = "acdD:efF:M:P:ps:Cvrw:o:S:t";
-
- int main(int argc, char **argv)
- {
-@@ -2208,6 +2208,7 @@ int main(int argc, char **argv)
- char *module = NULL;
- unsigned int i;
- unsigned int count = 0, plane_count = 0;
-+ int dont_stop = 0;
- unsigned int prop_count = 0;
- struct pipe_arg *pipe_args = NULL;
- struct plane_arg *plane_args = NULL;
-@@ -2314,6 +2315,9 @@ int main(int argc, char **argv)
-
- prop_count++;
- break;
-+ case 't':
-+ dont_stop = 1;
-+ break;
- default:
- usage(argv[0]);
- break;
-@@ -2464,7 +2468,7 @@ int main(int argc, char **argv)
- set_cursors(&dev, pipe_args, count);
-
- if (test_vsync)
-- test_page_flip(&dev, pipe_args, count);
-+ test_page_flip(&dev, pipe_args, count, dont_stop);
-
- if (drop_master)
- drmDropMaster(dev.fd);
diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
deleted file mode 100644
index 7a9bed64..00000000
--- a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
+++ /dev/null
@@ -1,4 +0,0 @@
-LIBDRM_ARAGO = ""
-LIBDRM_ARAGO:arago = "libdrm-arago.inc"
-
-require ${LIBDRM_ARAGO}
--
2.43.0
next prev parent reply other threads:[~2026-11-05 20:37 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-05 20:37 [meta-arago][master][PATCH 1/2] libdrm: Remove patch adding extra OMAP BO flags Andrew Davis
2026-11-05 20:37 ` Andrew Davis [this message]
2026-11-05 21:05 ` [meta-arago][master][PATCH 2/2] libdrm: Remove ltp-ddt test harness workaround Randolph Sapp
2026-11-05 21:00 ` [meta-arago][master][PATCH 1/2] libdrm: Remove patch adding extra OMAP BO flags PRC Automation
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261105203726.1306888-2-afd@ti.com \
--to=afd@ti.com \
--cc=denys@konsulko.com \
--cc=meta-arago@lists.yoctoproject.org \
--cc=reatmon@ti.com \
--cc=rs@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.