CC: [email protected] CC: [email protected] TO: "Nuno Sá" <[email protected]> CC: Guenter Roeck <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 509a15421674b9e1a3e1916939d0d0efd3e578da commit: 8412b410fa5e1e494a0fec84c3c462d49870d3f5 hwmon: Support ADI Fan Control IP date: 9 months ago :::::: branch date: 15 hours ago :::::: commit date: 9 months ago compiler: sh4-linux-gcc (GCC) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <[email protected]> cppcheck possible warnings: (new ones prefixed by >>, may not real problems) ^ >> fs/binfmt_elf_fdpic.c:895:6: warning: Variable 'seg' is reassigned a value >> before the old one has been used. 'break;' missing? [redundantAssignInSwitch] seg = loadmap->segs; ^ fs/binfmt_elf_fdpic.c:764:6: note: Variable 'seg' is reassigned a value before the old one has been used. 'break;' missing? seg = loadmap->segs; ^ fs/binfmt_elf_fdpic.c:895:6: note: Variable 'seg' is reassigned a value before the old one has been used. 'break;' missing? seg = loadmap->segs; ^ >> fs/binfmt_elf_fdpic.c:552:25: warning: Null pointer dereference [nullPointer] platform_len = strlen(k_base_platform) + 1; ^ fs/binfmt_elf_fdpic.c:868:6: warning: Variable 'seg' is reassigned a value before the old one has been used. 'break;' missing? [redundantAssignInSwitch] seg = mseg + 1; ^ fs/binfmt_elf_fdpic.c:764:6: note: Variable 'seg' is reassigned a value before the old one has been used. 'break;' missing? seg = loadmap->segs; ^ fs/binfmt_elf_fdpic.c:868:6: note: Variable 'seg' is reassigned a value before the old one has been used. 'break;' missing? seg = mseg + 1; ^ >> drivers/misc/ics932s401.c:138:38: warning: Shifting a negative value is >> technically undefined behaviour [shiftNegativeLHS] data->regs[regs_to_copy[i]] = temp >> 8; ^ -- >> net/mac802154/cfg.c:272:2: warning: Address of local auto-variable assigned >> to a function parameter. [autoVariables] *table = &sdata->sec.table; ^ -- >> drivers/input/gameport/ns558.c:103:27: warning: Shifting a negative value is >> technically undefined behaviour [shiftNegativeLHS] release_region(io & (-1 << (i - 1)), (1 << (i - 1))); ^ drivers/input/gameport/ns558.c:105:32: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] if (!request_region(io & (-1 << i), (1 << i), "ns558-isa")) ^ drivers/input/gameport/ns558.c:108:23: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] outb(0xff, io & (-1 << i)); ^ drivers/input/gameport/ns558.c:110:21: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] if (inb(io & (-1 << i)) != inb((io & (-1 << i)) + (1 << i) - 1)) b++; ^ drivers/input/gameport/ns558.c:114:28: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] release_region(io & (-1 << i), (1 << i)); ^ drivers/input/gameport/ns558.c:122:32: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] if (!request_region(io & (-1 << i), (1 << i), "ns558-isa")) ^ drivers/input/gameport/ns558.c:130:27: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] release_region(io & (-1 << i), (1 << i)); ^ drivers/input/gameport/ns558.c:142:56: warning: Shifting a negative value is technically undefined behaviour [shiftNegativeLHS] gameport_set_phys(port, "isa%04x/gameport0", io & (-1 << i)); ^ -- >> drivers/hwmon/axi-fan-control.c:355:9: warning: Identical condition 'ret', >> second condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/hwmon/axi-fan-control.c:338:6: note: first condition if (ret) ^ drivers/hwmon/axi-fan-control.c:355:9: note: second condition return ret; ^ -- >> drivers/hwmon/nct6775.c:2095:9: warning: %d in format string (no. 1) >> requires 'int' but the argument type is 'unsigned int'. >> [invalidPrintfArgType_sint] return sprintf(buf, "%dn", data->rpm[nr]); ^ drivers/hwmon/nct6775.c:2105:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] return sprintf(buf, "%dn", ^ drivers/hwmon/nct6775.c:2884:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] return sprintf(buf, "%dn", ^ drivers/hwmon/nct6775.c:3100:9: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] return sprintf(buf, "%dn", ^ >> drivers/hwmon/sht15.c:348:9: warning: Identical condition 'ret', second >> condition is always false [identicalConditionAfterEarlyExit] return ret; ^ drivers/hwmon/sht15.c:342:6: note: first condition if (ret) ^ drivers/hwmon/sht15.c:348:9: note: second condition return ret; ^ -- drivers/hwmon/w83627ehf.c:1029:1: warning: %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'. [invalidPrintfArgType_sint] fan_time_functions(fan_stop_time, W83627EHF_REG_FAN_STOP_TIME) ^ >> drivers/hwmon/w83627ehf.c:865:19: warning: Shifting a negative value is >> technically undefined behaviour [shiftNegativeLHS] reg |= (val - 1) << W83627EHF_PWM_ENABLE_SHIFT[channel]; ^ vim +/ret +355 drivers/hwmon/axi-fan-control.c 8412b410fa5e1e4 Nuno Sá 2019-10-09 330 8412b410fa5e1e4 Nuno Sá 2019-10-09 331 static int axi_fan_control_init(struct axi_fan_control_data *ctl, 8412b410fa5e1e4 Nuno Sá 2019-10-09 332 const struct device_node *np) 8412b410fa5e1e4 Nuno Sá 2019-10-09 333 { 8412b410fa5e1e4 Nuno Sá 2019-10-09 334 int ret; 8412b410fa5e1e4 Nuno Sá 2019-10-09 335 8412b410fa5e1e4 Nuno Sá 2019-10-09 336 /* get fan pulses per revolution */ 8412b410fa5e1e4 Nuno Sá 2019-10-09 337 ret = of_property_read_u32(np, "pulses-per-revolution", &ctl->ppr); 8412b410fa5e1e4 Nuno Sá 2019-10-09 338 if (ret) 8412b410fa5e1e4 Nuno Sá 2019-10-09 339 return ret; 8412b410fa5e1e4 Nuno Sá 2019-10-09 340 8412b410fa5e1e4 Nuno Sá 2019-10-09 341 /* 1, 2 and 4 are the typical and accepted values */ 8412b410fa5e1e4 Nuno Sá 2019-10-09 342 if (ctl->ppr != 1 && ctl->ppr != 2 && ctl->ppr != 4) 8412b410fa5e1e4 Nuno Sá 2019-10-09 343 return -EINVAL; 8412b410fa5e1e4 Nuno Sá 2019-10-09 344 /* 8412b410fa5e1e4 Nuno Sá 2019-10-09 345 * Enable all IRQs 8412b410fa5e1e4 Nuno Sá 2019-10-09 346 */ 8412b410fa5e1e4 Nuno Sá 2019-10-09 347 axi_iowrite(ADI_IRQ_MASK_OUT_ALL & 8412b410fa5e1e4 Nuno Sá 2019-10-09 348 ~(ADI_IRQ_SRC_NEW_MEASUR | ADI_IRQ_SRC_TACH_ERR | 8412b410fa5e1e4 Nuno Sá 2019-10-09 349 ADI_IRQ_SRC_PWM_CHANGED | ADI_IRQ_SRC_TEMP_INCREASE), 8412b410fa5e1e4 Nuno Sá 2019-10-09 350 ADI_REG_IRQ_MASK, ctl); 8412b410fa5e1e4 Nuno Sá 2019-10-09 351 8412b410fa5e1e4 Nuno Sá 2019-10-09 352 /* bring the device out of reset */ 8412b410fa5e1e4 Nuno Sá 2019-10-09 353 axi_iowrite(0x01, ADI_REG_RSTN, ctl); 8412b410fa5e1e4 Nuno Sá 2019-10-09 354 8412b410fa5e1e4 Nuno Sá 2019-10-09 @355 return ret; 8412b410fa5e1e4 Nuno Sá 2019-10-09 356 } 8412b410fa5e1e4 Nuno Sá 2019-10-09 357 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/[email protected] _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
