Hi Shashank,
On Mon, Apr 27, 2026 at 11:41:22AM +0900, Shashank Balaji wrote:
[...]
> --- a/drivers/hwtracing/coresight/coresight-core.c
> +++ b/drivers/hwtracing/coresight/coresight-core.c
> @@ -1694,7 +1694,7 @@ static void __exit coresight_exit(void)
> module_init(coresight_init);
> module_exit(coresight_exit);
>
> -int coresight_init_driver(const char *drv, struct amba_driver *amba_drv,
> +int __coresight_init_driver(const char *drv, struct amba_driver *amba_drv,
> struct platform_driver *pdev_drv, struct module
> *owner)
> {
> int ret;
> @@ -1713,7 +1713,7 @@ int coresight_init_driver(const char *drv, struct
> amba_driver *amba_drv,
> amba_driver_unregister(amba_drv);
> return ret;
> }
> -EXPORT_SYMBOL_GPL(coresight_init_driver);
> +EXPORT_SYMBOL_GPL(__coresight_init_driver);
For consistency, we usually use prefix "__" for internal functions.
Could you rename this function as:
int coresight_init_driver_with_owner(..., struct module *owner);
> +#define coresight_init_driver(drv, amba_drv, pdev_drv) \
> + __coresight_init_driver(drv, amba_drv, pdev_drv, THIS_MODULE)
> +int __coresight_init_driver(const char *drv, struct amba_driver *amba_drv,
> struct platform_driver *pdev_drv, struct module
> *owner);
Please first function declaration, then followed by the macro to
define coresight_init_driver().
With above changes:
Reviewed-by: Leo Yan <[email protected]>
