Messages in this thread | | | Date | Tue, 24 May 2016 13:27:06 +0200 | Subject | Re: [PATCH 1/3] iio: st_sensors: Init trigger before irq request | From | Linus Walleij <> |
| |
On Fri, May 13, 2016 at 8:43 PM, Crestez Dan Leonard <leonard.crestez@intel.com> wrote:
> This fixes a possible race where an interrupt arrives before complete > initialization and crashes because iio_trigger_get_drvdata returns NULL. > > Cc: Linus Walleij <linus.walleij@linaro.org> > Cc: Giuseppe Barba <giuseppe.barba@st.com> > Cc: Denis Ciocca <denis.ciocca@st.com> > Signed-off-by: Crestez Dan Leonard <leonard.crestez@intel.com> > --- > I ran into this while breaking the driver. But since the interrupt line can be > shared the handler should always be able to accept and ignore a call.
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
However I think my patch fixing the thread issue will collide by being close in context to this so it'd be great if you could resend it on top of "iio: st_sensors: switch to a threaded interrupt"?
Yours, Linus Walleij
| |