On Thu, 3 Jan 2019 12:16:49 +0100 Michal Simek <michal.simek@xxxxxxxxxx> wrote: > On 27. 12. 18 20:54, Aditya Pakki wrote: > > In function xadc_probe, xadc_write_adc_reg can return an error value > > when write fails. The fix checks for the return value consistent with > > other invocations of the latter function. > > > > Signed-off-by: Aditya Pakki <pakki001@xxxxxxx> > > --- > > drivers/iio/adc/xilinx-xadc-core.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/iio/adc/xilinx-xadc-core.c b/drivers/iio/adc/xilinx-xadc-core.c > > index 3f6be5ac049a..b13c61539d46 100644 > > --- a/drivers/iio/adc/xilinx-xadc-core.c > > +++ b/drivers/iio/adc/xilinx-xadc-core.c > > @@ -1273,8 +1273,10 @@ static int xadc_probe(struct platform_device *pdev) > > xadc->threshold[i] = 0xffff; > > else > > xadc->threshold[i] = 0; > > - xadc_write_adc_reg(xadc, XADC_REG_THRESHOLD(i), > > + ret = xadc_write_adc_reg(xadc, XADC_REG_THRESHOLD(i), > > xadc->threshold[i]); > > + if (ret) > > + goto err_free_irq; > > } > > > > /* Go to non-buffered mode */ > > > > Reviewed-by: Michal Simek <michal.simek@xxxxxxxxxx> > Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > Thanks, > Michal
