Re: [PATCH v2 00/13] OMAP mailbox FIFO removal

2024-06-26 Thread Andrew Davis
On 6/26/24 9:39 AM, Dominic Rath wrote: On 13.06.2024 14:22, Andrew Davis wrote: We looked into this some time ago, and noticed that the IRQ approach caused problems in the virtio/rpmsg code. I'd like to understand if your change was for the same reason, or something else we missed before.

Re: [PATCH v2 00/13] OMAP mailbox FIFO removal

2024-06-26 Thread Dominic Rath
On 13.06.2024 14:22, Andrew Davis wrote: We looked into this some time ago, and noticed that the IRQ approach caused problems in the virtio/rpmsg code. I'd like to understand if your change was for the same reason, or something else we missed before. It is most likely the same reason. Seems

Re: [PATCH v2 00/13] OMAP mailbox FIFO removal

2024-06-14 Thread Dominic Rath
On 13.06.2024 14:22, Andrew Davis wrote: We looked into this some time ago, and noticed that the IRQ approach caused problems in the virtio/rpmsg code. I'd like to understand if your change was for the same reason, or something else we missed before. It is most likely the same reason. Seems

Re: [PATCH v2 00/13] OMAP mailbox FIFO removal

2024-06-13 Thread Andrew Davis
On 6/13/24 2:58 AM, Dominic Rath wrote: Hello Andrew, On 10.04.2024 15:59, Andrew Davis wrote: Changes for v2:   - Use threaded irq as suggested by Hari and to   fix possible "scheduling while atomic" issue sorry for beeing late, I noticed this already got merged. I was wondering what

Re: [PATCH v2 00/13] OMAP mailbox FIFO removal

2024-06-13 Thread Dominic Rath
Hello Andrew, On 10.04.2024 15:59, Andrew Davis wrote: Changes for v2: - Use threaded irq as suggested by Hari and to fix possible "scheduling while atomic" issue sorry for beeing late, I noticed this already got merged. I was wondering what the reason was for ending up with the

[PATCH v2 00/13] OMAP mailbox FIFO removal

2024-04-10 Thread Andrew Davis
Hello all, Core of this series is the last patch removing the message FIFO from OMAP mailbox. This hurts our real-time performance. It was a legacy leftover from before the common mailbox framework anyway. The rest of the patches are cleanups found along the way. Thanks, Andrew Changes for v2: