[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH 4/9] acpi: add build_append_gas() hel
From: |
Auger Eric |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH 4/9] acpi: add build_append_gas() helper for Generic Address Structure |
Date: |
Tue, 27 Feb 2018 13:48:45 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
Hi,
On 22/02/18 13:42, Igor Mammedov wrote:
> it will help to add Generic Address Structure to ACPI tables
> without using packed C structures and avoid endianness
> issues as API doesn't need an explicit conversion.
>
> Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eric Auger <address@hidden>
Thanks
Eric
> ---
> include/hw/acpi/aml-build.h | 20 ++++++++++++++++++++
> hw/acpi/aml-build.c | 16 ++++++++++++++++
> 2 files changed, 36 insertions(+)
>
> diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h
> index 88d0738..8692ccc 100644
> --- a/include/hw/acpi/aml-build.h
> +++ b/include/hw/acpi/aml-build.h
> @@ -78,6 +78,15 @@ typedef enum {
> } AmlUpdateRule;
>
> typedef enum {
> + AML_AS_SYSTEM_MEMORY = 0X00,
> + AML_AS_SYSTEM_IO = 0X01,
> + AML_AS_PCI_CONFIG = 0X02,
> + AML_AS_EMBEDDED_CTRL = 0X03,
> + AML_AS_SMBUS = 0X04,
> + AML_AS_FFH = 0X7F,
> +} AmlAddressSpace;
> +
> +typedef enum {
> AML_SYSTEM_MEMORY = 0X00,
> AML_SYSTEM_IO = 0X01,
> AML_PCI_CONFIG = 0X02,
> @@ -389,6 +398,17 @@ int
> build_append_named_dword(GArray *array, const char *name_format, ...)
> GCC_FMT_ATTR(2, 3);
>
> +void build_append_gas(GArray *table, AmlAddressSpace as,
> + uint8_t bit_width, uint8_t bit_offset,
> + uint8_t access_width, uint64_t address);
> +
> +static inline void
> +build_append_gas_from_struct(GArray *table, const struct AcpiGenericAddress
> *s)
> +{
> + build_append_gas(table, s->space_id, s->bit_width, s->bit_offset,
> + s->access_width, s->address);
> +}
> +
> void build_srat_memory(AcpiSratMemoryAffinity *numamem, uint64_t base,
> uint64_t len, int node, MemoryAffinityFlags flags);
>
> diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c
> index 36a6cc4..3fef5f6 100644
> --- a/hw/acpi/aml-build.c
> +++ b/hw/acpi/aml-build.c
> @@ -258,6 +258,22 @@ static void build_append_int(GArray *table, uint64_t
> value)
> }
> }
>
> +/* Generic Address Structure (GAS)
> + * ACPI 2.0/3.0: 5.2.3.1 Generic Address Structure
> + * 2.0 compat note:
> + * @access_width must be 0, see ACPI 2.0:Table 5-1
> + */
> +void build_append_gas(GArray *table, AmlAddressSpace as,
> + uint8_t bit_width, uint8_t bit_offset,
> + uint8_t access_width, uint64_t address)
> +{
> + build_append_int_noprefix(table, as, 1);
> + build_append_int_noprefix(table, bit_width, 1);
> + build_append_int_noprefix(table, bit_offset, 1);
> + build_append_int_noprefix(table, access_width, 1);
> + build_append_int_noprefix(table, address, 8);
> +}
> +
> /*
> * Build NAME(XXXX, 0x00000000) where 0x00000000 is encoded as a dword,
> * and return the offset to 0x00000000 for runtime patching.
>
- [Qemu-arm] [PATCH 0/9] generalize build_fadt(), Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 1/9] acpi: remove unused acpi-dsdt.aml, Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 3/9] acpi: reuse AcpiGenericAddress instead of Acpi20GenericAddress, Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 2/9] pc: replace pm object initialization with one-liner in acpi_get_pm_info(), Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 4/9] acpi: add build_append_gas() helper for Generic Address Structure, Igor Mammedov, 2018/02/22
- Re: [Qemu-arm] [Qemu-devel] [PATCH 4/9] acpi: add build_append_gas() helper for Generic Address Structure,
Auger Eric <=
- [Qemu-arm] [PATCH 5/9] pc: acpi: isolate FADT specific data into AcpiFadtData structure, Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 6/9] pc: acpi: use build_append_foo() API to construct FADT, Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 7/9] acpi: move build_fadt() from i386 specific to generic ACPI source, Igor Mammedov, 2018/02/22
- [Qemu-arm] [PATCH 8/9] virt_arm: acpi: reuse common build_fadt(), Igor Mammedov, 2018/02/22