* [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli
@ 2016-11-02 5:31 Axl-zhang
2016-11-02 18:58 ` Maxime Ripard
2016-11-14 5:06 ` Vinod Koul
0 siblings, 2 replies; 3+ messages in thread
From: Axl-zhang @ 2016-11-02 5:31 UTC (permalink / raw)
To: Maxime Ripard, Chen-Yu Tsai, Dan Williams, Vinod Koul
Cc: dmaengine, linux-kernel, linux-arm-kernel, hao5781286
dma_pool_alloc does not initialize the value of the newly allocated
block for the v_lli, and the uninitilize value make the tests failed
which is on pine64 with dmatest.
we can fix it just change the "|=" to "=" for the v_lli->cfg.
Signed-off-by: Hao Zhang <hao5781286@gmail.com>
---
drivers/dma/sun6i-dma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/sun6i-dma.c b/drivers/dma/sun6i-dma.c
index 3835fcd..52b48eb 100644
--- a/drivers/dma/sun6i-dma.c
+++ b/drivers/dma/sun6i-dma.c
@@ -578,7 +578,7 @@ static struct dma_async_tx_descriptor *sun6i_dma_prep_dma_memcpy(
burst = convert_burst(8);
width = convert_buswidth(DMA_SLAVE_BUSWIDTH_4_BYTES);
- v_lli->cfg |= DMA_CHAN_CFG_SRC_DRQ(DRQ_SDRAM) |
+ v_lli->cfg = DMA_CHAN_CFG_SRC_DRQ(DRQ_SDRAM) |
DMA_CHAN_CFG_DST_DRQ(DRQ_SDRAM) |
DMA_CHAN_CFG_DST_LINEAR_MODE |
DMA_CHAN_CFG_SRC_LINEAR_MODE |
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli
2016-11-02 5:31 [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli Axl-zhang
@ 2016-11-02 18:58 ` Maxime Ripard
2016-11-14 5:06 ` Vinod Koul
1 sibling, 0 replies; 3+ messages in thread
From: Maxime Ripard @ 2016-11-02 18:58 UTC (permalink / raw)
To: Axl-zhang
Cc: Chen-Yu Tsai, Dan Williams, Vinod Koul, dmaengine, linux-kernel,
linux-arm-kernel
[-- Attachment #1: Type: text/plain, Size: 544 bytes --]
On Wed, Nov 02, 2016 at 01:31:12PM +0800, Axl-zhang wrote:
> dma_pool_alloc does not initialize the value of the newly allocated
> block for the v_lli, and the uninitilize value make the tests failed
> which is on pine64 with dmatest.
> we can fix it just change the "|=" to "=" for the v_lli->cfg.
>
> Signed-off-by: Hao Zhang <hao5781286@gmail.com>
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Thanks!
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli
2016-11-02 5:31 [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli Axl-zhang
2016-11-02 18:58 ` Maxime Ripard
@ 2016-11-14 5:06 ` Vinod Koul
1 sibling, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2016-11-14 5:06 UTC (permalink / raw)
To: Axl-zhang
Cc: Maxime Ripard, Chen-Yu Tsai, Dan Williams, dmaengine,
linux-kernel, linux-arm-kernel
On Wed, Nov 02, 2016 at 01:31:12PM +0800, Axl-zhang wrote:
> dma_pool_alloc does not initialize the value of the newly allocated
> block for the v_lli, and the uninitilize value make the tests failed
> which is on pine64 with dmatest.
> we can fix it just change the "|=" to "=" for the v_lli->cfg.
Applied after fixing the title. We dont need ARM.
Also spaces after each tag please..
--
~Vinod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-11-14 4:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-02 5:31 [PATCH v1] ARM:dmaengine:sun6i:fix the uninitialized value for v_lli Axl-zhang
2016-11-02 18:58 ` Maxime Ripard
2016-11-14 5:06 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).