cpu/amd/car: Fix checkpatch warnings
Fix line over 80 characters warnings and space after function name warning. Change-Id: Id5a5abaa06f8e285ff58436789318cb9cd3b7ac3 Signed-off-by: Evelyn Huang <evhuang@google.com> Reviewed-on: https://review.coreboot.org/19988 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
parent
208587e0f6
commit
acd02b5b3f
@ -4,7 +4,8 @@
|
|||||||
* original idea yhlu 6.2005 (assembler code)
|
* original idea yhlu 6.2005 (assembler code)
|
||||||
*
|
*
|
||||||
* Copyright (C) 2010 Rudolf Marek <r.marek@assembler.cz>
|
* Copyright (C) 2010 Rudolf Marek <r.marek@assembler.cz>
|
||||||
* Copyright (C) 2015 Timothy Pearson <tpearson@raptorengineeringinc.com>, Raptor Engineering
|
* Copyright (C) 2015 Timothy Pearson <tpearson@raptorengineeringinc.com>,
|
||||||
|
* Raptor Engineering
|
||||||
*
|
*
|
||||||
* This program is free software; you can redistribute it and/or modify
|
* This program is free software; you can redistribute it and/or modify
|
||||||
* it under the terms of the GNU General Public License as published by
|
* it under the terms of the GNU General Public License as published by
|
||||||
@ -31,7 +32,8 @@ static inline __attribute__((always_inline)) uint32_t amd_fam1x_cpu_family(void)
|
|||||||
return family;
|
return family;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline __attribute__((always_inline)) void disable_cache_as_ram(uint8_t skip_sharedc_config)
|
static inline __attribute__((always_inline))
|
||||||
|
void disable_cache_as_ram(uint8_t skip_sharedc_config)
|
||||||
{
|
{
|
||||||
msr_t msr;
|
msr_t msr;
|
||||||
uint32_t family;
|
uint32_t family;
|
||||||
@ -52,12 +54,17 @@ static inline __attribute__((always_inline)) void disable_cache_as_ram(uint8_t s
|
|||||||
#if CONFIG_DCACHE_RAM_SIZE > 0x18000
|
#if CONFIG_DCACHE_RAM_SIZE > 0x18000
|
||||||
wrmsr(MTRR_FIX_4K_D8000, msr);
|
wrmsr(MTRR_FIX_4K_D8000, msr);
|
||||||
#endif
|
#endif
|
||||||
/* disable fixed mtrr from now on, it will be enabled by ramstage again */
|
/* disable fixed mtrr from now on,
|
||||||
|
* it will be enabled by ramstage again
|
||||||
|
*/
|
||||||
msr = rdmsr(SYSCFG_MSR);
|
msr = rdmsr(SYSCFG_MSR);
|
||||||
msr.lo &= ~(SYSCFG_MSR_MtrrFixDramEn | SYSCFG_MSR_MtrrFixDramModEn);
|
msr.lo &= ~(SYSCFG_MSR_MtrrFixDramEn
|
||||||
|
| SYSCFG_MSR_MtrrFixDramModEn);
|
||||||
wrmsr(SYSCFG_MSR, msr);
|
wrmsr(SYSCFG_MSR, msr);
|
||||||
|
|
||||||
/* Set the default memory type and disable fixed and enable variable MTRRs */
|
/* Set the default memory type and
|
||||||
|
* disable fixed and enable variable MTRRs
|
||||||
|
*/
|
||||||
msr.hi = 0;
|
msr.hi = 0;
|
||||||
msr.lo = (1 << 11);
|
msr.lo = (1 << 11);
|
||||||
|
|
||||||
|
@ -48,20 +48,23 @@ static size_t backup_size(void)
|
|||||||
static void memcpy_(void *d, const void *s, size_t len)
|
static void memcpy_(void *d, const void *s, size_t len)
|
||||||
{
|
{
|
||||||
print_car_debug(" Copy [%08x-%08x] to [%08x - %08x] ...",
|
print_car_debug(" Copy [%08x-%08x] to [%08x - %08x] ...",
|
||||||
(uint32_t) s, (uint32_t) (s + len - 1), (uint32_t) d, (uint32_t) (d + len - 1));
|
(uint32_t) s, (uint32_t) (s + len - 1),
|
||||||
|
(uint32_t) d, (uint32_t) (d + len - 1));
|
||||||
memcpy(d, s, len);
|
memcpy(d, s, len);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void memset_(void *d, int val, size_t len)
|
static void memset_(void *d, int val, size_t len)
|
||||||
{
|
{
|
||||||
print_car_debug(" Fill [%08x-%08x] ...", (uint32_t) d, (uint32_t) (d + len - 1));
|
print_car_debug(" Fill [%08x-%08x] ...",
|
||||||
|
(uint32_t) d, (uint32_t) (d + len - 1));
|
||||||
memset(d, val, len);
|
memset(d, val, len);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int memcmp_(void *d, const void *s, size_t len)
|
static int memcmp_(void *d, const void *s, size_t len)
|
||||||
{
|
{
|
||||||
print_car_debug(" Compare [%08x-%08x] with [%08x - %08x] ...",
|
print_car_debug(" Compare [%08x-%08x] with [%08x - %08x] ...",
|
||||||
(uint32_t) s, (uint32_t) (s + len - 1), (uint32_t) d, (uint32_t) (d + len - 1));
|
(uint32_t) s, (uint32_t) (s + len - 1),
|
||||||
|
(uint32_t) d, (uint32_t) (d + len - 1));
|
||||||
return memcmp(d, s, len);
|
return memcmp(d, s, len);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -71,10 +74,13 @@ static void prepare_romstage_ramstack(int s3resume)
|
|||||||
print_car_debug("Prepare CAR migration and stack regions...");
|
print_car_debug("Prepare CAR migration and stack regions...");
|
||||||
|
|
||||||
if (s3resume) {
|
if (s3resume) {
|
||||||
void *resume_backup_memory = acpi_backup_container(CONFIG_RAMBASE, HIGH_MEMORY_SAVE);
|
void *resume_backup_memory =
|
||||||
|
acpi_backup_container(CONFIG_RAMBASE, HIGH_MEMORY_SAVE);
|
||||||
if (resume_backup_memory)
|
if (resume_backup_memory)
|
||||||
memcpy_(resume_backup_memory + HIGH_MEMORY_SAVE - backup_top,
|
memcpy_(resume_backup_memory
|
||||||
(void *)(CONFIG_RAMTOP - backup_top), backup_top);
|
+ HIGH_MEMORY_SAVE - backup_top,
|
||||||
|
(void *)(CONFIG_RAMTOP - backup_top),
|
||||||
|
backup_top);
|
||||||
}
|
}
|
||||||
memset_((void *)(CONFIG_RAMTOP - backup_top), 0, backup_top);
|
memset_((void *)(CONFIG_RAMTOP - backup_top), 0, backup_top);
|
||||||
|
|
||||||
@ -87,7 +93,8 @@ static void prepare_ramstage_region(int s3resume)
|
|||||||
print_car_debug("Prepare ramstage memory region...");
|
print_car_debug("Prepare ramstage memory region...");
|
||||||
|
|
||||||
if (s3resume) {
|
if (s3resume) {
|
||||||
void *resume_backup_memory = acpi_backup_container(CONFIG_RAMBASE, HIGH_MEMORY_SAVE);
|
void *resume_backup_memory =
|
||||||
|
acpi_backup_container(CONFIG_RAMBASE, HIGH_MEMORY_SAVE);
|
||||||
if (resume_backup_memory)
|
if (resume_backup_memory)
|
||||||
memcpy_(resume_backup_memory, (void *) CONFIG_RAMBASE,
|
memcpy_(resume_backup_memory, (void *) CONFIG_RAMBASE,
|
||||||
HIGH_MEMORY_SAVE - backup_top);
|
HIGH_MEMORY_SAVE - backup_top);
|
||||||
@ -118,7 +125,9 @@ void post_cache_as_ram(void)
|
|||||||
* boundary during romstage execution
|
* boundary during romstage execution
|
||||||
*/
|
*/
|
||||||
volatile uint32_t *lower_stack_boundary;
|
volatile uint32_t *lower_stack_boundary;
|
||||||
lower_stack_boundary = (void *)((CONFIG_DCACHE_RAM_BASE + CONFIG_DCACHE_RAM_SIZE) - CONFIG_DCACHE_BSP_STACK_SIZE);
|
lower_stack_boundary =
|
||||||
|
(void *)((CONFIG_DCACHE_RAM_BASE + CONFIG_DCACHE_RAM_SIZE)
|
||||||
|
- CONFIG_DCACHE_BSP_STACK_SIZE);
|
||||||
if ((*lower_stack_boundary) != 0xdeadbeef)
|
if ((*lower_stack_boundary) != 0xdeadbeef)
|
||||||
printk(BIOS_WARNING, "BSP overran lower stack boundary. Undefined behaviour may result!\n");
|
printk(BIOS_WARNING, "BSP overran lower stack boundary. Undefined behaviour may result!\n");
|
||||||
|
|
||||||
@ -159,7 +168,7 @@ void post_cache_as_ram(void)
|
|||||||
/* We do not come back. */
|
/* We do not come back. */
|
||||||
}
|
}
|
||||||
|
|
||||||
void cache_as_ram_new_stack (void)
|
void cache_as_ram_new_stack(void)
|
||||||
{
|
{
|
||||||
print_car_debug("Disabling cache as RAM now\n");
|
print_car_debug("Disabling cache as RAM now\n");
|
||||||
disable_cache_as_ram_bsp();
|
disable_cache_as_ram_bsp();
|
||||||
|
Loading…
x
Reference in New Issue
Block a user