diff mbox series

[-next] trusted-keys: Fix kernel-doc warnings in trusted-keys

Message ID 20230621074623.498647-1-cuigaosheng1@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] trusted-keys: Fix kernel-doc warnings in trusted-keys | expand

Commit Message

cuigaosheng June 21, 2023, 7:46 a.m. UTC
Fix kernel-doc warnings in trusted-keys:

security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
prototype for tpm_buf_append_auth(). Prototype was for
tpm2_buf_append_auth() instead.

Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
---
 security/keys/trusted-keys/trusted_tpm2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paul Moore June 21, 2023, 1:33 p.m. UTC | #1
On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
>
> Fix kernel-doc warnings in trusted-keys:
>
> security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> prototype for tpm_buf_append_auth(). Prototype was for
> tpm2_buf_append_auth() instead.
>
> Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> ---
>  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Paul Moore <paul@paul-moore.com>
Paul Moore July 25, 2023, 8:49 p.m. UTC | #2
On Wed, Jun 21, 2023 at 9:33 AM Paul Moore <paul@paul-moore.com> wrote:
>
> On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
> >
> > Fix kernel-doc warnings in trusted-keys:
> >
> > security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> > prototype for tpm_buf_append_auth(). Prototype was for
> > tpm2_buf_append_auth() instead.
> >
> > Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> > ---
> >  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Paul Moore <paul@paul-moore.com>

Jarkko, David, can one of you pick this up into your tree?
Paul Moore Aug. 3, 2023, 10 p.m. UTC | #3
On Tue, Jul 25, 2023 at 4:49 PM Paul Moore <paul@paul-moore.com> wrote:
> On Wed, Jun 21, 2023 at 9:33 AM Paul Moore <paul@paul-moore.com> wrote:
> > On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
> > >
> > > Fix kernel-doc warnings in trusted-keys:
> > >
> > > security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> > > prototype for tpm_buf_append_auth(). Prototype was for
> > > tpm2_buf_append_auth() instead.
> > >
> > > Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> > > ---
> > >  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > Reviewed-by: Paul Moore <paul@paul-moore.com>
>
> Jarkko, David, can one of you pick this up into your tree?

Guys, this patch is both trivial and obviously correct, please pick it
up so it can go up during the next merge window.
Mimi Zohar Aug. 3, 2023, 10:20 p.m. UTC | #4
On Thu, 2023-08-03 at 18:00 -0400, Paul Moore wrote:
> On Tue, Jul 25, 2023 at 4:49 PM Paul Moore <paul@paul-moore.com> wrote:
> > On Wed, Jun 21, 2023 at 9:33 AM Paul Moore <paul@paul-moore.com> wrote:
> > > On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
> > > >
> > > > Fix kernel-doc warnings in trusted-keys:
> > > >
> > > > security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> > > > prototype for tpm_buf_append_auth(). Prototype was for
> > > > tpm2_buf_append_auth() instead.
> > > >
> > > > Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> > > > ---
> > > >  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > Reviewed-by: Paul Moore <paul@paul-moore.com>
> >
> > Jarkko, David, can one of you pick this up into your tree?
> 
> Guys, this patch is both trivial and obviously correct, please pick it
> up so it can go up during the next merge window.

Paul, either Jarkko or I can queue a trusted-keys patch.  As this isn't
on the top of Jarkko's radar, I'll queue it.

Mimi
Paul Moore Aug. 3, 2023, 11:20 p.m. UTC | #5
On Thu, Aug 3, 2023 at 6:21 PM Mimi Zohar <zohar@linux.ibm.com> wrote:
>
> On Thu, 2023-08-03 at 18:00 -0400, Paul Moore wrote:
> > On Tue, Jul 25, 2023 at 4:49 PM Paul Moore <paul@paul-moore.com> wrote:
> > > On Wed, Jun 21, 2023 at 9:33 AM Paul Moore <paul@paul-moore.com> wrote:
> > > > On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
> > > > >
> > > > > Fix kernel-doc warnings in trusted-keys:
> > > > >
> > > > > security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> > > > > prototype for tpm_buf_append_auth(). Prototype was for
> > > > > tpm2_buf_append_auth() instead.
> > > > >
> > > > > Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> > > > > ---
> > > > >  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
> > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > Reviewed-by: Paul Moore <paul@paul-moore.com>
> > >
> > > Jarkko, David, can one of you pick this up into your tree?
> >
> > Guys, this patch is both trivial and obviously correct, please pick it
> > up so it can go up during the next merge window.
>
> Paul, either Jarkko or I can queue a trusted-keys patch.  As this isn't
> on the top of Jarkko's radar, I'll queue it.

Ah ha!  I was working under the assumption that everything under
security/keys/ was David and Jarko's responsibility, but now that I'm
looking at MAINTAINERS I see that isn't the case.  In fact it looks
like there are five (!) different "subsystems" under security/keys/
... I don't currently know enough about the different divisions there,
but it seems like we might want to see if we can do some consolidation
...

Anyway, thanks for picking this up Mimi.
Mimi Zohar Aug. 4, 2023, 11:57 a.m. UTC | #6
On Thu, 2023-08-03 at 19:20 -0400, Paul Moore wrote:
> On Thu, Aug 3, 2023 at 6:21 PM Mimi Zohar <zohar@linux.ibm.com> wrote:
> >
> > On Thu, 2023-08-03 at 18:00 -0400, Paul Moore wrote:
> > > On Tue, Jul 25, 2023 at 4:49 PM Paul Moore <paul@paul-moore.com> wrote:
> > > > On Wed, Jun 21, 2023 at 9:33 AM Paul Moore <paul@paul-moore.com> wrote:
> > > > > On Wed, Jun 21, 2023 at 3:46 AM Gaosheng Cui <cuigaosheng1@huawei.com> wrote:
> > > > > >
> > > > > > Fix kernel-doc warnings in trusted-keys:
> > > > > >
> > > > > > security/keys/trusted-keys/trusted_tpm2.c:203: warning: expecting
> > > > > > prototype for tpm_buf_append_auth(). Prototype was for
> > > > > > tpm2_buf_append_auth() instead.
> > > > > >
> > > > > > Signed-off-by: Gaosheng Cui <cuigaosheng1@huawei.com>
> > > > > > ---
> > > > > >  security/keys/trusted-keys/trusted_tpm2.c | 2 +-
> > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > >
> > > > > Reviewed-by: Paul Moore <paul@paul-moore.com>
> > > >
> > > > Jarkko, David, can one of you pick this up into your tree?
> > >
> > > Guys, this patch is both trivial and obviously correct, please pick it
> > > up so it can go up during the next merge window.
> >
> > Paul, either Jarkko or I can queue a trusted-keys patch.  As this isn't
> > on the top of Jarkko's radar, I'll queue it.
> 
> Ah ha!  I was working under the assumption that everything under
> security/keys/ was David and Jarko's responsibility, but now that I'm
> looking at MAINTAINERS I see that isn't the case.  In fact it looks
> like there are five (!) different "subsystems" under security/keys/
> ... I don't currently know enough about the different divisions there,
> but it seems like we might want to see if we can do some consolidation
> ...
> 
> Anyway, thanks for picking this up Mimi.

Thanks, Steven.  "6c95d71bad61 tracing: Fix kernel-doc warnings in
trace_seq.c " is in Linus' tree.

Mimi
diff mbox series

Patch

diff --git a/security/keys/trusted-keys/trusted_tpm2.c b/security/keys/trusted-keys/trusted_tpm2.c
index 2b2c8eb258d5..bc700f85f80b 100644
--- a/security/keys/trusted-keys/trusted_tpm2.c
+++ b/security/keys/trusted-keys/trusted_tpm2.c
@@ -186,7 +186,7 @@  int tpm2_key_priv(void *context, size_t hdrlen,
 }
 
 /**
- * tpm_buf_append_auth() - append TPMS_AUTH_COMMAND to the buffer.
+ * tpm2_buf_append_auth() - append TPMS_AUTH_COMMAND to the buffer.
  *
  * @buf: an allocated tpm_buf instance
  * @session_handle: session handle