[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Change in SaGv options

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30917 )

Change subject: soc/intel/cannonlake: Change in SaGv options
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/#/c/30917/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/30917/1//COMMIT_MSG@9 
PS1, Line 9: CML
> What does CML stand for?

Got it figured out, thanks for your explanation.

https://review.coreboot.org/#/c/30917/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/30917/2//COMMIT_MSG@9 
PS2, Line 9: CNL,WHL and CFL all are not using midfixed option in SaGv so keeping it for
           : CNL only
Does CNL use the midfixed option or not?