This patch fixes two unnecessary else conditions that were found by 
checkpatch.pl.

Signed-off-by: Serguey Parkhomovsky <sergueyparkhomov...@gmail.com>
---
 drivers/staging/rtl8712/rtl8712_efuse.c | 41 +++++++++++++++++----------------
 1 file changed, 21 insertions(+), 20 deletions(-)

diff --git a/drivers/staging/rtl8712/rtl8712_efuse.c 
b/drivers/staging/rtl8712/rtl8712_efuse.c
index c9eeb42..0e155d9 100644
--- a/drivers/staging/rtl8712/rtl8712_efuse.c
+++ b/drivers/staging/rtl8712/rtl8712_efuse.c
@@ -414,19 +414,19 @@ u8 r8712_efuse_pg_packet_write(struct _adapter *padapter, 
const u8 offset,
                                        bResult = false;
                        }
                        break;
-               } else { /* write header fail */
-                       bResult = false;
-                       if (0xFF == efuse_data)
-                               return bResult; /* nothing damaged. */
-                       /* call rescue procedure */
-                       if (fix_header(padapter, efuse_data, efuse_addr) ==
-                           false)
-                               return false; /* rescue fail */
-
-                       if (++repeat_times > _REPEAT_THRESHOLD_) /* fail */
-                               break;
-                       /* otherwise, take another risk... */
                }
+               /* write header fail */
+               bResult = false;
+               if (0xFF == efuse_data)
+                       return bResult; /* nothing damaged. */
+               /* call rescue procedure */
+               if (fix_header(padapter, efuse_data, efuse_addr) ==
+                   false)
+                       return false; /* rescue fail */
+
+               if (++repeat_times > _REPEAT_THRESHOLD_) /* fail */
+                       break;
+               /* otherwise, take another risk... */
        }
        return bResult;
 }
@@ -541,15 +541,16 @@ u8 r8712_efuse_map_write(struct _adapter *padapter, u16 
addr, u16 cnts,
                                }
                                idx++;
                                break;
-                       } else {
-                               if ((data[idx] != pktdata[i]) || (data[idx+1] !=
-                                    pktdata[i+1])) {
-                                       word_en &= ~BIT(i >> 1);
-                                       newdata[j++] = data[idx];
-                                       newdata[j++] = data[idx + 1];
-                               }
-                               idx += 2;
                        }
+
+                       if ((data[idx] != pktdata[i]) || (data[idx+1] !=
+                            pktdata[i+1])) {
+                               word_en &= ~BIT(i >> 1);
+                               newdata[j++] = data[idx];
+                               newdata[j++] = data[idx + 1];
+                       }
+                       idx += 2;
+
                        if (idx == cnts)
                                break;
                }
-- 
1.9.3

--
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/

Reply via email to