commit c790850ebe81d5b9fd7018b41e2fd7bda79db8c1
author Subrata Banik <subrata.banik@intel.com> Thu Mar 23 11:52:59 2017 +0530
committer Aaron Durbin <adurbin@chromium.org> Fri Mar 24 14:48:16 2017 +0100
tree a1cecdab27f0c1b9098dc13825bc5aa97440593b
parent 6e260fc873d38f2b836402e58c8660160b159e60

soc/intel/apollolake: Remove unused CAR_GLOBAL variable

Also move all local variable declaration at starting of function
block.

Change-Id: I774485a23b4b7d96a8dbd837da45553251dff3b0
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/18949
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>

src/soc/intel/apollolake/mmap_boot.c

diff --git a/src/soc/intel/apollolake/mmap_boot.c b/src/soc/intel/apollolake/mmap_boot.c
index a92fbb3..50d0ff0 100644
--- a/src/soc/intel/apollolake/mmap_boot.c
+++ b/src/soc/intel/apollolake/mmap_boot.c
@@ -58,7 +58,6 @@
  *
  */
 
-static size_t bios_start CAR_GLOBAL;
 static size_t bios_size CAR_GLOBAL;
 
 static struct mem_region_device shadow_dev CAR_GLOBAL;
@@ -66,7 +65,8 @@
 
 static void bios_mmap_init(void)
 {
-	size_t size;
+	size_t size, start, bios_end, bios_mapped_size;
+	uintptr_t base;
 
 	size = car_get_var(bios_size);
 
@@ -74,9 +74,6 @@
 	if (size != 0)
 		return;
 
-	size_t start, bios_end, bios_mapped_size;
-	uintptr_t base;
-
 	/*
 	 * BIOS_BFPREG provides info about BIOS Flash Primary Region
 	 * Base and Limit.
@@ -110,7 +107,6 @@
 				 start, bios_mapped_size,
 				 CONFIG_ROM_SIZE);
 
-	car_set_var(bios_start, start);
 	car_set_var(bios_size, size);
 }