Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

drm/radeon: remove set but not used variable 'tv_pll_cntl1'

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/radeon/radeon_legacy_tv.c: In function radeon_legacy_tv_mode_set:
drivers/gpu/drm/radeon/radeon_legacy_tv.c:538:24: warning: variable tv_pll_cntl1 set but not used [-Wunused-but-set-variable]

It is introduced by commit 4ce001abafaf ("drm/radeon/kms:
add initial radeon tv-out support."), but never used,
so remove it.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: zhengbin <zhengbin13@huawei.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

authored by

zhengbin and committed by
Alex Deucher
dc9b3dbd 77441f77

+1 -7
+1 -7
drivers/gpu/drm/radeon/radeon_legacy_tv.c
··· 537 537 uint32_t tv_master_cntl, tv_rgb_cntl, tv_dac_cntl; 538 538 uint32_t tv_modulator_cntl1, tv_modulator_cntl2; 539 539 uint32_t tv_vscaler_cntl1, tv_vscaler_cntl2; 540 - uint32_t tv_pll_cntl, tv_pll_cntl1, tv_ftotal; 540 + uint32_t tv_pll_cntl, tv_ftotal; 541 541 uint32_t tv_y_fall_cntl, tv_y_rise_cntl, tv_y_saw_tooth_cntl; 542 542 uint32_t m, n, p; 543 543 const uint16_t *hor_timing; ··· 708 708 ((n & RADEON_TV_N0LO_MASK) << RADEON_TV_N0LO_SHIFT) | 709 709 (((n >> 9) & RADEON_TV_N0HI_MASK) << RADEON_TV_N0HI_SHIFT) | 710 710 ((p & RADEON_TV_P_MASK) << RADEON_TV_P_SHIFT); 711 - 712 - tv_pll_cntl1 = (((4 & RADEON_TVPCP_MASK) << RADEON_TVPCP_SHIFT) | 713 - ((4 & RADEON_TVPVG_MASK) << RADEON_TVPVG_SHIFT) | 714 - ((1 & RADEON_TVPDC_MASK) << RADEON_TVPDC_SHIFT) | 715 - RADEON_TVCLK_SRC_SEL_TVPLL | 716 - RADEON_TVPLL_TEST_DIS); 717 711 718 712 tv_dac->tv.tv_uv_adr = 0xc8; 719 713