[Buildroot] [PATCH 7/7] board/kontron/smarc-sal28/genimage.cfg: fix option suffix

[Arnout Vandecappelle]

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