[v2] hw/dma: Align SiFive PDMA behavior with real hardware

[PATCH v2 0/4] hw/dma: Align SiFive PDMA behavior with real hardware

Posted by frank.chang@sifive.com 2 years, 7 months ago

From: Frank Chang <frank.chang@sifive.com>

Current QEMU PDMA doesn't align with real PDMA's behavior. This would
result in Linux dmatest failed. This patchest aligns with real PDMA's
behavior we tested on the real board. The golden results are performed
in U-Boot on the Unmatched board with PDMA supported.

Changelog:

v2:
  * Add comment to describe document errors.

Frank Chang (3):
  hw/dma: sifive_pdma: reset Next* registers when Control.claim is set
  hw/dma: sifive_pdma: claim bit must be set before DMA transactions
  hw/dma: sifive_pdma: don't set Control.error if 0 bytes to transfer

Green Wan (1):
  hw/dma: sifive_pdma: allow non-multiple transaction size transactions

 hw/dma/sifive_pdma.c | 54 +++++++++++++++++++++++++++++++++++---------
 1 file changed, 43 insertions(+), 11 deletions(-)

--
2.25.1

Re: [PATCH v2 0/4] hw/dma: Align SiFive PDMA behavior with real hardware

Posted by Frank Chang 2 years, 7 months ago

<frank.chang@sifive.com> 於 2021年9月12日 週日 下午8:46寫道：

> From: Frank Chang <frank.chang@sifive.com>
>
> Current QEMU PDMA doesn't align with real PDMA's behavior. This would
> result in Linux dmatest failed. This patchest aligns with real PDMA's
> behavior we tested on the real board. The golden results are performed
> in U-Boot on the Unmatched board with PDMA supported.
>
> Changelog:
>
> v2:
>   * Add comment to describe document errors.
>
> Frank Chang (3):
>   hw/dma: sifive_pdma: reset Next* registers when Control.claim is set
>   hw/dma: sifive_pdma: claim bit must be set before DMA transactions
>   hw/dma: sifive_pdma: don't set Control.error if 0 bytes to transfer
>
> Green Wan (1):
>   hw/dma: sifive_pdma: allow non-multiple transaction size transactions
>
>  hw/dma/sifive_pdma.c | 54 +++++++++++++++++++++++++++++++++++---------
>  1 file changed, 43 insertions(+), 11 deletions(-)
>
> --
> 2.25.1
>
>
>
I made a mistake while sending the patchset.
Please ignore this series, I will resend it again.
Sorry for the confusion.

Regards,
Frank Chang