Commit 6140cf20 authored by Jani Nikula's avatar Jani Nikula

drm: add drm_get_connector_force_name

Follow the naming in debugfs also for logging, add "unknown" for values
beyond the enumerated ones.

v2: add \n in connector_show, make internal to drm (Chris)

Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1487580708-29340-1-git-send-email-jani.nikula@intel.com
parent 0eb9acda
...@@ -128,22 +128,8 @@ static void drm_connector_get_cmdline_mode(struct drm_connector *connector) ...@@ -128,22 +128,8 @@ static void drm_connector_get_cmdline_mode(struct drm_connector *connector)
return; return;
if (mode->force) { if (mode->force) {
const char *s; DRM_INFO("forcing %s connector %s\n", connector->name,
drm_get_connector_force_name(mode->force));
switch (mode->force) {
case DRM_FORCE_OFF:
s = "OFF";
break;
case DRM_FORCE_ON_DIGITAL:
s = "ON - dig";
break;
default:
case DRM_FORCE_ON:
s = "ON";
break;
}
DRM_INFO("forcing %s connector %s\n", connector->name, s);
connector->force = mode->force; connector->force = mode->force;
} }
...@@ -492,6 +478,28 @@ const char *drm_get_connector_status_name(enum drm_connector_status status) ...@@ -492,6 +478,28 @@ const char *drm_get_connector_status_name(enum drm_connector_status status)
} }
EXPORT_SYMBOL(drm_get_connector_status_name); EXPORT_SYMBOL(drm_get_connector_status_name);
/**
* drm_get_connector_force_name - return a string for connector force
* @force: connector force to get name of
*
* Returns: const pointer to name.
*/
const char *drm_get_connector_force_name(enum drm_connector_force force)
{
switch (force) {
case DRM_FORCE_UNSPECIFIED:
return "unspecified";
case DRM_FORCE_OFF:
return "off";
case DRM_FORCE_ON:
return "on";
case DRM_FORCE_ON_DIGITAL:
return "digital";
default:
return "unknown";
}
}
#ifdef CONFIG_LOCKDEP #ifdef CONFIG_LOCKDEP
static struct lockdep_map connector_list_iter_dep_map = { static struct lockdep_map connector_list_iter_dep_map = {
.name = "drm_connector_list_iter" .name = "drm_connector_list_iter"
......
...@@ -142,6 +142,7 @@ int drm_mode_connector_set_obj_prop(struct drm_mode_object *obj, ...@@ -142,6 +142,7 @@ int drm_mode_connector_set_obj_prop(struct drm_mode_object *obj,
struct drm_property *property, struct drm_property *property,
uint64_t value); uint64_t value);
int drm_connector_create_standard_properties(struct drm_device *dev); int drm_connector_create_standard_properties(struct drm_device *dev);
const char *drm_get_connector_force_name(enum drm_connector_force force);
/* IOCTL */ /* IOCTL */
int drm_mode_connector_property_set_ioctl(struct drm_device *dev, int drm_mode_connector_property_set_ioctl(struct drm_device *dev,
......
...@@ -261,30 +261,8 @@ int drm_debugfs_cleanup(struct drm_minor *minor) ...@@ -261,30 +261,8 @@ int drm_debugfs_cleanup(struct drm_minor *minor)
static int connector_show(struct seq_file *m, void *data) static int connector_show(struct seq_file *m, void *data)
{ {
struct drm_connector *connector = m->private; struct drm_connector *connector = m->private;
const char *status;
switch (connector->force) { seq_printf(m, "%s\n", drm_get_connector_force_name(connector->force));
case DRM_FORCE_ON:
status = "on\n";
break;
case DRM_FORCE_ON_DIGITAL:
status = "digital\n";
break;
case DRM_FORCE_OFF:
status = "off\n";
break;
case DRM_FORCE_UNSPECIFIED:
status = "unspecified\n";
break;
default:
return 0;
}
seq_puts(m, status);
return 0; return 0;
} }
......
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