Thanks. We can certainly add support for "mxm" to seabios. It seems odd to me that this patch modifies the "generic" optionrom code. Is there a reason the code is checking for "mxm-30-sis" from optionsroms.c instead of from the existing vgahook_setup() code in vgahooks.c?
It seemed (to me) like the most logical place to put it, since it's right after a few other "config settings that impact VGA".
Similarly, is there a reason patch 1 modifies the generic src/romfile.c code to add a new romfile_loadfile_low() instead of having mxm_setup() call romfile_loadfile(), malloc_low(), and memcpy()?
Seems to me like adding extra steps for no reason as far as I know. In v1, I had a slightly different solution, and Gerd Hoffmann suggested adding a loadfile_romfile_low() instead.