Commit d3de2bb8 authored by Stefan Wahren's avatar Stefan Wahren Committed by Greg Kroah-Hartman

staging: vchiq_core: Use return value of mutex_lock_killable directly

Instead of saving the return value of mutex_lock_killable in a
local variable we could use the value directly.
Signed-off-by: default avatarStefan Wahren <stefan.wahren@i2se.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5b29aaaa
...@@ -1376,7 +1376,6 @@ resolve_bulks(VCHIQ_SERVICE_T *service, VCHIQ_BULK_QUEUE_T *queue) ...@@ -1376,7 +1376,6 @@ resolve_bulks(VCHIQ_SERVICE_T *service, VCHIQ_BULK_QUEUE_T *queue)
{ {
VCHIQ_STATE_T *state = service->state; VCHIQ_STATE_T *state = service->state;
int resolved = 0; int resolved = 0;
int rc;
while ((queue->process != queue->local_insert) && while ((queue->process != queue->local_insert) &&
(queue->process != queue->remote_insert)) { (queue->process != queue->remote_insert)) {
...@@ -1392,8 +1391,7 @@ resolve_bulks(VCHIQ_SERVICE_T *service, VCHIQ_BULK_QUEUE_T *queue) ...@@ -1392,8 +1391,7 @@ resolve_bulks(VCHIQ_SERVICE_T *service, VCHIQ_BULK_QUEUE_T *queue)
WARN_ON(!((int)(queue->local_insert - queue->process) > 0)); WARN_ON(!((int)(queue->local_insert - queue->process) > 0));
WARN_ON(!((int)(queue->remote_insert - queue->process) > 0)); WARN_ON(!((int)(queue->remote_insert - queue->process) > 0));
rc = mutex_lock_killable(&state->bulk_transfer_mutex); if (mutex_lock_killable(&state->bulk_transfer_mutex))
if (rc != 0)
break; break;
vchiq_transfer_bulk(bulk); vchiq_transfer_bulk(bulk);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment