[PATCH v2 1/4] lib: provide function hextoull()

Tom Rini trini at konsulko.com
Wed Nov 13 16:19:54 CET 2024


On Wed, Nov 13, 2024 at 07:42:57AM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Wed, 13 Nov 2024 at 07:12, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Wed, Nov 13, 2024 at 06:39:43AM -0700, Simon Glass wrote:
> > > Hi Heinrich,
> > >
> > > On Sun, 3 Nov 2024 at 15:42, Heinrich Schuchardt
> > > <heinrich.schuchardt at canonical.com> wrote:
> > > >
> > > > We often convert hexadecimal strings to hextoull(). Provide a wrapper
> > > > function to simple_strtoull() that does not require specifying the radix.
> > > >
> > > > Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> > > > ---
> > > > v2:
> > > >         no change
> > > > ---
> > > >  include/vsprintf.h | 13 +++++++++++++
> > > >  lib/strto.c        |  5 +++++
> > > >  2 files changed, 18 insertions(+)
> > >
> > > Could you add a test?
> >
> > We have tests already for this. But they weren't being run, which is
> > covered by 4/4.
> 
> I'm not seeing a test for hextoull(). Where are you looking?
> 
> There is a test for hextoul() in test/str_ut.c

Oh sorry, I was referring to the overall problem this series addresses.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241113/7a919bcf/attachment.sig>


More information about the U-Boot mailing list