[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d58fab14-a91c-45eb-94b5-c13ec3ff7984@amd.com>
Date: Mon, 2 Sep 2024 14:45:41 +0200
From: Michal Simek <michal.simek@....com>
To: Liao Chen <liaochen4@...wei.com>, linux-gpio@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Cc: linus.walleij@...aro.org, brgl@...ev.pl
Subject: Re: [PATCH -next] gpio: modepin: Enable module autoloading
On 9/2/24 13:58, Liao Chen wrote:
> Add MODULE_DEVICE_TABLE(), so modules could be properly autoloaded based
> on the alias from of_device_id table.
>
> Signed-off-by: Liao Chen <liaochen4@...wei.com>
> ---
> drivers/gpio/gpio-zynqmp-modepin.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpio/gpio-zynqmp-modepin.c b/drivers/gpio/gpio-zynqmp-modepin.c
> index a0d69387c153..2f3c9ebfa78d 100644
> --- a/drivers/gpio/gpio-zynqmp-modepin.c
> +++ b/drivers/gpio/gpio-zynqmp-modepin.c
> @@ -146,6 +146,7 @@ static const struct of_device_id modepin_platform_id[] = {
> { .compatible = "xlnx,zynqmp-gpio-modepin", },
> { }
> };
> +MODULE_DEVICE_TABLE(of, modepin_platform_id);
>
> static struct platform_driver modepin_platform_driver = {
> .driver = {
I did the same change internally some day ago but didn't send it.
Reviewed-by: Michal Simek <michal.simek@....com>
Thanks,
Michal
Powered by blists - more mailing lists