| Message ID | 3E528EFF1AE81A17+20250311131440.1468875-1-wangyuli@uniontech.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | MAINTAINERS: Add include/linux/tpm*.h to TPM maintainers | expand |
On Tue, Mar 11, 2025 at 09:14:40PM +0800, WangYuli wrote: > As of now, within include/linux, there are three header files > pertaining to the TPM driver: tpm.h, tpm_eventlg.h and tpm_command.h. > > Upon investigation when attempting code modifications, it has become > evident that these files are currently without a maintainer. > > In light of their intrinsic relationship with the TPM driver itself, > stewardship of these files should fall under the TPM subsystem. So > that scripts/get_maintainer.pl can provide more accurate output [SNIP] > > In passing, also include include/uapi/linux/vtpm_proxy.h for TPM > maintainers, as it is facing a comparable plight. > > Signed-off-by: WangYuli <wangyuli@uniontech.com> "Subnames" are separated with space, not with camel case. > --- > MAINTAINERS | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index ed7aa6867674..4a7b2e8b97de 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -24037,6 +24037,8 @@ Q: https://patchwork.kernel.org/project/linux-integrity/list/ > T: git git://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git > F: Documentation/devicetree/bindings/tpm/ > F: drivers/char/tpm/ > +F: include/linux/tpm*.h > +F: include/uapi/linux/vtpm_proxy.h > F: tools/testing/selftests/tpm2/ > > TPS546D24 DRIVER > -- > 2.47.2 > BR, Jarkko
Hi Jarkko, On 2025/3/12 14:23, Jarkko Sakkinen wrote: > "Subnames" are separated with space, not with camel case. > It's just a matter of regional cultural differences. In my country, we don't have the concept of "Subname" in names. We only have "Surname" and "Given name", and our convention is to put the surname first, followed by the given name, without a space in between. So I don't think I need to make any changes to my sign-off. Of course, as you are the TPM maintainer, I respect your opinion, and if you strongly insist, I can change my sign-off but only for all future TPM-related commits, perhaps by using Unicode characters instead of spelling it out with English letters. Thanks,
On Wed, Mar 12, 2025 at 02:48:51PM +0800, WangYuli wrote: > Hi Jarkko, > > On 2025/3/12 14:23, Jarkko Sakkinen wrote: > > "Subnames" are separated with space, not with camel case. > > > It's just a matter of regional cultural differences. > > In my country, we don't have the concept of "Subname" in names. > > We only have "Surname" and "Given name", and our convention is to put the > surname first, followed by the given name, without a space in between. > > So I don't think I need to make any changes to my sign-off. OK, I don't really mind the order, and I did not about the white space part. So with that Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org> > > Of course, as you are the TPM maintainer, I respect your opinion, and if you > strongly insist, I can change my sign-off but only for all future > TPM-related commits, perhaps by using Unicode characters instead of spelling > it out with English letters. It's always good to get educated, no worries. > > Thanks, > > -- > > 王昱力 > BR, Jarkko
diff --git a/MAINTAINERS b/MAINTAINERS index ed7aa6867674..4a7b2e8b97de 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -24037,6 +24037,8 @@ Q: https://patchwork.kernel.org/project/linux-integrity/list/ T: git git://git.kernel.org/pub/scm/linux/kernel/git/jarkko/linux-tpmdd.git F: Documentation/devicetree/bindings/tpm/ F: drivers/char/tpm/ +F: include/linux/tpm*.h +F: include/uapi/linux/vtpm_proxy.h F: tools/testing/selftests/tpm2/ TPS546D24 DRIVER
As of now, within include/linux, there are three header files pertaining to the TPM driver: tpm.h, tpm_eventlg.h and tpm_command.h. Upon investigation when attempting code modifications, it has become evident that these files are currently without a maintainer. In light of their intrinsic relationship with the TPM driver itself, stewardship of these files should fall under the TPM subsystem. So that scripts/get_maintainer.pl can provide more accurate output It should be noted that this change entails the omission of some commit_signer and author entries from the scripts/get_maintainer.pl output. A list is provided herein for future investigation: *tpm.h: Jarkko Sakkinen <jarkko@kernel.org> (commit_signer) James Bottomley <James.Bottomley@HansenPartnership.com> (commit_signer) Stefan Berger <stefanb@linux.ibm.com> (commit_signer) Ard Biesheuvel <ardb@kernel.org> (commit_signer) Roberto Sassu <roberto.sassu@huawei.com> (commit_signer) Al Viro <viro@zeniv.linux.org.uk> (author) *tpm_eventlog.h: Ard Biesheuvel <ardb@kernel.org> (commit_signer) Gregory Price <gourry@gourry.net> (commit_signer) In passing, also include include/uapi/linux/vtpm_proxy.h for TPM maintainers, as it is facing a comparable plight. Signed-off-by: WangYuli <wangyuli@uniontech.com> --- MAINTAINERS | 2 ++ 1 file changed, 2 insertions(+)