On Thu, Apr 30, 2020 at 11:59:53AM +0000, Satya Tangirala wrote:
> Add the crypto registers and structs defined in v2.1 of the JEDEC UFSHCI
> specification in preparation to add support for inline encryption to
> UFS.
> 
> Signed-off-by: Satya Tangirala <[email protected]>

Looks good, you can add:

    Reviewed-by: Eric Biggers <[email protected]>

One nit:

> diff --git a/drivers/scsi/ufs/ufshcd.h b/drivers/scsi/ufs/ufshcd.h
> index 6ffc08ad85f63..1eebb589159d6 100644
> --- a/drivers/scsi/ufs/ufshcd.h
> +++ b/drivers/scsi/ufs/ufshcd.h
> @@ -555,6 +555,12 @@ enum ufshcd_caps {
>        * for userspace to control the power management.
>        */
>       UFSHCD_CAP_RPM_AUTOSUSPEND                      = 1 << 6,
> +
> +     /*
> +      * This capability allows the host controller driver to use the
> +      * inline crypto engine, if it is present
> +      */
> +     UFSHCD_CAP_CRYPTO                               = (1 << 7),
>  };

The other values of this enum don't use parentheses.

- Eric


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to