On Tuesday 02 March 2010 04:45:01 pm Chase Douglas wrote: > On Tue, 2010-03-02 at 16:14 -0700, Bjorn Helgaas wrote: > > On Tuesday 02 March 2010 02:09:12 pm Chase Douglas wrote: > > > - printk("%sACPI: %s resource %s [0x%llx-0x%llx]" > > > + printk(KERN_INFO "ACPI: %s resource %s [0x%llx-0x%llx]" > > > " conflicts with ACPI region %s" > > > " [0x%llx-0x%llx]\n", > > > - acpi_enforce_resources == ENFORCE_RESOURCES_LAX > > > - ? KERN_WARNING : KERN_ERR, > > > ioport ? "I/O" : "Memory", res->name, > > > (long long) res->start, (long long) res->end, > > > res_list_elem->name, > > > > Since you're touching this anyway, can you convert this to use %pR? > > I see the potential usage of %p, but is the 'R' a typo? I haven't found > it in man 3 fprintf nor in a google search. Nope, see this description of the %p<x> kernel extensions: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=lib/vsprintf.c;hb=51c24aaacaea90c8e87f1dec75a2ac7622b593f8#l915 There are several examples already in the tree, including drivers/pci/setup-bus.c. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html