[coreboot-gerrit] [M] Change in coreboot[master]: mb/google/myst: Add new mainboard

[Karthik Ramasubramanian (Code Review)]

Attention is currently required from: Raul Rangel, Martin L Roth, Jon Murphy, Martin Roth, Fred Reitberger, Tim Van Patten, Felix Held, Mark Hasemeyer.

Karthik Ramasubramanian has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74093 )

Change subject: mb/google/myst: Add new mainboard
......................................................................

Patch Set 6:

(2 comments)

File src/mainboard/google/myst/chromeos.fmd:

https://review.coreboot.org/c/coreboot/+/74093/comment/648168f0_fc071c22 
PS2, Line 2: 	SI_BIOS {
> RECOVERY_MRC_CACHE should be 256K, same as the size of RW_MRC_CACHE […]

Fred, Why is the MRC region increasing? I would like to understand since the memory technology is the same in both Skyrim and Myst.

This has doubled between Guybrush and Skyrim. Now it has doubled again between Skyrim and Myst.

File src/mainboard/google/myst/chromeos.fmd:

https://review.coreboot.org/c/coreboot/+/74093/comment/98f5644d_a1ce8925 
PS3, Line 36: }
> RO should cover the lowest flash offsets.  See the birman `chromeos_phoenix. […]

Given the lack of clarity on how things are looking for 32 MB SPI support, do you think we can address the comments for 16 MB layout and land that as a first pass.

Once the required support has landed, we can add the 32 MB layout.