regulator: sm5703: Remove because it is unused and fails to build
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 9 Sep 2024 19:30:51 +0000 (21:30 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 13 Sep 2024 18:08:14 +0000 (19:08 +0100)
This file does not compile because <linux/mfd/sm5703.h> is missing.
In KConfig, it depends on MFD_SM5703.

Both MFD_SM5703 and the missing include rely on another patch that never
got merged. The last iteration related to this patch is [1].

So remove this dead-code and undo commit e8858ba89ca3 ("regulator:
sm5703-regulator: Add regulators support for SM5703 MFD")

[1]: https://lore.kernel.org/lkml/20220423085319.483524-5-markuss.broks@gmail.com/

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://patch.msgid.link/0f5da91a05e7343d290c88e3c583b674cf6219ac.1725910247.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/Kconfig
drivers/regulator/Makefile
drivers/regulator/sm5703-regulator.c [deleted file]

index 4b411a09c1a6708e2fd1775ca7b8328cda679b59..39297f7d8177193e51c99bc2b360c6d9936e62fe 100644 (file)
@@ -1373,13 +1373,6 @@ config REGULATOR_SLG51000
          The SLG51000 is seven compact and customizable low dropout
          regulators.
 
-config REGULATOR_SM5703
-       tristate "Silicon Mitus SM5703 regulators"
-       depends on MFD_SM5703
-       help
-         This driver provides support for voltage regulators of SM5703
-         multi-function device.
-
 config REGULATOR_STM32_BOOSTER
        tristate "STMicroelectronics STM32 BOOSTER"
        depends on ARCH_STM32 || COMPILE_TEST
index a61fa42b13c4d2457a8638521c8fed4186111d0f..3d5a803dce8a0556ba9557fa069c6e37593b3c69 100644 (file)
@@ -160,7 +160,6 @@ obj-$(CONFIG_REGULATOR_S5M8767) += s5m8767.o
 obj-$(CONFIG_REGULATOR_SC2731) += sc2731-regulator.o
 obj-$(CONFIG_REGULATOR_SKY81452) += sky81452-regulator.o
 obj-$(CONFIG_REGULATOR_SLG51000) += slg51000-regulator.o
-obj-$(CONFIG_REGULATOR_SM5703) += sm5703-regulator.o
 obj-$(CONFIG_REGULATOR_STM32_BOOSTER) += stm32-booster.o
 obj-$(CONFIG_REGULATOR_STM32_VREFBUF) += stm32-vrefbuf.o
 obj-$(CONFIG_REGULATOR_STM32_PWR) += stm32-pwr.o
diff --git a/drivers/regulator/sm5703-regulator.c b/drivers/regulator/sm5703-regulator.c
deleted file mode 100644 (file)
index 702461c..0000000
+++ /dev/null
@@ -1,170 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-only
-
-#include <linux/mfd/sm5703.h>
-#include <linux/module.h>
-#include <linux/mod_devicetable.h>
-#include <linux/platform_device.h>
-#include <linux/regmap.h>
-#include <linux/regulator/driver.h>
-#include <linux/regulator/of_regulator.h>
-
-enum sm5703_regulators {
-       SM5703_BUCK,
-       SM5703_LDO1,
-       SM5703_LDO2,
-       SM5703_LDO3,
-       SM5703_USBLDO1,
-       SM5703_USBLDO2,
-       SM5703_VBUS,
-       SM5703_MAX_REGULATORS,
-};
-
-static const int sm5703_ldo_voltagemap[] = {
-       1500000, 1800000, 2600000, 2800000, 3000000, 3300000,
-};
-
-static const int sm5703_buck_voltagemap[] = {
-       1000000, 1000000, 1000000, 1000000,
-       1000000, 1000000, 1000000, 1000000,
-       1000000, 1000000, 1000000, 1100000,
-       1200000, 1300000, 1400000, 1500000,
-       1600000, 1700000, 1800000, 1900000,
-       2000000, 2100000, 2200000, 2300000,
-       2400000, 2500000, 2600000, 2700000,
-       2800000, 2900000, 3000000, 3000000,
-};
-
-#define SM5703USBLDO(_name, _id)                                       \
-       [SM5703_USBLDO ## _id] = {                                      \
-               .name = _name,                                          \
-               .of_match = _name,                                      \
-               .regulators_node = "regulators",                        \
-               .type = REGULATOR_VOLTAGE,                              \
-               .id = SM5703_USBLDO ## _id,                             \
-               .ops = &sm5703_regulator_ops_fixed,                     \
-               .n_voltages = 1,                                        \
-               .fixed_uV = SM5703_USBLDO_MICROVOLT,                    \
-               .enable_reg = SM5703_REG_USBLDO12,                      \
-               .enable_mask = SM5703_REG_EN_USBLDO ##_id,              \
-               .owner                  = THIS_MODULE,                  \
-       }
-
-#define SM5703VBUS(_name)                                              \
-       [SM5703_VBUS] = {                                               \
-               .name = _name,                                          \
-               .of_match = _name,                                      \
-               .regulators_node = "regulators",                        \
-               .type = REGULATOR_VOLTAGE,                              \
-               .id = SM5703_VBUS,                                      \
-               .ops = &sm5703_regulator_ops_fixed,                     \
-               .n_voltages = 1,                                        \
-               .fixed_uV = SM5703_VBUS_MICROVOLT,                      \
-               .enable_reg = SM5703_REG_CNTL,                          \
-               .enable_mask = SM5703_OPERATION_MODE_MASK,              \
-               .enable_val = SM5703_OPERATION_MODE_USB_OTG_MODE,       \
-               .disable_val = SM5703_OPERATION_MODE_CHARGING_ON,       \
-               .owner                  = THIS_MODULE,                  \
-       }
-
-#define SM5703BUCK(_name)                                              \
-       [SM5703_BUCK] = {                                               \
-               .name = _name,                                          \
-               .of_match = _name,                                      \
-               .regulators_node = "regulators",                        \
-               .type = REGULATOR_VOLTAGE,                              \
-               .id = SM5703_BUCK,                                      \
-               .ops = &sm5703_regulator_ops,                           \
-               .n_voltages = ARRAY_SIZE(sm5703_buck_voltagemap),       \
-               .volt_table = sm5703_buck_voltagemap,                   \
-               .vsel_reg = SM5703_REG_BUCK,                            \
-               .vsel_mask = SM5703_BUCK_VOLT_MASK,                     \
-               .enable_reg = SM5703_REG_BUCK,                          \
-               .enable_mask = SM5703_REG_EN_BUCK,                      \
-               .owner                  = THIS_MODULE,                  \
-       }
-
-#define SM5703LDO(_name, _id)                                          \
-       [SM5703_LDO ## _id] = {                                         \
-               .name = _name,                                          \
-               .of_match = _name,                                      \
-               .regulators_node = "regulators",                        \
-               .type = REGULATOR_VOLTAGE,                              \
-               .id = SM5703_LDO ## _id,                                \
-               .ops = &sm5703_regulator_ops,                           \
-               .n_voltages = ARRAY_SIZE(sm5703_ldo_voltagemap),        \
-               .volt_table = sm5703_ldo_voltagemap,                    \
-               .vsel_reg = SM5703_REG_LDO ##_id,                       \
-               .vsel_mask = SM5703_LDO_VOLT_MASK,                      \
-               .enable_reg = SM5703_REG_LDO ##_id,                     \
-               .enable_mask = SM5703_LDO_EN,                           \
-               .owner                  = THIS_MODULE,                  \
-       }
-
-static const struct regulator_ops sm5703_regulator_ops = {
-       .enable                 = regulator_enable_regmap,
-       .disable                = regulator_disable_regmap,
-       .is_enabled             = regulator_is_enabled_regmap,
-       .list_voltage           = regulator_list_voltage_table,
-       .get_voltage_sel        = regulator_get_voltage_sel_regmap,
-       .set_voltage_sel        = regulator_set_voltage_sel_regmap,
-};
-
-static const struct regulator_ops sm5703_regulator_ops_fixed = {
-       .enable                 = regulator_enable_regmap,
-       .disable                = regulator_disable_regmap,
-       .is_enabled             = regulator_is_enabled_regmap,
-};
-
-static struct regulator_desc sm5703_regulators_desc[SM5703_MAX_REGULATORS] = {
-       SM5703BUCK("buck"),
-       SM5703LDO("ldo1", 1),
-       SM5703LDO("ldo2", 2),
-       SM5703LDO("ldo3", 3),
-       SM5703USBLDO("usbldo1", 1),
-       SM5703USBLDO("usbldo2", 2),
-       SM5703VBUS("vbus"),
-};
-
-static int sm5703_regulator_probe(struct platform_device *pdev)
-{
-       struct device *dev = &pdev->dev;
-       struct regulator_config config = { NULL, };
-       struct regulator_dev *rdev;
-       struct sm5703_dev *sm5703 = dev_get_drvdata(pdev->dev.parent);
-       int i;
-
-       config.dev = dev->parent;
-       config.regmap = sm5703->regmap;
-
-       for (i = 0; i < SM5703_MAX_REGULATORS; i++) {
-               rdev = devm_regulator_register(dev,
-                                              &sm5703_regulators_desc[i],
-                                              &config);
-               if (IS_ERR(rdev))
-                       return dev_err_probe(dev, PTR_ERR(rdev),
-                                            "Failed to register a regulator\n");
-       }
-
-       return 0;
-}
-
-static const struct platform_device_id sm5703_regulator_id[] = {
-       { "sm5703-regulator", 0 },
-       {}
-};
-MODULE_DEVICE_TABLE(platform, sm5703_regulator_id);
-
-static struct platform_driver sm5703_regulator_driver = {
-       .driver = {
-               .name = "sm5703-regulator",
-               .probe_type = PROBE_PREFER_ASYNCHRONOUS,
-       },
-       .probe  = sm5703_regulator_probe,
-       .id_table       = sm5703_regulator_id,
-};
-
-module_platform_driver(sm5703_regulator_driver);
-
-MODULE_DESCRIPTION("Silicon Mitus SM5703 LDO/Buck/USB regulator driver");
-MODULE_AUTHOR("Markuss Broks <markuss.broks@gmail.com>");
-MODULE_LICENSE("GPL");