Change in coreboot[master]: soc/intel/{skl,cnl,icl,jsl,elh,tgl}: make XTAL in s0ix conditional
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45723 )
Change subject: soc/intel/{skl,cnl,icl,jsl,elh,tgl}: make XTAL in s0ix conditional ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45723/2/src/soc/intel/cannonlake/ch... File src/soc/intel/cannonlake/chip.h:
https://review.coreboot.org/c/coreboot/+/45723/2/src/soc/intel/cannonlake/ch... PS2, Line 61: uint8_t s0ix_allow_xtal_on;
Thought about it a bit more and if my translation is correct, it's not just […]
I don't think this bit is really deciding whether to shut down XTAL. It basically controls whether PMC takes its state into consideration when making a decision about entering S0ix. Probably should be: xtal_blocks_s0ix?
Also, we might have to take USE_LEGACY_8254_TIMER into consideration i.e. if CONFIG_USE_LEGACY_8254_TIMER is enabled, then xtal_blocks_s0ix must be false.
-
Furquan Shaikh (Code Review)