[coreboot-gerrit] [S] Change in coreboot[master]: soc/mediatek/dptx.c: Remove set but unused variables

[Arthur Heymans (Code Review)]

Attention is currently required from: Hung-Te Lin, Yu-Ping Wu, Yidi Lin.

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74547 )

Change subject: soc/mediatek/dptx.c: Remove set but unused variables
......................................................................

soc/mediatek/dptx.c: Remove set but unused variables

This fixes clang warning about set but unused variables.

Change-Id: I3a3345e33380862d6939b61485f6d1eefa3d1815
Signed-off-by: Arthur Heymans 
---
M src/soc/mediatek/common/dp/dptx.c
1 file changed, 14 insertions(+), 4 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/74547/1

diff --git a/src/soc/mediatek/common/dp/dptx.c b/src/soc/mediatek/common/dp/dptx.c
index 08d288a..c123f88 100644
--- a/src/soc/mediatek/common/dp/dptx.c
+++ b/src/soc/mediatek/common/dp/dptx.c
@@ -445,7 +445,7 @@
 {
    u8 bpp;
    u16 sram_read_start = DPTX_TBC_BUF_READSTARTADRTHRD;
-	int tu_size, n_value, f_value, pixclk_mhz;
+	int tu_size, f_value, pixclk_mhz;

bpp = dptx_hal_get_colorbpp(mtk_dp);
    pixclk_mhz = mtk_dp->edid->mode.pixel_clock / 1000;
@@ -453,7 +453,6 @@
    	  (mtk_dp->train_info.linkrate * 27 *
    	   mtk_dp->train_info.linklane_count * 8);

-	n_value = tu_size / 10;
    f_value = tu_size % 10;
    printk(BIOS_DEBUG, "TU_size %d, FValue %d\n", tu_size, f_value);

@@ -950,7 +949,6 @@

static int dptx_set_trainingstart(struct mtk_dp *mtk_dp)
 {
-	int ret = DPTX_PASS;
    u8 lanecount;
    u8 linkrate;
    u8 buffer;
@@ -1006,7 +1004,7 @@
    	mtk_dp->train_info.eq_done = false;

dptx_training_changemode(mtk_dp);
-		ret = dptx_trainingflow(mtk_dp, linkrate, lanecount);
+		dptx_trainingflow(mtk_dp, linkrate, lanecount);

if (!mtk_dp->train_info.cr_done) {
    		/* CR fail and reduce link capability. */