diff mbox

[U-Boot,7/8] pmic: add tps62362 simple wrapper code

Message ID 1419287177-28401-7-git-send-email-balbi@ti.com
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Felipe Balbi Dec. 22, 2014, 10:26 p.m. UTC
This regulator is used with AM437x IDK to feed
VDD_MPU, without means to scale VDD_MPU we can't
support higher frequencies.

Signed-off-by: Felipe Balbi <balbi@ti.com>
---
 drivers/power/pmic/Makefile        |  1 +
 drivers/power/pmic/pmic_tps62362.c | 47 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 drivers/power/pmic/pmic_tps62362.c

Comments

Mugunthan V N Jan. 6, 2015, 8:58 a.m. UTC | #1
On Tuesday 23 December 2014 03:56 AM, Felipe Balbi wrote:
> This regulator is used with AM437x IDK to feed
> VDD_MPU, without means to scale VDD_MPU we can't
> support higher frequencies.
> 
> Signed-off-by: Felipe Balbi <balbi@ti.com>
> ---
>  drivers/power/pmic/Makefile        |  1 +
>  drivers/power/pmic/pmic_tps62362.c | 47 ++++++++++++++++++++++++++++++++++++++
>  2 files changed, 48 insertions(+)
>  create mode 100644 drivers/power/pmic/pmic_tps62362.c

There is a build error with this patch because tps62362.h is missing in
the commit.

Regards
Mugunthan V N
Felipe Balbi Jan. 6, 2015, 3:12 p.m. UTC | #2
On Tue, Jan 06, 2015 at 02:28:23PM +0530, Mugunthan V N wrote:
> On Tuesday 23 December 2014 03:56 AM, Felipe Balbi wrote:
> > This regulator is used with AM437x IDK to feed
> > VDD_MPU, without means to scale VDD_MPU we can't
> > support higher frequencies.
> > 
> > Signed-off-by: Felipe Balbi <balbi@ti.com>
> > ---
> >  drivers/power/pmic/Makefile        |  1 +
> >  drivers/power/pmic/pmic_tps62362.c | 47 ++++++++++++++++++++++++++++++++++++++
> >  2 files changed, 48 insertions(+)
> >  create mode 100644 drivers/power/pmic/pmic_tps62362.c
> 
> There is a build error with this patch because tps62362.h is missing in
> the commit.

looks like I forgot to git add :-s

yeah, I did. I'll resend it.
diff mbox

Patch

diff --git a/drivers/power/pmic/Makefile b/drivers/power/pmic/Makefile
index e7b07eb..985cfdb 100644
--- a/drivers/power/pmic/Makefile
+++ b/drivers/power/pmic/Makefile
@@ -14,5 +14,6 @@  obj-$(CONFIG_POWER_PFUZE100) += pmic_pfuze100.o
 obj-$(CONFIG_POWER_TPS65090_I2C) += pmic_tps65090.o
 obj-$(CONFIG_POWER_TPS65090_EC) += pmic_tps65090_ec.o
 obj-$(CONFIG_POWER_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/drivers/power/pmic/pmic_tps62362.c b/drivers/power/pmic/pmic_tps62362.c
new file mode 100644
index 0000000..2123685
--- /dev/null
+++ b/drivers/power/pmic/pmic_tps62362.c
@@ -0,0 +1,47 @@ 
+/*
+ * (C) Copyright 2014 Texas Instruments Incorporated -  http://www.ti.com
+ * Author: Felipe Balbi <balbi@ti.com>
+ *
+ * SPDX-License-Identifier:	GPL-2.0+
+ */
+
+#include <common.h>
+#include <i2c.h>
+#include <asm/errno.h>
+#include <power/pmic.h>
+#include <power/tps62362.h>
+
+/**
+ * tps62362_voltage_update() - Function to change a voltage level, as this
+ *			       is a multi-step process.
+ * @reg:	Register address to write to
+ * @volt_sel:	Voltage register value to write
+ * @return:	0 on success, 1 on failure
+ */
+int tps62362_voltage_update(unsigned char reg, unsigned char volt_sel)
+{
+	if (reg > TPS62362_NUM_REGS)
+		return 1;
+
+	return i2c_write(TPS62362_I2C_ADDR, reg, 1, &volt_sel, 1);
+}
+
+int power_tps62362_init(unsigned char bus)
+{
+	static const char name[] = "TPS62362";
+	struct pmic *p = pmic_alloc();
+
+	if (!p) {
+		printf("%s: POWER allocation error!\n", __func__);
+		return -ENOMEM;
+	}
+
+	p->name = name;
+	p->interface = PMIC_I2C;
+	p->number_of_regs = TPS62362_NUM_REGS;
+	p->hw.i2c.addr = TPS62362_I2C_ADDR;
+	p->hw.i2c.tx_num = 1;
+	p->bus = bus;
+
+	return 0;
+}