[PATCH 0/3] regulator: Minor cleanups

Chen-Yu Tsai wenst at chromium.org
Thu Aug 22 00:20:43 PDT 2024


Hi,

Here are some cleanups for some bits that I saw while reworking my I2C
device tree component prober to use of_regulator_bulk_get_all().
These are not directly related to that series, so I send them
separately here.

Patch 1 makes the regulator_get*() function misuse error message easier
to understand.

Patch 2 makes of_regulator_bulk_get_all() pass out the error code from
regulator_get() instead of replacing it with -EINVAL.

Patch 3 makes of_regulator_bulk_get_all() clean up *everything* and not
touch the passed in pointer on error, and explicitly documents this
behavior.

Please have a look and merge if possible.


Thanks
ChenYu

[1] https://lore.kernel.org/linux-arm-kernel/20240808095931.2649657-1-wenst@chromium.org/

Chen-Yu Tsai (3):
  regulator: Clarify error message for "id == NULL" in _regulator_get()
  regulator: Return actual error in of_regulator_bulk_get_all()
  regulator: Fully clean up on error in of_regulator_bulk_get_all()

 drivers/regulator/core.c         |  2 +-
 drivers/regulator/of_regulator.c | 23 +++++++++++++----------
 2 files changed, 14 insertions(+), 11 deletions(-)

-- 
2.46.0.184.g6999bdac58-goog




More information about the linux-arm-kernel mailing list