diff mbox series

video: fbdev: sis_main: Clean up some inconsistent indenting

Message ID 20220819110414.107565-1-jiapeng.chong@linux.alibaba.com (mailing list archive)
State Accepted, archived
Headers show
Series video: fbdev: sis_main: Clean up some inconsistent indenting | expand

Commit Message

Jiapeng Chong Aug. 19, 2022, 11:04 a.m. UTC
No functional modification involved.

drivers/video/fbdev/sis/sis_main.c:6165 sisfb_probe() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:4266 sisfb_post_300_rwtest() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:2388 SISDoSense() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:2531 SiS_Sense30x() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:2382 SISDoSense() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:2250 sisfb_sense_crt1() warn: inconsistent indenting.
drivers/video/fbdev/sis/sis_main.c:672 sisfb_validate_mode() warn: inconsistent indenting.

Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1934
Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 drivers/video/fbdev/sis/sis_main.c | 274 +++++++++++++++--------------
 1 file changed, 141 insertions(+), 133 deletions(-)

Comments

Helge Deller Aug. 22, 2022, 8:04 p.m. UTC | #1
On 8/19/22 13:04, Jiapeng Chong wrote:
> No functional modification involved.
>
> drivers/video/fbdev/sis/sis_main.c:6165 sisfb_probe() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:4266 sisfb_post_300_rwtest() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:2388 SISDoSense() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:2531 SiS_Sense30x() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:2382 SISDoSense() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:2250 sisfb_sense_crt1() warn: inconsistent indenting.
> drivers/video/fbdev/sis/sis_main.c:672 sisfb_validate_mode() warn: inconsistent indenting.
>
> Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1934
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>

applied.

Thanks!
Helge


