Re: [PATCH 1/3] test/cmd: Make some "ut dm" tests only available on sandbox

[Heinrich Schuchardt <xypron.glpk@gmx.de>]

On 10/28/24 17:48, Tom Rini wrote:
> Currently, the "dm" suite in unit tests (ut) is only available on
> sandbox. Make sure that all cmd tests that are part of this suite are
> only available on sandbox and not attempted to be run on hardware (where
> it will fail to be able to be started).
>
> Signed-off-by: Tom Rini <trini@konsulko.com>


Adding a comment to the sources describing that the tests require
sandbox drivers would be helpful.

Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>


> ---
>   test/cmd/Makefile | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/cmd/Makefile b/test/cmd/Makefile
> index 4b487c1d2cb4..fe7a2165af22 100644
> --- a/test/cmd/Makefile
> +++ b/test/cmd/Makefile
> @@ -25,16 +25,16 @@ obj-$(CONFIG_CMD_MEM_SEARCH) += mem_search.o
>   ifdef CONFIG_CMD_PCI
>   obj-$(CONFIG_CMD_PCI_MPS) += pci_mps.o
>   endif
> -obj-$(CONFIG_CMD_PINMUX) += pinmux.o
> -obj-$(CONFIG_CMD_PWM) += pwm.o
>   obj-$(CONFIG_CMD_SEAMA) += seama.o
>   ifdef CONFIG_SANDBOX
>   obj-$(CONFIG_CMD_MBR) += mbr.o
> +obj-$(CONFIG_CMD_PINMUX) += pinmux.o
> +obj-$(CONFIG_CMD_PWM) += pwm.o
>   obj-$(CONFIG_CMD_READ) += rw.o
>   obj-$(CONFIG_CMD_SETEXPR) += setexpr.o
> +obj-$(CONFIG_CMD_TEMPERATURE) += temperature.o
>   ifdef CONFIG_NET
>   obj-$(CONFIG_CMD_WGET) += wget.o
>   endif
>   obj-$(CONFIG_ARM_FFA_TRANSPORT) += armffa.o
>   endif
> -obj-$(CONFIG_CMD_TEMPERATURE) += temperature.o