[-next,0/2] iio: adc: mt6577_auxadc: Cleanup with the helpers

Message

Jinjie Ruan Aug. 26, 2023, 3:54 a.m. UTC

Use the dev_err_probe() helper to simplify error handling during probe.
This also handle scenario, when EDEFER is returned and useless
error is printed.

And use devm_add_action_or_reset() and devm_iio_device_register()
to simplify the code.

Jinjie Ruan (2):
  iio: adc: mt6577_auxadc: Simplify with dev_err_probe()
  iio: adc: mt6577_auxadc: Simplify with device managed function

 drivers/iio/adc/mt6577_auxadc.c | 60 +++++++++++++--------------------
 1 file changed, 23 insertions(+), 37 deletions(-)

Comments

Jonathan Cameron Aug. 27, 2023, 5:07 p.m. UTC | #1

On Sat, 26 Aug 2023 11:54:00 +0800
Jinjie Ruan <[e-mail]> wrote:

> Use the dev_err_probe() helper to simplify error handling during probe.
> This also handle scenario, when EDEFER is returned and useless
> error is printed.

Just a quick note to the driver maintainers.  Whilst I've applied this,
I won't be pushing that out as a non rebasing tree for a few weeks.
As such I'm very happy to add tags, or indeed to drop it if you see
any changes you would like.

Taking it quickly is mostly because I have a huge backlog having been
away and want to get the simple stuff off that list!

Thanks,

Jonathan

> 
> And use devm_add_action_or_reset() and devm_iio_device_register()
> to simplify the code.
> 
> Jinjie Ruan (2):
>   iio: adc: mt6577_auxadc: Simplify with dev_err_probe()
>   iio: adc: mt6577_auxadc: Simplify with device managed function
> 
>  drivers/iio/adc/mt6577_auxadc.c | 60 +++++++++++++--------------------
>  1 file changed, 23 insertions(+), 37 deletions(-)
>