[PATCH] staging: greybus: Reformat code in gb_operation_sync_timeout()

From: Thorsten Blum
Date: Sun Apr 13 2025 - 06:48:49 EST


Remove any unnecessary curly braces and combine 'else' and 'if' to an
'else if' to improve the code's readability and reduce indentation.

Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx>
---
drivers/greybus/operation.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/greybus/operation.c b/drivers/greybus/operation.c
index 8459e9bc0749..ba26504ccac3 100644
--- a/drivers/greybus/operation.c
+++ b/drivers/greybus/operation.c
@@ -1157,16 +1157,12 @@ int gb_operation_sync_timeout(struct gb_connection *connection, int type,
memcpy(operation->request->payload, request, request_size);

ret = gb_operation_request_send_sync_timeout(operation, timeout);
- if (ret) {
+ if (ret)
dev_err(&connection->hd->dev,
"%s: synchronous operation id 0x%04x of type 0x%02x failed: %d\n",
connection->name, operation->id, type, ret);
- } else {
- if (response_size) {
- memcpy(response, operation->response->payload,
- response_size);
- }
- }
+ else if (response_size)
+ memcpy(response, operation->response->payload, response_size);

gb_operation_put(operation);

--
2.49.0