Dear Simon Glass,
On Fri, 26 Mar 2021 at 07:08, Jaehoon Chung <jh80.ch...@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 <s...@chromium.org>
>
> Reviewed-by: Jaehoon Chung <jh80.ch...@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.
