diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2023-11-21 18:49:33 +0100 |
|---|---|---|
| committer | Lean Sheng Tan <sheng.tan@9elements.com> | 2023-11-23 14:42:59 +0000 |
| commit | 62f788e24469c0e12dbf427594cc8150ffa4336f (patch) | |
| tree | 623db985f59fa7cbff969a5be75f7f56ce58c57e /src/acpi/acpigen.c | |
| parent | 39b19f270c717dbf4b3c947341cb3f2f239eabe6 (diff) | |
acpigen.c: Add resource consumer functions for mmio
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Change-Id: Id9e4adcd976e1f56ef7f502d9df16dbefce95c3e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79217
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Tim Wawrzynczak <inforichland@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/acpi/acpigen.c')
| -rw-r--r-- | src/acpi/acpigen.c | 62 |
1 files changed, 37 insertions, 25 deletions
diff --git a/src/acpi/acpigen.c b/src/acpi/acpigen.c index fe36113fd7..ac95026562 100644 --- a/src/acpi/acpigen.c +++ b/src/acpi/acpigen.c @@ -2296,42 +2296,54 @@ void acpigen_resource_producer_io(u16 io_base, u16 io_limit) io_limit - io_base + 1); /* length */ } -static void acpigen_resource_producer_mmio32(u32 mmio_base, u32 mmio_limit, u16 type_flags) +static void acpigen_resource_mmio32(u32 mmio_base, u32 mmio_limit, u16 gen_flags, + u16 type_flags) { acpigen_resource_dword(RSRC_TYPE_MEM, /* res_type */ - ADDR_SPACE_GENERAL_FLAG_MAX_FIXED - | ADDR_SPACE_GENERAL_FLAG_MIN_FIXED - | ADDR_SPACE_GENERAL_FLAG_DEC_POS - | ADDR_SPACE_GENERAL_FLAG_PRODUCER, /* gen_flags */ - type_flags, /* type_flags */ - 0, /* gran */ - mmio_base, /* range_min */ - mmio_limit, /* range_max */ - 0x0, /* translation */ - mmio_limit - mmio_base + 1); /* length */ + gen_flags, /* gen_flags */ + type_flags, /* type_flags */ + 0, /* gran */ + mmio_base, /* range_min */ + mmio_limit, /* range_max */ + 0x0, /* translation */ + mmio_limit - mmio_base + 1); /* length */ } -static void acpigen_resource_producer_mmio64(u64 mmio_base, u64 mmio_limit, u16 type_flags) +static void acpigen_resource_mmio64(u64 mmio_base, u64 mmio_limit, u16 gen_flags, + u16 type_flags) { acpigen_resource_qword(RSRC_TYPE_MEM, /* res_type */ - ADDR_SPACE_GENERAL_FLAG_MAX_FIXED - | ADDR_SPACE_GENERAL_FLAG_MIN_FIXED - | ADDR_SPACE_GENERAL_FLAG_DEC_POS - | ADDR_SPACE_GENERAL_FLAG_PRODUCER, /* gen_flags */ - type_flags, /* type_flags */ - 0, /* gran */ - mmio_base, /* range_min */ - mmio_limit, /* range_max */ - 0x0, /* translation */ - mmio_limit - mmio_base + 1); /* length */ + gen_flags, /* gen_flags */ + type_flags, /* type_flags */ + 0, /* gran */ + mmio_base, /* range_min */ + mmio_limit, /* range_max */ + 0x0, /* translation */ + mmio_limit - mmio_base + 1); /* length */ } -void acpigen_resource_producer_mmio(u64 mmio_base, u64 mmio_limit, u16 type_flags) +static void acpigen_resource_mmio(u64 mmio_base, u64 mmio_limit, bool is_producer, u16 type_flags) { + const u16 gen_flags = ADDR_SPACE_GENERAL_FLAG_MAX_FIXED + | ADDR_SPACE_GENERAL_FLAG_MIN_FIXED + | ADDR_SPACE_GENERAL_FLAG_DEC_POS + | (is_producer ? ADDR_SPACE_GENERAL_FLAG_PRODUCER + : ADDR_SPACE_GENERAL_FLAG_CONSUMER); + if (mmio_base < 4ULL * GiB && mmio_limit < 4ULL * GiB) - acpigen_resource_producer_mmio32(mmio_base, mmio_limit, type_flags); + acpigen_resource_mmio32(mmio_base, mmio_limit, gen_flags, type_flags); else - acpigen_resource_producer_mmio64(mmio_base, mmio_limit, type_flags); + acpigen_resource_mmio64(mmio_base, mmio_limit, gen_flags, type_flags); +} + +void acpigen_resource_producer_mmio(u64 mmio_base, u64 mmio_limit, u16 type_flags) +{ + acpigen_resource_mmio(mmio_base, mmio_limit, true, type_flags); +} + +void acpigen_resource_consumer_mmio(u64 mmio_base, u64 mmio_limit, u16 type_flags) +{ + acpigen_resource_mmio(mmio_base, mmio_limit, false, type_flags); } void acpigen_write_ADR(uint64_t adr) |