Read through vesa.s a bit more throughly and experimented a bit, VESA is also configurable now! Thanks for the hint Andy, I didn't see the vesa_minx/y at first.

Just updated the PR to current main. @phil-opp would it be possible to get some feedback on this and hopefully merge it?

Sorry for the delay and thanks a lot for tackling this!

Looks good overall. Some ideas for improvements:

• It looks like you're trying to get an exact resolution match on UEFI, but only use the supplied resolution as minimum bounds on BIOS. This seems a bit inconsistent. The easiest solution would be to set treat the numbers as minima for UEFI too (and change the config field names accordingly).
• The only reason for the duplicated config write is that you need different import paths for the Config struct, right? Perhaps it is possible to instead make the parsed_config module public and then access it directly from bin/uefi.rs (via use bootloader::binary::parsed_config::CONFIG) instead of including the config there too?

Alright, got both of them done! UEFI now correctly treats it as minima (+ rename to minimum_framebuffer_*) and your idea of accessing the config module worked perfectly. Updated to current main as well.

@phil-opp Any updates? This is ready for merge.

I pushed another commit, however GitHub is telling me that the CI workflow is awaiting approval ("First-time contributors need a maintainer to approve running workflows", it links to this article).
Could you approve it? @phil-opp

Thanks!

Published as v0.10.9