Hi Vabhav,
> -----Original Message-----
> From: Vabhav Sharma <[email protected]>
> Sent: Monday, April 18, 2022 7:11 PM
> To: Fabio Estevam <[email protected]>; Varun Sethi <[email protected]>
> Cc: [email protected]; Horia Geanta <[email protected]>;
> Gaurav Jain <[email protected]>; [email protected]; Fabio
> Estevam <[email protected]>
> Subject: RE: [EXT] [PATCH v2] crypto: caam - fix i.MX6SX entropy delay value
>
>
>
> > -----Original Message-----
> > From: Fabio Estevam <[email protected]>
> > Sent: Sunday, April 17, 2022 12:41 AM
> > To: Varun Sethi <[email protected]>
> > Cc: [email protected]; Vabhav Sharma
> > <[email protected]>; Horia Geanta <[email protected]>; Gaurav
> > Jain <[email protected]>; [email protected]; Fabio
> > Estevam <[email protected]>
> > Subject: Re: [EXT] [PATCH v2] crypto: caam - fix i.MX6SX entropy delay
> > value
> >
> > Caution: EXT Email
> >
> > Hi Varun,
> >
> > On Sat, Apr 16, 2022 at 3:00 PM Varun Sethi <[email protected]> wrote:
> > >
> > > Hi Fabio,
> > > Vabhav is working on a fix for the Linux driver. He would be
> > > introducing a
> > new property in the CAAM device tree node, which would be used for
> > specifying the entropy delay value. This would make the solution generic.
> > This property is optional.
> >
> > Unfortunately, a devicetree property solution via optional property
> > would not work.
> >
> > Such a solution would not be backported to stable kernels and people
> > running old devicetree with new kernels would still face the problem.
> Please elaborate and specify the version for kernel, devicetree
> >
> > This problem is seen since kernel 5.10, so we need a kernel-only fix.
> Kernel 5.10 support devicetree, Do you mean customer using kernel without
> device tree?
Fabio's concern is about using new kernel image with an old device tree.
Fabio,
We feel that it would be better to have a provision to provide entropy delay parameter via device tree. This offers more flexibility.
For i.M6SX we can follow the approach proposed by you but at the same time we can still have the device tree provision.
Regards
Varun
On Mon, Apr 18, 2022 at 12:36 PM Varun Sethi <[email protected]> wrote:
> Fabio,
> We feel that it would be better to have a provision to provide entropy delay parameter via device tree. This offers more flexibility.
> For i.M6SX we can follow the approach proposed by you but at the same time we can still have the device tree provision.
Yes, agreed.