[PATCH V2 2/4] firmware: xilinx: Add ZynqMP SHA API for SHA3 functionality

[Harsha Harsha]

> -----Original Message-----
> From: Corentin Labbe <clabbe.montjoie at gmail.com>
> Sent: Friday, February 18, 2022 3:13 PM
> To: Harsha Harsha <harshah at xilinx.com>
> Cc: herbert at gondor.apana.org.au; davem at davemloft.net; linux-crypto at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; Michal Simek <michals at xilinx.com>; Sarat Chand Savitala <saratcha at xilinx.com>; Harsh Jain
> <harshj at xilinx.com>; git <git at xilinx.com>
> Subject: Re: [PATCH V2 2/4] firmware: xilinx: Add ZynqMP SHA API for SHA3 functionality
> 
> Le Fri, Feb 18, 2022 at 12:44:22AM +0530, Harsha a écrit :
> > This patch adds zynqmp_pm_sha_hash API in the ZynqMP firmware to compute
> > SHA3 hash of given data.
> >
> > Signed-off-by: Harsha <harsha.harsha at xilinx.com>
> > Acked-by: Michal Simek <michal.simek at xilinx.com>
> > ---
> 
> Hello
> 
> Your signed-off should contain your real name.

My complete name is Harsha which I have mentioned in the signed-off section.

> Furthermore why did you drop copyright from previous poster ?
> See https://patchwork.kernel.org/project/linux-crypto/cover/1556793282-17346-1-git-send-email-kalyani.akula@xilinx.com/ for
> reference.

I did not understand the comment. Do you want me to add Kalyani's name also in the Signed of section?

> 
> Furthermore, the previous poster didnt answered my questions about parallel processing and tests.

Since SHA3 HW engine in ZynqMPSoC does not support parallel processing of 2 hash requests, so we have changed our approach. 
Now  to support parallel processing of 2 hash requests, software fallback is being used for init, update, final, export and import in the ZynqMP SHA driver.
For digest, the calculation of SHA3 hash is done by the hardened SHA3 accelerator in Xilinx ZynqMP SoC.

Following tests have been done for the driver:
- Enabled kernel self tests and extra run-time crypto self tests
- Tested SHA hash computation for different sizes of data using userspace application
- Tested SHA hash computation using multiple updates of data using userspace application
- Tested parallel hash computation
- Tested using the tcrypt module

> 
> Regards

Regards,
Harsha