[U-Boot] [PATCH 1/2] [RFC] lib: Implement support for tiny sprintf()

Simon Glass
2016-05-31T19:51:32Z

Hi Marek,

On 1 June 2016 at 07:36, Marek Vasut <marex@denx.de> wrote:
> On 05/31/2016 08:19 PM, Simon Glass wrote:
>> Hi,
>>
>> On 31 May 2016 at 05:55, Tom Rini <trini@konsulko.com> wrote:
>>> On Thu, May 26, 2016 at 06:00:24PM +0200, Marek Vasut wrote:
>>>
>>>> Tweak the tiny printf code to also provide similarly tiny sprintf()
>>>> implementation. This is not comformant with POSIX for sure, but it
>>>> keeps the size down while still behaving rather reasonably.
>>>>
>>>> Signed-off-by: Marek Vasut <marex@denx.de>
>>>> Cc: Simon Glass <sjg@chromium.org>
>>>> Cc: Tom Rini <trini@konsulko.com>
>>>
>>> Reviewed-by: Tom Rini <trini@konsulko.com>
>>
>> Also please see this one:
>>
>> http://patchwork.ozlabs.org/patch/622267/
>>
>> It uses a function passed in.
>
> Oh, I knew I saw some s*printf() stuff from you. Works for me :)

OK good. It works for me too but I'm sure it can be improved (code
size, use of division, etc.), so see how you go.

Regards,
Simon