Module Name:    src
Committed By:   christos
Date:           Fri Jun 28 15:26:57 UTC 2013

Modified Files:
        src/sys/dev/pci: pm2fb.c

Log Message:
this could have never worked. Cleanup bogus code.
http://m00nbsd.net/ae123a9bae03f7dde5c6d654412daf5a.html


To generate a diff of this commit:
cvs rdiff -u -r1.21 -r1.22 src/sys/dev/pci/pm2fb.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/pci/pm2fb.c
diff -u src/sys/dev/pci/pm2fb.c:1.21 src/sys/dev/pci/pm2fb.c:1.22
--- src/sys/dev/pci/pm2fb.c:1.21	Sat Oct  6 10:41:40 2012
+++ src/sys/dev/pci/pm2fb.c	Fri Jun 28 11:26:57 2013
@@ -1,4 +1,4 @@
-/*	$NetBSD: pm2fb.c,v 1.21 2012/10/06 14:41:40 macallan Exp $	*/
+/*	$NetBSD: pm2fb.c,v 1.22 2013/06/28 15:26:57 christos Exp $	*/
 
 /*
  * Copyright (c) 2009, 2012 Michael Lorenz
@@ -31,7 +31,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pm2fb.c,v 1.21 2012/10/06 14:41:40 macallan Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pm2fb.c,v 1.22 2013/06/28 15:26:57 christos Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -1342,19 +1342,17 @@ pm2_setup_i2c(struct pm2fb_softc *sc)
 		 * best one we can support
 		 */
 		if (sc->sc_videomode == NULL) {
-			int n;
 			struct videomode *m = sc->sc_ei.edid_modes;
 
 			sort_modes(sc->sc_ei.edid_modes,
-			 	    &sc->sc_ei.edid_preferred_mode,
-				    sc->sc_ei.edid_nmodes);
-			while ((sc->sc_videomode == NULL) &&
-			       (n < sc->sc_ei.edid_nmodes)) {
-				if (MODE_IS_VALID(&m[n])) {
-					sc->sc_videomode = &m[n];
-				}
-				n++;
-			}
+			    &sc->sc_ei.edid_preferred_mode,
+			    sc->sc_ei.edid_nmodes);
+			if (sc->sc_videomode == NULL)
+				for (int n = 0; n < sc->sc_ei.edid_nmodes; n++)
+					if (MODE_IS_VALID(&m[n])) {
+						sc->sc_videomode = &m[n];
+						break;
+					}
 		}
 	}
 	if (sc->sc_videomode == NULL) {

Reply via email to