diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2016-12-28 12:35:06 +0100 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-01-03 17:38:14 +0100 |
commit | 1eef32d92b1c091eebe267fbd814be59e381700e (patch) | |
tree | a9e435ecae067d7e18ca648c9a3cf3fbf1470d81 /src/southbridge/nvidia/mcp55/lpc.c | |
parent | c258bc1ac6de973246121a6f58e76d03c4b0a208 (diff) |
sb/nvidia/mcp55: Fix P_state generation
amd_generate_powernow is never called by in lpc_slave_ops.
Move it to lpc_ops like on all other AMD southbridges.
TESTED on Gigabyte ga-m57sli-s4
Change-Id: I7db036e681d591a19e15dd3eaafb88b72a41bea1
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/17977
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/southbridge/nvidia/mcp55/lpc.c')
-rw-r--r-- | src/southbridge/nvidia/mcp55/lpc.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/src/southbridge/nvidia/mcp55/lpc.c b/src/southbridge/nvidia/mcp55/lpc.c index afc6e647b8..eeb6c1b1d0 100644 --- a/src/southbridge/nvidia/mcp55/lpc.c +++ b/src/southbridge/nvidia/mcp55/lpc.c @@ -240,10 +240,21 @@ static void mcp55_lpc_enable_resources(device_t dev) mcp55_lpc_enable_childrens_resources(dev); } +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) +static void southbridge_acpi_fill_ssdt_generator(device_t device) +{ + amd_generate_powernow(0, 0, 0); +} +#endif + static struct device_operations lpc_ops = { .read_resources = mcp55_lpc_read_resources, .set_resources = pci_dev_set_resources, .enable_resources = mcp55_lpc_enable_resources, +#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) + .acpi_fill_ssdt_generator = southbridge_acpi_fill_ssdt_generator, + .write_acpi_tables = acpi_write_hpet, +#endif .init = lpc_init, .scan_bus = scan_lpc_bus, .ops_pci = &mcp55_pci_ops, @@ -264,21 +275,11 @@ static const struct pci_driver lpc_driver __pci_driver = { .devices = lpc_ids, }; -#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) - -static void southbridge_acpi_fill_ssdt_generator(device_t device) -{ - amd_generate_powernow(0, 0, 0); -} - -#endif - static struct device_operations lpc_slave_ops = { .read_resources = mcp55_lpc_read_resources, .set_resources = pci_dev_set_resources, .enable_resources = pci_dev_enable_resources, #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) - .acpi_fill_ssdt_generator = southbridge_acpi_fill_ssdt_generator, .write_acpi_tables = acpi_write_hpet, #endif .init = lpc_slave_init, |