Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

hwmon: Fix checkpatch warning 'quoted string split across lines'

Cc: Corentin Labbe <corentin.labbe@geomatys.fr>
Cc: Mark M. Hoffman <mhoffman@lightlink.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: Juerg Haefliger <juergh@gmail.com>
Cc: Andreas Herrmann <herrmann.der.user@googlemail.com>
Cc: Rudolf Marek <r.marek@assembler.cz>
Cc: Jim Cromie <jim.cromie@gmail.com>
Cc: Roger Lucas <vt8231@hiddenengine.co.uk>
Cc: Marc Hulsman <m.hulsman@tudelft.nl>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>

+271 -249
+1 -2
drivers/hwmon/ad7314.c
··· 166 166 module_spi_driver(ad7314_driver); 167 167 168 168 MODULE_AUTHOR("Sonic Zhang <sonic.zhang@analog.com>"); 169 - MODULE_DESCRIPTION("Analog Devices AD7314, ADT7301 and ADT7302 digital" 170 - " temperature sensor driver"); 169 + MODULE_DESCRIPTION("Analog Devices AD7314, ADT7301 and ADT7302 digital temperature sensor driver"); 171 170 MODULE_LICENSE("GPL v2");
+4 -7
drivers/hwmon/adm1021.c
··· 312 312 int conv_rate, status, config, man_id, dev_id; 313 313 314 314 if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) { 315 - pr_debug("adm1021: detect failed, " 316 - "smbus byte data not supported!\n"); 315 + pr_debug("detect failed, smbus byte data not supported!\n"); 317 316 return -ENODEV; 318 317 } 319 318 ··· 323 324 324 325 /* Check unused bits */ 325 326 if ((status & 0x03) || (config & 0x3F) || (conv_rate & 0xF8)) { 326 - pr_debug("adm1021: detect failed, chip not detected!\n"); 327 + pr_debug("detect failed, chip not detected!\n"); 327 328 return -ENODEV; 328 329 } 329 330 ··· 352 353 else 353 354 type_name = "max1617"; 354 355 355 - pr_debug("adm1021: Detected chip %s at adapter %d, address 0x%02x.\n", 356 + pr_debug("Detected chip %s at adapter %d, address 0x%02x.\n", 356 357 type_name, i2c_adapter_id(adapter), client->addr); 357 358 strlcpy(info->type, type_name, I2C_NAME_SIZE); 358 359 ··· 367 368 368 369 data = devm_kzalloc(&client->dev, sizeof(struct adm1021_data), 369 370 GFP_KERNEL); 370 - if (!data) { 371 - pr_debug("adm1021: detect failed, devm_kzalloc failed!\n"); 371 + if (!data) 372 372 return -ENOMEM; 373 - } 374 373 375 374 i2c_set_clientdata(client, data); 376 375 data->type = id->driver_data;
+27 -26
drivers/hwmon/adm1026.c
··· 49 49 module_param_array(gpio_input, int, NULL, 0); 50 50 MODULE_PARM_DESC(gpio_input, "List of GPIO pins (0-16) to program as inputs"); 51 51 module_param_array(gpio_output, int, NULL, 0); 52 - MODULE_PARM_DESC(gpio_output, "List of GPIO pins (0-16) to program as " 53 - "outputs"); 52 + MODULE_PARM_DESC(gpio_output, 53 + "List of GPIO pins (0-16) to program as outputs"); 54 54 module_param_array(gpio_inverted, int, NULL, 0); 55 - MODULE_PARM_DESC(gpio_inverted, "List of GPIO pins (0-16) to program as " 56 - "inverted"); 55 + MODULE_PARM_DESC(gpio_inverted, 56 + "List of GPIO pins (0-16) to program as inverted"); 57 57 module_param_array(gpio_normal, int, NULL, 0); 58 - MODULE_PARM_DESC(gpio_normal, "List of GPIO pins (0-16) to program as " 59 - "normal/non-inverted"); 58 + MODULE_PARM_DESC(gpio_normal, 59 + "List of GPIO pins (0-16) to program as normal/non-inverted"); 60 60 module_param_array(gpio_fan, int, NULL, 0); 61 61 MODULE_PARM_DESC(gpio_fan, "List of GPIO pins (0-7) to program as fan tachs"); 62 62 ··· 372 372 dev_dbg(&client->dev, "ADM1026_REG_CONFIG1 is: 0x%02x\n", 373 373 data->config1); 374 374 if ((data->config1 & CFG1_MONITOR) == 0) { 375 - dev_dbg(&client->dev, "Monitoring not currently " 376 - "enabled.\n"); 375 + dev_dbg(&client->dev, 376 + "Monitoring not currently enabled.\n"); 377 377 } 378 378 if (data->config1 & CFG1_INT_ENABLE) { 379 - dev_dbg(&client->dev, "SMBALERT interrupts are " 380 - "enabled.\n"); 379 + dev_dbg(&client->dev, 380 + "SMBALERT interrupts are enabled.\n"); 381 381 } 382 382 if (data->config1 & CFG1_AIN8_9) { 383 - dev_dbg(&client->dev, "in8 and in9 enabled. " 384 - "temp3 disabled.\n"); 383 + dev_dbg(&client->dev, 384 + "in8 and in9 enabled. temp3 disabled.\n"); 385 385 } else { 386 - dev_dbg(&client->dev, "temp3 enabled. in8 and " 387 - "in9 disabled.\n"); 386 + dev_dbg(&client->dev, 387 + "temp3 enabled. in8 and in9 disabled.\n"); 388 388 } 389 389 if (data->config1 & CFG1_THERM_HOT) { 390 - dev_dbg(&client->dev, "Automatic THERM, PWM, " 391 - "and temp limits enabled.\n"); 390 + dev_dbg(&client->dev, 391 + "Automatic THERM, PWM, and temp limits enabled.\n"); 392 392 } 393 393 394 394 if (data->config3 & CFG3_GPIO16_ENABLE) { 395 - dev_dbg(&client->dev, "GPIO16 enabled. THERM " 396 - "pin disabled.\n"); 395 + dev_dbg(&client->dev, 396 + "GPIO16 enabled. THERM pin disabled.\n"); 397 397 } else { 398 - dev_dbg(&client->dev, "THERM pin enabled. " 399 - "GPIO16 disabled.\n"); 398 + dev_dbg(&client->dev, 399 + "THERM pin enabled. GPIO16 disabled.\n"); 400 400 } 401 401 if (data->config3 & CFG3_VREF_250) 402 402 dev_dbg(&client->dev, "Vref is 2.50 Volts.\n"); ··· 1798 1798 company = adm1026_read_value(client, ADM1026_REG_COMPANY); 1799 1799 verstep = adm1026_read_value(client, ADM1026_REG_VERSTEP); 1800 1800 1801 - dev_dbg(&adapter->dev, "Detecting device at %d,0x%02x with" 1802 - " COMPANY: 0x%02x and VERSTEP: 0x%02x\n", 1801 + dev_dbg(&adapter->dev, 1802 + "Detecting device at %d,0x%02x with COMPANY: 0x%02x and VERSTEP: 0x%02x\n", 1803 1803 i2c_adapter_id(client->adapter), client->addr, 1804 1804 company, verstep); 1805 1805 ··· 1811 1811 /* Analog Devices ADM1026 */ 1812 1812 } else if (company == ADM1026_COMPANY_ANALOG_DEV 1813 1813 && (verstep & 0xf0) == ADM1026_VERSTEP_GENERIC) { 1814 - dev_err(&adapter->dev, "Unrecognized stepping " 1815 - "0x%02x. Defaulting to ADM1026.\n", verstep); 1814 + dev_err(&adapter->dev, 1815 + "Unrecognized stepping 0x%02x. Defaulting to ADM1026.\n", 1816 + verstep); 1816 1817 } else if ((verstep & 0xf0) == ADM1026_VERSTEP_GENERIC) { 1817 - dev_err(&adapter->dev, "Found version/stepping " 1818 - "0x%02x. Assuming generic ADM1026.\n", 1818 + dev_err(&adapter->dev, 1819 + "Found version/stepping 0x%02x. Assuming generic ADM1026.\n", 1819 1820 verstep); 1820 1821 } else { 1821 1822 dev_dbg(&adapter->dev, "Autodetection failed\n");
+5 -4
drivers/hwmon/adm1029.c
··· 224 224 break; 225 225 default: 226 226 mutex_unlock(&data->update_lock); 227 - dev_err(&client->dev, "fan_div value %ld not " 228 - "supported. Choose one of 1, 2 or 4!\n", val); 227 + dev_err(&client->dev, 228 + "fan_div value %ld not supported. Choose one of 1, 2 or 4!\n", 229 + val); 229 230 return -EINVAL; 230 231 } 231 232 /* Update the value */ ··· 327 326 * There are no "official" CHIP ID, so actually 328 327 * we use Major/Minor revision for that 329 328 */ 330 - pr_info("adm1029: Unknown major revision %x, " 331 - "please let us know\n", chip_id); 329 + pr_info("Unknown major revision %x, please let us know\n", 330 + chip_id); 332 331 return -ENODEV; 333 332 } 334 333
+5 -4
drivers/hwmon/adm9240.c
··· 351 351 reg &= ~(3 << shift); 352 352 reg |= (fan_div << shift); 353 353 i2c_smbus_write_byte_data(client, ADM9240_REG_VID_FAN_DIV, reg); 354 - dev_dbg(&client->dev, "fan%d clock divider changed from %u " 355 - "to %u\n", nr + 1, 1 << old, 1 << fan_div); 354 + dev_dbg(&client->dev, 355 + "fan%d clock divider changed from %u to %u\n", 356 + nr + 1, 1 << old, 1 << fan_div); 356 357 } 357 358 358 359 /* ··· 700 699 /* start measurement cycle */ 701 700 i2c_smbus_write_byte_data(client, ADM9240_REG_CONFIG, 1); 702 701 703 - dev_info(&client->dev, "cold start: config was 0x%02x " 704 - "mode %u\n", conf, mode); 702 + dev_info(&client->dev, 703 + "cold start: config was 0x%02x mode %u\n", conf, mode); 705 704 } 706 705 } 707 706
+6 -4
drivers/hwmon/adt7411.c
··· 259 259 260 260 val = i2c_smbus_read_byte_data(client, ADT7411_REG_MANUFACTURER_ID); 261 261 if (val < 0 || val != ADT7411_MANUFACTURER_ID) { 262 - dev_dbg(&client->dev, "Wrong manufacturer ID. Got %d, " 263 - "expected %d\n", val, ADT7411_MANUFACTURER_ID); 262 + dev_dbg(&client->dev, 263 + "Wrong manufacturer ID. Got %d, expected %d\n", 264 + val, ADT7411_MANUFACTURER_ID); 264 265 return -ENODEV; 265 266 } 266 267 267 268 val = i2c_smbus_read_byte_data(client, ADT7411_REG_DEVICE_ID); 268 269 if (val < 0 || val != ADT7411_DEVICE_ID) { 269 - dev_dbg(&client->dev, "Wrong device ID. Got %d, " 270 - "expected %d\n", val, ADT7411_DEVICE_ID); 270 + dev_dbg(&client->dev, 271 + "Wrong device ID. Got %d, expected %d\n", 272 + val, ADT7411_DEVICE_ID); 271 273 return -ENODEV; 272 274 } 273 275
+13 -10
drivers/hwmon/asb100.c
··· 55 55 56 56 static unsigned short force_subclients[4]; 57 57 module_param_array(force_subclients, short, NULL, 0); 58 - MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " 59 - "{bus, clientaddr, subclientaddr1, subclientaddr2}"); 58 + MODULE_PARM_DESC(force_subclients, 59 + "List of subclient addresses: {bus, clientaddr, subclientaddr1, subclientaddr2}"); 60 60 61 61 /* Voltage IN registers 0-6 */ 62 62 #define ASB100_REG_IN(nr) (0x20 + (nr)) ··· 689 689 for (i = 2; i <= 3; i++) { 690 690 if (force_subclients[i] < 0x48 || 691 691 force_subclients[i] > 0x4f) { 692 - dev_err(&client->dev, "invalid subclient " 693 - "address %d; must be 0x48-0x4f\n", 692 + dev_err(&client->dev, 693 + "invalid subclient address %d; must be 0x48-0x4f\n", 694 694 force_subclients[i]); 695 695 err = -ENODEV; 696 696 goto ERROR_SC_2; ··· 708 708 } 709 709 710 710 if (sc_addr[0] == sc_addr[1]) { 711 - dev_err(&client->dev, "duplicate addresses 0x%x " 712 - "for subclients\n", sc_addr[0]); 711 + dev_err(&client->dev, 712 + "duplicate addresses 0x%x for subclients\n", 713 + sc_addr[0]); 713 714 err = -ENODEV; 714 715 goto ERROR_SC_2; 715 716 } 716 717 717 718 data->lm75[0] = i2c_new_dummy(adapter, sc_addr[0]); 718 719 if (!data->lm75[0]) { 719 - dev_err(&client->dev, "subclient %d registration " 720 - "at address 0x%x failed.\n", 1, sc_addr[0]); 720 + dev_err(&client->dev, 721 + "subclient %d registration at address 0x%x failed.\n", 722 + 1, sc_addr[0]); 721 723 err = -ENOMEM; 722 724 goto ERROR_SC_2; 723 725 } 724 726 725 727 data->lm75[1] = i2c_new_dummy(adapter, sc_addr[1]); 726 728 if (!data->lm75[1]) { 727 - dev_err(&client->dev, "subclient %d registration " 728 - "at address 0x%x failed.\n", 2, sc_addr[1]); 729 + dev_err(&client->dev, 730 + "subclient %d registration at address 0x%x failed.\n", 731 + 2, sc_addr[1]); 729 732 err = -ENOMEM; 730 733 goto ERROR_SC_3; 731 734 }
+1 -2
drivers/hwmon/coretemp.c
··· 411 411 * fixed for stepping D0 (6EC). 412 412 */ 413 413 if (c->x86_model == 0xe && c->x86_mask < 0xc && c->microcode < 0x39) { 414 - pr_err("Errata AE18 not fixed, update BIOS or " 415 - "microcode of the CPU!\n"); 414 + pr_err("Errata AE18 not fixed, update BIOS or microcode of the CPU!\n"); 416 415 return -ENODEV; 417 416 } 418 417 return 0;
+34 -34
drivers/hwmon/dme1737.c
··· 55 55 56 56 static bool probe_all_addr; 57 57 module_param(probe_all_addr, bool, 0); 58 - MODULE_PARM_DESC(probe_all_addr, "Include probing of non-standard LPC " 59 - "addresses"); 58 + MODULE_PARM_DESC(probe_all_addr, 59 + "Include probing of non-standard LPC addresses"); 60 60 61 61 /* Addresses to scan */ 62 62 static const unsigned short normal_i2c[] = {0x2c, 0x2d, 0x2e, I2C_CLIENT_END}; 63 63 64 64 enum chips { dme1737, sch5027, sch311x, sch5127 }; 65 + 66 + #define DO_REPORT "Please report to the driver maintainer." 65 67 66 68 /* --------------------------------------------------------------------- 67 69 * Registers ··· 568 566 val = i2c_smbus_read_byte_data(client, reg); 569 567 570 568 if (val < 0) { 571 - dev_warn(&client->dev, "Read from register " 572 - "0x%02x failed! Please report to the driver " 573 - "maintainer.\n", reg); 569 + dev_warn(&client->dev, 570 + "Read from register 0x%02x failed! %s\n", 571 + reg, DO_REPORT); 574 572 } 575 573 } else { /* ISA device */ 576 574 outb(reg, data->addr); ··· 589 587 res = i2c_smbus_write_byte_data(client, reg, val); 590 588 591 589 if (res < 0) { 592 - dev_warn(&client->dev, "Write to register " 593 - "0x%02x failed! Please report to the driver " 594 - "maintainer.\n", reg); 590 + dev_warn(&client->dev, 591 + "Write to register 0x%02x failed! %s\n", 592 + reg, DO_REPORT); 595 593 } 596 594 } else { /* ISA device */ 597 595 outb(reg, data->addr); ··· 1169 1167 /* Only valid for fan[1-4] */ 1170 1168 if (!(val == 1 || val == 2 || val == 4)) { 1171 1169 count = -EINVAL; 1172 - dev_warn(dev, "Fan type value %ld not " 1173 - "supported. Choose one of 1, 2, or 4.\n", 1170 + dev_warn(dev, 1171 + "Fan type value %ld not supported. Choose one of 1, 2, or 4.\n", 1174 1172 val); 1175 1173 goto exit; 1176 1174 } ··· 1296 1294 /* Only valid for pwm[1-3] */ 1297 1295 if (val < 0 || val > 2) { 1298 1296 count = -EINVAL; 1299 - dev_warn(dev, "PWM enable %ld not " 1300 - "supported. Choose one of 0, 1, or 2.\n", 1297 + dev_warn(dev, 1298 + "PWM enable %ld not supported. Choose one of 0, 1, or 2.\n", 1301 1299 val); 1302 1300 goto exit; 1303 1301 } ··· 1401 1399 if (!(val == 1 || val == 2 || val == 4 || 1402 1400 val == 6 || val == 7)) { 1403 1401 count = -EINVAL; 1404 - dev_warn(dev, "PWM auto channels zone %ld " 1405 - "not supported. Choose one of 1, 2, 4, 6, " 1402 + dev_warn(dev, 1403 + "PWM auto channels zone %ld not supported. Choose one of 1, 2, 4, 6, " 1406 1404 "or 7.\n", val); 1407 1405 goto exit; 1408 1406 } ··· 2180 2178 * selected attributes from read-only to read-writeable. 2181 2179 */ 2182 2180 if (data->config & 0x02) { 2183 - dev_info(dev, "Device is locked. Some attributes " 2184 - "will be read-only.\n"); 2181 + dev_info(dev, 2182 + "Device is locked. Some attributes will be read-only.\n"); 2185 2183 } else { 2186 2184 /* Change permissions of zone sysfs attributes */ 2187 2185 dme1737_chmod_group(dev, &dme1737_zone_chmod_group, ··· 2249 2247 /* Inform if part is not monitoring/started */ 2250 2248 if (!(data->config & 0x01)) { 2251 2249 if (!force_start) { 2252 - dev_err(dev, "Device is not monitoring. " 2253 - "Use the force_start load parameter to " 2254 - "override.\n"); 2250 + dev_err(dev, 2251 + "Device is not monitoring. Use the force_start load parameter to override.\n"); 2255 2252 return -EFAULT; 2256 2253 } 2257 2254 ··· 2290 2289 */ 2291 2290 if (dme1737_i2c_get_features(0x2e, data) && 2292 2291 dme1737_i2c_get_features(0x4e, data)) { 2293 - dev_warn(dev, "Failed to query Super-IO for optional " 2294 - "features.\n"); 2292 + dev_warn(dev, 2293 + "Failed to query Super-IO for optional features.\n"); 2295 2294 } 2296 2295 } 2297 2296 ··· 2318 2317 break; 2319 2318 } 2320 2319 2321 - dev_info(dev, "Optional features: pwm3=%s, pwm5=%s, pwm6=%s, " 2322 - "fan3=%s, fan4=%s, fan5=%s, fan6=%s.\n", 2320 + dev_info(dev, 2321 + "Optional features: pwm3=%s, pwm5=%s, pwm6=%s, fan3=%s, fan4=%s, fan5=%s, fan6=%s.\n", 2323 2322 (data->has_features & HAS_PWM(2)) ? "yes" : "no", 2324 2323 (data->has_features & HAS_PWM(4)) ? "yes" : "no", 2325 2324 (data->has_features & HAS_PWM(5)) ? "yes" : "no", ··· 2331 2330 reg = dme1737_read(data, DME1737_REG_TACH_PWM); 2332 2331 /* Inform if fan-to-pwm mapping differs from the default */ 2333 2332 if (client && reg != 0xa4) { /* I2C chip */ 2334 - dev_warn(dev, "Non-standard fan to pwm mapping: " 2335 - "fan1->pwm%d, fan2->pwm%d, fan3->pwm%d, " 2336 - "fan4->pwm%d. Please report to the driver " 2337 - "maintainer.\n", 2333 + dev_warn(dev, 2334 + "Non-standard fan to pwm mapping: fan1->pwm%d, fan2->pwm%d, fan3->pwm%d, fan4->pwm%d. %s\n", 2338 2335 (reg & 0x03) + 1, ((reg >> 2) & 0x03) + 1, 2339 - ((reg >> 4) & 0x03) + 1, ((reg >> 6) & 0x03) + 1); 2336 + ((reg >> 4) & 0x03) + 1, ((reg >> 6) & 0x03) + 1, 2337 + DO_REPORT); 2340 2338 } else if (!client && reg != 0x24) { /* ISA chip */ 2341 - dev_warn(dev, "Non-standard fan to pwm mapping: " 2342 - "fan1->pwm%d, fan2->pwm%d, fan3->pwm%d. " 2343 - "Please report to the driver maintainer.\n", 2339 + dev_warn(dev, 2340 + "Non-standard fan to pwm mapping: fan1->pwm%d, fan2->pwm%d, fan3->pwm%d. %s\n", 2344 2341 (reg & 0x03) + 1, ((reg >> 2) & 0x03) + 1, 2345 - ((reg >> 4) & 0x03) + 1); 2342 + ((reg >> 4) & 0x03) + 1, DO_REPORT); 2346 2343 } 2347 2344 2348 2345 /* ··· 2354 2355 DME1737_REG_PWM_CONFIG(ix)); 2355 2356 if ((data->has_features & HAS_PWM(ix)) && 2356 2357 (PWM_EN_FROM_REG(data->pwm_config[ix]) == -1)) { 2357 - dev_info(dev, "Switching pwm%d to " 2358 - "manual mode.\n", ix + 1); 2358 + dev_info(dev, 2359 + "Switching pwm%d to manual mode.\n", 2360 + ix + 1); 2359 2361 data->pwm_config[ix] = PWM_EN_TO_REG(1, 2360 2362 data->pwm_config[ix]); 2361 2363 dme1737_write(data, DME1737_REG_PWM(ix), 0);
+2 -2
drivers/hwmon/fam15h_power.c
··· 189 189 190 190 /* result not allowed to be >= 256W */ 191 191 if ((tmp >> 16) >= 256) 192 - dev_warn(&f4->dev, "Bogus value for ProcessorPwrWatts " 193 - "(processor_pwr_watts>=%u)\n", 192 + dev_warn(&f4->dev, 193 + "Bogus value for ProcessorPwrWatts (processor_pwr_watts>=%u)\n", 194 194 (unsigned int) (tmp >> 16)); 195 195 196 196 /* convert to microWatt */
+5 -4
drivers/hwmon/fschmd.c
··· 463 463 v = 3; 464 464 break; 465 465 default: 466 - dev_err(dev, "fan_div value %lu not supported. " 467 - "Choose one of 2, 4 or 8!\n", v); 466 + dev_err(dev, 467 + "fan_div value %lu not supported. Choose one of 2, 4 or 8!\n", 468 + v); 468 469 return -EINVAL; 469 470 } 470 471 ··· 1250 1249 } 1251 1250 if (i == ARRAY_SIZE(watchdog_minors)) { 1252 1251 data->watchdog_miscdev.minor = 0; 1253 - dev_warn(&client->dev, "Couldn't register watchdog chardev " 1254 - "(due to no free minor)\n"); 1252 + dev_warn(&client->dev, 1253 + "Couldn't register watchdog chardev (due to no free minor)\n"); 1255 1254 } 1256 1255 mutex_unlock(&watchdog_data_mutex); 1257 1256
+3 -2
drivers/hwmon/gl518sm.c
··· 344 344 val = 3; 345 345 break; 346 346 default: 347 - dev_err(dev, "Invalid fan clock divider %lu, choose one " 348 - "of 1, 2, 4 or 8\n", val); 347 + dev_err(dev, 348 + "Invalid fan clock divider %lu, choose one of 1, 2, 4 or 8\n", 349 + val); 349 350 return -EINVAL; 350 351 } 351 352
+11 -10
drivers/hwmon/ibmaem.c
··· 289 289 err = ipmi_create_user(data->interface, &driver_data.ipmi_hndlrs, 290 290 data, &data->user); 291 291 if (err < 0) { 292 - dev_err(bmc, "Unable to register user with IPMI " 293 - "interface %d\n", data->interface); 292 + dev_err(bmc, 293 + "Unable to register user with IPMI interface %d\n", 294 + data->interface); 294 295 return -EACCES; 295 296 } 296 297 ··· 329 328 struct aem_ipmi_data *data = user_msg_data; 330 329 331 330 if (msg->msgid != data->tx_msgid) { 332 - dev_err(data->bmc_device, "Mismatch between received msgid " 333 - "(%02x) and transmitted msgid (%02x)!\n", 331 + dev_err(data->bmc_device, 332 + "Mismatch between received msgid (%02x) and transmitted msgid (%02x)!\n", 334 333 (int)msg->msgid, 335 334 (int)data->tx_msgid); 336 335 ipmi_free_recv_msg(msg); ··· 576 575 /* Register with hwmon */ 577 576 data->hwmon_dev = hwmon_device_register(&data->pdev->dev); 578 577 if (IS_ERR(data->hwmon_dev)) { 579 - dev_err(&data->pdev->dev, "Unable to register hwmon " 580 - "device for IPMI interface %d\n", 578 + dev_err(&data->pdev->dev, 579 + "Unable to register hwmon device for IPMI interface %d\n", 581 580 probe->interface); 582 581 res = PTR_ERR(data->hwmon_dev); 583 582 goto hwmon_reg_err; ··· 716 715 /* Register with hwmon */ 717 716 data->hwmon_dev = hwmon_device_register(&data->pdev->dev); 718 717 if (IS_ERR(data->hwmon_dev)) { 719 - dev_err(&data->pdev->dev, "Unable to register hwmon " 720 - "device for IPMI interface %d\n", 718 + dev_err(&data->pdev->dev, 719 + "Unable to register hwmon device for IPMI interface %d\n", 721 720 probe->interface); 722 721 res = PTR_ERR(data->hwmon_dev); 723 722 goto hwmon_reg_err; ··· 769 768 770 769 while (!aem_find_aem2(probe, &fi_resp, i)) { 771 770 if (fi_resp.major != 2) { 772 - dev_err(probe->bmc_device, "Unknown AEM v%d; please " 773 - "report this to the maintainer.\n", 771 + dev_err(probe->bmc_device, 772 + "Unknown AEM v%d; please report this to the maintainer.\n", 774 773 fi_resp.major); 775 774 i++; 776 775 continue;
+9 -8
drivers/hwmon/ibmpex.c
··· 163 163 data->sensor_major = data->rx_msg_data[0]; 164 164 data->sensor_minor = data->rx_msg_data[1]; 165 165 166 - dev_info(data->bmc_device, "Found BMC with sensor interface " 167 - "v%d.%d %d-%02d-%02d on interface %d\n", 166 + dev_info(data->bmc_device, 167 + "Found BMC with sensor interface v%d.%d %d-%02d-%02d on interface %d\n", 168 168 data->sensor_major, 169 169 data->sensor_minor, 170 170 extract_value(data->rx_msg_data, 2), ··· 478 478 err = ipmi_create_user(data->interface, &driver_data.ipmi_hndlrs, 479 479 data, &data->user); 480 480 if (err < 0) { 481 - dev_err(dev, "Unable to register user with IPMI " 482 - "interface %d\n", data->interface); 481 + dev_err(dev, 482 + "Unable to register user with IPMI interface %d\n", 483 + data->interface); 483 484 goto out; 484 485 } 485 486 ··· 502 501 data->hwmon_dev = hwmon_device_register(data->bmc_device); 503 502 504 503 if (IS_ERR(data->hwmon_dev)) { 505 - dev_err(data->bmc_device, "Unable to register hwmon " 506 - "device for IPMI interface %d\n", 504 + dev_err(data->bmc_device, 505 + "Unable to register hwmon device for IPMI interface %d\n", 507 506 data->interface); 508 507 goto out_user; 509 508 } ··· 568 567 struct ibmpex_bmc_data *data = (struct ibmpex_bmc_data *)user_msg_data; 569 568 570 569 if (msg->msgid != data->tx_msgid) { 571 - dev_err(data->bmc_device, "Mismatch between received msgid " 572 - "(%02x) and transmitted msgid (%02x)!\n", 570 + dev_err(data->bmc_device, 571 + "Mismatch between received msgid (%02x) and transmitted msgid (%02x)!\n", 573 572 (int)msg->msgid, 574 573 (int)data->tx_msgid); 575 574 ipmi_free_recv_msg(msg);
+2 -2
drivers/hwmon/k8temp.c
··· 200 200 */ 201 201 if (model >= 0x40) { 202 202 data->swap_core_select = 1; 203 - dev_warn(&pdev->dev, "Temperature readouts might be wrong - " 204 - "check erratum #141\n"); 203 + dev_warn(&pdev->dev, 204 + "Temperature readouts might be wrong - check erratum #141\n"); 205 205 } 206 206 207 207 /*
+6 -4
drivers/hwmon/lm78.c
··· 386 386 data->fan_div[nr] = 3; 387 387 break; 388 388 default: 389 - dev_err(dev, "fan_div value %ld not " 390 - "supported. Choose one of 1, 2, 4 or 8!\n", val); 389 + dev_err(dev, 390 + "fan_div value %ld not supported. Choose one of 1, 2, 4 or 8!\n", 391 + val); 391 392 mutex_unlock(&data->update_lock); 392 393 return -EINVAL; 393 394 } ··· 637 636 goto err_nodev; 638 637 639 638 if (lm78_alias_detect(client, i)) { 640 - dev_dbg(&adapter->dev, "Device at 0x%02x appears to " 641 - "be the same as ISA device\n", address); 639 + dev_dbg(&adapter->dev, 640 + "Device at 0x%02x appears to be the same as ISA device\n", 641 + address); 642 642 goto err_nodev; 643 643 } 644 644
+3 -2
drivers/hwmon/lm80.c
··· 286 286 data->fan_div[nr] = 3; 287 287 break; 288 288 default: 289 - dev_err(&client->dev, "fan_div value %ld not " 290 - "supported. Choose one of 1, 2, 4 or 8!\n", val); 289 + dev_err(&client->dev, 290 + "fan_div value %ld not supported. Choose one of 1, 2, 4 or 8!\n", 291 + val); 291 292 mutex_unlock(&data->update_lock); 292 293 return -EINVAL; 293 294 }
+2 -2
drivers/hwmon/lm85.c
··· 1293 1293 company = lm85_read_value(client, LM85_REG_COMPANY); 1294 1294 verstep = lm85_read_value(client, LM85_REG_VERSTEP); 1295 1295 1296 - dev_dbg(&adapter->dev, "Detecting device at 0x%02x with " 1297 - "COMPANY: 0x%02x and VERSTEP: 0x%02x\n", 1296 + dev_dbg(&adapter->dev, 1297 + "Detecting device at 0x%02x with COMPANY: 0x%02x and VERSTEP: 0x%02x\n", 1298 1298 address, company, verstep); 1299 1299 1300 1300 /* All supported chips have the version in common */
+20 -16
drivers/hwmon/lm93.c
··· 818 818 if (value >= 0) { 819 819 return value; 820 820 } else { 821 - dev_warn(&client->dev, "lm93: read byte data failed, " 822 - "address 0x%02x.\n", reg); 821 + dev_warn(&client->dev, 822 + "lm93: read byte data failed, address 0x%02x.\n", 823 + reg); 823 824 mdelay(i + 3); 824 825 } 825 826 ··· 839 838 result = i2c_smbus_write_byte_data(client, reg, value); 840 839 841 840 if (result < 0) 842 - dev_warn(&client->dev, "lm93: write byte data failed, " 843 - "0x%02x at address 0x%02x.\n", value, reg); 841 + dev_warn(&client->dev, 842 + "lm93: write byte data failed, 0x%02x at address 0x%02x.\n", 843 + value, reg); 844 844 845 845 return result; 846 846 } ··· 856 854 if (value >= 0) { 857 855 return value; 858 856 } else { 859 - dev_warn(&client->dev, "lm93: read word data failed, " 860 - "address 0x%02x.\n", reg); 857 + dev_warn(&client->dev, 858 + "lm93: read word data failed, address 0x%02x.\n", 859 + reg); 861 860 mdelay(i + 3); 862 861 } 863 862 ··· 877 874 result = i2c_smbus_write_word_data(client, reg, value); 878 875 879 876 if (result < 0) 880 - dev_warn(&client->dev, "lm93: write word data failed, " 881 - "0x%04x at address 0x%02x.\n", value, reg); 877 + dev_warn(&client->dev, 878 + "lm93: write word data failed, 0x%04x at address 0x%02x.\n", 879 + value, reg); 882 880 883 881 return result; 884 882 } ··· 902 898 if (result == lm93_block_read_cmds[fbn].len) { 903 899 break; 904 900 } else { 905 - dev_warn(&client->dev, "lm93: block read data failed, " 906 - "command 0x%02x.\n", 901 + dev_warn(&client->dev, 902 + "lm93: block read data failed, command 0x%02x.\n", 907 903 lm93_block_read_cmds[fbn].cmd); 908 904 mdelay(i + 3); 909 905 } ··· 2676 2672 return; 2677 2673 } 2678 2674 2679 - dev_warn(&client->dev, "timed out waiting for sensor " 2680 - "chip to signal ready!\n"); 2675 + dev_warn(&client->dev, 2676 + "timed out waiting for sensor chip to signal ready!\n"); 2681 2677 } 2682 2678 2683 2679 /* Return 0 if detection is successful, -ENODEV otherwise */ ··· 2737 2733 dev_dbg(&client->dev, "using SMBus block data transactions\n"); 2738 2734 update = lm93_update_client_full; 2739 2735 } else if ((LM93_SMBUS_FUNC_MIN & func) == LM93_SMBUS_FUNC_MIN) { 2740 - dev_dbg(&client->dev, "disabled SMBus block data " 2741 - "transactions\n"); 2736 + dev_dbg(&client->dev, 2737 + "disabled SMBus block data transactions\n"); 2742 2738 update = lm93_update_client_min; 2743 2739 } else { 2744 - dev_dbg(&client->dev, "detect failed, " 2745 - "smbus byte and/or word data not supported!\n"); 2740 + dev_dbg(&client->dev, 2741 + "detect failed, smbus byte and/or word data not supported!\n"); 2746 2742 return -ENODEV; 2747 2743 } 2748 2744
+12 -13
drivers/hwmon/pc87360.c
··· 1190 1190 confreg[3] = superio_inb(sioaddr, 0x25); 1191 1191 1192 1192 if (confreg[2] & 0x40) { 1193 - pr_info("Using thermistors for " 1194 - "temperature monitoring\n"); 1193 + pr_info("Using thermistors for temperature monitoring\n"); 1195 1194 } 1196 1195 if (confreg[3] & 0xE0) { 1197 1196 pr_info("VID inputs routed (mode %u)\n", ··· 1270 1271 if (data->address[i] 1271 1272 && !devm_request_region(dev, extra_isa[i], PC87360_EXTENT, 1272 1273 pc87360_driver.driver.name)) { 1273 - dev_err(dev, "Region 0x%x-0x%x already " 1274 - "in use!\n", extra_isa[i], 1275 - extra_isa[i]+PC87360_EXTENT-1); 1274 + dev_err(dev, 1275 + "Region 0x%x-0x%x already in use!\n", 1276 + extra_isa[i], extra_isa[i]+PC87360_EXTENT-1); 1276 1277 return -EBUSY; 1277 1278 } 1278 1279 } ··· 1434 1435 if (init >= 2 && data->innr) { 1435 1436 reg = pc87360_read_value(data, LD_IN, NO_BANK, 1436 1437 PC87365_REG_IN_CONVRATE); 1437 - dev_info(&pdev->dev, "VLM conversion set to " 1438 - "1s period, 160us delay\n"); 1438 + dev_info(&pdev->dev, 1439 + "VLM conversion set to 1s period, 160us delay\n"); 1439 1440 pc87360_write_value(data, LD_IN, NO_BANK, 1440 1441 PC87365_REG_IN_CONVRATE, 1441 1442 (reg & 0xC0) | 0x11); ··· 1449 1450 if (init >= init_in[i]) { 1450 1451 /* Forcibly enable voltage channel */ 1451 1452 if (!(reg & CHAN_ENA)) { 1452 - dev_dbg(&pdev->dev, "Forcibly " 1453 - "enabling in%d\n", i); 1453 + dev_dbg(&pdev->dev, "Forcibly enabling in%d\n", 1454 + i); 1454 1455 pc87360_write_value(data, LD_IN, i, 1455 1456 PC87365_REG_IN_STATUS, 1456 1457 (reg & 0x68) | 0x87); ··· 1574 1575 data->fan_status[nr] += 0x20; 1575 1576 data->fan_min[nr] >>= 1; 1576 1577 data->fan[nr] >>= 1; 1577 - dev_dbg(dev, "Increasing " 1578 - "clock divider to %d for fan %d\n", 1578 + dev_dbg(dev, 1579 + "Increasing clock divider to %d for fan %d\n", 1579 1580 FAN_DIV_FROM_REG(data->fan_status[nr]), nr + 1); 1580 1581 } 1581 1582 } else { ··· 1586 1587 data->fan_status[nr] -= 0x20; 1587 1588 data->fan_min[nr] <<= 1; 1588 1589 data->fan[nr] <<= 1; 1589 - dev_dbg(dev, "Decreasing " 1590 - "clock divider to %d for fan %d\n", 1590 + dev_dbg(dev, 1591 + "Decreasing clock divider to %d for fan %d\n", 1591 1592 FAN_DIV_FROM_REG(data->fan_status[nr]), 1592 1593 nr + 1); 1593 1594 }
+7 -6
drivers/hwmon/pc87427.c
··· 627 627 pc87427_readall_pwm(data, nr); 628 628 mode = data->pwm_enable[nr] & PWM_ENABLE_MODE_MASK; 629 629 if (mode != PWM_MODE_MANUAL && mode != PWM_MODE_OFF) { 630 - dev_notice(dev, "Can't set PWM%d duty cycle while not in " 631 - "manual mode\n", nr + 1); 630 + dev_notice(dev, 631 + "Can't set PWM%d duty cycle while not in manual mode\n", 632 + nr + 1); 632 633 mutex_unlock(&data->lock); 633 634 return -EPERM; 634 635 } ··· 1246 1245 1247 1246 val = superio_inb(sioaddr, SIOREG_MAP); 1248 1247 if (val & 0x01) { 1249 - pr_warn("Logical device 0x%02x is memory-mapped, " 1250 - "can't use\n", logdev[i]); 1248 + pr_warn("Logical device 0x%02x is memory-mapped, can't use\n", 1249 + logdev[i]); 1251 1250 continue; 1252 1251 } 1253 1252 1254 1253 val = (superio_inb(sioaddr, SIOREG_IOBASE) << 8) 1255 1254 | superio_inb(sioaddr, SIOREG_IOBASE + 1); 1256 1255 if (!val) { 1257 - pr_info("I/O base address not set for logical device " 1258 - "0x%02x\n", logdev[i]); 1256 + pr_info("I/O base address not set for logical device 0x%02x\n", 1257 + logdev[i]); 1259 1258 continue; 1260 1259 } 1261 1260 sio_data->address[i] = val;
+6 -6
drivers/hwmon/sch56xx-common.c
··· 161 161 break; 162 162 } 163 163 if (i == max_busy_polls + max_lazy_polls) { 164 - pr_err("Max retries exceeded reading virtual " 165 - "register 0x%04hx (%d)\n", reg, 1); 164 + pr_err("Max retries exceeded reading virtual register 0x%04hx (%d)\n", 165 + reg, 1); 166 166 return -EIO; 167 167 } 168 168 ··· 178 178 break; 179 179 180 180 if (i == 0) 181 - pr_warn("EC reports: 0x%02x reading virtual register " 182 - "0x%04hx\n", (unsigned int)val, reg); 181 + pr_warn("EC reports: 0x%02x reading virtual register 0x%04hx\n", 182 + (unsigned int)val, reg); 183 183 } 184 184 if (i == max_busy_polls) { 185 - pr_err("Max retries exceeded reading virtual " 186 - "register 0x%04hx (%d)\n", reg, 2); 185 + pr_err("Max retries exceeded reading virtual register 0x%04hx (%d)\n", 186 + reg, 2); 187 187 return -EIO; 188 188 } 189 189
+3 -2
drivers/hwmon/sis5595.c
··· 456 456 data->fan_div[nr] = 3; 457 457 break; 458 458 default: 459 - dev_err(dev, "fan_div value %ld not " 460 - "supported. Choose one of 1, 2, 4 or 8!\n", val); 459 + dev_err(dev, 460 + "fan_div value %ld not supported. Choose one of 1, 2, 4 or 8!\n", 461 + val); 461 462 mutex_unlock(&data->update_lock); 462 463 return -EINVAL; 463 464 }
+3 -4
drivers/hwmon/thmc50.c
··· 41 41 static unsigned short adm1022_temp3[16]; 42 42 static unsigned int adm1022_temp3_num; 43 43 module_param_array(adm1022_temp3, ushort, &adm1022_temp3_num, 0); 44 - MODULE_PARM_DESC(adm1022_temp3, "List of adapter,address pairs " 45 - "to enable 3rd temperature (ADM1022 only)"); 44 + MODULE_PARM_DESC(adm1022_temp3, 45 + "List of adapter,address pairs to enable 3rd temperature (ADM1022 only)"); 46 46 47 47 /* Many THMC50 constants specified below */ 48 48 ··· 312 312 const char *type_name; 313 313 314 314 if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) { 315 - pr_debug("thmc50: detect failed, " 316 - "smbus byte data not supported!\n"); 315 + pr_debug("thmc50: detect failed, smbus byte data not supported!\n"); 317 316 return -ENODEV; 318 317 } 319 318
+2 -2
drivers/hwmon/tmp102.c
··· 155 155 156 156 if (!i2c_check_functionality(client->adapter, 157 157 I2C_FUNC_SMBUS_WORD_DATA)) { 158 - dev_err(&client->dev, "adapter doesn't support SMBus word " 159 - "transactions\n"); 158 + dev_err(&client->dev, 159 + "adapter doesn't support SMBus word transactions\n"); 160 160 return -ENODEV; 161 161 } 162 162
+3 -2
drivers/hwmon/tmp401.c
··· 451 451 return -EINVAL; 452 452 453 453 if (val != 1) { 454 - dev_err(dev, "temp_reset_history value %ld not" 455 - " supported. Use 1 to reset the history!\n", val); 454 + dev_err(dev, 455 + "temp_reset_history value %ld not supported. Use 1 to reset the history!\n", 456 + val); 456 457 return -EINVAL; 457 458 } 458 459 i2c_smbus_write_byte_data(to_i2c_client(dev),
+3 -4
drivers/hwmon/tmp421.c
··· 208 208 /* Start conversions (disable shutdown if necessary) */ 209 209 config = i2c_smbus_read_byte_data(client, TMP421_CONFIG_REG_1); 210 210 if (config < 0) { 211 - dev_err(&client->dev, "Could not read configuration" 212 - " register (%d)\n", config); 211 + dev_err(&client->dev, 212 + "Could not read configuration register (%d)\n", config); 213 213 return -ENODEV; 214 214 } 215 215 ··· 322 322 module_i2c_driver(tmp421_driver); 323 323 324 324 MODULE_AUTHOR("Andre Prendel <andre.prendel@gmx.de>"); 325 - MODULE_DESCRIPTION("Texas Instruments TMP421/422/423 temperature sensor" 326 - " driver"); 325 + MODULE_DESCRIPTION("Texas Instruments TMP421/422/423 temperature sensor driver"); 327 326 MODULE_LICENSE("GPL");
+5 -4
drivers/hwmon/via686a.c
··· 889 889 890 890 address = val & ~(VIA686A_EXTENT - 1); 891 891 if (address == 0) { 892 - dev_err(&dev->dev, "base address not set - upgrade BIOS " 893 - "or use force_addr=0xaddr\n"); 892 + dev_err(&dev->dev, 893 + "base address not set - upgrade BIOS or use force_addr=0xaddr\n"); 894 894 return -ENODEV; 895 895 } 896 896 ··· 899 899 return -ENODEV; 900 900 if (!(val & 0x0001)) { 901 901 if (!force_addr) { 902 - dev_warn(&dev->dev, "Sensors disabled, enable " 903 - "with force_addr=0x%x\n", address); 902 + dev_warn(&dev->dev, 903 + "Sensors disabled, enable with force_addr=0x%x\n", 904 + address); 904 905 return -ENODEV; 905 906 } 906 907
+13 -10
drivers/hwmon/vt1211.c
··· 571 571 break; 572 572 default: 573 573 count = -EINVAL; 574 - dev_warn(dev, "fan div value %ld not supported. " 575 - "Choose one of 1, 2, 4, or 8.\n", val); 574 + dev_warn(dev, 575 + "fan div value %ld not supported. Choose one of 1, 2, 4, or 8.\n", 576 + val); 576 577 goto EXIT; 577 578 } 578 579 vt1211_write8(data, VT1211_REG_FAN_DIV, ··· 675 674 break; 676 675 default: 677 676 count = -EINVAL; 678 - dev_warn(dev, "pwm mode %ld not supported. " 679 - "Choose one of 0 or 2.\n", val); 677 + dev_warn(dev, 678 + "pwm mode %ld not supported. Choose one of 0 or 2.\n", 679 + val); 680 680 goto EXIT; 681 681 } 682 682 vt1211_write8(data, VT1211_REG_PWM_CTL, ··· 702 700 case SHOW_SET_PWM_AUTO_CHANNELS_TEMP: 703 701 if (val < 1 || val > 7) { 704 702 count = -EINVAL; 705 - dev_warn(dev, "temp channel %ld not supported. " 706 - "Choose a value between 1 and 7.\n", val); 703 + dev_warn(dev, 704 + "temp channel %ld not supported. Choose a value between 1 and 7.\n", 705 + val); 707 706 goto EXIT; 708 707 } 709 708 if (!ISTEMP(val - 1, data->uch_config)) { ··· 1328 1325 1329 1326 if ((uch_config < -1) || (uch_config > 31)) { 1330 1327 err = -EINVAL; 1331 - pr_warn("Invalid UCH configuration %d. " 1332 - "Choose a value between 0 and 31.\n", uch_config); 1328 + pr_warn("Invalid UCH configuration %d. Choose a value between 0 and 31.\n", 1329 + uch_config); 1333 1330 goto EXIT; 1334 1331 } 1335 1332 1336 1333 if ((int_mode < -1) || (int_mode > 0)) { 1337 1334 err = -EINVAL; 1338 - pr_warn("Invalid interrupt mode %d. " 1339 - "Only mode 0 is supported.\n", int_mode); 1335 + pr_warn("Invalid interrupt mode %d. Only mode 0 is supported.\n", 1336 + int_mode); 1340 1337 goto EXIT; 1341 1338 } 1342 1339
+3 -2
drivers/hwmon/vt8231.c
··· 573 573 data->fan_div[nr] = 3; 574 574 break; 575 575 default: 576 - dev_err(dev, "fan_div value %ld not supported. " 577 - "Choose one of 1, 2, 4 or 8!\n", val); 576 + dev_err(dev, 577 + "fan_div value %ld not supported. Choose one of 1, 2, 4 or 8!\n", 578 + val); 578 579 mutex_unlock(&data->update_lock); 579 580 return -EINVAL; 580 581 }
+15 -16
drivers/hwmon/w83627ehf.c
··· 840 840 && (reg >= 0xff || (sio_data->kind == nct6775 841 841 && reg == 0x00)) 842 842 && data->fan_div[i] < 0x07) { 843 - dev_dbg(dev, "Increasing fan%d " 844 - "clock divider from %u to %u\n", 843 + dev_dbg(dev, 844 + "Increasing fan%d clock divider from %u to %u\n", 845 845 i + 1, div_from_reg(data->fan_div[i]), 846 846 div_from_reg(data->fan_div[i] + 1)); 847 847 data->fan_div[i]++; ··· 1110 1110 */ 1111 1111 data->fan_min[nr] = 254; 1112 1112 new_div = 7; /* 128 == (1 << 7) */ 1113 - dev_warn(dev, "fan%u low limit %lu below minimum %u, set to " 1114 - "minimum\n", nr + 1, val, 1115 - data->fan_from_reg_min(254, 7)); 1113 + dev_warn(dev, 1114 + "fan%u low limit %lu below minimum %u, set to minimum\n", 1115 + nr + 1, val, data->fan_from_reg_min(254, 7)); 1116 1116 } else if (!reg) { 1117 1117 /* 1118 1118 * Speed above this value cannot possibly be represented, ··· 1120 1120 */ 1121 1121 data->fan_min[nr] = 1; 1122 1122 new_div = 0; /* 1 == (1 << 0) */ 1123 - dev_warn(dev, "fan%u low limit %lu above maximum %u, set to " 1124 - "maximum\n", nr + 1, val, 1125 - data->fan_from_reg_min(1, 0)); 1123 + dev_warn(dev, 1124 + "fan%u low limit %lu above maximum %u, set to maximum\n", 1125 + nr + 1, val, data->fan_from_reg_min(1, 0)); 1126 1126 } else { 1127 1127 /* 1128 1128 * Automatically pick the best divider, i.e. the one such ··· 2396 2396 en_vrm10 = superio_inb(sio_data->sioreg, 2397 2397 SIO_REG_EN_VRM10); 2398 2398 if ((en_vrm10 & 0x08) && data->vrm == 90) { 2399 - dev_warn(dev, "Setting VID input " 2400 - "voltage to TTL\n"); 2399 + dev_warn(dev, 2400 + "Setting VID input voltage to TTL\n"); 2401 2401 superio_outb(sio_data->sioreg, 2402 2402 SIO_REG_EN_VRM10, 2403 2403 en_vrm10 & ~0x08); 2404 2404 } else if (!(en_vrm10 & 0x08) 2405 2405 && data->vrm == 100) { 2406 - dev_warn(dev, "Setting VID input " 2407 - "voltage to VRM10\n"); 2406 + dev_warn(dev, 2407 + "Setting VID input voltage to VRM10\n"); 2408 2408 superio_outb(sio_data->sioreg, 2409 2409 SIO_REG_EN_VRM10, 2410 2410 en_vrm10 | 0x08); ··· 2420 2420 if (err) 2421 2421 goto exit_release; 2422 2422 } else { 2423 - dev_info(dev, "VID pins in output mode, CPU VID not " 2424 - "available\n"); 2423 + dev_info(dev, 2424 + "VID pins in output mode, CPU VID not available\n"); 2425 2425 } 2426 2426 } 2427 2427 ··· 2795 2795 /* Activate logical device if needed */ 2796 2796 val = superio_inb(sioaddr, SIO_REG_ENABLE); 2797 2797 if (!(val & 0x01)) { 2798 - pr_warn("Forcibly enabling Super-I/O. " 2799 - "Sensor is probably unusable.\n"); 2798 + pr_warn("Forcibly enabling Super-I/O. Sensor is probably unusable.\n"); 2800 2799 superio_outb(sioaddr, SIO_REG_ENABLE, val | 0x01); 2801 2800 } 2802 2801
+19 -17
drivers/hwmon/w83781d.c
··· 64 64 /* Insmod parameters */ 65 65 static unsigned short force_subclients[4]; 66 66 module_param_array(force_subclients, short, NULL, 0); 67 - MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " 68 - "{bus, clientaddr, subclientaddr1, subclientaddr2}"); 67 + MODULE_PARM_DESC(force_subclients, 68 + "List of subclient addresses: {bus, clientaddr, subclientaddr1, subclientaddr2}"); 69 69 70 70 static bool reset; 71 71 module_param(reset, bool, 0); ··· 826 826 data->sens[nr] = val; 827 827 break; 828 828 case W83781D_DEFAULT_BETA: 829 - dev_warn(dev, "Sensor type %d is deprecated, please use 4 " 830 - "instead\n", W83781D_DEFAULT_BETA); 829 + dev_warn(dev, 830 + "Sensor type %d is deprecated, please use 4 instead\n", 831 + W83781D_DEFAULT_BETA); 831 832 /* fall through */ 832 833 case 4: /* thermistor */ 833 834 tmp = w83781d_read_value(data, W83781D_REG_SCFG1); ··· 875 874 for (i = 2; i <= 3; i++) { 876 875 if (force_subclients[i] < 0x48 || 877 876 force_subclients[i] > 0x4f) { 878 - dev_err(&new_client->dev, "Invalid subclient " 879 - "address %d; must be 0x48-0x4f\n", 877 + dev_err(&new_client->dev, 878 + "Invalid subclient address %d; must be 0x48-0x4f\n", 880 879 force_subclients[i]); 881 880 err = -EINVAL; 882 881 goto ERROR_SC_1; ··· 911 910 for (i = 0; i < num_sc; i++) { 912 911 data->lm75[i] = i2c_new_dummy(adapter, sc_addr[i]); 913 912 if (!data->lm75[i]) { 914 - dev_err(&new_client->dev, "Subclient %d " 915 - "registration at address 0x%x " 916 - "failed.\n", i, sc_addr[i]); 913 + dev_err(&new_client->dev, 914 + "Subclient %d registration at address 0x%x failed.\n", 915 + i, sc_addr[i]); 917 916 err = -ENOMEM; 918 917 if (i == 1) 919 918 goto ERROR_SC_3; ··· 1177 1176 goto err_nodev; 1178 1177 1179 1178 if (val1 <= 0x30 && w83781d_alias_detect(client, val1)) { 1180 - dev_dbg(&adapter->dev, "Device at 0x%02x appears to " 1181 - "be the same as ISA device\n", address); 1179 + dev_dbg(&adapter->dev, 1180 + "Device at 0x%02x appears to be the same as ISA device\n", 1181 + address); 1182 1182 goto err_nodev; 1183 1183 } 1184 1184 ··· 1369 1367 * as I see very little reason why this would be needed at 1370 1368 * all. 1371 1369 */ 1372 - dev_info(dev, "If reset=1 solved a problem you were " 1373 - "having, please report!\n"); 1370 + dev_info(dev, 1371 + "If reset=1 solved a problem you were having, please report!\n"); 1374 1372 1375 1373 /* save these registers */ 1376 1374 i = w83781d_read_value(data, W83781D_REG_BEEP_CONFIG); ··· 1427 1425 /* Enable temp2 */ 1428 1426 tmp = w83781d_read_value(data, W83781D_REG_TEMP2_CONFIG); 1429 1427 if (tmp & 0x01) { 1430 - dev_warn(dev, "Enabling temp2, readings " 1431 - "might not make sense\n"); 1428 + dev_warn(dev, 1429 + "Enabling temp2, readings might not make sense\n"); 1432 1430 w83781d_write_value(data, W83781D_REG_TEMP2_CONFIG, 1433 1431 tmp & 0xfe); 1434 1432 } ··· 1438 1436 tmp = w83781d_read_value(data, 1439 1437 W83781D_REG_TEMP3_CONFIG); 1440 1438 if (tmp & 0x01) { 1441 - dev_warn(dev, "Enabling temp3, " 1442 - "readings might not make sense\n"); 1439 + dev_warn(dev, 1440 + "Enabling temp3, readings might not make sense\n"); 1443 1441 w83781d_write_value(data, 1444 1442 W83781D_REG_TEMP3_CONFIG, tmp & 0xfe); 1445 1443 }
+2 -2
drivers/hwmon/w83791d.c
··· 56 56 57 57 static unsigned short force_subclients[4]; 58 58 module_param_array(force_subclients, short, NULL, 0); 59 - MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " 60 - "{bus, clientaddr, subclientaddr1, subclientaddr2}"); 59 + MODULE_PARM_DESC(force_subclients, 60 + "List of subclient addresses: {bus, clientaddr, subclientaddr1, subclientaddr2}"); 61 61 62 62 static bool reset; 63 63 module_param(reset, bool, 0);
+7 -6
drivers/hwmon/w83792d.c
··· 54 54 55 55 static unsigned short force_subclients[4]; 56 56 module_param_array(force_subclients, short, NULL, 0); 57 - MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " 58 - "{bus, clientaddr, subclientaddr1, subclientaddr2}"); 57 + MODULE_PARM_DESC(force_subclients, 58 + "List of subclient addresses: {bus, clientaddr, subclientaddr1, subclientaddr2}"); 59 59 60 60 static bool init; 61 61 module_param(init, bool, 0); ··· 951 951 for (i = 2; i <= 3; i++) { 952 952 if (force_subclients[i] < 0x48 || 953 953 force_subclients[i] > 0x4f) { 954 - dev_err(&new_client->dev, "invalid subclient " 955 - "address %d; must be 0x48-0x4f\n", 954 + dev_err(&new_client->dev, 955 + "invalid subclient address %d; must be 0x48-0x4f\n", 956 956 force_subclients[i]); 957 957 err = -ENODEV; 958 958 goto ERROR_SC_0; ··· 969 969 if (!(val & 0x80)) { 970 970 if ((data->lm75[0] != NULL) && 971 971 ((val & 0x7) == ((val >> 4) & 0x7))) { 972 - dev_err(&new_client->dev, "duplicate addresses 0x%x, " 973 - "use force_subclient\n", data->lm75[0]->addr); 972 + dev_err(&new_client->dev, 973 + "duplicate addresses 0x%x, use force_subclient\n", 974 + data->lm75[0]->addr); 974 975 err = -ENODEV; 975 976 goto ERROR_SC_1; 976 977 }
+4 -4
drivers/hwmon/w83793.c
··· 59 59 60 60 static unsigned short force_subclients[4]; 61 61 module_param_array(force_subclients, short, NULL, 0); 62 - MODULE_PARM_DESC(force_subclients, "List of subclient addresses: " 63 - "{bus, clientaddr, subclientaddr1, subclientaddr2}"); 62 + MODULE_PARM_DESC(force_subclients, 63 + "List of subclient addresses: {bus, clientaddr, subclientaddr1, subclientaddr2}"); 64 64 65 65 static bool reset; 66 66 module_param(reset, bool, 0); ··· 1921 1921 } 1922 1922 if (i == ARRAY_SIZE(watchdog_minors)) { 1923 1923 data->watchdog_miscdev.minor = 0; 1924 - dev_warn(&client->dev, "Couldn't register watchdog chardev " 1925 - "(due to no free minor)\n"); 1924 + dev_warn(&client->dev, 1925 + "Couldn't register watchdog chardev (due to no free minor)\n"); 1926 1926 } 1927 1927 1928 1928 mutex_unlock(&watchdog_data_mutex);
+5 -4
drivers/hwmon/w83795.c
··· 2120 2120 &w83795_in[i][3].dev_attr.attr, 2121 2121 S_IRUGO); 2122 2122 if (err_max || err_min) 2123 - dev_warn(&client->dev, "Failed to set in%d limits " 2124 - "read-only (%d, %d)\n", i, err_max, err_min); 2123 + dev_warn(&client->dev, 2124 + "Failed to set in%d limits read-only (%d, %d)\n", 2125 + i, err_max, err_min); 2125 2126 else 2126 - dev_info(&client->dev, "in%d limits set dynamically " 2127 - "from VID\n", i); 2127 + dev_info(&client->dev, 2128 + "in%d limits set dynamically from VID\n", i); 2128 2129 } 2129 2130 } 2130 2131