Commit 71860632 authored by Luc Van Oostenryck's avatar Luc Van Oostenryck Committed by Kleber Sacilotto de Souza

drm/radeon: fix mode_valid's return type

BugLink: https://bugs.launchpad.net/bugs/1791953

[ Upstream commit 7a47f20e ]

The method struct drm_connector_helper_funcs::mode_valid is defined
as returning an 'enum drm_mode_status' but the driver implementation
for this method uses an 'int' for it.

Fix this by using 'enum drm_mode_status' in the driver too.
Signed-off-by: default avatarLuc Van Oostenryck <luc.vanoostenryck@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarSasha Levin <alexander.levin@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarStefan Bader <stefan.bader@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent cbc596d0
...@@ -844,7 +844,7 @@ static int radeon_lvds_get_modes(struct drm_connector *connector) ...@@ -844,7 +844,7 @@ static int radeon_lvds_get_modes(struct drm_connector *connector)
return ret; return ret;
} }
static int radeon_lvds_mode_valid(struct drm_connector *connector, static enum drm_mode_status radeon_lvds_mode_valid(struct drm_connector *connector,
struct drm_display_mode *mode) struct drm_display_mode *mode)
{ {
struct drm_encoder *encoder = radeon_best_single_encoder(connector); struct drm_encoder *encoder = radeon_best_single_encoder(connector);
...@@ -993,7 +993,7 @@ static int radeon_vga_get_modes(struct drm_connector *connector) ...@@ -993,7 +993,7 @@ static int radeon_vga_get_modes(struct drm_connector *connector)
return ret; return ret;
} }
static int radeon_vga_mode_valid(struct drm_connector *connector, static enum drm_mode_status radeon_vga_mode_valid(struct drm_connector *connector,
struct drm_display_mode *mode) struct drm_display_mode *mode)
{ {
struct drm_device *dev = connector->dev; struct drm_device *dev = connector->dev;
...@@ -1136,7 +1136,7 @@ static int radeon_tv_get_modes(struct drm_connector *connector) ...@@ -1136,7 +1136,7 @@ static int radeon_tv_get_modes(struct drm_connector *connector)
return 1; return 1;
} }
static int radeon_tv_mode_valid(struct drm_connector *connector, static enum drm_mode_status radeon_tv_mode_valid(struct drm_connector *connector,
struct drm_display_mode *mode) struct drm_display_mode *mode)
{ {
if ((mode->hdisplay > 1024) || (mode->vdisplay > 768)) if ((mode->hdisplay > 1024) || (mode->vdisplay > 768))
...@@ -1477,7 +1477,7 @@ static void radeon_dvi_force(struct drm_connector *connector) ...@@ -1477,7 +1477,7 @@ static void radeon_dvi_force(struct drm_connector *connector)
radeon_connector->use_digital = true; radeon_connector->use_digital = true;
} }
static int radeon_dvi_mode_valid(struct drm_connector *connector, static enum drm_mode_status radeon_dvi_mode_valid(struct drm_connector *connector,
struct drm_display_mode *mode) struct drm_display_mode *mode)
{ {
struct drm_device *dev = connector->dev; struct drm_device *dev = connector->dev;
...@@ -1778,7 +1778,7 @@ radeon_dp_detect(struct drm_connector *connector, bool force) ...@@ -1778,7 +1778,7 @@ radeon_dp_detect(struct drm_connector *connector, bool force)
return ret; return ret;
} }
static int radeon_dp_mode_valid(struct drm_connector *connector, static enum drm_mode_status radeon_dp_mode_valid(struct drm_connector *connector,
struct drm_display_mode *mode) struct drm_display_mode *mode)
{ {
struct drm_device *dev = connector->dev; struct drm_device *dev = connector->dev;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment