Module Name: src Committed By: macallan Date: Wed Sep 16 16:52:54 UTC 2015
Modified Files: src/sys/dev/pci: gffb.c pm2fb.c r128fb.c voodoofb.c Log Message: prop_dictionary_get_bool() doesn't return FALSE if the property doesn't exist, so initialize things first. No more console stealing on sparc64. To generate a diff of this commit: cvs rdiff -u -r1.9 -r1.10 src/sys/dev/pci/gffb.c cvs rdiff -u -r1.27 -r1.28 src/sys/dev/pci/pm2fb.c cvs rdiff -u -r1.38 -r1.39 src/sys/dev/pci/r128fb.c cvs rdiff -u -r1.49 -r1.50 src/sys/dev/pci/voodoofb.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/gffb.c diff -u src/sys/dev/pci/gffb.c:1.9 src/sys/dev/pci/gffb.c:1.10 --- src/sys/dev/pci/gffb.c:1.9 Sun Jun 29 03:43:06 2014 +++ src/sys/dev/pci/gffb.c Wed Sep 16 16:52:54 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: gffb.c,v 1.9 2014/06/29 03:43:06 tsutsui Exp $ */ +/* $NetBSD: gffb.c,v 1.10 2015/09/16 16:52:54 macallan Exp $ */ /* * Copyright (c) 2013 Michael Lorenz @@ -35,7 +35,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: gffb.c,v 1.9 2014/06/29 03:43:06 tsutsui Exp $"); +__KERNEL_RCSID(0, "$NetBSD: gffb.c,v 1.10 2015/09/16 16:52:54 macallan Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -181,7 +181,7 @@ gffb_attach(device_t parent, device_t se struct wsemuldisplaydev_attach_args aa; prop_dictionary_t dict; unsigned long defattr; - bool is_console; + bool is_console = FALSE; int i, j, f; uint8_t cmap[768]; Index: src/sys/dev/pci/pm2fb.c diff -u src/sys/dev/pci/pm2fb.c:1.27 src/sys/dev/pci/pm2fb.c:1.28 --- src/sys/dev/pci/pm2fb.c:1.27 Wed Dec 17 16:50:08 2014 +++ src/sys/dev/pci/pm2fb.c Wed Sep 16 16:52:54 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: pm2fb.c,v 1.27 2014/12/17 16:50:08 macallan Exp $ */ +/* $NetBSD: pm2fb.c,v 1.28 2015/09/16 16:52:54 macallan Exp $ */ /* * Copyright (c) 2009, 2012 Michael Lorenz @@ -31,7 +31,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: pm2fb.c,v 1.27 2014/12/17 16:50:08 macallan Exp $"); +__KERNEL_RCSID(0, "$NetBSD: pm2fb.c,v 1.28 2015/09/16 16:52:54 macallan Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -315,7 +315,7 @@ pm2fb_attach(device_t parent, device_t s struct wsemuldisplaydev_attach_args aa; prop_dictionary_t dict; unsigned long defattr; - bool is_console; + bool is_console = FALSE; uint32_t flags; int i; Index: src/sys/dev/pci/r128fb.c diff -u src/sys/dev/pci/r128fb.c:1.38 src/sys/dev/pci/r128fb.c:1.39 --- src/sys/dev/pci/r128fb.c:1.38 Wed Oct 9 17:18:23 2013 +++ src/sys/dev/pci/r128fb.c Wed Sep 16 16:52:54 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: r128fb.c,v 1.38 2013/10/09 17:18:23 macallan Exp $ */ +/* $NetBSD: r128fb.c,v 1.39 2015/09/16 16:52:54 macallan Exp $ */ /* * Copyright (c) 2007, 2012 Michael Lorenz @@ -31,7 +31,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: r128fb.c,v 1.38 2013/10/09 17:18:23 macallan Exp $"); +__KERNEL_RCSID(0, "$NetBSD: r128fb.c,v 1.39 2015/09/16 16:52:54 macallan Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -201,7 +201,7 @@ r128fb_attach(device_t parent, device_t struct wsemuldisplaydev_attach_args aa; prop_dictionary_t dict; unsigned long defattr; - bool is_console; + bool is_console = FALSE; int i, j; uint32_t reg, flags; uint8_t cmap[768]; Index: src/sys/dev/pci/voodoofb.c diff -u src/sys/dev/pci/voodoofb.c:1.49 src/sys/dev/pci/voodoofb.c:1.50 --- src/sys/dev/pci/voodoofb.c:1.49 Thu Jul 24 09:39:58 2014 +++ src/sys/dev/pci/voodoofb.c Wed Sep 16 16:52:54 2015 @@ -1,4 +1,4 @@ -/* $NetBSD: voodoofb.c,v 1.49 2014/07/24 09:39:58 macallan Exp $ */ +/* $NetBSD: voodoofb.c,v 1.50 2015/09/16 16:52:54 macallan Exp $ */ /* * Copyright (c) 2005, 2006, 2012 Michael Lorenz @@ -32,7 +32,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: voodoofb.c,v 1.49 2014/07/24 09:39:58 macallan Exp $"); +__KERNEL_RCSID(0, "$NetBSD: voodoofb.c,v 1.50 2015/09/16 16:52:54 macallan Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -694,7 +694,7 @@ static bool voodoofb_is_console(struct voodoofb_softc *sc) { prop_dictionary_t dict; - bool console; + bool console = FALSE; dict = device_properties(sc->sc_dev); prop_dictionary_get_bool(dict, "is_console", &console);