[coreboot-gerrit] Change in coreboot[master]: util/ifdtool: Add Jasperlake platform support under IFDv2

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37111 )

Change subject: util/ifdtool: Add Jasperlake platform support under IFDv2
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/37111/5/util/ifdtool/ifdtool.c 
File util/ifdtool/ifdtool.c:

https://review.coreboot.org/c/coreboot/+/37111/5/util/ifdtool/ifdtool.c@1641 
PS5, Line 1641: } else if (!strcmp(optarg, "jsl")) {
              :                                 platform = PLATFORM_JSL;
> The original idea of adding this argument was to ensure ifdtool can group platforms with same IFD ve […]

The only significant difference I can see between APL and GLK descriptors is that FLMAP1[24..31] went from 0x13 on APL to 0x17 on GLK, and that half of FLCOMP was redefined, since the EC is now a possible master, which has its own FLMSTR5 register at FMBA + 0x10.

w.r.t. the SPI controller, I just made GLK work using the same code as APL, so its hardware must be the same.