From: Heinrich Schuchardt <xypron.glpk@gmx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 06/16] linux/time.h: include vsprintf.h
Date: Tue, 26 Nov 2019 08:31:18 +0100 [thread overview]
Message-ID: <c4a874b0-f043-059f-4902-682653eb6cdc@gmx.de> (raw)
In-Reply-To: <6cce30a5-7e79-9c49-2d78-a3a38da7e143@gmx.de>
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
next prev parent reply other threads:[~2019-11-26 7:31 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-13 0:44 [U-Boot] [PATCH v3 00/16] import x509/pkcs7 parsers from linux AKASHI Takahiro
2019-11-13 0:44 ` [U-Boot] [PATCH v3 01/16] linux_compat: move kmemdup() from ubifs.c to linux_compat.c AKASHI Takahiro
2019-11-26 3:20 ` Heinrich Schuchardt
2019-12-06 21:48 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 02/16] rtc.h: add struct udevice declaration AKASHI Takahiro
2019-12-06 21:48 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 03/16] rtc: move date.c from drivers/rtc/ to lib/ AKASHI Takahiro
2019-12-06 21:48 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 04/16] lib: add mktime64() for linux compatibility AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 05/16] include: kernel.h: include printk.h AKASHI Takahiro
2019-11-26 3:35 ` Heinrich Schuchardt
2019-11-27 1:02 ` AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-12-09 1:21 ` AKASHI Takahiro
2019-11-13 0:44 ` [U-Boot] [PATCH v3 06/16] linux/time.h: include vsprintf.h AKASHI Takahiro
2019-11-26 3:56 ` Heinrich Schuchardt
2019-11-26 7:31 ` Heinrich Schuchardt [this message]
2019-11-27 1:27 ` AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 07/16] cmd: add asn1_compiler AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 08/16] Makefile: add build script for asn1 parsers AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 09/16] lib: add asn1 decoder AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 10/16] doc: add README for asn1 compiler and decoder AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 11/16] lib: add oid registry utility AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 12/16] lib: crypto: add public key utility AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:44 ` [U-Boot] [PATCH v3 13/16] lib: crypto: add rsa public key parser AKASHI Takahiro
2019-12-06 21:49 ` Tom Rini
2019-11-13 0:45 ` [U-Boot] [PATCH v3 14/16] lib: crypto: add x509 parser AKASHI Takahiro
2019-12-06 21:50 ` Tom Rini
2019-12-07 20:51 ` Heinrich Schuchardt
2019-12-07 22:34 ` Tom Rini
2019-12-09 0:59 ` AKASHI Takahiro
2019-11-13 0:45 ` [U-Boot] [PATCH v3 15/16] lib: crypto: add pkcs7 message parser AKASHI Takahiro
2019-12-06 21:50 ` Tom Rini
2019-11-13 0:45 ` [U-Boot] [PATCH v3 16/16] test: add asn1 unit test AKASHI Takahiro
2019-12-06 21:50 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c4a874b0-f043-059f-4902-682653eb6cdc@gmx.de \
--to=xypron.glpk@gmx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox