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 <[email protected]> > Reviewed-by: AngeloGioacchino Del Regno > <[email protected]>
Reviewed-by: Daniel Thompson <[email protected]> > --- > 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 >
