[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Add Acoustic features

[Nick Vaccaro (Code Review)]

Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45052 )

Change subject: soc/intel/tigerlake: Add Acoustic features
......................................................................

Patch Set 8:

(2 comments)

https://review.coreboot.org/c/coreboot/+/45052/8/src/soc/intel/tigerlake/fsp... 
File src/soc/intel/tigerlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/45052/8/src/soc/intel/tigerlake/fsp... 
PS8, Line 260: 	for (i = 0; i < ARRAY_SIZE(config->SlowSlewRate); i++)
Why not do this as a single memcpy using ARRAY_SIZE * sizeof array element?

https://review.coreboot.org/c/coreboot/+/45052/8/src/soc/intel/tigerlake/fsp... 
PS8, Line 264: 	for (i = 0; i < ARRAY_SIZE(config->FastPkgCRampDisable); i++)
Why not do this as a single memcpy using ARRAY_SIZE * sizeof array element?