On Mon, Jun 19, 2023 at 08:16:47PM +0200, Köry Maincent wrote: > On Mon, 19 Jun 2023 00:48:00 +0300 > Serge Semin <fancer.lancer@xxxxxxxxx> wrote: > > > Seems reasonable especially seeing there is a code with a similar > > semantic in the dw_hdma_v0_core_write_chunk() method. > > > > Reviewed-by: Serge Semin <fancer.lancer@xxxxxxxxx> > > > > Just curious whether we really need to have the local IRQs left > > enabled for the remote device setup... The only case I have in mind is > > that it would be useful to signal a remote end-point host of such > > event in some application-specific environment. It sounds exotic but > > still possible. > > Thanks for your review. > Yes, we do need to let local IRQs enabled. I have tested to remove them and it > prevent the remote setup to function correctly on my board. Maybe it needs to be > set to know internally when the transfer is done, but it seems weird. I haven't > a full explanation for now. Ok. Thanks for checking it out. -Serge(y)
