Add small changes to address the review comments from Paval which were posted after the driver was applied: https://lore.kernel.org/linux-arm-msm/20230325170957.GA2904@bug/#t https://lore.kernel.org/linux-arm-msm/ZB8wbv93+Vmx3trt@xxxxxxxxxx/ 1) Add a sentence in Kconfig to explain the driver can be compiled as a module 2) Strobe off the LED channel before setting flash current 3) Put the child node while register flash LED device failed. Changes in v2: Sperate the fixes into 3 independent changes. Signed-off-by: Fenglin Wu <quic_fenglinw@xxxxxxxxxxx> --- Fenglin Wu (3): leds: flash: leds-qcom-flash: declare the driver as a module leds: flash: leds-qcom-flash: turn off LED before setting flash current leds: flash: leds-qcom-flash: put child node if registration failed drivers/leds/flash/Kconfig | 2 ++ drivers/leds/flash/leds-qcom-flash.c | 5 +++++ 2 files changed, 7 insertions(+) --- base-commit: 7a2b85a76b05f16e8e788d035afcacdd4faa1d71 change-id: 20230725-leds-qcom-flash-driver-tiny-fixes-f6fd8d90eb94 Best regards, -- Fenglin Wu <quic_fenglinw@xxxxxxxxxxx>
