Re: [PATCH] nvmem: add QTI SDAM driver

[sthella]

On 2019-12-20 05:59, Bjorn Andersson wrote:
> On Thu 19 Dec 02:29 PST 2019, sthella@xxxxxxxxxxxxxx wrote:
>
> > On 2019-12-18 11:44, Bjorn Andersson wrote:
> > > On Tue 17 Dec 01:20 PST 2019, Shyam Kumar Thella wrote:
> [..]
> > > > +subsys_initcall(sdam_init);
> > >
> > > module_platform_driver(sdam_driver), unless you have some strong
> > > arguments for why this needs to be subsys_initcall
> > There are some critical sybsystems which depend on nvmem data. So I 
> > would
> > prefer using subsys_initcall().
>
> How critical? Needed to kernel module loading?
>
> Can you please document this need somehow? (either a comment here or
> something in the commit message). Be specific.
Sure. I will add the need in commit message in its next patch.
> THanks,
> Bjorn