From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=100.103.45.18 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1792771698; b=aV28ee8Bl5gfkMtFIn7SRIqFoyq1v9NNTRPLtH5y+hG/M7AA5tFDbdkWjIMUNxXGPHY4 ji5PW5JTMa/4hpK5N4TloS8l1E+5ed7KUjlXrt77Y+O2txq0efcUiwUI4fn2NCBoIGstx /7kol1dpAt4JHNLms6fKOyUZxNJJAjacK86mvXx0srhayV0ieWbjU0aLw1YAGjSwCK8W5 ugMJz0o67aSA8cpun9bSSt///HpoUCmBqFv1FRd8D/glEeQ+MjjXmlUYiw6DUUXPvV3Uy iBu10+z6zdap5oKaU9zWimdBYTf96xpLmPICdftu/1Td6ncfRtNJH6xkAVw5wjQPqfQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1792771698; h=Received:DKIM-Signature:Date:From:To:Cc:Subject:Message-ID: References:MIME-Version:Content-Type:Content-Disposition:In-Reply-To; bh=TmhafjD6XNoUqXmShAafFQ+tXkd+0JlWrZOP6ljtztU=; b=EWQL/i1VbbE763cYmOpvpBdALBLKLf3d6gRew8RbSej5CUmtW1b4m6jy0cfCkgdTWCHa xUW/6ogcgTft9R2L9IVxLUcvEoPFQJU/cyusiGIQJwUOlZ+qpu37cSXsBFxHjrJvxhWtX jlrlkOUas46sy9UW4utJrvX49d6IhIjyUzC4A4Fsc0xEyWRynMd05Jx0WTCdiElr3dfAM mhPB9ymEI4sc5h43h8KHnvhPdO4CQhZW8OMfBBAKGrIFkXegfFZIyNR7jmqmrImz1pOnv yAktzNe+ftN+/iDdng0V86P7xF0RqOTZbPrZCRkJEwUBe8BesmznQoulaEI/wyYglaQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=JAV0ueGa; arc=none smtp.remote-ip=100.103.45.18 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with UTF8SMTPS id 6D8F4606A31B; Fri, 23 Oct 2026 16:08:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=JAV0ueGa Received: by smtp.kernel.org (Postfix) with UTF8SMTPSA id 82CAA1F01828; Fri, 23 Oct 2026 16:08:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1792771698; bh=TmhafjD6XNoUqXmShAafFQ+tXkd+0JlWrZOP6ljtztU=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=JAV0ueGa3gNMEFHhMfVpX/62RQSLvQG1JT5+7E3SsptzCOVWz0KQmuhNGLopo0LLI UZXg7BeklmQHJ2Hli6d8DWxg+4moGbiEFZKMYKMIKWV9v/ECFeErl93KkHOe08HVTD 7LOSQ+a55OArj/XDWsDABd7eQ00HJBA9rS0reTYc7ahi+XeXroh2HtgRyRNLi/ukP7 Sub/8Sy1M1aY8QZemS9MmbejS5InFDn5O+cqeYeZY0gznK78rSoPSKOV6/2/w5/pmq 499edhbDzaOkC5gF8a4Qp9qwwDVhMvHqbXjjOXgSQwd6zuBpsKceCuXLwUQ0Ddy22h OTN6qFn/OJ+OQ== Date: Fri, 23 Oct 2026 19:08:13 +0300 From: Jarkko Sakkinen To: Linus Torvalds Cc: Peter Huewe , Jason Gunthorpe , David Howells , keyrings@vger.kernel.org, linux-integrity@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [GIT PULL] TPM DEVICE DRIVER changes for v7.4-rc1 Message-ID: References: X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Oct 23, 2026 at 06:11:03PM +0300, Jarkko Sakkinen wrote: > On Fri, Oct 23, 2026 at 06:05:05PM +0300, Jarkko Sakkinen wrote: > > The following changes since commit 722658cac48a6ba49f5506fd74d010ad9dbb2f05: > > > > keys/trusted_keys: reuse TPM version in option handling (2026-10-20 18:15:40 +0300) > > > > are available in the Git repository at: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git tags/tpm-v7.4-rc1 > > > > for you to fetch changes up to 836952696d0260cca1de6d3300cd8fcd5687fe67: > > > > tpm: reject duplicate PCR banks in tpm2_get_pcr_allocation (2026-10-21 13:20:58 +0300) > > > > ---------------------------------------------------------------- > > tpm: changes for v7.4-rc1 > > > > FEATURES AND IMPROVEMENTS > > ========================= > > > > - tpm_crb behaviour was improved by putting the chip into an idle state for > > each transmission. This improves firmware behavior as the kernel now > > better cancels its state as if it had never touched it in the first > > place. > > - One spurious (but harmless) wmb() was dropped from tpm_pm_resume(). > > - tpm_tis_i2c now requests the optional reset control and deasserts it > > before communicating with the TPM. > > - SIMPLE_DEV_PM_OPS was replaced with DEFINE_SIMPLE_DEV_PM_OPS across the > > whole driver stack. > > > > BUGS > > ==== > > > > - tpm2_probe() misinterpreted a transport error on TPM2_GetCapability > > as the chip not being probed, and the chip was detected as a TPM 1 chip. > > This can happen in reality only in the extremely rare case of bus > > corruption. Despite the odds, a fix was landed, as it is still > > technically a legit bug fix. > > - It is not really worth the time and energy to go into details as tpm_nsc > > is a somewhat legacy driver, but one null pointer dereference bug and > > another causing use-after-free were fixed. > > - read() of /dev/tpm{,rm}0 with zero size caused a pending response to be > > consumed. This was fixed simply by an immediate return. > > > > Signed-off-by: Jarkko Sakkinen > > > > ---------------------------------------------------------------- > > Alexander Sverdlin (1): > > tpm: tpm2_probe: propagate transport errors > > > > Li Jun (2): > > char: tpm: Use SIMPLE_DEV_PM_OPS for ibmvtpm power management > > tpm: use DEFINE_SIMPLE_DEV_PM_OPS and pm_sleep_ptr() > > > > Mario Limonciello (1): > > tpm: Call cmd_ready/go_idle for each command transmission > > > > Pei Xiao (3): > > tpm: tpm_nsc: fix NULL pointer dereference on init failure > > tpm: tpm_nsc: stop using the cleanup callback as dev.release > > tpm: fix zero-length read discarding the pending response > > > > Richard Lyu (1): > > tpm: Remove ineffective wmb() from tpm_pm_resume() > > > > Stefano Radaelli (1): > > tpm: tis_i2c: Deassert optional reset line before probing > > > > Yuqi Xu (1): > > tpm: reject duplicate PCR banks in tpm2_get_pcr_allocation > > > > drivers/char/tpm/tpm-chip.c | 24 ------------------------ > > drivers/char/tpm/tpm-dev-common.c | 3 +++ > > drivers/char/tpm/tpm-interface.c | 28 ++++++++++++++++++++++------ > > drivers/char/tpm/tpm2-cmd.c | 21 ++++++++++++++++++++- > > drivers/char/tpm/tpm_atmel.c | 4 ++-- > > drivers/char/tpm/tpm_i2c_atmel.c | 4 ++-- > > drivers/char/tpm/tpm_i2c_infineon.c | 4 ++-- > > drivers/char/tpm/tpm_i2c_nuvoton.c | 4 ++-- > > drivers/char/tpm/tpm_ibmvtpm.c | 5 +---- > > drivers/char/tpm/tpm_infineon.c | 7 +++---- > > drivers/char/tpm/tpm_nsc.c | 12 ++++++++---- > > drivers/char/tpm/tpm_tis.c | 4 ++-- > > drivers/char/tpm/tpm_tis_core.h | 2 -- > > drivers/char/tpm/tpm_tis_i2c.c | 16 ++++++++++++++-- > > drivers/char/tpm/tpm_tis_i2c_cr50.c | 4 ++-- > > drivers/char/tpm/tpm_tis_spi_main.c | 4 ++-- > > drivers/char/tpm/tpm_tis_synquacer.c | 4 ++-- > > 17 files changed, 87 insertions(+), 63 deletions(-) > > > > My templates are a bit out-of-sync, thus the divergence in subject > lines. I'm working on my pull request scripts, and address this while at > it so that subjects, tags etc. have same stuff in the same format. Just > started distract me when I realized this after looking at what went to > lore so though to mention. > > Br, Jarkko I need to redo this PR mishap happened with linux-next so please ignore. I had my own tree in wrong state. For tags and subject I'll use MAINTAINERS defined subsystem name. Br, Jarkko