The parentheses are in the wrong place here so we pass the bits per
pixel as zero.

Fixes: abbee6238775 ("drm/mgag200: Added resolution and bandwidth limits for 
various G200e products.")
Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
---
Static analysis.  Not tested.

diff --git a/drivers/gpu/drm/mgag200/mgag200_mode.c 
b/drivers/gpu/drm/mgag200/mgag200_mode.c
index 68e5d9c94475..fb50a9ddaae8 100644
--- a/drivers/gpu/drm/mgag200/mgag200_mode.c
+++ b/drivers/gpu/drm/mgag200/mgag200_mode.c
@@ -1620,8 +1620,8 @@ static int mga_vga_mode_valid(struct drm_connector 
*connector,
                        return MODE_VIRTUAL_X;
                if (mode->vdisplay > 1024)
                        return MODE_VIRTUAL_Y;
-               if (mga_vga_calculate_mode_bandwidth(mode,
-                       bpp > (31877 * 1024)))
+               if (mga_vga_calculate_mode_bandwidth(mode, bpp) >
+                   (31877 * 1024))
                        return MODE_BANDWIDTH;
        } else if (mdev->type == G200_EV &&
                (mga_vga_calculate_mode_bandwidth(mode, bpp)
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to