On 11/26/19 4:56 AM, Heinrich Schuchardt wrote:
On 11/13/19 1:44 AM, AKASHI Takahiro wrote:
Without this commit, time.h possibly causes a build error as
asctime_r() uses sprintf().
asctime_r() is not a Linux symbol (as of next-20191119)
ctime_r() and asctime_r() are defined as inline functions. ctime_r() is
used in multiple places and so we may end up duplicating code. So I
would prefer the inline functions in time.h to be moved to a separate C
file in lib/.
Best regards
Heinrich
Could it be that in one of your C files you simply didn't follow the
U-Boot coding style convention to include common.h first and we don't
need this patch at all?
https://www.denx.de/wiki/U-Boot/CodingStyle#Include_files
Best regards
Heinrich
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot