[U-Boot] [PATCH 0/2] power: pmic: tps65218: Add DCDC3 configuration

Some boards like am437x-gp-evm require dcdc3 also to be configured as it feeds on to ddr. Hence add the capability as well.
Keerthy (2): power: pmic: tps65218: Add DCDC3 configuration board: ti: AM43XX: Add ddr voltage rail configuration
board/ti/am43xx/board.c | 7 +++++++ drivers/power/pmic/pmic_tps65218.c | 3 ++- include/power/tps65218.h | 1 + 3 files changed, 10 insertions(+), 1 deletion(-)

Some boards like am437x-gp-evm require dcdc3 also to be configured as it feeds on to ddr. Hence add the capability as well.
Signed-off-by: Keerthy j-keerthy@ti.com --- drivers/power/pmic/pmic_tps65218.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/power/pmic/pmic_tps65218.c b/drivers/power/pmic/pmic_tps65218.c index c5e768a..911f639 100644 --- a/drivers/power/pmic/pmic_tps65218.c +++ b/drivers/power/pmic/pmic_tps65218.c @@ -96,7 +96,8 @@ int tps65218_reg_write(uchar prot_level, uchar dest_reg, uchar dest_val, int tps65218_voltage_update(uchar dc_cntrl_reg, uchar volt_sel) { if ((dc_cntrl_reg != TPS65218_DCDC1) && - (dc_cntrl_reg != TPS65218_DCDC2)) + (dc_cntrl_reg != TPS65218_DCDC2) && + (dc_cntrl_reg != TPS65218_DCDC3)) return 1;
/* set voltage level */

On Fri, Jun 02, 2017 at 03:00:30PM +0530, Keerthy wrote:
Some boards like am437x-gp-evm require dcdc3 also to be configured as it feeds on to ddr. Hence add the capability as well.
Signed-off-by: Keerthy j-keerthy@ti.com
Reviewed-by: Tom Rini trini@konsulko.com

On Friday 02 June 2017 03:00 PM, Keerthy wrote:
Some boards like am437x-gp-evm require dcdc3 also to be configured as it feeds on to ddr. Hence add the capability as well.
Signed-off-by: Keerthy j-keerthy@ti.com
Reviewed-by: Lokesh Vutla lokeshvutla@ti.com
Thanks and regards, Lokesh

On 06/02/2017 06:30 PM, Keerthy wrote:
Some boards like am437x-gp-evm require dcdc3 also to be configured as it feeds on to ddr. Hence add the capability as well.
Signed-off-by: Keerthy j-keerthy@ti.com
Applied to u-boot-mmc for pmic. Thanks!
Best Regards, Jaehoon Chung
drivers/power/pmic/pmic_tps65218.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/power/pmic/pmic_tps65218.c b/drivers/power/pmic/pmic_tps65218.c index c5e768a..911f639 100644 --- a/drivers/power/pmic/pmic_tps65218.c +++ b/drivers/power/pmic/pmic_tps65218.c @@ -96,7 +96,8 @@ int tps65218_reg_write(uchar prot_level, uchar dest_reg, uchar dest_val, int tps65218_voltage_update(uchar dc_cntrl_reg, uchar volt_sel) { if ((dc_cntrl_reg != TPS65218_DCDC1) &&
(dc_cntrl_reg != TPS65218_DCDC2))
(dc_cntrl_reg != TPS65218_DCDC2) &&
(dc_cntrl_reg != TPS65218_DCDC3))
return 1;
/* set voltage level */

Add ddr voltage rail (dcdc3) configuration. Set the dcdc3 DDR supply to 1.35V.
Signed-off-by: Keerthy j-keerthy@ti.com --- board/ti/am43xx/board.c | 7 +++++++ include/power/tps65218.h | 1 + 2 files changed, 8 insertions(+)
diff --git a/board/ti/am43xx/board.c b/board/ti/am43xx/board.c index c15d8fa..2c02909 100644 --- a/board/ti/am43xx/board.c +++ b/board/ti/am43xx/board.c @@ -441,6 +441,13 @@ void scale_vcores_generic(u32 m) printf("%s failure\n", __func__); return; } + + /* Set DCDC3 (DDR) voltage */ + if (tps65218_voltage_update(TPS65218_DCDC3, + TPS65218_DCDC3_VOLT_SEL_1350MV)) { + printf("%s failure\n", __func__); + return; + } }
void scale_vcores_idk(u32 m) diff --git a/include/power/tps65218.h b/include/power/tps65218.h index e3538e2..43b9c9a 100644 --- a/include/power/tps65218.h +++ b/include/power/tps65218.h @@ -63,6 +63,7 @@ enum { #define TPS65218_DCDC_VOLT_SEL_1200MV 0x23 #define TPS65218_DCDC_VOLT_SEL_1260MV 0x29 #define TPS65218_DCDC_VOLT_SEL_1330MV 0x30 +#define TPS65218_DCDC3_VOLT_SEL_1350MV 0x12
#define TPS65218_CC_STAT (BIT(0) | BIT(1)) #define TPS65218_STATE (BIT(2) | BIT(3))

On Fri, Jun 02, 2017 at 03:00:31PM +0530, Keerthy wrote:
Add ddr voltage rail (dcdc3) configuration. Set the dcdc3 DDR supply to 1.35V.
Signed-off-by: Keerthy j-keerthy@ti.com
Reviewed-by: Tom Rini trini@konsulko.com

On Friday 02 June 2017 03:00 PM, Keerthy wrote:
Add ddr voltage rail (dcdc3) configuration. Set the dcdc3 DDR supply to 1.35V.
Signed-off-by: Keerthy j-keerthy@ti.com
Reviewed-by: Lokesh Vutla lokeshvutla@ti.com
Thanks and regards, Lokesh

On 06/02/2017 06:30 PM, Keerthy wrote:
Add ddr voltage rail (dcdc3) configuration. Set the dcdc3 DDR supply to 1.35V.
Signed-off-by: Keerthy j-keerthy@ti.com
Applied to u-boot-mmc for pmic. Thanks!
Best Regards, Jaehoon Chung
board/ti/am43xx/board.c | 7 +++++++ include/power/tps65218.h | 1 + 2 files changed, 8 insertions(+)
diff --git a/board/ti/am43xx/board.c b/board/ti/am43xx/board.c index c15d8fa..2c02909 100644 --- a/board/ti/am43xx/board.c +++ b/board/ti/am43xx/board.c @@ -441,6 +441,13 @@ void scale_vcores_generic(u32 m) printf("%s failure\n", __func__); return; }
- /* Set DCDC3 (DDR) voltage */
- if (tps65218_voltage_update(TPS65218_DCDC3,
TPS65218_DCDC3_VOLT_SEL_1350MV)) {
printf("%s failure\n", __func__);
return;
- }
}
void scale_vcores_idk(u32 m) diff --git a/include/power/tps65218.h b/include/power/tps65218.h index e3538e2..43b9c9a 100644 --- a/include/power/tps65218.h +++ b/include/power/tps65218.h @@ -63,6 +63,7 @@ enum { #define TPS65218_DCDC_VOLT_SEL_1200MV 0x23 #define TPS65218_DCDC_VOLT_SEL_1260MV 0x29 #define TPS65218_DCDC_VOLT_SEL_1330MV 0x30 +#define TPS65218_DCDC3_VOLT_SEL_1350MV 0x12
#define TPS65218_CC_STAT (BIT(0) | BIT(1)) #define TPS65218_STATE (BIT(2) | BIT(3))
participants (4)
-
Jaehoon Chung
-
Keerthy
-
Lokesh Vutla
-
Tom Rini