On Tue, Jan 17, 2017 at 10:06:26AM +0200, Oded Gabbay wrote: > On Tue, Jan 17, 2017 at 9:30 AM, Masahiro Yamada > <yamada.masah...@socionext.com> wrote: > > 2016-12-28 16:36 GMT+09:00 Oded Gabbay <oded.gab...@gmail.com>: > >> In the tiny-printf implementation, there is no support for %.*s. This patch > >> checks if CONFIG_USE_TINY_PRINTF is defined and if so, prints a different > >> debug statement which doesn't use %.*s > >> > >> Signed-off-by: Oded Gabbay <oded.gab...@gmail.com> > >> Cc: Simon Glass <s...@chromium.org> > >> --- > >> common/spl/spl.c | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/common/spl/spl.c b/common/spl/spl.c > >> index f7df834..7c4744d 100644 > >> --- a/common/spl/spl.c > >> +++ b/common/spl/spl.c > >> @@ -115,9 +115,14 @@ int spl_parse_image_header(struct spl_image_info > >> *spl_image, > >> } > >> spl_image->os = image_get_os(header); > >> spl_image->name = image_get_name(header); > >> +#ifdef CONFIG_USE_TINY_PRINTF > >> + debug("spl: payload image: %s load addr: 0x%x size: %d\n", > >> + spl_image->name, spl_image->load_addr, > >> spl_image->size); > >> +#else > >> debug("spl: payload image: %.*s load addr: 0x%x size: > >> %d\n", > >> (int)sizeof(spl_image->name), spl_image->name, > >> spl_image->load_addr, spl_image->size); > >> +#endif > >> } else { > >> #ifdef CONFIG_SPL_PANIC_ON_RAW_IMAGE > > > > > > Same here. > > > > Please do not patch around with CONFIG_USE_TINY_PRINTF. > > > > What you need to do is to fix tiny_printf() implementation. > > > > > > -- > > Best Regards > > Masahiro Yamada > > ok, I can accept that but how would you like to fix it ?
I would suggest un-winding things such that we get useful information in both cases with USE_TINY_PRINTF being the limiting factor. I am of the opinion that USE_TINY_PRINTF should be the default within SPL. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot