<br><br><div class="gmail_quote">2010/3/6 Rudolf Marek <span dir="ltr"><<a href="mailto:r.marek@assembler.cz">r.marek@assembler.cz</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Hi,<br>
<br>
Following patch changes the K8M890 VGA handling. It reverts the framebuffer size<br>
to option based (similar what Uwe did) and also it uses GFXUMA to handle the<br>
high_tables_start offset from memory top.<br>
<br>
To satisfy the CMOS option users (Hi, libv! ;) I added also a possibility to do<br>
that through CMOS.<br>
<br>
Signed-off-by: Rudolf Marek <<a href="mailto:r.marek@assembler.cz">r.marek@assembler.cz</a>><br></blockquote><div><br>+choice<br>+    prompt "Framebuffer size"<br>+    default K8M890_VIDEO_MB_32MB<br>+    depends on SOUTHBRIDGE_VIA_K8T890_VGA_EN<br>
+<br>+config K8M890_VIDEO_MB_32MB<br>+    bool "32MB"<br>+config K8M890_VIDEO_MB_64MB<br>+    bool "64MB"<br>+config K8M890_VIDEO_MB_128MB<br>+    bool "128MB"<br>+config K8M890_VIDEO_MB_256MB<br>
+    bool "256MB"<br>+config K8M890_VIDEO_MB_CMOS<br>+    bool "Use CMOS option"<br><br>Shouldn't there be a config K8M890_VIDEO_MB_OFF choice?<br><br>+endchoice<br>+<br>+config VIDEO_MB<br>+    int<br>
+    default 0   if K8M890_VIDEO_MB_OFF<br><br>I didn't see any comments documenting the meaning of fbbits.  At first I thought it  was log2 of the MB of memory....<br><br>+        uma_memory_size = 4 << (fbbits + 20);<br>
+    } else {<br>+        uma_memory_size = (CONFIG_VIDEO_MB << 20);<br>     }<br><br>And this definition looks a little different, but maybe it's equivalent?<br><br>+    fbbits = ((log2(uma_memory_size >> 20) - 2) << 4);<br>
+    printk_info("K8M890: Using a %dMB framebuffer.\n", (unsigned int) (uma_memory_size >> 20));<br> <br>It seems like clarity would be good, since it's a user option.<br><br>With a few minor things addressed: <br>
<br>Acked-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br><br>Thanks,<br>Myles<br></div></div><br>