Hi Ludovic, On Thu, 28 Nov 2019 at 15:06, Ludovic BARRE <ludovic.barre@xxxxxx> wrote: > > hi Ulf > > just a gentleman ping about this thread. > > small summarize: > This patch return an IRQ_WAKE_THREAD only when the variant is > busy_timeout capable and a datatimeout occurs on R1B request. > > So the threaded irq is called only to treat this specific error. > Normally, there is no impact on HW flow control or for legacy variants. Yes, this should work. > > In your previous message, you seem to suggest using threaded irq to > manage HW flow control (pio mode). But Like you mention below, the mmci > legacy could timing sensitive. > > For the moment, I prefer to use the threaded irq just to manage this > error. If needed, the irq threade could be extended later. > > What do you think about that? Yes, that's fine! I have another minor comment on the code, though, but posting that separately. [...] Kind regards Uffe
