staging: xgifb: Remove unneeded parentheses
authorJanani Ravichandran <janani.rvchndrn@gmail.com>
Thu, 11 Feb 2016 22:30:16 +0000 (17:30 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 12 Feb 2016 03:50:11 +0000 (19:50 -0800)
Remove unneeded parentheses around the right hand side of assignments as
they are not needed. Semantic patch:

@@
expression a, b, c;
@@

(
  a = (b == c)
|
  a =
- (
  b
- )
)

Signed-off-by: Janani Ravichandran <janani.rvchndrn@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/xgifb/XGI_main_26.c
drivers/staging/xgifb/vb_setmode.c

index f11cea17f2b8adb34e1a7df1389bcfd0a4557fa6..cc55a17f9b7f765015ebe84c42558b8f7d375271 100644 (file)
@@ -655,26 +655,26 @@ static void XGIfb_pre_setmode(struct xgifb_video_info *xgifb_info)
 
        switch (xgifb_info->display2) {
        case XGIFB_DISP_CRT:
-               cr30 = (SIS_VB_OUTPUT_CRT2 | SIS_SIMULTANEOUS_VIEW_ENABLE);
+               cr30 = SIS_VB_OUTPUT_CRT2 | SIS_SIMULTANEOUS_VIEW_ENABLE;
                cr31 |= SIS_DRIVER_MODE;
                break;
        case XGIFB_DISP_LCD:
-               cr30 = (SIS_VB_OUTPUT_LCD | SIS_SIMULTANEOUS_VIEW_ENABLE);
+               cr30 = SIS_VB_OUTPUT_LCD | SIS_SIMULTANEOUS_VIEW_ENABLE;
                cr31 |= SIS_DRIVER_MODE;
                break;
        case XGIFB_DISP_TV:
                if (xgifb_info->TV_type == TVMODE_HIVISION)
-                       cr30 = (SIS_VB_OUTPUT_HIVISION
-                                       | SIS_SIMULTANEOUS_VIEW_ENABLE);
+                       cr30 = SIS_VB_OUTPUT_HIVISION
+                                       | SIS_SIMULTANEOUS_VIEW_ENABLE;
                else if (xgifb_info->TV_plug == TVPLUG_SVIDEO)
-                       cr30 = (SIS_VB_OUTPUT_SVIDEO
-                                       | SIS_SIMULTANEOUS_VIEW_ENABLE);
+                       cr30 = SIS_VB_OUTPUT_SVIDEO
+                                       | SIS_SIMULTANEOUS_VIEW_ENABLE;
                else if (xgifb_info->TV_plug == TVPLUG_COMPOSITE)
-                       cr30 = (SIS_VB_OUTPUT_COMPOSITE
-                                       | SIS_SIMULTANEOUS_VIEW_ENABLE);
+                       cr30 = SIS_VB_OUTPUT_COMPOSITE
+                                       | SIS_SIMULTANEOUS_VIEW_ENABLE;
                else if (xgifb_info->TV_plug == TVPLUG_SCART)
-                       cr30 = (SIS_VB_OUTPUT_SCART
-                                       | SIS_SIMULTANEOUS_VIEW_ENABLE);
+                       cr30 = SIS_VB_OUTPUT_SCART
+                                       | SIS_SIMULTANEOUS_VIEW_ENABLE;
                cr31 |= SIS_DRIVER_MODE;
 
                if (XGIfb_tvmode == 1 || xgifb_info->TV_type == TVMODE_PAL)
index de96da181bd201afb527aad15a02a3ff642504e1..2078d57e97a41732aa941fb62b9ed6b98b24671f 100644 (file)
@@ -1899,8 +1899,8 @@ static void XGI_GetVBInfo(unsigned short ModeIdIndex,
        push <<= 8;
        tempax = temp << 8;
        tempbx = tempbx | tempax;
-       temp = (SetCRT2ToDualEdge | SetCRT2ToYPbPr525750 | XGI_SetCRT2ToLCDA
-               | SetInSlaveMode | DisableCRT2Display);
+       temp = SetCRT2ToDualEdge | SetCRT2ToYPbPr525750 | XGI_SetCRT2ToLCDA
+               | SetInSlaveMode | DisableCRT2Display;
        temp = 0xFFFF ^ temp;
        tempbx &= temp;
 
@@ -2881,7 +2881,7 @@ static void XGI_SetGroup1(unsigned short ModeIdIndex,
        xgifb_reg_set(pVBInfo->Part1Port, 0x0C, temp);
        temp = tempcx & 0x00FF;
        xgifb_reg_set(pVBInfo->Part1Port, 0x0D, temp);
-       tempcx = (pVBInfo->VGAVT - 1);
+       tempcx = pVBInfo->VGAVT - 1;
        temp = tempcx & 0x00FF;
 
        xgifb_reg_set(pVBInfo->Part1Port, 0x0E, temp);
@@ -2919,7 +2919,7 @@ static void XGI_SetGroup1(unsigned short ModeIdIndex,
        temp = tempbx & 0x00FF;
        xgifb_reg_set(pVBInfo->Part1Port, 0x10, temp);
        temp = ((tempbx & 0xFF00) >> 8) << 4;
-       temp = ((tempcx & 0x000F) | (temp));
+       temp = (tempcx & 0x000F) | (temp);
        xgifb_reg_set(pVBInfo->Part1Port, 0x11, temp);
        tempax = 0;
 
@@ -4074,7 +4074,7 @@ static void XGI_SetGroup4(unsigned short ModeIdIndex,
        tempcx |= 0x04000;
 
        if (tempeax <= tempebx) {
-               tempcx = (tempcx & (~0x4000));
+               tempcx = tempcx & (~0x4000);
                tempeax = pVBInfo->VGAVDE;
        } else {
                tempeax -= tempebx;
@@ -4124,7 +4124,7 @@ static void XGI_SetGroup4(unsigned short ModeIdIndex,
                temp = (tempax & 0xFF00) >> 8;
                temp = (temp & 0x0003) << 4;
                xgifb_reg_set(pVBInfo->Part4Port, 0x1E, temp);
-               temp = (tempax & 0x00FF);
+               temp = tempax & 0x00FF;
                xgifb_reg_set(pVBInfo->Part4Port, 0x1D, temp);
 
                if (pVBInfo->VBInfo & (SetCRT2ToTV | SetCRT2ToHiVision)) {
@@ -4926,7 +4926,7 @@ static void XGI_SetCRT2ModeRegs(struct vb_device_info *pVBInfo)
                        tempcl -= ModeVGA;
                        if (tempcl >= 0) {
                                /* BT Color */
-                               tempah = (0x008 >> tempcl);
+                               tempah = 0x008 >> tempcl;
                                if (tempah == 0)
                                        tempah = 1;
                                tempah |= 0x040;