<p dir="ltr"><br>
Am 03.09.2012 18:07 schrieb "Christian Gmeiner" <<a href="mailto:christian.gmeiner@gmail.com">christian.gmeiner@gmail.com</a>>:<br>
><br>
> 2012/9/3 Kevin O'Connor <<a href="mailto:kevin@koconnor.net">kevin@koconnor.net</a>>:<br>
> > On Sat, Sep 01, 2012 at 05:12:55PM +0200, Christian Gmeiner wrote:<br>
> >> As all supported graphic adapters are supporting VGA we should<br>
> >> list them in int10 (AX = 4F00h). Also clext.c and bochsvga.c<br>
> >> are haveing a function named like is_XXX_mode() which checks<br>
> >> if its a default VGA mode or a VESA mode. If its a normal VGA<br>
> >> mode stdvga_set_mode() gets called.<br>
> > [...]<br>
> >>  void<br>
> >>  stdvga_list_modes(u16 seg, u16 *dest, u16 *last)<br>
> >>  {<br>
> >> +    int i;<br>
> >> +    for (i = 0; i < ARRAY_SIZE(vga_modes); i++) {<br>
> >> +        struct stdvga_mode_s *stdmode_g = &vga_modes[i];<br>
> >> +        u16 mode = GET_GLOBAL(stdmode_g->mode);<br>
> >> +        if (mode == 0xffff)<br>
> >> +            continue;<br>
> >> +        SET_FARVAR(seg, *dest, mode);<br>
> >> +        dest++;<br>
> >> +    }<br>
> >> +<br>
> >>      SET_FARVAR(seg, *dest, 0xffff);<br>
> >>  }<br>
> ><br>
> > This is a good idea.  However, I don't think vbe_104f01() will<br>
> > correctly handle all the standard vga modes as it is coded today.<br>
> ><br>
><br>
> I will have a closer look at it during my following patches. If you want you<br>
> can skip this patch.<br>
></p>
<p dir="ltr">Maybe it would be better to use this patch and update vga_modes array to reflect the modes we can do now.</p>
<p dir="ltr">> ---<br>
> Christian Gmeiner, MSc<br>
</p>