Change in coreboot[master]: soc/intel/{skl,cnl,icl,jsl,elh,tgl}: make XTAL S0ix qualification opt...

[Nico Huber (Code Review)]

Nico Huber 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 S0ix qualification optional
......................................................................

Patch Set 3:

(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;
> > 8254 clock gating is required for XTAL shutdown
> > XTAL shutdown is necessary for S0ix unless disqualification bit is set.
> Exactly. That's what I understood, too.

I was there too, then saw CB:22237. Why did they want to clear the bit?

And most of all, why do you want to clear the bit then?