Marshall Dawson has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38146 )
Change subject: amdblocks/biosram: Do small reformatting ......................................................................
amdblocks/biosram: Do small reformatting
Remove two blank lines. Reorder functions by read/write sizes.
Change-Id: I7bd6ed44546d49b65135a98e424a5669d90f2867 Signed-off-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/soc/amd/common/block/acpimmio/biosram.c 1 file changed, 6 insertions(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/46/38146/1
diff --git a/src/soc/amd/common/block/acpimmio/biosram.c b/src/soc/amd/common/block/acpimmio/biosram.c index 814fdf3..1b1fcad 100644 --- a/src/soc/amd/common/block/acpimmio/biosram.c +++ b/src/soc/amd/common/block/acpimmio/biosram.c @@ -38,12 +38,6 @@ return (biosram_read8(reg + sizeof(uint8_t)) << 8 | biosram_read8(reg)); }
-static uint32_t biosram_read32(uint8_t reg) -{ - uint32_t value = biosram_read16(reg + sizeof(uint16_t)) << 16; - return value | biosram_read16(reg); -} - static void biosram_write16(uint8_t reg, uint16_t value) { biosram_write8(reg, value & 0xff); @@ -51,6 +45,12 @@ biosram_write8(reg + sizeof(uint8_t), value & 0xff); }
+static uint32_t biosram_read32(uint8_t reg) +{ + uint32_t value = biosram_read16(reg + sizeof(uint16_t)) << 16; + return value | biosram_read16(reg); +} + static void biosram_write32(uint8_t reg, uint32_t value) { biosram_write16(reg, value & 0xffff); @@ -58,7 +58,6 @@ biosram_write16(reg + sizeof(uint16_t), value & 0xffff); }
- /* Access to BIOSRAM is only allowed through the abstractions below. */
void *get_ap_entry_ptr(void) @@ -71,7 +70,6 @@ biosram_write32(BIOSRAM_AP_ENTRY, (uintptr_t)entry); }
- void backup_top_of_low_cacheable(uintptr_t ramtop) { biosram_write32(BIOSRAM_CBMEM_TOP, ramtop);