Received-SPF: pass (google.com: domain of linux-crypto+bounces-3918-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
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:53 +0530
Message-Id: <20240429062855.923595-4-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?4bOvnVGuT/zkjWQvJKLzt30kN/bGjr+ZTpXUnHgcY30DtzmfTLxqM1RJRMrM?=
 =?us-ascii?Q?fA5vk5WPbVOtaJjIvlH9diVVrhHbLQQNi7FLMhWh1zVHvutyovB6/JXeKhoB?=
 =?us-ascii?Q?XF+kSrSQCUhrLqe5tymounU/4TqzpLxqapUg4FT9GXNf0RMIN+cLvQuOCI/s?=
 =?us-ascii?Q?lg3Dex8Lj4sXoKLFv9MAjjSlnWbYiu9uPM7HexCs3K4ESnOvOdQEGTMe2Bb8?=
 =?us-ascii?Q?VsCpY2dfHtY/GTXjJ3iVhUez9EB8h5cMpMGVEAT5FJ71FZdG3dI/z5md3yGr?=
 =?us-ascii?Q?xBqd1nYqpChNDREt7HCYoloLTYOqsZxIvasURsP6WtQefz72V0cgtDFZBqbz?=
 =?us-ascii?Q?oeHDw5EpxoEQjjUgRrXOlEU3YwYTnkd8Mb0kNdp4YuYVreCBQ3iMf7MWALYw?=
 =?us-ascii?Q?/axm9cvY29vpEd2AglXk2Q5fzlO6eIiVp8inIdHtx/r7aFQ88WVJmfJx0S+q?=
 =?us-ascii?Q?8+B3kShnFGbY88gmm4SL93HNwBQnVzjIYJ6rIs6hFqBb0KzpP0NU4H5UOHF+?=
 =?us-ascii?Q?LX4QgMVuFUqTR90/VBmaGSz9m4135IzzuHw+tZIxOnY+j4Wt9Uj6VYC7lmVS?=
 =?us-ascii?Q?n4zUzAnfxBUqvs6ZWh/rqg0hXaVS8KejPiwtpjzzbQaq4zQ4o1IRBkqczGYA?=
 =?us-ascii?Q?KspARScBB0DOZ94a9MqkR0sRxX+ZzULk13K8906RAjfb7sVUKvqLprQ0BeGY?=
 =?us-ascii?Q?PMO9I2HebxWYEmuNPJJAhVtPr85sP/pOUHRzEtmuOY/Shl30Dvkn7qeJ1Jq1?=
 =?us-ascii?Q?fynm+oQMy2W3jfFsG57EQWChidD6KDC+PVm9i8P8LITTrjhAFK8LdVoop3JB?=
 =?us-ascii?Q?z3tmUgEpOAxHFKfsb5oy3AglemB5ZIc+4V+unj4k9G7RlDV5qDOoMGMrVnoU?=
 =?us-ascii?Q?HVDZMogef4UqIToWNw6BkRbNwR6vrBRBvwed+9zyo+eyDE1d+ZTSOw4wTrjf?=
 =?us-ascii?Q?s5VGlyaLnHCg0740BOI83qxQUmXWFRGB6W0dRdCNr2y6gfzT5+9o5tTUbv02?=
 =?us-ascii?Q?rICdaArvRgJ6EDUAvW7+VqziVGYAVTaJbB2n04vqBAnLTSlPSKLDVL7JvlJm?=
 =?us-ascii?Q?8lGQE6diBBuYhkvjLfUX/RktTTUNv/E7o3Cr+RcbaIsJwfqGV/TiaM5REoDO?=
 =?us-ascii?Q?IHj/356VshqkajvLZ0uBU6tY1kpCAOBnG38BEr3KEGRFAS6wEFksZGQCgT42?=
 =?us-ascii?Q?cGnjTZUi1V20xYeNOWbqyvXBzg0KKtvrHmHoY3WxbyXjwBanllpWEbW0NOUi?=
 =?us-ascii?Q?LROCjBohXlbuu/Za7qGkHgSxbRocvx1xMs/YIkp0NNta5PABLFTCcSzbemhz?=
 =?us-ascii?Q?crrrLMGCwDstPNidSzWpjPw9mIdCN0865aCG7hhdAdxELCjTkEN7cTgOwUrT?=
 =?us-ascii?Q?b3G7Wwwr15pwIEmyelGELj24V5Q8dHkDqASRFOZSYFDV2xV64tmHUjtdhK6K?=
 =?us-ascii?Q?V7gCyf8Cl4A9Ig1KZ4ieHXabfcmzSStzXBHB9Gcpp6TALznen9M2EAClAQW7?=
 =?us-ascii?Q?8JEFVi3Borm3SyeFTw/bB1xeb4OoolydOOzCJBkz0UG5esZHLf0WD4ItD8KP?=
 =?us-ascii?Q?9s6hr7SOw4epjOaBBWjVGaTZdwUIE9YW1cvfDO97?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6f104762-348d-4fd7-a0b7-08dc6816066b
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:38.7899
 (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: DeKAnxSzJpxdIeVvGDBK/oBtGCjLL8IVtmnUF4do847ctyY20Dy4T1aiPnH5ueCWbK2xAA0PqzR58aFET1Xkmw==
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