@@ -14,7 +14,7 @@ obj-$(CONFIG_AXP209_POWER) += axp209.o
obj-$(CONFIG_AXP221_POWER) += axp221.o
obj-$(CONFIG_AXP305_POWER) += axp305.o
obj-$(CONFIG_AXP313_POWER) += axp313.o
-obj-$(CONFIG_AXP717_POWER) += axp717.o
+obj-$(CONFIG_AXP717_POWER) += axp_spl.o
obj-$(CONFIG_AXP809_POWER) += axp809.o
obj-$(CONFIG_AXP818_POWER) += axp818.o
endif
deleted file mode 100644
@@ -1,92 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0+
-/*
- * AXP717 SPL driver
- * (C) Copyright 2024 Arm Ltd.
- */
-
-#include <common.h>
-#include <command.h>
-#include <errno.h>
-#include <asm/arch/pmic_bus.h>
-#include <axp_pmic.h>
-
-enum axp717_reg {
- AXP717_CHIP_VERSION = 0x3,
- AXP717_SHUTDOWN = 0x27,
- AXP717_OUTPUT_CTRL1 = 0x80,
- AXP717_DCDC1_VOLTAGE = 0x83,
-};
-
-#define AXP717_CHIP_VERSION_MASK 0xc8
-#define AXP717_DCDC_1220MV_OFFSET 71
-#define AXP717_POWEROFF (1U << 0)
-#define DCDC_DVM_ENABLE (1U << 7)
-
-static u8 axp_mvolt_to_cfg(int mvolt, int min, int max, int div)
-{
- if (mvolt < min)
- mvolt = min;
- else if (mvolt > max)
- mvolt = max;
-
- return (mvolt - min) / div;
-}
-
-static int axp_set_dcdc(int dcdc_num, unsigned int mvolt)
-{
- int ret;
- u8 cfg = DCDC_DVM_ENABLE;
-
- if (dcdc_num < 1 || dcdc_num > 3)
- return -EINVAL;
-
- if (mvolt >= 1220)
- cfg |= AXP717_DCDC_1220MV_OFFSET +
- axp_mvolt_to_cfg(mvolt, 1220,
- dcdc_num == 3 ? 1840 : 1540, 20);
- else
- cfg |= axp_mvolt_to_cfg(mvolt, 500, 1200, 10);
-
- if (mvolt == 0)
- return pmic_bus_clrbits(AXP717_OUTPUT_CTRL1,
- 1U << (dcdc_num -1));
-
- ret = pmic_bus_write(AXP717_DCDC1_VOLTAGE + dcdc_num - 1, cfg);
- if (ret)
- return ret;
-
- return pmic_bus_setbits(AXP717_OUTPUT_CTRL1, 1U << (dcdc_num - 1));
-}
-
-int axp_set_dcdc1(unsigned int mvolt)
-{
- return axp_set_dcdc(1, mvolt);
-}
-
-int axp_set_dcdc2(unsigned int mvolt)
-{
- return axp_set_dcdc(2, mvolt);
-}
-
-int axp_set_dcdc3(unsigned int mvolt)
-{
- return axp_set_dcdc(3, mvolt);
-}
-
-int axp_init(void)
-{
- return pmic_bus_init();
-}
-
-#if !CONFIG_IS_ENABLED(ARM_PSCI_FW) && !IS_ENABLED(CONFIG_SYSRESET_CMD_POWEROFF)
-int do_poweroff(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
-{
- pmic_bus_setbits(AXP717_SHUTDOWN, AXP717_POWEROFF);
-
- /* infinite loop during shutdown */
- while (1) {}
-
- /* not reached */
- return 0;
-}
-#endif
We now have a generic AXP SPL driver implementation, that already covers the DC/DC converters of the AXP717 PMIC. Remove the old, dedicated driver and switch to the new generic driver. This should not introduce any change in behaviour. Signed-off-by: Andre Przywara <andre.przywara@arm.com> --- drivers/power/Makefile | 2 +- drivers/power/axp717.c | 92 ------------------------------------------ 2 files changed, 1 insertion(+), 93 deletions(-) delete mode 100644 drivers/power/axp717.c