Messages in this thread | | | Date | Wed, 20 Dec 2023 09:12:40 +0100 | From | Johan Hovold <> | Subject | Re: [RESEND PATCH] leds: ledtrig-pattern: Use last_repeat when applying hw pattern |
| |
On Tue, Jul 19, 2022 at 11:30:33PM +0200, Marijn Suijten wrote: > `last_repeat` holds the actual value requested by the user whereas > `repeat` is a software iteration variable that is unused in hardware > patterns. > > Furthermore `last_repeat` is the field returned to the user when reading > the `repeat` sysfs property. This field is initialized to `-1` which is > - together with `1` - the only valid value in the upcoming Qualcomm LPG > driver. It is thus unexpected when `repeat` with an initialization > value of `0` is passed into the the driver, when the sysfs property > clearly presents a value of `-1`. > > Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org> > Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> > Tested-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Looks correct to me:
Reviewed-by: Johan Hovold <johan+linaro@kernel.org>
| |