diff options
| author | Jeroen Hofstee <jeroen@myspectrum.nl> | 2014-07-10 22:33:00 +0200 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-07-18 17:53:23 -0400 |
| commit | d7b2d9df02631e6f0ecbfbc4a4e01459d57d0d98 (patch) | |
| tree | 84a83775298caff9c01972d0578e6e77359a6ff1 /lib | |
| parent | 3ea664c7c339a788341b47f1eb0aa98eee18a721 (diff) | |
| download | talos-obmc-uboot-d7b2d9df02631e6f0ecbfbc4a4e01459d57d0d98.tar.gz talos-obmc-uboot-d7b2d9df02631e6f0ecbfbc4a4e01459d57d0d98.zip | |
lib:vsprintf: reduce scope of pack_hex_byte
pack_hex_byte is only used when CONFIG_CMD_NET is
defined so limit it to that scope. This prevents
a clang warning.
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/vsprintf.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/lib/vsprintf.c b/lib/vsprintf.c index 60874dae3e..7ec758e40f 100644 --- a/lib/vsprintf.c +++ b/lib/vsprintf.c @@ -28,17 +28,6 @@ /* some reluctance to put this into a new limits.h, so it is here */ #define INT_MAX ((int)(~0U>>1)) -static const char hex_asc[] = "0123456789abcdef"; -#define hex_asc_lo(x) hex_asc[((x) & 0x0f)] -#define hex_asc_hi(x) hex_asc[((x) & 0xf0) >> 4] - -static inline char *pack_hex_byte(char *buf, u8 byte) -{ - *buf++ = hex_asc_hi(byte); - *buf++ = hex_asc_lo(byte); - return buf; -} - unsigned long simple_strtoul(const char *cp, char **endp, unsigned int base) { @@ -434,6 +423,17 @@ static char *string(char *buf, char *end, char *s, int field_width, } #ifdef CONFIG_CMD_NET +static const char hex_asc[] = "0123456789abcdef"; +#define hex_asc_lo(x) hex_asc[((x) & 0x0f)] +#define hex_asc_hi(x) hex_asc[((x) & 0xf0) >> 4] + +static inline char *pack_hex_byte(char *buf, u8 byte) +{ + *buf++ = hex_asc_hi(byte); + *buf++ = hex_asc_lo(byte); + return buf; +} + static char *mac_address_string(char *buf, char *end, u8 *addr, int field_width, int precision, int flags) { |
