[coreboot-gerrit] [L] Change in coreboot[master]: mb/intel: add Archer City CRB support

[Paul Menzel (Code Review)]

Attention is currently required from: Patrick Rudolph, Simon Chou, Johnny Lin, Shuming Chu (Shuming), Arthur Heymans, Juan Sanchez.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/71968 )

Change subject: mb/intel: add Archer City CRB support
......................................................................

Patch Set 28:

(2 comments)

File src/mainboard/intel/archercity_crb/romstage.c:

https://review.coreboot.org/c/coreboot/+/71968/comment/f771c530_0b414a94 
PS27, Line 83: 		mupd->FspmConfig.DelayAfterPCIeLinkTraining = 2000;
> I think the above comment already has mentioned ms.

I meant remove the redundant comment, and just put the unit as a comment on the same line.

https://review.coreboot.org/c/coreboot/+/71968/comment/d4d44114_d2c9a8b9 
PS27, Line 96: 			"SerialIoUartDebugEnable to %d\n", FSP_LOG, FSP_LOG_DEFAULT);
> Ack. Put into online would exceed 96 characters.

Exceeding the limit is allowed for print strings. The reasoning is, that it’s easier to grep.