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

printf: Remove some L modifier uses 

We use the L modifier in a non-standard way (for
long long instead of long double, which we have no
business with).
clang complains, to reduce its use, to make
emulation/qemu-x86 happier.
Long term, we should consider eliminating public uses
of 'L' (but internal use in vtxprintf to denote
long long is fine)

Change-Id: If9a17d9ae9925cdc8736445e7d5eedc59c7028c6
Signed-off-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-on: http://review.coreboot.org/781
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Patrick Georgi 2012-03-11 19:31:03 +01:00
parent ccee6256b4
commit 5161509132
3 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/devices/device.c
View File

@ -322,7 +322,7 @@ static void compute_resources(struct bus *bus, struct resource *bridge,
(resource->flags & IORESOURCE_ASSIGNED)) { (resource->flags & IORESOURCE_ASSIGNED)) {
printk(BIOS_ERR, printk(BIOS_ERR,
"Resource limit looks wrong! (no APIC?)\n"); "Resource limit looks wrong! (no APIC?)\n");
printk(BIOS_ERR, "%s %02lx limit %08Lx\n", printk(BIOS_ERR, "%s %02lx limit %08llx\n",
dev_path(dev), resource->index, resource->limit); dev_path(dev), resource->index, resource->limit);
} }
@ -627,7 +627,7 @@ static void avoid_fixed_resources(struct device *dev)
for (res = dev->resource_list; res; res = res->next) { for (res = dev->resource_list; res; res = res->next) {
if ((res->flags & IORESOURCE_FIXED)) if ((res->flags & IORESOURCE_FIXED))
continue; continue;
printk(BIOS_SPEW, "%s:@%s %02lx limit %08Lx\n", __func__, printk(BIOS_SPEW, "%s:@%s %02lx limit %08llx\n", __func__,
dev_path(dev), res->index, res->limit); dev_path(dev), res->index, res->limit);
if ((res->flags & MEM_MASK) == PREF_TYPE && if ((res->flags & MEM_MASK) == PREF_TYPE &&
(res->limit < limits.pref.limit)) (res->limit < limits.pref.limit))
@ -660,9 +660,9 @@ static void avoid_fixed_resources(struct device *dev)
else else
continue; continue;
printk(BIOS_SPEW, "%s2: %s@%02lx limit %08Lx\n", __func__, printk(BIOS_SPEW, "%s2: %s@%02lx limit %08llx\n", __func__,
dev_path(dev), res->index, res->limit); dev_path(dev), res->index, res->limit);
printk(BIOS_SPEW, "\tlim->base %08Lx lim->limit %08Lx\n", printk(BIOS_SPEW, "\tlim->base %08llx lim->limit %08llx\n",
lim->base, lim->limit); lim->base, lim->limit);
/* Is the resource outside the limits? */ /* Is the resource outside the limits? */

4
src/devices/device_util.c
View File

@ -571,7 +571,7 @@ void report_resource_stored(device_t dev, struct resource *resource,
sprintf(buf, "bus %02x ", dev->link_list->secondary); sprintf(buf, "bus %02x ", dev->link_list->secondary);
#endif #endif
} }
printk(BIOS_DEBUG, "%s %02lx <- [0x%010Lx - 0x%010Lx] size 0x%08Lx " printk(BIOS_DEBUG, "%s %02lx <- [0x%010llx - 0x%010llx] size 0x%08llx "
"gran 0x%02x %s%s%s\n", dev_path(dev), resource->index, "gran 0x%02x %s%s%s\n", dev_path(dev), resource->index,
base, end, resource->size, resource->gran, buf, base, end, resource->size, resource->gran, buf,
resource_type(resource), comment); resource_type(resource), comment);
@ -786,7 +786,7 @@ void show_one_resource(int debug_level, struct device *dev,
*/ */
do_printk(debug_level, "%s %02lx <- [0x%010llx - 0x%010llx] " do_printk(debug_level, "%s %02lx <- [0x%010llx - 0x%010llx] "
"size 0x%08Lx gran 0x%02x %s%s%s\n", dev_path(dev), "size 0x%08llx gran 0x%02x %s%s%s\n", dev_path(dev),
resource->index, base, end, resource->size, resource->gran, resource->index, base, end, resource->size, resource->gran,
buf, resource_type(resource), comment); buf, resource_type(resource), comment);
} }

2
src/devices/pnp_device.c
View File

@ -107,7 +107,7 @@ void pnp_read_resources(device_t dev)
static void pnp_set_resource(device_t dev, struct resource *resource) static void pnp_set_resource(device_t dev, struct resource *resource)
{ {
if (!(resource->flags & IORESOURCE_ASSIGNED)) { if (!(resource->flags & IORESOURCE_ASSIGNED)) {
printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010Lx " printk(BIOS_ERR, "ERROR: %s %02lx %s size: 0x%010llx "
"not assigned\n", dev_path(dev), resource->index, "not assigned\n", dev_path(dev), resource->index,
resource_type(resource), resource->size); resource_type(resource), resource->size);
return; return;