Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 22 Sep 2022 16:46:59 +0200
From:   Dominik Brodowski <linux@dominikbrodowski.net>
To:     Xiu Jianfeng <xiujianfeng@huawei.com>
Cc:     linux-kernel@vger.kernel.org
Subject: Re: [PATCH] pcmcia: Add __init/__exit annotations to module
 init/exit funcs
Message-ID: <Yyx1Y6OuLmYMQL6O@owl.dominikbrodowski.net>
References: <20220922112323.26353-1-xiujianfeng@huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220922112323.26353-1-xiujianfeng@huawei.com>
Precedence: bulk

Am Thu, Sep 22, 2022 at 07:23:23PM +0800 schrieb Xiu Jianfeng:
> Add missing __init/__exit annotations to module init/exit funcs.
> 
> Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>

Applied to the pcmcia tree, sans the patch to vrc4171_card.c which is
already on its way out of the kernel.

Thanks,
	Dominik

> ---
>  drivers/pcmcia/i82092.c       | 4 ++--
>  drivers/pcmcia/vrc4171_card.c | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/pcmcia/i82092.c b/drivers/pcmcia/i82092.c
> index 192c9049d654..a335748bdef5 100644
> --- a/drivers/pcmcia/i82092.c
> +++ b/drivers/pcmcia/i82092.c
> @@ -661,12 +661,12 @@ static int i82092aa_set_mem_map(struct pcmcia_socket *socket,
>  	return 0;
>  }
>  
> -static int i82092aa_module_init(void)
> +static int __init i82092aa_module_init(void)
>  {
>  	return pci_register_driver(&i82092aa_pci_driver);
>  }
>  
> -static void i82092aa_module_exit(void)
> +static void __exit i82092aa_module_exit(void)
>  {
>  	pci_unregister_driver(&i82092aa_pci_driver);
>  	if (sockets[0].io_base > 0)
> diff --git a/drivers/pcmcia/vrc4171_card.c b/drivers/pcmcia/vrc4171_card.c
> index 177d77892144..ebaeb582539a 100644
> --- a/drivers/pcmcia/vrc4171_card.c
> +++ b/drivers/pcmcia/vrc4171_card.c
> @@ -699,7 +699,7 @@ static struct platform_driver vrc4171_card_driver = {
>  	},
>  };
>  
> -static int vrc4171_card_init(void)
> +static int __init vrc4171_card_init(void)
>  {
>  	int retval;
>  
> @@ -733,7 +733,7 @@ static int vrc4171_card_init(void)
>  	return 0;
>  }
>  
> -static void vrc4171_card_exit(void)
> +static void __exit vrc4171_card_exit(void)
>  {
>  	free_irq(vrc4171_irq, vrc4171_sockets);
>  	vrc4171_remove_sockets();
> -- 
> 2.17.1
>