On Fri, May 21, 2021 at 11:03:00AM +0200, Gerd Hoffmann wrote:
> On Thu, May 20, 2021 at 01:55:12PM -0400, Kevin O'Connor wrote:
> > On Tue, May 04, 2021 at 07:43:35PM -0400, Elliot Killick via SeaBIOS wrote:
> > > Hi,
> > > 
> > > This patch series adds support for all HD and QXGA resolutions.
> > > 
> > > I've taken Gerd's advice on the v1 patch and split the formatting
> > > improvments into a separate patch.
> > > 
> > > I also tried to take Gerd's advice on removing some of the older colors
> > > depths for higher resolutions. However, upon doing this Windows was no
> > > longer recognizing some of the resolutions so I had to add them back. I
> > > split that up into a separate patch in case anyone wants to try and
> > > reproduce the issue. But, it should be fine to include those common
> > > color depths anyway for maximal compatibility.
> > 
> > Thanks, and sorry for the delay in responding.
> > 
> > In general it looks fine to me, but I was hoping Gerd could take a
> > look as well.
> 
> Looks sane to me.
> 
> > As one minor nit, I think it would be preferable to
> > squash patch 2 and 3 as it seems odd to remove the modes just to add
> > them back again.
> 
> You mean squash 1+3 I guess?

Sorry, yes.

-Kevin
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-le...@seabios.org

Reply via email to