Skip to contentSkip to conversation

configs: trats2: enable the max77686 regulator config

PATCH Superseded Jaehoon Chung 2 participants 5 messages Jan 25, 2018 1 message not in the archive Library code

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): CGME20180125070638epcas2p208e91c4ab6d16e860cbc2b386fa15ff7@epcas2p2.samsung.com

#1 lore raw

PATCH 1/3 configs: trats2: enable the max77686 regulator config

Enable the CONFIG_DM_REGULATOR_MAX77686 for using regulator driver.

---
 configs/trats2_defconfig | 2 ++
 1 file changed, 2 insertions(+)
configs/trats2_defconfig +2 -0
index aaa25a9bc7..dab62530e7 100644
@@ -46,6 +46,8 @@ CONFIG_MMC_SDHCI_SDMA=y
 CONFIG_MMC_SDHCI_S5P=y
 CONFIG_DM_PMIC=y
 CONFIG_DM_PMIC_MAX77686=y
+CONFIG_DM_REGULATOR=y
+CONFIG_DM_REGULATOR_MAX77686=y
 CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_GADGET=y
Signature · 1 line
2.15.1
#2 lore raw

PATCH 2/3 power: pmic_max77686: remove the old pmic_max77686 file

max77686 pmic is supporting with max77686.c under pmic/ and regulator/ direnctroy. Remove pmic_max77686.c what didn't use anywhere. Instead, enable CONFIG_DM_REGULATOR_MAX77686 and CONFIG_DM_PMIC_MAX77686.

---
 drivers/power/pmic/Makefile        |   1 -
 drivers/power/pmic/pmic_max77686.c | 304 -------------------------------------
 2 files changed, 305 deletions(-)
 delete mode 100644 drivers/power/pmic/pmic_max77686.c
drivers/power/pmic/Makefile +0 -1
index 7d6c583d34..265b7cb2f6 100644
@@ -29,7 +29,6 @@ obj-$(CONFIG_POWER_MAX77696) += pmic_max77696.o
 obj-$(CONFIG_POWER_MAX8998) += pmic_max8998.o
 obj-$(CONFIG_POWER_MAX8997) += pmic_max8997.o
 obj-$(CONFIG_POWER_MUIC_MAX8997) += muic_max8997.o
-obj-$(CONFIG_POWER_MAX77686) += pmic_max77686.o
 obj-$(CONFIG_POWER_PFUZE100) += pmic_pfuze100.o
 obj-$(CONFIG_POWER_PFUZE3000) += pmic_pfuze3000.o
 obj-$(CONFIG_POWER_TPS65217) += pmic_tps65217.o
drivers/power/pmic/pmic_max77686.c deleted +0 -304

304 lines not on this page · raw message

Signature · 1 line
2.15.1
#3 lore raw

PATCH 2/3 power: pmic_max77686: remove the old pmic_max77686 file

Quoted 332 lines · On Thu, 25 Jan 2018 16:06:36 +0900 Jaehoon Chung <[email protected]> wrote:

max77686 pmic is supporting with max77686.c under pmic/ and regulator/ direnctroy. Remove pmic_max77686.c what didn't use anywhere. Instead, enable CONFIG_DM_REGULATOR_MAX77686 and CONFIG_DM_PMIC_MAX77686.

---
 drivers/power/pmic/Makefile        |   1 -
 drivers/power/pmic/pmic_max77686.c | 304

------------------------------------- 2 files changed, 305 deletions(-) delete mode 100644 drivers/power/pmic/pmic_max77686.c

drivers/power/pmic/Makefile +0 -1
index 7d6c583d34..265b7cb2f6 100644
@@ -29,7 +29,6 @@ obj-$(CONFIG_POWER_MAX77696) += pmic_max77696.o
 obj-$(CONFIG_POWER_MAX8998) += pmic_max8998.o
 obj-$(CONFIG_POWER_MAX8997) += pmic_max8997.o
 obj-$(CONFIG_POWER_MUIC_MAX8997) += muic_max8997.o
-obj-$(CONFIG_POWER_MAX77686) += pmic_max77686.o
 obj-$(CONFIG_POWER_PFUZE100) += pmic_pfuze100.o
 obj-$(CONFIG_POWER_PFUZE3000) += pmic_pfuze3000.o
 obj-$(CONFIG_POWER_TPS65217) += pmic_tps65217.o
