summaryrefslogtreecommitdiffstats
path: root/drivers/video/fbdev/sis
diff options
context:
space:
mode:
authorSam Ravnborg <sam@ravnborg.org>2020-12-06 20:02:47 +0100
committerSam Ravnborg <sam@ravnborg.org>2020-12-08 18:36:52 +0100
commit604057e984c969cf9f45ded572823e68f4c4b825 (patch)
treec025b3a9b8953a5041b9ef240ee7c8bfda72a162 /drivers/video/fbdev/sis
parentvideo: fbdev: controlfb: Fix set but not used warnings (diff)
downloadlinux-604057e984c969cf9f45ded572823e68f4c4b825.tar.xz
linux-604057e984c969cf9f45ded572823e68f4c4b825.zip
video: fbdev: sis: Drop useless call to SiS_GetResInfo()
Coverity reported: Useless call (USELESS_CALL) side_effect_free: Calling SiS_GetResInfo(SiS_Pr, ModeNo, ModeIdIndex) is only useful for its return value, which is ignored. And this is correct - so drop the call. Signed-off-by: Sam Ravnborg <sam@ravnborg.org> Reported-by: Colin Ian King <colin.king@canonical.com> Addresses-Coverity: ("Useless call") Acked-by: Thomas Zimmermann <tzimmermann@suse.de> Cc: Colin Ian King <colin.king@canonical.com> Cc: Thomas Winischhofer <thomas@winischhofer.net> Link: https://patchwork.freedesktop.org/patch/msgid/20201206190247.1861316-14-sam@ravnborg.org
Diffstat (limited to 'drivers/video/fbdev/sis')
-rw-r--r--drivers/video/fbdev/sis/init.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/video/fbdev/sis/init.c b/drivers/video/fbdev/sis/init.c
index b77ea1a8825a..b568c646a76c 100644
--- a/drivers/video/fbdev/sis/init.c
+++ b/drivers/video/fbdev/sis/init.c
@@ -2659,7 +2659,6 @@ SiS_SetCRT1ModeRegs(struct SiS_Private *SiS_Pr, unsigned short ModeNo,
if(SiS_Pr->UseCustomMode) {
infoflag = SiS_Pr->CInfoFlag;
} else {
- SiS_GetResInfo(SiS_Pr, ModeNo, ModeIdIndex);
if(ModeNo > 0x13) {
infoflag = SiS_Pr->SiS_RefIndex[RRTI].Ext_InfoFlag;
}