Hi Krzysztof, ... > /** > - * amba_driver_register - register an AMBA device driver > + * __amba_driver_register - register an AMBA device driver > * @drv: amba device driver structure > + * @owner: owning module/driver > * > * Register an AMBA device driver with the Linux device model > * core. If devices pre-exist, the drivers probe function will > * be called. > */ > -int amba_driver_register(struct amba_driver *drv) > +int __amba_driver_register(struct amba_driver *drv, ... > +/* > + * use a macro to avoid include chaining to get THIS_MODULE > + */ Should the documentation be moved here? Well... I don't see any documentation linking this file yet, but in case it comes we want documented amba_driver_register() rather than __amba_driver_register(). Andi > +#define amba_driver_register(drv) \ > + __amba_driver_register(drv, THIS_MODULE) > +