Re: [PATCH v3 2/2] leds: lm3601x: Introduce the lm3601x LED driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dan,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on robh/for-next]
[also build test WARNING on v4.17-rc4 next-20180509]
[cannot apply to j.anaszewski-leds/for-next]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Dan-Murphy/dt-bindings-lm3601x-Introduce-the-lm3601x-driver/20180509-232022
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next


coccinelle warnings: (new ones prefixed by >>)

>> drivers/leds/leds-lm3601x.c:241:35-36: WARNING: Use ARRAY_SIZE

vim +241 drivers/leds/leds-lm3601x.c

   224	
   225	static int lm3601x_strobe_set(struct led_classdev_flash *fled_cdev,
   226					bool state)
   227	{
   228		struct lm3601x_led *led = fled_cdev_to_led(fled_cdev);
   229		int ret;
   230		int current_timeout;
   231		int reg_count;
   232		int i;
   233		int timeout_reg_val = 0;
   234	
   235		mutex_lock(&led->lock);
   236	
   237		ret = regmap_read(led->regmap, LM3601X_CFG_REG, &current_timeout);
   238		if (ret < 0)
   239			goto out;
   240	
 > 241		reg_count = sizeof(strobe_timeouts) / sizeof(strobe_timeouts[0]);
   242		for (i = 0; i < reg_count; i++) {
   243			if (led->strobe_timeout > strobe_timeouts[i].timeout)
   244				continue;
   245	
   246			if (led->strobe_timeout <= strobe_timeouts[i].timeout) {
   247				timeout_reg_val = (strobe_timeouts[i].reg_val << 1);
   248				break;
   249			}
   250	
   251			ret = -EINVAL;
   252			goto out;
   253		}
   254	
   255		if (led->strobe_timeout != current_timeout)
   256			ret = regmap_update_bits(led->regmap, LM3601X_CFG_REG,
   257						0x1e, timeout_reg_val);
   258	
   259		if (state)
   260			ret = regmap_update_bits(led->regmap, LM3601X_ENABLE_REG,
   261						LM3601X_MODE_STROBE,
   262						LM3601X_MODE_STROBE);
   263		else
   264			ret = regmap_update_bits(led->regmap, LM3601X_ENABLE_REG,
   265						LM3601X_MODE_STROBE, LED_OFF);
   266	
   267		ret = lm3601x_read_faults(led);
   268	out:
   269		mutex_unlock(&led->lock);
   270		return ret;
   271	}
   272	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux