From 4644061aaccb77edefa4c8369de62dabdd5bdaf1 Mon Sep 17 00:00:00 2001 From: Andrew Morton Date: Thu, 29 Nov 2012 14:18:37 +1100 Subject: [PATCH] drivers-firmware-dmi_scanc-fetch-dmi-version-from-smbios-if-it-exists-checkpatch-fixes WARNING: Prefer pr_info(... to printk(KERN_INFO, ... #56: FILE: drivers/firmware/dmi_scan.c:426: + printk(KERN_INFO "SMBIOS %d.%d present.\n", WARNING: Prefer pr_info(... to printk(KERN_INFO, ... #61: FILE: drivers/firmware/dmi_scan.c:431: + printk(KERN_INFO "Legacy DMI %d.%d present.\n", WARNING: Prefer pr_debug(... to printk(KERN_DEBUG, ... #85: FILE: drivers/firmware/dmi_scan.c:455: + printk(KERN_DEBUG "SMBIOS version fixup(2.%d->2.%d)\n", WARNING: Prefer pr_debug(... to printk(KERN_DEBUG, ... #90: FILE: drivers/firmware/dmi_scan.c:460: + printk(KERN_DEBUG "SMBIOS version fixup(2.%d->2.%d)\n", total: 0 errors, 4 warnings, 104 lines checked ./patches/drivers-firmware-dmi_scanc-fetch-dmi-version-from-smbios-if-it-exists.patch has style problems, please review. If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Feng Jin Cc: Jean Delvare Cc: Zhenzhong Duan Signed-off-by: Andrew Morton --- drivers/firmware/dmi_scan.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware/dmi_scan.c index 111f3bab3a32..fd3ae6290d71 100644 --- a/drivers/firmware/dmi_scan.c +++ b/drivers/firmware/dmi_scan.c @@ -423,12 +423,12 @@ static int __init dmi_present(const char __iomem *p) if (dmi_walk_early(dmi_decode) == 0) { if (dmi_ver) - printk(KERN_INFO "SMBIOS %d.%d present.\n", + pr_info("SMBIOS %d.%d present.\n", dmi_ver >> 8, dmi_ver & 0xFF); else { dmi_ver = (buf[14] & 0xF0) << 4 | (buf[14] & 0x0F); - printk(KERN_INFO "Legacy DMI %d.%d present.\n", + pr_info("Legacy DMI %d.%d present.\n", dmi_ver >> 8, dmi_ver & 0xFF); } dmi_dump_ids(); @@ -452,13 +452,12 @@ static int __init smbios_present(const char __iomem *p) switch (dmi_ver) { case 0x021F: case 0x0221: - printk(KERN_DEBUG "SMBIOS version fixup(2.%d->2.%d)\n", + pr_debug("SMBIOS version fixup(2.%d->2.%d)\n", dmi_ver & 0xFF, 3); dmi_ver = 0x0203; break; case 0x0233: - printk(KERN_DEBUG "SMBIOS version fixup(2.%d->2.%d)\n", - 51, 6); + pr_debug("SMBIOS version fixup(2.%d->2.%d)\n", 51, 6); dmi_ver = 0x0206; break; } -- 2.39.5