usb: cdnsp: Fixes incorrect value in ISOC TRB

Fixes issue with priority of operator. Operator "|" priority is
higher then "? :".
To improve the readability the operator "? :" has been replaced with
"if ()" statement.

Fixes: 3d82904559 ("usb: cdnsp: cdns3 Add main part of Cadence USBSSP DRD Driver")
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Pawel Laszczak <pawell@cadence.com>
Signed-off-by: Peter Chen <peter.chen@kernel.org>
This commit is contained in:
Pawel Laszczak 2021-03-08 08:31:03 +01:00 committed by Peter Chen
parent a38fd87484
commit d0dcd90b7f
No known key found for this signature in database
GPG Key ID: 4859298150D671BB
1 changed files with 4 additions and 1 deletions

View File

@ -2197,7 +2197,10 @@ static int cdnsp_queue_isoc_tx(struct cdnsp_device *pdev,
* inverted in the first TDs isoc TRB. * inverted in the first TDs isoc TRB.
*/ */
field = TRB_TYPE(TRB_ISOC) | TRB_TLBPC(last_burst_pkt) | field = TRB_TYPE(TRB_ISOC) | TRB_TLBPC(last_burst_pkt) |
start_cycle ? 0 : 1 | TRB_SIA | TRB_TBC(burst_count); TRB_SIA | TRB_TBC(burst_count);
if (!start_cycle)
field |= TRB_CYCLE;
/* Fill the rest of the TRB fields, and remaining normal TRBs. */ /* Fill the rest of the TRB fields, and remaining normal TRBs. */
for (i = 0; i < trbs_per_td; i++) { for (i = 0; i < trbs_per_td; i++) {