staging: vchiq_core: avoid ternary operator for set_service_state
authorStefan Wahren <stefan.wahren@i2se.com>
Sun, 23 Jan 2022 20:02:18 +0000 (21:02 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 25 Jan 2022 15:30:16 +0000 (16:30 +0100)
There is already a check for service->sync, so use this instead of a
separate ternary operator to update the service state.

Reviewed-by: Nicolas Saenz Julienne <nsaenz@kernel.org>
Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
Link: https://lore.kernel.org/r/1642968143-19281-14-git-send-email-stefan.wahren@i2se.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c

index 7e901be..6127244 100644 (file)
@@ -1521,15 +1521,17 @@ parse_open(struct vchiq_state *state, struct vchiq_header *header)
                        if (queue_message_sync(state, NULL, openack_id, memcpy_copy_callback,
                                               &ack_payload, sizeof(ack_payload), 0) == VCHIQ_RETRY)
                                goto bail_not_ready;
+
+                       /* The service is now open */
+                       set_service_state(service, VCHIQ_SRVSTATE_OPENSYNC);
                } else {
                        if (queue_message(state, NULL, openack_id, memcpy_copy_callback,
                                          &ack_payload, sizeof(ack_payload), 0) == VCHIQ_RETRY)
                                goto bail_not_ready;
-               }
 
-               /* The service is now open */
-               set_service_state(service, service->sync ? VCHIQ_SRVSTATE_OPENSYNC
-                                       : VCHIQ_SRVSTATE_OPEN);
+                       /* The service is now open */
+                       set_service_state(service, VCHIQ_SRVSTATE_OPEN);
+               }
        }
 
        /* Success - the message has been dealt with */