soc/amd/stoneyridge: Remove "else" after a return
File ramtop.c has one instance of if()/else where the if tests for top mem in lower 4GiB, and returns just before the "else" statement. These "else" statements are not needed. BUG=b:117648025 TEST=Build and boot grunt. Change-Id: Iba16a416e78dae75a95a11d38179161c5a11b2ad Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/29247 Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
b66757fc58
commit
8c614f2017
1 changed files with 5 additions and 5 deletions
|
@ -63,11 +63,11 @@ void *cbmem_top(void)
|
|||
|
||||
if (!tom.lo)
|
||||
return 0;
|
||||
else
|
||||
/* 8MB alignment to keep MTRR usage low */
|
||||
return (void *)ALIGN_DOWN(restore_top_of_low_cacheable()
|
||||
- CONFIG_SMM_TSEG_SIZE
|
||||
- BERT_REGION_MAX_SIZE, 8*MiB);
|
||||
|
||||
/* 8MB alignment to keep MTRR usage low */
|
||||
return (void *)ALIGN_DOWN(restore_top_of_low_cacheable()
|
||||
- CONFIG_SMM_TSEG_SIZE
|
||||
- BERT_REGION_MAX_SIZE, 8*MiB);
|
||||
}
|
||||
|
||||
static uintptr_t smm_region_start(void)
|
||||
|
|
Loading…
Reference in a new issue