qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] compiler error in CONFIG_S3VGA
@ 2005-04-17 18:52 Bernhard Fischer
  2005-04-17 19:31 ` Fabrice Bellard
  0 siblings, 1 reply; 3+ messages in thread
From: Bernhard Fischer @ 2005-04-17 18:52 UTC (permalink / raw)
  To: qemu-devel

[-- Attachment #1: Type: text/plain, Size: 59 bytes --]

Hi,

qemu/hw/vga.c: typo: s/uinr32_t/uint32_t/

Thank you,

[-- Attachment #2: qemu.typedef_typo_in_DEBUG.diff --]
[-- Type: text/plain, Size: 497 bytes --]

diff -X excl -rduNp qemu.oorig/hw/vga.c qemu/hw/vga.c
--- qemu.oorig/hw/vga.c	2005-02-10 23:00:27.000000000 +0100
+++ qemu/hw/vga.c	2005-04-17 02:47:10.000000000 +0200
@@ -956,7 +956,7 @@ static void vga_get_offsets(VGAState *s,
         line_offset = s->cr[0x13];
 #ifdef CONFIG_S3VGA
         {
-            uinr32_t v;
+            uint32_t v;
             v = (s->cr[0x51] >> 4) & 3; /* S3 extension */
             if (v == 0)
                 v = (s->cr[0x43] >> 2) & 1; /* S3 extension */


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] compiler error in CONFIG_S3VGA
  2005-04-17 18:52 [Qemu-devel] compiler error in CONFIG_S3VGA Bernhard Fischer
@ 2005-04-17 19:31 ` Fabrice Bellard
  2005-04-17 21:01   ` Bernhard Fischer
  0 siblings, 1 reply; 3+ messages in thread
From: Fabrice Bellard @ 2005-04-17 19:31 UTC (permalink / raw)
  To: qemu-devel

In fact the real fix is to delete the S3VGA stuff - it has never worked 
and I have no plan to support it. Moveover, a correct implementation 
would be made in another driver as the cirrus_vga driver.

Fabrice.

Bernhard Fischer wrote:
> Hi,
> 
> qemu/hw/vga.c: typo: s/uinr32_t/uint32_t/
> 
> Thank you,
> 
> 
> ------------------------------------------------------------------------
> 
> diff -X excl -rduNp qemu.oorig/hw/vga.c qemu/hw/vga.c
> --- qemu.oorig/hw/vga.c	2005-02-10 23:00:27.000000000 +0100
> +++ qemu/hw/vga.c	2005-04-17 02:47:10.000000000 +0200
> @@ -956,7 +956,7 @@ static void vga_get_offsets(VGAState *s,
>          line_offset = s->cr[0x13];
>  #ifdef CONFIG_S3VGA
>          {
> -            uinr32_t v;
> +            uint32_t v;
>              v = (s->cr[0x51] >> 4) & 3; /* S3 extension */
>              if (v == 0)
>                  v = (s->cr[0x43] >> 2) & 1; /* S3 extension */
> 
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] compiler error in CONFIG_S3VGA
  2005-04-17 19:31 ` Fabrice Bellard
@ 2005-04-17 21:01   ` Bernhard Fischer
  0 siblings, 0 replies; 3+ messages in thread
From: Bernhard Fischer @ 2005-04-17 21:01 UTC (permalink / raw)
  To: qemu-devel

On Sun, Apr 17, 2005 at 09:31:32PM +0200, Fabrice Bellard wrote:
>In fact the real fix is to delete the S3VGA stuff - it has never worked 
>and I have no plan to support it. Moveover, a correct implementation 
>would be made in another driver as the cirrus_vga driver.

Also remove the DEBUG_TB_CHECK path, which gives me errors at link-time
(page_get_flags) if CONFIG_USER_ONLY is not defined?

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-04-17 21:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-04-17 18:52 [Qemu-devel] compiler error in CONFIG_S3VGA Bernhard Fischer
2005-04-17 19:31 ` Fabrice Bellard
2005-04-17 21:01   ` Bernhard Fischer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).