Commit 1ca28218 authored by Joe Perches's avatar Joe Perches Committed by Jean Delvare

hwmon: (w83781d) Use pr_fmt and pr_<level>

Added #define pr_fmt KBUILD_MODNAME ": " fmt
Converted printks to pr_<level>
Coalesced any long formats
Removed prefixes from formats

[JD: Optimize repeated debug messages]
Signed-off-by: default avatarJoe Perches <joe@perches.com>
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
parent 5e24e0c3
...@@ -33,6 +33,8 @@ ...@@ -33,6 +33,8 @@
*/ */
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#include <linux/module.h> #include <linux/module.h>
#include <linux/init.h> #include <linux/init.h>
#include <linux/slab.h> #include <linux/slab.h>
...@@ -1798,8 +1800,7 @@ w83781d_isa_found(unsigned short address) ...@@ -1798,8 +1800,7 @@ w83781d_isa_found(unsigned short address)
* individually for the probing phase. */ * individually for the probing phase. */
for (port = address; port < address + W83781D_EXTENT; port++) { for (port = address; port < address + W83781D_EXTENT; port++) {
if (!request_region(port, 1, "w83781d")) { if (!request_region(port, 1, "w83781d")) {
pr_debug("w83781d: Failed to request port 0x%x\n", pr_debug("Failed to request port 0x%x\n", port);
port);
goto release; goto release;
} }
} }
...@@ -1811,7 +1812,7 @@ w83781d_isa_found(unsigned short address) ...@@ -1811,7 +1812,7 @@ w83781d_isa_found(unsigned short address)
if (inb_p(address + 2) != val if (inb_p(address + 2) != val
|| inb_p(address + 3) != val || inb_p(address + 3) != val
|| inb_p(address + 7) != val) { || inb_p(address + 7) != val) {
pr_debug("w83781d: Detection failed at step 1\n"); pr_debug("Detection failed at step %d\n", 1);
goto release; goto release;
} }
#undef REALLY_SLOW_IO #undef REALLY_SLOW_IO
...@@ -1820,14 +1821,14 @@ w83781d_isa_found(unsigned short address) ...@@ -1820,14 +1821,14 @@ w83781d_isa_found(unsigned short address)
MSB (busy flag) should be clear initially, set after the write. */ MSB (busy flag) should be clear initially, set after the write. */
save = inb_p(address + W83781D_ADDR_REG_OFFSET); save = inb_p(address + W83781D_ADDR_REG_OFFSET);
if (save & 0x80) { if (save & 0x80) {
pr_debug("w83781d: Detection failed at step 2\n"); pr_debug("Detection failed at step %d\n", 2);
goto release; goto release;
} }
val = ~save & 0x7f; val = ~save & 0x7f;
outb_p(val, address + W83781D_ADDR_REG_OFFSET); outb_p(val, address + W83781D_ADDR_REG_OFFSET);
if (inb_p(address + W83781D_ADDR_REG_OFFSET) != (val | 0x80)) { if (inb_p(address + W83781D_ADDR_REG_OFFSET) != (val | 0x80)) {
outb_p(save, address + W83781D_ADDR_REG_OFFSET); outb_p(save, address + W83781D_ADDR_REG_OFFSET);
pr_debug("w83781d: Detection failed at step 3\n"); pr_debug("Detection failed at step %d\n", 3);
goto release; goto release;
} }
...@@ -1835,7 +1836,7 @@ w83781d_isa_found(unsigned short address) ...@@ -1835,7 +1836,7 @@ w83781d_isa_found(unsigned short address)
outb_p(W83781D_REG_CONFIG, address + W83781D_ADDR_REG_OFFSET); outb_p(W83781D_REG_CONFIG, address + W83781D_ADDR_REG_OFFSET);
val = inb_p(address + W83781D_DATA_REG_OFFSET); val = inb_p(address + W83781D_DATA_REG_OFFSET);
if (val & 0x80) { if (val & 0x80) {
pr_debug("w83781d: Detection failed at step 4\n"); pr_debug("Detection failed at step %d\n", 4);
goto release; goto release;
} }
outb_p(W83781D_REG_BANK, address + W83781D_ADDR_REG_OFFSET); outb_p(W83781D_REG_BANK, address + W83781D_ADDR_REG_OFFSET);
...@@ -1844,19 +1845,19 @@ w83781d_isa_found(unsigned short address) ...@@ -1844,19 +1845,19 @@ w83781d_isa_found(unsigned short address)
val = inb_p(address + W83781D_DATA_REG_OFFSET); val = inb_p(address + W83781D_DATA_REG_OFFSET);
if ((!(save & 0x80) && (val != 0xa3)) if ((!(save & 0x80) && (val != 0xa3))
|| ((save & 0x80) && (val != 0x5c))) { || ((save & 0x80) && (val != 0x5c))) {
pr_debug("w83781d: Detection failed at step 5\n"); pr_debug("Detection failed at step %d\n", 5);
goto release; goto release;
} }
outb_p(W83781D_REG_I2C_ADDR, address + W83781D_ADDR_REG_OFFSET); outb_p(W83781D_REG_I2C_ADDR, address + W83781D_ADDR_REG_OFFSET);
val = inb_p(address + W83781D_DATA_REG_OFFSET); val = inb_p(address + W83781D_DATA_REG_OFFSET);
if (val < 0x03 || val > 0x77) { /* Not a valid I2C address */ if (val < 0x03 || val > 0x77) { /* Not a valid I2C address */
pr_debug("w83781d: Detection failed at step 6\n"); pr_debug("Detection failed at step %d\n", 6);
goto release; goto release;
} }
/* The busy flag should be clear again */ /* The busy flag should be clear again */
if (inb_p(address + W83781D_ADDR_REG_OFFSET) & 0x80) { if (inb_p(address + W83781D_ADDR_REG_OFFSET) & 0x80) {
pr_debug("w83781d: Detection failed at step 7\n"); pr_debug("Detection failed at step %d\n", 7);
goto release; goto release;
} }
...@@ -1871,7 +1872,7 @@ w83781d_isa_found(unsigned short address) ...@@ -1871,7 +1872,7 @@ w83781d_isa_found(unsigned short address)
found = 1; found = 1;
if (found) if (found)
pr_info("w83781d: Found a %s chip at %#x\n", pr_info("Found a %s chip at %#x\n",
val == 0x30 ? "W83782D" : "W83781D", (int)address); val == 0x30 ? "W83782D" : "W83781D", (int)address);
release: release:
...@@ -1894,21 +1895,19 @@ w83781d_isa_device_add(unsigned short address) ...@@ -1894,21 +1895,19 @@ w83781d_isa_device_add(unsigned short address)
pdev = platform_device_alloc("w83781d", address); pdev = platform_device_alloc("w83781d", address);
if (!pdev) { if (!pdev) {
err = -ENOMEM; err = -ENOMEM;
printk(KERN_ERR "w83781d: Device allocation failed\n"); pr_err("Device allocation failed\n");
goto exit; goto exit;
} }
err = platform_device_add_resources(pdev, &res, 1); err = platform_device_add_resources(pdev, &res, 1);
if (err) { if (err) {
printk(KERN_ERR "w83781d: Device resource addition failed " pr_err("Device resource addition failed (%d)\n", err);
"(%d)\n", err);
goto exit_device_put; goto exit_device_put;
} }
err = platform_device_add(pdev); err = platform_device_add(pdev);
if (err) { if (err) {
printk(KERN_ERR "w83781d: Device addition failed (%d)\n", pr_err("Device addition failed (%d)\n", err);
err);
goto exit_device_put; goto exit_device_put;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment