Hi Michael, All,

Il giorno 9 nov 2021, alle ore 09:05, Michael Walle <michael@walle.cc> ha scritto:

Am 2021-11-09 00:05, schrieb Giulio Benetti:
Option 'offset = 4k' must be 'option = 4K'.

Why is this is a 'must'? Apparently it works, no?

This is a standard I’ve proposed and it’s been accepted by committing all the 102 patches.
There is not an existing standard and I’ve created one, that’s all.

So if anything
it is for unifying; but even then it's moving in the wrong
direction. There is no "K" prefix in si units. Stictly speaking
there is no "KB"; there is a "kB" or a "KiB".

Here we had 2 choices using genimage “K” or “k”. I went for “K” since the most of the files had “K”.
And anyway both “K” and “k” are not that good, but it’s not our fault, it depends on genimage input format.

Yann already explained the differences between notations here:
https://lists.buildroot.org/pipermail/buildroot/2021-November/627885.html

You can follow-up the thread and the motivations on such thing.

Feel free to propose a modification on this topic since it’s “fresh”. I have to integrate a bit the manual and write a Linter for this kind of files so it’s better I know it early.

Thank you
Best regards
Giulio Benetti
Benetti Engineering sas


Anyway. I'll let the buildroot maintainers decide on this.

-michael