[PATCH 1/1] image: usage of value ~0UL for intrd_high

Tom Rini trini at konsulko.com
Sat Jan 9 19:58:36 CET 2021


On Sat, Jan 09, 2021 at 08:47:07PM +0200, Andy Shevchenko wrote:
> On Sat, Jan 9, 2021 at 8:06 PM Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> >
> > The comment for initrd_high in the coding and in README were contradicting
> > and neither fully described what the coding does.
> >
> > Clarify the usage of the special value ~0UL for the environment variable
> > initrd_high.
> 
> All those F:s are hard to read in the comments and documentation and
> typo prone. I would prefer to rephrase like "all 1:s value in 32- or
> 64-bit format" or alike.

If we're going to improve this we should also note it's discouraged
unless you know for certain there will be no overlap and it's strongly
discouraged in default environments.

-- 
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/20210109/bbdd029b/attachment.sig>


More information about the U-Boot mailing list