mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-09-06 18:05:37 +00:00
acpi: applesmc: use AcpiDevAmlIfClass:build_dev_aml to provide device's AML
.. and clean up not longer needed conditionals in DSTD build code. applesmc AML will be fetched and included when ISA bridge will build its own AML code (incl. attached devices). Expected AML change: the device under separate _SB.PCI0.ISA scope is moved directly under Device(ISA) node. Signed-off-by: Igor Mammedov <imammedo@redhat.com> Acked-by: Gerd Hoffmann <kraxel@redhat.com> Message-Id: <20220608135340.3304695-25-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
parent
cde55744e9
commit
1d7ea91500
|
@ -121,7 +121,6 @@ typedef struct AcpiMiscInfo {
|
|||
const unsigned char *dsdt_code;
|
||||
unsigned dsdt_size;
|
||||
uint16_t pvpanic_port;
|
||||
uint16_t applesmc_io_base;
|
||||
} AcpiMiscInfo;
|
||||
|
||||
typedef struct AcpiBuildPciBusHotplugState {
|
||||
|
@ -307,7 +306,6 @@ static void acpi_get_misc_info(AcpiMiscInfo *info)
|
|||
info->tpm_version = tpm_get_version(tpm_find());
|
||||
#endif
|
||||
info->pvpanic_port = pvpanic_port();
|
||||
info->applesmc_io_base = applesmc_port();
|
||||
}
|
||||
|
||||
/*
|
||||
|
@ -1800,26 +1798,6 @@ build_dsdt(GArray *table_data, BIOSLinker *linker,
|
|||
aml_append(dsdt, scope);
|
||||
}
|
||||
|
||||
if (misc->applesmc_io_base) {
|
||||
scope = aml_scope("\\_SB.PCI0.ISA");
|
||||
dev = aml_device("SMC");
|
||||
|
||||
aml_append(dev, aml_name_decl("_HID", aml_eisaid("APP0001")));
|
||||
/* device present, functioning, decoding, not shown in UI */
|
||||
aml_append(dev, aml_name_decl("_STA", aml_int(0xB)));
|
||||
|
||||
crs = aml_resource_template();
|
||||
aml_append(crs,
|
||||
aml_io(AML_DECODE16, misc->applesmc_io_base, misc->applesmc_io_base,
|
||||
0x01, APPLESMC_MAX_DATA_LENGTH)
|
||||
);
|
||||
aml_append(crs, aml_irq_no_flags(6));
|
||||
aml_append(dev, aml_name_decl("_CRS", crs));
|
||||
|
||||
aml_append(scope, dev);
|
||||
aml_append(dsdt, scope);
|
||||
}
|
||||
|
||||
if (misc->pvpanic_port) {
|
||||
scope = aml_scope("\\_SB.PCI0.ISA");
|
||||
|
||||
|
|
|
@ -37,10 +37,14 @@
|
|||
#include "qemu/module.h"
|
||||
#include "qemu/timer.h"
|
||||
#include "qom/object.h"
|
||||
#include "hw/acpi/acpi_aml_interface.h"
|
||||
|
||||
/* #define DEBUG_SMC */
|
||||
|
||||
#define APPLESMC_DEFAULT_IOBASE 0x300
|
||||
#define TYPE_APPLE_SMC "isa-applesmc"
|
||||
#define APPLESMC_MAX_DATA_LENGTH 32
|
||||
#define APPLESMC_PROP_IO_BASE "iobase"
|
||||
|
||||
enum {
|
||||
APPLESMC_DATA_PORT = 0x00,
|
||||
|
@ -347,14 +351,35 @@ static Property applesmc_isa_properties[] = {
|
|||
DEFINE_PROP_END_OF_LIST(),
|
||||
};
|
||||
|
||||
static void build_applesmc_aml(AcpiDevAmlIf *adev, Aml *scope)
|
||||
{
|
||||
Aml *crs;
|
||||
AppleSMCState *s = APPLE_SMC(adev);
|
||||
uint32_t iobase = s->iobase;
|
||||
Aml *dev = aml_device("SMC");
|
||||
|
||||
aml_append(dev, aml_name_decl("_HID", aml_eisaid("APP0001")));
|
||||
/* device present, functioning, decoding, not shown in UI */
|
||||
aml_append(dev, aml_name_decl("_STA", aml_int(0xB)));
|
||||
crs = aml_resource_template();
|
||||
aml_append(crs,
|
||||
aml_io(AML_DECODE16, iobase, iobase, 0x01, APPLESMC_MAX_DATA_LENGTH)
|
||||
);
|
||||
aml_append(crs, aml_irq_no_flags(6));
|
||||
aml_append(dev, aml_name_decl("_CRS", crs));
|
||||
aml_append(scope, dev);
|
||||
}
|
||||
|
||||
static void qdev_applesmc_class_init(ObjectClass *klass, void *data)
|
||||
{
|
||||
DeviceClass *dc = DEVICE_CLASS(klass);
|
||||
AcpiDevAmlIfClass *adevc = ACPI_DEV_AML_IF_CLASS(klass);
|
||||
|
||||
dc->realize = applesmc_isa_realize;
|
||||
dc->reset = qdev_applesmc_isa_reset;
|
||||
device_class_set_props(dc, applesmc_isa_properties);
|
||||
set_bit(DEVICE_CATEGORY_MISC, dc->categories);
|
||||
adevc->build_dev_aml = build_applesmc_aml;
|
||||
}
|
||||
|
||||
static const TypeInfo applesmc_isa_info = {
|
||||
|
@ -362,6 +387,10 @@ static const TypeInfo applesmc_isa_info = {
|
|||
.parent = TYPE_ISA_DEVICE,
|
||||
.instance_size = sizeof(AppleSMCState),
|
||||
.class_init = qdev_applesmc_class_init,
|
||||
.interfaces = (InterfaceInfo[]) {
|
||||
{ TYPE_ACPI_DEV_AML_IF },
|
||||
{ },
|
||||
},
|
||||
};
|
||||
|
||||
static void applesmc_register_types(void)
|
||||
|
|
|
@ -16,20 +16,6 @@ OBJECT_DECLARE_TYPE(ISADevice, ISADeviceClass, ISA_DEVICE)
|
|||
#define TYPE_ISA_BUS "ISA"
|
||||
OBJECT_DECLARE_SIMPLE_TYPE(ISABus, ISA_BUS)
|
||||
|
||||
#define TYPE_APPLE_SMC "isa-applesmc"
|
||||
#define APPLESMC_MAX_DATA_LENGTH 32
|
||||
#define APPLESMC_PROP_IO_BASE "iobase"
|
||||
|
||||
static inline uint16_t applesmc_port(void)
|
||||
{
|
||||
Object *obj = object_resolve_path_type("", TYPE_APPLE_SMC, NULL);
|
||||
|
||||
if (obj) {
|
||||
return object_property_get_uint(obj, APPLESMC_PROP_IO_BASE, NULL);
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
#define TYPE_ISADMA "isa-dma"
|
||||
|
||||
typedef struct IsaDmaClass IsaDmaClass;
|
||||
|
|
Loading…
Reference in a new issue