[PATCH v1] tpm/tpm_ftpm_tee: Use UUID API for exporting the UUID

[PATCH v1] tpm/tpm_ftpm_tee: Use UUID API for exporting the UUID

[Andy Shevchenko]

There is export_uuid() function which exports uuid_t to the u8 array.
Use it instead of open coding variant.

This allows to hide the uuid_t internals.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/char/tpm/tpm_ftpm_tee.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/char/tpm/tpm_ftpm_tee.c b/drivers/char/tpm/tpm_ftpm_tee.c
index 22bf553ccf9df..2491a2cb54a22 100644
--- a/drivers/char/tpm/tpm_ftpm_tee.c
+++ b/drivers/char/tpm/tpm_ftpm_tee.c
@@ -241,7 +241,7 @@ static int ftpm_tee_probe(struct platform_device *pdev)
 
 	/* Open a session with fTPM TA */
 	memset(&sess_arg, 0, sizeof(sess_arg));
-	memcpy(sess_arg.uuid, ftpm_ta_uuid.b, TEE_IOCTL_UUID_LEN);
+	export_uuid(sess_arg.uuid, &ftpm_ta_uuid);
 	sess_arg.clnt_login = TEE_IOCTL_LOGIN_PUBLIC;
 	sess_arg.num_params = 0;
 
-- 
2.26.1

Re: [PATCH v1] tpm/tpm_ftpm_tee: Use UUID API for exporting the UUID

[Jarkko Sakkinen]

On Wed, Apr 22, 2020 at 04:01:35PM +0300, Andy Shevchenko wrote:
> There is export_uuid() function which exports uuid_t to the u8 array.
> Use it instead of open coding variant.
> 
> This allows to hide the uuid_t internals.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Acked-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>

I guess no need for backports?

/Jarkko

Re: [PATCH v1] tpm/tpm_ftpm_tee: Use UUID API for exporting the UUID

[Andy Shevchenko]

On Wed, Apr 22, 2020 at 06:07:42PM +0300, Jarkko Sakkinen wrote:
> On Wed, Apr 22, 2020 at 04:01:35PM +0300, Andy Shevchenko wrote:
> > There is export_uuid() function which exports uuid_t to the u8 array.
> > Use it instead of open coding variant.
> > 
> > This allows to hide the uuid_t internals.
> > 
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> Acked-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>

Thanks!

> I guess no need for backports?

No, the API appears in v5.7-rc1.


-- 
With Best Regards,
Andy Shevchenko