nb/intel/haswell: Clean up haswell.h header

Drop unused chipset type macros, remove unnecessary guards and
reorganize contents so that headers can be included at the top.
Also drop the inclusion from ASL, as it is no longer necessary.

Tested with BUILD_TIMELESS=1, Asrock B85M Pro4 remains identical.

Change-Id: I6fcc0d428d0fdbf410bcbeb6ae4809870b7b498f
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51889
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Angel Pons 2021-03-27 20:32:16 +01:00 committed by Patrick Georgi
parent 45f449416d
commit 9fa141898e
2 changed files with 6 additions and 38 deletions

View File

@ -1,7 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */ /* SPDX-License-Identifier: GPL-2.0-only */
#include "../haswell.h"
Name (_HID, EISAID ("PNP0A08")) // PCIe Name (_HID, EISAID ("PNP0A08")) // PCIe
Name (_CID, EISAID ("PNP0A03")) // PCI Name (_CID, EISAID ("PNP0A03")) // PCI

View File

@ -3,21 +3,18 @@
#ifndef __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__ #ifndef __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__
#define __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__ #define __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__
/* Chipset types */ #include <device/device.h>
#define HASWELL_MOBILE 0 #include <northbridge/intel/common/fixed_bars.h>
#define HASWELL_DESKTOP 1
#define HASWELL_SERVER 2
#include "memmap.h" #include "memmap.h"
#include "registers/dmibar.h"
/* Everything below this line is ignored in the DSDT */ #include "registers/epbar.h"
#ifndef __ACPI__ #include "registers/host_bridge.h"
#include "registers/mchbar.h"
/* Device 0:0.0 PCI configuration space (Host Bridge) */ /* Device 0:0.0 PCI configuration space (Host Bridge) */
#define HOST_BRIDGE PCI_DEV(0, 0, 0) #define HOST_BRIDGE PCI_DEV(0, 0, 0)
#include "registers/host_bridge.h"
/* Device 0:1.0 PCI configuration space (PCIe Graphics) */ /* Device 0:1.0 PCI configuration space (PCIe Graphics) */
#define PEG_CAP 0xa2 #define PEG_CAP 0xa2
@ -51,15 +48,6 @@
#define MSAC 0x62 /* Multi Size Aperture Control */ #define MSAC 0x62 /* Multi Size Aperture Control */
/*
* MCHBAR
*/
#include <northbridge/intel/common/fixed_bars.h>
/* As there are many registers, define them on a separate file */
#include "registers/mchbar.h"
#define ARCHDIS 0xff0 /* DMA Remap Engine Policy Control */ #define ARCHDIS 0xff0 /* DMA Remap Engine Policy Control */
#define DMAR_LCKDN (1 << 31) #define DMAR_LCKDN (1 << 31)
#define SPCAPCTRL (1 << 25) #define SPCAPCTRL (1 << 25)
@ -68,20 +56,6 @@
#define GLBIOTLBINV (1 << 1) #define GLBIOTLBINV (1 << 1)
#define GLBCTXTINV (1 << 0) #define GLBCTXTINV (1 << 0)
/*
* EPBAR - Egress Port Root Complex Register Block
*/
#include "registers/epbar.h"
/*
* DMIBAR
*/
#include "registers/dmibar.h"
#ifndef __ASSEMBLER__
void mb_late_romstage_setup(void); /* optional */ void mb_late_romstage_setup(void); /* optional */
void haswell_early_initialization(void); void haswell_early_initialization(void);
@ -90,12 +64,8 @@ void haswell_unhide_peg(void);
void report_platform_info(void); void report_platform_info(void);
#include <device/device.h>
struct acpi_rsdp; struct acpi_rsdp;
unsigned long northbridge_write_acpi_tables(const struct device *device, unsigned long start, unsigned long northbridge_write_acpi_tables(const struct device *device, unsigned long start,
struct acpi_rsdp *rsdp); struct acpi_rsdp *rsdp);
#endif /* __ASSEMBLER__ */
#endif /* __ACPI__ */
#endif /* __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__ */ #endif /* __NORTHBRIDGE_INTEL_HASWELL_HASWELL_H__ */