RE: [PATCH 1/2] crypto: caam - Change structure type representing DECO MID

From: Gaurav Jain
Date: Mon Jul 17 2023 - 01:42:25 EST


Reviewed-by: Gaurav Jain <gaurav.jain@xxxxxxx>

> -----Original Message-----
> From: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx>
> Sent: Wednesday, July 12, 2023 11:37 AM
> To: Horia Geanta <horia.geanta@xxxxxxx>; Varun Sethi <V.Sethi@xxxxxxx>;
> Pankaj Gupta <pankaj.gupta@xxxxxxx>; Gaurav Jain <gaurav.jain@xxxxxxx>;
> herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; linux-
> crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Cc: Franck Lenormand <franck.lenormand@xxxxxxx>; Meenakshi Aggarwal
> <meenakshi.aggarwal@xxxxxxx>
> Subject: [PATCH 1/2] crypto: caam - Change structure type representing DECO
> MID
>
> From: Franck LENORMAND <franck.lenormand@xxxxxxx>
>
> The structure partid is not suitable to represent the DECO MID register.
>
> This patch replace partid by masterid which is more appropriate.
>
> Signed-off-by: Franck LENORMAND <franck.lenormand@xxxxxxx>
> Signed-off-by: Horia Geantă <horia.geanta@xxxxxxx>
> Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx>
> ---
> drivers/crypto/caam/regs.h | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/drivers/crypto/caam/regs.h b/drivers/crypto/caam/regs.h index
> 189e74c21f0c..0f87bd365582 100644
> --- a/drivers/crypto/caam/regs.h
> +++ b/drivers/crypto/caam/regs.h
> @@ -459,12 +459,6 @@ struct masterid {
> u32 liodn_ls; /* LIODN for non-sequence and seq access */
> };
>
> -/* Partition ID for DMA configuration */ -struct partid {
> - u32 rsvd1;
> - u32 pidr; /* partition ID, DECO */
> -};
> -
> /* RNGB test mode (replicated twice in some configurations) */
> /* Padded out to 0x100 */
> struct rngtst {
> @@ -590,7 +584,7 @@ struct caam_ctrl {
> u32 deco_rsr; /* DECORSR - Deco Request Source */
> u32 rsvd11;
> u32 deco_rq; /* DECORR - DECO Request */
> - struct partid deco_mid[5]; /* DECOxLIODNR - 1 per DECO */
> + struct masterid deco_mid[5]; /* DECOxLIODNR - 1 per DECO */
> u32 rsvd5[22];
>
> /* DECO Availability/Reset Section 120-3ff */
> --
> 2.25.1