Re: [PATCH v3 1/3] w1: ds2482: Add regulator support

From: Stefan Wahren
Date: Mon Nov 25 2024 - 10:49:45 EST


Hi Kryštof,

+ ret = devm_regulator_get_enable(&client->dev, "vcc");
+ if (ret) {
+ dev_err(&client->dev, "Fail to enable regulator\n");
+ return ret;
+ }


I'm fine with using devm_regulator_get_enable() because it doesn't break
existing DTS like imx6ull-tarragon-master, which doesn't have vcc defined.
But please use dev_err_probe() here instead and s/Fail/Failed/ in the
error message.

Best regards