[coreboot-gerrit] Patch set updated for coreboot: 2c1139d device_ops: add device_t argument to write_acpi_tables
Alexander Couzens (lynxis@fe80.eu)
gerrit at coreboot.org
Mon Apr 13 01:17:31 CEST 2015
Alexander Couzens (lynxis at fe80.eu) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9599
-gerrit
commit 2c1139d5bd2bdf42059abef9b21400b2eeaecbc9
Author: Alexander Couzens <lynxis at fe80.eu>
Date: Sun Apr 12 22:28:37 2015 +0200
device_ops: add device_t argument to write_acpi_tables
`device_t device` is missing as argument. Every device_op function
should have a `device_t device` argument.
Change-Id: I1ba4bfa0ac36a09a82b108249158c80c50f9f5fd
Signed-off-by: Alexander Couzens <lynxis at fe80.eu>
---
src/arch/x86/boot/acpi.c | 4 ++--
src/arch/x86/include/arch/acpi.h | 2 +-
src/include/device/device.h | 2 +-
src/mainboard/amd/serengeti_cheetah/acpi_tables.c | 2 +-
src/mainboard/amd/serengeti_cheetah/mainboard.h | 2 +-
src/mainboard/amd/serengeti_cheetah_fam10/acpi_tables.c | 3 ++-
src/mainboard/amd/serengeti_cheetah_fam10/mainboard.h | 2 +-
src/mainboard/emulation/qemu-i440fx/acpi.h | 2 +-
src/mainboard/getac/p470/acpi_tables.c | 4 +++-
src/mainboard/getac/p470/mainboard.h | 3 ++-
src/mainboard/iwill/dk8_htx/acpi_tables.c | 4 +++-
src/mainboard/iwill/dk8_htx/mainboard.h | 2 +-
src/northbridge/amd/agesa/family12/northbridge.c | 3 ++-
src/northbridge/amd/agesa/family14/northbridge.c | 3 ++-
src/northbridge/amd/agesa/family15/northbridge.c | 3 ++-
src/northbridge/amd/agesa/family15rl/northbridge.c | 3 ++-
src/northbridge/amd/agesa/family15tn/northbridge.c | 3 ++-
src/northbridge/amd/agesa/family16kb/northbridge.c | 3 ++-
src/northbridge/amd/amdfam10/acpi.c | 5 +++--
src/northbridge/amd/amdfam10/amdfam10.h | 3 ++-
src/northbridge/amd/amdk8/northbridge.c | 4 +++-
src/northbridge/amd/pi/00630F01/northbridge.c | 3 ++-
src/northbridge/amd/pi/00730F01/northbridge.c | 3 ++-
src/northbridge/intel/gm45/acpi.c | 4 +++-
src/northbridge/intel/gm45/gm45.h | 10 +++++++++-
src/southbridge/intel/lynxpoint/lpc.c | 4 +++-
26 files changed, 58 insertions(+), 28 deletions(-)
diff --git a/src/arch/x86/boot/acpi.c b/src/arch/x86/boot/acpi.c
index f7f3e6a..eadb24b 100644
--- a/src/arch/x86/boot/acpi.c
+++ b/src/arch/x86/boot/acpi.c
@@ -466,7 +466,7 @@ void acpi_create_hpet(acpi_hpet_t *hpet)
header->checksum = acpi_checksum((void *)hpet, sizeof(acpi_hpet_t));
}
-unsigned long acpi_write_hpet(unsigned long current, acpi_rsdp_t *rsdp)
+unsigned long acpi_write_hpet(device_t device, unsigned long current, acpi_rsdp_t *rsdp)
{
acpi_hpet_t *hpet;
@@ -820,7 +820,7 @@ unsigned long write_acpi_tables(unsigned long start)
for (dev = all_devices; dev; dev = dev->next) {
if (dev->ops && dev->ops->write_acpi_tables) {
- current = dev->ops->write_acpi_tables(current, rsdp);
+ current = dev->ops->write_acpi_tables(dev, current, rsdp);
ALIGN_CURRENT;
}
}
diff --git a/src/arch/x86/include/arch/acpi.h b/src/arch/x86/include/arch/acpi.h
index 25d1bca..226f20a 100644
--- a/src/arch/x86/include/arch/acpi.h
+++ b/src/arch/x86/include/arch/acpi.h
@@ -530,7 +530,7 @@ void acpi_create_slit(acpi_slit_t *slit);
#if ENV_RAMSTAGE
void acpi_create_hpet(acpi_hpet_t *hpet);
-unsigned long acpi_write_hpet(unsigned long start, acpi_rsdp_t *rsdp);
+unsigned long acpi_write_hpet(device_t device, unsigned long start, acpi_rsdp_t *rsdp);
/* cpu/intel/speedstep/acpi.c */
void generate_cpu_entries(device_t device);
diff --git a/src/include/device/device.h b/src/include/device/device.h
index ec538d5..1c43389 100644
--- a/src/include/device/device.h
+++ b/src/include/device/device.h
@@ -56,7 +56,7 @@ struct device_operations {
void (*get_smbios_strings)(device_t dev, struct smbios_type11 *t);
#endif
#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES) && IS_ENABLED(CONFIG_PER_DEVICE_ACPI_TABLES)
- unsigned long (*write_acpi_tables)(unsigned long start, struct acpi_rsdp *rsdp);
+ unsigned long (*write_acpi_tables)(device_t dev, unsigned long start, struct acpi_rsdp *rsdp);
void (*acpi_fill_ssdt_generator)(device_t dev);
void (*acpi_inject_dsdt_generator)(device_t dev);
#endif
diff --git a/src/mainboard/amd/serengeti_cheetah/acpi_tables.c b/src/mainboard/amd/serengeti_cheetah/acpi_tables.c
index 168793e..e0fe77d 100644
--- a/src/mainboard/amd/serengeti_cheetah/acpi_tables.c
+++ b/src/mainboard/amd/serengeti_cheetah/acpi_tables.c
@@ -122,7 +122,7 @@ unsigned long acpi_fill_madt(unsigned long current)
return current;
}
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp)
+unsigned long mainboard_write_acpi_tables(device_t dev, unsigned long start, acpi_rsdp_t *rsdp)
{
unsigned long current;
acpi_header_t *ssdtx;
diff --git a/src/mainboard/amd/serengeti_cheetah/mainboard.h b/src/mainboard/amd/serengeti_cheetah/mainboard.h
index 80d8f2a..fc2dcaf 100644
--- a/src/mainboard/amd/serengeti_cheetah/mainboard.h
+++ b/src/mainboard/amd/serengeti_cheetah/mainboard.h
@@ -1 +1 @@
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp);
+unsigned long mainboard_write_acpi_tables(device_t device, unsigned long start, acpi_rsdp_t *rsdp);
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/acpi_tables.c b/src/mainboard/amd/serengeti_cheetah_fam10/acpi_tables.c
index 51265bc..f17aa05 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/acpi_tables.c
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/acpi_tables.c
@@ -131,7 +131,8 @@ unsigned long acpi_fill_madt(unsigned long current)
return current;
}
-unsigned long mainboard_write_acpi_tables(unsigned long current,
+unsigned long mainboard_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_header_t *ssdtx;
diff --git a/src/mainboard/amd/serengeti_cheetah_fam10/mainboard.h b/src/mainboard/amd/serengeti_cheetah_fam10/mainboard.h
index 8f17804..b152b04 100644
--- a/src/mainboard/amd/serengeti_cheetah_fam10/mainboard.h
+++ b/src/mainboard/amd/serengeti_cheetah_fam10/mainboard.h
@@ -1,2 +1,2 @@
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp);
+unsigned long mainboard_write_acpi_tables(device_t device, unsigned long start, acpi_rsdp_t *rsdp);
diff --git a/src/mainboard/emulation/qemu-i440fx/acpi.h b/src/mainboard/emulation/qemu-i440fx/acpi.h
index c0d4e6e..f2ffdeb 100644
--- a/src/mainboard/emulation/qemu-i440fx/acpi.h
+++ b/src/mainboard/emulation/qemu-i440fx/acpi.h
@@ -1 +1 @@
-unsigned long northbridge_write_acpi_tables(unsigned long start);
+unsigned long northbridge_write_acpi_tables(device_t device, unsigned long start);
diff --git a/src/mainboard/getac/p470/acpi_tables.c b/src/mainboard/getac/p470/acpi_tables.c
index d421101..55345c9 100644
--- a/src/mainboard/getac/p470/acpi_tables.c
+++ b/src/mainboard/getac/p470/acpi_tables.c
@@ -131,7 +131,9 @@ unsigned long acpi_fill_srat(unsigned long current)
}
#define ALIGN_CURRENT current = (ALIGN(current, 16))
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp)
+unsigned long mainboard_write_acpi_tables(device_t device,
+ unsigned long start,
+ acpi_rsdp_t *rsdp)
{
unsigned long current;
acpi_header_t *ecdt;
diff --git a/src/mainboard/getac/p470/mainboard.h b/src/mainboard/getac/p470/mainboard.h
index 45d961b..56f5455 100644
--- a/src/mainboard/getac/p470/mainboard.h
+++ b/src/mainboard/getac/p470/mainboard.h
@@ -1,4 +1,5 @@
struct acpi_rsdp;
-unsigned long mainboard_write_acpi_tables(unsigned long start,
+unsigned long mainboard_write_acpi_tables(device_t device,
+ unsigned long start,
struct acpi_rsdp *rsdp);
diff --git a/src/mainboard/iwill/dk8_htx/acpi_tables.c b/src/mainboard/iwill/dk8_htx/acpi_tables.c
index e1c5c80..e724557 100644
--- a/src/mainboard/iwill/dk8_htx/acpi_tables.c
+++ b/src/mainboard/iwill/dk8_htx/acpi_tables.c
@@ -129,7 +129,9 @@ unsigned long acpi_fill_madt(unsigned long current)
return current;
}
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp)
+unsigned long mainboard_write_acpi_tables(device_t device,
+ unsigned long start,
+ acpi_rsdp_t *rsdp)
{
unsigned long current;
acpi_header_t *ssdtx;
diff --git a/src/mainboard/iwill/dk8_htx/mainboard.h b/src/mainboard/iwill/dk8_htx/mainboard.h
index 80d8f2a..fc2dcaf 100644
--- a/src/mainboard/iwill/dk8_htx/mainboard.h
+++ b/src/mainboard/iwill/dk8_htx/mainboard.h
@@ -1 +1 @@
-unsigned long mainboard_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp);
+unsigned long mainboard_write_acpi_tables(device_t device, unsigned long start, acpi_rsdp_t *rsdp);
diff --git a/src/northbridge/amd/agesa/family12/northbridge.c b/src/northbridge/amd/agesa/family12/northbridge.c
index da208da..c4d549c 100644
--- a/src/northbridge/amd/agesa/family12/northbridge.c
+++ b/src/northbridge/amd/agesa/family12/northbridge.c
@@ -844,7 +844,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/agesa/family14/northbridge.c b/src/northbridge/amd/agesa/family14/northbridge.c
index 86372ea..8cdf044 100644
--- a/src/northbridge/amd/agesa/family14/northbridge.c
+++ b/src/northbridge/amd/agesa/family14/northbridge.c
@@ -833,7 +833,8 @@ unsigned long acpi_fill_hest(acpi_hest_t *hest)
return (unsigned long)current;
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/agesa/family15/northbridge.c b/src/northbridge/amd/agesa/family15/northbridge.c
index 7263d62..32909c9 100644
--- a/src/northbridge/amd/agesa/family15/northbridge.c
+++ b/src/northbridge/amd/agesa/family15/northbridge.c
@@ -534,7 +534,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/agesa/family15rl/northbridge.c b/src/northbridge/amd/agesa/family15rl/northbridge.c
index d5406ef..3417c65 100644
--- a/src/northbridge/amd/agesa/family15rl/northbridge.c
+++ b/src/northbridge/amd/agesa/family15rl/northbridge.c
@@ -510,7 +510,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/agesa/family15tn/northbridge.c b/src/northbridge/amd/agesa/family15tn/northbridge.c
index 3f1caaa..d417824 100644
--- a/src/northbridge/amd/agesa/family15tn/northbridge.c
+++ b/src/northbridge/amd/agesa/family15tn/northbridge.c
@@ -509,7 +509,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/agesa/family16kb/northbridge.c b/src/northbridge/amd/agesa/family16kb/northbridge.c
index 958a037..89549d3 100644
--- a/src/northbridge/amd/agesa/family16kb/northbridge.c
+++ b/src/northbridge/amd/agesa/family16kb/northbridge.c
@@ -510,7 +510,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/amdfam10/acpi.c b/src/northbridge/amd/amdfam10/acpi.c
index 63ec306..3afbf72 100644
--- a/src/northbridge/amd/amdfam10/acpi.c
+++ b/src/northbridge/amd/amdfam10/acpi.c
@@ -330,8 +330,9 @@ void northbridge_acpi_write_vars(device_t device)
acpigen_pop_len();
}
-unsigned long northbridge_write_acpi_tables(unsigned long current,
- struct acpi_rsdp *rsdp)
+unsigned long northbridge_write_acpi_tables(device_t device,
+ unsigned long current,
+ struct acpi_rsdp *rsdp)
{
acpi_srat_t *srat;
acpi_slit_t *slit;
diff --git a/src/northbridge/amd/amdfam10/amdfam10.h b/src/northbridge/amd/amdfam10/amdfam10.h
index 4518165..f5d5ff9 100644
--- a/src/northbridge/amd/amdfam10/amdfam10.h
+++ b/src/northbridge/amd/amdfam10/amdfam10.h
@@ -1074,7 +1074,8 @@ BOOL AMD_CB_ManualBUIDSwapList(u8 Node, u8 Link, const u8 **List);
struct acpi_rsdp;
-unsigned long northbridge_write_acpi_tables(unsigned long start,
+unsigned long northbridge_write_acpi_tables(device_t device,
+ unsigned long start,
struct acpi_rsdp *rsdp);
void northbridge_acpi_write_vars(device_t device);
diff --git a/src/northbridge/amd/amdk8/northbridge.c b/src/northbridge/amd/amdk8/northbridge.c
index a17aa65..9f8b296 100644
--- a/src/northbridge/amd/amdk8/northbridge.c
+++ b/src/northbridge/amd/amdk8/northbridge.c
@@ -580,7 +580,9 @@ static void mcf0_control_init(struct device *dev)
}
#if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-static unsigned long northbridge_write_acpi_tables(unsigned long start, acpi_rsdp_t *rsdp)
+static unsigned long northbridge_write_acpi_tables(device_t device,
+ unsigned long start,
+ acpi_rsdp_t *rsdp)
{
unsigned long current;
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/pi/00630F01/northbridge.c b/src/northbridge/amd/pi/00630F01/northbridge.c
index f58622e..3fa1c40 100644
--- a/src/northbridge/amd/pi/00630F01/northbridge.c
+++ b/src/northbridge/amd/pi/00630F01/northbridge.c
@@ -504,7 +504,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/amd/pi/00730F01/northbridge.c b/src/northbridge/amd/pi/00730F01/northbridge.c
index 7c33ee3..a23b6ed 100644
--- a/src/northbridge/amd/pi/00730F01/northbridge.c
+++ b/src/northbridge/amd/pi/00730F01/northbridge.c
@@ -498,7 +498,8 @@ static void northbridge_fill_ssdt_generator(device_t device)
acpigen_pop_len();
}
-static unsigned long agesa_write_acpi_tables(unsigned long current,
+static unsigned long agesa_write_acpi_tables(device_t device,
+ unsigned long current,
acpi_rsdp_t *rsdp)
{
acpi_srat_t *srat;
diff --git a/src/northbridge/intel/gm45/acpi.c b/src/northbridge/intel/gm45/acpi.c
index 58498ed..2dbd56d 100644
--- a/src/northbridge/intel/gm45/acpi.c
+++ b/src/northbridge/intel/gm45/acpi.c
@@ -110,7 +110,9 @@ static unsigned long acpi_fill_dmar(unsigned long current)
}
#define ALIGN_CURRENT current = (ALIGN(current, 16))
-unsigned long northbridge_write_acpi_tables(unsigned long start, struct acpi_rsdp *rsdp)
+unsigned long northbridge_write_acpi_tables(device_t device,
+ unsigned long start,
+ struct acpi_rsdp *rsdp)
{
unsigned long current;
#if CONFIG_IOMMU
diff --git a/src/northbridge/intel/gm45/gm45.h b/src/northbridge/intel/gm45/gm45.h
index a31ea7d..6ea8cde 100644
--- a/src/northbridge/intel/gm45/gm45.h
+++ b/src/northbridge/intel/gm45/gm45.h
@@ -21,10 +21,12 @@
#ifndef __NORTHBRIDGE_INTEL_GM45_GM45_H__
#define __NORTHBRIDGE_INTEL_GM45_GM45_H__
+
#include "southbridge/intel/i82801ix/i82801ix.h"
#ifndef __ACPI__
+#include <rules.h>
#include <stdint.h>
typedef enum {
@@ -435,8 +437,14 @@ u32 decode_igd_gtt_size(u32 gsm);
void init_iommu(void);
+#if ENV_RAMSTAGE
+
+#include <device/device.h>
+
struct acpi_rsdp;
-unsigned long northbridge_write_acpi_tables(unsigned long start, struct acpi_rsdp *rsdp);
+unsigned long northbridge_write_acpi_tables(device_t device, unsigned long start, struct acpi_rsdp *rsdp);
#endif
+
+#endif /* !__ACPI__ */
#endif /* __NORTHBRIDGE_INTEL_GM45_GM45_H__ */
diff --git a/src/southbridge/intel/lynxpoint/lpc.c b/src/southbridge/intel/lynxpoint/lpc.c
index c033022..bfa874e 100644
--- a/src/southbridge/intel/lynxpoint/lpc.c
+++ b/src/southbridge/intel/lynxpoint/lpc.c
@@ -780,7 +780,9 @@ static void southbridge_inject_dsdt(device_t dev)
}
#define ALIGN_CURRENT current = (ALIGN(current, 16))
-static unsigned long southbridge_write_acpi_tables(unsigned long start, struct acpi_rsdp *rsdp)
+static unsigned long southbridge_write_acpi_tables(device_t device,
+ unsigned long start,
+ struct acpi_rsdp *rsdp)
{
unsigned long current;
acpi_hpet_t *hpet;
More information about the coreboot-gerrit
mailing list