* [PATCH 2/3] power: pmic: Provide DM_PMIC support for tps65217 driver
2022-02-22 8:03 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
@ 2022-02-22 8:03 ` Lukasz Majewski
2022-03-06 23:14 ` Jaehoon Chung
2022-02-22 8:04 ` [PATCH 3/3] defconfig: Enable DM_PMIC and DM PMIC_TPS65217 on AM335x EVM board Lukasz Majewski
` (3 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Lukasz Majewski @ 2022-02-22 8:03 UTC (permalink / raw)
To: Tom Rini, u-boot
Cc: Lukasz Majewski, Andre Przywara, Bin Meng, Jaehoon Chung,
Marek Behún, Quentin Schulz, Samuel Holland, Simon Glass,
Stephan Gerhold
The tps65217 PMIC driver is used with am335x SoC based designs.
It is used in the SPL (MLO) as well, so the DM conversion only is
for u-boot proper.
This driver only allows simple reading/writing/dumping of the content
of its registers and requires the DM_I2C for proper operation.
Moreover, new CONFIG_PMIC_TPS65217 has been introduced in Kconfig
to be used with boards, which both support DM_PMIC and DM_I2C.
Signed-off-by: Lukasz Majewski <lukma@denx.de>
---
drivers/power/pmic/Kconfig | 8 +++
drivers/power/pmic/pmic_tps65217.c | 82 ++++++++++++++++++++++++++++++
2 files changed, 90 insertions(+)
diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
index ce0adb18a4..c7739228da 100644
--- a/drivers/power/pmic/Kconfig
+++ b/drivers/power/pmic/Kconfig
@@ -306,6 +306,14 @@ config PMIC_TPS65090
only, and you can enable the regulator/charger drivers separately if
required.
+config PMIC_TPS65217
+ bool "Enable driver for Texas Instruments TPS65217 PMIC"
+ ---help---
+ The TPS65217 is a PMIC containing several LDOs, DC to DC convertors,
+ FETs and a battery charger. This driver provides register access
+ only, and you can enable the regulator/charger drivers separately if
+ required.
+
config PMIC_PALMAS
bool "Enable driver for Texas Instruments PALMAS PMIC"
---help---
diff --git a/drivers/power/pmic/pmic_tps65217.c b/drivers/power/pmic/pmic_tps65217.c
index c7f532df4d..ccbf223593 100644
--- a/drivers/power/pmic/pmic_tps65217.c
+++ b/drivers/power/pmic/pmic_tps65217.c
@@ -6,8 +6,13 @@
#include <common.h>
#include <i2c.h>
+#include <dm.h>
+#include <errno.h>
+#include <fdtdec.h>
+#include <power/pmic.h>
#include <power/tps65217.h>
+#if !CONFIG_IS_ENABLED(DM_PMIC)
struct udevice *tps65217_dev __section(".data") = NULL;
/**
@@ -148,3 +153,80 @@ int power_tps65217_init(unsigned char bus)
#endif
return 0;
}
+#else /* CONFIG_IS_ENABLED(DM_PMIC) */
+static const struct pmic_child_info pmic_children_info[] = {
+ { .prefix = "ldo", .driver = "tps65217_ldo" },
+ { },
+};
+
+static int tps65217_reg_count(struct udevice *dev)
+{
+ return TPS65217_PMIC_NUM_OF_REGS;
+}
+
+static int tps65217_write(struct udevice *dev, uint reg, const uint8_t *buff,
+ int len)
+{
+ if (dm_i2c_write(dev, reg, buff, len)) {
+ pr_err("write error to device: %p register: %#x!\n", dev, reg);
+ return -EIO;
+ }
+
+ return 0;
+}
+
+static int tps65217_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
+{
+ int ret;
+
+ ret = dm_i2c_read(dev, reg, buff, len);
+ if (ret) {
+ pr_err("read error %d from device: %p register: %#x!\n", ret,
+ dev, reg);
+ return -EIO;
+ }
+
+ return 0;
+}
+
+static int tps65217_bind(struct udevice *dev)
+{
+ ofnode regulators_node;
+ int children;
+
+ regulators_node = dev_read_subnode(dev, "regulators");
+ if (!ofnode_valid(regulators_node)) {
+ debug("%s: %s regulators subnode not found!\n", __func__,
+ dev->name);
+ return -ENXIO;
+ }
+
+ debug("%s: '%s' - found regulators subnode\n", __func__, dev->name);
+
+ children = pmic_bind_children(dev, regulators_node, pmic_children_info);
+ if (!children)
+ debug("%s: %s - no child found\n", __func__, dev->name);
+
+ /* Always return success for this device */
+ return 0;
+}
+
+static struct dm_pmic_ops tps65217_ops = {
+ .reg_count = tps65217_reg_count,
+ .read = tps65217_read,
+ .write = tps65217_write,
+};
+
+static const struct udevice_id tps65217_ids[] = {
+ { .compatible = "ti,tps65217" },
+ { }
+};
+
+U_BOOT_DRIVER(pmic_tps65217) = {
+ .name = "tps65217 pmic",
+ .id = UCLASS_PMIC,
+ .of_match = tps65217_ids,
+ .bind = tps65217_bind,
+ .ops = &tps65217_ops,
+};
+#endif
--
2.20.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 2/3] power: pmic: Provide DM_PMIC support for tps65217 driver
2022-02-22 8:03 ` [PATCH 2/3] power: pmic: Provide DM_PMIC support for tps65217 driver Lukasz Majewski
@ 2022-03-06 23:14 ` Jaehoon Chung
0 siblings, 0 replies; 11+ messages in thread
From: Jaehoon Chung @ 2022-03-06 23:14 UTC (permalink / raw)
To: Lukasz Majewski, Tom Rini, u-boot
Cc: Andre Przywara, Bin Meng, Marek Behún, Quentin Schulz,
Samuel Holland, Simon Glass, Stephan Gerhold
On 2/22/22 17:03, Lukasz Majewski wrote:
> The tps65217 PMIC driver is used with am335x SoC based designs.
>
> It is used in the SPL (MLO) as well, so the DM conversion only is
> for u-boot proper.
>
> This driver only allows simple reading/writing/dumping of the content
> of its registers and requires the DM_I2C for proper operation.
>
> Moreover, new CONFIG_PMIC_TPS65217 has been introduced in Kconfig
> to be used with boards, which both support DM_PMIC and DM_I2C.
>
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
Best Regards,
Jaehoon Chung
> ---
>
> drivers/power/pmic/Kconfig | 8 +++
> drivers/power/pmic/pmic_tps65217.c | 82 ++++++++++++++++++++++++++++++
> 2 files changed, 90 insertions(+)
>
> diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig
> index ce0adb18a4..c7739228da 100644
> --- a/drivers/power/pmic/Kconfig
> +++ b/drivers/power/pmic/Kconfig
> @@ -306,6 +306,14 @@ config PMIC_TPS65090
> only, and you can enable the regulator/charger drivers separately if
> required.
>
> +config PMIC_TPS65217
> + bool "Enable driver for Texas Instruments TPS65217 PMIC"
> + ---help---
> + The TPS65217 is a PMIC containing several LDOs, DC to DC convertors,
> + FETs and a battery charger. This driver provides register access
> + only, and you can enable the regulator/charger drivers separately if
> + required.
> +
> config PMIC_PALMAS
> bool "Enable driver for Texas Instruments PALMAS PMIC"
> ---help---
> diff --git a/drivers/power/pmic/pmic_tps65217.c b/drivers/power/pmic/pmic_tps65217.c
> index c7f532df4d..ccbf223593 100644
> --- a/drivers/power/pmic/pmic_tps65217.c
> +++ b/drivers/power/pmic/pmic_tps65217.c
> @@ -6,8 +6,13 @@
>
> #include <common.h>
> #include <i2c.h>
> +#include <dm.h>
> +#include <errno.h>
> +#include <fdtdec.h>
> +#include <power/pmic.h>
> #include <power/tps65217.h>
>
> +#if !CONFIG_IS_ENABLED(DM_PMIC)
> struct udevice *tps65217_dev __section(".data") = NULL;
>
> /**
> @@ -148,3 +153,80 @@ int power_tps65217_init(unsigned char bus)
> #endif
> return 0;
> }
> +#else /* CONFIG_IS_ENABLED(DM_PMIC) */
> +static const struct pmic_child_info pmic_children_info[] = {
> + { .prefix = "ldo", .driver = "tps65217_ldo" },
> + { },
> +};
> +
> +static int tps65217_reg_count(struct udevice *dev)
> +{
> + return TPS65217_PMIC_NUM_OF_REGS;
> +}
> +
> +static int tps65217_write(struct udevice *dev, uint reg, const uint8_t *buff,
> + int len)
> +{
> + if (dm_i2c_write(dev, reg, buff, len)) {
> + pr_err("write error to device: %p register: %#x!\n", dev, reg);
> + return -EIO;
> + }
> +
> + return 0;
> +}
> +
> +static int tps65217_read(struct udevice *dev, uint reg, uint8_t *buff, int len)
> +{
> + int ret;
> +
> + ret = dm_i2c_read(dev, reg, buff, len);
> + if (ret) {
> + pr_err("read error %d from device: %p register: %#x!\n", ret,
> + dev, reg);
> + return -EIO;
> + }
> +
> + return 0;
> +}
> +
> +static int tps65217_bind(struct udevice *dev)
> +{
> + ofnode regulators_node;
> + int children;
> +
> + regulators_node = dev_read_subnode(dev, "regulators");
> + if (!ofnode_valid(regulators_node)) {
> + debug("%s: %s regulators subnode not found!\n", __func__,
> + dev->name);
> + return -ENXIO;
> + }
> +
> + debug("%s: '%s' - found regulators subnode\n", __func__, dev->name);
> +
> + children = pmic_bind_children(dev, regulators_node, pmic_children_info);
> + if (!children)
> + debug("%s: %s - no child found\n", __func__, dev->name);
> +
> + /* Always return success for this device */
> + return 0;
> +}
> +
> +static struct dm_pmic_ops tps65217_ops = {
> + .reg_count = tps65217_reg_count,
> + .read = tps65217_read,
> + .write = tps65217_write,
> +};
> +
> +static const struct udevice_id tps65217_ids[] = {
> + { .compatible = "ti,tps65217" },
> + { }
> +};
> +
> +U_BOOT_DRIVER(pmic_tps65217) = {
> + .name = "tps65217 pmic",
> + .id = UCLASS_PMIC,
> + .of_match = tps65217_ids,
> + .bind = tps65217_bind,
> + .ops = &tps65217_ops,
> +};
> +#endif
^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/3] defconfig: Enable DM_PMIC and DM PMIC_TPS65217 on AM335x EVM board
2022-02-22 8:03 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
2022-02-22 8:03 ` [PATCH 2/3] power: pmic: Provide DM_PMIC support for tps65217 driver Lukasz Majewski
@ 2022-02-22 8:04 ` Lukasz Majewski
2022-03-03 9:29 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
` (2 subsequent siblings)
4 siblings, 0 replies; 11+ messages in thread
From: Lukasz Majewski @ 2022-02-22 8:04 UTC (permalink / raw)
To: Tom Rini, u-boot; +Cc: Lukasz Majewski, Andrew F. Davis
As this board now supports DM_I2C, it is safe for it to also use the
DM converted PMIC tps65217 driver.
Signed-off-by: Lukasz Majewski <lukma@denx.de>
---
configs/am335x_boneblack_vboot_defconfig | 3 +++
configs/am335x_evm_defconfig | 3 +++
configs/am335x_evm_spiboot_defconfig | 3 +++
configs/am335x_hs_evm_defconfig | 3 +++
configs/am335x_hs_evm_uart_defconfig | 3 +++
include/configs/am335x_evm.h | 1 -
6 files changed, 15 insertions(+), 1 deletion(-)
diff --git a/configs/am335x_boneblack_vboot_defconfig b/configs/am335x_boneblack_vboot_defconfig
index c79ffde91c..d46b2ac470 100644
--- a/configs/am335x_boneblack_vboot_defconfig
+++ b/configs/am335x_boneblack_vboot_defconfig
@@ -64,6 +64,9 @@ CONFIG_DM_ETH=y
CONFIG_PHY_GIGE=y
CONFIG_MII=y
CONFIG_DRIVER_TI_CPSW=y
+CONFIG_DM_PMIC=y
+# CONFIG_SPL_DM_PMIC is not set
+CONFIG_PMIC_TPS65217=y
CONFIG_SPI=y
CONFIG_DM_SPI=y
CONFIG_OMAP3_SPI=y
diff --git a/configs/am335x_evm_defconfig b/configs/am335x_evm_defconfig
index 33d0fb818f..7f7556971b 100644
--- a/configs/am335x_evm_defconfig
+++ b/configs/am335x_evm_defconfig
@@ -83,6 +83,9 @@ CONFIG_DM_ETH=y
CONFIG_PHY_GIGE=y
CONFIG_MII=y
CONFIG_DRIVER_TI_CPSW=y
+CONFIG_DM_PMIC=y
+# CONFIG_SPL_DM_PMIC is not set
+CONFIG_PMIC_TPS65217=y
CONFIG_SPI=y
CONFIG_DM_SPI=y
CONFIG_OMAP3_SPI=y
diff --git a/configs/am335x_evm_spiboot_defconfig b/configs/am335x_evm_spiboot_defconfig
index f8acb7e1a9..f7be718b58 100644
--- a/configs/am335x_evm_spiboot_defconfig
+++ b/configs/am335x_evm_spiboot_defconfig
@@ -77,6 +77,9 @@ CONFIG_PHY_SMSC=y
CONFIG_DM_ETH=y
CONFIG_MII=y
CONFIG_DRIVER_TI_CPSW=y
+CONFIG_DM_PMIC=y
+# CONFIG_SPL_DM_PMIC is not set
+CONFIG_PMIC_TPS65217=y
CONFIG_SPI=y
CONFIG_DM_SPI=y
CONFIG_OMAP3_SPI=y
diff --git a/configs/am335x_hs_evm_defconfig b/configs/am335x_hs_evm_defconfig
index 993fbbcd39..fbb2d1f382 100644
--- a/configs/am335x_hs_evm_defconfig
+++ b/configs/am335x_hs_evm_defconfig
@@ -70,6 +70,9 @@ CONFIG_PHY_SMSC=y
CONFIG_DM_ETH=y
CONFIG_MII=y
CONFIG_DRIVER_TI_CPSW=y
+CONFIG_DM_PMIC=y
+# CONFIG_SPL_DM_PMIC is not set
+CONFIG_PMIC_TPS65217=y
CONFIG_SPI=y
CONFIG_DM_SPI=y
CONFIG_OMAP3_SPI=y
diff --git a/configs/am335x_hs_evm_uart_defconfig b/configs/am335x_hs_evm_uart_defconfig
index ecb0526c33..b7008025de 100644
--- a/configs/am335x_hs_evm_uart_defconfig
+++ b/configs/am335x_hs_evm_uart_defconfig
@@ -72,6 +72,9 @@ CONFIG_PHY_SMSC=y
CONFIG_DM_ETH=y
CONFIG_MII=y
CONFIG_DRIVER_TI_CPSW=y
+CONFIG_DM_PMIC=y
+# CONFIG_SPL_DM_PMIC is not set
+CONFIG_PMIC_TPS65217=y
CONFIG_SPI=y
CONFIG_DM_SPI=y
CONFIG_OMAP3_SPI=y
diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
index 7b02d91e47..42660f09d6 100644
--- a/include/configs/am335x_evm.h
+++ b/include/configs/am335x_evm.h
@@ -166,7 +166,6 @@
#define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
/* PMIC support */
-#define CONFIG_PMIC_TPS65217
#define CONFIG_POWER_TPS65910
/* SPL */
--
2.20.1
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-02-22 8:03 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
2022-02-22 8:03 ` [PATCH 2/3] power: pmic: Provide DM_PMIC support for tps65217 driver Lukasz Majewski
2022-02-22 8:04 ` [PATCH 3/3] defconfig: Enable DM_PMIC and DM PMIC_TPS65217 on AM335x EVM board Lukasz Majewski
@ 2022-03-03 9:29 ` Lukasz Majewski
2022-03-06 23:13 ` Jaehoon Chung
2022-03-10 18:52 ` Tom Rini
4 siblings, 0 replies; 11+ messages in thread
From: Lukasz Majewski @ 2022-03-03 9:29 UTC (permalink / raw)
To: Tom Rini, u-boot
Cc: Andre Przywara, Enric Balletbo i Serra, Govindaraji Sivanantham,
Hannes Schmelzer, Heiko Schocher, Hiremath Gireesh, Jaehoon Chung,
Marcin Niestroj, Marek Behún, Patrick Delaunay,
Priyanka Jain, Quentin Schulz, Rick Chen, Samuel Holland,
Simon Glass, Sjoerd Simons, Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 5423 bytes --]
Hi Tom,
> Up till now the CONFIG_POWER_TPS65217 has been defined in several
> header files for am335x SoC.
>
> This patch renames it to CONFIG_PMIC_TPS65217, which better reflects
> the role of this IC circuit.
>
Gentle ping on this patch series.
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> ---
>
> drivers/power/pmic/Makefile | 2 +-
> include/configs/am335x_evm.h | 2 +-
> include/configs/am335x_guardian.h | 2 +-
> include/configs/am335x_shc.h | 2 +-
> include/configs/am335x_sl50.h | 2 +-
> include/configs/brppt1.h | 2 +-
> include/configs/bur_am335x_common.h | 2 +-
> include/configs/chiliboard.h | 2 +-
> scripts/config_whitelist.txt | 2 +-
> 9 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile
> index 401cde32cf..584d6e0e78 100644
> --- a/drivers/power/pmic/Makefile
> +++ b/drivers/power/pmic/Makefile
> @@ -36,7 +36,7 @@ obj-$(CONFIG_POWER_MUIC_MAX8997) += muic_max8997.o
> obj-$(CONFIG_POWER_PCA9450) += pmic_pca9450.o
> obj-$(CONFIG_POWER_PFUZE100) += pmic_pfuze100.o
> obj-$(CONFIG_POWER_PFUZE3000) += pmic_pfuze3000.o
> -obj-$(CONFIG_POWER_TPS65217) += pmic_tps65217.o
> +obj-$(CONFIG_PMIC_TPS65217) += pmic_tps65217.o
> obj-$(CONFIG_POWER_TPS65218) += pmic_tps62362.o
> obj-$(CONFIG_POWER_TPS65218) += pmic_tps65218.o
> obj-$(CONFIG_POWER_TPS65910) += pmic_tps65910.o
> diff --git a/include/configs/am335x_evm.h
> b/include/configs/am335x_evm.h index 9070845b7a..7b02d91e47 100644
> --- a/include/configs/am335x_evm.h
> +++ b/include/configs/am335x_evm.h
> @@ -166,7 +166,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /*
> UART5 */
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
> #define CONFIG_POWER_TPS65910
>
> /* SPL */
> diff --git a/include/configs/am335x_guardian.h
> b/include/configs/am335x_guardian.h index 10a95a10a0..e6e7000529
> 100644 --- a/include/configs/am335x_guardian.h
> +++ b/include/configs/am335x_guardian.h
> @@ -95,7 +95,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /*
> UART5 */
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* Bootcount using the RTC block */
> #define CONFIG_SYS_BOOTCOUNT_LE
> diff --git a/include/configs/am335x_shc.h
> b/include/configs/am335x_shc.h index 5ed4eb3b3c..5ac46bf42b 100644
> --- a/include/configs/am335x_shc.h
> +++ b/include/configs/am335x_shc.h
> @@ -156,7 +156,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /*
> UART5 */
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* SPL */
>
> diff --git a/include/configs/am335x_sl50.h
> b/include/configs/am335x_sl50.h index 0da98975ad..e8233c6cca 100644
> --- a/include/configs/am335x_sl50.h
> +++ b/include/configs/am335x_sl50.h
> @@ -50,7 +50,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /*
> UART5 */
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
> #define CONFIG_POWER_TPS65910
>
> /* SPL */
> diff --git a/include/configs/brppt1.h b/include/configs/brppt1.h
> index d5458edbdb..a9d4b84873 100644
> --- a/include/configs/brppt1.h
> +++ b/include/configs/brppt1.h
> @@ -22,7 +22,7 @@
> #define V_OSCK 26000000 /* Clock
> output from T2 */ #define V_SCLK
> (V_OSCK)
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /*
> * When we have NAND flash we expect to be making use of mtdparts,
> diff --git a/include/configs/bur_am335x_common.h
> b/include/configs/bur_am335x_common.h index 9b2e8b5c6e..129356b87f
> 100644 --- a/include/configs/bur_am335x_common.h
> +++ b/include/configs/bur_am335x_common.h
> @@ -26,7 +26,7 @@
> /* Timer information */
> #define CONFIG_SYS_PTV 2 /* Divisor:
> 2^(PTV+1) => 8 */ #define CONFIG_SYS_TIMERBASE
> 0x48040000 /* Use Timer2 */ -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> #include <asm/arch/omap.h>
>
> diff --git a/include/configs/chiliboard.h
> b/include/configs/chiliboard.h index fe49627263..85e8edab23 100644
> --- a/include/configs/chiliboard.h
> +++ b/include/configs/chiliboard.h
> @@ -107,7 +107,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /*
> UART5 */
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* SPL */
> /* Bootcount using the RTC block */
> diff --git a/scripts/config_whitelist.txt
> b/scripts/config_whitelist.txt index 6b21e3918d..cf007c08a5 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -640,6 +640,7 @@ CONFIG_PM
> CONFIG_PMC_BR_PRELIM
> CONFIG_PMC_OR_PRELIM
> CONFIG_PME_PLAT_CLK_DIV
> +CONFIG_PMIC_TPS65217
> CONFIG_PMU
> CONFIG_PMW_BASE
> CONFIG_POST
> @@ -658,7 +659,6 @@ CONFIG_POWER_PFUZE3000_I2C_ADDR
> CONFIG_POWER_SPI
> CONFIG_POWER_TPS62362
> CONFIG_POWER_TPS65090_EC
> -CONFIG_POWER_TPS65217
> CONFIG_POWER_TPS65218
> CONFIG_POWER_TPS65910
> CONFIG_PPC_CLUSTER_START
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-02-22 8:03 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
` (2 preceding siblings ...)
2022-03-03 9:29 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
@ 2022-03-06 23:13 ` Jaehoon Chung
2022-03-10 18:52 ` Tom Rini
4 siblings, 0 replies; 11+ messages in thread
From: Jaehoon Chung @ 2022-03-06 23:13 UTC (permalink / raw)
To: Lukasz Majewski, Tom Rini, u-boot
Cc: Andre Przywara, Enric Balletbo i Serra, Govindaraji Sivanantham,
Hannes Schmelzer, Heiko Schocher, Hiremath Gireesh,
Marcin Niestroj, Marek Behún, Patrick Delaunay,
Priyanka Jain, Quentin Schulz, Rick Chen, Samuel Holland,
Simon Glass, Sjoerd Simons, Stefan Roese, Stephan Gerhold
On 2/22/22 17:03, Lukasz Majewski wrote:
> Up till now the CONFIG_POWER_TPS65217 has been defined in several header
> files for am335x SoC.
>
> This patch renames it to CONFIG_PMIC_TPS65217, which better reflects the
> role of this IC circuit.
>
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
Best Regards,
Jaehoon Chung
> ---
>
> drivers/power/pmic/Makefile | 2 +-
> include/configs/am335x_evm.h | 2 +-
> include/configs/am335x_guardian.h | 2 +-
> include/configs/am335x_shc.h | 2 +-
> include/configs/am335x_sl50.h | 2 +-
> include/configs/brppt1.h | 2 +-
> include/configs/bur_am335x_common.h | 2 +-
> include/configs/chiliboard.h | 2 +-
> scripts/config_whitelist.txt | 2 +-
> 9 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile
> index 401cde32cf..584d6e0e78 100644
> --- a/drivers/power/pmic/Makefile
> +++ b/drivers/power/pmic/Makefile
> @@ -36,7 +36,7 @@ obj-$(CONFIG_POWER_MUIC_MAX8997) += muic_max8997.o
> obj-$(CONFIG_POWER_PCA9450) += pmic_pca9450.o
> obj-$(CONFIG_POWER_PFUZE100) += pmic_pfuze100.o
> obj-$(CONFIG_POWER_PFUZE3000) += pmic_pfuze3000.o
> -obj-$(CONFIG_POWER_TPS65217) += pmic_tps65217.o
> +obj-$(CONFIG_PMIC_TPS65217) += pmic_tps65217.o
> obj-$(CONFIG_POWER_TPS65218) += pmic_tps62362.o
> obj-$(CONFIG_POWER_TPS65218) += pmic_tps65218.o
> obj-$(CONFIG_POWER_TPS65910) += pmic_tps65910.o
> diff --git a/include/configs/am335x_evm.h b/include/configs/am335x_evm.h
> index 9070845b7a..7b02d91e47 100644
> --- a/include/configs/am335x_evm.h
> +++ b/include/configs/am335x_evm.h
> @@ -166,7 +166,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
>
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
> #define CONFIG_POWER_TPS65910
>
> /* SPL */
> diff --git a/include/configs/am335x_guardian.h b/include/configs/am335x_guardian.h
> index 10a95a10a0..e6e7000529 100644
> --- a/include/configs/am335x_guardian.h
> +++ b/include/configs/am335x_guardian.h
> @@ -95,7 +95,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
>
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* Bootcount using the RTC block */
> #define CONFIG_SYS_BOOTCOUNT_LE
> diff --git a/include/configs/am335x_shc.h b/include/configs/am335x_shc.h
> index 5ed4eb3b3c..5ac46bf42b 100644
> --- a/include/configs/am335x_shc.h
> +++ b/include/configs/am335x_shc.h
> @@ -156,7 +156,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
>
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* SPL */
>
> diff --git a/include/configs/am335x_sl50.h b/include/configs/am335x_sl50.h
> index 0da98975ad..e8233c6cca 100644
> --- a/include/configs/am335x_sl50.h
> +++ b/include/configs/am335x_sl50.h
> @@ -50,7 +50,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
>
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
> #define CONFIG_POWER_TPS65910
>
> /* SPL */
> diff --git a/include/configs/brppt1.h b/include/configs/brppt1.h
> index d5458edbdb..a9d4b84873 100644
> --- a/include/configs/brppt1.h
> +++ b/include/configs/brppt1.h
> @@ -22,7 +22,7 @@
> #define V_OSCK 26000000 /* Clock output from T2 */
> #define V_SCLK (V_OSCK)
>
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /*
> * When we have NAND flash we expect to be making use of mtdparts,
> diff --git a/include/configs/bur_am335x_common.h b/include/configs/bur_am335x_common.h
> index 9b2e8b5c6e..129356b87f 100644
> --- a/include/configs/bur_am335x_common.h
> +++ b/include/configs/bur_am335x_common.h
> @@ -26,7 +26,7 @@
> /* Timer information */
> #define CONFIG_SYS_PTV 2 /* Divisor: 2^(PTV+1) => 8 */
> #define CONFIG_SYS_TIMERBASE 0x48040000 /* Use Timer2 */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> #include <asm/arch/omap.h>
>
> diff --git a/include/configs/chiliboard.h b/include/configs/chiliboard.h
> index fe49627263..85e8edab23 100644
> --- a/include/configs/chiliboard.h
> +++ b/include/configs/chiliboard.h
> @@ -107,7 +107,7 @@
> #define CONFIG_SYS_NS16550_COM6 0x481aa000 /* UART5 */
>
> /* PMIC support */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>
> /* SPL */
> /* Bootcount using the RTC block */
> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
> index 6b21e3918d..cf007c08a5 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -640,6 +640,7 @@ CONFIG_PM
> CONFIG_PMC_BR_PRELIM
> CONFIG_PMC_OR_PRELIM
> CONFIG_PME_PLAT_CLK_DIV
> +CONFIG_PMIC_TPS65217
> CONFIG_PMU
> CONFIG_PMW_BASE
> CONFIG_POST
> @@ -658,7 +659,6 @@ CONFIG_POWER_PFUZE3000_I2C_ADDR
> CONFIG_POWER_SPI
> CONFIG_POWER_TPS62362
> CONFIG_POWER_TPS65090_EC
> -CONFIG_POWER_TPS65217
> CONFIG_POWER_TPS65218
> CONFIG_POWER_TPS65910
> CONFIG_PPC_CLUSTER_START
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-02-22 8:03 ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Lukasz Majewski
` (3 preceding siblings ...)
2022-03-06 23:13 ` Jaehoon Chung
@ 2022-03-10 18:52 ` Tom Rini
2022-03-11 12:09 ` Lukasz Majewski
4 siblings, 1 reply; 11+ messages in thread
From: Tom Rini @ 2022-03-10 18:52 UTC (permalink / raw)
To: Lukasz Majewski
Cc: u-boot, Andre Przywara, Enric Balletbo i Serra,
Govindaraji Sivanantham, Hannes Schmelzer, Heiko Schocher,
Hiremath Gireesh, Jaehoon Chung, Marcin Niestroj,
Marek Behún, Patrick Delaunay, Priyanka Jain, Quentin Schulz,
Rick Chen, Samuel Holland, Simon Glass, Sjoerd Simons,
Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 990 bytes --]
On Tue, Feb 22, 2022 at 09:03:58AM +0100, Lukasz Majewski wrote:
> Up till now the CONFIG_POWER_TPS65217 has been defined in several header
> files for am335x SoC.
>
> This patch renames it to CONFIG_PMIC_TPS65217, which better reflects the
> role of this IC circuit.
>
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> ---
>
> drivers/power/pmic/Makefile | 2 +-
> include/configs/am335x_evm.h | 2 +-
> include/configs/am335x_guardian.h | 2 +-
> include/configs/am335x_shc.h | 2 +-
> include/configs/am335x_sl50.h | 2 +-
> include/configs/brppt1.h | 2 +-
> include/configs/bur_am335x_common.h | 2 +-
> include/configs/chiliboard.h | 2 +-
> scripts/config_whitelist.txt | 2 +-
> 9 files changed, 9 insertions(+), 9 deletions(-)
This needs to be migrated to Kconfig fully / correctly as well, not just
changing the whitelist entry.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-03-10 18:52 ` Tom Rini
@ 2022-03-11 12:09 ` Lukasz Majewski
2022-03-11 12:16 ` Tom Rini
0 siblings, 1 reply; 11+ messages in thread
From: Lukasz Majewski @ 2022-03-11 12:09 UTC (permalink / raw)
To: Tom Rini
Cc: u-boot, Andre Przywara, Enric Balletbo i Serra,
Govindaraji Sivanantham, Hannes Schmelzer, Heiko Schocher,
Hiremath Gireesh, Jaehoon Chung, Marcin Niestroj,
Marek Behún, Patrick Delaunay, Priyanka Jain, Quentin Schulz,
Rick Chen, Samuel Holland, Simon Glass, Sjoerd Simons,
Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 1660 bytes --]
Hi Tom,
> On Tue, Feb 22, 2022 at 09:03:58AM +0100, Lukasz Majewski wrote:
> > Up till now the CONFIG_POWER_TPS65217 has been defined in several
> > header files for am335x SoC.
> >
> > This patch renames it to CONFIG_PMIC_TPS65217, which better
> > reflects the role of this IC circuit.
> >
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> > ---
> >
> > drivers/power/pmic/Makefile | 2 +-
> > include/configs/am335x_evm.h | 2 +-
> > include/configs/am335x_guardian.h | 2 +-
> > include/configs/am335x_shc.h | 2 +-
> > include/configs/am335x_sl50.h | 2 +-
> > include/configs/brppt1.h | 2 +-
> > include/configs/bur_am335x_common.h | 2 +-
> > include/configs/chiliboard.h | 2 +-
> > scripts/config_whitelist.txt | 2 +-
> > 9 files changed, 9 insertions(+), 9 deletions(-)
>
> This needs to be migrated to Kconfig fully / correctly as well, not
> just changing the whitelist entry.
>
But there is follow up patch, which adds proper entry to Kconfig with
new name:
https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
The idea was to first rename the CONFIG_POWER_TPS65217 to
CONFIG_PMIC_TPS65217 (as a single patch) and then provide DM support
for TPS65217 with proper (renamed) Kconfig entry.
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-03-11 12:09 ` Lukasz Majewski
@ 2022-03-11 12:16 ` Tom Rini
2022-03-11 13:35 ` Lukasz Majewski
0 siblings, 1 reply; 11+ messages in thread
From: Tom Rini @ 2022-03-11 12:16 UTC (permalink / raw)
To: Lukasz Majewski
Cc: u-boot, Andre Przywara, Enric Balletbo i Serra,
Govindaraji Sivanantham, Hannes Schmelzer, Heiko Schocher,
Hiremath Gireesh, Jaehoon Chung, Marcin Niestroj,
Marek Behún, Patrick Delaunay, Priyanka Jain, Quentin Schulz,
Rick Chen, Samuel Holland, Simon Glass, Sjoerd Simons,
Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 1854 bytes --]
On Fri, Mar 11, 2022 at 01:09:08PM +0100, Lukasz Majewski wrote:
> Hi Tom,
>
> > On Tue, Feb 22, 2022 at 09:03:58AM +0100, Lukasz Majewski wrote:
> > > Up till now the CONFIG_POWER_TPS65217 has been defined in several
> > > header files for am335x SoC.
> > >
> > > This patch renames it to CONFIG_PMIC_TPS65217, which better
> > > reflects the role of this IC circuit.
> > >
> > > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > > Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> > > ---
> > >
> > > drivers/power/pmic/Makefile | 2 +-
> > > include/configs/am335x_evm.h | 2 +-
> > > include/configs/am335x_guardian.h | 2 +-
> > > include/configs/am335x_shc.h | 2 +-
> > > include/configs/am335x_sl50.h | 2 +-
> > > include/configs/brppt1.h | 2 +-
> > > include/configs/bur_am335x_common.h | 2 +-
> > > include/configs/chiliboard.h | 2 +-
> > > scripts/config_whitelist.txt | 2 +-
> > > 9 files changed, 9 insertions(+), 9 deletions(-)
> >
> > This needs to be migrated to Kconfig fully / correctly as well, not
> > just changing the whitelist entry.
> >
>
> But there is follow up patch, which adds proper entry to Kconfig with
> new name:
>
> https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
>
> The idea was to first rename the CONFIG_POWER_TPS65217 to
> CONFIG_PMIC_TPS65217 (as a single patch) and then provide DM support
> for TPS65217 with proper (renamed) Kconfig entry.
Please re-order the series so we don't do what you're doing here and be
sure to fully migrate everyone. I noticed this as part of finding and
fixing:
https://patchwork.ozlabs.org/project/uboot/patch/20220311121248.2801589-1-trini@konsulko.com/
yesterday and saw a bunch of PMIC symbols get listed.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-03-11 12:16 ` Tom Rini
@ 2022-03-11 13:35 ` Lukasz Majewski
2022-03-11 13:52 ` Tom Rini
0 siblings, 1 reply; 11+ messages in thread
From: Lukasz Majewski @ 2022-03-11 13:35 UTC (permalink / raw)
To: Tom Rini
Cc: u-boot, Andre Przywara, Enric Balletbo i Serra,
Govindaraji Sivanantham, Hannes Schmelzer, Heiko Schocher,
Hiremath Gireesh, Jaehoon Chung, Marcin Niestroj,
Marek Behún, Patrick Delaunay, Priyanka Jain, Quentin Schulz,
Rick Chen, Samuel Holland, Simon Glass, Sjoerd Simons,
Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 2542 bytes --]
Hi Tom,
> On Fri, Mar 11, 2022 at 01:09:08PM +0100, Lukasz Majewski wrote:
> > Hi Tom,
> >
> > > On Tue, Feb 22, 2022 at 09:03:58AM +0100, Lukasz Majewski wrote:
> > > > Up till now the CONFIG_POWER_TPS65217 has been defined in
> > > > several header files for am335x SoC.
> > > >
> > > > This patch renames it to CONFIG_PMIC_TPS65217, which better
> > > > reflects the role of this IC circuit.
> > > >
> > > > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > > > Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> > > > ---
> > > >
> > > > drivers/power/pmic/Makefile | 2 +-
> > > > include/configs/am335x_evm.h | 2 +-
> > > > include/configs/am335x_guardian.h | 2 +-
> > > > include/configs/am335x_shc.h | 2 +-
> > > > include/configs/am335x_sl50.h | 2 +-
> > > > include/configs/brppt1.h | 2 +-
> > > > include/configs/bur_am335x_common.h | 2 +-
> > > > include/configs/chiliboard.h | 2 +-
> > > > scripts/config_whitelist.txt | 2 +-
> > > > 9 files changed, 9 insertions(+), 9 deletions(-)
> > >
> > > This needs to be migrated to Kconfig fully / correctly as well,
> > > not just changing the whitelist entry.
> > >
> >
> > But there is follow up patch, which adds proper entry to Kconfig
> > with new name:
> >
> > https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
> >
> > The idea was to first rename the CONFIG_POWER_TPS65217 to
> > CONFIG_PMIC_TPS65217 (as a single patch) and then provide DM support
> > for TPS65217 with proper (renamed) Kconfig entry.
>
> Please re-order the series so we don't do what you're doing here and
> be sure to fully migrate everyone. I noticed this as part of finding
> and fixing:
> https://patchwork.ozlabs.org/project/uboot/patch/20220311121248.2801589-1-trini@konsulko.com/
> yesterday and saw a bunch of PMIC symbols get listed.
>
I can squash those two patches:
https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-1-lukma@denx.de/
https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
However, the drawback is that we would have the move to Kconfig type of
patch with the rename.
Would it be OK?
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
2022-03-11 13:35 ` Lukasz Majewski
@ 2022-03-11 13:52 ` Tom Rini
0 siblings, 0 replies; 11+ messages in thread
From: Tom Rini @ 2022-03-11 13:52 UTC (permalink / raw)
To: Lukasz Majewski
Cc: u-boot, Andre Przywara, Enric Balletbo i Serra,
Govindaraji Sivanantham, Hannes Schmelzer, Heiko Schocher,
Hiremath Gireesh, Jaehoon Chung, Marcin Niestroj,
Marek Behún, Patrick Delaunay, Priyanka Jain, Quentin Schulz,
Rick Chen, Samuel Holland, Simon Glass, Sjoerd Simons,
Stefan Roese, Stephan Gerhold
[-- Attachment #1: Type: text/plain, Size: 2580 bytes --]
On Fri, Mar 11, 2022 at 02:35:45PM +0100, Lukasz Majewski wrote:
> Hi Tom,
>
> > On Fri, Mar 11, 2022 at 01:09:08PM +0100, Lukasz Majewski wrote:
> > > Hi Tom,
> > >
> > > > On Tue, Feb 22, 2022 at 09:03:58AM +0100, Lukasz Majewski wrote:
> > > > > Up till now the CONFIG_POWER_TPS65217 has been defined in
> > > > > several header files for am335x SoC.
> > > > >
> > > > > This patch renames it to CONFIG_PMIC_TPS65217, which better
> > > > > reflects the role of this IC circuit.
> > > > >
> > > > > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > > > > Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
> > > > > ---
> > > > >
> > > > > drivers/power/pmic/Makefile | 2 +-
> > > > > include/configs/am335x_evm.h | 2 +-
> > > > > include/configs/am335x_guardian.h | 2 +-
> > > > > include/configs/am335x_shc.h | 2 +-
> > > > > include/configs/am335x_sl50.h | 2 +-
> > > > > include/configs/brppt1.h | 2 +-
> > > > > include/configs/bur_am335x_common.h | 2 +-
> > > > > include/configs/chiliboard.h | 2 +-
> > > > > scripts/config_whitelist.txt | 2 +-
> > > > > 9 files changed, 9 insertions(+), 9 deletions(-)
> > > >
> > > > This needs to be migrated to Kconfig fully / correctly as well,
> > > > not just changing the whitelist entry.
> > > >
> > >
> > > But there is follow up patch, which adds proper entry to Kconfig
> > > with new name:
> > >
> > > https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
> > >
> > > The idea was to first rename the CONFIG_POWER_TPS65217 to
> > > CONFIG_PMIC_TPS65217 (as a single patch) and then provide DM support
> > > for TPS65217 with proper (renamed) Kconfig entry.
> >
> > Please re-order the series so we don't do what you're doing here and
> > be sure to fully migrate everyone. I noticed this as part of finding
> > and fixing:
> > https://patchwork.ozlabs.org/project/uboot/patch/20220311121248.2801589-1-trini@konsulko.com/
> > yesterday and saw a bunch of PMIC symbols get listed.
> >
>
> I can squash those two patches:
> https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-1-lukma@denx.de/
> https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lukma@denx.de/
>
> However, the drawback is that we would have the move to Kconfig type of
> patch with the rename.
>
> Would it be OK?
Yes, that's fine. And some patches to migrate other CONFIG.*POWER.*
symbols would be much appreciated too.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread