[v3,0/2] Qcom Geni exit path cleanups

Message

Andi Shyti Dec. 27, 2024, 10:32 p.m. UTC

Hi,

I am submitting two trivial cleanups in this series. The first
replaces all instances of dev_err with dev_err_probe throughout
the probe function for consistency. The second improves the error
exit path by introducing a single 'goto' label for better
maintainability.

Thank you Mukesh and Vladimir for your reviews.

Thank you,
Andi

Changelog:
----------
v2 -> v3:
 - Don't forget to assign ret the proper value.
 - Add tags in patch 1.
v1 -> v2:
 - Updated the final return statement to return 'ret' instead of
   '0' for consistency. Since 'ret' already holds the value '0',
   this change is purely aesthetic.
 - Renamed the exit label from 'err_off' to 'err_resources' for
   improved clarity and alignment with its purpose.

Andi Shyti (2):
  i2c: qcom-geni: Use dev_err_probe in the probe function
  i2c: qcom-geni: Simplify error handling in probe function

 drivers/i2c/busses/i2c-qcom-geni.c | 56 ++++++++++++++----------------
 1 file changed, 27 insertions(+), 29 deletions(-)

Comments

Andi Shyti Dec. 30, 2024, 2:02 a.m. UTC | #1

Hi,

> Andi Shyti (2):
>   i2c: qcom-geni: Use dev_err_probe in the probe function
>   i2c: qcom-geni: Simplify error handling in probe function

merged to i2c/i2c-host.

Thanks,
Andi