Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-11 Thread Tom Rini
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 
> > > > > Reviewed-by: 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(-)
> > > > 
> > > > 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-lu...@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-tr...@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-lu...@denx.de/
> https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lu...@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


signature.asc
Description: PGP signature


Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-11 Thread Lukasz Majewski
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 
> > > > Reviewed-by: 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(-)
> > > 
> > > 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-lu...@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-tr...@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-lu...@denx.de/
https://patchwork.ozlabs.org/project/uboot/patch/20220222080401.6859-2-lu...@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: lu...@denx.de


pgpiWpQAWcRDM.pgp
Description: OpenPGP digital signature


Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-11 Thread Tom Rini
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 
> > > Reviewed-by: 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(-)  
> > 
> > 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-lu...@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-tr...@konsulko.com/
yesterday and saw a bunch of PMIC symbols get listed.

-- 
Tom


signature.asc
Description: PGP signature


Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-11 Thread Lukasz Majewski
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 
> > Reviewed-by: 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(-)  
> 
> 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-lu...@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: lu...@denx.de


pgpCrrxh7rIvx.pgp
Description: OpenPGP digital signature


Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-10 Thread Tom Rini
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 
> Reviewed-by: 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(-)

This needs to be migrated to Kconfig fully / correctly as well, not just
changing the whitelist entry.

-- 
Tom


signature.asc
Description: PGP signature


Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-06 Thread Jaehoon Chung
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 

Reviewed-by: Jaehoon Chung 

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   2600  /* 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 0x4804  /* Use Timer2 */
> -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>  
>  #include 
>  
> 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
>  

Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-03-03 Thread Lukasz Majewski
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 
> ---
> 
>  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   2600  /* 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
> 0x4804/* Use Timer2 */ -#define CONFIG_POWER_TPS65217
> +#define CONFIG_PMIC_TPS65217
>  
>  #include 
>  
> 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
>  

[PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217

2022-02-22 Thread Lukasz Majewski
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 
---

 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_COM60x481aa000  /* 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_COM60x481aa000  /* 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_COM60x481aa000  /* 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_COM60x481aa000  /* 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 2600  /* 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   0x4804  /* Use Timer2 */
-#define CONFIG_POWER_TPS65217
+#define CONFIG_PMIC_TPS65217
 
 #include 
 
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_COM60x481aa000  /* 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
-- 
2.20.1