[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Enable ACPI timer emulation if PM timer is disa...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34563 )

Change subject: soc/intel/cannonlake: Enable ACPI timer emulation if PM timer is disabled
......................................................................

Patch Set 3:
> Patch Set 3:
> > Created a revert of this change as If0d7fa6f9766c7c4e2fa9e846c179adc6a4e1681
>
> I guess we should first ask why this check was added?
>
> Aamir, Subrata, Furquan, do you know the reason for this
> change? The BIOS spec seems to suggest to always enable
> it. Are there any downsides?
>
> Please always mention the reason for a change in the
> commit message.

Posted response on https://review.coreboot.org/c/coreboot/+/45000.