On 7/28/2024 7:41 AM, Gokul Sriram P wrote:
On 7/18/2024 9:59 PM, Jeffrey Hugo wrote:
On 7/18/2024 12:13 AM, Gokul Sriram Palanisamy wrote:
From: Karthick Shanmugham <quic_kartshan@xxxxxxxxxxx>
When mhi_irq_handler is a shared interrupt handler. It is the shared
interrupt handlers responsibility to identify its own interrupt and exit
quickly if it is not. If there is no pending event in the event ring
handled, exit the interrupt context returning IRQ_NONE denoting the
interrupt either doesn't belong to this event ring or not handled.
How was this found?
Internally, the proprietary cnss drivers shares the same msi interrupt
for streaming certain debug information.
In that scenario, the interrupt didn't reach the corresponding shared
handler since the mhi_irq_handler( ) returned IRQ_HANDLED when there is
actually no pending mhi event.
This resulted in debug information not collected.
You should probably explain this in the commit text.
Also, are you sure other parts of the irq handler don't need to be updated?
