* [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID
@ 2022-05-04 9:14 Andy Shevchenko
2022-05-05 16:39 ` Jakub Kicinski
2022-05-06 2:00 ` patchwork-bot+netdevbpf
0 siblings, 2 replies; 4+ messages in thread
From: Andy Shevchenko @ 2022-05-04 9:14 UTC (permalink / raw)
To: Andy Shevchenko, Allen Pais, netdev, linux-mips, linux-kernel
Cc: Rafał Miłecki, Michael Chan, Florian Fainelli,
Christoph Hellwig
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>
Reviewed-by: Christoph Hellwig <hch@lst.de>
---
v4: added tag (Christoph), resent with 126858db81a5 (in next) in mind (Florian)
drivers/firmware/broadcom/tee_bnxt_fw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/firmware/broadcom/tee_bnxt_fw.c b/drivers/firmware/broadcom/tee_bnxt_fw.c
index a5bf4c3f6dc7..40e3183a3d11 100644
--- a/drivers/firmware/broadcom/tee_bnxt_fw.c
+++ b/drivers/firmware/broadcom/tee_bnxt_fw.c
@@ -197,7 +197,7 @@ static int tee_bnxt_fw_probe(struct device *dev)
return -ENODEV;
/* Open session with Bnxt load Trusted App */
- memcpy(sess_arg.uuid, bnxt_device->id.uuid.b, TEE_IOCTL_UUID_LEN);
+ export_uuid(sess_arg.uuid, &bnxt_device->id.uuid);
sess_arg.clnt_login = TEE_IOCTL_LOGIN_PUBLIC;
sess_arg.num_params = 0;
--
2.35.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID
2022-05-04 9:14 [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID Andy Shevchenko
@ 2022-05-05 16:39 ` Jakub Kicinski
2022-05-05 17:17 ` Florian Fainelli
2022-05-06 2:00 ` patchwork-bot+netdevbpf
1 sibling, 1 reply; 4+ messages in thread
From: Jakub Kicinski @ 2022-05-05 16:39 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Allen Pais, netdev, linux-mips, linux-kernel,
Rafał Miłecki, Michael Chan, Florian Fainelli,
Christoph Hellwig
On Wed, 4 May 2022 12:14:07 +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>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> ---
> v4: added tag (Christoph), resent with 126858db81a5 (in next) in mind (Florian)
Judging by the history of the file this may go via the tee tree or
net-next. Since tee was not CCed I presume the latter is preferred.
Please let us know if that's incorrect otherwise we'll apply tomorrow :)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID
2022-05-05 16:39 ` Jakub Kicinski
@ 2022-05-05 17:17 ` Florian Fainelli
0 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2022-05-05 17:17 UTC (permalink / raw)
To: Jakub Kicinski, Andy Shevchenko
Cc: Allen Pais, netdev, linux-mips, linux-kernel,
Rafał Miłecki, Michael Chan, Christoph Hellwig
On 5/5/22 09:39, Jakub Kicinski wrote:
> On Wed, 4 May 2022 12:14:07 +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>
>> Reviewed-by: Christoph Hellwig <hch@lst.de>
>> ---
>> v4: added tag (Christoph), resent with 126858db81a5 (in next) in mind (Florian)
>
> Judging by the history of the file this may go via the tee tree or
> net-next. Since tee was not CCed I presume the latter is preferred.
> Please let us know if that's incorrect otherwise we'll apply tomorrow :)
This file has historically been without a maintainer or tree, but since
it is somewhat related to the bnxt driver, I eventually signed up
Michael to also review such patches:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=126858db81a5094d20885bc59621c3b9497f9048
If you could apply it via netdev-next/master that owuld be great, thanks!
--
Florian
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID
2022-05-04 9:14 [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID Andy Shevchenko
2022-05-05 16:39 ` Jakub Kicinski
@ 2022-05-06 2:00 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-05-06 2:00 UTC (permalink / raw)
To: Andy Shevchenko
Cc: apais, netdev, linux-mips, linux-kernel, zajec5, michael.chan,
f.fainelli, hch
Hello:
This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:
On Wed, 4 May 2022 12:14:07 +0300 you 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>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
>
> [...]
Here is the summary with links:
- [v4,1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID
https://git.kernel.org/netdev/net-next/c/10b4a11fe70f
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-05-06 2:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-04 9:14 [PATCH v4 1/1] firmware: tee_bnxt: Use UUID API for exporting the UUID Andy Shevchenko
2022-05-05 16:39 ` Jakub Kicinski
2022-05-05 17:17 ` Florian Fainelli
2022-05-06 2:00 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).