[coreboot-gerrit] Change in coreboot[master]: amd/picasso/verstage: replace rsa accel with modexp

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45987 )

Change subject: amd/picasso/verstage: replace rsa accel with modexp
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45987/4/src/soc/amd/picasso/psp_ver... 
File src/soc/amd/picasso/psp_verstage/vboot_crypto.c:

https://review.coreboot.org/c/coreboot/+/45987/4/src/soc/amd/picasso/psp_ver... 
PS4, Line 132: 		sig_swapped[i] = swab32(inout_32[key->arrsize - i - 1]);
> AFAIK no.

Yes, do not read the hardware buffer from the PSP as an array of bytes when it shall be interpreted as an array of dwords. This might be cumbersome to implement, though.