linux-kernel - Re: [PATCH] regulator: Move regulator drivers to subsys

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 28 Apr 2009 08:18:20 +0300
From:	Mike Rapoport <mike@...pulab.co.il>
To:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
CC:	Liam Girdwood <lrg@...mlogic.co.uk>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] regulator: Move regulator drivers to subsys_initcall()



Mark Brown wrote:
> Regulators need to be available early in init in order to allow them
> to be available for consumers when requested. This is generally done
> by registering them at subsys_initcall() time but not all regulator
> drivers have done that. Convert these drivers to do so in order to
> mimimise future support.
> 
> Signed-off-by: Mark Brown <broonie@...nsource.wolfsonmicro.com>
> ---
>  drivers/regulator/da903x.c             |    2 +-
>  drivers/regulator/fixed.c              |    2 +-
>  drivers/regulator/pcf50633-regulator.c |    2 +-
>  drivers/regulator/wm8400-regulator.c   |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/regulator/da903x.c b/drivers/regulator/da903x.c
> index 72b1549..2a491bf 100644
> --- a/drivers/regulator/da903x.c
> +++ b/drivers/regulator/da903x.c
> @@ -504,7 +504,7 @@ static int __init da903x_regulator_init(void)
>  {
>  	return platform_driver_register(&da903x_regulator_driver);
>  }
> -module_init(da903x_regulator_init);
> +subsys_initcall(da903x_regulator_init);

For DA903x part

Acked-by: Mike Rapoport <mike@...pulab.co.il>

>  static void __exit da903x_regulator_exit(void)
>  {
> diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
> index 23d5546..6e0bede 100644
> --- a/drivers/regulator/fixed.c
> +++ b/drivers/regulator/fixed.c
> @@ -117,7 +117,7 @@ static int __init regulator_fixed_voltage_init(void)
>  {
>  	return platform_driver_register(&regulator_fixed_voltage_driver);
>  }
> -module_init(regulator_fixed_voltage_init);
> +subsys_initcall(regulator_fixed_voltage_init);
>  
>  static void __exit regulator_fixed_voltage_exit(void)
>  {
> diff --git a/drivers/regulator/pcf50633-regulator.c b/drivers/regulator/pcf50633-regulator.c
> index cd761d8..8e14900 100644
> --- a/drivers/regulator/pcf50633-regulator.c
> +++ b/drivers/regulator/pcf50633-regulator.c
> @@ -316,7 +316,7 @@ static int __init pcf50633_regulator_init(void)
>  {
>  	return platform_driver_register(&pcf50633_regulator_driver);
>  }
> -module_init(pcf50633_regulator_init);
> +subsys_initcall(pcf50633_regulator_init);
>  
>  static void __exit pcf50633_regulator_exit(void)
>  {
> diff --git a/drivers/regulator/wm8400-regulator.c b/drivers/regulator/wm8400-regulator.c
> index 1574260..01a6c95 100644
> --- a/drivers/regulator/wm8400-regulator.c
> +++ b/drivers/regulator/wm8400-regulator.c
> @@ -380,7 +380,7 @@ static int __init wm8400_regulator_init(void)
>  {
>  	return platform_driver_register(&wm8400_regulator_driver);
>  }
> -module_init(wm8400_regulator_init);
> +subsys_initcall(wm8400_regulator_init);
>  
>  static void __exit wm8400_regulator_exit(void)
>  {

-- 
Sincerely yours,
Mike.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/