From: Miquel Raynal <miquel.raynal@bootlin.com>
To: Tom Rini <trini@konsulko.com>
Cc: Fabio Estevam <festevam@gmail.com>,
Svyatoslav Ryhel <clamor95@gmail.com>,
Simon Glass <sjg@chromium.org>,
Jaehoon Chung <jh80.chung@samsung.com>,
Lukasz Majewski <lukma@denx.de>,
Sean Anderson <seanga2@gmail.com>,
Anatolij Gustschin <agust@denx.de>, Peng Fan <peng.fan@nxp.com>,
Mario Six <mario.six@gdsys.cc>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
u-boot@lists.denx.de, Ian Ray <ian.ray@gehealthcare.com>,
Michael Nazzareno Trimarchi <michael@amarulasolutions.com>,
Dario Binacchi <dario.binacchi@amarulasolutions.com>,
Adam Ford <aford173@gmail.com>, Marek Vasut <marex@denx.de>
Subject: Re: [PATCH v4 00/12] Add imx8mp video support
Date: Thu, 03 Apr 2025 09:30:56 +0200 [thread overview]
Message-ID: <878qohpuq7.fsf@bootlin.com> (raw)
In-Reply-To: <20250328155711.GV93000@bill-the-cat> (Tom Rini's message of "Fri, 28 Mar 2025 09:57:11 -0600")
Hi Tom,
>> I need the tests to run on the test.dts and not the default sandbox.dts.
>> The documentation states to use -T for that when doing the checks
>> locally. Unfortunately, the Azure pipelines do use the default sandbox
>> DT. I can add few mostly useless nodes to the default DTS for the tests
>> to pass, but I feel like this is not the correct approach.
>>
>> Is there a way to tell Azure to run some of the unit tests (especially
>> the DM ones) only with the test DT?
>
> I saw the older email first, sorry. But I think the docs have become
> disconnected from how things have been done practically.
Actually, tests are actually running with the test.dts (with the option
-T) otherwise I don't think it would work. In practice I haven't found
the proof of it, but I strongly believe it is not using the default DTS.
However, many tests depend on Kconfig symbols to be enabled, which means
we need to constantly enable more support in the various sandbox
defconfigs to make them run. For instance the video_bridge tests I
rebased on top of where only running on the regular sandbox defconfig,
not the others.
I believe it is all sorted out now, CI is green, so let me respin this
series :-)
Thanks,
Miquèl
next prev parent reply other threads:[~2025-04-03 12:36 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-29 10:21 [PATCH v4 00/12] Add imx8mp video support Miquel Raynal
2025-01-29 10:21 ` [PATCH v4 01/12] dm: doc: Fix example Miquel Raynal
2025-01-29 10:21 ` [PATCH v4 02/12] dm: core: Add a helper to retrieve devices through graph endpoints Miquel Raynal
2025-01-29 10:21 ` [PATCH v4 03/12] sandbox: Add a fake DSI controller and link it to the panel Miquel Raynal
2025-01-29 10:21 ` [PATCH v4 04/12] test: dm: test-fdt: Add checks for uclass_get_device_by_endpoint() Miquel Raynal
2025-01-29 10:21 ` [PATCH v4 05/12] power-domain: Add refcounting Miquel Raynal
2025-02-08 11:09 ` Fabio Estevam
2025-03-03 8:43 ` Miquel Raynal
2025-03-04 13:13 ` Simon Glass
2025-03-04 13:55 ` Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 06/12] clk: Ensure the parent clocks are enabled while reparenting Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 07/12] clk: imx8mp: Add media related clocks Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 08/12] imx: power-domain: Describe the i.MX8 MEDIAMIX domain Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 09/12] imx: power-domain: Add support for the MEDIAMIX control block Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 10/12] video: imx: Fix Makefile in order to be able to add other imx drivers Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 11/12] video: imx: Add LDB driver Miquel Raynal
2025-01-29 10:22 ` [PATCH v4 12/12] video: imx: Add LCDIF driver Miquel Raynal
2025-03-03 12:31 ` [PATCH v4 00/12] Add imx8mp video support Fabio Estevam
2025-03-03 17:06 ` Tom Rini
2025-03-03 17:14 ` Svyatoslav Ryhel
2025-03-13 18:26 ` Fabio Estevam
2025-03-26 15:01 ` Miquel Raynal
2025-03-27 13:31 ` Fabio Estevam
2025-03-27 14:14 ` Tom Rini
2025-03-28 8:39 ` Miquel Raynal
2025-03-28 15:57 ` Tom Rini
2025-03-28 16:31 ` Miquel Raynal
2025-04-03 7:30 ` Miquel Raynal [this message]
2025-03-03 17:18 ` Svyatoslav Ryhel
2025-03-05 18:12 ` Fabio Estevam
2025-03-05 18:17 ` Tom Rini
2025-03-06 12:38 ` Fabio Estevam
2025-03-06 12:51 ` Miquel Raynal
2025-03-06 13:00 ` Fabio Estevam
2025-03-06 14:05 ` Tom Rini
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=878qohpuq7.fsf@bootlin.com \
--to=miquel.raynal@bootlin.com \
--cc=aford173@gmail.com \
--cc=agust@denx.de \
--cc=clamor95@gmail.com \
--cc=dario.binacchi@amarulasolutions.com \
--cc=festevam@gmail.com \
--cc=ian.ray@gehealthcare.com \
--cc=jh80.chung@samsung.com \
--cc=lukma@denx.de \
--cc=marex@denx.de \
--cc=mario.six@gdsys.cc \
--cc=michael@amarulasolutions.com \
--cc=peng.fan@nxp.com \
--cc=seanga2@gmail.com \
--cc=sjg@chromium.org \
--cc=thomas.petazzoni@bootlin.com \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox