RE: [tpmdd-devel] [PATCH v3 4/7] tpm: infrastructure for TPM spaces

2 messages, 2 authors, 2017-03-17 · open the first message on its own page

RE: [tpmdd-devel] [PATCH v3 4/7] tpm: infrastructure for TPM spaces

From: <hidden>
Date: 2017-03-17 18:57:44

quoted hunk
diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-
interface.c
index 20b1fe3..db5ffe9 100644
--- a/drivers/char/tpm/tpm-interface.c
+++ b/drivers/char/tpm/tpm-interface.c
@@ -376,11 +376,12 @@ static bool tpm_validate_command(struct
tpm_chip *chip, const u8 *cmd,
  *     0 when the operation is successful.
  *     A negative number for system errors (errno).
  */
-ssize_t tpm_transmit(struct tpm_chip *chip, const u8 *buf, size_t bufsiz,
-		     unsigned int flags)
+ssize_t tpm_transmit(struct tpm_chip *chip, struct tpm_space *space,
+		     u8 *buf, size_t bufsiz, unsigned int flags)
When adding parameters, please also update the parameter documentation at the top of the function. It is missing for the new parameter "space" here.

Alexander
????{.n?+???????+%?????????w??{.n?+????{??????????v?^?)????w*jg???
???????j????G??????
???j:+v???w?j?m?????
??
?w?????f???h?????????

Re: [tpmdd-devel] [PATCH v3 4/7] tpm: infrastructure for TPM spaces

From: Jarkko Sakkinen <hidden>
Date: 2017-03-17 20:46:37

On Fri, Mar 17, 2017 at 03:41:38PM +0000, Alexander.Steffen at infineon.com wrote:
quoted
diff --git a/drivers/char/tpm/tpm-interface.c b/drivers/char/tpm/tpm-
interface.c
index 20b1fe3..db5ffe9 100644
--- a/drivers/char/tpm/tpm-interface.c
+++ b/drivers/char/tpm/tpm-interface.c
@@ -376,11 +376,12 @@ static bool tpm_validate_command(struct
tpm_chip *chip, const u8 *cmd,
  *     0 when the operation is successful.
  *     A negative number for system errors (errno).
  */
-ssize_t tpm_transmit(struct tpm_chip *chip, const u8 *buf, size_t bufsiz,
-		     unsigned int flags)
+ssize_t tpm_transmit(struct tpm_chip *chip, struct tpm_space *space,
+		     u8 *buf, size_t bufsiz, unsigned int flags)
When adding parameters, please also update the parameter documentation
at the top of the function. It is missing for the new parameter
"space" here.
I missed that. I'll update the commits with the proper documentation.
Alexander
/Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-security-module" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help