Thread (2 messages) 2 messages, 2 authors, 2014-03-07

Re: [patch -resend] sisfb: fix 1280x720 resolution support

From: Tomi Valkeinen <hidden>
Date: 2014-03-07 12:20:51
Also in: kernel-janitors

On 07/03/14 13:18, Dan Carpenter wrote:
quoted hunk ↗ jump to hunk
It uses the wrong mode index because there is no break statement.

Signed-off-by: Dan Carpenter <redacted>
diff --git a/drivers/video/fbdev/sis/init.c b/drivers/video/fbdev/sis/init.c
index 4f26bc28e60b..bd40f5ecd901 100644
--- a/drivers/video/fbdev/sis/init.c
+++ b/drivers/video/fbdev/sis/init.c
@@ -651,6 +651,7 @@ SiS_GetModeID_LCD(int VGAEngine, unsigned int VBFlags, int HDisplay, int VDispla
 	     switch(VDisplay) {
 	     case 720:
 		ModeIndex = ModeIndex_1280x720[Depth];
+		break;
 	     case 768:
 		if(VGAEngine == SIS_300_VGA) {
 		   ModeIndex = ModeIndex_300_1280x768[Depth];
Thanks, queued for 3.15.

 Tomi

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help