On Mon, Nov 21, 2022 at 2:35 PM Lionel DEBIEVE <lionel.debieve@xxxxxxxxxxx> wrote: > + writel_relaxed(readl_relaxed(cryp->regs + CRYP_CR) | CR_KEYRDEN, cryp->regs + CRYP_CR); > > Why are you still using CRYP_CR here, would it be better to use cryp->caps->cr to keep compatibility? I noticed that the first two registers were not moved in the memory map so I just kept CR and SR as hard defines. It could be argued that the code will be containing one less memory reference and addition so it is more efficient, but it's so little so I am fine to change it to an offset in the caps if you prefer it to be uniform. Yours, Linus Walleij
