[PATCH 00/15] lib: Add support for a decimal 0m prefix for numbers

Wolfgang Denk wd at denx.de
Wed Jul 21 09:52:50 CEST 2021


Hi,

In message <20210720160547.GM9379 at bill-the-cat> you wrote:
> 
> > So for example (10)123 would mean decimal 123? I don't know how we
> > would parse brackets separately from expressions though.
>
> (123)10 would be "123" in decimal.  Which is indeed a mouthful.  But it
> would also be generic and (123)16 would be 0x123.  So the parsing
> shouldn't be too hard, for most commands.  But then yes, expressions
> become quite hard.

Come on, guys, be serious!  This is a boot loader.  Size matters.

Do we _really_ need all this, and is it worth the code size?

Simon's patches include some cleanup, which probably even reduces
the size, so good.

But whether it's 0m123 or 0t123 or 0!123 or ... is pretty much
irrelevant - chose one symbol, use it, and be done with that.



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Do you suppose the reason the ends of the `Intel Inside'  logo  don't
match up is that it was drawn on a Pentium?


More information about the U-Boot mailing list