[PATCH 3/3] snow: Enable unit tests

Minkyu Kang promsoft at gmail.com
Fri Apr 23 10:32:27 CEST 2021


Dear Simon Glass,

On Fri, 26 Mar 2021 at 07:08, Jaehoon Chung <jh80.chung at samsung.com> wrote:

> On 3/25/21 6:44 AM, Simon Glass wrote:
> > It is useful to have a board with unit tests enabled, to check that this
> > does not break.
> >
> > Let's choose snow, since it is not under active development and it is
> > glorious.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
>
> Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>
>
> Best Regards,
> Jaehoon Chung
>
> > ---
> >
> >  configs/snow_defconfig | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/configs/snow_defconfig b/configs/snow_defconfig
> > index 8b646cac67a..0277062428e 100644
> > --- a/configs/snow_defconfig
> > +++ b/configs/snow_defconfig
> > @@ -92,3 +92,5 @@ CONFIG_VIDEO_BRIDGE_NXP_PTN3460=y
> >  CONFIG_LCD=y
> >  CONFIG_TPM=y
> >  CONFIG_ERRNO_STR=y
> > +CONFIG_UNIT_TEST=y
> > +# CONFIG_UT_LIB_ASN1 is not set
> >
>
>
On my system, compiler warnings have occurred.
Is it fine for you?

test/lib/strlcat.c:51:39: warning: format ‘%lx’ expects argument of type
‘long unsigned int’, but argument 7 has type ‘size_t’ {aka ‘unsigned int’}
[-Wformat=]
   51 |                          "Expected %#lx (%ld), got %#lx (%ld)",
      |                                    ~~~^
      |                                       |
      |                                       long unsigned int
      |                                    %#x
   52 |                          expected, expected, actual, actual);
      |                          ~~~~~~~~
      |                          |
      |                          size_t {aka unsigned int}

-- 
Thanks,
Minkyu Kang.


More information about the U-Boot mailing list