(quoted diff) +0 -0

b/drivers/power/pmic/pmic_max77686.c deleted file mode 100644 index 8e653316d1..0000000000

drivers/power/pmic/pmic_max77686.c deleted +0 -41
@@ -1,304 +0,0 @@
-/*
- *  Copyright (C) 2012 Samsung Electronics
- *  Rajeshwari Shinde <[email protected]>
- *
- * SPDX-License-Identifier:	GPL-2.0+
- */
-
-#include <common.h>
-#include <fdtdec.h>
-#include <i2c.h>
-#include <power/pmic.h>
-#include <power/max77686_pmic.h>
-#include <errno.h>
-
-DECLARE_GLOBAL_DATA_PTR;
-
-static const char max77686_buck_addr[] = {
-	0xff, 0x10, 0x12, 0x1c, 0x26, 0x30, 0x32, 0x34, 0x36, 0x38
-};
-
-static unsigned int max77686_ldo_volt2hex(int ldo, ulong uV)
-{
-	unsigned int hex = 0;
-
-	switch (ldo) {
-	case 1:
-	case 2:
-	case 6:
-	case 7:
-	case 8:
-	case 15:
-		hex = (uV - 800000) / 25000;
-		break;
-	default:
-		hex = (uV - 800000) / 50000;
-	}
-
-	if (hex >= 0 && hex <= MAX77686_LDO_VOLT_MAX_HEX)
-		return hex;
-
-	debug("%s: %ld is wrong voltage value for LDO%d\n",

__func__, uV, ldo);

drivers/power/pmic/pmic_max77686.c deleted +0 -9
-	return 0;
-}
-
-static int max77686_buck_volt2hex(int buck, ulong uV)
-{
-	int hex = 0;
-
-	if (buck < 5 || buck > 9) {
-		debug("%s: buck %d is not supported\n", __func__,

buck);

drivers/power/pmic/pmic_max77686.c deleted +0 -19
-		return -EINVAL;
-	}
-
-	hex = (uV - 750000) / 50000;
-
-	if (hex >= 0 && hex <= MAX77686_BUCK_VOLT_MAX_HEX)
-		return hex;
-
-	debug("%s: %ld is wrong voltage value for BUCK%d\n",
-	      __func__, uV, buck);
-	return -EINVAL;
-}
-
-int max77686_set_ldo_voltage(struct pmic *p, int ldo, ulong uV)
-{
-	unsigned int val, ret, hex, adr;
-
-	if (ldo < 1 || ldo > 26) {
-		printf("%s: %d is wrong ldo number\n", __func__,

ldo);

drivers/power/pmic/pmic_max77686.c deleted +0 -53
-		return -EINVAL;
-	}
-
-	adr = MAX77686_REG_PMIC_LDO1CTRL1 + ldo - 1;
-	hex = max77686_ldo_volt2hex(ldo, uV);
-
-	if (!hex)
-		return -EINVAL;
-
-	ret = pmic_reg_read(p, adr, &val);
-	if (ret)
-		return ret;
-
-	val &= ~MAX77686_LDO_VOLT_MASK;
-	val |= hex;
-	ret |= pmic_reg_write(p, adr, val);
-
-	return ret;
-}
-
-int max77686_set_buck_voltage(struct pmic *p, int buck, ulong uV)
-{
-	unsigned int val, adr;
-	int hex, ret;
-
-	if (buck < 5 || buck > 9) {
-		printf("%s: %d is an unsupported bucket number\n",
-		       __func__, buck);
-		return -EINVAL;
-	}
-
-	adr = max77686_buck_addr[buck] + 1;
-	hex = max77686_buck_volt2hex(buck, uV);
-
-	if (hex < 0)
-		return hex;
-
-	ret = pmic_reg_read(p, adr, &val);
-	if (ret)
-		return ret;
-
-	val &= ~MAX77686_BUCK_VOLT_MASK;
-	ret |= pmic_reg_write(p, adr, val | hex);
-
-	return ret;
-}
-
-int max77686_set_ldo_mode(struct pmic *p, int ldo, char opmode)
-{
-	unsigned int val, ret, adr, mode;
-
-	if (ldo < 1 || 26 < ldo) {
-		printf("%s: %d is wrong ldo number\n", __func__,

ldo);

drivers/power/pmic/pmic_max77686.c deleted +0 -62
-		return -EINVAL;
-	}
-
-	adr = MAX77686_REG_PMIC_LDO1CTRL1 + ldo - 1;
-
-	/* mode */
-	switch (opmode) {
-	case OPMODE_OFF:
-		mode = MAX77686_LDO_MODE_OFF;
-		break;
-	case OPMODE_STANDBY:
-		switch (ldo) {
-		case 2:
-		case 6:
-		case 7:
-		case 8:
-		case 10:
-		case 11:
-		case 12:
-		case 14:
-		case 15:
-		case 16:
-			mode = MAX77686_LDO_MODE_STANDBY;
-			break;
-		default:
-			mode = 0xff;
-		}
-		break;
-	case OPMODE_LPM:
-		mode = MAX77686_LDO_MODE_LPM;
-		break;
-	case OPMODE_ON:
-		mode = MAX77686_LDO_MODE_ON;
-		break;
-	default:
-		mode = 0xff;
-	}
-
-	if (mode == 0xff) {
-		printf("%s: %d is not supported on LDO%d\n",
-		       __func__, opmode, ldo);
-		return -ENOTSUPP;
-	}
-
-	ret = pmic_reg_read(p, adr, &val);
-	if (ret)
-		return ret;
-
-	val &= ~MAX77686_LDO_MODE_MASK;
-	val |= mode;
-	ret |= pmic_reg_write(p, adr, val);
-
-	return ret;
-}
-
-int max77686_set_buck_mode(struct pmic *p, int buck, char opmode)
-{
-	unsigned int val, ret, mask, adr, size, mode, mode_shift;
-
-	size = ARRAY_SIZE(max77686_buck_addr);
-	if (buck >= size) {
-		printf("%s: %d is wrong buck number\n", __func__,

buck);

drivers/power/pmic/pmic_max77686.c deleted +0 -30
-		return -EINVAL;
-	}
-
-	adr = max77686_buck_addr[buck];
-
-	/* mask */
-	switch (buck) {
-	case 2:
-	case 3:
-	case 4:
-		mode_shift = MAX77686_BUCK_MODE_SHIFT_2;
-		break;
-	default:
-		mode_shift = MAX77686_BUCK_MODE_SHIFT_1;
-	}
-
-	mask = MAX77686_BUCK_MODE_MASK << mode_shift;
-
-	/* mode */
-	switch (opmode) {
-	case OPMODE_OFF:
-		mode = MAX77686_BUCK_MODE_OFF << mode_shift;
-		break;
-	case OPMODE_STANDBY:
-		switch (buck) {
-		case 1:
-		case 2:
-		case 3:
-		case 4:
-			mode = MAX77686_BUCK_MODE_STANDBY <<

mode_shift;

drivers/power/pmic/pmic_max77686.c deleted +0 -56
-			break;
-		default:
-			mode = 0xff;
-		}
-		break;
-	case OPMODE_LPM:
-		switch (buck) {
-		case 2:
-		case 3:
-		case 4:
-			mode = MAX77686_BUCK_MODE_LPM << mode_shift;
-			break;
-		default:
-			mode = 0xff;
-		}
-		break;
-	case OPMODE_ON:
-		mode = MAX77686_BUCK_MODE_ON << mode_shift;
-		break;
-	default:
-		mode = 0xff;
-	}
-
-	if (mode == 0xff) {
-		printf("%s: %d is not supported on BUCK%d\n",
-		       __func__, opmode, buck);
-		return -ENOTSUPP;
-	}
-
-	ret = pmic_reg_read(p, adr, &val);
-	if (ret)
-		return ret;
-
-	val &= ~mask;
-	val |= mode;
-	ret |= pmic_reg_write(p, adr, val);
-
-	return ret;
-}
-
-int pmic_init(unsigned char bus)
-{
-	static const char name[] = "MAX77686_PMIC";
-	struct pmic *p = pmic_alloc();
-#if CONFIG_IS_ENABLED(OF_CONTROL)
-	const void *blob = gd->fdt_blob;
-	int node, parent, tmp;
-#endif
-
-	if (!p) {
-		printf("%s: POWER allocation error!\n", __func__);
-		return -ENOMEM;
-	}
-
-#if CONFIG_IS_ENABLED(OF_CONTROL)
-	node = fdtdec_next_compatible(blob, 0,

COMPAT_MAXIM_MAX77686_PMIC);

drivers/power/pmic/pmic_max77686.c deleted +0 -2
-	if (node < 0) {
-		debug("PMIC: No node for PMIC Chip in device

tree\n");

drivers/power/pmic/pmic_max77686.c deleted +0 -22
-		debug("node = %d\n", node);
-		return -ENODEV;
-	}
-
-	parent = fdt_parent_offset(blob, node);
-	if (parent < 0) {
-		debug("%s: Cannot find node parent\n", __func__);
-		return -ENODEV;
-	}
-
-	/* tmp since p->bus is unsigned */
-	tmp = i2c_get_bus_num_fdt(parent);
-	if (tmp < 0) {
-		debug("%s: Cannot find I2C bus\n", __func__);
-		return -ENODEV;
-	}
-	p->bus = tmp;
-	p->hw.i2c.addr = fdtdec_get_int(blob, node, "reg", 9);
-#else
-	p->bus = bus;
-	p->hw.i2c.addr = MAX77686_I2C_ADDR;
-#endif

drivers/power/pmic/pmic_max77686.c line 304

Quoted 10 lines
drivers/power/pmic/pmic_max77686.c deleted +0 -10
-
-	p->name = name;
-	p->interface = PMIC_I2C;
-	p->number_of_regs = MAX77686_NUM_OF_REGS;
-	p->hw.i2c.tx_num = 1;
-
-	puts("Board PMIC init\n");
-
-	return 0;
-}

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-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de -------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180125/48453930/attachment.sig>

#4 lore raw

PATCH 3/3 lib: fdtdec: drop the old compatible about max77686

In reply to Jaehoon Chung

Drop the old compatible about max77686.

---
 lib/fdtdec.c | 1 -
 1 file changed, 1 deletion(-)
lib/fdtdec.c +0 -1
index df9d9ae433..58830aa370 100644
@@ -48,7 +48,6 @@ static const char * const compat_names[COMPAT_COUNT] = {
 	COMPAT(SAMSUNG_EXYNOS_MIPI_DSI, "samsung,exynos-mipi-dsi"),
 	COMPAT(SAMSUNG_EXYNOS_DWMMC, "samsung,exynos-dwmmc"),
 	COMPAT(SAMSUNG_EXYNOS_MMC, "samsung,exynos-mmc"),
-	COMPAT(MAXIM_MAX77686_PMIC, "maxim,max77686"),
 	COMPAT(GENERIC_SPI_FLASH, "spi-flash"),
 	COMPAT(MAXIM_98095_CODEC, "maxim,max98095-codec"),
 	COMPAT(SAMSUNG_EXYNOS5_I2C, "samsung,exynos5-hsi2c"),
Signature · 1 line
2.15.1
#5 lore raw

PATCH 1/3 configs: trats2: enable the max77686 regulator config

In reply to Jaehoon Chung
Quoted 12 lines · On Thu, 25 Jan 2018 16:06:35 +0900 Jaehoon Chung <[email protected]> wrote:

Enable the CONFIG_DM_REGULATOR_MAX77686 for using regulator driver.

---
 configs/trats2_defconfig | 2 ++
 1 file changed, 2 insertions(+)
configs/trats2_defconfig +0 -0
index aaa25a9bc7..dab62530e7 100644

configs/trats2_defconfig CONFIG_MMC_SDHCI_SDMA=y · line 53

Quoted 8 lines
configs/trats2_defconfig +2 -0
@@ -46,6 +46,8 @@ CONFIG_MMC_SDHCI_SDMA=y
 CONFIG_MMC_SDHCI_S5P=y
 CONFIG_DM_PMIC=y
 CONFIG_DM_PMIC_MAX77686=y
+CONFIG_DM_REGULATOR=y
+CONFIG_DM_REGULATOR_MAX77686=y
 CONFIG_USB=y
 CONFIG_DM_USB=y
 CONFIG_USB_GADGET=y

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-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de -------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180125/bb00069f/attachment.sig>