GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

arch/x86: Replace some __SMM__ guards 

We generally do not guard source in attempts to reduce
the final object sizes, but rely on garbage collection.

Most of the __unused attributes inserted here will be
removed when remaining __SIMPLE_DEVICE__ guards can
be removed.

Change-Id: I2440931fab4f41d7e8249c082e6c9b5a9cd0ef13
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36641
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Kyösti Mälkki 2019-11-05 18:50:38 +02:00
parent be5317f6d0
commit 21d6a27ac0
14 changed files with 72 additions and 91 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/mainboard/google/stout/ec.c
View File

@ -17,6 +17,7 @@
#include <bootmode.h> #include <bootmode.h>
#include <types.h> #include <types.h>
#include <console/console.h> #include <console/console.h>
#include <cpu/x86/smm.h>
#include <ec/quanta/it8518/ec.h> #include <ec/quanta/it8518/ec.h>
#include <device/device.h> #include <device/device.h>
#include <device/pci.h> #include <device/pci.h>
@ -25,11 +26,6 @@
#include <elog.h> #include <elog.h>
#include "ec.h" #include "ec.h"
#ifdef __SMM__
#include <cpu/x86/smm.h>
#endif
#ifndef __SMM__
void stout_ec_init(void) void stout_ec_init(void)
{ {
@ -59,8 +55,6 @@ void stout_ec_init(void)
// TODO: Power Limit Setting // TODO: Power Limit Setting
} }
#else // SMM
void stout_ec_finalize_smm(void) void stout_ec_finalize_smm(void)
{ {
u8 ec_reg, critical_shutdown = 0; u8 ec_reg, critical_shutdown = 0;
@ -105,4 +99,3 @@ void stout_ec_finalize_smm(void)
write_pmbase32(PM1_CNT, read_pmbase32(PM1_CNT) | (0xf << 10)); write_pmbase32(PM1_CNT, read_pmbase32(PM1_CNT) | (0xf << 10));
} }
} }
#endif //__SMM__

7
src/soc/intel/baytrail/pmutil.c
View File

