Hi Herbert, If you have no comments then please apply the patch > -----Original Message----- > From: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx> > Sent: Wednesday, March 1, 2023 11:09 AM > To: Pankaj Gupta <pankaj.gupta@xxxxxxx>; Vijay Balakrishna > <vijayb@xxxxxxxxxxxxxxxxxxx>; Horia Geanta <horia.geanta@xxxxxxx>; Varun > Sethi <V.Sethi@xxxxxxx>; Gaurav Jain <gaurav.jain@xxxxxxx>; > herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; linux- > crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Franck Lenormand > <franck.lenormand@xxxxxxx> > Cc: code@xxxxxxxxxxx > Subject: RE: [EXT] Re: [PATCH] drivers: crypto: caam: jr: add .shutdown hook > > Copyright updated in another patch and sent v2, both patches are modifying > same file so no need to make any changes in this patch. > > > -----Original Message----- > > From: Pankaj Gupta <pankaj.gupta@xxxxxxx> > > Sent: Monday, February 27, 2023 1:03 PM > > To: Vijay Balakrishna <vijayb@xxxxxxxxxxxxxxxxxxx>; Meenakshi Aggarwal > > <meenakshi.aggarwal@xxxxxxx>; Horia Geanta <horia.geanta@xxxxxxx>; > > Varun Sethi <V.Sethi@xxxxxxx>; Gaurav Jain <gaurav.jain@xxxxxxx>; > > herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; linux- > > crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Franck Lenormand > > <franck.lenormand@xxxxxxx> > > Cc: code@xxxxxxxxxxx > > Subject: RE: [EXT] Re: [PATCH] drivers: crypto: caam: jr: add > > .shutdown hook > > > > Change the year in the License header. > > > > After changing, you can add the reviewed by me. > > Reviewed-By: Pankaj Gupta <pankaj.gupta@xxxxxxx> > > > > > -----Original Message----- > > > From: Vijay Balakrishna <vijayb@xxxxxxxxxxxxxxxxxxx> > > > Sent: Tuesday, February 21, 2023 11:02 PM > > > To: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx>; Horia Geanta > > > <horia.geanta@xxxxxxx>; Varun Sethi <V.Sethi@xxxxxxx>; Pankaj Gupta > > > <pankaj.gupta@xxxxxxx>; Gaurav Jain <gaurav.jain@xxxxxxx>; > > > herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx; linux- > > > crypto@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Franck > > > Lenormand <franck.lenormand@xxxxxxx> > > > Cc: code@xxxxxxxxxxx > > > Subject: [EXT] Re: [PATCH] drivers: crypto: caam: jr: add .shutdown > > > hook > > > > > > Caution: EXT Email > > > > > > On 2/20/2023 9:40 PM, meenakshi.aggarwal@xxxxxxx wrote: > > > > From: Gaurav Jain <gaurav.jain@xxxxxxx> > > > > > > > > add .shutdown hook in caam_jr driver to support kexec boot > > > > > > > > Signed-off-by: Gaurav Jain <gaurav.jain@xxxxxxx> > > > > > > Tested-by: Vijay Balakrishna <vijayb@xxxxxxxxxxxxxxxxxxx> > > > > > > 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)