[SeaBIOS] [PATCH v3 0/2] Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

Uwe Kleine-König posted 2 patches 10 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/seabios tags/patchew/20190731155115.16571-1-uwe@kleine-koenig.org
vgasrc/cbvga.c     | 80 ++++++----------------------------------------
vgasrc/svgamodes.c |  8 +++++
2 files changed, 17 insertions(+), 71 deletions(-)

[SeaBIOS] [PATCH v3 0/2] Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

Posted by Uwe Kleine-König 10 weeks ago
Hello,

this is my attempt to address the review comments I got for v2:

 - Gerd pointed out that cbvesa should only use the modes where memmodel
   == MM_DIRECT.

 - Kevin noted that reading global variables need the GET_GLOBAL
   wrapper. (I would have expected that the compiler does the right
   thing here. Probably I'm still to wet behind the ears here :-)

Note this is only compile tested.

Uwe Kleine-König (2):
  cbvga: reuse svga modes definitions from svgamodes.c
  Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

 vgasrc/cbvga.c     | 80 ++++++----------------------------------------
 vgasrc/svgamodes.c |  8 +++++
 2 files changed, 17 insertions(+), 71 deletions(-)

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

[SeaBIOS] Re: [PATCH v3 0/2] Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

Posted by Uwe Kleine-König 5 days ago
On 7/31/19 5:51 PM, Uwe Kleine-König wrote:
> Hello,
> 
> this is my attempt to address the review comments I got for v2:
> 
>  - Gerd pointed out that cbvesa should only use the modes where memmodel
>    == MM_DIRECT.
> 
>  - Kevin noted that reading global variables need the GET_GLOBAL
>    wrapper. (I would have expected that the compiler does the right
>    thing here. Probably I'm still to wet behind the ears here :-)
> 
> Note this is only compile tested.

Is there anything I have to do here to get these patches merged?

Best regards
Uwe

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

[SeaBIOS] Re: [PATCH v3 0/2] Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

Posted by Kevin O'Connor 3 days ago
On Thu, Oct 10, 2019 at 05:43:30PM +0200, Uwe Kleine-König wrote:
> On 7/31/19 5:51 PM, Uwe Kleine-König wrote:
> > Hello,
> > 
> > this is my attempt to address the review comments I got for v2:
> > 
> >  - Gerd pointed out that cbvesa should only use the modes where memmodel
> >    == MM_DIRECT.
> > 
> >  - Kevin noted that reading global variables need the GET_GLOBAL
> >    wrapper. (I would have expected that the compiler does the right
> >    thing here. Probably I'm still to wet behind the ears here :-)
> > 
> > Note this is only compile tested.
> 
> Is there anything I have to do here to get these patches merged?

The patches look fine to me.  The commits should have a signed-off-by
line though.  Gerd - do you have any further comments on this?

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

[SeaBIOS] Re: [PATCH v3 0/2] Add additional resolutions for 16:9 displays: 1600x900 and 2560x1440

Posted by Uwe Kleine-König 1 day ago
Hello Kevin,

On 10/13/19 4:07 AM, Kevin O'Connor wrote:
> The commits should have a signed-off-by
> line though.

Not sure what is meant here. Do you miss an S-o-b by me or by Gerd?

If the former and assuming the intention is that its meaning is similar
to that in the kernel: INAL, but I think if you want to have an at least
small chance that this formalism has any juristic meaning you need to
formalize it. Currently you ship GPLv3 and LGPLv3 in seabios.git but
many source files don't have their own copyright notes that declare
which should be effective[1]. Also there is no file that documents what
Signed-off-by should actually mean.

Just my 0.02€
Uwe

[1] I only checked a few files (vgasrc/ramfb.c, vgasrc/svgamodes.c) by
hand and assumed the other files to be in a similar situation. A quick
glance using licensecheck from
https://metacpan.org/release/App-Licensecheck:

seabios$ git ls-files | xargs licensecheck | sed "s/.*: //" | sort | uniq -c
2008 is > 1013 at /usr/share/perl5/String/Copyright.pm line 194.
      1 BSD (2 clause)
     10 GPL
      4 GPL GENERATED FILE
     16 GPL (v2.0)
      1 GPL (v2) GENERATED FILE
    102 LGPL (v3)
      1 MIT/X11 (BSD like)
      3 *No copyright* GENERATED FILE
      1 *No copyright* GPL (v2.0)
      7 *No copyright* GPL (v2 or later) (with incorrect FSF address)
      3 *No copyright* LGPL (v3)
    101 *No copyright* UNKNOWN
      6 UNKNOWN

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