Received-SPF: pass (google.com: domain of linux-crypto+bounces-3917-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Pankaj Gupta <pankaj.gupta@nxp.com>
To: gaurav.jain@nxp.com,
	horia.geanta@nxp.com,
	V.Sethi@nxp.com,
	herbert@gondor.apana.org.au,
	davem@davemloft.net,
	iuliana.prodan@nxp.com,
	linux-crypto@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	linux-imx@nxp.com
Cc: Pankaj Gupta <pankaj.gupta@nxp.com>
Subject: [PATCH v4 1/2] caam: init-clk based on caam-page0-access
Date: Mon, 29 Apr 2024 11:58:52 +0530
Message-Id: <20240429062855.923595-3-pankaj.gupta@nxp.com>
In-Reply-To: <20240429062855.923595-1-pankaj.gupta@nxp.com>
References: <20240429062855.923595-1-pankaj.gupta@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?E5Z4jiNGsn/t1Kfnd5t7HrI+qY+m80iT8PmG+76M2DPDzPxlCItcP4yeIKdS?=
 =?us-ascii?Q?H1kUD0NL36B1wMerrCmlBx3r7DrDUja6l9PW0jc5+FjpE2NjoNr1P6zzyREB?=
 =?us-ascii?Q?L0rNQdmHCp8m4g+jX9+qNVvVsvsvpofV1g6y4M1EMIDgADjqi/bvn89aRuwQ?=
 =?us-ascii?Q?TGm3O4FiMYezlIkn9IGARaWQxoljKNq3xpfiGeqz7InXrX+i2KedXsjbkL4p?=
 =?us-ascii?Q?ftMlq3ySNBKHe2fmyxb8NToRikuVXL667J/j3bKA5fw/NsxhzzkMHGGTC23u?=
 =?us-ascii?Q?EVJ8EzvQi26LBoewADrj7UuQx9GiUrSdfMq4CLXF/KfiyCW/l2m0CcYqa98n?=
 =?us-ascii?Q?93mIAKXwozkVrTm3zJeRAs0WxmBDCTzqNy3u4LHbGCXgsMcpMDs1UMjcaDMF?=
 =?us-ascii?Q?1F4Rqu1BrR3pkBGhwiOHYgAbSVK9oFDPtQRKVlGgoYUMRTf0vP3FQMfLvk70?=
 =?us-ascii?Q?QLlTKKWU7o6dLkyzpedpuATU+3Fjc6J19jtmA1SGyUEsbq2U2GHAXTQxEvqg?=
 =?us-ascii?Q?5j7NrEHvieaWb9SAMhIhPqF4CJRWvgTrIC4wcqupCegsVDTqibHWbpVVdFJ6?=
 =?us-ascii?Q?X0dPyhht6wpzCbgQ0cOIig8eiK3Xf0odpKBvjYMa0SBabdG+1kUnss8hzJhx?=
 =?us-ascii?Q?MC5H26npRjBWQVlUVI4iuX5n51OJkP7ieUpTPeMLmZm2LOcho2Yd+J11ByRJ?=
 =?us-ascii?Q?ZfHiajhH/x03ERv70NlL7Lubvg1iEDevLwub1Al6oFYKNBbE0+xDBOnPbjXe?=
 =?us-ascii?Q?xee1QZ7Bc3JPdP3BknNlu8xZJGmLwrK9Yb5bltTInNEBqXgixhFVu/NGGI2i?=
 =?us-ascii?Q?ophB/UUxgNL4xBp5WQliuZmRKJ/b4OTxYNcW8kb0X+G7UC/2LuXIouN7EyTp?=
 =?us-ascii?Q?DKNN0h9s5WrTlmp1ZilCB/MrqvTNioTOQo36MjgHvhnJmdWZ3l1VX7ZncEnz?=
 =?us-ascii?Q?GuVJfdAvGfcy2ACMlvfVfxs1wvDTFdxH3VRWX0HCjbI7JSKNYJbcyJ7UfYbi?=
 =?us-ascii?Q?jU+mPsoQi33Y+QcjG42vhN8LwRasLhqlteMy9vhmcQBYG596tNYkZrO+EH2v?=
 =?us-ascii?Q?/3pf5P0/qK/CUs4AhLI6/QR3al0QE+9Eqw3sWOx83C1QpF08b94aOK/55L40?=
 =?us-ascii?Q?PswyiJVUddTgeMSXyoLEYomv7wIXlWS0ufgrjrFctwoRc+MbhPnQovM8NsLa?=
 =?us-ascii?Q?6pWDqPFpEoZlxVqOYV5EvFsrDhDpzhSj9dH4QFDrklbSC2BefrWwR5shF0Bc?=
 =?us-ascii?Q?Nz0Y3bJkQJwgp9e0Ep7ALCEMZo8UkbX7Acss3/R2Q6GeIIX7cSuHH1KLaFou?=
 =?us-ascii?Q?OsiVG1q17UpVEcppfGBfNovxOAN9/IXnyYJ3ivkIoKnsJjW1YxoxZTlamzRw?=
 =?us-ascii?Q?uwkDjnPqglHtoGJfusM4wqEmjrm+YxWuijGa9XX11275ClVQgs3Wft/uCI8H?=
 =?us-ascii?Q?lIwXRE7A6pGKdnZb6M9PaSNAVguJ7260iY3pSXdzkuBt2atXlUwqC03Doijo?=
 =?us-ascii?Q?PvIF7dTVqT5W5YyQ1UeGL0W10ax9UTOkgsMt1bSyuepxwbGI0PpKRHyYNs+1?=
 =?us-ascii?Q?MSLWjFia6eNjNmJ/rbawtReOpgleX21HQSPDV/Aj?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 26355f26-3b23-4f37-e80b-08dc681603e2
X-MS-Exchange-CrossTenant-AuthSource: AM9PR04MB8604.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Apr 2024 06:31:34.8272
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ttUqLv5opveEYOl6dAJIQKnIt8Zj0lMDGZ/8Y8ONq8cTra2rNKe4yDrr3cLXAHbk8tzopuD5cB0YeOdhivgxAQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DUZPR04MB9919

CAAM clock initializat is done based on the basis of soc specific
info stored in struct caam_imx_data:
- caam-page0-access flag
- num_clks

CAAM driver needs to be aware of access rights to CAAM control page
i.e., page0, to do things differently.

Signed-off-by: Pankaj Gupta <pankaj.gupta@nxp.com>
Reviewed-by: Gaurav Jain <gaurav.jain@nxp.com>
---
 drivers/crypto/caam/ctrl.c | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/drivers/crypto/caam/ctrl.c b/drivers/crypto/caam/ctrl.c
index bdf367f3f679..02363c467cf3 100644
--- a/drivers/crypto/caam/ctrl.c
+++ b/drivers/crypto/caam/ctrl.c
@@ -512,6 +512,7 @@ static const struct of_device_id caam_match[] = {
 MODULE_DEVICE_TABLE(of, caam_match);
 
 struct caam_imx_data {
+	bool page0_access;
 	const struct clk_bulk_data *clks;
 	int num_clks;
 };
@@ -524,6 +525,7 @@ static const struct clk_bulk_data caam_imx6_clks[] = {
 };
 
 static const struct caam_imx_data caam_imx6_data = {
+	.page0_access = true,
 	.clks = caam_imx6_clks,
 	.num_clks = ARRAY_SIZE(caam_imx6_clks),
 };
@@ -534,6 +536,7 @@ static const struct clk_bulk_data caam_imx7_clks[] = {
 };
 
 static const struct caam_imx_data caam_imx7_data = {
+	.page0_access = true,
 	.clks = caam_imx7_clks,
 	.num_clks = ARRAY_SIZE(caam_imx7_clks),
 };
@@ -545,6 +548,7 @@ static const struct clk_bulk_data caam_imx6ul_clks[] = {
 };
 
 static const struct caam_imx_data caam_imx6ul_data = {
+	.page0_access = true,
 	.clks = caam_imx6ul_clks,
 	.num_clks = ARRAY_SIZE(caam_imx6ul_clks),
 };
@@ -554,6 +558,7 @@ static const struct clk_bulk_data caam_vf610_clks[] = {
 };
 
 static const struct caam_imx_data caam_vf610_data = {
+	.page0_access = true,
 	.clks = caam_vf610_clks,
 	.num_clks = ARRAY_SIZE(caam_vf610_clks),
 };
@@ -860,6 +865,7 @@ static int caam_probe(struct platform_device *pdev)
 	int pg_size;
 	int BLOCK_OFFSET = 0;
 	bool reg_access = true;
+	const struct caam_imx_data *imx_soc_data;
 
 	ctrlpriv = devm_kzalloc(&pdev->dev, sizeof(*ctrlpriv), GFP_KERNEL);
 	if (!ctrlpriv)
@@ -894,12 +900,20 @@ static int caam_probe(struct platform_device *pdev)
 			return -EINVAL;
 		}
 
+		imx_soc_data = imx_soc_match->data;
+		reg_access = reg_access && imx_soc_data->page0_access;
+		/*
+		 * CAAM clocks cannot be controlled from kernel.
+		 */
+		if (!imx_soc_data->num_clks)
+			goto iomap_ctrl;
+
 		ret = init_clocks(dev, imx_soc_match->data);
 		if (ret)
 			return ret;
 	}
 
-
+iomap_ctrl:
 	/* Get configuration properties from device tree */
 	/* First, get register page */
 	ctrl = devm_of_iomap(dev, nprop, 0, NULL);
-- 
2.34.1