linux-kernel - RE: [PATCH] caam: imx: fix the built-in caam driver cannot match soc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM6PR04MB60535A5DB643B8A5B07E1C55E2759@AM6PR04MB6053.eurprd04.prod.outlook.com>
Date:   Fri, 12 May 2023 10:57:22 +0000
From:   Alice Guo <alice.guo@....com>
To:     Pankaj Gupta <pankaj.gupta@....com>,
        Gaurav Jain <gaurav.jain@....com>,
        "herbert@...dor.apana.org.au" <herbert@...dor.apana.org.au>,
        "davem@...emloft.net" <davem@...emloft.net>
CC:     "linux-crypto@...r.kernel.org" <linux-crypto@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        dl-linux-imx <linux-imx@....com>,
        Aisheng Dong <aisheng.dong@....com>,
        Horia Geanta <horia.geanta@....com>,
        Peng Fan <peng.fan@....com>
Subject: RE: [PATCH] caam: imx: fix the built-in caam driver cannot match
 soc_id

Acked-by: Alice Guo <alice.guo@....com>

Best Regards,
Alice Guo

> -----Original Message-----
> From: Pankaj Gupta <pankaj.gupta@....com>
> Sent: Friday, May 12, 2023 11:11 PM
> To: Gaurav Jain <gaurav.jain@....com>; herbert@...dor.apana.org.au;
> davem@...emloft.net
> Cc: linux-crypto@...r.kernel.org; linux-kernel@...r.kernel.org; dl-linux-imx
> <linux-imx@....com>; Pankaj Gupta <pankaj.gupta@....com>; Alice Guo
> <alice.guo@....com>; Aisheng Dong <aisheng.dong@....com>; Horia Geanta
> <horia.geanta@....com>; Peng Fan <peng.fan@....com>
> Subject: [PATCH] caam: imx: fix the built-in caam driver cannot match soc_id
> 
> Since, CAAM driver is probed before soc_device_attribute done as part of:
> - drivers/soc/imx/soc-imx8m.c   (for i.MX8M)
> - drivers/firmware/imx/ele_mu.c (EdgeLock Enclave kernel driver, for
> i.MX8ULP)
> 
> It is needed to return -EPROBE_DEFER, after calling soc_device_match() in
> drivers/crypto/caam/ctrl.c.
> 
> soc_device_match returns NULL for:
> - i.MX8M
> - i.MX8ULP,
> can be considered that the SoC device has not been probed yet.
> Hence, it returns -EPROBE_DEFER directly.
> 
> caam: imx: change to use of_match_node in run_descriptor_deco0
> 
> Providing imx8m_machine_match to match:
> - i.MX8M{Q,M,N,P},
> - i.MX8ULP,
> so as to start using of_match_node, to simplify the code.
> 
> Signed-off-by: Alice Guo <alice.guo@....com>
> Signed-off-by: Pankaj Gupta <pankaj.gupta@....com>
> Reviewed-by: Dong Aisheng <aisheng.dong@....com>
> Reviewed-by: Horia Geanta <horia.geanta@....com>
> Acked-by: Peng Fan <peng.fan@....com>
> ---
>  drivers/crypto/caam/ctrl.c | 17 +++++++++++++----
>  1 file changed, 13 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/crypto/caam/ctrl.c b/drivers/crypto/caam/ctrl.c index
> bedcc2ab3a00..d8c528363f10 100644
> --- a/drivers/crypto/caam/ctrl.c
> +++ b/drivers/crypto/caam/ctrl.c
> @@ -79,6 +79,15 @@ static void build_deinstantiation_desc(u32 *desc, int
> handle)
>  	append_jump(desc, JUMP_CLASS_CLASS1 | JUMP_TYPE_HALT);  }
> 
> +static const struct of_device_id imx8m_machine_match[] = {
> +	{ .compatible = "fsl,imx8mm", },
> +	{ .compatible = "fsl,imx8mn", },
> +	{ .compatible = "fsl,imx8mp", },
> +	{ .compatible = "fsl,imx8mq", },
> +	{ .compatible = "fsl,imx8ulp", },
> +	{ }
> +};
> +
>  /*
>   * run_descriptor_deco0 - runs a descriptor on DECO0, under direct control
> of
>   *			  the software (no JR/QI used).
> @@ -105,10 +114,7 @@ static inline int run_descriptor_deco0(struct device
> *ctrldev, u32 *desc,
>  	     * Apparently on i.MX8M{Q,M,N,P} it doesn't matter if virt_en == 1
>  	     * and the following steps should be performed regardless
>  	     */
> -	    of_machine_is_compatible("fsl,imx8mq") ||
> -	    of_machine_is_compatible("fsl,imx8mm") ||
> -	    of_machine_is_compatible("fsl,imx8mn") ||
> -	    of_machine_is_compatible("fsl,imx8mp")) {
> +	    of_match_node(imx8m_machine_match, of_root)) {
>  		clrsetbits_32(&ctrl->deco_rsr, 0, DECORSR_JR0);
> 
>  		while (!(rd_reg32(&ctrl->deco_rsr) & DECORSR_VALID) && @@
> -646,6 +652,9 @@ static int caam_probe(struct platform_device *pdev)
>  	nprop = pdev->dev.of_node;
> 
>  	imx_soc_match = soc_device_match(caam_imx_soc_table);
> +	if (!imx_soc_match && of_match_node(imx8m_machine_match, of_root))
> +		return -EPROBE_DEFER;
> +
>  	caam_imx = (bool)imx_soc_match;
> 
>  	if (imx_soc_match) {
> --
> 2.34.1