From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4672AC27C40 for ; Thu, 24 Aug 2023 18:24:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Cc:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Q3RHI8mWLStNjejZZAWGTXumTJXRncd2T4v83m0EC94=; b=BRxQYUrYF4CHw+ rc1Xp0u8ulPuokeGLf17/tiSo4T/2smXhxzIor8mVqLkHlUvFh33bUNsmRdAVdUGaQnHQeMUv1top r66Yem5OU62bjFd+pFFe3ACPwJLEDl3l8I5llrbVL5lcnE1YazpeZ1mfM94ZfL6pCsjhXq50TiMuF dTkidtnqGyp/fUi7O2eQyel74OQOExPPMjw8P9VkFCmfCFzj7AQg1mSsGh7LaFKr8egw3R0H5OYTS WYqi2Acr95K3iQqCJSIJGfuIZ24xbp7X5CYa5z7GLG+fitO/NqbLCupOBlhVulAL7r+AuL6qPe0+Z wz16KUO8GeuHCz+ZHyrA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qZF04-003aG9-08; Thu, 24 Aug 2023 18:24:08 +0000 Received: from mail-ed1-x532.google.com ([2a00:1450:4864:20::532]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qZF01-003aFU-1D for linux-arm-kernel@lists.infradead.org; Thu, 24 Aug 2023 18:24:07 +0000 Received: by mail-ed1-x532.google.com with SMTP id 4fb4d7f45d1cf-52a4737a08fso238435a12.3 for ; Thu, 24 Aug 2023 11:24:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1692901441; x=1693506241; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=973UgM2tzR/G8NExNMzGXq0EzRi09qe9/9zNN6E8lMI=; b=PgFZjIqOeTYZaIqnkdWJN0JpGsLz6pds74ERM6kipziYTraXLxac82r18VqDdgZTrR 4IyJSJ1+ZjHqBrOJKSXZb4gs7MCpn9JSQemvYdWeIJDBj93G5/sc+0vbZkriQQ+Q6qpU YgKB4AwaDuhJjtapZ4DkZLDGvmykBevbyMj+sCnjQuBCkN8/yUtPym5lJ2w5dV7QtIEC FXAXLPci/dGvQCAbTyPJ9iDw2y78tfq7+aagKsrN8XBGqjYD1SeeXdDXGXOr1ciza3Ku XDVK3EcLAgU0v5PiFUrbRwdMpWpZcPyqleXpa6oisrSdYdFuq4c039djjOys8IGvUcnO kcSg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692901441; x=1693506241; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=973UgM2tzR/G8NExNMzGXq0EzRi09qe9/9zNN6E8lMI=; b=gKDeMK5Vo221hWE7WX4jFFmXZ1oxnhvqj6+FtJqdPSxu5MSGY32nJYVxLpxA4+4ObU fMDMr/bJDdPX0a93wGV/QCvC6To6FCu+lkBKHV29KuyyWgr0DFkKUHAi9E9x1fl90xYT G+neV+Q5Uy0ft5BX20NY4Q/szcKTE7EfjfvStlihsowLkvF7bNo74d8KQLPc1jyHGjtw 7yDroeC4IoWyU2Q4LNq7AKHLgZGu9DSXQ21jLvEGykUwA1ID9nUFVVYRoejs+eshH676 Cj5LaCDu8K2zBlzZM8JJF2SQodv0opY+zRHYbmF6v5YMelua1+x0TAjFSXL9JaBClDij utjA== X-Gm-Message-State: AOJu0YzX8SyH7eaAa/u7vKpSv5CXA/bRoKhh/UddTCD3LvfkKQoyAyrm 2aCBw72HL7C8/63NcrxKnWAHgg== X-Google-Smtp-Source: AGHT+IHysWAOwXtctQ35hBuhVVjnyCtdz9zXSYsBKT7gP4iLcIu+DWaCtsznh/M+WApmQvx33ZugEA== X-Received: by 2002:aa7:ccc9:0:b0:525:7cd4:34da with SMTP id y9-20020aa7ccc9000000b005257cd434damr11971151edt.40.1692901441126; Thu, 24 Aug 2023 11:24:01 -0700 (PDT) Received: from [192.168.0.22] ([77.252.47.198]) by smtp.gmail.com with ESMTPSA id u8-20020aa7d0c8000000b00525503fac84sm36880edo.25.2023.08.24.11.23.59 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 24 Aug 2023 11:24:00 -0700 (PDT) Message-ID: Date: Thu, 24 Aug 2023 20:23:58 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH v5 08/11] firmware: imx: enable trng Content-Language: en-US To: Pankaj Gupta , shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, clin@suse.com, conor+dt@kernel.org, pierre.gondois@arm.com, ping.bai@nxp.com, xiaoning.wang@nxp.com, wei.fang@nxp.com, peng.fan@nxp.com, haibo.chen@nxp.com, festevam@gmail.com, linux-imx@nxp.com, davem@davemloft.net, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, gaurav.jain@nxp.com, alexander.stein@ew.tq-group.com, sahil.malhotra@nxp.com, aisheng.dong@nxp.com, V.Sethi@nxp.com References: <20230823073330.1712721-1-pankaj.gupta@nxp.com> <20230823073330.1712721-9-pankaj.gupta@nxp.com> From: Krzysztof Kozlowski In-Reply-To: <20230823073330.1712721-9-pankaj.gupta@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230824_112405_487100_8FDBF356 X-CRM114-Status: GOOD ( 23.33 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 23/08/2023 09:33, Pankaj Gupta wrote: > Enabled trng on imx93 platform through enclave fw. > > Signed-off-by: Gaurav Jain > Signed-off-by: Pankaj Gupta > --- > drivers/firmware/imx/Kconfig | 9 +++ > drivers/firmware/imx/Makefile | 3 +- > drivers/firmware/imx/ele_base_msg.c | 88 +++++++++++++++++++++++ > drivers/firmware/imx/ele_common.c | 44 +++++++++++- > drivers/firmware/imx/ele_common.h | 2 + > drivers/firmware/imx/ele_fw_api.c | 54 ++++++++++++++ > drivers/firmware/imx/ele_trng.c | 48 +++++++++++++ > drivers/firmware/imx/se_fw.c | 23 +++++- > include/linux/firmware/imx/ele_base_msg.h | 11 +++ > include/linux/firmware/imx/ele_fw_api.h | 5 ++ > 10 files changed, 283 insertions(+), 4 deletions(-) > create mode 100644 drivers/firmware/imx/ele_trng.c > firmware is not a place for RNG driver. You did not even CC RNG folks... > + > +int ele_trng_init(struct device *dev) > +{ > + struct ele_trng *trng; > + int ret; > + > + trng = devm_kzalloc(dev, sizeof(*trng), GFP_KERNEL); > + if (!trng) > + return -ENOMEM; > + > + trng->dev = dev; > + trng->rng.name = "ele-trng"; > + trng->rng.read = ele_get_hwrng; > + trng->rng.priv = (unsigned long)trng; > + trng->rng.quality = 1024; > + > + dev_dbg(dev, "registering ele-trng\n"); > + > + ret = devm_hwrng_register(dev, &trng->rng); > + if (ret) > + return ret; > + > + dev_info(dev, "Successfully registered ele-trng\n"); Drop all simple trace replacements. Use tracing. > + return 0; > +} > + > +int ele_get_hwrng(struct hwrng *rng, > + void *data, size_t len, bool wait) > +{ > + struct ele_trng *trng = (struct ele_trng *)rng->priv; > + > + return ele_get_random(trng->dev, data, len); > +} > diff --git a/drivers/firmware/imx/se_fw.c b/drivers/firmware/imx/se_fw.c > index 88300c41d62b..6083c42dcda3 100644 > --- a/drivers/firmware/imx/se_fw.c > +++ b/drivers/firmware/imx/se_fw.c > @@ -21,10 +21,10 @@ > #include > #include > #include > -#include > #include > > #include "se_fw.h" > +#include "ele_common.h" > > #define SOC_ID_OF_IMX8ULP 0x084D > #define SOC_ID_OF_IMX93 0x9300 > @@ -43,6 +43,9 @@ struct imx_info { > uint8_t *pool_name; > bool reserved_dma_ranges; > bool init_fw; > + /* platform specific flag to enable/disable the ELE True RNG */ > + bool start_rng; > + bool enable_ele_trng; > }; > > static LIST_HEAD(priv_data_list); > @@ -58,6 +61,8 @@ static const struct imx_info imx8ulp_info = { > .pool_name = "sram-pool", > .reserved_dma_ranges = true, > .init_fw = false, > + .start_rng = true, > + .enable_ele_trng = false, > }; > > static const struct imx_info imx93_info = { > @@ -71,6 +76,8 @@ static const struct imx_info imx93_info = { > .pool_name = NULL, > .reserved_dma_ranges = true, > .init_fw = true, > + .start_rng = true, > + .enable_ele_trng = true, > }; > > static const struct of_device_id se_fw_match[] = { > @@ -1140,9 +1147,23 @@ static int se_fw_probe(struct platform_device *pdev) > } > } > > + /* start ele rng */ > + if (info->start_rng) { > + ret = ele_do_start_rng(dev); > + if (ret) > + dev_err(dev, "Failed to start ele rng\n"); > + } > + > + if (!ret && info->enable_ele_trng) { > + ret = ele_trng_init(dev); > + if (ret) > + dev_err(dev, "Failed to init ele-trng\n"); > + } > + > pr_info("i.MX secure-enclave: %s's mu#%d interface to firmware, configured.\n", dev_info > info->se_name, > priv->ele_mu_id); > + Not related. > return devm_of_platform_populate(dev); > Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel