Messages in this thread | | | From | Stephan Mueller <> | Subject | Re: [PATCH 3/3] hwrng: msm - Add support for prng v2 | Date | Wed, 27 Jun 2018 09:51:48 +0200 |
| |
Am Mittwoch, 27. Juni 2018, 09:01:44 CEST schrieb Vinod:
Hi Vinod,
> > > Currently am doing bunch of initialization in .probe (platform driver) > > > and I think recommendation would be to move that to .cra_init, which > > > seem > > > plausible but I don't have pdev to read hw_resource etc.. so would still > > > need to get that. > > > > It seems that your allocation during probe relates to the hardware > > resource > > where you only have one in the system. Thus, doing the allocation here > > makes sense. And, you do not want to perform probe or such resource > > allocation once per crypto API RNG instance allocation. As said, there > > can be multiple or even they can be allocated and deallocated frequently. > > This in particular applies if your driver's "stdrng" has the highest prio > > which means that it will be allocated and deallocated frequently. > > Right, that is how I visualized it. > > Is there a way where we can tweak the register API to pass hw_resource > pointer and get that back? Would that work with the security model in > crypto.
I would not see an easy way for that. The core register API of the kernel crypto API would need to be changed. > > I do not like globals and somehow don't feel that we should do it that > way
Granted, I concur here.
Ciao Stephan
| |