diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-01-25 09:26:55 -0500 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2018-01-30 12:05:25 -0500 |
| commit | 9eb8d7a93c81fdff68189979caff22a058891d15 (patch) | |
| tree | 1b0a70ab540b44b5eda515a92b9eec789e5450f8 | |
| parent | b46310eecd9e23ec140ee2e1004f9335e776da82 (diff) | |
drm/mgag200: fix a test in mga_vga_mode_valid()
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.carpenter@oracle.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20180125142655.GA23885@mwanda
| -rw-r--r-- | drivers/gpu/drm/mgag200/mgag200_mode.c | 4 |
1 files changed, 2 insertions, 2 deletions
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, | |||
| 1620 | return MODE_VIRTUAL_X; | 1620 | return MODE_VIRTUAL_X; |
| 1621 | if (mode->vdisplay > 1024) | 1621 | if (mode->vdisplay > 1024) |
| 1622 | return MODE_VIRTUAL_Y; | 1622 | return MODE_VIRTUAL_Y; |
| 1623 | if (mga_vga_calculate_mode_bandwidth(mode, | 1623 | if (mga_vga_calculate_mode_bandwidth(mode, bpp) > |
| 1624 | bpp > (31877 * 1024))) | 1624 | (31877 * 1024)) |
| 1625 | return MODE_BANDWIDTH; | 1625 | return MODE_BANDWIDTH; |
| 1626 | } else if (mdev->type == G200_EV && | 1626 | } else if (mdev->type == G200_EV && |
| 1627 | (mga_vga_calculate_mode_bandwidth(mode, bpp) | 1627 | (mga_vga_calculate_mode_bandwidth(mode, bpp) |