linux-kernel - Re: [PATCH v2 3/3] tpm: Add a driver for Loongson TPM device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Z65tuC722nnuhWEO@kernel.org>
Date: Fri, 14 Feb 2025 00:10:00 +0200
From: Jarkko Sakkinen <jarkko@...nel.org>
To: Qunqin Zhao <zhaoqunqin@...ngson.cn>
Cc: lee@...nel.org, herbert@...dor.apana.org.au, davem@...emloft.net,
	peterhuewe@....de, linux-kernel@...r.kernel.org,
	loongarch@...ts.linux.dev, linux-crypto@...r.kernel.org,
	jgg@...pe.ca, linux-integrity@...r.kernel.org,
	Yinggang Gu <guyinggang@...ngson.cn>
Subject: Re: [PATCH v2 3/3] tpm: Add a driver for Loongson TPM device

On Thu, Feb 13, 2025 at 11:56:25AM +0800, Qunqin Zhao wrote:
> 
> 在 2025/2/13 上午5:28, Jarkko Sakkinen 写道:
> > On Wed, Feb 12, 2025 at 11:31:13AM +0800, Qunqin Zhao wrote:
> > > TPM2.0 is implemented in Loongson security engine. This is the driver
> > TPM 2.0 or just TPM2 (either is fine with me). Quick reminder what
> > Loongson security engine (I could not know that and it would be nice
> > to have that in git log).
> 
> Looks like just TPM2 is better.

Either is fine :-)

> 
> Some explanations about Loongson security engine will be added to git log.
> 
> > 
> > > for it.
> > > 
> > > Co-developed-by: Yinggang Gu <guyinggang@...ngson.cn>
> > > Signed-off-by: Yinggang Gu <guyinggang@...ngson.cn>
> > > Signed-off-by: Qunqin Zhao <zhaoqunqin@...ngson.cn>
> > > ---
> > >   MAINTAINERS                 |   1 +
> > >   drivers/char/tpm/Kconfig    |   9 ++++
> > >   drivers/char/tpm/Makefile   |   1 +
> > >   drivers/char/tpm/tpm_lsse.c | 104 ++++++++++++++++++++++++++++++++++++
> > >   4 files changed, 115 insertions(+)
> > >   create mode 100644 drivers/char/tpm/tpm_lsse.c
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 6493d58436..6aad0f08ad 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -13484,6 +13484,7 @@ LOONGSON CRYPTO DRIVER
> > >   M:	Qunqin Zhao <zhaoqunqin@...ngson.com>
> > >   L:	linux-crypto@...r.kernel.org
> > >   S:	Maintained
> > > +F:	drivers/char/tpm/tpm_lsse.c
> > >   F:	drivers/crypto/loongson/
> > >   LOONGSON-2 APB DMA DRIVER
> > Probably MAINTAINERS update should be a separate patch.
> 
> Some  MAINTAINERS updates are not  separated form the driver patch.  Like
> the submit of "drivers/mfd/max7714*".
> 
> So it seems whether the updates to MAINTAINERS  are separated or not is OK.

I'd prefer them separated from code changes. They are separate tasks
per se.

BR, Jarkko