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);