nb/intel/haswell: Simplify GMA SSDT generator
Simplify generation of GMA SSDT, using updated naming convention. If acpi_fill_ssdt is being invoked, then we know the IGD device is present and enabled, so we can skip those checks. And the SSDT generator now checks that the gfx struct is populated, so we can skip that too. Change-Id: Icd9caf622dd4c46b13589ebb772138b25888752f Signed-off-by: Matt DeVillier <matt.devillier@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/39948 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
parent
ae51bb4ba9
commit
41c4eb5fa6
|
@ -524,24 +524,11 @@ static void gma_func0_init(struct device *dev)
|
||||||
intel_gma_restore_opregion();
|
intel_gma_restore_opregion();
|
||||||
}
|
}
|
||||||
|
|
||||||
const struct i915_gpu_controller_info *intel_gma_get_controller_info(void)
|
static void gma_generate_ssdt(struct device *dev)
|
||||||
{
|
{
|
||||||
struct device *dev = pcidev_on_root(2, 0);
|
const struct northbridge_intel_haswell_config *chip = dev->chip_info;
|
||||||
if (!dev) {
|
|
||||||
return NULL;
|
|
||||||
}
|
|
||||||
struct northbridge_intel_haswell_config *chip = dev->chip_info;
|
|
||||||
return &chip->gfx;
|
|
||||||
}
|
|
||||||
|
|
||||||
static void gma_ssdt(struct device *device)
|
drivers_intel_gma_displays_ssdt_generate(&chip->gfx);
|
||||||
{
|
|
||||||
const struct i915_gpu_controller_info *gfx = intel_gma_get_controller_info();
|
|
||||||
if (!gfx) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
drivers_intel_gma_displays_ssdt_generate(gfx);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static unsigned long gma_write_acpi_tables(struct device *const dev, unsigned long current,
|
static unsigned long gma_write_acpi_tables(struct device *const dev, unsigned long current,
|
||||||
|
@ -573,15 +560,15 @@ static struct pci_operations gma_pci_ops = {
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct device_operations gma_func0_ops = {
|
static struct device_operations gma_func0_ops = {
|
||||||
.read_resources = pci_dev_read_resources,
|
.read_resources = pci_dev_read_resources,
|
||||||
.set_resources = pci_dev_set_resources,
|
.set_resources = pci_dev_set_resources,
|
||||||
.enable_resources = pci_dev_enable_resources,
|
.enable_resources = pci_dev_enable_resources,
|
||||||
.init = gma_func0_init,
|
.init = gma_func0_init,
|
||||||
.acpi_fill_ssdt = gma_ssdt,
|
.acpi_fill_ssdt = gma_generate_ssdt,
|
||||||
.scan_bus = NULL,
|
.scan_bus = NULL,
|
||||||
.enable = NULL,
|
.enable = NULL,
|
||||||
.ops_pci = &gma_pci_ops,
|
.ops_pci = &gma_pci_ops,
|
||||||
.write_acpi_tables = gma_write_acpi_tables,
|
.write_acpi_tables = gma_write_acpi_tables,
|
||||||
};
|
};
|
||||||
|
|
||||||
static const unsigned short pci_device_ids[] = {
|
static const unsigned short pci_device_ids[] = {
|
||||||
|
|
Loading…
Reference in New Issue