[Buildroot] [PATCH 7/7] board/kontron/smarc-sal28/genimage.cfg: fix option suffix
Arnout Vandecappelle
arnout at mind.be
Tue Nov 9 09:23:27 UTC 2021
On 09/11/2021 09:50, Yann E. MORIN wrote:
> Michael, All,
>
> On 2021-11-09 09:04 +0100, Michael Walle spake thusly:
>> 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? 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".
>
> This SI reference was also my comment on the preceding series, but
> unfortunately, that ship has sailed now that the series has been
> applied...
What has been applied is uppercase K everywhere. Lowercase k looks like a SI
unit, i.e. 1000, so I think uppercase is a little better. To be fully correct,
we should use KiB instead (which apparently works since genimage apparently
ignores everything after the first letter). But as you say, that ship has sailed...
Anyway, this stuff is never going to be fully accurate, so anything is OK in
my book, as long as it's consistent.
Regards,
Arnout
More information about the buildroot
mailing list