[src/trunk]: src/sys/dev/pci if we find composity sync enabled, leave it on i...

[macallan <macallan%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/0901fe9cfdca
branches:  trunk
changeset: 936563:0901fe9cfdca
user:      macallan <macallan%NetBSD.org@localhost>
date:      Thu Jul 30 21:29:20 2020 +0000

description:
if we find composity sync enabled, leave it on instead of unconditionally
enabling it on sparc*

diffstat:

 sys/dev/pci/machfb.c |  17 +++++++++--------
 1 files changed, 9 insertions(+), 8 deletions(-)

diffs (52 lines):

diff -r ece6ec7dd89d -r 0901fe9cfdca sys/dev/pci/machfb.c
--- a/sys/dev/pci/machfb.c      Thu Jul 30 21:25:43 2020 +0000
+++ b/sys/dev/pci/machfb.c      Thu Jul 30 21:29:20 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: machfb.c,v 1.99 2020/07/05 09:23:42 martin Exp $       */
+/*     $NetBSD: machfb.c,v 1.100 2020/07/30 21:29:20 macallan Exp $    */
 
 /*
  * Copyright (c) 2002 Bang Jun-Young
@@ -34,7 +34,7 @@
 
 #include <sys/cdefs.h>
 __KERNEL_RCSID(0,
-       "$NetBSD: machfb.c,v 1.99 2020/07/05 09:23:42 martin Exp $");
+       "$NetBSD: machfb.c,v 1.100 2020/07/30 21:29:20 macallan Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -147,6 +147,7 @@
        uint8_t sc_edid_data[1024];
        struct edid_info sc_ei;
        int sc_setmode;
+       int sc_gen_cntl;
 
        u_char sc_cmap_red[256];
        u_char sc_cmap_green[256];
@@ -613,7 +614,11 @@
        }
 
        sc->sc_console = mach64_is_console(sc);
-       aprint_debug("gen_cntl: %08x\n", regr(sc, CRTC_GEN_CNTL));
+       sc->sc_gen_cntl = regr(sc, CRTC_GEN_CNTL);
+       aprint_debug("gen_cntl: %08x\n", sc->sc_gen_cntl);
+       sc->sc_gen_cntl &= CRTC_CSYNC_EN;
+       aprint_normal_dev(sc->sc_dev, "found composite sync %s\n",
+           sc->sc_gen_cntl ? "enabled" : "disabled");
 
 #define MODE_IS_VALID(m) ((sc->ramdac_freq >= (m)->dot_clock) && \
                          ((m)->hdisplay <= 1280))
@@ -996,11 +1001,7 @@
        regw(sc, CRTC_OFF_PITCH, (sc->virt_x >> 3) << 22);
 
        regw(sc, CRTC_GEN_CNTL, crtc->gen_cntl | crtc->color_depth |
-/* XXX this unconditionally enables composite sync on SPARC */
-#ifdef __sparc__
-           CRTC_CSYNC_EN |
-#endif
-           CRTC_EXT_DISP_EN | CRTC_EXT_EN);
+           sc->sc_gen_cntl | CRTC_EXT_DISP_EN | CRTC_EXT_EN);
 }
 
 static int