Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1530537

Re: [PATCH] drivers: net: davinci_mdio: use builtin_platform_driver

From David Miller <davem@davemloft.net>
Newsgroups linux.kernel
Subject Re: [PATCH] drivers: net: davinci_mdio: use builtin_platform_driver
Date 2016-11-25 22:30 +0100
Message-ID <sHwed-5RF-1@gated-at.bofh.it> (permalink)
References <sGH21-69j-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


From: Geliang Tang <geliangtang@gmail.com>
Date: Wed, 23 Nov 2016 22:45:43 +0800

> @@ -536,11 +536,7 @@ static struct platform_driver davinci_mdio_driver = {
>  	.remove = davinci_mdio_remove,
>  };
>  
> -static int __init davinci_mdio_init(void)
> -{
> -	return platform_driver_register(&davinci_mdio_driver);
> -}
> -device_initcall(davinci_mdio_init);
> +builtin_platform_driver(davinci_mdio_driver);
>  

As noted by others this is not a correct transformation, the existing
code works properly when modular.  But it will not with this change.

device_initcall() is rerouted to module_init() inside of a module
build, whereas the thing builtin_platform_driver() expands to does
not.

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH] drivers: net: davinci_mdio: use builtin_platform_driver David Miller <davem@davemloft.net> - 2016-11-25 22:30 +0100

csiph-web