Hi Bryan, Em ter, 23 de abr de 2019 às 07:20, Bryan O'Donoghue <bryan.odonog...@linaro.org> escreveu: > > Use __sec_set_jr_context_normal() to set job-ring ownership rather than the > current in-line array walk. > > Signed-off-by: Bryan O'Donoghue <bryan.odonog...@linaro.org> > --- > drivers/crypto/fsl/jr.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/drivers/crypto/fsl/jr.c b/drivers/crypto/fsl/jr.c > index 7b13aa4a61..65982b8369 100644 > --- a/drivers/crypto/fsl/jr.c > +++ b/drivers/crypto/fsl/jr.c > @@ -616,7 +616,6 @@ int sec_init_idx(uint8_t sec_idx) > { > ccsr_sec_t *sec = (void *)SEC_ADDR(sec_idx); > uint32_t mcr = sec_in32(&sec->mcfgr); > - uint32_t jrown_ns; > int i;
We may also need to remove this variable otherwise we get build warning below: drivers/crypto/fsl/jr.c: In function 'sec_init_idx': drivers/crypto/fsl/jr.c:625:6: warning: unused variable 'i' [-Wunused-variable] int i; ^ Thanks for submitting this patch set. I couldn't get encrypted boot working in my first attempt, doing the exact same procedure with commit 22191ac35344 ("drivers/crypto/fsl: assign job-rings to non-TrustZone") reverted works fine. I will take a better look in your patch set and let you know if I find something. Best Regards, Breno Matheus Lima _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot