Kyösti Mälkki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34878 )
Change subject: amd/picasso: Rename ramtop.c to memmap.c ......................................................................
amd/picasso: Rename ramtop.c to memmap.c
Use a name consistent with the more recent soc/intel.
Change-Id: I491e609bed00dc79c628b321c74ad7f4cc31b5fe Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com --- M src/soc/amd/picasso/Makefile.inc R src/soc/amd/picasso/memmap.c 2 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/34878/1
diff --git a/src/soc/amd/picasso/Makefile.inc b/src/soc/amd/picasso/Makefile.inc index 7f37192..67d8904 100644 --- a/src/soc/amd/picasso/Makefile.inc +++ b/src/soc/amd/picasso/Makefile.inc @@ -44,7 +44,7 @@ romstage-y += pmutil.c romstage-y += reset.c romstage-y += smbus.c -romstage-y += ramtop.c +romstage-y += memmap.c romstage-$(CONFIG_PICASSO_UART) += uart.c romstage-y += tsc_freq.c romstage-y += southbridge.c @@ -62,7 +62,7 @@
postcar-y += monotonic_timer.c postcar-$(CONFIG_PICASSO_UART) += uart.c -postcar-y += ramtop.c +postcar-y += memmap.c postcar-$(CONFIG_VBOOT_MEASURED_BOOT) += i2c.c postcar-y += tsc_freq.c
@@ -80,7 +80,7 @@ ramstage-y += sata.c ramstage-y += sm.c ramstage-y += smbus.c -ramstage-y += ramtop.c +ramstage-y += memmap.c ramstage-$(CONFIG_HAVE_SMI_HANDLER) += smi.c ramstage-$(CONFIG_HAVE_SMI_HANDLER) += smi_util.c ramstage-$(CONFIG_PICASSO_UART) += uart.c diff --git a/src/soc/amd/picasso/ramtop.c b/src/soc/amd/picasso/memmap.c similarity index 100% rename from src/soc/amd/picasso/ramtop.c rename to src/soc/amd/picasso/memmap.c
Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34878 )
Change subject: amd/picasso: Rename ramtop.c to memmap.c ......................................................................
Patch Set 1: Code-Review+2
I think this was my original preference and someone else liked ramtop better, so thanks.
Kyösti Mälkki has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/34878 )
Change subject: amd/picasso: Rename ramtop.c to memmap.c ......................................................................
amd/picasso: Rename ramtop.c to memmap.c
Use a name consistent with the more recent soc/intel.
Change-Id: I491e609bed00dc79c628b321c74ad7f4cc31b5fe Signed-off-by: Kyösti Mälkki kyosti.malkki@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/34878 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/soc/amd/picasso/Makefile.inc R src/soc/amd/picasso/memmap.c 2 files changed, 3 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Marshall Dawson: Looks good to me, approved
diff --git a/src/soc/amd/picasso/Makefile.inc b/src/soc/amd/picasso/Makefile.inc index 7f37192..67d8904 100644 --- a/src/soc/amd/picasso/Makefile.inc +++ b/src/soc/amd/picasso/Makefile.inc @@ -44,7 +44,7 @@ romstage-y += pmutil.c romstage-y += reset.c romstage-y += smbus.c -romstage-y += ramtop.c +romstage-y += memmap.c romstage-$(CONFIG_PICASSO_UART) += uart.c romstage-y += tsc_freq.c romstage-y += southbridge.c @@ -62,7 +62,7 @@
postcar-y += monotonic_timer.c postcar-$(CONFIG_PICASSO_UART) += uart.c -postcar-y += ramtop.c +postcar-y += memmap.c postcar-$(CONFIG_VBOOT_MEASURED_BOOT) += i2c.c postcar-y += tsc_freq.c
@@ -80,7 +80,7 @@ ramstage-y += sata.c ramstage-y += sm.c ramstage-y += smbus.c -ramstage-y += ramtop.c +ramstage-y += memmap.c ramstage-$(CONFIG_HAVE_SMI_HANDLER) += smi.c ramstage-$(CONFIG_HAVE_SMI_HANDLER) += smi_util.c ramstage-$(CONFIG_PICASSO_UART) += uart.c diff --git a/src/soc/amd/picasso/ramtop.c b/src/soc/amd/picasso/memmap.c similarity index 100% rename from src/soc/amd/picasso/ramtop.c rename to src/soc/amd/picasso/memmap.c