Remove some warnings, mainly from format strings which didn't match the

arguments.

Signed-off-by: Myles Watson <mylesgw@gmail.com>
Acked-by: Peter Stuge <peter@stuge.se>


git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3931 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
Myles Watson 2009-02-09 17:52:54 +00:00
parent 4505948fae
commit c4ddbff706
18 changed files with 44 additions and 42 deletions

View File

@ -251,7 +251,7 @@ unsigned long lb_table_fini(struct lb_header *head)
head->table_checksum = compute_ip_checksum(first_rec, head->table_bytes); head->table_checksum = compute_ip_checksum(first_rec, head->table_bytes);
head->header_checksum = 0; head->header_checksum = 0;
head->header_checksum = compute_ip_checksum(head, sizeof(*head)); head->header_checksum = compute_ip_checksum(head, sizeof(*head));
printk_debug("Wrote coreboot table at: %p - %p checksum %lx\n", printk_debug("Wrote coreboot table at: %p - %p checksum %x\n",
head, rec, head->table_checksum); head, rec, head->table_checksum);
return (unsigned long)rec; return (unsigned long)rec;
} }
@ -415,16 +415,16 @@ unsigned long write_coreboot_table(
low_table_end = (unsigned long)head; low_table_end = (unsigned long)head;
} }
printk_debug("Adjust low_table_end from 0x%08x to ", low_table_end); printk_debug("Adjust low_table_end from 0x%08lx to ", low_table_end);
low_table_end += 0xfff; // 4K aligned low_table_end += 0xfff; // 4K aligned
low_table_end &= ~0xfff; low_table_end &= ~0xfff;
printk_debug("0x%08x \n", low_table_end); printk_debug("0x%08lx \n", low_table_end);
/* The Linux kernel assumes this region is reserved */ /* The Linux kernel assumes this region is reserved */
printk_debug("Adjust rom_table_end from 0x%08x to ", rom_table_end); printk_debug("Adjust rom_table_end from 0x%08lx to ", rom_table_end);
rom_table_end += 0xffff; // 64K align rom_table_end += 0xffff; // 64K align
rom_table_end &= ~0xffff; rom_table_end &= ~0xffff;
printk_debug("0x%08x \n", rom_table_end); printk_debug("0x%08lx \n", rom_table_end);
#if (HAVE_OPTION_TABLE == 1) #if (HAVE_OPTION_TABLE == 1)
{ {

View File

@ -91,7 +91,7 @@ unsigned long copy_pirq_routing_table(unsigned long addr)
addr &= ~15; addr &= ~15;
/* This table must be betweeen 0xf0000 & 0x100000 */ /* This table must be betweeen 0xf0000 & 0x100000 */
printk_info("Copying Interrupt Routing Table to 0x%08x... ", addr); printk_info("Copying Interrupt Routing Table to 0x%08lx... ", addr);
memcpy((void *)addr, &intel_irq_routing_table, intel_irq_routing_table.size); memcpy((void *)addr, &intel_irq_routing_table, intel_irq_routing_table.size);
printk_info("done.\n"); printk_info("done.\n");
verify_copy_pirq_routing_table(addr); verify_copy_pirq_routing_table(addr);

View File

@ -92,7 +92,7 @@ struct lb_memory *write_tables(void)
mpc_start &= ~1023; mpc_start &= ~1023;
rom_table_start = mpc_start; rom_table_start = mpc_start;
} }
printk_debug("move mptable from 0x%0x to 0x%0x, size 0x%0x\n", low_table_end, mpc_start, mptable_size); printk_debug("move mptable from 0x%0lx to 0x%0x, size 0x%0x\n", low_table_end, mpc_start, mptable_size);
memcpy((unsigned char *)mpc_start, (unsigned char *)low_table_end, mptable_size); memcpy((unsigned char *)mpc_start, (unsigned char *)low_table_end, mptable_size);
smp_write_floating_table_physaddr(low_table_end - SMP_FLOATING_TABLE_LEN, mpc_start); smp_write_floating_table_physaddr(low_table_end - SMP_FLOATING_TABLE_LEN, mpc_start);
memset((unsigned char *)low_table_end, '\0', mptable_size); memset((unsigned char *)low_table_end, '\0', mptable_size);

View File

@ -225,7 +225,7 @@ void cpu_initialize(void)
info = cpu_info(); info = cpu_info();
printk_notice("Initializing CPU #%d\n", info->index); printk_notice("Initializing CPU #%ld\n", info->index);
cpu = info->cpu; cpu = info->cpu;
if (!cpu) { if (!cpu) {
@ -261,7 +261,7 @@ void cpu_initialize(void)
cpu->ops->init(cpu); cpu->ops->init(cpu);
} }
printk_info("CPU #%d initialized\n", info->index); printk_info("CPU #%ld initialized\n", info->index);
return; return;
} }

