Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39098 )
Change subject: soc/intel/tigerlake: Integrate Legacy 8254 timer support ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/39098/2/src/soc/intel/tigerlake/fsp... File src/soc/intel/tigerlake/fsp_params_tgl.c:
https://review.coreboot.org/c/coreboot/+/39098/2/src/soc/intel/tigerlake/fsp... PS2, Line 139: Enable8254ClockGating
we will update FSP header by EOD.
Ack
https://review.coreboot.org/c/coreboot/+/39098/2/src/soc/intel/tigerlake/fsp... PS2, Line 140: 1
yes, make sense, will clean for other SOC as well at once
Ack