On Tue Aug 15, 2023 at 11:06 AM EEST, Li Zetao wrote:
> Use the module_platform_driver macro to simplify the code, which is the
> same as declaring with module_init() and module_exit().
>
> Signed-off-by: Li Zetao <lizetao1@xxxxxxxxxx>
> ---
> drivers/char/tpm/tpm_tis_synquacer.c | 18 +-----------------
> 1 file changed, 1 insertion(+), 17 deletions(-)
>
> diff --git a/drivers/char/tpm/tpm_tis_synquacer.c b/drivers/char/tpm/tpm_tis_synquacer.c
> index 49278746b0e2..f7d5e76b505e 100644
> --- a/drivers/char/tpm/tpm_tis_synquacer.c
> +++ b/drivers/char/tpm/tpm_tis_synquacer.c
> @@ -162,23 +162,7 @@ static struct platform_driver tis_synquacer_drv = {
> },
> };
>
> -static int __init tpm_tis_synquacer_module_init(void)
> -{
> - int rc;
> -
> - rc = platform_driver_register(&tis_synquacer_drv);
> - if (rc)
> - return rc;
> -
> - return 0;
> -}
> -
> -static void __exit tpm_tis_synquacer_module_exit(void)
> -{
> - platform_driver_unregister(&tis_synquacer_drv);
> -}
> +module_platform_driver(tis_synquacer_drv);
>
> -module_init(tpm_tis_synquacer_module_init);
> -module_exit(tpm_tis_synquacer_module_exit);
> MODULE_DESCRIPTION("TPM MMIO Driver for Socionext SynQuacer platform");
> MODULE_LICENSE("GPL");
> --
> 2.34.1
Reviewed-by: Jarkko Sakkinen <jarkko@xxxxxxxxxx>
BR, Jarkko
