diff options
| author | Ahmad Fatoum <a.fatoum@pengutronix.de> | 2022-01-03 13:02:05 +0100 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2022-01-05 09:01:17 +0100 |
| commit | dc8c0a9425a7be301e03e77489a25b14e20b383a (patch) | |
| tree | fd928bb7fc0fa98ad2e1e643a6358a5f99491844 | |
| parent | 1b0679d9b8d58697da9fb563bb3509d29ad43c28 (diff) | |
| download | barebox-dc8c0a9425a7be301e03e77489a25b14e20b383a.tar.gz barebox-dc8c0a9425a7be301e03e77489a25b14e20b383a.tar.xz | |
parseopt: drop unused, duplicate, parseopt_u16
We always have sizeof(unsigned short) == sizeof(uint16_t), so
parseopt_u16() is exactly equivalent to parseopt_hu().
The former is unused anyway, so just drop it.
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Link: https://lore.barebox.org/20220103120205.1728778-1-a.fatoum@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| -rw-r--r-- | include/parseopt.h | 1 | ||||
| -rw-r--r-- | lib/parseopt.c | 31 |
2 files changed, 0 insertions, 32 deletions
diff --git a/include/parseopt.h b/include/parseopt.h index 5a40bdc219..a944c3655f 100644 --- a/include/parseopt.h +++ b/include/parseopt.h @@ -6,7 +6,6 @@ void parseopt_llu_suffix(const char *options, const char *opt, void parseopt_b(const char *options, const char *opt, bool *val); void parseopt_hu(const char *options, const char *opt, unsigned short *val); -void parseopt_u16(const char *options, const char *opt, uint16_t *val); void parseopt_str(const char *options, const char *opt, char **val); #endif /* __PARSEOPT_H__ */ diff --git a/lib/parseopt.c b/lib/parseopt.c index 70983066d9..d0a27d8bf3 100644 --- a/lib/parseopt.c +++ b/lib/parseopt.c @@ -59,37 +59,6 @@ again: *val = v; } -void parseopt_u16(const char *options, const char *opt, uint16_t *val) -{ - const char *start; - size_t optlen = strlen(opt); - ulong v; - char *endp; - -again: - start = strstr(options, opt); - - if (!start) - return; - - if (start > options && start[-1] != ',') { - options = start; - goto again; - } - - if (start[optlen] != '=') { - options = start; - goto again; - } - - v = simple_strtoul(start + optlen + 1, &endp, 0); - if (v > U16_MAX) - return; - - if (*endp == ',' || *endp == '\0') - *val = v; -} - void parseopt_str(const char *options, const char *opt, char **val) { const char *start; |