These are only used if SST_DEBUG_FUNC is set. Declare them under the same clause as their use. Fixes the following W=1 kernel build warning(s): drivers/video/fbdev/sstfb.c: In function ‘sst_detect_ics’: drivers/video/fbdev/sstfb.c:934:26: warning: variable ‘n_clk1_b’ set but not used [-Wunused-but-set-variable] drivers/video/fbdev/sstfb.c:934:16: warning: variable ‘n_clk0_7’ set but not used [-Wunused-but-set-variable] drivers/video/fbdev/sstfb.c:934:6: warning: variable ‘n_clk0_1’ set but not used [-Wunused-but-set-variable] Cc: Alex Dewar <alex.dewar90@xxxxxxxxx> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> Cc: Ghozlane Toumi <gtoumi@xxxxxxxxxxx> Cc: James Simmons <jsimmons@xxxxxxxxxxxxx> Cc: Paul Mundt <lethal@xxxxxxxxxxxxxxxxx> Cc: Urs Ganse <ursg@xxxxxx> Cc: Helge Deller <deller@xxxxxx> Cc: Alan Cox <alan@xxxxxxxxxx> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx Cc: linux-fbdev@xxxxxxxxxxxxxxx Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx> --- drivers/video/fbdev/sstfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/fbdev/sstfb.c b/drivers/video/fbdev/sstfb.c index c05cdabeb11c2..b8813a0b63b1d 100644 --- a/drivers/video/fbdev/sstfb.c +++ b/drivers/video/fbdev/sstfb.c @@ -931,7 +931,7 @@ static int sst_detect_ics(struct fb_info *info) { struct sstfb_par *par = info->par; int m_clk0_1, m_clk0_7, m_clk1_b; - int n_clk0_1, n_clk0_7, n_clk1_b; + int __maybe_unused n_clk0_1, __maybe_unused n_clk0_7, __maybe_unused n_clk1_b; int i; for (i = 0; i<5; i++ ) { -- 2.25.1 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel