New patch to review for coreboot: c71730d src/device/root_device.c: Spew out when mainboard

[Paul Menzel]

Paul Menzel ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2612

-gerrit

commit c71730d79f9e989ca3c898acb522b8d6f96917df
Author: Paul Menzel 
Date:   Thu Mar 7 15:19:25 2013 +0100

src/device/root_device.c: Spew out when mainboard

For example change for ASRock E350M1

[…]
        Mainboard E350M1 Enable.
    →   Enable mainboard ASROCK E350M1
        scan_static_bus for Root Device
        […]

Peter’s suggestion [1]: Not sure how this looks like, when dev is printed out.

printk(BIOS_INFO "%s " CONFIG_MAINBOARD_DIR ". dev=0x%p\n", __func__, dev);

TODO
    0. Decide on level: `BIOS_INFO` or `BIOS_SPEW`?
    1. Decide on wording.
    2. Decide if `CONFIG_MAINBOARD_DIR` or `mainboard_name` should be used.
    3. Adapt all boards.
    4. Add test.

[1] http://www.coreboot.org/pipermail/coreboot/2013-February/074621.html

Change-Id: Ie22cfac202f6dd8060860d9600312fe5af499612
    Signed-off-by: Paul Menzel 
---
 src/device/root_device.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/device/root_device.c b/src/device/root_device.c
index 49fa711..f285283 100644
--- a/src/device/root_device.c
+++ b/src/device/root_device.c
@@ -136,6 +136,8 @@ static void root_dev_enable_resources(device_t dev)
  */
 static unsigned int root_dev_scan_bus(device_t root, unsigned int max)
 {
+	printk(BIOS_SPEW, "Enable mainboard %s\n", mainboard_name);
+
    return scan_static_bus(root, max);
 }