[coreboot-gerrit] Change in coreboot[master]: soc/intel/jasperlake: Update acoustic noise related parameters

[Maulik V Vaghela (Code Review)]

Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49187 )

Change subject: soc/intel/jasperlake: Update acoustic noise related parameters
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/49187/1/src/soc/intel/jasperlake/fs... 
File src/soc/intel/jasperlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/49187/1/src/soc/intel/jasperlake/fs... 
PS1, Line 240:
> Check for superfluous whitespace in the tree (lint-stable-003-whitespace): File src/soc/intel/jasper […]

Done

https://review.coreboot.org/c/coreboot/+/49187/1/src/soc/intel/jasperlake/fs... 
PS1, Line 244: 0
> For which VR domain is this? Why is the assumption here that the mainboard setting is for VR domain- […]

Hi Furquan,
Jasperlake only supports VR domain 0 and it doesn't have multiple VR domains as in TGL.
That is the reason we only set it for VR domain 0.
Reference: JSL EDS vol1 (Document# 613601, chapter 3.4.1)