mbox series

[v1,0/5] dma/xlnx-zdma: Bug fixes

Message ID 20200402134721.27863-1-edgar.iglesias@gmail.com (mailing list archive)
Headers show
Series dma/xlnx-zdma: Bug fixes | expand

Message

Edgar E. Iglesias April 2, 2020, 1:47 p.m. UTC
From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>

Hi,

This series fixes a couple of bugs we've ran into with some
proprietary test code and drivers using the Xilinx zDMA.

Cheers,
Edgar

Edgar E. Iglesias (5):
  dma/xlnx-zdma: Remove comment
  dma/xlnx-zdma: Populate DBG0.CMN_BUF_FREE
  dma/xlnx-zdma: Clear DMA_DONE when halting
  dma/xlnx-zdma: Advance the descriptor address when stopping
  dma/xlnx-zdma: Reorg to fix CUR_DSCR

 hw/dma/xlnx-zdma.c | 56 ++++++++++++++++++++++++----------------------
 1 file changed, 29 insertions(+), 27 deletions(-)

Comments

Peter Maydell April 3, 2020, 6:53 p.m. UTC | #1
On Thu, 2 Apr 2020 at 14:46, Edgar E. Iglesias <edgar.iglesias@gmail.com> wrote:
>
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>
> Hi,
>
> This series fixes a couple of bugs we've ran into with some
> proprietary test code and drivers using the Xilinx zDMA.

Thanks; as these are bugfixes I've applied them to target-arm.next
for 5.0. I'd appreciate it if you could take a look at the
endianness bugs I mention in a reply to patch 5 (probably
that is 5.1 work).

-- PMM
Edgar E. Iglesias April 4, 2020, 12:29 p.m. UTC | #2
On Fri, Apr 03, 2020 at 07:53:24PM +0100, Peter Maydell wrote:
> On Thu, 2 Apr 2020 at 14:46, Edgar E. Iglesias <edgar.iglesias@gmail.com> wrote:
> >
> > From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
> >
> > Hi,
> >
> > This series fixes a couple of bugs we've ran into with some
> > proprietary test code and drivers using the Xilinx zDMA.
> 
> Thanks; as these are bugfixes I've applied them to target-arm.next
> for 5.0. I'd appreciate it if you could take a look at the
> endianness bugs I mention in a reply to patch 5 (probably
> that is 5.1 work).


Thanks for pointing those out!

I've just posted fixes for the endianness related descriptor loading
bugs but left some of the C struct issues for 5.1.

Cheers,
Edgar