> ---
>  drivers/video/fbdev/sis/sis_main.c | 274 +++++++++++++++--------------
>  1 file changed, 141 insertions(+), 133 deletions(-)
>
> diff --git a/drivers/video/fbdev/sis/sis_main.c b/drivers/video/fbdev/sis/sis_main.c
> index 7114c5c17c91..ac4680a74d78 100644
> --- a/drivers/video/fbdev/sis/sis_main.c
> +++ b/drivers/video/fbdev/sis/sis_main.c
> @@ -650,37 +650,37 @@ sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
>  	u16 xres=0, yres, myres;
>
>  #ifdef CONFIG_FB_SIS_300
> -	if(ivideo->sisvga_engine == SIS_300_VGA) {
> -		if(!(sisbios_mode[myindex].chipset & MD_SIS300))
> +	if (ivideo->sisvga_engine == SIS_300_VGA) {
> +		if (!(sisbios_mode[myindex].chipset & MD_SIS300))
>  			return -1 ;
>  	}
>  #endif
>  #ifdef CONFIG_FB_SIS_315
> -	if(ivideo->sisvga_engine == SIS_315_VGA) {
> -		if(!(sisbios_mode[myindex].chipset & MD_SIS315))
> +	if (ivideo->sisvga_engine == SIS_315_VGA) {
> +		if (!(sisbios_mode[myindex].chipset & MD_SIS315))
>  			return -1;
>  	}
>  #endif
>
>  	myres = sisbios_mode[myindex].yres;
>
> -	switch(vbflags & VB_DISPTYPE_DISP2) {
> +	switch (vbflags & VB_DISPTYPE_DISP2) {
>
>  	case CRT2_LCD:
>  		xres = ivideo->lcdxres; yres = ivideo->lcdyres;
>
> -		if((ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL848) &&
> -		   (ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL856)) {
> -			if(sisbios_mode[myindex].xres > xres)
> +		if ((ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL848) &&
> +		    (ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL856)) {
> +			if (sisbios_mode[myindex].xres > xres)
>  				return -1;
> -			if(myres > yres)
> +			if (myres > yres)
>  				return -1;
>  		}
>
> -		if(ivideo->sisfb_fstn) {
> -			if(sisbios_mode[myindex].xres == 320) {
> -				if(myres == 240) {
> -					switch(sisbios_mode[myindex].mode_no[1]) {
> +		if (ivideo->sisfb_fstn) {
> +			if (sisbios_mode[myindex].xres == 320) {
> +				if (myres == 240) {
> +					switch (sisbios_mode[myindex].mode_no[1]) {
>  						case 0x50: myindex = MODE_FSTN_8;  break;
>  						case 0x56: myindex = MODE_FSTN_16; break;
>  						case 0x53: return -1;
> @@ -689,7 +689,7 @@ sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
>  			}
>  		}
>
> -		if(SiS_GetModeID_LCD(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
> +		if (SiS_GetModeID_LCD(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
>  			 	sisbios_mode[myindex].yres, 0, ivideo->sisfb_fstn,
>  			 	ivideo->SiS_Pr.SiS_CustomT, xres, yres, ivideo->vbflags2) < 0x14) {
>  			return -1;
> @@ -697,14 +697,14 @@ sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
>  		break;
>
>  	case CRT2_TV:
> -		if(SiS_GetModeID_TV(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
> +		if (SiS_GetModeID_TV(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
>  				sisbios_mode[myindex].yres, 0, ivideo->vbflags2) < 0x14) {
>  			return -1;
>  		}
>  		break;
>
>  	case CRT2_VGA:
> -		if(SiS_GetModeID_VGA2(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
> +		if (SiS_GetModeID_VGA2(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
>  				sisbios_mode[myindex].yres, 0, ivideo->vbflags2) < 0x14) {
>  			return -1;
>  		}
> @@ -2205,82 +2205,88 @@ static bool sisfb_test_DDC1(struct sis_video_info *ivideo)
>
>  static void sisfb_sense_crt1(struct sis_video_info *ivideo)
>  {
> -    bool mustwait = false;
> -    u8  sr1F, cr17;
> +	bool mustwait = false;
> +	u8  sr1F, cr17;
>  #ifdef CONFIG_FB_SIS_315
> -    u8  cr63=0;
> +	u8  cr63 = 0;
>  #endif
> -    u16 temp = 0xffff;
> -    int i;
> +	u16 temp = 0xffff;
> +	int i;
> +
> +	sr1F = SiS_GetReg(SISSR, 0x1F);
> +	SiS_SetRegOR(SISSR, 0x1F, 0x04);
> +	SiS_SetRegAND(SISSR, 0x1F, 0x3F);
>
> -    sr1F = SiS_GetReg(SISSR, 0x1F);
> -    SiS_SetRegOR(SISSR, 0x1F, 0x04);
> -    SiS_SetRegAND(SISSR, 0x1F, 0x3F);
> -    if(sr1F & 0xc0) mustwait = true;
> +	if (sr1F & 0xc0)
> +		mustwait = true;
>
>  #ifdef CONFIG_FB_SIS_315
> -    if(ivideo->sisvga_engine == SIS_315_VGA) {
> -       cr63 = SiS_GetReg(SISCR, ivideo->SiS_Pr.SiS_MyCR63);
> -       cr63 &= 0x40;
> -       SiS_SetRegAND(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF);
> -    }
> +	if (ivideo->sisvga_engine == SIS_315_VGA) {
> +		cr63 = SiS_GetReg(SISCR, ivideo->SiS_Pr.SiS_MyCR63);
> +		cr63 &= 0x40;
> +		SiS_SetRegAND(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF);
> +	}
>  #endif
>
> -    cr17 = SiS_GetReg(SISCR, 0x17);
> -    cr17 &= 0x80;
> -    if(!cr17) {
> -       SiS_SetRegOR(SISCR, 0x17, 0x80);
> -       mustwait = true;
> -       SiS_SetReg(SISSR, 0x00, 0x01);
> -       SiS_SetReg(SISSR, 0x00, 0x03);
> -    }
> +	cr17 = SiS_GetReg(SISCR, 0x17);
> +	cr17 &= 0x80;
>
> -    if(mustwait) {
> -       for(i=0; i < 10; i++) sisfbwaitretracecrt1(ivideo);
> -    }
> +	if (!cr17) {
> +		SiS_SetRegOR(SISCR, 0x17, 0x80);
> +		mustwait = true;
> +		SiS_SetReg(SISSR, 0x00, 0x01);
> +		SiS_SetReg(SISSR, 0x00, 0x03);
> +	}
>
> +	if (mustwait) {
> +		for (i = 0; i < 10; i++)
> +			sisfbwaitretracecrt1(ivideo);
> +	}
>  #ifdef CONFIG_FB_SIS_315
> -    if(ivideo->chip >= SIS_330) {
> -       SiS_SetRegAND(SISCR, 0x32, ~0x20);
> -       if(ivideo->chip >= SIS_340) {
> -	   SiS_SetReg(SISCR, 0x57, 0x4a);
> -       } else {
> -	   SiS_SetReg(SISCR, 0x57, 0x5f);
> -       }
> -	SiS_SetRegOR(SISCR, 0x53, 0x02);
> -	while ((SiS_GetRegByte(SISINPSTAT)) & 0x01)    break;
> -	while (!((SiS_GetRegByte(SISINPSTAT)) & 0x01)) break;
> -	if ((SiS_GetRegByte(SISMISCW)) & 0x10) temp = 1;
> -	SiS_SetRegAND(SISCR, 0x53, 0xfd);
> -	SiS_SetRegAND(SISCR, 0x57, 0x00);
> -    }
> +	if (ivideo->chip >= SIS_330) {
> +		SiS_SetRegAND(SISCR, 0x32, ~0x20);
> +		if (ivideo->chip >= SIS_340)
> +			SiS_SetReg(SISCR, 0x57, 0x4a);
> +		else
> +			SiS_SetReg(SISCR, 0x57, 0x5f);
> +
> +		SiS_SetRegOR(SISCR, 0x53, 0x02);
> +		while ((SiS_GetRegByte(SISINPSTAT)) & 0x01)
> +			break;
> +		while (!((SiS_GetRegByte(SISINPSTAT)) & 0x01))
> +			break;
> +		if ((SiS_GetRegByte(SISMISCW)) & 0x10)
> +			temp = 1;
> +
> +		SiS_SetRegAND(SISCR, 0x53, 0xfd);
> +		SiS_SetRegAND(SISCR, 0x57, 0x00);
> +	}
>  #endif
>
> -    if(temp == 0xffff) {
> -       i = 3;
> -       do {
> -	  temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
> -		ivideo->sisvga_engine, 0, 0, NULL, ivideo->vbflags2);
> -       } while(((temp == 0) || (temp == 0xffff)) && i--);
> +	if (temp == 0xffff) {
> +		i = 3;
>
> -       if((temp == 0) || (temp == 0xffff)) {
> -          if(sisfb_test_DDC1(ivideo)) temp = 1;
> -       }
> -    }
> +		do {
> +			temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
> +			ivideo->sisvga_engine, 0, 0, NULL, ivideo->vbflags2);
> +		} while (((temp == 0) || (temp == 0xffff)) && i--);
>
> -    if((temp) && (temp != 0xffff)) {
> -       SiS_SetRegOR(SISCR, 0x32, 0x20);
> -    }
> +		if ((temp == 0) || (temp == 0xffff)) {
> +			if (sisfb_test_DDC1(ivideo))
> +				temp = 1;
> +		}
> +	}
> +
> +	if ((temp) && (temp != 0xffff))
> +		SiS_SetRegOR(SISCR, 0x32, 0x20);
>
>  #ifdef CONFIG_FB_SIS_315
> -    if(ivideo->sisvga_engine == SIS_315_VGA) {
> -	SiS_SetRegANDOR(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF, cr63);
> -    }
> +	if (ivideo->sisvga_engine == SIS_315_VGA)
> +		SiS_SetRegANDOR(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF, cr63);
>  #endif
>
> -    SiS_SetRegANDOR(SISCR, 0x17, 0x7F, cr17);
> -
> -    SiS_SetReg(SISSR, 0x1F, sr1F);
> +	SiS_SetRegANDOR(SISCR, 0x17, 0x7F, cr17);
> +	SiS_SetReg(SISSR, 0x1F, sr1F);
>  }
>
>  /* Determine and detect attached devices on SiS30x */
> @@ -2294,25 +2300,25 @@ static void SiS_SenseLCD(struct sis_video_info *ivideo)
>  	ivideo->SiS_Pr.PanelSelfDetected = false;
>
>  	/* LCD detection only for TMDS bridges */
> -	if(!(ivideo->vbflags2 & VB2_SISTMDSBRIDGE))
> +	if (!(ivideo->vbflags2 & VB2_SISTMDSBRIDGE))
>  		return;
> -	if(ivideo->vbflags2 & VB2_30xBDH)
> +	if (ivideo->vbflags2 & VB2_30xBDH)
>  		return;
>
>  	/* If LCD already set up by BIOS, skip it */
>  	reg = SiS_GetReg(SISCR, 0x32);
> -	if(reg & 0x08)
> +	if (reg & 0x08)
>  		return;
>
>  	realcrtno = 1;
> -	if(ivideo->SiS_Pr.DDCPortMixup)
> +	if (ivideo->SiS_Pr.DDCPortMixup)
>  		realcrtno = 0;
>
>  	/* Check DDC capabilities */
>  	temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags, ivideo->sisvga_engine,
>  				realcrtno, 0, &buffer[0], ivideo->vbflags2);
>
> -	if((!temp) || (temp == 0xffff) || (!(temp & 0x02)))
> +	if ((!temp) || (temp == 0xffff) || (!(temp & 0x02)))
>  		return;
>
>  	/* Read DDC data */
> @@ -2321,17 +2327,17 @@ static void SiS_SenseLCD(struct sis_video_info *ivideo)
>  		temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
>  				ivideo->sisvga_engine, realcrtno, 1,
>  				&buffer[0], ivideo->vbflags2);
> -	} while((temp) && i--);
> +	} while ((temp) && i--);
>
> -	if(temp)
> +	if (temp)
>  		return;
>
>  	/* No digital device */
> -	if(!(buffer[0x14] & 0x80))
> +	if (!(buffer[0x14] & 0x80))
>  		return;
>
>  	/* First detailed timing preferred timing? */
> -	if(!(buffer[0x18] & 0x02))
> +	if (!(buffer[0x18] & 0x02))
>  		return;
>
>  	xres = buffer[0x38] | ((buffer[0x3a] & 0xf0) << 4);
> @@ -2339,26 +2345,26 @@ static void SiS_SenseLCD(struct sis_video_info *ivideo)
>
>  	switch(xres) {
>  		case 1024:
> -			if(yres == 768)
> +			if (yres == 768)
>  				paneltype = 0x02;
>  			break;
>  		case 1280:
> -			if(yres == 1024)
> +			if (yres == 1024)
>  				paneltype = 0x03;
>  			break;
>  		case 1600:
> -			if((yres == 1200) && (ivideo->vbflags2 & VB2_30xC))
> +			if ((yres == 1200) && (ivideo->vbflags2 & VB2_30xC))
>  				paneltype = 0x0b;
>  			break;
>  	}
>
> -	if(!paneltype)
> +	if (!paneltype)
>  		return;
>
> -	if(buffer[0x23])
> +	if (buffer[0x23])
>  		cr37 |= 0x10;
>
> -	if((buffer[0x47] & 0x18) == 0x18)
> +	if ((buffer[0x47] & 0x18) == 0x18)
>  		cr37 |= ((((buffer[0x47] & 0x06) ^ 0x06) << 5) | 0x20);
>  	else
>  		cr37 |= 0xc0;
> @@ -2373,31 +2379,34 @@ static void SiS_SenseLCD(struct sis_video_info *ivideo)
>
>  static int SISDoSense(struct sis_video_info *ivideo, u16 type, u16 test)
>  {
> -    int temp, mytest, result, i, j;
> -
> -    for(j = 0; j < 10; j++) {
> -       result = 0;
> -       for(i = 0; i < 3; i++) {
> -          mytest = test;
> -	   SiS_SetReg(SISPART4, 0x11, (type & 0x00ff));
> -          temp = (type >> 8) | (mytest & 0x00ff);
> -	  SiS_SetRegANDOR(SISPART4, 0x10, 0xe0, temp);
> -          SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1500);
> -          mytest >>= 8;
> -          mytest &= 0x7f;
> -	   temp = SiS_GetReg(SISPART4, 0x03);
> -          temp ^= 0x0e;
> -          temp &= mytest;
> -          if(temp == mytest) result++;
> +	int temp, mytest, result, i, j;
> +
> +	for (j = 0; j < 10; j++) {
> +		result = 0;
> +		for (i = 0; i < 3; i++) {
> +			mytest = test;
> +			SiS_SetReg(SISPART4, 0x11, (type & 0x00ff));
> +			temp = (type >> 8) | (mytest & 0x00ff);
> +			SiS_SetRegANDOR(SISPART4, 0x10, 0xe0, temp);
> +			SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1500);
> +			mytest >>= 8;
> +			mytest &= 0x7f;
> +			temp = SiS_GetReg(SISPART4, 0x03);
> +			temp ^= 0x0e;
> +			temp &= mytest;
> +			if (temp == mytest)
> +				result++;
>  #if 1
> -	  SiS_SetReg(SISPART4, 0x11, 0x00);
> -	  SiS_SetRegAND(SISPART4, 0x10, 0xe0);
> -	  SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1000);
> +			SiS_SetReg(SISPART4, 0x11, 0x00);
> +			SiS_SetRegAND(SISPART4, 0x10, 0xe0);
> +			SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1000);
>  #endif
> -       }
> -       if((result == 0) || (result >= 2)) break;
> -    }
> -    return result;
> +		}
> +
> +		if ((result == 0) || (result >= 2))
> +			break;
> +	}
> +	return result;
>  }
>
>  static void SiS_Sense30x(struct sis_video_info *ivideo)
> @@ -4263,18 +4272,17 @@ static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
>  	unsigned int k, RankCapacity, PageCapacity, BankNumHigh, BankNumMid;
>  	unsigned int PhysicalAdrOtherPage, PhysicalAdrHigh, PhysicalAdrHalfPage;
>
> -	 for(k = 0; k < ARRAY_SIZE(SiS_DRAMType); k++) {
> -
> +	for (k = 0; k < ARRAY_SIZE(SiS_DRAMType); k++) {
>  		RankCapacity = buswidth * SiS_DRAMType[k][3];
>
> -		if(RankCapacity != PseudoRankCapacity)
> +		if (RankCapacity != PseudoRankCapacity)
>  			continue;
>
> -		if((SiS_DRAMType[k][2] + SiS_DRAMType[k][0]) > PseudoAdrPinCount)
> +		if ((SiS_DRAMType[k][2] + SiS_DRAMType[k][0]) > PseudoAdrPinCount)
>  			continue;
>
>  		BankNumHigh = RankCapacity * 16 * iteration - 1;
> -		if(iteration == 3) {             /* Rank No */
> +		if (iteration == 3) {             /* Rank No */
>  			BankNumMid  = RankCapacity * 16 - 1;
>  		} else {
>  			BankNumMid  = RankCapacity * 16 * iteration / 2 - 1;
> @@ -4288,18 +4296,22 @@ static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
>  		SiS_SetRegAND(SISSR, 0x15, 0xFB); /* Test */
>  		SiS_SetRegOR(SISSR, 0x15, 0x04);  /* Test */
>  		sr14 = (SiS_DRAMType[k][3] * buswidth) - 1;
> -		if(buswidth == 4)      sr14 |= 0x80;
> -		else if(buswidth == 2) sr14 |= 0x40;
> +
> +		if (buswidth == 4)
> +			sr14 |= 0x80;
> +		else if (buswidth == 2)
> +			sr14 |= 0x40;
> +
>  		SiS_SetReg(SISSR, 0x13, SiS_DRAMType[k][4]);
>  		SiS_SetReg(SISSR, 0x14, sr14);
>
>  		BankNumHigh <<= 16;
>  		BankNumMid <<= 16;
>
> -		if((BankNumHigh + PhysicalAdrHigh      >= mapsize) ||
> -		   (BankNumMid  + PhysicalAdrHigh      >= mapsize) ||
> -		   (BankNumHigh + PhysicalAdrHalfPage  >= mapsize) ||
> -		   (BankNumHigh + PhysicalAdrOtherPage >= mapsize))
> +		if ((BankNumHigh + PhysicalAdrHigh >= mapsize) ||
> +		    (BankNumMid  + PhysicalAdrHigh >= mapsize) ||
> +		    (BankNumHigh + PhysicalAdrHalfPage  >= mapsize) ||
> +		    (BankNumHigh + PhysicalAdrOtherPage >= mapsize))
>  			continue;
>
>  		/* Write data */
> @@ -4313,7 +4325,7 @@ static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
>  				(FBAddr + BankNumHigh + PhysicalAdrOtherPage));
>
>  		/* Read data */
> -		if(readw(FBAddr + BankNumHigh + PhysicalAdrHigh) == PhysicalAdrHigh)
> +		if (readw(FBAddr + BankNumHigh + PhysicalAdrHigh) == PhysicalAdrHigh)
>  			return 1;
>  	}
>
> @@ -6155,24 +6167,20 @@ static int sisfb_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  #endif
>
>  #ifdef CONFIG_FB_SIS_315
> -		if(ivideo->sisvga_engine == SIS_315_VGA) {
> +		if (ivideo->sisvga_engine == SIS_315_VGA) {
>  			int result = 1;
> -		/*	if((ivideo->chip == SIS_315H)   ||
> -			   (ivideo->chip == SIS_315)    ||
> -			   (ivideo->chip == SIS_315PRO) ||
> -			   (ivideo->chip == SIS_330)) {
> -				sisfb_post_sis315330(pdev);
> -			} else */ if(ivideo->chip == XGI_20) {
> +
> +			if (ivideo->chip == XGI_20) {
>  				result = sisfb_post_xgi(pdev);
>  				ivideo->sisfb_can_post = 1;
> -			} else if((ivideo->chip == XGI_40) && ivideo->haveXGIROM) {
> +			} else if ((ivideo->chip == XGI_40) && ivideo->haveXGIROM) {
>  				result = sisfb_post_xgi(pdev);
>  				ivideo->sisfb_can_post = 1;
>  			} else {
>  				printk(KERN_INFO "sisfb: Card is not "
>  					"POSTed and sisfb can't do this either.\n");
>  			}
> -			if(!result) {
> +			if (!result) {
>  				printk(KERN_ERR "sisfb: Failed to POST card\n");
>  				ret = -ENODEV;
>  				goto error_3;
diff mbox series

Patch

diff --git a/drivers/video/fbdev/sis/sis_main.c b/drivers/video/fbdev/sis/sis_main.c
index 7114c5c17c91..ac4680a74d78 100644
--- a/drivers/video/fbdev/sis/sis_main.c
+++ b/drivers/video/fbdev/sis/sis_main.c
@@ -650,37 +650,37 @@  sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
 	u16 xres=0, yres, myres;
 
 #ifdef CONFIG_FB_SIS_300
-	if(ivideo->sisvga_engine == SIS_300_VGA) {
-		if(!(sisbios_mode[myindex].chipset & MD_SIS300))
+	if (ivideo->sisvga_engine == SIS_300_VGA) {
+		if (!(sisbios_mode[myindex].chipset & MD_SIS300))
 			return -1 ;
 	}
 #endif
 #ifdef CONFIG_FB_SIS_315
-	if(ivideo->sisvga_engine == SIS_315_VGA) {
-		if(!(sisbios_mode[myindex].chipset & MD_SIS315))
+	if (ivideo->sisvga_engine == SIS_315_VGA) {
+		if (!(sisbios_mode[myindex].chipset & MD_SIS315))
 			return -1;
 	}
 #endif
 
 	myres = sisbios_mode[myindex].yres;
 
-	switch(vbflags & VB_DISPTYPE_DISP2) {
+	switch (vbflags & VB_DISPTYPE_DISP2) {
 
 	case CRT2_LCD:
 		xres = ivideo->lcdxres; yres = ivideo->lcdyres;
 
-		if((ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL848) &&
-		   (ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL856)) {
-			if(sisbios_mode[myindex].xres > xres)
+		if ((ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL848) &&
+		    (ivideo->SiS_Pr.SiS_CustomT != CUT_PANEL856)) {
+			if (sisbios_mode[myindex].xres > xres)
 				return -1;
-			if(myres > yres)
+			if (myres > yres)
 				return -1;
 		}
 
-		if(ivideo->sisfb_fstn) {
-			if(sisbios_mode[myindex].xres == 320) {
-				if(myres == 240) {
-					switch(sisbios_mode[myindex].mode_no[1]) {
+		if (ivideo->sisfb_fstn) {
+			if (sisbios_mode[myindex].xres == 320) {
+				if (myres == 240) {
+					switch (sisbios_mode[myindex].mode_no[1]) {
 						case 0x50: myindex = MODE_FSTN_8;  break;
 						case 0x56: myindex = MODE_FSTN_16; break;
 						case 0x53: return -1;
@@ -689,7 +689,7 @@  sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
 			}
 		}
 
-		if(SiS_GetModeID_LCD(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
+		if (SiS_GetModeID_LCD(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
 			 	sisbios_mode[myindex].yres, 0, ivideo->sisfb_fstn,
 			 	ivideo->SiS_Pr.SiS_CustomT, xres, yres, ivideo->vbflags2) < 0x14) {
 			return -1;
@@ -697,14 +697,14 @@  sisfb_validate_mode(struct sis_video_info *ivideo, int myindex, u32 vbflags)
 		break;
 
 	case CRT2_TV:
-		if(SiS_GetModeID_TV(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
+		if (SiS_GetModeID_TV(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
 				sisbios_mode[myindex].yres, 0, ivideo->vbflags2) < 0x14) {
 			return -1;
 		}
 		break;
 
 	case CRT2_VGA:
-		if(SiS_GetModeID_VGA2(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
+		if (SiS_GetModeID_VGA2(ivideo->sisvga_engine, vbflags, sisbios_mode[myindex].xres,
 				sisbios_mode[myindex].yres, 0, ivideo->vbflags2) < 0x14) {
 			return -1;
 		}
@@ -2205,82 +2205,88 @@  static bool sisfb_test_DDC1(struct sis_video_info *ivideo)
 
 static void sisfb_sense_crt1(struct sis_video_info *ivideo)
 {
-    bool mustwait = false;
-    u8  sr1F, cr17;
+	bool mustwait = false;
+	u8  sr1F, cr17;
 #ifdef CONFIG_FB_SIS_315
-    u8  cr63=0;
+	u8  cr63 = 0;
 #endif
-    u16 temp = 0xffff;
-    int i;
+	u16 temp = 0xffff;
+	int i;
+
+	sr1F = SiS_GetReg(SISSR, 0x1F);
+	SiS_SetRegOR(SISSR, 0x1F, 0x04);
+	SiS_SetRegAND(SISSR, 0x1F, 0x3F);
 
-    sr1F = SiS_GetReg(SISSR, 0x1F);
-    SiS_SetRegOR(SISSR, 0x1F, 0x04);
-    SiS_SetRegAND(SISSR, 0x1F, 0x3F);
-    if(sr1F & 0xc0) mustwait = true;
+	if (sr1F & 0xc0)
+		mustwait = true;
 
 #ifdef CONFIG_FB_SIS_315
-    if(ivideo->sisvga_engine == SIS_315_VGA) {
-       cr63 = SiS_GetReg(SISCR, ivideo->SiS_Pr.SiS_MyCR63);
-       cr63 &= 0x40;
-       SiS_SetRegAND(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF);
-    }
+	if (ivideo->sisvga_engine == SIS_315_VGA) {
+		cr63 = SiS_GetReg(SISCR, ivideo->SiS_Pr.SiS_MyCR63);
+		cr63 &= 0x40;
+		SiS_SetRegAND(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF);
+	}
 #endif
 
-    cr17 = SiS_GetReg(SISCR, 0x17);
-    cr17 &= 0x80;
-    if(!cr17) {
-       SiS_SetRegOR(SISCR, 0x17, 0x80);
-       mustwait = true;
-       SiS_SetReg(SISSR, 0x00, 0x01);
-       SiS_SetReg(SISSR, 0x00, 0x03);
-    }
+	cr17 = SiS_GetReg(SISCR, 0x17);
+	cr17 &= 0x80;
 
-    if(mustwait) {
-       for(i=0; i < 10; i++) sisfbwaitretracecrt1(ivideo);
-    }
+	if (!cr17) {
+		SiS_SetRegOR(SISCR, 0x17, 0x80);
+		mustwait = true;
+		SiS_SetReg(SISSR, 0x00, 0x01);
+		SiS_SetReg(SISSR, 0x00, 0x03);
+	}
 
+	if (mustwait) {
+		for (i = 0; i < 10; i++)
+			sisfbwaitretracecrt1(ivideo);
+	}
 #ifdef CONFIG_FB_SIS_315
-    if(ivideo->chip >= SIS_330) {
-       SiS_SetRegAND(SISCR, 0x32, ~0x20);
-       if(ivideo->chip >= SIS_340) {
-	   SiS_SetReg(SISCR, 0x57, 0x4a);
-       } else {
-	   SiS_SetReg(SISCR, 0x57, 0x5f);
-       }
-	SiS_SetRegOR(SISCR, 0x53, 0x02);
-	while ((SiS_GetRegByte(SISINPSTAT)) & 0x01)    break;
-	while (!((SiS_GetRegByte(SISINPSTAT)) & 0x01)) break;
-	if ((SiS_GetRegByte(SISMISCW)) & 0x10) temp = 1;
-	SiS_SetRegAND(SISCR, 0x53, 0xfd);
-	SiS_SetRegAND(SISCR, 0x57, 0x00);
-    }
+	if (ivideo->chip >= SIS_330) {
+		SiS_SetRegAND(SISCR, 0x32, ~0x20);
+		if (ivideo->chip >= SIS_340)
+			SiS_SetReg(SISCR, 0x57, 0x4a);
+		else
+			SiS_SetReg(SISCR, 0x57, 0x5f);
+
+		SiS_SetRegOR(SISCR, 0x53, 0x02);
+		while ((SiS_GetRegByte(SISINPSTAT)) & 0x01)
+			break;
+		while (!((SiS_GetRegByte(SISINPSTAT)) & 0x01))
+			break;
+		if ((SiS_GetRegByte(SISMISCW)) & 0x10)
+			temp = 1;
+
+		SiS_SetRegAND(SISCR, 0x53, 0xfd);
+		SiS_SetRegAND(SISCR, 0x57, 0x00);
+	}
 #endif
 
-    if(temp == 0xffff) {
-       i = 3;
-       do {
-	  temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
-		ivideo->sisvga_engine, 0, 0, NULL, ivideo->vbflags2);
-       } while(((temp == 0) || (temp == 0xffff)) && i--);
+	if (temp == 0xffff) {
+		i = 3;
 
-       if((temp == 0) || (temp == 0xffff)) {
-          if(sisfb_test_DDC1(ivideo)) temp = 1;
-       }
-    }
+		do {
+			temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
+			ivideo->sisvga_engine, 0, 0, NULL, ivideo->vbflags2);
+		} while (((temp == 0) || (temp == 0xffff)) && i--);
 
-    if((temp) && (temp != 0xffff)) {
-       SiS_SetRegOR(SISCR, 0x32, 0x20);
-    }
+		if ((temp == 0) || (temp == 0xffff)) {
+			if (sisfb_test_DDC1(ivideo))
+				temp = 1;
+		}
+	}
+
+	if ((temp) && (temp != 0xffff))
+		SiS_SetRegOR(SISCR, 0x32, 0x20);
 
 #ifdef CONFIG_FB_SIS_315
-    if(ivideo->sisvga_engine == SIS_315_VGA) {
-	SiS_SetRegANDOR(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF, cr63);
-    }
+	if (ivideo->sisvga_engine == SIS_315_VGA)
+		SiS_SetRegANDOR(SISCR, ivideo->SiS_Pr.SiS_MyCR63, 0xBF, cr63);
 #endif
 
-    SiS_SetRegANDOR(SISCR, 0x17, 0x7F, cr17);
-
-    SiS_SetReg(SISSR, 0x1F, sr1F);
+	SiS_SetRegANDOR(SISCR, 0x17, 0x7F, cr17);
+	SiS_SetReg(SISSR, 0x1F, sr1F);
 }
 
 /* Determine and detect attached devices on SiS30x */
@@ -2294,25 +2300,25 @@  static void SiS_SenseLCD(struct sis_video_info *ivideo)
 	ivideo->SiS_Pr.PanelSelfDetected = false;
 
 	/* LCD detection only for TMDS bridges */
-	if(!(ivideo->vbflags2 & VB2_SISTMDSBRIDGE))
+	if (!(ivideo->vbflags2 & VB2_SISTMDSBRIDGE))
 		return;
-	if(ivideo->vbflags2 & VB2_30xBDH)
+	if (ivideo->vbflags2 & VB2_30xBDH)
 		return;
 
 	/* If LCD already set up by BIOS, skip it */
 	reg = SiS_GetReg(SISCR, 0x32);
-	if(reg & 0x08)
+	if (reg & 0x08)
 		return;
 
 	realcrtno = 1;
-	if(ivideo->SiS_Pr.DDCPortMixup)
+	if (ivideo->SiS_Pr.DDCPortMixup)
 		realcrtno = 0;
 
 	/* Check DDC capabilities */
 	temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags, ivideo->sisvga_engine,
 				realcrtno, 0, &buffer[0], ivideo->vbflags2);
 
-	if((!temp) || (temp == 0xffff) || (!(temp & 0x02)))
+	if ((!temp) || (temp == 0xffff) || (!(temp & 0x02)))
 		return;
 
 	/* Read DDC data */
@@ -2321,17 +2327,17 @@  static void SiS_SenseLCD(struct sis_video_info *ivideo)
 		temp = SiS_HandleDDC(&ivideo->SiS_Pr, ivideo->vbflags,
 				ivideo->sisvga_engine, realcrtno, 1,
 				&buffer[0], ivideo->vbflags2);
-	} while((temp) && i--);
+	} while ((temp) && i--);
 
-	if(temp)
+	if (temp)
 		return;
 
 	/* No digital device */
-	if(!(buffer[0x14] & 0x80))
+	if (!(buffer[0x14] & 0x80))
 		return;
 
 	/* First detailed timing preferred timing? */
-	if(!(buffer[0x18] & 0x02))
+	if (!(buffer[0x18] & 0x02))
 		return;
 
 	xres = buffer[0x38] | ((buffer[0x3a] & 0xf0) << 4);
@@ -2339,26 +2345,26 @@  static void SiS_SenseLCD(struct sis_video_info *ivideo)
 
 	switch(xres) {
 		case 1024:
-			if(yres == 768)
+			if (yres == 768)
 				paneltype = 0x02;
 			break;
 		case 1280:
-			if(yres == 1024)
+			if (yres == 1024)
 				paneltype = 0x03;
 			break;
 		case 1600:
-			if((yres == 1200) && (ivideo->vbflags2 & VB2_30xC))
+			if ((yres == 1200) && (ivideo->vbflags2 & VB2_30xC))
 				paneltype = 0x0b;
 			break;
 	}
 
-	if(!paneltype)
+	if (!paneltype)
 		return;
 
-	if(buffer[0x23])
+	if (buffer[0x23])
 		cr37 |= 0x10;
 
-	if((buffer[0x47] & 0x18) == 0x18)
+	if ((buffer[0x47] & 0x18) == 0x18)
 		cr37 |= ((((buffer[0x47] & 0x06) ^ 0x06) << 5) | 0x20);
 	else
 		cr37 |= 0xc0;
@@ -2373,31 +2379,34 @@  static void SiS_SenseLCD(struct sis_video_info *ivideo)
 
 static int SISDoSense(struct sis_video_info *ivideo, u16 type, u16 test)
 {
-    int temp, mytest, result, i, j;
-
-    for(j = 0; j < 10; j++) {
-       result = 0;
-       for(i = 0; i < 3; i++) {
-          mytest = test;
-	   SiS_SetReg(SISPART4, 0x11, (type & 0x00ff));
-          temp = (type >> 8) | (mytest & 0x00ff);
-	  SiS_SetRegANDOR(SISPART4, 0x10, 0xe0, temp);
-          SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1500);
-          mytest >>= 8;
-          mytest &= 0x7f;
-	   temp = SiS_GetReg(SISPART4, 0x03);
-          temp ^= 0x0e;
-          temp &= mytest;
-          if(temp == mytest) result++;
+	int temp, mytest, result, i, j;
+
+	for (j = 0; j < 10; j++) {
+		result = 0;
+		for (i = 0; i < 3; i++) {
+			mytest = test;
+			SiS_SetReg(SISPART4, 0x11, (type & 0x00ff));
+			temp = (type >> 8) | (mytest & 0x00ff);
+			SiS_SetRegANDOR(SISPART4, 0x10, 0xe0, temp);
+			SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1500);
+			mytest >>= 8;
+			mytest &= 0x7f;
+			temp = SiS_GetReg(SISPART4, 0x03);
+			temp ^= 0x0e;
+			temp &= mytest;
+			if (temp == mytest)
+				result++;
 #if 1
-	  SiS_SetReg(SISPART4, 0x11, 0x00);
-	  SiS_SetRegAND(SISPART4, 0x10, 0xe0);
-	  SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1000);
+			SiS_SetReg(SISPART4, 0x11, 0x00);
+			SiS_SetRegAND(SISPART4, 0x10, 0xe0);
+			SiS_DDC2Delay(&ivideo->SiS_Pr, 0x1000);
 #endif
-       }
-       if((result == 0) || (result >= 2)) break;
-    }
-    return result;
+		}
+
+		if ((result == 0) || (result >= 2))
+			break;
+	}
+	return result;
 }
 
 static void SiS_Sense30x(struct sis_video_info *ivideo)
@@ -4263,18 +4272,17 @@  static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
 	unsigned int k, RankCapacity, PageCapacity, BankNumHigh, BankNumMid;
 	unsigned int PhysicalAdrOtherPage, PhysicalAdrHigh, PhysicalAdrHalfPage;
 
-	 for(k = 0; k < ARRAY_SIZE(SiS_DRAMType); k++) {
-
+	for (k = 0; k < ARRAY_SIZE(SiS_DRAMType); k++) {
 		RankCapacity = buswidth * SiS_DRAMType[k][3];
 
-		if(RankCapacity != PseudoRankCapacity)
+		if (RankCapacity != PseudoRankCapacity)
 			continue;
 
-		if((SiS_DRAMType[k][2] + SiS_DRAMType[k][0]) > PseudoAdrPinCount)
+		if ((SiS_DRAMType[k][2] + SiS_DRAMType[k][0]) > PseudoAdrPinCount)
 			continue;
 
 		BankNumHigh = RankCapacity * 16 * iteration - 1;
-		if(iteration == 3) {             /* Rank No */
+		if (iteration == 3) {             /* Rank No */
 			BankNumMid  = RankCapacity * 16 - 1;
 		} else {
 			BankNumMid  = RankCapacity * 16 * iteration / 2 - 1;
@@ -4288,18 +4296,22 @@  static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
 		SiS_SetRegAND(SISSR, 0x15, 0xFB); /* Test */
 		SiS_SetRegOR(SISSR, 0x15, 0x04);  /* Test */
 		sr14 = (SiS_DRAMType[k][3] * buswidth) - 1;
-		if(buswidth == 4)      sr14 |= 0x80;
-		else if(buswidth == 2) sr14 |= 0x40;
+
+		if (buswidth == 4)
+			sr14 |= 0x80;
+		else if (buswidth == 2)
+			sr14 |= 0x40;
+
 		SiS_SetReg(SISSR, 0x13, SiS_DRAMType[k][4]);
 		SiS_SetReg(SISSR, 0x14, sr14);
 
 		BankNumHigh <<= 16;
 		BankNumMid <<= 16;
 
-		if((BankNumHigh + PhysicalAdrHigh      >= mapsize) ||
-		   (BankNumMid  + PhysicalAdrHigh      >= mapsize) ||
-		   (BankNumHigh + PhysicalAdrHalfPage  >= mapsize) ||
-		   (BankNumHigh + PhysicalAdrOtherPage >= mapsize))
+		if ((BankNumHigh + PhysicalAdrHigh >= mapsize) ||
+		    (BankNumMid  + PhysicalAdrHigh >= mapsize) ||
+		    (BankNumHigh + PhysicalAdrHalfPage  >= mapsize) ||
+		    (BankNumHigh + PhysicalAdrOtherPage >= mapsize))
 			continue;
 
 		/* Write data */
@@ -4313,7 +4325,7 @@  static int sisfb_post_300_rwtest(struct sis_video_info *ivideo, int iteration,
 				(FBAddr + BankNumHigh + PhysicalAdrOtherPage));
 
 		/* Read data */
-		if(readw(FBAddr + BankNumHigh + PhysicalAdrHigh) == PhysicalAdrHigh)
+		if (readw(FBAddr + BankNumHigh + PhysicalAdrHigh) == PhysicalAdrHigh)
 			return 1;
 	}
 
@@ -6155,24 +6167,20 @@  static int sisfb_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 #endif
 
 #ifdef CONFIG_FB_SIS_315
-		if(ivideo->sisvga_engine == SIS_315_VGA) {
+		if (ivideo->sisvga_engine == SIS_315_VGA) {
 			int result = 1;
-		/*	if((ivideo->chip == SIS_315H)   ||
-			   (ivideo->chip == SIS_315)    ||
-			   (ivideo->chip == SIS_315PRO) ||
-			   (ivideo->chip == SIS_330)) {
-				sisfb_post_sis315330(pdev);
-			} else */ if(ivideo->chip == XGI_20) {
+
+			if (ivideo->chip == XGI_20) {
 				result = sisfb_post_xgi(pdev);
 				ivideo->sisfb_can_post = 1;
-			} else if((ivideo->chip == XGI_40) && ivideo->haveXGIROM) {
+			} else if ((ivideo->chip == XGI_40) && ivideo->haveXGIROM) {
 				result = sisfb_post_xgi(pdev);
 				ivideo->sisfb_can_post = 1;
 			} else {
 				printk(KERN_INFO "sisfb: Card is not "
 					"POSTed and sisfb can't do this either.\n");
 			}
-			if(!result) {
+			if (!result) {
 				printk(KERN_ERR "sisfb: Failed to POST card\n");
 				ret = -ENODEV;
 				goto error_3;