View File

@ -481,10 +481,10 @@ void x86_exception(struct eregs *info)
} }
#else /* !CONFIG_GDB_STUB */ #else /* !CONFIG_GDB_STUB */
printk_emerg( printk_emerg(
"Unexpected Exception: %d @ %02x:%08lx - Halting\n" "Unexpected Exception: %d @ %02x:%08x - Halting\n"
"Code: %d eflags: %08lx\n" "Code: %d eflags: %08x\n"
"eax: %08lx ebx: %08lx ecx: %08lx edx: %08lx\n" "eax: %08x ebx: %08x ecx: %08x edx: %08x\n"
"edi: %08lx esi: %08lx ebp: %08lx esp: %08lx\n", "edi: %08x esi: %08x ebp: %08x esp: %08x\n",
info->vector, info->cs, info->eip, info->vector, info->cs, info->eip,
info->error_code, info->eflags, info->error_code, info->eflags,
info->eax, info->ebx, info->ecx, info->edx, info->eax, info->ebx, info->ecx, info->edx,

View File

@ -236,7 +236,7 @@ void smp_write_intsrc_pci_bridge(struct mp_config_table *mc,
if ((child->class >> 16) != PCI_BASE_CLASS_BRIDGE) { if ((child->class >> 16) != PCI_BASE_CLASS_BRIDGE) {
/* pci device */ /* pci device */
printk_debug("route irq: %s %04x\n", dev_path(child)); printk_debug("route irq: %s\n", dev_path(child));
for (i = 0; i < 4; i++) for (i = 0; i < 4; i++)
smp_write_intsrc(mc, irqtype, irqflag, srcbus, (slot<<2)|i, dstapic, dstirq_x[i]); smp_write_intsrc(mc, irqtype, irqflag, srcbus, (slot<<2)|i, dstapic, dstirq_x[i]);
goto next; goto next;
@ -246,7 +246,7 @@ void smp_write_intsrc_pci_bridge(struct mp_config_table *mc,
case PCI_CLASS_BRIDGE_PCI: case PCI_CLASS_BRIDGE_PCI:
case PCI_CLASS_BRIDGE_PCMCIA: case PCI_CLASS_BRIDGE_PCMCIA:
case PCI_CLASS_BRIDGE_CARDBUS: case PCI_CLASS_BRIDGE_CARDBUS:
printk_debug("route irq bridge: %s %04x\n", dev_path(child)); printk_debug("route irq bridge: %s\n", dev_path(child));
smp_write_intsrc_pci_bridge(mc, irqtype, irqflag, child, dstapic, dstirq_x); smp_write_intsrc_pci_bridge(mc, irqtype, irqflag, child, dstapic, dstirq_x);
} }

View File

@ -76,7 +76,7 @@ int verify_ip_checksum(
memcpy(n_desc, buff, 2); memcpy(n_desc, buff, 2);
} }
if (checksum != cb->ip_checksum) { if (checksum != cb->ip_checksum) {
printk_err("Image checksum: %04x != computed checksum: %04x\n", printk_err("Image checksum: %04x != computed checksum: %04lx\n",
cb->ip_checksum, checksum); cb->ip_checksum, checksum);
} }
return checksum == cb->ip_checksum; return checksum == cb->ip_checksum;
@ -500,7 +500,7 @@ static int load_elf_segments(
/* Zero the extra bytes between middle & end */ /* Zero the extra bytes between middle & end */
if (middle < end) { if (middle < end) {
printk_debug("Clearing Segment: addr: 0x%016lx memsz: 0x%016lx\n", printk_debug("Clearing Segment: addr: 0x%016lx memsz: 0x%016lx\n",
(unsigned long)middle, end - middle); (unsigned long)middle, (unsigned long)(end - middle));
/* Zero the extra bytes */ /* Zero the extra bytes */
memset(middle, 0, end - middle); memset(middle, 0, end - middle);
@ -590,7 +590,7 @@ int elfload(struct lb_memory *mem,
/* Reset to booting from this image as late as possible */ /* Reset to booting from this image as late as possible */
boot_successful(); boot_successful();
printk_debug("Jumping to boot code at 0x%x\n", entry); printk_debug("Jumping to boot code at 0x%p\n", entry);
post_code(0xfe); post_code(0xfe);
/* Jump to kernel */ /* Jump to kernel */

View File

@ -231,7 +231,7 @@ static inline void clear_2M_ram(unsigned long basek, struct mtrr_state *mtrr_sta
size = (limitk - basek) << 10; size = (limitk - basek) << 10;
addr = map_2M_page(basek >> 11); addr = map_2M_page(basek >> 11);
if (addr == MAPPING_ERROR) { if (addr == MAPPING_ERROR) {
printk_err("Cannot map page: %x\n", basek >> 11); printk_err("Cannot map page: %lx\n", basek >> 11);
return; return;
} }
@ -312,7 +312,7 @@ static void init_ecc_memory(unsigned node_id)
begink = CONFIG_LB_MEM_TOPK; begink = CONFIG_LB_MEM_TOPK;
} }
printk_debug("Clearing memory %uK - %uK: ", begink, endk); printk_debug("Clearing memory %luK - %luK: ", begink, endk);
/* Save the normal state */ /* Save the normal state */
save_mtrr_state(&mtrr_state); save_mtrr_state(&mtrr_state);
@ -465,7 +465,9 @@ static inline void k8_errata(void)
extern void model_fxx_update_microcode(unsigned cpu_deviceid); extern void model_fxx_update_microcode(unsigned cpu_deviceid);
int init_processor_name(void); int init_processor_name(void);
#if CONFIG_USBDEBUG_DIRECT
static unsigned ehci_debug_addr; static unsigned ehci_debug_addr;
#endif
void model_fxx_init(device_t dev) void model_fxx_init(device_t dev)
{ {

View File

@ -55,7 +55,7 @@ void setup_lapic(void)
LAPIC_DELIVERY_MODE_NMI) LAPIC_DELIVERY_MODE_NMI)
); );
printk_debug(" apic_id: 0x%02x ", lapicid()); printk_debug(" apic_id: 0x%02lx ", lapicid());
#else /* !NEED_LLAPIC */ #else /* !NEED_LLAPIC */
/* Only Pentium Pro and later have those MSR stuff */ /* Only Pentium Pro and later have those MSR stuff */

View File

@ -85,14 +85,14 @@ static int lapic_start_cpu(unsigned long apicid)
send_status = lapic_read(LAPIC_ICR) & LAPIC_ICR_BUSY; send_status = lapic_read(LAPIC_ICR) & LAPIC_ICR_BUSY;
} while (send_status && (timeout++ < 1000)); } while (send_status && (timeout++ < 1000));
if (timeout >= 1000) { if (timeout >= 1000) {
printk_err("CPU %d: First apic write timed out. Disabling\n", printk_err("CPU %ld: First apic write timed out. Disabling\n",
apicid); apicid);
// too bad. // too bad.
printk_err("ESR is 0x%x\n", lapic_read(LAPIC_ESR)); printk_err("ESR is 0x%lx\n", lapic_read(LAPIC_ESR));
if (lapic_read(LAPIC_ESR)) { if (lapic_read(LAPIC_ESR)) {
printk_err("Try to reset ESR\n"); printk_err("Try to reset ESR\n");
lapic_write_around(LAPIC_ESR, 0); lapic_write_around(LAPIC_ESR, 0);
printk_err("ESR is 0x%x\n", lapic_read(LAPIC_ESR)); printk_err("ESR is 0x%lx\n", lapic_read(LAPIC_ESR));
} }
return 0; return 0;
} }
@ -114,7 +114,7 @@ static int lapic_start_cpu(unsigned long apicid)
send_status = lapic_read(LAPIC_ICR) & LAPIC_ICR_BUSY; send_status = lapic_read(LAPIC_ICR) & LAPIC_ICR_BUSY;
} while (send_status && (timeout++ < 1000)); } while (send_status && (timeout++ < 1000));
if (timeout >= 1000) { if (timeout >= 1000) {
printk_err("CPU %d: Second apic write timed out. Disabling\n", printk_err("CPU %ld: Second apic write timed out. Disabling\n",
apicid); apicid);
// too bad. // too bad.
return 0; return 0;
@ -345,8 +345,6 @@ void stop_this_cpu(void)
/* C entry point of secondary cpus */ /* C entry point of secondary cpus */
void secondary_cpu_init(void) void secondary_cpu_init(void)
{ {
unsigned long cpunum;
atomic_inc(&active_cpus); atomic_inc(&active_cpus);
#if SERIAL_CPU_INIT == 1 #if SERIAL_CPU_INIT == 1
#if CONFIG_MAX_CPUS>2 #if CONFIG_MAX_CPUS>2

View File

@ -241,7 +241,7 @@ static unsigned int range_to_mtrr(unsigned int reg,
align = max_align; align = max_align;
} }
sizek = 1 << align; sizek = 1 << align;
printk_debug("Setting variable MTRR %d, base: %4dMB, range: %4dMB, type %s\n", printk_debug("Setting variable MTRR %d, base: %4ldMB, range: %4ldMB, type %s\n",
reg, range_startk >>10, sizek >> 10, reg, range_startk >>10, sizek >> 10,
(type==MTRR_TYPE_UNCACHEABLE)?"UC": (type==MTRR_TYPE_UNCACHEABLE)?"UC":
((type==MTRR_TYPE_WRBACK)?"WB":"Other") ((type==MTRR_TYPE_WRBACK)?"WB":"Other")

View File

@ -467,7 +467,7 @@ void report_resource_stored(device_t dev, struct resource *resource, const char
#endif #endif
} }
printk_debug( printk_debug(
"%s %02x <- [0x%010Lx - 0x%010Lx] size 0x%08Lx gran 0x%02x %s%s%s\n", "%s %02lx <- [0x%010Lx - 0x%010Lx] size 0x%08Lx gran 0x%02x %s%s%s\n",
dev_path(dev), dev_path(dev),
resource->index, resource->index,
base, end, base, end,

View File

@ -207,7 +207,7 @@ struct resource *pci_get_resource(struct device *dev, unsigned long index)
if (moving == 0) { if (moving == 0) {
if (value != 0) { if (value != 0) {
printk_debug( printk_debug(
"%s register %02x(%08x), read-only ignoring it\n", "%s register %02lx(%08lx), read-only ignoring it\n",
dev_path(dev), index, value); dev_path(dev), index, value);
} }
resource->flags = 0; resource->flags = 0;
@ -311,7 +311,7 @@ static void pci_get_rom_resource(struct device *dev, unsigned long index)
if (moving == 0) { if (moving == 0) {
if (value != 0) { if (value != 0) {
printk_debug("%s register %02x(%08x), read-only ignoring it\n", printk_debug("%s register %02lx(%08lx), read-only ignoring it\n",
dev_path(dev), index, value); dev_path(dev), index, value);
} }
resource->flags = 0; resource->flags = 0;
@ -459,7 +459,7 @@ static void pci_set_resource(struct device *dev, struct resource *resource)
/* Make certain the resource has actually been set */ /* Make certain the resource has actually been set */
if (!(resource->flags & IORESOURCE_ASSIGNED)) { if (!(resource->flags & IORESOURCE_ASSIGNED)) {
printk_err("ERROR: %s %02x %s size: 0x%010Lx not assigned\n", printk_err("ERROR: %s %02lx %s size: 0x%010Lx not assigned\n",
dev_path(dev), resource->index, dev_path(dev), resource->index,
resource_type(resource), resource_type(resource),
resource->size); resource->size);
@ -546,7 +546,7 @@ static void pci_set_resource(struct device *dev, struct resource *resource)
else { else {
/* Don't let me think I stored the resource */ /* Don't let me think I stored the resource */
resource->flags &= ~IORESOURCE_STORED; resource->flags &= ~IORESOURCE_STORED;
printk_err("ERROR: invalid resource->index %x\n", printk_err("ERROR: invalid resource->index %lx\n",
resource->index); resource->index);
} }
report_resource_stored(dev, resource, ""); report_resource_stored(dev, resource, "");

View File

@ -45,7 +45,7 @@ struct rom_header * pci_rom_probe(struct device *dev)
return NULL; return NULL;
} }
printk_debug("rom address for %s = %x\n", dev_path(dev), rom_address); printk_debug("rom address for %s = %lx\n", dev_path(dev), rom_address);
if(!dev->on_mainboard) { if(!dev->on_mainboard) {
/* enable expansion ROM address decoding */ /* enable expansion ROM address decoding */
@ -110,12 +110,12 @@ struct rom_header *pci_rom_load(struct device *dev, struct rom_header *rom_heade
extern device_t vga_pri; // the primary vga device, defined in device.c extern device_t vga_pri; // the primary vga device, defined in device.c
if (dev != vga_pri) return NULL; // only one VGA supported if (dev != vga_pri) return NULL; // only one VGA supported
#endif #endif
printk_debug("copying VGA ROM Image from 0x%x to 0x%x, 0x%x bytes\n", printk_debug("copying VGA ROM Image from %p to 0x%x, 0x%x bytes\n",
rom_header, PCI_VGA_RAM_IMAGE_START, rom_size); rom_header, PCI_VGA_RAM_IMAGE_START, rom_size);
memcpy((void *)PCI_VGA_RAM_IMAGE_START, rom_header, rom_size); memcpy((void *)PCI_VGA_RAM_IMAGE_START, rom_header, rom_size);
return (struct rom_header *) (PCI_VGA_RAM_IMAGE_START); return (struct rom_header *) (PCI_VGA_RAM_IMAGE_START);
} else { } else {
printk_debug("copying non-VGA ROM Image from 0x%x to 0x%x, 0x%x bytes\n", printk_debug("copying non-VGA ROM Image from %p to %p, 0x%x bytes\n",
rom_header, pci_ram_image_start, rom_size); rom_header, pci_ram_image_start, rom_size);
memcpy(pci_ram_image_start, rom_header, rom_size); memcpy(pci_ram_image_start, rom_header, rom_size);
pci_ram_image_start += rom_size; pci_ram_image_start += rom_size;

View File

@ -103,7 +103,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_err("ERROR: %s %02x %s size: 0x%010Lx not assigned\n", printk_err("ERROR: %s %02lx %s size: 0x%010Lx not assigned\n",
dev_path(dev), resource->index, dev_path(dev), resource->index,
resource_type(resource), resource_type(resource),
resource->size); resource->size);
@ -121,7 +121,7 @@ static void pnp_set_resource(device_t dev, struct resource *resource)
pnp_set_irq(dev, resource->index, resource->base); pnp_set_irq(dev, resource->index, resource->base);
} }
else { else {
printk_err("ERROR: %s %02x unknown resource type\n", printk_err("ERROR: %s %02lx unknown resource type\n",
dev_path(dev), resource->index); dev_path(dev), resource->index);
return; return;
} }

View File

@ -72,7 +72,7 @@ unsigned long write_pirq_routing_table(unsigned long addr)
addr &= ~15; addr &= ~15;
/* This table must be betweeen 0xf0000 & 0x100000 */ /* This table must be betweeen 0xf0000 & 0x100000 */
printk_info("Writing IRQ routing tables to 0x%x...", addr); printk_info("Writing IRQ routing tables to 0x%lx...", addr);
pirq = (void *)(addr); pirq = (void *)(addr);
v = (uint8_t *)(addr); v = (uint8_t *)(addr);

View File

@ -314,7 +314,7 @@ static void ck804_lpc_enable_childrens_resources(device_t dev)
continue; continue;
base = res->base; base = res->base;
end = resource_end(res); end = resource_end(res);
printk_debug("ck804 lpc decode:%s, base=0x%08x, end=0x%08x\r\n", dev_path(child), base, end); printk_debug("ck804 lpc decode:%s, base=0x%08lx, end=0x%08lx\r\n", dev_path(child), base, end);
switch (base) { switch (base) {
case 0x3f8: // COM1 case 0x3f8: // COM1
reg |= (1 << 0); reg |= (1 << 0);

View File

@ -11,6 +11,7 @@
#include <device/pci_ops.h> #include <device/pci_ops.h>
#include "ck804.h" #include "ck804.h"
#if CK804_SATA_RESET_FOR_ATAPI
static void sata_com_reset(struct device *dev, unsigned reset) static void sata_com_reset(struct device *dev, unsigned reset)
// reset = 1 : reset // reset = 1 : reset
// reset = 0 : clear // reset = 0 : clear
@ -21,7 +22,7 @@ static void sata_com_reset(struct device *dev, unsigned reset)
base = (uint32_t *) pci_read_config32(dev, 0x24); base = (uint32_t *) pci_read_config32(dev, 0x24);
printk_debug("base = %08x\r\n", base); printk_debug("base = %08lx\n", base);
if (reset) { if (reset) {
*(base + 4) = 0xffffffff; *(base + 4) = 0xffffffff;
@ -71,6 +72,7 @@ static void sata_com_reset(struct device *dev, unsigned reset)
printk_debug("loop=%d, *(base+0x44)=%08x\r\n", loop, dword); printk_debug("loop=%d, *(base+0x44)=%08x\r\n", loop, dword);
} }
} }
#endif
static void sata_init(struct device *dev) static void sata_init(struct device *dev)
{ {
@ -132,7 +134,7 @@ static void sata_init(struct device *dev)
dword |= 2; dword |= 2;
pci_write_config32(dev, 0xf8, dword); pci_write_config32(dev, 0xf8, dword);
#if 0 #if CK804_SATA_RESET_FOR_ATAPI
dword = pci_read_config32(dev, 0xac); dword = pci_read_config32(dev, 0xac);
dword &= ~((1 << 13) | (1 << 14)); dword &= ~((1 << 13) | (1 << 14));
dword |= (1 << 13) | (0 << 14); dword |= (1 << 13) | (0 << 14);