[PATCH v2 5/6] usb: dwc3: exynos: Use devm_regulator_bulk_get_enable() helper function
Krzysztof Kozlowski
krzysztof.kozlowski at linaro.org
Thu Apr 4 00:23:30 PDT 2024
On 04/04/2024 09:13, Anand Moon wrote:
> Use devm_regulator_bulk_get_enable() instead of open coded
> 'devm_regulator_get(), regulator_enable(), regulator_disable().
I fail to see how did you replace open-coded suspend/resume paths.
>
> Signed-off-by: Anand Moon <linux.amoon at gmail.com>
> ---
> V2: no changes, did not find any regression in pm suspend/resume.
No, that's not equivalent code. No explanation in commit msg.
You already got comments on this and nothing improved. You just entirely
ignored received comments. That's not how it works.
I don't think you understand the code and Linux driver model. This patch
repeats several previous attempts with similar issues: no logic behind a
change.
NAK.
Best regards,
Krzysztof
More information about the linux-arm-kernel
mailing list