[S] Change in coreboot[main]: mb/intel/ptlrvp: Update flashmap to allocate 14MB to BIOS

[Jamie Ryu (Code Review)]

Attention is currently required from: Bora Guvendik, Jayvik Desai, Jérémy Compostella, Subrata Banik, Wonkyu Kim, Wonkyu Kim, Zhixing Ma, [e-mail].

Jamie Ryu has posted comments on this change by Jamie Ryu. ( https://review.coreboot.org/c/coreboot/+/87035?usp=email )

Change subject: mb/intel/ptlrvp: Update flashmap to allocate 14MB to BIOS
......................................................................

Patch Set 4:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/87035/comment/4d78c255_ab4997b5?usp... :
PS2, Line 9: This updates flashmap descriptor to allocate 18MB to SI_ME
           : and 14MB for BIOS. PTL RVP coreboot is used with several types of RVP boards and this layout with 14MB BIOS is very convenient for debugging and creating coreboot for certain use cases and tests purpose.
> This updates the Flashmap (FMAP) descriptor to allocate 18MB to Silicon […]

Thanks! The commit message is updated accordingly.

https://review.coreboot.org/c/coreboot/+/87035/comment/c2842fd5_ff61bda3?usp... :
PS2, Line 12: Build the ptlrvp variant(ES) and check flashmap of the coreboot
            : is updated correctly.
> TEST=Build the ptlrvp variant (ES) and check if the flashmap of the […]

Thanks. it is updated by the patchset#4.

File src/mainboard/intel/ptlrvp/chromeos.fmd:

https://review.coreboot.org/c/coreboot/+/87035/comment/a1b61abc_ce23abc3?usp... :
PS2, Line 14: 	# This section starts at the 16M boundary in SPI flash.
            : 		# PTL does not support a region crossing this boundary,
            : 		# because the SPI flash is memory-mapped into two non-
            : 		# contiguous windows.
> Is this still correct ? It looks like SI_ME_PAD starts at 16 MB. So RW_SECTION_B cannot be at 16 MB.

The comments is removed from fmd file in patchset#4.