Change in coreboot[master]: soc/mediatek/mt8186: fix variable type

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/59567 )

Change subject: soc/mediatek/mt8186: fix variable type
......................................................................

soc/mediatek/mt8186: fix variable type

The types of pwrap_read_field()'s return value and pwrap_write_field()'s
`val` argument are u16, so correct the usage in MT6366.

TEST=build pass
BUG=b:202871018

Signed-off-by: Rex-BC Chen 
Change-Id: Ie05ab65ecd9b8ea1379ef74393285c4f5d2db8a4
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59567
Tested-by: build bot (Jenkins) 
Reviewed-by: Yu-Ping Wu 
---
M src/soc/mediatek/mt8186/mt6366.c
1 file changed, 13 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Yu-Ping Wu: Looks good to me, approved

diff --git a/src/soc/mediatek/mt8186/mt6366.c b/src/soc/mediatek/mt8186/mt6366.c
index 406c524..450b783 100644
--- a/src/soc/mediatek/mt8186/mt6366.c
+++ b/src/soc/mediatek/mt8186/mt6366.c
@@ -473,7 +473,7 @@

static u32 pmic_get_vcore_vol(void)
 {
-	u32 vol_reg;
+	u16 vol_reg;

vol_reg = pwrap_read_field(PMIC_VCORE_DBG0, 0x7F, 0);
    return 500000 + vol_reg * 6250;
@@ -481,7 +481,7 @@

static void pmic_set_vcore_vol(u32 vcore_uv)
 {
-	u32 vol_reg;
+	u16 vol_reg;

assert(vcore_uv >= 500000);
    assert(vcore_uv <= 1100000);
@@ -539,7 +539,7 @@

static u32 pmic_get_vdram1_vol(void)
 {
-	u32 vol_reg;
+	u16 vol_reg;

vol_reg = pwrap_read_field(PMIC_VDRAM1_DBG0, 0x7F, 0);
    return 500000 + vol_reg * 12500;
@@ -547,7 +547,7 @@

static void pmic_set_vdram1_vol(u32 vdram_uv)
 {
-	u32 vol_reg;
+	u16 vol_reg;

assert(vdram_uv >= 500000);
    assert(vdram_uv <= 1300000);
@@ -562,7 +562,7 @@
 static u32 pmic_get_vddq_vol(void)
 {
    int efuse_votrim;
-	u32 cali_trim;
+	u16 cali_trim;

if (!pwrap_read_field(PMIC_VDDQ_OP_EN, 0x1, 15))
    	return 0;
@@ -575,7 +575,8 @@

static void pmic_set_vddq_vol(u32 vddq_uv)
 {
-	int target_mv, dram2_ori_mv, cali_offset_uv, cali_trim;
+	int target_mv, dram2_ori_mv, cali_offset_uv;
+	u16 cali_trim;

assert(vddq_uv >= 530000);
    assert(vddq_uv <= 680000);
@@ -606,7 +607,8 @@

static u32 pmic_get_vmch_vol(void)
 {
-	u32 vol_reg, ret;
+	u32 ret;
+	u16 vol_reg;

vol_reg = pwrap_read_field(PMIC_VMCH_ANA_CON0, 0x7, 8);

@@ -630,7 +632,7 @@

static void pmic_set_vmch_vol(u32 vmch_uv)
 {
-	u32 val = 0;
+	u16 val = 0;

switch (vmch_uv) {
    case 2900000:
@@ -656,7 +658,8 @@

static u32 pmic_get_vmc_vol(void)
 {
-	u32 vol_reg, ret;
+	u32 ret;
+	u16 vol_reg;

vol_reg = pwrap_read_field(PMIC_VMC_ANA_CON0, 0xF, 8);

@@ -683,7 +686,7 @@

static void pmic_set_vmc_vol(u32 vmc_uv)
 {
-	u32 val = 0;
+	u16 val = 0;

switch (vmc_uv) {
    case 1800000: