atiext: Convert tables to C and fix for radeon

[SeaBIOS] [PATCH v2 0/2] atiext: Convert tables to C and fix for radeon

Posted by Chad Jablonski 2 weeks, 3 days ago

Full testing requires the QEMU ati-vga CP engine patch series under
review on qemu-devel:
https://lore.kernel.org/qemu-devel/20260719161536.774061-1-chad@jablonski.xyz/

Changes from v1:
- Convert tables to C (Kevin O'Connor)
- Use actual header size for size field via sizeof

Chad Jablonski (1):
  atiext: Fix bios tables for radeon KMS driver

Kevin O'Connor (1):
  atiext: Implement ATI tables in atiext.c

 vgasrc/ati-tables.S | 43 ------------------------------
 vgasrc/atiext.c     | 65 +++++++++++++++++++++++++++++++++++++++++++++
 vgasrc/vgaentry.S   |  6 ++++-
 3 files changed, 70 insertions(+), 44 deletions(-)
 delete mode 100644 vgasrc/ati-tables.S

-- 
2.54.0

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

[SeaBIOS] Re: [PATCH v2 0/2] atiext: Convert tables to C and fix for radeon

Posted by Kevin O'Connor via SeaBIOS 2 weeks, 3 days ago

On Tue, Jul 28, 2026 at 10:54:34AM -0400, Chad Jablonski wrote:
> Full testing requires the QEMU ati-vga CP engine patch series under
> review on qemu-devel:
> https://lore.kernel.org/qemu-devel/20260719161536.774061-1-chad@jablonski.xyz/

Thanks.  It seems fine to me.

Quick question - can you confirm that you've tested this patch series
on the current QEMU ati-vga?  (That is, by "full testing" you are
referring to testing of the new features.)

-Kevin


> 
> Changes from v1:
> - Convert tables to C (Kevin O'Connor)
> - Use actual header size for size field via sizeof
> 
> Chad Jablonski (1):
>   atiext: Fix bios tables for radeon KMS driver
> 
> Kevin O'Connor (1):
>   atiext: Implement ATI tables in atiext.c
> 
>  vgasrc/ati-tables.S | 43 ------------------------------
>  vgasrc/atiext.c     | 65 +++++++++++++++++++++++++++++++++++++++++++++
>  vgasrc/vgaentry.S   |  6 ++++-
>  3 files changed, 70 insertions(+), 44 deletions(-)
>  delete mode 100644 vgasrc/ati-tables.S
> 
> --
> 2.54.0
> 

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

[SeaBIOS] Re: [PATCH v2 0/2] atiext: Convert tables to C and fix for radeon

Posted by Chad Jablonski 2 weeks, 3 days ago

>
> Quick question - can you confirm that you've tested this patch series
> on the current QEMU ati-vga?  (That is, by "full testing" you are
> referring to testing of the new features.)
>

You know what. I just tested it on QEMU v11.1.0-rc1 and KMS works
without any of the other mentioned patches. I think I made an assumption
somewhere along the way that the other patches were required. But they're not!
They enable Xorg with acceleration but that's not really important here. I'll
remove mention of the other series in v3. I think that's a much simpler story.
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org