Commit bbfcac5f authored by Vakul Garg's avatar Vakul Garg Committed by Herbert Xu

crypto: caam/jr - Remove extra memory barrier during job ring dequeue

In function caam_jr_dequeue(), a full memory barrier is used before
writing response job ring's register to signal removal of the completed
job. Therefore for writing the register, we do not need another write
memory barrier. Hence it is removed by replacing the call to wr_reg32()
with a newly defined function wr_reg32_relaxed().
Signed-off-by: default avatarVakul Garg <vakul.garg@nxp.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent f5a2aeb8
...@@ -213,7 +213,7 @@ static void caam_jr_dequeue(unsigned long devarg) ...@@ -213,7 +213,7 @@ static void caam_jr_dequeue(unsigned long devarg)
mb(); mb();
/* set done */ /* set done */
wr_reg32(&jrp->rregs->outring_rmvd, 1); wr_reg32_relaxed(&jrp->rregs->outring_rmvd, 1);
jrp->out_ring_read_index = (jrp->out_ring_read_index + 1) & jrp->out_ring_read_index = (jrp->out_ring_read_index + 1) &
(JOBR_DEPTH - 1); (JOBR_DEPTH - 1);
......
...@@ -96,6 +96,14 @@ cpu_to_caam(16) ...@@ -96,6 +96,14 @@ cpu_to_caam(16)
cpu_to_caam(32) cpu_to_caam(32)
cpu_to_caam(64) cpu_to_caam(64)
static inline void wr_reg32_relaxed(void __iomem *reg, u32 data)
{
if (caam_little_end)
writel_relaxed(data, reg);
else
writel_relaxed(cpu_to_be32(data), reg);
}
static inline void wr_reg32(void __iomem *reg, u32 data) static inline void wr_reg32(void __iomem *reg, u32 data)
{ {
if (caam_little_end) if (caam_little_end)
......
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