Re: [PATCH] leds: lp3952: Store leds[LP3952_LED_ALL] in struct lp3952_led_array

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

 



On 07/18/2016 08:53 AM, Axel Lin wrote:


2016-07-18 14:48 GMT+08:00 Jacek Anaszewski <j.anaszewski@xxxxxxxxxxx
<mailto:j.anaszewski@xxxxxxxxxxx>>:

    Hi Axel,

    Thanks for catching this. I missed also redundant devm_kfree here.
    Would it be OK for you if I merged your cleanup with the original
    patch and added your Reviewed-by to the commit message instead?


That's fine, you can add Reviewed-by: Axel Lin <axel.lin@xxxxxxxxxx
<mailto:axel.lin@xxxxxxxxxx>>.

    This driver hasn't been pushed to the mainline yet, so it would be
    nice not to introduce its original version with some shortcomings
    that need to be immediately fixed.

Fixed up the original patch and updated for-next branch of
linux-leds.git.

--
Best regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux