| Message ID | 20130905035128.GA15824@redhat.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
Dave Jones <davej@redhat.com> writes: > The indentation here implies this was meant to be a multi-statement > if, but it lacks the braces. > > Signed-off-by: Dave Jones <davej@fedoraproject.org> Thanks, applied. I added "ath10k:" to the patch title.
diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c index 33af467..31b69d3 100644 --- a/drivers/net/wireless/ath/ath10k/pci.c +++ b/drivers/net/wireless/ath/ath10k/pci.c @@ -1200,7 +1200,7 @@ static void ath10k_pci_tx_pipe_cleanup(struct hif_ce_pipe_info *pipe_info) while (ath10k_ce_cancel_send_next(ce_hdl, (void **)&netbuf, &ce_data, &nbytes, &id) == 0) { - if (netbuf != CE_SENDLIST_ITEM_CTXT) + if (netbuf != CE_SENDLIST_ITEM_CTXT) { /* * Indicate the completion to higer layer to free * the buffer @@ -1209,6 +1209,7 @@ static void ath10k_pci_tx_pipe_cleanup(struct hif_ce_pipe_info *pipe_info) ar_pci->msg_callbacks_current.tx_completion(ar, netbuf, id); + } } }
The indentation here implies this was meant to be a multi-statement if, but it lacks the braces. Signed-off-by: Dave Jones <davej@fedoraproject.org> -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html