@ -16,7 +16,9 @@
#include <stdint.h> #include <stdint.h>
#include <arch/acpi.h> #include <arch/acpi.h>
#include <arch/io.h> #include <arch/io.h>
#include <device/device.h>
#include <device/mmio.h> #include <device/mmio.h>
#include <device/pci.h>
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include <cbmem.h> #include <cbmem.h>
#include <console/console.h> #include <console/console.h>
@ -37,10 +39,7 @@ static inline pci_devfn_t get_pcu_dev(void)
return pcu_dev; return pcu_dev;
} }
#else /* !__SMM__ */ #else
#include <device/device.h>
#include <device/pci.h>
static struct device *pcu_dev; static struct device *pcu_dev;
static struct device *get_pcu_dev(void) static struct device *get_pcu_dev(void)
{ {

8
src/soc/intel/braswell/pmutil.c
View File

@ -17,7 +17,9 @@
#include <arch/acpi.h> #include <arch/acpi.h>
#include <arch/io.h> #include <arch/io.h>
#include <assert.h> #include <assert.h>
#include <device/device.h>
#include <device/mmio.h> #include <device/mmio.h>
#include <device/pci.h>
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include <console/console.h> #include <console/console.h>
#include <soc/iomap.h> #include <soc/iomap.h>
@ -37,9 +39,7 @@ static inline pci_devfn_t get_pcu_dev(void)
return pcu_dev; return pcu_dev;
} }
#else /* ENV_SMM */ #else /* __SIMPLE_DEVICE__ */
#include <device/device.h>
#include <device/pci.h>
static struct device *pcu_dev; static struct device *pcu_dev;
static struct device *get_pcu_dev(void) static struct device *get_pcu_dev(void)
@ -48,7 +48,7 @@ static struct device *get_pcu_dev(void)
pcu_dev = pcidev_on_root(PCU_DEV, 0); pcu_dev = pcidev_on_root(PCU_DEV, 0);
return pcu_dev; return pcu_dev;
} }
#endif /* ENV_SMM */ #endif /* __SIMPLE_DEVICE__ */
uint16_t get_pmbase(void) uint16_t get_pmbase(void)
{ {

6
src/soc/intel/broadwell/xhci.c
View File

@ -24,7 +24,7 @@
#include <soc/xhci.h> #include <soc/xhci.h>
#include <soc/cpu.h> #include <soc/cpu.h>
#ifdef __SMM__ #ifdef __SIMPLE_DEVICE__
static u8 *usb_xhci_mem_base(pci_devfn_t dev) static u8 *usb_xhci_mem_base(pci_devfn_t dev)
{ {
u32 mem_base = pci_read_config32(dev, PCI_BASE_ADDRESS_0); u32 mem_base = pci_read_config32(dev, PCI_BASE_ADDRESS_0);
@ -196,7 +196,7 @@ void usb_xhci_sleep_prepare(pci_devfn_t dev, u8 slp_typ)
pci_or_config16(dev, XHCI_PWR_CTL_STS, XHCI_PWR_CTL_STATUS_PME); pci_or_config16(dev, XHCI_PWR_CTL_STS, XHCI_PWR_CTL_STATUS_PME);
pci_or_config16(dev, XHCI_PWR_CTL_STS, XHCI_PWR_CTL_ENABLE_PME); pci_or_config16(dev, XHCI_PWR_CTL_STS, XHCI_PWR_CTL_ENABLE_PME);
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
static void xhci_init(struct device *dev) static void xhci_init(struct device *dev)
{ {
@ -235,4 +235,4 @@ static const struct pci_driver pch_usb_xhci __pci_driver = {
.vendor = PCI_VENDOR_ID_INTEL, .vendor = PCI_VENDOR_ID_INTEL,
.devices = pci_device_ids, .devices = pci_device_ids,
}; };
#endif /* !__SMM__ */ #endif /* !__SIMPLE_DEVICE__ */

25
src/southbridge/intel/bd82x6x/me.c
View File

@ -26,6 +26,8 @@
#include <device/mmio.h> #include <device/mmio.h>
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include <console/console.h> #include <console/console.h>
#include <device/device.h>
#include <device/pci.h>
#include <device/pci_ids.h> #include <device/pci_ids.h>
#include <device/pci_def.h> #include <device/pci_def.h>
#include <string.h> #include <string.h>
@ -33,11 +35,6 @@
#include <elog.h> #include <elog.h>
#include <halt.h> #include <halt.h>
#ifndef __SMM__
#include <device/device.h>
#include <device/pci.h>
#endif
#include "me.h" #include "me.h"
#include "pch.h" #include "pch.h"
@ -45,9 +42,8 @@
#include <vendorcode/google/chromeos/gnvs.h> #include <vendorcode/google/chromeos/gnvs.h>
#endif #endif
#ifndef __SMM__
/* Path that the BIOS should take based on ME state */ /* Path that the BIOS should take based on ME state */
static const char *me_bios_path_values[] = { static const char *me_bios_path_values[] __unused = {
[ME_NORMAL_BIOS_PATH] = "Normal", [ME_NORMAL_BIOS_PATH] = "Normal",
[ME_S3WAKE_BIOS_PATH] = "S3 Wake", [ME_S3WAKE_BIOS_PATH] = "S3 Wake",
[ME_ERROR_BIOS_PATH] = "Error", [ME_ERROR_BIOS_PATH] = "Error",
@ -55,7 +51,6 @@ static const char *me_bios_path_values[] = {
[ME_DISABLE_BIOS_PATH] = "Disable", [ME_DISABLE_BIOS_PATH] = "Disable",
[ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update", [ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update",
}; };
#endif
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
@ -112,7 +107,7 @@ static inline void mei_write_dword_ptr(void *ptr, int offset)
mei_dump(ptr, dword, offset, "WRITE"); mei_dump(ptr, dword, offset, "WRITE");
} }
#ifndef __SMM__ #ifndef __SIMPLE_DEVICE__
static inline void pci_read_dword_ptr(struct device *dev, void *ptr, int offset) static inline void pci_read_dword_ptr(struct device *dev, void *ptr, int offset)
{ {
u32 dword = pci_read_config32(dev, offset); u32 dword = pci_read_config32(dev, offset);
@ -346,9 +341,8 @@ static inline int mei_sendrecv(struct mei_header *mei, struct mkhi_header *mkhi,
return 0; return 0;
} }
#ifdef __SMM__
/* Send END OF POST message to the ME */ /* Send END OF POST message to the ME */
static int mkhi_end_of_post(void) static int __unused mkhi_end_of_post(void)
{ {
struct mkhi_header mkhi = { struct mkhi_header mkhi = {
.group_id = MKHI_GROUP_ID_GEN, .group_id = MKHI_GROUP_ID_GEN,
@ -370,7 +364,6 @@ static int mkhi_end_of_post(void)
printk(BIOS_INFO, "ME: END OF POST message successful\n"); printk(BIOS_INFO, "ME: END OF POST message successful\n");
return 0; return 0;
} }
#endif
/* Get ME firmware version */ /* Get ME firmware version */
static int __unused mkhi_get_fw_version(void) static int __unused mkhi_get_fw_version(void)
@ -486,7 +479,8 @@ int mkhi_global_reset(void)
} }
#endif #endif
#ifdef __SMM__ #ifdef __SIMPLE_DEVICE__
static void intel_me7_finalize_smm(void) static void intel_me7_finalize_smm(void)
{ {
struct me_hfs hfs; struct me_hfs hfs;
@ -536,7 +530,8 @@ void intel_me_finalize_smm(void)
printk(BIOS_ERR, "No finalize handler for ME %08x.\n", did); printk(BIOS_ERR, "No finalize handler for ME %08x.\n", did);
} }
} }
#else /* !__SMM__ */
#else
/* Determine the path that we should take based on ME status */ /* Determine the path that we should take based on ME status */
static me_bios_path intel_me_path(struct device *dev) static me_bios_path intel_me_path(struct device *dev)
@ -748,4 +743,4 @@ static const struct pci_driver intel_me __pci_driver = {
.device = 0x1c3a, .device = 0x1c3a,
}; };
#endif /* !__SMM__ */ #endif /* __SIMPLE_DEVICE__ */

27
src/southbridge/intel/bd82x6x/me_8.x.c
View File

@ -24,6 +24,8 @@
#include <arch/acpi.h> #include <arch/acpi.h>
#include <device/mmio.h> #include <device/mmio.h>
#include <device/device.h>
#include <device/pci.h>
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include <console/console.h> #include <console/console.h>
#include <device/pci_ids.h> #include <device/pci_ids.h>
@ -33,11 +35,6 @@
#include <elog.h> #include <elog.h>
#include <halt.h> #include <halt.h>
#ifndef __SMM__
#include <device/device.h>
#include <device/pci.h>
#endif
#include "me.h" #include "me.h"
#include "pch.h" #include "pch.h"
@ -46,9 +43,8 @@
#include <vendorcode/google/chromeos/gnvs.h> #include <vendorcode/google/chromeos/gnvs.h>
#endif #endif
#ifndef __SMM__
/* Path that the BIOS should take based on ME state */ /* Path that the BIOS should take based on ME state */
static const char *me_bios_path_values[] = { static const char *me_bios_path_values[] __unused = {
[ME_NORMAL_BIOS_PATH] = "Normal", [ME_NORMAL_BIOS_PATH] = "Normal",
[ME_S3WAKE_BIOS_PATH] = "S3 Wake", [ME_S3WAKE_BIOS_PATH] = "S3 Wake",
[ME_ERROR_BIOS_PATH] = "Error", [ME_ERROR_BIOS_PATH] = "Error",
@ -57,7 +53,6 @@ static const char *me_bios_path_values[] = {
[ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update", [ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update",
}; };
static int intel_me_read_mbp(me_bios_payload *mbp_data); static int intel_me_read_mbp(me_bios_payload *mbp_data);
#endif
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
@ -115,7 +110,7 @@ static inline void mei_write_dword_ptr(void *ptr, int offset)
mei_dump(ptr, dword, offset, "WRITE"); mei_dump(ptr, dword, offset, "WRITE");
} }
#ifndef __SMM__ #ifndef __SIMPLE_DEVICE__
static inline void pci_read_dword_ptr(struct device *dev, void *ptr, int offset) static inline void pci_read_dword_ptr(struct device *dev, void *ptr, int offset)
{ {
u32 dword = pci_read_config32(dev, offset); u32 dword = pci_read_config32(dev, offset);
@ -453,10 +448,8 @@ static int mkhi_global_reset(void)
} }
#endif #endif
#ifdef __SMM__
/* Send END OF POST message to the ME */ /* Send END OF POST message to the ME */
static int mkhi_end_of_post(void) static int __unused mkhi_end_of_post(void)
{ {
struct mkhi_header mkhi = { struct mkhi_header mkhi = {
.group_id = MKHI_GROUP_ID_GEN, .group_id = MKHI_GROUP_ID_GEN,
@ -482,6 +475,8 @@ static int mkhi_end_of_post(void)
return 0; return 0;
} }
#ifdef __SIMPLE_DEVICE__
void intel_me8_finalize_smm(void) void intel_me8_finalize_smm(void)
{ {
struct me_hfs hfs; struct me_hfs hfs;
@ -517,7 +512,7 @@ void intel_me8_finalize_smm(void)
RCBA32_OR(FD2, PCH_DISABLE_MEI1); RCBA32_OR(FD2, PCH_DISABLE_MEI1);
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
/* Determine the path that we should take based on ME status */ /* Determine the path that we should take based on ME status */
static me_bios_path intel_me_path(struct device *dev) static me_bios_path intel_me_path(struct device *dev)
@ -752,6 +747,8 @@ static const struct pci_driver intel_me __pci_driver = {
.device = 0x1e3a, .device = 0x1e3a,
}; };
#endif /* !__SIMPLE_DEVICE__ */
/****************************************************************************** /******************************************************************************
* */ * */
static u32 me_to_host_words_pending(void) static u32 me_to_host_words_pending(void)
@ -783,7 +780,7 @@ static u32 host_to_me_words_room(void)
* mbp seems to be following its own flow, let's retrieve it in a dedicated * mbp seems to be following its own flow, let's retrieve it in a dedicated
* function. * function.
*/ */
static int intel_me_read_mbp(me_bios_payload *mbp_data) static int __unused intel_me_read_mbp(me_bios_payload *mbp_data)
{ {
mbp_header mbp_hdr; mbp_header mbp_hdr;
mbp_item_header mbp_item_hdr; mbp_item_header mbp_item_hdr;
@ -907,5 +904,3 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data)
return 0; return 0;
} }
#endif /* !__SMM__ */

7
src/southbridge/intel/bd82x6x/pch.c
View File

@ -17,12 +17,9 @@
#include <console/console.h> #include <console/console.h>
#include <delay.h> #include <delay.h>
#ifdef __SMM__
#include <device/pci_def.h>
#else /* !__SMM__ */
#include <device/device.h> #include <device/device.h>
#include <device/pci.h> #include <device/pci.h>
#endif #include <device/pci_def.h>
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include <string.h> #include <string.h>
@ -145,7 +142,7 @@ void pch_iobp_update(u32 address, u32 andvalue, u32 orvalue)
return; return;
} }
#ifndef __SMM__ #ifndef __SIMPLE_DEVICE__
/* Set bit in function disable register to hide this device */ /* Set bit in function disable register to hide this device */
static void pch_hide_devfn(unsigned int devfn) static void pch_hide_devfn(unsigned int devfn)
{ {

2
src/southbridge/intel/common/gpio.c
View File

@ -35,7 +35,7 @@
static u16 get_gpio_base(void) static u16 get_gpio_base(void)
{ {
#if defined(__SMM__) #ifdef __SIMPLE_DEVICE__
/* Don't assume GPIO_BASE is still the same */ /* Don't assume GPIO_BASE is still the same */
return pci_read_config16(PCH_LPC_DEV, GPIO_BASE) & 0xfffe; return pci_read_config16(PCH_LPC_DEV, GPIO_BASE) & 0xfffe;
#else #else

2
src/southbridge/intel/common/pmbase.c
View File

@ -38,7 +38,7 @@
u16 lpc_get_pmbase(void) u16 lpc_get_pmbase(void)
{ {
#if defined(__SMM__) #ifdef __SIMPLE_DEVICE__
/* Don't assume PMBASE is still the same */ /* Don't assume PMBASE is still the same */
return pci_read_config16(PCH_LPC_DEV, PMBASE) & 0xfffc; return pci_read_config16(PCH_LPC_DEV, PMBASE) & 0xfffc;
#else #else

28
src/southbridge/intel/ibexpeak/me.c
View File

@ -23,20 +23,17 @@
*/ */
#include <arch/acpi.h> #include <arch/acpi.h>
#include <device/mmio.h>
#include <device/pci_ops.h>
#include <console/console.h> #include <console/console.h>
#include <device/pci_ids.h> #include <device/device.h>
#include <device/mmio.h>
#include <device/pci.h>
#include <device/pci_def.h> #include <device/pci_def.h>
#include <device/pci_ids.h>
#include <device/pci_ops.h>
#include <string.h> #include <string.h>
#include <delay.h> #include <delay.h>
#include <elog.h> #include <elog.h>
#ifndef __SMM__
#include <device/device.h>
#include <device/pci.h>
#endif
#include "me.h" #include "me.h"
#include "pch.h" #include "pch.h"
@ -44,9 +41,8 @@
#include <vendorcode/google/chromeos/gnvs.h> #include <vendorcode/google/chromeos/gnvs.h>
#endif #endif
#ifndef __SMM__
/* Path that the BIOS should take based on ME state */ /* Path that the BIOS should take based on ME state */
static const char *me_bios_path_values[] = { static const char *me_bios_path_values[] __unused = {
[ME_NORMAL_BIOS_PATH] = "Normal", [ME_NORMAL_BIOS_PATH] = "Normal",
[ME_S3WAKE_BIOS_PATH] = "S3 Wake", [ME_S3WAKE_BIOS_PATH] = "S3 Wake",
[ME_ERROR_BIOS_PATH] = "Error", [ME_ERROR_BIOS_PATH] = "Error",
@ -54,7 +50,6 @@ static const char *me_bios_path_values[] = {
[ME_DISABLE_BIOS_PATH] = "Disable", [ME_DISABLE_BIOS_PATH] = "Disable",
[ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update", [ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update",
}; };
#endif
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
@ -111,7 +106,7 @@ static inline void mei_write_dword_ptr(void *ptr, int offset)
mei_dump(ptr, dword, offset, "WRITE"); mei_dump(ptr, dword, offset, "WRITE");
} }
#ifndef __SMM__ #ifndef __SIMPLE_DEVICE__
static inline void pci_read_dword_ptr(struct device *dev,void *ptr, static inline void pci_read_dword_ptr(struct device *dev,void *ptr,
int offset) int offset)
{ {
@ -131,7 +126,6 @@ static inline void write_host_csr(struct mei_csr *csr)
mei_write_dword_ptr(csr, MEI_H_CSR); mei_write_dword_ptr(csr, MEI_H_CSR);
} }
#ifdef __SMM__
static inline void read_me_csr(struct mei_csr *csr) static inline void read_me_csr(struct mei_csr *csr)
{ {
mei_read_dword_ptr(csr, MEI_ME_CSR_HA); mei_read_dword_ptr(csr, MEI_ME_CSR_HA);
@ -348,7 +342,7 @@ static inline int mei_sendrecv(struct mei_header *mei, struct mkhi_header *mkhi,
} }
/* Send END OF POST message to the ME */ /* Send END OF POST message to the ME */
static int mkhi_end_of_post(void) static int __unused mkhi_end_of_post(void)
{ {
struct mkhi_header mkhi = { struct mkhi_header mkhi = {
.group_id = MKHI_GROUP_ID_GEN, .group_id = MKHI_GROUP_ID_GEN,
@ -371,6 +365,8 @@ static int mkhi_end_of_post(void)
return 0; return 0;
} }
#ifdef __SIMPLE_DEVICE__
static void intel_me7_finalize_smm(void) static void intel_me7_finalize_smm(void)
{ {
struct me_hfs hfs; struct me_hfs hfs;
@ -420,7 +416,7 @@ void intel_me_finalize_smm(void)
printk(BIOS_ERR, "No finalize handler for ME %08x.\n", did); printk(BIOS_ERR, "No finalize handler for ME %08x.\n", did);
} }
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
/* Determine the path that we should take based on ME status */ /* Determine the path that we should take based on ME status */
static me_bios_path intel_me_path(struct device *dev) static me_bios_path intel_me_path(struct device *dev)
@ -629,4 +625,4 @@ static const struct pci_driver intel_me __pci_driver = {
.devices = pci_device_ids .devices = pci_device_ids
}; };
#endif /* !__SMM__ */ #endif /* !__SIMPLE_DEVICE__ */

26
src/southbridge/intel/lynxpoint/me_9.x.c
View File

@ -45,9 +45,8 @@
#include <vendorcode/google/chromeos/gnvs.h> #include <vendorcode/google/chromeos/gnvs.h>
#endif #endif
#ifndef __SMM__
/* Path that the BIOS should take based on ME state */ /* Path that the BIOS should take based on ME state */
static const char *me_bios_path_values[] = { static const char *me_bios_path_values[] __unused = {
[ME_NORMAL_BIOS_PATH] = "Normal", [ME_NORMAL_BIOS_PATH] = "Normal",
[ME_S3WAKE_BIOS_PATH] = "S3 Wake", [ME_S3WAKE_BIOS_PATH] = "S3 Wake",
[ME_ERROR_BIOS_PATH] = "Error", [ME_ERROR_BIOS_PATH] = "Error",
@ -56,7 +55,6 @@ static const char *me_bios_path_values[] = {
[ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update", [ME_FIRMWARE_UPDATE_BIOS_PATH] = "Firmware Update",
}; };
static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev); static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev);
#endif
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
@ -557,10 +555,8 @@ static int mkhi_global_reset(void)
} }
#endif #endif
#ifdef __SMM__
/* Send END OF POST message to the ME */ /* Send END OF POST message to the ME */
static int mkhi_end_of_post(void) static int __unused mkhi_end_of_post(void)
{ {
struct mkhi_header mkhi = { struct mkhi_header mkhi = {
.group_id = MKHI_GROUP_ID_GEN, .group_id = MKHI_GROUP_ID_GEN,
@ -579,6 +575,8 @@ static int mkhi_end_of_post(void)
return 0; return 0;
} }
#ifdef __SIMPLE_DEVICE__
void intel_me_finalize_smm(void) void intel_me_finalize_smm(void)
{ {
struct me_hfs hfs; struct me_hfs hfs;
@ -619,7 +617,7 @@ void intel_me_finalize_smm(void)
RCBA32_OR(FD2, PCH_DISABLE_MEI1); RCBA32_OR(FD2, PCH_DISABLE_MEI1);
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
static inline int mei_sendrecv_icc(struct icc_header *icc, static inline int mei_sendrecv_icc(struct icc_header *icc,
void *req_data, int req_bytes, void *req_data, int req_bytes,
@ -901,6 +899,8 @@ static const struct pci_driver intel_me __pci_driver = {
.devices= pci_device_ids, .devices= pci_device_ids,
}; };
#endif /* !__SIMPLE_DEVICE__ */
/****************************************************************************** /******************************************************************************
* */ * */
static u32 me_to_host_words_pending(void) static u32 me_to_host_words_pending(void)
@ -938,7 +938,7 @@ struct mbp_payload {
* mbp seems to be following its own flow, let's retrieve it in a dedicated * mbp seems to be following its own flow, let's retrieve it in a dedicated
* function. * function.
*/ */
static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev) static int __unused intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev)
{ {
mbp_header mbp_hdr; mbp_header mbp_hdr;
u32 me2host_pending; u32 me2host_pending;
@ -947,7 +947,11 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev)
struct mbp_payload *mbp; struct mbp_payload *mbp;
int i; int i;
#ifdef __SIMPLE_DEVICE__
pci_read_dword_ptr(PCI_BDF(dev), &hfs2, PCI_ME_HFS2);
#else
pci_read_dword_ptr(dev, &hfs2, PCI_ME_HFS2); pci_read_dword_ptr(dev, &hfs2, PCI_ME_HFS2);
#endif
if (!hfs2.mbp_rdy) { if (!hfs2.mbp_rdy) {
printk(BIOS_ERR, "ME: MBP not ready\n"); printk(BIOS_ERR, "ME: MBP not ready\n");
@ -1057,8 +1061,10 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev)
return 0; return 0;
mbp_failure: mbp_failure:
#ifdef __SIMPLE_DEVICE__
intel_me_mbp_give_up(PCI_BDF(dev));
#else
intel_me_mbp_give_up(dev); intel_me_mbp_give_up(dev);
#endif
return -1; return -1;
} }
#endif /* !__SMM__ */

4
src/southbridge/intel/lynxpoint/pch.c
View File

@ -90,7 +90,7 @@ u16 get_gpiobase(void)
return gpiobase; return gpiobase;
} }
#ifndef __SMM__ #ifndef __SIMPLE_DEVICE__
/* Put device in D3Hot Power State */ /* Put device in D3Hot Power State */
static void pch_enable_d3hot(struct device *dev) static void pch_enable_d3hot(struct device *dev)
@ -330,4 +330,4 @@ struct chip_operations southbridge_intel_lynxpoint_ops = {
.enable_dev = pch_enable, .enable_dev = pch_enable,
}; };
#endif /* __SMM__ */ #endif /* __SIMPLE_DEVICE__ */

6
src/southbridge/intel/lynxpoint/usb_ehci.c
View File

@ -24,7 +24,7 @@
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include "pch.h" #include "pch.h"
#ifdef __SMM__ #ifdef __SIMPLE_DEVICE__
void usb_ehci_disable(pci_devfn_t dev) void usb_ehci_disable(pci_devfn_t dev)
{ {
@ -132,7 +132,7 @@ void usb_ehci_sleep_prepare(pci_devfn_t dev, u8 slp_typ)
} }
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
static void usb_ehci_clock_gating(struct device *dev) static void usb_ehci_clock_gating(struct device *dev)
{ {
@ -202,4 +202,4 @@ static const struct pci_driver pch_usb_ehci __pci_driver = {
.devices = pci_device_ids, .devices = pci_device_ids,
}; };
#endif /* !__SMM__ */ #endif /* !__SIMPLE_DEVICE__ */

6
src/southbridge/intel/lynxpoint/usb_xhci.c
View File

@ -166,7 +166,7 @@ static void usb_xhci_reset_usb3(struct device *dev, int all)
usb_xhci_reset_status_usb3(mem_base, port); usb_xhci_reset_status_usb3(mem_base, port);
} }
#ifdef __SMM__ #ifdef __SIMPLE_DEVICE__
/* Handler for XHCI controller on entry to S3/S4/S5 */ /* Handler for XHCI controller on entry to S3/S4/S5 */
void usb_xhci_sleep_prepare(pci_devfn_t dev, u8 slp_typ) void usb_xhci_sleep_prepare(pci_devfn_t dev, u8 slp_typ)
@ -251,7 +251,7 @@ void usb_xhci_route_all(void)
usb_xhci_reset_usb3(PCH_XHCI_DEV, 1); usb_xhci_reset_usb3(PCH_XHCI_DEV, 1);
} }
#else /* !__SMM__ */ #else /* !__SIMPLE_DEVICE__ */
static void usb_xhci_clock_gating(struct device *dev) static void usb_xhci_clock_gating(struct device *dev)
{ {
@ -395,4 +395,4 @@ static const struct pci_driver pch_usb_xhci __pci_driver = {
.vendor = PCI_VENDOR_ID_INTEL, .vendor = PCI_VENDOR_ID_INTEL,
.devices = pci_device_ids, .devices = pci_device_ids,
}; };
#endif /* !__SMM__ */ #endif /* !__SIMPLE_DEVICE__ */