staging/xgifb: Remove redundant if statement
authorPeter Huewe <peterhuewe@gmx.de>
Sun, 3 Feb 2013 03:08:45 +0000 (04:08 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 4 Feb 2013 01:26:14 +0000 (19:26 -0600)
The code checks twice for if (pVBInfo->VBInfo & SetCRT2ToTV) without any
changes in between -> we can remove the second check.
And while at it we can also save the temp variable and use tempbx
directly.

Signed-off-by: Peter Huewe <peterhuewe@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/xgifb/vb_setmode.c

index 2e7277e86ebd739b77148d6a253303427c12d292..93f5d46b3435290284824f416d8f592ba3e95dd8 100644 (file)
@@ -2197,31 +2197,25 @@ static void XGI_GetVBInfo(unsigned short ModeNo, unsigned short ModeIdIndex,
 static void XGI_GetTVInfo(unsigned short ModeNo, unsigned short ModeIdIndex,
                struct vb_device_info *pVBInfo)
 {
-       unsigned short temp, tempbx = 0, resinfo = 0, modeflag, index1;
-
-       tempbx = 0;
-       resinfo = 0;
+       unsigned short tempbx = 0, resinfo = 0, modeflag, index1;
 
        if (pVBInfo->VBInfo & SetCRT2ToTV) {
                modeflag = XGI330_EModeIDTable[ModeIdIndex].Ext_ModeFlag;
                resinfo = XGI330_EModeIDTable[ModeIdIndex].Ext_RESINFO;
 
-               if (pVBInfo->VBInfo & SetCRT2ToTV) {
-                       temp = xgifb_reg_get(pVBInfo->P3d4, 0x35);
-                       tempbx = temp;
-                       if (tempbx & TVSetPAL) {
-                               tempbx &= (SetCHTVOverScan |
-                                          TVSetPALM |
-                                          TVSetPALN |
-                                          TVSetPAL);
-                               if (tempbx & TVSetPALM)
-                                       /* set to NTSC if PAL-M */
-                                       tempbx &= ~TVSetPAL;
-                       } else
-                               tempbx &= (SetCHTVOverScan |
-                                          TVSetNTSCJ |
-                                          TVSetPAL);
-               }
+               tempbx = xgifb_reg_get(pVBInfo->P3d4, 0x35);
+               if (tempbx & TVSetPAL) {
+                       tempbx &= (SetCHTVOverScan |
+                                  TVSetPALM |
+                                  TVSetPALN |
+                                  TVSetPAL);
+                       if (tempbx & TVSetPALM)
+                               /* set to NTSC if PAL-M */
+                               tempbx &= ~TVSetPAL;
+               } else
+                       tempbx &= (SetCHTVOverScan |
+                                  TVSetNTSCJ |
+                                  TVSetPAL);
 
                if (pVBInfo->IF_DEF_LVDS == 0) {
                        if (pVBInfo->VBInfo & SetCRT2ToSCART)