diff mbox

Refuse to load driver for depth-8 X server.

Message ID 1361573188-4249-1-git-send-email-cworth@cworth.org (mailing list archive)
State New, archived
Headers show

Commit Message

Carl Worth Feb. 22, 2013, 10:46 p.m. UTC 
The driver is not functional at this depth, so tell the user as much and
bail out.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=31375
---
 src/intel_driver.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Eric Anholt Feb. 23, 2013, 12:50 a.m. UTC | #1 
Carl Worth <cworth@cworth.org> writes:

> The driver is not functional at this depth, so tell the user as much and
> bail out.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=31375

Yeah, we shouldn't be supporting 8bpp.

Reviewed-by: Eric Anholt <eric@anholt.net>
diff mbox

Patch

diff --git a/src/intel_driver.c b/src/intel_driver.c
index 7807106..7f11978 100644
--- a/src/intel_driver.c
+++ b/src/intel_driver.c
@@ -532,15 +532,15 @@  static Bool I830PreInit(ScrnInfoPtr scrn, int flags)
 		return FALSE;
 
 	switch (scrn->depth) {
-	case 8:
 	case 15:
 	case 16:
 	case 24:
 	case 30:
 		break;
+	case 8:
 	default:
 		xf86DrvMsg(scrn->scrnIndex, X_ERROR,
-			   "Given depth (%d) is not supported by I830 driver\n",
+			   "Given depth (%d) is not supported by intel driver\n",
 			   scrn->depth);
 		return FALSE;
 	}