[coreboot-gerrit] Patch set updated for coreboot: 3abf888 device/device.c: Improve output in `init_dev()`

[Paul Menzel (paulepanter@users.sourceforge.net)]

Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10455

-gerrit

commit 3abf8885cedb39789246e68005a121ca0f67d3f9
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sun Jun 7 20:28:17 2015 +0200

    device/device.c: Improve output in `init_dev()`
    
    Since commit 05294292 (device tree: track init times) there are two
    lines printed for each init() call of a device, when
    `HAVE_MONOTONIC_TIMER` is selected.
    
    	[…]
    	CPU_CLUSTER: 0 init 12708 usecs
    	DOMAIN: 0000 init
    	DOMAIN: 0000 init 1 usecs
    	PCI: 00:00.0 init
    	Northbridge init
    	PCI: 00:00.0 init 2 usecs
    	PCI: 00:01.0 init
    	PCI: 00:01.0 init 1 usecs
    	PCI: 00:01.1 init
    	PCI: 00:01.1 init 1 usecs
    	PCI: 00:11.0 init
    	PCI: 00:11.0 init 1 usecs
    	PCI: 00:14.0 init
    	PCI: 00:14.0 init 1 usecs
    	PCI: 00:14.3 init
    	SB800 - Late.c - lpc_init - Start.
    	RTC Init
    	RTC: coreboot checksum invalid
    	SB800 - Late.c - lpc_init - End.
    	[…]
    
    Improve the output by changing the wording to.
    
    	%s init ...\n
    	init()
    	%s init finished in %ld usecs\n
    
    Note, that `%s init ... done in %ld usecs` is not possible as the
    function `init()` can also print messages.
    
    Change-Id: I7132cd650911dba680f060d6073a5a09c879b24c
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/device/device.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/device/device.c b/src/device/device.c
index 6bdeae1..a768ff7 100644
--- a/src/device/device.c
+++ b/src/device/device.c
@@ -1144,11 +1144,11 @@ static void init_dev(struct device *dev)
 			       dev_path(dev->bus->dev), dev->bus->link_num);
 		}
 
-		printk(BIOS_DEBUG, "%s init\n", dev_path(dev));
+		printk(BIOS_DEBUG, "%s init ...\n", dev_path(dev));
 		dev->initialized = 1;
 		dev->ops->init(dev);
 #if CONFIG_HAVE_MONOTONIC_TIMER
-		printk(BIOS_DEBUG, "%s init %ld usecs\n", dev_path(dev),
+		printk(BIOS_DEBUG, "%s init finished in %ld usecs\n", dev_path(dev),
 			stopwatch_duration_usecs(&sw));
 #endif
 	}