From 8f0b3e546a6ee8501ac5c34b6522ae52b8104e2d Mon Sep 17 00:00:00 2001
From: Angel Pons <th3fanbus@gmail.com>
Date: Tue, 15 Sep 2020 01:03:22 +0200
Subject: nb/intel/pineview: Place raminit definitions in raminit.h

There's no need to have implementation details in a public header.

Tested with BUILD_TIMELESS=1, Foxconn D41S remains identical.

Change-Id: I0bfd6ee72347249302ee073081f670b315aa40e4
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45388
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
---
 src/northbridge/intel/pineview/pineview.h | 117 ------------------------------
 1 file changed, 117 deletions(-)

(limited to 'src/northbridge/intel/pineview/pineview.h')

diff --git a/src/northbridge/intel/pineview/pineview.h b/src/northbridge/intel/pineview/pineview.h
index 03fa318a14..cafda8ace6 100644
--- a/src/northbridge/intel/pineview/pineview.h
+++ b/src/northbridge/intel/pineview/pineview.h
@@ -10,12 +10,6 @@
 #define BOOT_PATH_RESET		1
 #define BOOT_PATH_RESUME	2
 
-#define SYSINFO_DIMM_NOT_POPULATED	0x00
-#define SYSINFO_DIMM_X16SS		0x01
-#define SYSINFO_DIMM_X16DS		0x02
-#define SYSINFO_DIMM_X8DS		0x05
-#define SYSINFO_DIMM_X8DDS		0x06
-
 /* Device 0:0.0 PCI configuration space (Host Bridge) */
 #define HOST_BRIDGE	PCI_DEV(0, 0, 0)
 
@@ -71,117 +65,6 @@
 #define DMIBAR16(x) *((volatile u16 *)(DEFAULT_DMIBAR + (x)))
 #define DMIBAR32(x) *((volatile u32 *)(DEFAULT_DMIBAR + (x)))
 
-enum fsb_clk {
-	FSB_CLOCK_667MHz = 0,
-	FSB_CLOCK_800MHz = 1,
-};
-
-enum mem_clk {
-	MEM_CLOCK_667MHz = 0,
-	MEM_CLOCK_800MHz = 1,
-};
-
-enum ddr {
-	DDR2 = 2,
-	DDR3 = 3,
-};
-
-enum chip_width { /* as in DDR3 spd */
-	CHIP_WIDTH_x4	= 0,
-	CHIP_WIDTH_x8	= 1,
-	CHIP_WIDTH_x16	= 2,
-	CHIP_WIDTH_x32	= 3,
-};
-
-enum chip_cap { /* as in DDR3 spd */
-	CHIP_CAP_256M	= 0,
-	CHIP_CAP_512M	= 1,
-	CHIP_CAP_1G	= 2,
-	CHIP_CAP_2G	= 3,
-	CHIP_CAP_4G	= 4,
-	CHIP_CAP_8G	= 5,
-	CHIP_CAP_16G	= 6,
-};
-
-struct timings {
-	unsigned int	CAS;
-	enum fsb_clk	fsb_clock;
-	enum mem_clk	mem_clock;
-	unsigned int	tRAS;
-	unsigned int	tRP;
-	unsigned int	tRCD;
-	unsigned int	tWR;
-	unsigned int	tRFC;
-	unsigned int	tWTR;
-	unsigned int	tRRD;
-	unsigned int	tRTP;
-};
-
-struct dimminfo {
-	unsigned int	card_type; /* 0x0: unpopulated,
-				      0xa - 0xf: raw card type A - F */
-	u8		type;
-	enum chip_width	width;
-	enum chip_cap	chip_capacity;
-	unsigned int	page_size; /* of whole DIMM in Bytes (4096 or 8192) */
-	unsigned int	sides;
-	unsigned int	banks;
-	unsigned int	ranks;
-	unsigned int	rows;
-	unsigned int	cols;
-	unsigned int	cas_latencies;
-	unsigned int	tAAmin;
-	unsigned int	tCKmin;
-	unsigned int	tWR;
-	unsigned int	tRP;
-	unsigned int	tRCD;
-	unsigned int	tRAS;
-	unsigned int	rank_capacity_mb; /* per rank in Megabytes */
-	u8		spd_data[256];
-};
-
-struct pllparam {
-	u8 kcoarse[2][72];
-	u8 pi[2][72];
-	u8 dben[2][72];
-	u8 dbsel[2][72];
-	u8 clkdelay[2][72];
-};
-
-struct sysinfo {
-	u8 maxpi;
-	u8 pioffset;
-	u8 pi[8];
-	u16 coarsectrl;
-	u16 coarsedelay;
-	u16 mediumphase;
-	u16 readptrdelay;
-
-	int		txt_enabled;
-	int		cores;
-	int		boot_path;
-	int		max_ddr2_mhz;
-	int		max_ddr3_mt;
-	int		max_fsb_mhz;
-	int		max_render_mhz;
-	int		enable_igd;
-	int		enable_peg;
-	u16		ggc;
-
-	int		dimm_config[2];
-	int		dimms_per_ch;
-	int		spd_type;
-	int		channel_capacity[2];
-	struct timings	selected_timings;
-	struct dimminfo	dimms[4];
-	u8		spd_map[4];
-
-	u8 nodll;
-	u8 async;
-	u8 dt0mode;
-	u8 mvco4x;		/* 0 (8x) or 1 (4x) */
-};
-
 void pineview_early_init(void);
 u32 decode_igd_memory_size(const u32 gms);
 u32 decode_igd_gtt_size(const u32 gsm);
-- 
cgit v1.2.3