Re: [PATCH 1/2] mfd: max77693: Allow building as a module

From: Lee Jones
Date: Thu Apr 07 2016 - 09:29:56 EST


On Mon, 04 Apr 2016, Krzysztof Kozlowski wrote:

> The consumer of max77693 regulators on Trats2 board (samsung-usb2-phy
> driver) supports deferred probing so the max77693 main MFD driver can be
> built now as a module. This gives more flexibility and removes manual
> ordering of init calls.
>
> Suggested-by: Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx>
> Cc: Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx>
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
> ---
> drivers/mfd/Kconfig | 4 ++--
> drivers/mfd/max77693.c | 14 ++------------
> 2 files changed, 4 insertions(+), 14 deletions(-)

I assume this can be taken immediately and doesn't depend on anything
external to the set?

For my own reference:
Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>

> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> index eea61e349e26..aa1e7c3f82ee 100644
> --- a/drivers/mfd/Kconfig
> +++ b/drivers/mfd/Kconfig
> @@ -543,8 +543,8 @@ config MFD_MAX77686
> of the device.
>
> config MFD_MAX77693
> - bool "Maxim Semiconductor MAX77693 PMIC Support"
> - depends on I2C=y
> + tristate "Maxim Semiconductor MAX77693 PMIC Support"
> + depends on I2C
> select MFD_CORE
> select REGMAP_I2C
> select REGMAP_IRQ
> diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
> index b83b7a7da1ae..78e501feb96c 100644
> --- a/drivers/mfd/max77693.c
> +++ b/drivers/mfd/max77693.c
> @@ -368,6 +368,7 @@ static const struct of_device_id max77693_dt_match[] = {
> { .compatible = "maxim,max77693" },
> {},
> };
> +MODULE_DEVICE_TABLE(of, max77693_dt_match);
> #endif
>
> static struct i2c_driver max77693_i2c_driver = {
> @@ -381,18 +382,7 @@ static struct i2c_driver max77693_i2c_driver = {
> .id_table = max77693_i2c_id,
> };
>
> -static int __init max77693_i2c_init(void)
> -{
> - return i2c_add_driver(&max77693_i2c_driver);
> -}
> -/* init early so consumer devices can complete system boot */
> -subsys_initcall(max77693_i2c_init);
> -
> -static void __exit max77693_i2c_exit(void)
> -{
> - i2c_del_driver(&max77693_i2c_driver);
> -}
> -module_exit(max77693_i2c_exit);
> +module_i2c_driver(max77693_i2c_driver);
>
> MODULE_DESCRIPTION("MAXIM 77693 multi-function core driver");
> MODULE_AUTHOR("SangYoung, Son <hello.son@xxxxxxxxxxx>");

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog