[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Enables CSE Lite driver for JSL platform in the...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48280 )

Change subject: soc/intel/jasperlake: Enables CSE Lite driver for JSL platform in the romstage
......................................................................

Patch Set 6: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/48280/6/src/soc/intel/jasperlake/ro... 
File src/soc/intel/jasperlake/romstage/romstage.c:

https://review.coreboot.org/c/coreboot/+/48280/6/src/soc/intel/jasperlake/ro... 
PS6, Line 140: cse_fw_sync() must be called after DRAM initialization
It would be helpful to also add the information that HMRFPO (which is used by cse_fw_sync()) is expected to be executed only after DRAM initialization.