Changeset a71c158 in mainline for kernel/arch/ia32/src/drivers/vesa.c

Timestamp:

2009-08-21T14:12:45Z (15 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

0e6dce8, b50b5af2, e5792d1

Parents:

90c8b8d

Message:

kernel output devices now suport multiple instances (except ski and sgcn, which respect the same interface, but behave as singletons)
if more than one output device gets initialized, the output is cloned to all of them
get rid of arch_grab_console() and arch_release_console() (output devices can implement a generic "redraw" method, input devices respect the "silent" global variable)
related cleanups and modifications

File:

• kernel/arch/ia32/src/drivers/vesa.c (modified) (5 diffs)

kernel/arch/ia32/src/drivers/vesa.c

@@ -27 +27 @@
  */
 
-/** @addtogroup ia32    
+/** @addtogroup ia32
  * @{
  */
@@ -40 +40 @@
 #include <genarch/fb/visuals.h>
 #include <arch/drivers/vesa.h>
+#include <console/chardev.h>
+#include <console/console.h>
 #include <putchar.h>
 #include <mm/page.h>
@@ -66 +68 @@
 uint8_t vesa_blue_pos;
 
-int vesa_present(void)
+bool vesa_init(void)
 {
-        if ((vesa_width != 0xffff) && (vesa_height != 0xffff))
-                return true;
+        if ((vesa_width == 0xffff) || (vesa_height == 0xffff))
+                return false;
         
-        return false;
-}
-
-void vesa_init(void)
-{
-        unsigned int visual;
+        visual_t visual;
         
         switch (vesa_bpp) {
@@ -97 +94 @@
                 break;
         default:
-                panic("Unsupported bits per pixel.");
+                LOG("Unsupported bits per pixel.");
+                return false;
         }
         
@@ -108 +106 @@
                 .visual = visual,
         };
-        fb_init(&vesa_props);
-}
-
-void vesa_redraw(void)
-{
-        fb_redraw();
+        
+        outdev_t *fbdev = fb_init(&vesa_props);
+        if (!fbdev)
+                return false;
+        
+        stdout_wire(fbdev);
+        return true;
 }