Message-ID: <d253d5ba-8e57-dc20-dfb4-08aac9e92a4c@linux.microsoft.com>
Date:   Tue, 21 Feb 2023 09:32:13 -0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.1
Subject: Re: [PATCH] drivers: crypto: caam: jr: add .shutdown hook
Content-Language: en-US
To:     meenakshi.aggarwal@nxp.com, horia.geanta@nxp.com, V.sethi@nxp.com,
        pankaj.gupta@nxp.com, gaurav.jain@nxp.com,
        herbert@gondor.apana.org.au, davem@davemloft.net,
        linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org,
        franck.lenormand@nxp.com
Cc:     code@tyhicks.com
References: <20230220105033.1449263-1-meenakshi.aggarwal@nxp.com>
 <20230221054047.2140558-1-meenakshi.aggarwal@nxp.com>
From:   Vijay Balakrishna <vijayb@linux.microsoft.com>
In-Reply-To: <20230221054047.2140558-1-meenakshi.aggarwal@nxp.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 2/20/2023 9:40 PM, meenakshi.aggarwal@nxp.com wrote:
> From: Gaurav Jain <gaurav.jain@nxp.com>
> 
> add .shutdown hook in caam_jr driver to support kexec boot
> 
> Signed-off-by: Gaurav Jain <gaurav.jain@nxp.com>

Tested-by: Vijay Balakrishna <vijayb@linux.microsoft.com>

Thanks,
Vijay

> ---
>   drivers/crypto/caam/jr.c | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/crypto/caam/jr.c b/drivers/crypto/caam/jr.c
> index 8745fe3cb575..a2a99d09b4ad 100644
> --- a/drivers/crypto/caam/jr.c
> +++ b/drivers/crypto/caam/jr.c
> @@ -198,6 +198,11 @@ static int caam_jr_remove(struct platform_device *pdev)
>   	return ret;
>   }
>   
> +static void caam_jr_platform_shutdown(struct platform_device *pdev)
> +{
> +	caam_jr_remove(pdev);
> +}
> +
>   /* Main per-ring interrupt handler */
>   static irqreturn_t caam_jr_interrupt(int irq, void *st_dev)
>   {
> @@ -653,6 +658,7 @@ static struct platform_driver caam_jr_driver = {
>   	},
>   	.probe       = caam_jr_probe,
>   	.remove      = caam_jr_remove,
> +	.shutdown = caam_jr_platform_shutdown,
>   };
>   
>   static int __init jr_driver_init(void)