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.barysh...@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.armstr...@linaro.org>

Reply via email to