Skip to content

Commit

Permalink
soc: fsl: dpio: remove set but not used 'addr_cena'
Browse files Browse the repository at this point in the history
This addresses the following gcc warning with "make W=1":

drivers/soc/fsl/dpio/qbman-portal.c: In function
‘qbman_swp_enqueue_multiple_direct’:
drivers/soc/fsl/dpio/qbman-portal.c:650:11: warning: variable
‘addr_cena’ set but not used [-Wunused-but-set-variable]
  650 |  uint64_t addr_cena;
      |           ^~~~~~~~~

Reported-by: Hulk Robot <[email protected]>
Signed-off-by: Jason Yan <[email protected]>
Reported-by: kernel test robot <[email protected]>
Reviewed-by: Krzysztof Kozlowski <[email protected]>
Signed-off-by: Li Yang <[email protected]>
  • Loading branch information
JasonYanHw authored and Li Yang committed Sep 22, 2020
1 parent 750cf40 commit 72f7fe2
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions drivers/soc/fsl/dpio/qbman-portal.c
Original file line number Diff line number Diff line change
Expand Up @@ -647,7 +647,6 @@ int qbman_swp_enqueue_multiple_direct(struct qbman_swp *s,
const uint32_t *cl = (uint32_t *)d;
uint32_t eqcr_ci, eqcr_pi, half_mask, full_mask;
int i, num_enqueued = 0;
uint64_t addr_cena;

spin_lock(&s->access_spinlock);
half_mask = (s->eqcr.pi_ci_mask>>1);
Expand Down Expand Up @@ -701,7 +700,6 @@ int qbman_swp_enqueue_multiple_direct(struct qbman_swp *s,

/* Flush all the cacheline without load/store in between */
eqcr_pi = s->eqcr.pi;
addr_cena = (size_t)s->addr_cena;
for (i = 0; i < num_enqueued; i++)
eqcr_pi++;
s->eqcr.pi = eqcr_pi & full_mask;
Expand Down

0 comments on commit 72f7fe2

Please sign in to comment.