Commit 1b30b989 authored by Vakul Garg's avatar Vakul Garg Committed by Herbert Xu

crypto: caam/jr - Removed redundant vars from job ring private data

For each job ring, the variable 'ringsize' is initialised but never
used. Similarly variables 'inp_ring_write_index' and 'head' always track
the same value and instead of 'inp_ring_write_index', caam_jr_enqueue()
can use 'head' itself. Both these variables have been removed.
Signed-off-by: default avatarVakul Garg <vakul.garg@nxp.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent a118dfa0
...@@ -49,10 +49,8 @@ struct caam_drv_private_jr { ...@@ -49,10 +49,8 @@ struct caam_drv_private_jr {
atomic_t tfm_count ____cacheline_aligned; atomic_t tfm_count ____cacheline_aligned;
/* Job ring info */ /* Job ring info */
int ringsize; /* Size of rings (assume input = output) */
struct caam_jrentry_info *entinfo; /* Alloc'ed 1 per ring entry */ struct caam_jrentry_info *entinfo; /* Alloc'ed 1 per ring entry */
spinlock_t inplock ____cacheline_aligned; /* Input ring index lock */ spinlock_t inplock ____cacheline_aligned; /* Input ring index lock */
int inp_ring_write_index; /* Input index "tail" */
u32 inpring_avail; /* Number of free entries in input ring */ u32 inpring_avail; /* Number of free entries in input ring */
int head; /* entinfo (s/w ring) head index */ int head; /* entinfo (s/w ring) head index */
dma_addr_t *inpring; /* Base of input ring, alloc DMA-safe */ dma_addr_t *inpring; /* Base of input ring, alloc DMA-safe */
......
...@@ -358,7 +358,7 @@ int caam_jr_enqueue(struct device *dev, u32 *desc, ...@@ -358,7 +358,7 @@ int caam_jr_enqueue(struct device *dev, u32 *desc,
head_entry->cbkarg = areq; head_entry->cbkarg = areq;
head_entry->desc_addr_dma = desc_dma; head_entry->desc_addr_dma = desc_dma;
jrp->inpring[jrp->inp_ring_write_index] = cpu_to_caam_dma(desc_dma); jrp->inpring[head] = cpu_to_caam_dma(desc_dma);
/* /*
* Guarantee that the descriptor's DMA address has been written to * Guarantee that the descriptor's DMA address has been written to
...@@ -367,8 +367,6 @@ int caam_jr_enqueue(struct device *dev, u32 *desc, ...@@ -367,8 +367,6 @@ int caam_jr_enqueue(struct device *dev, u32 *desc,
*/ */
smp_wmb(); smp_wmb();
jrp->inp_ring_write_index = (jrp->inp_ring_write_index + 1) &
(JOBR_DEPTH - 1);
jrp->head = (head + 1) & (JOBR_DEPTH - 1); jrp->head = (head + 1) & (JOBR_DEPTH - 1);
/* /*
...@@ -434,7 +432,6 @@ static int caam_jr_init(struct device *dev) ...@@ -434,7 +432,6 @@ static int caam_jr_init(struct device *dev)
jrp->entinfo[i].desc_addr_dma = !0; jrp->entinfo[i].desc_addr_dma = !0;
/* Setup rings */ /* Setup rings */
jrp->inp_ring_write_index = 0;
jrp->out_ring_read_index = 0; jrp->out_ring_read_index = 0;
jrp->head = 0; jrp->head = 0;
jrp->tail = 0; jrp->tail = 0;
...@@ -444,7 +441,6 @@ static int caam_jr_init(struct device *dev) ...@@ -444,7 +441,6 @@ static int caam_jr_init(struct device *dev)
wr_reg32(&jrp->rregs->inpring_size, JOBR_DEPTH); wr_reg32(&jrp->rregs->inpring_size, JOBR_DEPTH);
wr_reg32(&jrp->rregs->outring_size, JOBR_DEPTH); wr_reg32(&jrp->rregs->outring_size, JOBR_DEPTH);
jrp->ringsize = JOBR_DEPTH;
jrp->inpring_avail = JOBR_DEPTH; jrp->inpring_avail = JOBR_DEPTH;
spin_lock_init(&jrp->inplock); spin_lock_init(&jrp->inplock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment