Some macros changed to inline function. There is only one container_of it those functions. Sleep times adjusted to shorter ones. Data sheet gives some absolute minimum times but selected times contains some margin to add overall robustness. Comments added to sleep times. Chips don't have HW reset line. If the chip is connected to always on power rail, there is no power on ramp either at system restart. It is possible that the chip is in some odd state at restart and the detection may fail. SW reset command is now send before detection to set chip to known state for the detection. Patches are done to top my lp5523 and lp5521 driver patches which are waiting to go in. Samu Onkalo (6): leds: lp5521: Change some macros to functions leds: lp5523: Change some macros to functions leds: lp5521: Adjust delays and add comments to them leds: lp5523: Adjust delays and add comments to them leds: lp5521: Perform SW reset before detection leds: lp5523: Perform SW reset before detection drivers/leds/leds-lp5521.c | 50 +++++++++++++++++++++++++++++--------------- drivers/leds/leds-lp5523.c | 44 +++++++++++++++++++++----------------- 2 files changed, 57 insertions(+), 37 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html