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 <peterhu...@gmx.de>
---
 drivers/staging/xgifb/vb_setmode.c |   34 ++++++++++++++--------------------
 1 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/drivers/staging/xgifb/vb_setmode.c 
b/drivers/staging/xgifb/vb_setmode.c
index 2e7277e..93f5d46 100644
--- a/drivers/staging/xgifb/vb_setmode.c
+++ b/drivers/staging/xgifb/vb_setmode.c
@@ -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)
-- 
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/

Reply via email to