Hello James, On 12/12/2017 10:17 PM, James Ettle wrote: > OK, I built a kernel 4.14.5 from vanilla kernel.org sources using the Fedora > .config (couldn't get the Fedora package to build). > > I see no difference with both [0, 1] patches applied and the tpm modules > loaded -- no keyboard or touchpad. > > [Note: I'm not *consciously* using the TPM for anything, but /dev/tpm0 is > present and have never initialised it. I don't know if anything in Fedora > uses it by default if present.] >
On a default Fedora install the TPM isn't used, but the CLKRUN_EN toggling happens for all I/O operations with the TPM and the driver does access the TPM on its probe function. So even if you don't use it, the driver will mess with the CLKRUN signal. Jarkko and Azhar, I wonder what's the solution for this. I understand the quirk is needed for some systems, but on the other hand breaks functionality for other devices. I think that at the very least this should be disabled by default and have a Kconfig option or module parameter to enable if needed. Since as James said, it causes regressions even for people that are not using the TPM device. Although probably reverting the offending commits is the right thing to do until a proper solution is proposed. Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat
