[U-Boot] [PATCH v2 01/10] tiny-printf: Reduce size by removing ctype

Simon Glass sjg at chromium.org
Mon Oct 21 23:26:43 UTC 2019


The ctype array is brought into the image, adding 256 bytes, when it is
unlikely to be needed. The extra code for %p is only present when DEBUG
is defined, so let's drop ctype as well unless DEBUG is defined.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

Changes in v2: None

 lib/tiny-printf.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/lib/tiny-printf.c b/lib/tiny-printf.c
index ebef92fc9f6..632b4249142 100644
--- a/lib/tiny-printf.c
+++ b/lib/tiny-printf.c
@@ -289,8 +289,15 @@ static int _vprintf(struct printf_info *info, const char *fmt, va_list va)
 				break;
 			case 'p':
 				pointer(info, fmt, va_arg(va, void *));
+				/*
+				 * Skip this because it pulls in _ctype which is
+				 * 256 bytes, and we don't generally implement
+				 * pointer anyway
+				 */
+#ifdef DEBUG
 				while (isalnum(fmt[0]))
 					fmt++;
+#endif
 				break;
 			case '%':
 				out(info, '%');
-- 
2.23.0.866.gb869b98d4c-goog



More information about the U-Boot mailing list