Clean up E7520/7320/7525 quirk printk.
authorDave Jones <davej@redhat.com>
Fri, 6 Jul 2007 18:59:43 +0000 (14:59 -0400)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Sat, 7 Jul 2007 20:53:13 +0000 (13:53 -0700)
The printk level in this printk is bogus, as the previous printk
didn't have a terminating \n resulting in ..

Intel E7520/7320/7525 detected.<6>Disabling irq balancing and affinity

It also never printed a \n at all in the case where we didn't do
the quirk.

Change it to only make noise if it actually does something useful.

Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/i386/kernel/quirks.c

index 9f6ab1789bb05fc81593c47211b16bc45be894e1..6722469c2633841c7ad8f29fb5355e9b7df02b42 100644 (file)
@@ -20,8 +20,6 @@ static void __devinit quirk_intel_irqbalance(struct pci_dev *dev)
        if (rev > 0x9)
                return;
 
-       printk(KERN_INFO "Intel E7520/7320/7525 detected.");
-
        /* enable access to config space*/
        pci_read_config_byte(dev, 0xf4, &config);
        pci_write_config_byte(dev, 0xf4, config|0x2);
@@ -30,7 +28,8 @@ static void __devinit quirk_intel_irqbalance(struct pci_dev *dev)
        raw_pci_ops->read(0, 0, 0x40, 0x4c, 2, &word);
 
        if (!(word & (1 << 13))) {
-               printk(KERN_INFO "Disabling irq balancing and affinity\n");
+               printk(KERN_INFO "Intel E7520/7320/7525 detected. "
+                       "Disabling irq balancing and affinity\n");
 #ifdef CONFIG_IRQBALANCE
                irqbalance_disable("");
 #endif