src: Fix various spelling and whitespace issues.
This fixes some spelling and whitespace issues that I came across while working on various things in the tree. There are no functional changes. Change-Id: I33bc77282f2f94a1fc5f1bc713e44f72db20c1ab Signed-off-by: Martin Roth <martinroth@google.com> Reviewed-on: https://review.coreboot.org/13016 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
parent
bda8a04b01
commit
2f91403303
|
@ -92,7 +92,7 @@ static void enable_var_mtrr(unsigned char deftype)
|
|||
wrmsr(MTRR_DEF_TYPE_MSR, msr);
|
||||
}
|
||||
|
||||
/* fms: find most sigificant bit set, stolen from Linux Kernel Source. */
|
||||
/* fms: find most significant bit set, stolen from Linux Kernel Source. */
|
||||
static inline unsigned int fms(unsigned int x)
|
||||
{
|
||||
int r;
|
||||
|
@ -209,7 +209,7 @@ static struct memranges *get_physical_address_space(void)
|
|||
filter_vga_wrcomb);
|
||||
|
||||
/* The address space below 4GiB is special. It needs to be
|
||||
* covered entirly by range entries so that MTRR calculations
|
||||
* covered entirely by range entries so that MTRR calculations
|
||||
* can be properly done for the full 32-bit address space.
|
||||
* Therefore, ensure holes are filled up to 4GiB as
|
||||
* uncacheable */
|
||||
|
|
|
@ -90,7 +90,7 @@ gpioEarlyInit(
|
|||
for (Index = 0; Index < MAX_GPIO_NO; Index++) {
|
||||
if (!(((Index >= GPIO_RSVD_ZONE0_S) && (Index <= GPIO_RSVD_ZONE0_E)) || ((Index >= GPIO_RSVD_ZONE1_S) && (Index <= GPIO_RSVD_ZONE1_E)))) {
|
||||
if ((StripInfo >= boardRevC) || ((Index != GPIO_111) && (Index != GPIO_113))) {
|
||||
// Configure multi-funtion
|
||||
// Configure multi-function
|
||||
Mmio8_And_Or (IoMuxMmioAddr, Index, 0x00, (gpio_table[Index].select & ~NonGpio));
|
||||
}
|
||||
// Configure GPIO
|
||||
|
|
|
@ -88,7 +88,7 @@ gpioEarlyInit(
|
|||
for (Index = 0; Index < MAX_GPIO_NO; Index++) {
|
||||
if (!(((Index >= GPIO_RSVD_ZONE0_S) && (Index <= GPIO_RSVD_ZONE0_E)) || ((Index >= GPIO_RSVD_ZONE1_S) && (Index <= GPIO_RSVD_ZONE1_E)))) {
|
||||
if ((StripInfo >= boardRevC) || ((Index != GPIO_111) && (Index != GPIO_113))) {
|
||||
// Configure multi-funtion
|
||||
// Configure multi-function
|
||||
Mmio8_And_Or (IoMuxMmioAddr, Index, 0x00, (gpio_table[Index].select & ~NonGpio));
|
||||
}
|
||||
// Configure GPIO
|
||||
|
|
|
@ -64,7 +64,7 @@ index:
|
|||
Write:
|
||||
- Write the data to DctOffsetData
|
||||
- Write register num to DctOffset with DctAccessWrite = 1
|
||||
- poll the DctAccessDone untio it = 1
|
||||
- poll the DctAccessDone until it = 1
|
||||
*/
|
||||
|
||||
|
||||
|
@ -153,7 +153,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 100 = reserved
|
||||
* 101 = reserved
|
||||
* 110 = reserved
|
||||
* 111 = Interleve on A[12] and A[13] and A[14] (8 nodes)
|
||||
* 111 = Interleave on A[12] and A[13] and A[14] (8 nodes)
|
||||
* [15:11] Reserved
|
||||
* [13:16] DRAM Base Address i Bits 39-24
|
||||
* This field defines the upper address bits of a 40-bit address
|
||||
|
@ -232,7 +232,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 100 = 2.5 Memory Clocks
|
||||
* 101 = 3 Memory Clocks
|
||||
* 110 = 3.5 Memory Clocks
|
||||
* 111 = Reseved
|
||||
* 111 = Reserved
|
||||
* [15: 7] Reserved
|
||||
* [16:16] AltVidC3MemClkTriEn (AltVID Memory Clock Tristate Enable)
|
||||
* Enables the DDR memory clocks to be tristated when alternate VID
|
||||
|
@ -244,19 +244,19 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* [18:18] DqsRcvEnTrain (DQS Receiver Enable Training Mode)
|
||||
* 0 = Normal DQS Receiver enable operation
|
||||
* 1 = DQS receiver enable training mode
|
||||
* [31:19] reverved
|
||||
* [31:19] reserved
|
||||
*/
|
||||
PCI_ADDR(0, 0x18, 2, 0x78), 0xfff80000, (6<<4)|(6<<0),
|
||||
|
||||
/* DRAM Initialization Register
|
||||
* F2:0x7C
|
||||
* [15: 0] MrsAddress (Address for MRS/EMRS Commands)
|
||||
* this field specifies the dsata driven on the DRAM address pins
|
||||
* this field specifies the data driven on the DRAM address pins
|
||||
* 15-0 for MRS and EMRS commands
|
||||
* [18:16] MrsBank (Bank Address for MRS/EMRS Commands)
|
||||
* this files specifies the data driven on the DRAM bank pins for
|
||||
* the MRS and EMRS commands
|
||||
* [23:19] reverved
|
||||
* [23:19] reserved
|
||||
* [24:24] SendPchgAll (Send Precharge All Command)
|
||||
* Setting this bit causes the DRAM controller to send a precharge
|
||||
* all command. This bit is cleared by the hardware after the
|
||||
|
@ -268,7 +268,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* [26:26] SendMrsCmd (Send MRS/EMRS Command)
|
||||
* Setting this bit causes the DRAM controller to send the MRS or
|
||||
* EMRS command defined by the MrsAddress and MrsBank fields. This
|
||||
* bit is cleared by the hardware adter the commmand completes
|
||||
* bit is cleared by the hardware after the command completes
|
||||
* [27:27] DeassertMemRstX (De-assert Memory Reset)
|
||||
* Setting this bit causes the DRAM controller to de-assert the
|
||||
* memory reset pin. This bit cannot be used to assert the memory
|
||||
|
@ -276,7 +276,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* [28:28] AssertCke (Assert CKE)
|
||||
* setting this bit causes the DRAM controller to assert the CKE
|
||||
* pins. This bit cannot be used to de-assert the CKE pins
|
||||
* [30:29] reverved
|
||||
* [30:29] reserved
|
||||
* [31:31] EnDramInit (Enable DRAM Initialization)
|
||||
* Setting this bit puts the DRAM controller in a BIOS controlled
|
||||
* DRAM initialization mode. BIOS must clear this bit aster DRAM
|
||||
|
@ -388,7 +388,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 111 = 9 bus clocks
|
||||
* [ 7: 7] Reserved
|
||||
* [ 9: 8] Twtr (Internal DRAM Write-to-Read Command Delay,
|
||||
* minium write-to-read delay when both access the same chip select)
|
||||
* Minimum write-to-read delay when both access the same chip select)
|
||||
* 00 = Reserved
|
||||
* 01 = 1 bus clocks
|
||||
* 10 = 2 bus clocks
|
||||
|
@ -408,7 +408,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 00 = 2 bus clocks ( 1 idle cycle on the bus)
|
||||
* 01 = 3 bus clocks ( 2 idle cycles on the bus)
|
||||
* 10 = 4 bus clocks ( 3 idle cycles on the bus)
|
||||
* 11 = 5 bus clocks ( 4 idel cycles on the bus)
|
||||
* 11 = 5 bus clocks ( 4 idle cycles on the bus)
|
||||
* [17:16] Tref (Refresh Rate)
|
||||
* 00 = Undefined behavior
|
||||
* 01 = Reserved
|
||||
|
@ -487,7 +487,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 010 = 333MHz
|
||||
* 011 = reserved
|
||||
* 1xx = reserved
|
||||
* [ 3: 3] MemClkFreqVal (Memory Clock Freqency Valid)
|
||||
* [ 3: 3] MemClkFreqVal (Memory Clock Frequency Valid)
|
||||
* 1 = BIOS need to set the bit when setting up MemClkFreq to
|
||||
* the proper value
|
||||
* [ 7: 4] MaxAsyncLat ( Maximum Asynchronous Latency)
|
||||
|
@ -523,7 +523,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* MEMCLK cycle
|
||||
* 1 = One additional MEMCLK of setup time is provided on all
|
||||
* DRAM address and control signals except CS, CKE, and ODT;
|
||||
* i.e., these signals are drivern for two MEMCLK cycles
|
||||
* i.e., these signals are driven for two MEMCLK cycles
|
||||
* rather than one
|
||||
* [21:21] Reserved
|
||||
* [22:22] BankSwizzleMode ( Bank Swizzle Mode),
|
||||
|
@ -565,7 +565,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* 1 When bit enables support for mismatched DIMMs when using
|
||||
* 128-bit DRAM interface, the Width128 no effect, only for
|
||||
* AM2 and s1g1
|
||||
* [ 5: 5] DCC_EN ( Dynamica Idle Cycle Counter Enable)
|
||||
* [ 5: 5] DCC_EN ( Dynamic Idle Cycle Counter Enable)
|
||||
* When set to 1, indicates that each entry in the page tables
|
||||
* dynamically adjusts the idle cycle limit based on page
|
||||
* Conflict/Page Miss (PC/PM) traffic
|
||||
|
@ -581,7 +581,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
* [ 9: 9] DramEnabled ( DRAM Enabled)
|
||||
* When Set, this bit indicates that the DRAM is enabled, this
|
||||
* bit is set by hardware after DRAM initialization or on an exit
|
||||
* from self refresh. The DRAM controller is intialized after the
|
||||
* from self refresh. The DRAM controller is initialized after the
|
||||
* hardware-controlled initialization process ( initiated by the
|
||||
* F2 0x90[DramInit]) completes or when the BIOS-controlled
|
||||
* initialization process completes (F2 0x7c(EnDramInit] is
|
||||
|
@ -603,7 +603,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
|
||||
/* DRAM Scrub Control Register
|
||||
* F3:0x58
|
||||
* [ 4: 0] DRAM Scrube Rate
|
||||
* [ 4: 0] DRAM Scrub Rate
|
||||
* [ 7: 5] reserved
|
||||
* [12: 8] L2 Scrub Rate
|
||||
* [15:13] reserved
|
||||
|
@ -646,7 +646,7 @@ static void sdram_set_registers(const struct mem_controller *ctrl, struct sys_in
|
|||
PCI_ADDR(0, 0x18, 3, 0x5C), 0x0000003e, 0x00000000,
|
||||
/* DRAM Scrub Address High Register
|
||||
* F3:0x60
|
||||
* [ 7: 0] DRAM Scrubb Address 39-32
|
||||
* [ 7: 0] DRAM Scrub Address 39-32
|
||||
* [31: 8] Reserved
|
||||
*/
|
||||
PCI_ADDR(0, 0x18, 3, 0x60), 0xffffff00, 0x00000000,
|
||||
|
@ -673,7 +673,7 @@ index:
|
|||
Write:
|
||||
- Write the data to DctOffsetData
|
||||
- Write register num to DctOffset with DctAccessWrite = 1
|
||||
- poll the DctAccessDone untio it = 1
|
||||
- poll the DctAccessDone until it = 1
|
||||
|
||||
*/
|
||||
int i;
|
||||
|
@ -1161,7 +1161,7 @@ static unsigned long interleave_chip_selects(const struct mem_controller *ctrl,
|
|||
csbase_inc <<=1;
|
||||
}
|
||||
|
||||
/* Compute the initial values for csbase and csbask.
|
||||
/* Compute the initial values for csbase and csmask.
|
||||
* In csbase just set the enable bit and the base to zero.
|
||||
* In csmask set the mask bits for the size and page level interleave.
|
||||
*/
|
||||
|
@ -1196,7 +1196,7 @@ static unsigned long order_chip_selects(const struct mem_controller *ctrl)
|
|||
/* Remember which registers we have used in the high 8 bits of tom */
|
||||
tom = 0;
|
||||
for (;;) {
|
||||
/* Find the largest remaining canidate */
|
||||
/* Find the largest remaining candidate */
|
||||
unsigned index, canidate;
|
||||
uint32_t csbase, csmask;
|
||||
unsigned size;
|
||||
|
@ -1220,12 +1220,12 @@ static unsigned long order_chip_selects(const struct mem_controller *ctrl)
|
|||
if (tom & (1 << (index + 24))) {
|
||||
continue;
|
||||
}
|
||||
/* I have a new canidate */
|
||||
/* I have a new candidate */
|
||||
csbase = value;
|
||||
canidate = index;
|
||||
}
|
||||
|
||||
/* See if I have found a new canidate */
|
||||
/* See if I have found a new candidate */
|
||||
if (csbase == 0) {
|
||||
break;
|
||||
}
|
||||
|
@ -1352,7 +1352,7 @@ static long spd_handle_unbuffered_dimms(const struct mem_controller *ctrl,
|
|||
if (is_opteron(ctrl)) {
|
||||
#if 0
|
||||
if ( registered != (meminfo->dimm_mask & ((1<<DIMM_SOCKETS)-1)) ) {
|
||||
meminfo->dimm_mask &= (registered | (registered << DIMM_SOCKETS) ); //disable unbuffed dimm
|
||||
meminfo->dimm_mask &= (registered | (registered << DIMM_SOCKETS) ); //disable unbuffered dimm
|
||||
// die("Mixed buffered and registered dimms not supported");
|
||||
}
|
||||
//By yhlu for debug M2, s1g1 can do dual channel, but it use unbuffer DIMM
|
||||
|
@ -1503,7 +1503,7 @@ static long spd_enable_2channels(const struct mem_controller *ctrl, struct mem_i
|
|||
printk(BIOS_SPEW, "Enabling dual channel memory\n");
|
||||
dcl = pci_read_config32(ctrl->f2, DRAM_CONFIG_LOW);
|
||||
dcl &= ~DCL_BurstLength32; /* 32byte mode may be preferred in platforms that include graphics controllers that generate a lot of 32-bytes system memory accesses
|
||||
32byte mode is not supported when the DRAM interface is 128 bits wides, even 32byte mode is set, system still use 64 byte mode */
|
||||
32byte mode is not supported when the DRAM interface is 128 bits wide, even 32byte mode is set, system still use 64 byte mode */
|
||||
dcl |= DCL_Width128;
|
||||
pci_write_config32(ctrl->f2, DRAM_CONFIG_LOW, dcl);
|
||||
meminfo->is_Width128 = 1;
|
||||
|
@ -1720,8 +1720,8 @@ static int find_optimum_spd_latency(u32 spd_device, unsigned *min_latency, unsig
|
|||
/* Compute the lowest cas latency which can be expressed in this
|
||||
* particular SPD EEPROM. You can store at most settings for 3
|
||||
* contiguous CAS latencies, so by taking the highest CAS
|
||||
* latency maked as supported in the SPD and subtracting 2 you
|
||||
* get the lowest expressable CAS latency. That latency is not
|
||||
* latency marked as supported in the SPD and subtracting 2 you
|
||||
* get the lowest expressible CAS latency. That latency is not
|
||||
* necessarily supported, but a (maybe invalid) entry exists
|
||||
* for it.
|
||||
*/
|
||||
|
@ -2495,7 +2495,7 @@ static void set_max_async_latency(const struct mem_controller *ctrl, const struc
|
|||
dch = pci_read_config32(ctrl->f2, DRAM_CONFIG_HIGH);
|
||||
dch &= ~(DCH_MaxAsyncLat_MASK << DCH_MaxAsyncLat_SHIFT);
|
||||
|
||||
//FIXME: We need to use Max of DqsRcvEnDelay + 6ns here: After trainning and get that from index reg 0x10, 0x13, 0x16, 0x19, 0x30, 0x33, 0x36, 0x39
|
||||
//FIXME: We need to use Max of DqsRcvEnDelay + 6ns here: After training and get that from index reg 0x10, 0x13, 0x16, 0x19, 0x30, 0x33, 0x36, 0x39
|
||||
async_lat = 6 + 6;
|
||||
|
||||
|
||||
|
|
|
@ -585,7 +585,7 @@ static unsigned TrainRcvrEn(const struct mem_controller *ctrl, unsigned Pass, st
|
|||
print_debug_dqs("\tTrainRcvEn51: channel ",channel, 1);
|
||||
|
||||
/* for each rank */
|
||||
/* there are four recriver pairs, loosely associated with CS */
|
||||
/* there are four receiver pairs, loosely associated with CS */
|
||||
for( receiver = 0; (receiver < 8) && (!Errors); receiver+=2)
|
||||
{
|
||||
|
||||
|
@ -664,14 +664,14 @@ static unsigned TrainRcvrEn(const struct mem_controller *ctrl, unsigned Pass, st
|
|||
/* FIXME: 64bit MUX */
|
||||
|
||||
if(is_Width128) {
|
||||
/* Program current Receiver enable delay chaannel b */
|
||||
/* Program current Receiver enable delay channel b */
|
||||
pci_write_config32_index_wait(ctrl->f2, 0x98, index+ 0x20, RcvrEnDly);
|
||||
}
|
||||
|
||||
/* Program the MaxAsyncLat filed with the
|
||||
current DQS receiver enable setting plus 6ns
|
||||
*/
|
||||
/*Porgram MaxAsyncLat to correspond with current delay */
|
||||
/* Program MaxAsyncLat to correspond with current delay */
|
||||
SetMaxAL_RcvrDly(ctrl, RcvrEnDly);
|
||||
|
||||
CurrTest = DQS_FAIL;
|
||||
|
@ -755,7 +755,7 @@ static unsigned TrainRcvrEn(const struct mem_controller *ctrl, unsigned Pass, st
|
|||
}
|
||||
|
||||
if(Pass == DQS_FIRST_PASS) {
|
||||
// We need a better value for DQSPos trainning
|
||||
// We need a better value for DQSPos training
|
||||
RcvrEnDly = RcvrEnDlyRmin /* + RCVREN_MARGIN * T1000/64/50 */;
|
||||
} else {
|
||||
RcvrEnDly = RcvrEnDlyRmin;
|
||||
|
@ -1641,7 +1641,7 @@ static void set_var_mtrr_dqs(
|
|||
}
|
||||
|
||||
|
||||
/* fms: find most sigificant bit set, stolen from Linux Kernel Source. */
|
||||
/* fms: find most significant bit set, stolen from Linux Kernel Source. */
|
||||
static inline unsigned int fms(unsigned int x)
|
||||
{
|
||||
int r;
|
||||
|
@ -1653,7 +1653,7 @@ static inline unsigned int fms(unsigned int x)
|
|||
return r;
|
||||
}
|
||||
|
||||
/* fls: find least sigificant bit set */
|
||||
/* fls: find least significant bit set */
|
||||
static inline unsigned int fls(unsigned int x)
|
||||
{
|
||||
int r;
|
||||
|
|
|
@ -34,12 +34,12 @@
|
|||
#define DCT_STATUS_LOAD_REDUCED 4 /* Load-Reduced DIMMs support */
|
||||
#define DCT_STATUS_OnDimmMirror 24 /* OnDimmMirror support */
|
||||
|
||||
/* PCI Defintions */
|
||||
#define FUN_HT 0 /* Funtion 0 Access */
|
||||
#define FUN_MAP 1 /* Funtion 1 Access */
|
||||
#define FUN_DCT 2 /* Funtion 2 Access */
|
||||
#define FUN_MISC 3 /* Funtion 3 Access */
|
||||
#define FUN_ADD_DCT 0xF /* Funtion 2 Additional Register Access */
|
||||
/* PCI Definitions */
|
||||
#define FUN_HT 0 /* Function 0 Access */
|
||||
#define FUN_MAP 1 /* Function 1 Access */
|
||||
#define FUN_DCT 2 /* Function 2 Access */
|
||||
#define FUN_MISC 3 /* Function 3 Access */
|
||||
#define FUN_ADD_DCT 0xF /* Function 2 Additional Register Access */
|
||||
#define BOTH_DCTS 2 /* The access is independent of DCTs */
|
||||
#define PCI_MIN_LOW 0 /* Lowest possible PCI register location */
|
||||
#define PCI_MAX_HIGH 31 /* Highest possible PCI register location */
|
||||
|
@ -60,7 +60,7 @@
|
|||
#define DRAM_CONT_ADD_ECC_PHASE_REC_CTRL 0x52
|
||||
#define DRAM_CONT_ADD_WRITE_LEV_ERROR_REG 0x53
|
||||
|
||||
/* CPU Register defintions */
|
||||
/* CPU Register definitions */
|
||||
|
||||
/* Register Bit Location */
|
||||
#define DctAccessDone 31
|
||||
|
@ -155,7 +155,7 @@ typedef struct _sDCTStruct
|
|||
u8 ErrStatus[MAX_ERRORS]; /* Minor Error codes for DCT0 and 1 */
|
||||
u8 DimmValid[MAX_TOTAL_DIMMS]; /* Indicates which DIMMs are valid for */
|
||||
/* Total Number of DIMMs(per Node) */
|
||||
u8 WLTotalDelay[MAX_BYTE_LANES];/* Write Levelization Toral Delay */
|
||||
u8 WLTotalDelay[MAX_BYTE_LANES];/* Write Levelization Total Delay */
|
||||
/* per byte lane */
|
||||
u8 MaxDimmsInstalled; /* Max Dimms Installed for current DCT */
|
||||
u8 DimmRanks[MAX_TOTAL_DIMMS]; /* Total Number of Ranks(per Dimm) */
|
||||
|
|
Loading…
Reference in New Issue