[PATCH] drm/radeon: eliminate redundant connector_names table

[Alex Deucher]

On Fri, May 4, 2012 at 11:25 AM, Ilija Hadzic
<ihadzic at research.bell-labs.com> wrote:
> connector_names table is just a repeat of information that
> already exists in drm_connector_enum_list and the same string
> can be retrieved using drm_get_connector_name function.
>
> Nuke the redundant table and use the proper function to retrieve
> the connector name.
>
> Signed-off-by: Ilija Hadzic <ihadzic at research.bell-labs.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/radeon/radeon_display.c |   20 +-------------------
>  1 files changed, 1 insertions(+), 19 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c
> index 8086c96..60aecc5 100644
> --- a/drivers/gpu/drm/radeon/radeon_display.c
> +++ b/drivers/gpu/drm/radeon/radeon_display.c
> @@ -572,24 +572,6 @@ static const char *encoder_names[36] = {
>        "TRAVIS",
>  };
>
> -static const char *connector_names[15] = {
> -       "Unknown",
> -       "VGA",
> -       "DVI-I",
> -       "DVI-D",
> -       "DVI-A",
> -       "Composite",
> -       "S-video",
> -       "LVDS",
> -       "Component",
> -       "DIN",
> -       "DisplayPort",
> -       "HDMI-A",
> -       "HDMI-B",
> -       "TV",
> -       "eDP",
> -};
> -
>  static const char *hpd_names[6] = {
>        "HPD1",
>        "HPD2",
> @@ -612,7 +594,7 @@ static void radeon_print_display_setup(struct drm_device *dev)
>        list_for_each_entry(connector, &dev->mode_config.connector_list, head) {
>                radeon_connector = to_radeon_connector(connector);
>                DRM_INFO("Connector %d:\n", i);
> -               DRM_INFO("  %s\n", connector_names[connector->connector_type]);
> +               DRM_INFO("  %s\n", drm_get_connector_name(connector));
>                if (radeon_connector->hpd.hpd != RADEON_HPD_NONE)
>                        DRM_INFO("  %s\n", hpd_names[radeon_connector->hpd.hpd]);
>                if (radeon_connector->ddc_bus) {
> --
> 1.7.8.5
>
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel