From 736adc5a11119b0f2cd7355283fc2716b3dff0b1 Mon Sep 17 00:00:00 2001 From: Augustin Cavalier Date: Sat, 20 Jul 2019 09:52:14 -0400 Subject: [PATCH] XHCI: Tweak formatting of FinishTransfers logic. No functional change intended. --- src/add-ons/kernel/busses/usb/xhci.cpp | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/src/add-ons/kernel/busses/usb/xhci.cpp b/src/add-ons/kernel/busses/usb/xhci.cpp index d9c21b6ce7..57e49212a1 100644 --- a/src/add-ons/kernel/busses/usb/xhci.cpp +++ b/src/add-ons/kernel/busses/usb/xhci.cpp @@ -2802,16 +2802,14 @@ XHCI::FinishTransfers() TRACE("transfer not successful, actualLength=%" B_PRIuSIZE "\n", actualLength); } - if (callbackStatus == B_OK) { - if (directionIn && actualLength > 0) { - TRACE("copying in iov count %ld\n", transfer->VectorCount()); - status_t status = transfer->PrepareKernelAccess(); - if (status == B_OK) { - ReadDescriptor(td, transfer->Vector(), - transfer->VectorCount()); - } else { - callbackStatus = status; - } + if (callbackStatus == B_OK && directionIn && actualLength > 0) { + TRACE("copying in iov count %ld\n", transfer->VectorCount()); + status_t status = transfer->PrepareKernelAccess(); + if (status == B_OK) { + ReadDescriptor(td, transfer->Vector(), + transfer->VectorCount()); + } else { + callbackStatus = status; } } transfer->Finished(callbackStatus, actualLength);