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 arthur@aheymans.xyz --- 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. */