All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lukasz Majewski <lukma@denx.de>
To: Tom Rini <trini@konsulko.com>, u-boot@lists.denx.de
Cc: "Andre Przywara" <andre.przywara@arm.com>,
	"Enric Balletbo i Serra" <enric.balletbo@collabora.com>,
	"Govindaraji Sivanantham" <Govindaraji.Sivanantham@in.bosch.com>,
	"Hannes Schmelzer" <hannes.schmelzer@br-automation.com>,
	"Heiko Schocher" <hs@denx.de>,
	"Hiremath Gireesh" <Gireesh.Hiremath@in.bosch.com>,
	"Jaehoon Chung" <jh80.chung@samsung.com>,
	"Marcin Niestroj" <m.niestroj@grinn-global.com>,
	"Marek Behún" <marek.behun@nic.cz>,
	"Patrick Delaunay" <patrick.delaunay@foss.st.com>,
	"Priyanka Jain" <priyanka.jain@nxp.com>,
	"Quentin Schulz" <quentin.schulz@theobroma-systems.com>,
	"Rick Chen" <rick@andestech.com>,
	"Samuel Holland" <samuel@sholland.org>,
	"Simon Glass" <sjg@chromium.org>,
	"Sjoerd Simons" <sjoerd.simons@collabora.co.uk>,
	"Stefan Roese" <sr@denx.de>,
	"Stephan Gerhold" <stephan@gerhold.net>
Subject: Re: [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217
Date: Thu, 3 Mar 2022 10:29:06 +0100	[thread overview]
Message-ID: <20220303102906.45a61c6f@ktm> (raw)
In-Reply-To: <20220222080401.6859-1-lukma@denx.de>

[-- 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 --]

  parent reply	other threads:[~2022-03-03  9:29 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20220222080425epcas1p2d9b60c85b505dbc59fa34fdb61400d28@epcas1p2.samsung.com>
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-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
2022-03-03  9:29   ` Lukasz Majewski [this message]
2022-03-06 23:13   ` [PATCH 1/3] power: Rename CONFIG_POWER_TPS65217 with CONFIG_PMIC_TPS65217 Jaehoon Chung
2022-03-10 18:52   ` Tom Rini
2022-03-11 12:09     ` Lukasz Majewski
2022-03-11 12:16       ` Tom Rini
2022-03-11 13:35         ` Lukasz Majewski
2022-03-11 13:52           ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220303102906.45a61c6f@ktm \
    --to=lukma@denx.de \
    --cc=Gireesh.Hiremath@in.bosch.com \
    --cc=Govindaraji.Sivanantham@in.bosch.com \
    --cc=andre.przywara@arm.com \
    --cc=enric.balletbo@collabora.com \
    --cc=hannes.schmelzer@br-automation.com \
    --cc=hs@denx.de \
    --cc=jh80.chung@samsung.com \
    --cc=m.niestroj@grinn-global.com \
    --cc=marek.behun@nic.cz \
    --cc=patrick.delaunay@foss.st.com \
    --cc=priyanka.jain@nxp.com \
    --cc=quentin.schulz@theobroma-systems.com \
    --cc=rick@andestech.com \
    --cc=samuel@sholland.org \
    --cc=sjg@chromium.org \
    --cc=sjoerd.simons@collabora.co.uk \
    --cc=sr@denx.de \
    --cc=stephan@gerhold.net \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.