Skip to contentSkip to conversation

powerpc/t4240qds: add support of reading Core voltage

PATCH New shh.xie at gmail.com 3 participants 3 messages Aug 25, 2015 – Oct 15, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 powerpc/t4240qds: add support of reading Core voltage

From: Shaohui Xie <[email protected]>

A U-boot CMD vdd_read is implemented to read Core voltage.

---
 board/freescale/t4qds/t4240qds.c | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)
board/freescale/t4qds/t4240qds.c +30 -0
index 4f2cccd..56273e4 100644
@@ -887,8 +887,38 @@ static int do_vdd_adjust(cmd_tbl_t *cmdtp,
 	return 0;
 }
 
+static int do_vdd_read(cmd_tbl_t *cmdtp,
+		       int flag, int argc,
+		       char * const argv[])
+{
+	int vdd, ret;
+
+	if (argc < 1)
+		return CMD_RET_USAGE;
+
+	ret = select_i2c_ch_pca9547(I2C_MUX_CH_VOL_MONITOR);
+	if (ret) {
+		printf("VID: I2c failed to switch channel\n");
+		return 0;
+	}
+
+	vdd = read_voltage();
+	if (vdd < 0)
+		printf("VID: Could not read voltage\n");
+	else
+		printf("VID: Core voltage is@%d mV\n", vdd);
+
+	return 0;
+}
+
 U_BOOT_CMD(
 	vdd_override, 2, 0, do_vdd_adjust,
 	"Override VDD",
 	"- override with the voltage specified in mV, eg. 1050"
 );
+
+U_BOOT_CMD(
+	vdd_read, 1, 0, do_vdd_read,
+	"read VDD",
+	" - Read the voltage specified in mV"
+)
Signature · 1 line
2.1.0.27.g96db324
#1 lore raw

PATCH 1/1 powerpc/t4240qds: add support of reading Core voltage

Quoted 43 lines · shh.xie at gmail.com

From: Shaohui Xie <[email protected]>

A U-boot CMD vdd_read is implemented to read Core voltage.

---
 board/freescale/t4qds/t4240qds.c | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)
board/freescale/t4qds/t4240qds.c +24 -0
index 4f2cccd..56273e4 100644
@@ -887,8 +887,38 @@ static int do_vdd_adjust(cmd_tbl_t *cmdtp,
 	return 0;
 }
 
+static int do_vdd_read(cmd_tbl_t *cmdtp,
+		       int flag, int argc,
+		       char * const argv[])
+{
+	int vdd, ret;
+
+	if (argc < 1)
+		return CMD_RET_USAGE;
+
+	ret = select_i2c_ch_pca9547(I2C_MUX_CH_VOL_MONITOR);
+	if (ret) {
+		printf("VID: I2c failed to switch channel\n");
+		return 0;
+	}
+
+	vdd = read_voltage();
+	if (vdd < 0)
+		printf("VID: Could not read voltage\n");
+	else
+		printf("VID: Core voltage is at %d mV\n", vdd);
+
+	return 0;
+}
+
 U_BOOT_CMD(

board/freescale/t4qds/t4240qds.c static int do_vdd_adjust(cmd_tbl_t *cmdtp, · line 924

Quoted 10 lines
board/freescale/t4qds/t4240qds.c +6 -0
 	vdd_override, 2, 0, do_vdd_adjust,
 	"Override VDD",
 	"- override with the voltage specified in mV, eg. 1050"
 );
+
+U_BOOT_CMD(
+	vdd_read, 1, 0, do_vdd_read,
+	"read VDD",
+	" - Read the voltage specified in mV"
+)

Can you explain why you need this command? You already get the voltage if you run bdinfo. If you override vdd, you also get a print.

York

#2 lore raw

PATCH 1/1 powerpc/t4240qds: add support of reading Core voltage

Quoted 8 lines · shh.xie at gmail.com
Quoted 3 lines

From: Shaohui Xie <[email protected]>

A U-boot CMD vdd_read is implemented to read Core voltage.

Can you explain why you need this command? You already get the voltage if you run bdinfo. If you override vdd, you also get a print.

[S.H] there was a request that the commands override_vdd and vdd_read should be Supported pair wise on T4240QDS, like what T2080RDB does...

So if the request is not really necessary, we can drop it.

Thanks!
Shaohui