On Tue, 22 Nov 2022 20:05:38 +0100 Volker Rümelin vr_qemu@t-online.de wrote:
Since QEMU commit dbce582a119 (acpi: pc/q35: drop ad-hoc PCI-ISA bridge AML routines and let bus ennumeration generate AML)
The commit dbce582a119 doesn't exist in the QEMU repository. It's commit 47a373faa6.
Right, I must have grabbed id from another branch
Gerd, Can you fix up commit message when applying or should I respin patch?
With best regards, Volker
SeaBIOS fails to parse ISA bridge AML with:
parse_termlist: parse error, skip from 92/517 ... ACPI: no PS/2 keyboard presentdue to Alias term in DSDT which isn't handled by SeaBIOS properly. Add dumb Alias parsing which just skips over term, so the rest of AML could be parsed successfully.
Signed-off-by: Igor Mammedovimammedo@redhat.com Reported-by: Volker Rümelinvr_qemu@t-online.de
src/fw/dsdt_parser.c | 4 ++++ 1 file changed, 4 insertions(+)
diff --git a/src/fw/dsdt_parser.c b/src/fw/dsdt_parser.c index eb5496f3..2ac82821 100644 --- a/src/fw/dsdt_parser.c +++ b/src/fw/dsdt_parser.c @@ -417,6 +417,10 @@ static int parse_termobj(struct parse_state *s, break; case 0x01: /* one */ break;
- case 0x06: /* AliasOp */
offset += parse_namestring(s, ptr + offset, "SourceObject");offset += parse_namestring(s, ptr + offset, "AliasObject");break; case 0x08: /* name op */ offset += parse_namestring(s, ptr + offset, "name"); offset += parse_termobj(s, ptr + offset);