From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rasmus Villemoes Date: Fri, 20 Jun 2014 19:59:57 +0000 Subject: [PATCH] drivers/video/fbdev/sis: Add missing else Message-Id: <1403294397-18262-1-git-send-email-linux@rasmusvillemoes.dk> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jean-Christophe Plagniol-Villard , Tomi Valkeinen Cc: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, Rasmus Villemoes The surrounding code and the fact that the previous two if's are rendered useless without this "else" (since SIS_340=55 and XGI_20=75 are greater than SIS_661=15) suggests that the "else" is indeed missing. Signed-off-by: Rasmus Villemoes --- drivers/video/fbdev/sis/init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/sis/init.c b/drivers/video/fbdev/sis/init.c index bd40f5e..dfe3eb7 100644 --- a/drivers/video/fbdev/sis/init.c +++ b/drivers/video/fbdev/sis/init.c @@ -1511,7 +1511,7 @@ SiS_Get310DRAMType(struct SiS_Private *SiS_Pr) } else if(SiS_Pr->ChipType >= SIS_340) { /* TODO */ data = 0; - } if(SiS_Pr->ChipType >= SIS_661) { + } else if(SiS_Pr->ChipType >= SIS_661) { if(SiS_Pr->SiS_ROMNew) { data = ((SiS_GetReg(SiS_Pr->SiS_P3d4,0x78) & 0xc0) >> 6); } else { -- 1.9.2