On Wed, Jan 15, 2025 at 06:22:55PM +0000, Conor Dooley wrote:
Moving it out of the interrupt handler entirely probably makes the code a lot easier to reason about, so I think I'd actually rather do that. That said, I don't think I would want that change going into fixes as it's a pretty intrusive change, although given where we are in the cycle it ain't as much of a problem since I'd not be sending it until after 6.13 is released anyway.
OTOH the patch you sent will lock up hard if the RX FIFO gets stuck for some other reason which is always a bit concerning...