From df4b30bee2b0d08fcf7fb0df37974ddb77d72ce6 Mon Sep 17 00:00:00 2001 From: Marius Strobl Date: Sat, 4 Jun 2005 21:15:27 +0000 Subject: [PATCH] - In creator_configure() when probed for the high-level console return the number of registered adapters instead of determining again whether stdout is a supported card (and which might have failed to attach and register). - Drop creator_set_mode() and move the relevant parts to creator_fill_rect() and creator_putc() respectively. This is a bit cleaner than having to make sure that creator_set_mode() was called before creator_fill_rect() or creator_putc() are used and matches better what Xorg does. - Fix a bug in the handling of the FBIOSCURSOR IOCTL; the code was meant to return ENODEV for all invocations expect when used to disable the cursor and not just when used for enabling the cursor. - In case the adapter is the OFW stdout move its OFW cursor to the start of the last line on halt so OFW output doesn't get intermixed with what FreeBSD left on the screen. With hindsight this is what the faking of a hardware cursor which was removed in the last revision really was about, i.e. to keep the OFW updated about the current cursor position. The new approach however is simpler while producing the same result and doesn't cause the first letter of the OFW output to be turned into a blank and a newline. - Add variable names to the prototypes of creator_cursor_*() which were added in the last revision and list them alphabetically in order to match the style of this file. --- sys/dev/fb/creator.c | 67 +++++++++++++------------------ sys/dev/fb/creatorreg.h | 7 ++-- sys/sparc64/creator/creator.h | 7 ++-- sys/sparc64/creator/creator_upa.c | 9 +++++ 4 files changed, 46 insertions(+), 44 deletions(-) diff --git a/sys/dev/fb/creator.c b/sys/dev/fb/creator.c index 13c894e219f..f2bdcd87dcb 100644 --- a/sys/dev/fb/creator.c +++ b/sys/dev/fb/creator.c @@ -81,12 +81,12 @@ static vi_putc_t creator_putc; static vi_puts_t creator_puts; static vi_putm_t creator_putm; +static void creator_cursor_enable(struct creator_softc *sc, int onoff); +static void creator_cursor_install(struct creator_softc *sc); static void creator_ras_fifo_wait(struct creator_softc *sc, int n); static void creator_ras_setbg(struct creator_softc *sc, int bg); static void creator_ras_setfg(struct creator_softc *sc, int fg); static void creator_ras_wait(struct creator_softc *sc); -static void creator_cursor_enable(struct creator_softc *, int); -static void creator_cursor_install(struct creator_softc *); static video_switch_t creatorvidsw = { .probe = creator_probe, @@ -191,7 +191,22 @@ creator_configure(int flags) char buf[32]; int i; + /* + * For the high-level console probing return the number of + * registered adapters. + */ + if (!(flags & VIO_PROBE_ONLY)) { + for (i = 0; vid_find_adapter(CREATOR_DRIVER_NAME, i) >= 0; i++) + ; + return (i); + } + + /* Low-level console probing and initialization. */ + sc = &creator_softc; + if (sc->sc_va.va_flags & V_ADP_REGISTERED) + goto found; + if ((chosen = OF_finddevice("/chosen")) == -1) return (0); if (OF_getprop(chosen, "stdout", &stdout, sizeof(stdout)) == -1) @@ -201,14 +216,7 @@ creator_configure(int flags) if (OF_getprop(output, "name", buf, sizeof(buf)) == -1) return (0); if (strcmp(buf, "SUNW,ffb") == 0 || strcmp(buf, "SUNW,afb") == 0) { - /* - * When being called a second time, i.e. during - * sc_probe_unit(), just return at this point. - * Note that the polarity of the VIO_PROBE_ONLY - * flag is somewhat non-intuitive. - */ - if (!(flags & VIO_PROBE_ONLY)) - goto found; + sc->sc_flags = CREATOR_CONSOLE; if (strcmp(buf, "SUNW,afb") == 0) sc->sc_flags |= CREATOR_AFB; sc->sc_node = output; @@ -297,12 +305,6 @@ creator_init(int unit, video_adapter_t *adp, int flags) } } - /* - * FFB_DAC_CFG_TGEN_VIDE must be turned on for creator_set_mode() - * to take effect. - */ - creator_blank_display(adp, V_DISPLAY_ON); - creator_set_mode(adp, 0); creator_blank_display(adp, V_DISPLAY_BLANK); adp->va_flags |= V_ADP_COLOR | V_ADP_BORDER | V_ADP_INITIALIZED; @@ -331,20 +333,8 @@ creator_query_mode(video_adapter_t *adp, video_info_t *info) static int creator_set_mode(video_adapter_t *adp, int mode) { - struct creator_softc *sc; - sc = (struct creator_softc *)adp; - creator_ras_fifo_wait(sc, 4); - FFB_WRITE(sc, FFB_FBC, FFB_FBC_ROP, FBC_ROP_NEW); - FFB_WRITE(sc, FFB_FBC, FFB_FBC_DRAWOP, FBC_DRAWOP_RECTANGLE); - FFB_WRITE(sc, FFB_FBC, FFB_FBC_PMASK, 0xffffffff); - FFB_WRITE(sc, FFB_FBC, FFB_FBC_FONTINC, 0x10000); - - creator_ras_setbg(sc, 0x0); - creator_ras_setfg(sc, 0xffffff); - creator_ras_wait(sc); - - return (0); + return (ENODEV); } static int @@ -491,13 +481,12 @@ creator_ioctl(video_adapter_t *adp, u_long cmd, caddr_t data) break; case FBIOSCURSOR: fbc = (struct fbcursor *)data; - if (fbc->set & FB_CUR_SETCUR) { - if (fbc->enable == 0) { - creator_cursor_enable(sc, 0); - sc->sc_flags &= ~CREATOR_CUREN; - } else - return (ENODEV); - } + if (fbc->set & FB_CUR_SETCUR && fbc->enable == 0) { + creator_cursor_enable(sc, 0); + sc->sc_flags &= ~CREATOR_CUREN; + } else + return (ENODEV); + break; break; default: return (fb_commonioctl(adp, cmd, data)); @@ -518,7 +507,8 @@ creator_fill_rect(video_adapter_t *adp, int val, int x, int y, int cx, int cy) struct creator_softc *sc; sc = (struct creator_softc *)adp; - creator_ras_fifo_wait(sc, 2); + creator_ras_fifo_wait(sc, 3); + FFB_WRITE(sc, FFB_FBC, FFB_FBC_PMASK, 0xffffffff); FFB_WRITE(sc, FFB_FBC, FFB_FBC_ROP, FBC_ROP_NEW); FFB_WRITE(sc, FFB_FBC, FFB_FBC_DRAWOP, FBC_DRAWOP_RECTANGLE); creator_ras_setfg(sc, cmap[val & 0xf]); @@ -593,10 +583,11 @@ creator_putc(video_adapter_t *adp, vm_offset_t off, u_int8_t c, u_int8_t a) p = (uint16_t *)sc->sc_font + (c * adp->va_info.vi_cheight); creator_ras_setfg(sc, cmap[a & 0xf]); creator_ras_setbg(sc, cmap[(a >> 4) & 0xf]); - creator_ras_fifo_wait(sc, 2 + adp->va_info.vi_cheight); + creator_ras_fifo_wait(sc, 3 + adp->va_info.vi_cheight); FFB_WRITE(sc, FFB_FBC, FFB_FBC_FONTXY, ((row + sc->sc_ymargin) << 16) | (col + sc->sc_xmargin)); FFB_WRITE(sc, FFB_FBC, FFB_FBC_FONTW, adp->va_info.vi_cwidth); + FFB_WRITE(sc, FFB_FBC, FFB_FBC_FONTINC, 0x10000); for (i = 0; i < adp->va_info.vi_cheight; i++) { FFB_WRITE(sc, FFB_FBC, FFB_FBC_FONT, *p++ << 16); } diff --git a/sys/dev/fb/creatorreg.h b/sys/dev/fb/creatorreg.h index bb8162fc0f4..ce79582360e 100644 --- a/sys/dev/fb/creatorreg.h +++ b/sys/dev/fb/creatorreg.h @@ -234,9 +234,10 @@ struct creator_softc { int sc_flags; #define CREATOR_AFB (1 << 0) -#define CREATOR_CUREN (1 << 1) -#define CREATOR_CURINV (1 << 2) -#define CREATOR_PAC1 (1 << 3) +#define CREATOR_CONSOLE (1 << 1) +#define CREATOR_CUREN (1 << 2) +#define CREATOR_CURINV (1 << 3) +#define CREATOR_PAC1 (1 << 4) }; #endif /* !_DEV_FB_CREATOR_H_ */ diff --git a/sys/sparc64/creator/creator.h b/sys/sparc64/creator/creator.h index bb8162fc0f4..ce79582360e 100644 --- a/sys/sparc64/creator/creator.h +++ b/sys/sparc64/creator/creator.h @@ -234,9 +234,10 @@ struct creator_softc { int sc_flags; #define CREATOR_AFB (1 << 0) -#define CREATOR_CUREN (1 << 1) -#define CREATOR_CURINV (1 << 2) -#define CREATOR_PAC1 (1 << 3) +#define CREATOR_CONSOLE (1 << 1) +#define CREATOR_CUREN (1 << 2) +#define CREATOR_CURINV (1 << 3) +#define CREATOR_PAC1 (1 << 4) }; #endif /* !_DEV_FB_CREATOR_H_ */ diff --git a/sys/sparc64/creator/creator_upa.c b/sys/sparc64/creator/creator_upa.c index d7b1bb8d978..82c550f169b 100644 --- a/sys/sparc64/creator/creator_upa.c +++ b/sys/sparc64/creator/creator_upa.c @@ -301,4 +301,13 @@ creator_shutdown(void *v) FFB_WRITE(sc, FFB_DAC, FFB_DAC_VALUE2, sc->sc_flags & CREATOR_CURINV ? 0 : FFB_DAC_CUR_CTRL_P0 | FFB_DAC_CUR_CTRL_P1); + /* + * In case this is the console set the cursor of the stdout + * instance to the start of the last line so OFW output ends + * up beneath what FreeBSD left on the screen. + */ + if (sc->sc_flags & CREATOR_CONSOLE) { + OF_interpret("stdout @ is my-self 0 to column#", 0); + OF_interpret("stdout @ is my-self #lines 1 - to line#", 0); + } }