Re: [PATCH] crypto: use signed variable to store status and error checking

From: Giovanni Cabiddu
Date: Wed Jan 05 2022 - 07:14:01 EST


Thanks for your patch.

Patches to the qat driver have the following headline
crypto: qat - ...
not just
crypto: ...

On Wed, Jan 05, 2022 at 02:38:58AM +0500, Muhammad Usama Anjum wrote:
> ret should be signed. adf_cfg_get_param_value and match_string return
Use () when refer to the function().

> signed statuses. The return status may be saved wrongly in unsigned ret
> variable. Correct the data type of ret to signed int.
>
> Signed-off-by: Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx>
After fixing,
Acked-by: Giovanni Cabiddu <giovanni.cabiddu@xxxxxxxxx>

Regards,

--
Giovanni