]> git.karo-electronics.de Git - linux-beck.git/commitdiff
staging: xgifb: remove unnecessary parentheses
authorGargi Sharma <gs051095@gmail.com>
Thu, 15 Sep 2016 19:07:10 +0000 (00:37 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 16 Sep 2016 08:27:52 +0000 (10:27 +0200)
! has higher priority than && or ||, so remove parentheses around
! expressions to make the code more readable.

Signed-off-by: Gargi Sharma <gs051095@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/xgifb/vb_setmode.c

index 316209ff983559ad0100434004ae13684383fa46..af3f4d0fbd82d58aaa2655b2633d12e6a0e16726 100644 (file)
@@ -214,7 +214,7 @@ static unsigned char XGI_AjustCRT2Rate(unsigned short ModeIdIndex,
                if (!(pVBInfo->VBInfo & TVSetPAL) &&
                    (modeflag & NoSupportSimuTV) &&
                    (pVBInfo->VBInfo & SetInSlaveMode) &&
-                   (!(pVBInfo->VBInfo & SetNotSimuMode)))
+                   !(pVBInfo->VBInfo & SetNotSimuMode))
                        return 0;
        }
 
@@ -1655,7 +1655,7 @@ static unsigned char XGI_GetVCLKPtr(unsigned short RefreshRateTableIndex,
        modeflag = XGI330_EModeIDTable[ModeIdIndex].Ext_ModeFlag;
 
        if ((pVBInfo->SetFlag & ProgrammingCRT2) &&
-           (!(pVBInfo->LCDInfo & EnableScalingLCD))) { /* {LCDA/LCDB} */
+           !(pVBInfo->LCDInfo & EnableScalingLCD)) { /* {LCDA/LCDB} */
                index = XGI_GetLCDCapPtr(pVBInfo);
                tempal = pVBInfo->LCDCapList[index].LCD_VCLK;
 
@@ -1716,7 +1716,7 @@ static void XGI_GetVCLKLen(unsigned char tempal, unsigned char *di_0,
 {
        if (pVBInfo->VBType & (VB_SIS301 | VB_SIS301B | VB_SIS302B
                        | VB_SIS301LV | VB_SIS302LV | VB_XGI301C)) {
-               if ((!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA)) &&
+               if (!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA) &&
                    (pVBInfo->SetFlag & ProgrammingCRT2)) {
                        *di_0 = XGI_VBVCLKData[tempal].Part4_A;
                        *di_1 = XGI_VBVCLKData[tempal].Part4_B;
@@ -1741,8 +1741,8 @@ static void XGI_SetCRT2ECLK(unsigned short ModeIdIndex,
        for (i = 0; i < 4; i++) {
                xgifb_reg_and_or(pVBInfo->P3d4, 0x31, ~0x30,
                                (unsigned short) (0x10 * i));
-               if ((!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA)) &&
-                               (!(pVBInfo->VBInfo & SetInSlaveMode))) {
+               if (!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA) &&
+                               !(pVBInfo->VBInfo & SetInSlaveMode)) {
                        xgifb_reg_set(pVBInfo->P3c4, 0x2e, di_0);
                        xgifb_reg_set(pVBInfo->P3c4, 0x2f, di_1);
                } else {
@@ -1986,7 +1986,7 @@ static void XGI_GetVBInfo(unsigned short ModeIdIndex,
        }
 
        if (!(tempbx & DisableCRT2Display)) {
-               if ((!(tempbx & DriverMode)) || (!(modeflag & CRT2Mode))) {
+               if (!(tempbx & DriverMode) || !(modeflag & CRT2Mode)) {
                        if (!(tempbx & XGI_SetCRT2ToLCDA))
                                tempbx |= (SetInSlaveMode | SetSimuScanMode);
                }
@@ -2132,7 +2132,7 @@ static unsigned char XGI_GetLCDInfo(unsigned short ModeIdIndex,
 
        if ((pVBInfo->LCDResInfo == Panel_1400x1050) &&
            (pVBInfo->VBInfo & SetCRT2ToLCD) && (resinfo == 9) &&
-           (!(tempbx & EnableScalingLCD)))
+           !(tempbx & EnableScalingLCD))
                /*
                 * set to center in 1280x1024 LCDB
                 * for Panel_1400x1050
@@ -3922,7 +3922,7 @@ static void XGI_SetTap4Regs(struct vb_device_info *pVBInfo)
                xgifb_reg_set(pVBInfo->Part2Port, i, Tap4TimingPtr->Reg[j]);
 
        if ((pVBInfo->VBInfo & SetCRT2ToTV) &&
-           (!(pVBInfo->VBInfo & SetCRT2ToHiVision))) {
+           !(pVBInfo->VBInfo & SetCRT2ToHiVision)) {
                /* Set Vertical Scaling */
                Tap4TimingPtr = XGI_GetTap4Ptr(1, pVBInfo);
                for (i = 0xC0, j = 0; i < 0xFF; i++, j++)
@@ -3932,7 +3932,7 @@ static void XGI_SetTap4Regs(struct vb_device_info *pVBInfo)
        }
 
        if ((pVBInfo->VBInfo & SetCRT2ToTV) &&
-           (!(pVBInfo->VBInfo & SetCRT2ToHiVision)))
+           !(pVBInfo->VBInfo & SetCRT2ToHiVision))
                /* Enable V.Scaling */
                xgifb_reg_and_or(pVBInfo->Part2Port, 0x4E, ~0x14, 0x04);
        else
@@ -4142,8 +4142,8 @@ static void XGI_SetGroup4(unsigned short ModeIdIndex,
                                        | TVSetHiVision))) {
                                temp |= 0x0001;
                                if ((pVBInfo->VBInfo & SetInSlaveMode) &&
-                                               (!(pVBInfo->TVInfo
-                                                               & TVSimuMode)))
+                                               !(pVBInfo->TVInfo
+                                                               & TVSimuMode))
                                        temp &= (~0x0001);
                        }
                }
@@ -4476,7 +4476,7 @@ static void XGI_DisableBridge(struct xgifb_video_info *xgifb_info,
 
                if ((pVBInfo->VBInfo &
                        (DisableCRT2Display | SetSimuScanMode)) ||
-                   ((!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA)) &&
+                   (!(pVBInfo->VBInfo & XGI_SetCRT2ToLCDA) &&
                    (pVBInfo->VBInfo &
                        (SetCRT2ToRAMDAC | SetCRT2ToLCD | SetCRT2ToTV))))
                        xgifb_reg_or(pVBInfo->Part1Port, 0x00, 0x80);
@@ -4587,7 +4587,7 @@ static void XGI_GetTVPtrIndex2(unsigned short *tempbx, unsigned char *tempcl,
 
        if (pVBInfo->VBType & (VB_SIS301B | VB_SIS302B | VB_SIS301LV
                        | VB_SIS302LV | VB_XGI301C)) {
-               if ((!(pVBInfo->VBInfo & SetInSlaveMode)) || (pVBInfo->TVInfo
+               if (!(pVBInfo->VBInfo & SetInSlaveMode) || (pVBInfo->TVInfo
                                & TVSimuMode)) {
                        *tempbx += 8;
                        *tempcl += 1;
@@ -4988,8 +4988,8 @@ reg_and_or:
        if (pVBInfo->VBInfo & (SetCRT2ToRAMDAC | SetCRT2ToTV | SetCRT2ToLCD
                        | XGI_SetCRT2ToLCDA)) {
                tempah &= (~0x08);
-               if ((pVBInfo->ModeType == ModeVGA) && (!(pVBInfo->VBInfo
-                               & SetInSlaveMode))) {
+               if ((pVBInfo->ModeType == ModeVGA) && !(pVBInfo->VBInfo
+                               & SetInSlaveMode)) {
                        tempah |= 0x010;
                }
                tempah |= 0x080;
@@ -5486,7 +5486,7 @@ unsigned char XGISetModeNew(struct xgifb_video_info *xgifb_info,
                XGI_DisableBridge(xgifb_info, HwDeviceExtension, pVBInfo);
 
                if (pVBInfo->VBInfo & (SetSimuScanMode | XGI_SetCRT2ToLCDA) ||
-                       (!(pVBInfo->VBInfo & SwitchCRT2))) {
+                       !(pVBInfo->VBInfo & SwitchCRT2)) {
                        XGI_SetCRT1Group(xgifb_info, HwDeviceExtension, ModeNo,
                                        ModeIdIndex, pVBInfo);