Skip to content

Commit 54c8b5b

Browse files
Youri QuerryLi Yang
authored andcommitted
soc: fsl: dpio: rename the enqueue descriptor variable
The struct qbman_eq_desc 'd' variable declaration is covering one of the function parameters. This has no functional impact since this function parameter was not used after the new declaration. Even so, rename the variable so that we make the code more readable. Fixes: 3b2abda ("soc: fsl: dpio: Replace QMAN array mode with ring mode enqueue") Signed-off-by: Youri Querry <[email protected]> Signed-off-by: Ioana Ciornei <[email protected]> Signed-off-by: Li Yang <[email protected]>
1 parent a7ff7dc commit 54c8b5b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

drivers/soc/fsl/dpio/qbman-portal.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -688,9 +688,9 @@ int qbman_swp_enqueue_multiple_direct(struct qbman_swp *s,
688688
p = (s->addr_cena + QBMAN_CENA_SWP_EQCR(eqcr_pi & half_mask));
689689
p[0] = cl[0] | s->eqcr.pi_vb;
690690
if (flags && (flags[i] & QBMAN_ENQUEUE_FLAG_DCA)) {
691-
struct qbman_eq_desc *d = (struct qbman_eq_desc *)p;
691+
struct qbman_eq_desc *eq_desc = (struct qbman_eq_desc *)p;
692692

693-
d->dca = (1 << QB_ENQUEUE_CMD_DCA_EN_SHIFT) |
693+
eq_desc->dca = (1 << QB_ENQUEUE_CMD_DCA_EN_SHIFT) |
694694
((flags[i]) & QBMAN_EQCR_DCA_IDXMASK);
695695
}
696696
eqcr_pi++;
@@ -770,9 +770,9 @@ int qbman_swp_enqueue_multiple_mem_back(struct qbman_swp *s,
770770
p = (s->addr_cena + QBMAN_CENA_SWP_EQCR(eqcr_pi & half_mask));
771771
p[0] = cl[0] | s->eqcr.pi_vb;
772772
if (flags && (flags[i] & QBMAN_ENQUEUE_FLAG_DCA)) {
773-
struct qbman_eq_desc *d = (struct qbman_eq_desc *)p;
773+
struct qbman_eq_desc *eq_desc = (struct qbman_eq_desc *)p;
774774

775-
d->dca = (1 << QB_ENQUEUE_CMD_DCA_EN_SHIFT) |
775+
eq_desc->dca = (1 << QB_ENQUEUE_CMD_DCA_EN_SHIFT) |
776776
((flags[i]) & QBMAN_EQCR_DCA_IDXMASK);
777777
}
778778
eqcr_pi++;

0 commit comments

Comments
 (0)