summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorDuncan Laurie <dlaurie@chromium.org>2016-02-09 09:17:56 -0800
committerStefan Reinauer <stefan.reinauer@coreboot.org>2016-02-13 20:03:14 +0100
commitbc2c0a325abca26a7b99e98dfd17a5c7bed2a04f (patch)
tree3e4a9e13f902b2daf04e3c479aa4224c96e78aa6 /src/lib
parent37a54b7f65c31fba63c0b45fcb1ed20a07ea9a66 (diff)
downloadcoreboot-bc2c0a325abca26a7b99e98dfd17a5c7bed2a04f.tar.xz
FMAP: Clean up debug output
Reduce the debug output from FMAP lookups. When we had one or two FMAP lookups in a boot this was not a big deal, but now that we do many lookups it is a lot of unnecessary output duplication. This change reduces these 3 lines: FMAP: area VBLOCK_A found FMAP: offset: 200000 FMAP: size: 65536 bytes To just one line: FMAP: area VBLOCK_A found @ 200000 (65536 bytes) And makes the header output only print once: FMAP: Found "FMAP" version 1.0 at c10000. FMAP: base = 0 size = 1000000 #areas = 29 BUG=chrome-os-partner:40635 BRANCH=glados TEST=boot on chell and enjoy non-truncated memconsole Change-Id: Ib5862b8bfad113a700faae89089557094aa6d499 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Original-Commit-Id: 6890f36536d4ae6fc4988fc8191b0cff4e33e2e6 Original-Change-Id: Ifefee1ab26e6ee406de552880fbbd5b7916fcadd Original-Signed-off-by: Duncan Laurie <dlaurie@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/326887 Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: https://review.coreboot.org/13695 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/fmap.c19
1 files changed, 12 insertions, 7 deletions
diff --git a/src/lib/fmap.c b/src/lib/fmap.c
index 028bc2538b..54e54ba6b7 100644
--- a/src/lib/fmap.c
+++ b/src/lib/fmap.c
@@ -13,6 +13,7 @@
* GNU General Public License for more details.
*/
+#include <arch/early_variables.h>
#include <boot_device.h>
#include <console/console.h>
#include <fmap.h>
@@ -24,6 +25,8 @@
* See http://code.google.com/p/flashmap/ for more information on FMAP.
*/
+static int fmap_print_once CAR_GLOBAL;
+
int find_fmap_directory(struct region_device *fmrd)
{
const struct region_device *boot;
@@ -50,10 +53,13 @@ int find_fmap_directory(struct region_device *fmrd)
return -1;
}
- printk(BIOS_DEBUG, "FMAP: Found \"%s\" version %d.%d at %zx.\n",
- fmap->name, fmap->ver_major, fmap->ver_minor, offset);
- printk(BIOS_DEBUG, "FMAP: base = %llx size = %x #areas = %d\n",
- (long long)fmap->base, fmap->size, fmap->nareas);
+ if (!car_get_var(fmap_print_once)) {
+ printk(BIOS_DEBUG, "FMAP: Found \"%s\" version %d.%d at %zx.\n",
+ fmap->name, fmap->ver_major, fmap->ver_minor, offset);
+ printk(BIOS_DEBUG, "FMAP: base = %llx size = %x #areas = %d\n",
+ (long long)fmap->base, fmap->size, fmap->nareas);
+ car_set_var(fmap_print_once, 1);
+ }
fmap_size += fmap->nareas * sizeof(struct fmap_area);
@@ -97,9 +103,8 @@ int fmap_locate_area(const char *name, struct region *ar)
continue;
}
- printk(BIOS_DEBUG, "FMAP: area %s found\n", name);
- printk(BIOS_DEBUG, "FMAP: offset: %x\n", area->offset);
- printk(BIOS_DEBUG, "FMAP: size: %d bytes\n", area->size);
+ printk(BIOS_DEBUG, "FMAP: area %s found @ %x (%d bytes)\n",
+ name, area->offset, area->size);
ar->offset = area->offset;
ar->size = area->size;