On Tue, 12 Mar 2019 at 04:38, Thierry Reding <thierry.red...@gmail.com> wrote: > > From: Thierry Reding <tred...@nvidia.com> > > When phys_addr_t printf specifier support was first introduced in commit > 1eebd14b7902 ("vsprintf: Add modifier for phys_addr_t"), it was enabled > only if CONFIG_CMD_NET was selected. Since physical addresses are not > unique to networking support it doesn't make sense to conditionally add > it in those cases only. Move support for it outside of the CMD_NET guard > so that the specifier is always supported. > > Signed-off-by: Thierry Reding <tred...@nvidia.com> > --- > lib/vsprintf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Is there a code-size impact here? Perhaps instead it should be a new Kconfig option, implied by CMD_NET? Reviewed-by: Simon Glass <s...@chromium.org> Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot