Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dcb9360e authored by Elise Lennion's avatar Elise Lennion Committed by Greg Kroah-Hartman
Browse files

staging: xgifb: Compress return logic into one line. 



Simplify return logic to avoid unnecessary variable assignments.

Coccinelle was used with this semantic patch:
@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;

Signed-off-by: default avatarElise Lennion <elise.lennion@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent ce439c42

drivers/staging/xgifb/vb_setmode.c

+3 −6
Original line number Diff line number Diff line
@@ -1682,13 +1682,11 @@ static unsigned char XGI_GetVCLKPtr(unsigned short RefreshRateTableIndex, 
			}



			if (pVBInfo->TVInfo & TVSetYPbPr750p) {

				tempal = XGI_YPbPr750pVCLK;

				return tempal;

				return XGI_YPbPr750pVCLK;

			}



			if (pVBInfo->TVInfo & TVSetYPbPr525p) {

				tempal = YPbPr525pVCLK;

				return tempal;

				return YPbPr525pVCLK;

			}



			tempal = NTSC1024VCLK;
@@ -1705,8 +1703,7 @@ static unsigned char XGI_GetVCLKPtr(unsigned short RefreshRateTableIndex, 
	} /* {End of VB} */



	inb((pVBInfo->P3ca + 0x02));

	tempal = XGI330_RefIndex[RefreshRateTableIndex].Ext_CRTVCLK;

	return tempal;

	return XGI330_RefIndex[RefreshRateTableIndex].Ext_CRTVCLK;

}



static void XGI_GetVCLKLen(unsigned char tempal, unsigned char *di_0,