[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Disable hybrid storage mode in CSE Lite RO boot

[Jamie Ryu (Code Review)]

Jamie Ryu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42282 )

Change subject: soc/intel/tigerlake: Disable hybrid storage mode in CSE Lite RO boot
......................................................................

Patch Set 16: Code-Review+1

(4 comments)

https://review.coreboot.org/c/coreboot/+/42282/15//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42282/15//COMMIT_MSG@10 
PS15, Line 10: a Optane
> an

Done

https://review.coreboot.org/c/coreboot/+/42282/15//COMMIT_MSG@10 
PS15, Line 10: strap  configuration
> Please use just one space.

Done

https://review.coreboot.org/c/coreboot/+/42282/15//COMMIT_MSG@14 
PS15, Line 14: CSE Lite supports the strap override when running on CSE RW partition,
> Please add a blank line above to separate paragraphs visibly.

Done

https://review.coreboot.org/c/coreboot/+/42282/15/src/soc/intel/tigerlake/fs... 
File src/soc/intel/tigerlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/42282/15/src/soc/intel/tigerlake/fs... 
PS15, Line 268: 		printk(BIOS_DEBUG, "cse_lite: CSE RO boot in recovery mode\n");
> Make it level INFO, and mention that hybrid storage mode is disabled?

Done. Thanks for comments!