[coreboot-gerrit] Change in coreboot[master]: amdblocks/biosram: Do small reformatting

[Michał Żygowski (Code Review)]

Michał Żygowski has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38146 )

Change subject: amdblocks/biosram: Do small reformatting
......................................................................

amdblocks/biosram: Do small reformatting

Remove two blank lines and reorder functions by read/write sizes.

Change-Id: I7bd6ed44546d49b65135a98e424a5669d90f2867
Signed-off-by: Marshall Dawson 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/38146
Reviewed-by: Paul Menzel 
Reviewed-by: Kyösti Mälkki 
Reviewed-by: Michał Żygowski 
Tested-by: build bot (Jenkins) 
---
M src/soc/amd/common/block/acpimmio/biosram.c
1 file changed, 6 insertions(+), 8 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Kyösti Mälkki: Looks good to me, approved
  Paul Menzel: Looks good to me, but someone else must approve
  Michał Żygowski: Looks good to me, approved

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);