[coreboot-gerrit] [L] Change in coreboot[master]: util/cbfstool: Add '%<alignment>' in flashmap descriptor

[Julius Werner (Code Review)]

Attention is currently required from: Aaron Durbin, Furquan Shaikh, Hung-Te Lin.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37262?usp=email )

Change subject: util/cbfstool: Add '%<alignment>' in flashmap descriptor
......................................................................

Patch Set 15: Code-Review+2

(2 comments)

File util/cbfstool/default.fmd:

https://review.coreboot.org/c/coreboot/+/37262/comment/eaa5c478_55ef7860 :
PS13, Line 18:  %0
> When I try to build a image (Minimal x86 fake board): […]

Yeah, I think aligning the FMAP to 4K would be the best solution in that case.

File util/cbfstool/fmd_scanner.l:

https://review.coreboot.org/c/coreboot/+/37262/comment/f72d1b04_94e820db :
PS13, Line 31: .                           return *yytext;
> ',' is removed.

Okay. I guess I'm still confused why the `%` token needs to be treated differently. Why can't you just let `%` fall through to the default `return *yytext;` rule too?