From: Markus Elfring elfring@users.sourceforge.net Date: Fri, 31 Oct 2025 19:50:39 +0100
A pointer was assigned to a variable. The same pointer was used for the destination parameter of a memcpy() call. This function is documented in the way that the same value is returned. Thus convert two separate statements into a direct variable assignment for the return value from a memory copy action.
The source code was transformed by using the Coccinelle software.
Signed-off-by: Markus Elfring elfring@users.sourceforge.net --- drivers/greybus/es2.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/greybus/es2.c b/drivers/greybus/es2.c index 7630a36ecf81..d71d37b41a66 100644 --- a/drivers/greybus/es2.c +++ b/drivers/greybus/es2.c @@ -194,8 +194,7 @@ static int output_async(struct es2_ap_dev *es2, void *req, u16 size, u8 cmd) return -ENOMEM; }
- buf = (u8 *)dr + sizeof(*dr); - memcpy(buf, req, size); + buf = memcpy((u8 *)dr + sizeof(*dr), req, size);
dr->bRequest = cmd; dr->bRequestType = USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_INTERFACE;