Maybe the previous discussions in CB:33300 are useful?

Yeah, if we want to do explicit numbers I would vote for FMAP/512K or RO_FRID/1 instead of the %. I think the '/' stands for division and alignment is sort of a "is divisible by". For this here I suggested '%' to stand for modulo, because it's the "unaligned" sigil (as in "you may have a non-zero modulus for this section). But maybe I'm also widely overthinking all of this. :D (Also I think the '%' looks a bit better than '/' without another number behind it.)

I don't think we really need explicit numbers, though, I think that just makes this more complicated and more boilerplate-y again. In practice, we'll only ever care about 4K. The FMAP is sort of a "soft" requirement (make the alignment as large as makes sense for this platform), I don't think a hard constraint is very useful for that. In practice you usually just put in an explicit, largely-aligned offset for the FMAP section and then let the other RO sections flow around that.

View Change

To view, visit change 37262. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I26b394590c28667a4afcd521c7caa2009b5b98a9
Gerrit-Change-Number: 37262
Gerrit-PatchSet: 8
Gerrit-Owner: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 28 Nov 2019 07:02:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment