From 554f3e75cf4ad74419dbfa9fe41d00cdf6955327 Mon Sep 17 00:00:00 2001 From: Andrew Morton Date: Wed, 26 Sep 2012 11:34:31 +1000 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