e1000: fix missing link interrupt check brackets
authorAaron Conole <aconole@redhat.com>
Tue, 22 Mar 2016 21:37:14 +0000 (17:37 -0400)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 31 Mar 2016 15:09:23 +0000 (17:09 +0200)
The device lsc interrupt check has a misleading whitespace around it which
can be improved by adding appropriate braces to the check. Since the ret
variable was checked after previous assignment, this introduces no functional
change.

Fixes: 921a72008f76 ("e1000: add Rx interrupt handler")

Signed-off-by: Aaron Conole <aconole@redhat.com>
Acked-by: Panu Matilainen <pmatilai@redhat.com>
drivers/net/e1000/em_ethdev.c

index 1f0a7f4..9b5e1af 100644 (file)
@@ -666,13 +666,14 @@ eth_em_start(struct rte_eth_dev *dev)
 
        if (rte_intr_allow_others(intr_handle)) {
                /* check if lsc interrupt is enabled */
-               if (dev->data->dev_conf.intr_conf.lsc != 0)
+               if (dev->data->dev_conf.intr_conf.lsc != 0) {
                        ret = eth_em_interrupt_setup(dev);
                        if (ret) {
                                PMD_INIT_LOG(ERR, "Unable to setup interrupts");
                                em_dev_clear_queues(dev);
                                return ret;
                        }
+               }
        } else {
                rte_intr_callback_unregister(intr_handle,
                                                eth_em_interrupt_handler,