GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add name field for device 

The constant field "name" in chip_operations is common to multiple
different devices within a chip and cannot reflect the actual device
as found on the platform.

The intention is that a driver sets dev->name as part of the device
enumeration sequence with the detected hardware type and revision.
The field is for debug print use only.

Change-Id: Ib7bf90ba3c618ad0cb715d80d6a937ceaae0adcf
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/1634
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
Kyösti Mälkki 2012-10-10 23:14:28 +03:00 committed by Stefan Reinauer
parent a74af56dc1
commit 7d54eb8e23
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/devices/device_util.c
View File

@ -230,7 +230,9 @@ const char *dev_path(device_t dev)
const char *dev_name(device_t dev) const char *dev_name(device_t dev)
{ {
if (dev->chip_ops && dev->chip_ops->name) if (dev->name)
return dev->name;
else if (dev->chip_ops && dev->chip_ops->name)
return dev->chip_ops->name; return dev->chip_ops->name;
else else
return "unknown"; return "unknown";

1
src/include/device/device.h
View File

@ -101,6 +101,7 @@ struct device {
struct device_operations *ops; struct device_operations *ops;
#ifndef __PRE_RAM__ #ifndef __PRE_RAM__
const struct chip_operations *chip_ops; const struct chip_operations *chip_ops;
const char *name;
#endif #endif
ROMSTAGE_CONST void *chip_info; ROMSTAGE_CONST void *chip_info;
}; };