<Acked-by> :- Ruchika Gupta <ruchika.gupta@xxxxxxxxxxxxx> Tested on P4080DS. Ported and tested on LS1 platform also (This platform has the virtualization enabled). Thanks, Ruchika > -----Original Message----- > From: Horia Geanta [mailto:horia.geanta@xxxxxxxxxxxxx] > Sent: Monday, July 21, 2014 6:33 PM > To: Herbert Xu; linux-crypto@xxxxxxxxxxxxxxx; Gupta Ruchika-R66431; Phillips > Kim-R1AAHA > Cc: David S. Miller > Subject: [PATCH] crypto: caam - fix DECO RSR polling > > RSR (Request Source Register) is not used when virtualization is disabled, > thus don't poll for Valid bit. > > Besides this, if used, timeout has to be reinitialized. > > Signed-off-by: Horia Geanta <horia.geanta@xxxxxxxxxxxxx> > --- > Only compile-tested. > Ruchika / Kim, please review / test. > > drivers/crypto/caam/ctrl.c | 11 +++++++---- > 1 file changed, 7 insertions(+), 4 deletions(-) > > diff --git a/drivers/crypto/caam/ctrl.c b/drivers/crypto/caam/ctrl.c index > c6e9d3b2d502..84d4b95c761e 100644 > --- a/drivers/crypto/caam/ctrl.c > +++ b/drivers/crypto/caam/ctrl.c > @@ -89,12 +89,15 @@ static inline int run_descriptor_deco0(struct device > *ctrldev, u32 *desc, > /* Set the bit to request direct access to DECO0 */ > topregs = (struct caam_full __iomem *)ctrlpriv->ctrl; > > - if (ctrlpriv->virt_en == 1) > + if (ctrlpriv->virt_en == 1) { > setbits32(&topregs->ctrl.deco_rsr, DECORSR_JR0); > > - while (!(rd_reg32(&topregs->ctrl.deco_rsr) & DECORSR_VALID) && > - --timeout) > - cpu_relax(); > + while (!(rd_reg32(&topregs->ctrl.deco_rsr) & DECORSR_VALID) && > + --timeout) > + cpu_relax(); > + > + timeout = 100000; > + } > > setbits32(&topregs->ctrl.deco_rq, DECORR_RQD0ENABLE); > > -- > 1.8.3.1 -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html