Hi, On Thu, Apr 4, 2024 at 3:08 AM Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > > Use .init_load_uA part of the bulk regulator API instead of calling > register_set_load() manually. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > drivers/gpu/drm/panel/panel-novatek-nt36672a.c | 11 +++-------- > 1 file changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/panel/panel-novatek-nt36672a.c b/drivers/gpu/drm/panel/panel-novatek-nt36672a.c > index 33fb3d715e54..3886372415c2 100644 > --- a/drivers/gpu/drm/panel/panel-novatek-nt36672a.c > +++ b/drivers/gpu/drm/panel/panel-novatek-nt36672a.c > @@ -605,21 +605,16 @@ static int nt36672a_panel_add(struct nt36672a_panel *pinfo) > struct device *dev = &pinfo->link->dev; > int i, ret; > > - for (i = 0; i < ARRAY_SIZE(pinfo->supplies); i++) > + for (i = 0; i < ARRAY_SIZE(pinfo->supplies); i++) { > pinfo->supplies[i].supply = nt36672a_regulator_names[i]; > + pinfo->supplies[i].init_load_uA = nt36672a_regulator_enable_loads[i]; > + } > > ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(pinfo->supplies), > pinfo->supplies); > if (ret < 0) > return dev_err_probe(dev, ret, "failed to get regulators\n"); > > - for (i = 0; i < ARRAY_SIZE(pinfo->supplies); i++) { > - ret = regulator_set_load(pinfo->supplies[i].consumer, > - nt36672a_regulator_enable_loads[i]); > - if (ret) > - return dev_err_probe(dev, ret, "failed to set regulator enable loads\n"); > - } Thanks for the cleanup! I happened to notice this commit and I'm curious why you didn't take the next step and move to devm_regulator_bulk_get_const(). That would have allowed you to avoid the "for" loop above that copies the data ;-) -Doug