[PATCH 3/3] staging/gdm72xx: Remove duplicated code in gdm_qos.c

[Peter Huewe]

The first branch of the if statement is ended with a return thus there
is no need for an else if, and thus we can move the duplicated code to
the top and use it for the other two branches.

Signed-off-by: Peter Huewe <peterhu...@gmx.de>
---
 drivers/staging/gdm72xx/gdm_qos.c |   42 +++++++++++++++----------------------
 1 files changed, 17 insertions(+), 25 deletions(-)

diff --git a/drivers/staging/gdm72xx/gdm_qos.c 
b/drivers/staging/gdm72xx/gdm_qos.c
index d48994b..b795353 100644
--- a/drivers/staging/gdm72xx/gdm_qos.c
+++ b/drivers/staging/gdm72xx/gdm_qos.c
@@ -365,20 +365,24 @@ void gdm_recv_qos_hci_packet(void *nic_ptr, u8 *buf, int 
size)
                spin_unlock_irqrestore(&qcb->qos_lock, flags);
                send_qos_list(nic, &send_list);
                return;
-       } else if (subCmdEvt == QOS_ADD) {
-               pos = 6;
-
-               SFID = ((buf[pos++]<<24)&0xff000000);
-               SFID += ((buf[pos++]<<16)&0xff0000);
-               SFID += ((buf[pos++]<<8)&0xff00);
-               SFID += (buf[pos++]);
-
-               index = get_csr(qcb, SFID, 1);
-               if (index == -1) {
-                       netdev_err(nic->netdev, "QoS ERROR: csr Update 
Error\n");
-                       return;
-               }
+       }
 
+       /* subCmdEvt == QOS_ADD || subCmdEvt == QOS_CHANG_DEL */
+       pos = 6;
+       SFID = ((buf[pos++]<<24)&0xff000000);
+       SFID += ((buf[pos++]<<16)&0xff0000);
+       SFID += ((buf[pos++]<<8)&0xff00);
+       SFID += (buf[pos++]);
+
+       index = get_csr(qcb, SFID, 1);
+       if (index == -1) {
+               netdev_err(nic->netdev,
+                          "QoS ERROR: csr Update Error / Wrong index (%d) \n",
+                          index);
+               return;
+       }
+
+       if (subCmdEvt == QOS_ADD) {
                netdev_dbg(nic->netdev, "QOS_ADD SFID = 0x%x, index=%d\n",
                           SFID, index);
 
@@ -420,18 +424,6 @@ void gdm_recv_qos_hci_packet(void *nic_ptr, u8 *buf, int 
size)
                qcb->qos_limit_size = 254/qcb->qos_list_cnt;
                spin_unlock_irqrestore(&qcb->qos_lock, flags);
        } else if (subCmdEvt == QOS_CHANGE_DEL) {
-               pos = 6;
-               SFID = ((buf[pos++]<<24)&0xff000000);
-               SFID += ((buf[pos++]<<16)&0xff0000);
-               SFID += ((buf[pos++]<<8)&0xff00);
-               SFID += (buf[pos++]);
-               index = get_csr(qcb, SFID, 1);
-               if (index == -1) {
-                       netdev_err(nic->netdev, "QoS ERROR: Wrong index(%d)\n",
-                                  index);
-                       return;
-               }
-
                netdev_dbg(nic->netdev, "QOS_CHANGE_DEL SFID = 0x%x, 
index=%d\n",
                           SFID, index);
 
-- 
1.7.8.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/