RE: [PATCH] iio: core: Propagate error codes from OF layer to client drivers
From: Opensource [Adam Thomson]
Date: Tue Aug 26 2014 - 10:42:51 EST
On August 26, 2014 14:48, Ivan T. Ivanov wrote:
> On Tue, 2014-08-26 at 06:25 -0700, Guenter Roeck wrote:
> > On 08/26/2014 12:51 AM, Ivan T. Ivanov wrote:
> > > Hi,
> > >
> > [ ... ]
> > >
> > > Another, less intrusive, solution will be if we revert last patch and explicitly
> > > check for EPROBE_DEFER on of_ by_name() return. How this sounds?
> > >
> > How is that different to the just accepted patch ?
> You mean this one. I have prepared fix last Friday and forget to
> check again before posting it, sorry. Please ignore my patch.
>  iio:inkern: fix overwritten -EPROBE_DEFER in of_iio_channel_get_by_name
Apologies on my part for fixing one problem and introducing another. Didn't see
that in my testing, and missed that potential return value when examining the
code. At the time It looked like the parent function would only expect NULL
pointers for failure, especially given the non CONFIG_OF definitions of the
functions. Should've delved further. :(