[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 02/47] acpi: add acpi_scope() term
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v2 02/47] acpi: add acpi_scope() term |
Date: |
Fri, 23 Jan 2015 11:36:50 +0100 |
On Fri, 23 Jan 2015 10:02:54 +0200
"Michael S. Tsirkin" <address@hidden> wrote:
> On Thu, Jan 22, 2015 at 02:49:46PM +0000, Igor Mammedov wrote:
> > Signed-off-by: Igor Mammedov <address@hidden>
> > ---
> > hw/acpi/acpi-build-utils.c | 18 ++++++++++++++++++
> > include/hw/acpi/acpi-build-utils.h | 4 ++++
> > 2 files changed, 22 insertions(+)
> >
> > diff --git a/hw/acpi/acpi-build-utils.c b/hw/acpi/acpi-build-utils.c
> > index 547ecaa..e13d748 100644
> > --- a/hw/acpi/acpi-build-utils.c
> > +++ b/hw/acpi/acpi-build-utils.c
> > @@ -306,3 +306,21 @@ void aml_append(AcpiAml *parent_ctx, AcpiAml child)
> > build_append_array(parent_ctx->buf, child.buf);
> > build_free_array(child.buf);
> > }
> > +
> > +static AcpiAml aml_allocate_internal(uint8_t op, AcpiBlockFlags flags)
> > +{
> > + AcpiAml var = { .op = op, .block_flags = flags };
> > + var.buf = build_alloc_array();
> > + return var;
> > +}
> > +
>
> Is there also allocate_external? If not - static is enough to
> know it's not an API function.
There isn't, I'll fix it up.
>
> > +/* ACPI 5.0: 20.2.5.1 Namespace Modifier Objects Encoding: DefScope */
> > +AcpiAml GCC_FMT_ATTR(1, 2) acpi_scope(const char *name_format, ...)
> > +{
> > + va_list ap;
> > + AcpiAml var = aml_allocate_internal(0x10 /* ScopeOp */, PACKAGE);
> > + va_start(ap, name_format);
> > + build_append_namestringv(var.buf, name_format, ap);
> > + va_end(ap);
> > + return var;
> > +}
> > diff --git a/include/hw/acpi/acpi-build-utils.h
> > b/include/hw/acpi/acpi-build-utils.h
> > index 64e7ec3..bbb786b 100644
> > --- a/include/hw/acpi/acpi-build-utils.h
> > +++ b/include/hw/acpi/acpi-build-utils.h
> > @@ -21,6 +21,10 @@ typedef struct AcpiAml {
> >
> > void aml_append(AcpiAml *parent_ctx, AcpiAml child);
> >
> > +/* Block ASL object primitives */
> > +AcpiAml GCC_FMT_ATTR(1, 2) acpi_scope(const char *name_format, ...);
> > +
> > +/* other helpers */
> > GArray *build_alloc_array(void);
> > void build_free_array(GArray *array);
> > void build_prepend_byte(GArray *array, uint8_t val);
> > --
> > 1.8.3.1
>
- [Qemu-devel] [PATCH v2 00/47] ACPI refactoring: replace template patching with C ASL API, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 04/47] acpi: add acpi_method() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 02/47] acpi: add acpi_scope() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 03/47] acpi: add acpi_device() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 05/47] acpi: add acpi_if() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 07/47] acpi: factor out ACPI const int packing out build_append_value(), Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 12/47] acpi: add acpi_store() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 15/47] acpi: add acpi_call1(), acpi_call2(), acpi_call3(), acpi_call4() helpers, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 13/47] acpi: add acpi_and() term, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 08/47] acpi: extend build_append_{value|int}() to support 64-bit values, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 18/47] pc: acpi-build: drop unsupported PM1b_CNT.SLP_TYP, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 16/47] pc: acpi-build: drop template patching and create PCI bus tree dynamically, Igor Mammedov, 2015/01/22
- [Qemu-devel] [PATCH v2 14/47] acpi: add acpi_notify() term, Igor Mammedov, 2015/01/22