Messages in this thread | | | Date | Tue, 5 Oct 2021 10:20:36 +0100 | From | Daniel Thompson <> | Subject | Re: [PATCH 06/10] backlight: qcom-wled: Remove unnecessary 4th default string in wled3 |
| |
On Mon, Oct 04, 2021 at 09:27:37PM +0200, Marijn Suijten wrote: > The previous commit improves num_strings parsing to not go over the > maximum of 3 strings for wled3 anymore. Likewise this default index for > a hypothetical 4th string is invalid and could access registers that are > not mapped to the desired purpose. > Removing this value gets rid of undesired confusion and avoids the > possibility of accessing registers at this offset even if the 4th array > element is used by accident. > > Signed-off-by: Marijn Suijten <marijn.suijten@somainline.org> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
> --- > drivers/video/backlight/qcom-wled.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/backlight/qcom-wled.c b/drivers/video/backlight/qcom-wled.c > index 66ce77ee3099..9ec1bdd374d2 100644 > --- a/drivers/video/backlight/qcom-wled.c > +++ b/drivers/video/backlight/qcom-wled.c > @@ -946,7 +946,7 @@ static const struct wled_config wled3_config_defaults = { > .cs_out_en = false, > .ext_gen = false, > .cabc = false, > - .enabled_strings = {0, 1, 2, 3}, > + .enabled_strings = {0, 1, 2}, > }; > > static int wled4_setup(struct wled *wled) > -- > 2.33.0 >
| |