Messages in this thread | | | Date | Wed, 24 Apr 2024 08:53:17 +0200 | From | Neil Armstrong <> | Subject | Re: [PATCH 5/6] drm/panel: novatek-nt36672a: stop calling regulator_set_load manually |
| |
On 04/04/2024 12:08, Dmitry Baryshkov 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@linaro.org> > --- > 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"); > - } > - > pinfo->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW); > if (IS_ERR(pinfo->reset_gpio)) > return dev_err_probe(dev, PTR_ERR(pinfo->reset_gpio), >
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
| |