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[1]. I have prepared fix last Friday and forget to > check again before posting it, sorry. Please ignore my patch. > > Thanks, > Ivan > > [1] 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. :( ��.n��������+%������w��{.n�����{��(��)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