On Tue, Jan 17, 2023 at 3:01 PM Bough Chen <haibo.chen@xxxxxxx> wrote: > > On Tue, Dec 20, 2022 at 10:02 AM <haibo.chen@xxxxxxx> wrote: > > > > > From: Haibo Chen <haibo.chen@xxxxxxx> > > > > > > To support Android GKI, need to build this driver as module. > > > So change the config as tristate type, and add module license. > > > > > > Signed-off-by: Jindong Yue <jindong.yue@xxxxxxx> > > > Signed-off-by: Haibo Chen <haibo.chen@xxxxxxx> > > > > OK... > > > > > builtin_platform_driver(vf610_gpio_driver); > > > > Then you want something like module_platform_driver() here. > > Hi Linus, > > Here, though need to build this gpio driver as module due to the Android GKI, but we do not really need to > rmmod/unbind this driver, so do not need to implement the _exit or remove callback. > In this case, still need to change to module_platform_driver()? > If so, after use the module_platform_driver(), is it okay to not implement the __exit /remove callback? Yes, our project is about maintaining and improving the Linux kernel, GKI is not a project we are running. If they need stuff from Linux, it happens on our terms, and that is that modularization should be self-motivated. Check this article: https://lwn.net/Articles/872209/ Yours, Linus Walleij