[PATCH 1/1] test: bootflow test depends on CONFIG_VIDEO

[Heinrich Schuchardt]

Tom Rini <trini at konsulko.com> schrieb am Mi., 31. Mai 2023, 16:02:

> On Wed, May 31, 2023 at 10:50:52AM +0200, Heinrich Schuchardt wrote:
>
> > qemu_arm64_defconfig with UNIT_TEST=y does not build.
> >
> > CONFIG_EXPO depends on CONFIG_VIDEO. Hence on platforms without video we
> > must not enable unit tests invoking expo functions.
> >
> > Fixes: fb1451bec2a5 ("bootstd: Add tests for bootstd including all
> uclasses")
> > Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> > ---
> >  test/boot/Makefile | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/test/boot/Makefile b/test/boot/Makefile
> > index 22ed61c8fa..665017ba3d 100644
> > --- a/test/boot/Makefile
> > +++ b/test/boot/Makefile
> > @@ -2,7 +2,10 @@
> >  #
> >  # Copyright 2021 Google LLC
> >
> > -obj-$(CONFIG_BOOTSTD) += bootdev.o bootstd_common.o bootflow.o
> bootmeth.o
> > +ifdef CONFIG_BOOTSTD
> > +obj-y += bootstd_common.o bootmeth.o
> > +obj-$(VIDEO) += bootdev.o bootflow.o
> > +endif
> >  obj-$(CONFIG_FIT) += image.o
> >
> >  obj-$(CONFIG_EXPO) += expo.o
>
> OK, but the functionality itself doesn't depend on video, it looks like
> maybe only the bootflow_menu_theme test itself, so lets try and solve
> this differently.
>
> --
> Tom
>

Yes I should have used CONFIG_EXPO not CONFIG_VIDEO .

Regards

Heinrich

>