[v4] iio: adc: bcm_iproc_adc: Simplify probe error handling

[PATCH v4 0/3] iio: adc: bcm_iproc_adc: Simplify probe error handling

mdshahid03@gmail.com posted 3 patches 2 months ago

Diff against v3 v5 v6
Download series mbox

There is a newer version of this series

drivers/iio/adc/bcm_iproc_adc.c | 54 +++++++++++++++------------------
1 file changed, 24 insertions(+), 30 deletions(-)

Expand all Fold all

[PATCH v4 0/3] iio: adc: bcm_iproc_adc: Simplify probe error handling

Posted by mdshahid03@gmail.com 2 months ago

From: Mohammad Shahid <mdshahid03@gmail.com>

Hi,

This series simplifies the probe error handling in bcm_iproc_adc.

Changes in v4:
- Move the redundant iproc_adc_enable() error message removal into the
  cleanup patch as suggested by Joshua Crofts.

Mohammad Shahid (3):
  iio: adc: bcm_iproc_adc: Remove redundant probe error messages
  iio: adc: bcm_iproc_adc: Introduce local device pointer
  iio: adc: bcm_iproc_adc: Convert probe error handling to
    dev_err_probe()

 drivers/iio/adc/bcm_iproc_adc.c | 54 +++++++++++++++------------------
 1 file changed, 24 insertions(+), 30 deletions(-)

-- 
2.43.0

Re: [PATCH v4 0/3] iio: adc: bcm_iproc_adc: Simplify probe error handling

Posted by Jonathan Cameron 2 months ago

On Sat,  1 Aug 2026 14:39:50 +0530
mdshahid03@gmail.com wrote:

> From: Mohammad Shahid <mdshahid03@gmail.com>
> 
> Hi,
> 
> This series simplifies the probe error handling in bcm_iproc_adc.
Hi Mohammad

See comments on v3 + slow down. It wastes your time and that of reviewers
if you post new versions too quickly.

Jonathan

> 
> Changes in v4:
> - Move the redundant iproc_adc_enable() error message removal into the
>   cleanup patch as suggested by Joshua Crofts.
> 
> Mohammad Shahid (3):
>   iio: adc: bcm_iproc_adc: Remove redundant probe error messages
>   iio: adc: bcm_iproc_adc: Introduce local device pointer
>   iio: adc: bcm_iproc_adc: Convert probe error handling to
>     dev_err_probe()
> 
>  drivers/iio/adc/bcm_iproc_adc.c | 54 +++++++++++++++------------------
>  1 file changed, 24 insertions(+), 30 deletions(-)
>