[U-Boot] [PATCH v3] arm: printf() is not available in some SPL configurations

2011-11-22 Thread Christian Riesch
This patch avoids build breakage for SPLs that do not support printf. Signed-off-by: Christian Riesch Cc: Wolfgang Denk Cc: Tom Rini Acked-by: Tom Rini --- Changes for v3: - Removed extra white space - Separated patch from patchset [1] [1] http://lists.denx.de/pipermail/u-boot/2011-November/1

Re: [U-Boot] [PATCH v3] arm: printf() is not available in some SPL configurations

2011-11-22 Thread Andreas Bießmann
Dear Christian, sorry I haven't followed the discussion so far. I have a little pointer too. Am 22.11.2011 09:26, schrieb Christian Riesch: > This patch avoids build breakage for SPLs that do not support printf. > > Signed-off-by: Christian Riesch > Cc: Wolfgang Denk > Cc: Tom Rini > Acked-by

Re: [U-Boot] [PATCH v3] arm: printf() is not available in some SPL configurations

2011-11-22 Thread Christian Riesch
Hi Andreas, Thank you for your comments! On Tue, Nov 22, 2011 at 9:49 AM, Andreas Bießmann wrote: > Dear Christian, > > sorry I haven't followed the discussion so far. I have a little pointer too. > > Am 22.11.2011 09:26, schrieb Christian Riesch: >> This patch avoids build breakage for SPLs that

Re: [U-Boot] [PATCH v3] arm: printf() is not available in some SPL configurations

2011-11-22 Thread Scott Wood
On 11/22/2011 02:49 AM, Andreas Bießmann wrote: > Well, you can do it that way but I guess there are several places where > printf() will be needed by some drivers in SPL. Therefore I think it is > better to provide printf() for SPL than to ifdef out all the printf's. > > So how about providing an