OSDN Git Service

video: fbdev: aty: Delete unused variable in radeon_monitor
authorSam Ravnborg <sam@ravnborg.org>
Sat, 28 Nov 2020 22:40:50 +0000 (23:40 +0100)
committerSam Ravnborg <sam@ravnborg.org>
Mon, 30 Nov 2020 18:28:09 +0000 (19:28 +0100)
Fix warning about variable that is asssigned a value but never used.
The variable was indeed never used so delete it.

Keep the call to radeon_probe_i2c_connector() as it may have
side-effects. It is unlikely but I could not verify that is was safe to
drop the call.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linux-fbdev@vger.kernel.org
Link: https://patchwork.freedesktop.org/patch/msgid/20201128224114.1033617-5-sam@ravnborg.org
drivers/video/fbdev/aty/radeon_monitor.c

index 9966c58..df55e23 100644 (file)
@@ -488,12 +488,10 @@ void radeon_probe_screens(struct radeonfb_info *rinfo,
 #if defined(DEBUG) && defined(CONFIG_FB_RADEON_I2C)
                {
                        u8 *EDIDs[4] = { NULL, NULL, NULL, NULL };
-                       int mon_types[4] = {MT_NONE, MT_NONE, MT_NONE, MT_NONE};
                        int i;
 
                        for (i = 0; i < 4; i++)
-                               mon_types[i] = radeon_probe_i2c_connector(rinfo,
-                                                                         i+1, &EDIDs[i]);
+                               radeon_probe_i2c_connector(rinfo, i + 1, &EDIDs[i]);
                }
 #endif /* DEBUG */
                /*