Re: [SeaBIOS] [PATCH 2/2] vgabios: Limit the range of the VBE number of "pages" parameter.

2013-09-14 Thread Kevin O'Connor
On Sat, Sep 14, 2013 at 09:11:02AM +0200, Paul Menzel wrote: > Am Freitag, den 13.09.2013, 16:24 -0400 schrieb Kevin O'Connor: > > Looking at the output of other VGA BIOS implementations, it appears > > that the number of available video pages reported is always between 1 > > and 127. > > That is

Re: [SeaBIOS] [PATCH 2/2] vgabios: Limit the range of the VBE number of "pages" parameter.

2013-09-14 Thread Peter Stuge
Paul Menzel wrote: > > +if (pages < 2) > > +pages++; > > Can pages be 0? If yes, it would be 0 again after substracting 1. Should > it be `pages = 1` instead of `pages++`? pages=2; //Peter ___ SeaBIOS mailing list SeaBIOS@seabios.org http

Re: [SeaBIOS] [PATCH 2/2] vgabios: Limit the range of the VBE number of "pages" parameter.

2013-09-14 Thread Paul Menzel
Am Freitag, den 13.09.2013, 16:24 -0400 schrieb Kevin O'Connor: > Looking at the output of other VGA BIOS implementations, it appears > that the number of available video pages reported is always between 1 > and 127. That is including 1 and 127, right? > Signed-off-by: Kevin O'Connor > --- > vg

[SeaBIOS] [PATCH 2/2] vgabios: Limit the range of the VBE number of "pages" parameter.

2013-09-13 Thread Kevin O'Connor
Looking at the output of other VGA BIOS implementations, it appears that the number of available video pages reported is always between 1 and 127. Signed-off-by: Kevin O'Connor --- vgasrc/vbe.c | 4 1 file changed, 4 insertions(+) diff --git a/vgasrc/vbe.c b/vgasrc/vbe.c index d962333..2c0