RE: [PATCH V4 4/9] pinctrl: imx8mn: Support building as module

From: Anson Huang
Date: Thu Jun 11 2020 - 04:48:47 EST


Hi, Daniel

> Subject: Re: [PATCH V4 4/9] pinctrl: imx8mn: Support building as module
>
> Maybe this is obvious but I would really like to see an explanation
>
> of why we are switching from arch_initcall to platform_init.
>
> Commit message act as documentation for the reviewers.

Yes, I noticed this, and it looks like unnecessary change, and I just replied in
mail that I will change it back to arch_initcall() in order to make sure nothing
changed for built-in config. Below is what I replied in cover-letter mail:

" I will keep the arch_initcall() there in next version patch series, it can make sure
the change does NOT impact built-in config. For module build, the arch_initcall()
will be same as module_init(), user needs to insmod the .ko with correct sequence."

Thanks,
Anson