> +#define fpga_mgr_register_full(parent, info) \ > + __fpga_mgr_register_full(parent, info, THIS_MODULE) > struct fpga_manager * > -fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info); > +__fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info, > + struct module *owner); > > +#define fpga_mgr_register(parent, name, mops, priv) \ > + __fpga_mgr_register(parent, name, mops, priv, THIS_MODULE) > struct fpga_manager * > -fpga_mgr_register(struct device *parent, const char *name, > - const struct fpga_manager_ops *mops, void *priv); > +__fpga_mgr_register(struct device *parent, const char *name, > + const struct fpga_manager_ops *mops, void *priv, struct module *owner); > + > void fpga_mgr_unregister(struct fpga_manager *mgr); > > +#define devm_fpga_mgr_register_full(parent, info) \ > + __devm_fpga_mgr_register_full(parent, info, THIS_MODULE) > struct fpga_manager * > -devm_fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info); > +__devm_fpga_mgr_register_full(struct device *parent, const struct fpga_manager_info *info, > + struct module *owner); Add a line here. I can do it myself if you agree. There is still a RFC prefix for this patch. Are you ready to get it merged? If yes, Acked-by: Xu Yilun <yilun.xu@xxxxxxxxx> Next time if you think patches are ready for serious review and merge, drop the RFC prefix. That avoids an extra query. Thanks, Yilun > +#define devm_fpga_mgr_register(parent, name, mops, priv) \ > + __devm_fpga_mgr_register(parent, name, mops, priv, THIS_MODULE) > struct fpga_manager * > -devm_fpga_mgr_register(struct device *parent, const char *name, > - const struct fpga_manager_ops *mops, void *priv); > +__devm_fpga_mgr_register(struct device *parent, const char *name, > + const struct fpga_manager_ops *mops, void *priv, > + struct module *owner); > > #endif /*_LINUX_FPGA_MGR_H */ > -- > 2.43.0 > >