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?

take care,
  Gerd

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

Reply via email to