github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-06-07 22:14:04 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

dmaengine: pl330: _loop_cyclic: fixup loopcnt is too large

Browse Source 
This patch fixup dma transfer data lost if loop cnt is larger
than 256.

Change-Id: Id49302cdcc1ac871d03070ce07eaa7653e54408c
Signed-off-by: Sugar Zhang <sugar.zhang@rock-chips.com>
This commit is contained in:
Sugar Zhang 2018-08-24 16:16:10 +08:00 committed by Tao Huang
parent 4e3b147bc1
commit 46020da40c
1 changed files with 42 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
drivers/dma/pl330.c
View File

@ -1350,19 +1350,14 @@ static inline int _loop(struct pl330_dmac *pl330, unsigned dry_run, u8 buf[],
return off;
}
static inline int _loop_cyclic(struct pl330_dmac *pl330, unsigned int dry_run,
u8 buf[], unsigned long bursts,
const struct _xfer_spec *pxs, int ev)
static int _period(struct pl330_dmac *pl330, unsigned int dry_run, u8 buf[],
unsigned long bursts, const struct _xfer_spec *pxs, int ev)
{
int cyc, off, num_dregs;
unsigned int lcnt0, lcnt1, ljmp0, ljmp1, ljmpfe;
unsigned int lcnt1, ljmp1;
int cyc, off = 0, num_dregs = 0;
struct _arg_LPEND lpend;
struct pl330_xfer *x = &pxs->desc->px;
off = 0;
ljmpfe = off;
lcnt0 = pxs->desc->num_periods;
if (bursts > 256) {
lcnt1 = 256;
cyc = bursts / 256;
@ -1371,14 +1366,6 @@ static inline int _loop_cyclic(struct pl330_dmac *pl330, unsigned int dry_run,
cyc = 1;
}
/* forever loop */
off += _emit_MOV(dry_run, &buf[off], SAR, x->src_addr);
off += _emit_MOV(dry_run, &buf[off], DAR, x->dst_addr);
/* loop0 */
off += _emit_LP(dry_run, &buf[off], 0, lcnt0);
ljmp0 = off;
/* loop1 */
off += _emit_LP(dry_run, &buf[off], 1, lcnt1);
ljmp1 = off;
@ -1419,12 +1406,50 @@ static inline int _loop_cyclic(struct pl330_dmac *pl330, unsigned int dry_run,
off += _emit_SEV(dry_run, &buf[off], ev);
return off;
}
static inline int _loop_cyclic(struct pl330_dmac *pl330, unsigned int dry_run,
u8 buf[], unsigned long bursts,
const struct _xfer_spec *pxs, int ev)
{
int off, periods, residue, i;
unsigned int lcnt0, ljmp0, ljmpfe;
struct _arg_LPEND lpend;
struct pl330_xfer *x = &pxs->desc->px;
off = 0;
ljmpfe = off;
lcnt0 = pxs->desc->num_periods;
periods = 1;
while (lcnt0 > 256) {
periods++;
lcnt0 = pxs->desc->num_periods / periods;
}
residue = pxs->desc->num_periods % periods;
/* forever loop */
off += _emit_MOV(dry_run, &buf[off], SAR, x->src_addr);
off += _emit_MOV(dry_run, &buf[off], DAR, x->dst_addr);
/* loop0 */
off += _emit_LP(dry_run, &buf[off], 0, lcnt0);
ljmp0 = off;
for (i = 0; i < periods; i++)
off += _period(pl330, dry_run, &buf[off], bursts, pxs, ev);
lpend.cond = ALWAYS;
lpend.forever = false;
lpend.loop = 0;
lpend.bjump = off - ljmp0;
off += _emit_LPEND(dry_run, &buf[off], &lpend);
for (i = 0; i < residue; i++)
off += _period(pl330, dry_run, &buf[off], bursts, pxs, ev);
lpend.cond = ALWAYS;
lpend.forever = true;
lpend.loop = 1;