[U-Boot] [PATCH 2/3] power: pmic_max77686: remove the old pmic_max77686 file

Lukasz Majewski lukma at denx.de
Thu Jan 25 09:19:07 UTC 2018


On Thu, 25 Jan 2018 16:06:36 +0900
Jaehoon Chung <jh80.chung at samsung.com> 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.
> 
> Signed-off-by: Jaehoon Chung <jh80.chung at samsung.com>
> ---
>  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
> 
> diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile
> index 7d6c583d34..265b7cb2f6 100644
> --- a/drivers/power/pmic/Makefile
> +++ b/drivers/power/pmic/Makefile
> @@ -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
> diff --git a/drivers/power/pmic/pmic_max77686.c
> b/drivers/power/pmic/pmic_max77686.c deleted file mode 100644
> index 8e653316d1..0000000000
> --- a/drivers/power/pmic/pmic_max77686.c
> +++ /dev/null
> @@ -1,304 +0,0 @@
> -/*
> - *  Copyright (C) 2012 Samsung Electronics
> - *  Rajeshwari Shinde <rajeshwari.s at samsung.com>
> - *
> - * 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);
> -	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);
> -		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);
> -		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);
> -		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);
> -		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;
> -			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);
> -	if (node < 0) {
> -		debug("PMIC: No node for PMIC Chip in device
> tree\n");
> -		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
> -
> -	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;
> -}

Acked-by: Lukasz Majewski <lukma at denx.de>

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>


More information about the U-Boot mailing list