From patchwork Tue Aug 9 13:41:29 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Vetter X-Patchwork-Id: 9271235 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8C83460754 for ; Tue, 9 Aug 2016 13:42:28 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7ECD927D0C for ; Tue, 9 Aug 2016 13:42:28 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 73B50283DF; Tue, 9 Aug 2016 13:42:28 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.1 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 25E3227D0C for ; Tue, 9 Aug 2016 13:42:28 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E10716E585; Tue, 9 Aug 2016 13:42:15 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mail-wm0-x242.google.com (mail-wm0-x242.google.com [IPv6:2a00:1450:400c:c09::242]) by gabe.freedesktop.org (Postfix) with ESMTPS id 87F336E598 for ; Tue, 9 Aug 2016 13:42:11 +0000 (UTC) Received: by mail-wm0-x242.google.com with SMTP id i5so3186889wmg.2 for ; Tue, 09 Aug 2016 06:42:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=v2xASD/qejB0q710nsK06wOmImm4WZeun8Y6BiFrtW4=; b=XpBlrEOzzSgC+zvj1+g4t+UBm230POmWA2/OtDz53l7YpQJ9mJlIDC/SsZ2gz06oOz IPxq18/wZdsezUmxjR5YZqDei3O65iA9I/L9LzzVgyG3rSvyaMcArsr2SGG9libIYH8T J+mh4gul+SXiw4tADoedGa7zPh1xtkIWVNCaY= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=v2xASD/qejB0q710nsK06wOmImm4WZeun8Y6BiFrtW4=; b=QEBrzJbpYrW4gvLl3lJZpayhlhpKAD87/Mvb4Zi8ZPpcYePB3ukduE82XtZ5d+cVH3 o6BXk1Rus9lvIjz+RDp7Um8qnYOh4vClR1VXEU4MAbDMNE3b4Z6UTgxlWbyDByZMNR0L bEBb6Izl5/cmdbTYDFfQ+49OCegguLiUAOOB7SmQUJ8IGR8K1NUyQRHHO4/+Qu2SOGPD hszMqjUBdxWktnPATyjD8FoWu1DswA76VRyk2WsmxOgaZ+n32As0FwaopC7dC6Jb2HGY I18khTXt4JIUPKEE0vxO9z03zDj5BYElqKoSAzspEXp1Y4yPdLfpLYne21B1XC2TYcY4 pk1w== X-Gm-Message-State: AEkoouu16yvAw7tnxTqAsNVyFIMBoPFthQ7aeMRMPWiFt5yu5vk0Tg2MRKf5i9zOiLZjXg== X-Received: by 10.194.189.229 with SMTP id gl5mr90327753wjc.168.1470750129901; Tue, 09 Aug 2016 06:42:09 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:56b5:0:ac27:b86c:7764:9429]) by smtp.gmail.com with ESMTPSA id p1sm38077003wjd.37.2016.08.09.06.42.07 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 09 Aug 2016 06:42:08 -0700 (PDT) From: Daniel Vetter To: DRI Development Date: Tue, 9 Aug 2016 15:41:29 +0200 Message-Id: <1470750091-16627-19-git-send-email-daniel.vetter@ffwll.ch> X-Mailer: git-send-email 2.8.1 In-Reply-To: <1470750091-16627-1-git-send-email-daniel.vetter@ffwll.ch> References: <1470750091-16627-1-git-send-email-daniel.vetter@ffwll.ch> Cc: Daniel Vetter , Intel Graphics Development Subject: [Intel-gfx] [PATCH 18/20] drm: Remove display_info->min/max_(h|v)max X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP No one looks at it, only i915/gma500 lvds even bother to fill it out. I guess a very old plan was to use this for filtering modes, but that's already done within the edid parser. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/gma500/mdfld_dsi_output.c | 5 ----- drivers/gpu/drm/gma500/psb_intel_lvds.c | 9 --------- drivers/gpu/drm/i915/intel_lvds.c | 11 ----------- include/drm/drm_connector.h | 3 --- 4 files changed, 28 deletions(-) diff --git a/drivers/gpu/drm/gma500/mdfld_dsi_output.c b/drivers/gpu/drm/gma500/mdfld_dsi_output.c index 907cb51795c3..acb3848ef1c9 100644 --- a/drivers/gpu/drm/gma500/mdfld_dsi_output.c +++ b/drivers/gpu/drm/gma500/mdfld_dsi_output.c @@ -335,11 +335,6 @@ static int mdfld_dsi_connector_get_modes(struct drm_connector *connector) struct drm_display_mode *dup_mode = NULL; struct drm_device *dev = connector->dev; - connector->display_info.min_vfreq = 0; - connector->display_info.max_vfreq = 200; - connector->display_info.min_hfreq = 0; - connector->display_info.max_hfreq = 200; - if (fixed_mode) { dev_dbg(dev->dev, "fixed_mode %dx%d\n", fixed_mode->hdisplay, fixed_mode->vdisplay); diff --git a/drivers/gpu/drm/gma500/psb_intel_lvds.c b/drivers/gpu/drm/gma500/psb_intel_lvds.c index e55733ca46d2..fd7c91254841 100644 --- a/drivers/gpu/drm/gma500/psb_intel_lvds.c +++ b/drivers/gpu/drm/gma500/psb_intel_lvds.c @@ -530,15 +530,6 @@ static int psb_intel_lvds_get_modes(struct drm_connector *connector) if (ret) return ret; - /* Didn't get an EDID, so - * Set wide sync ranges so we get all modes - * handed to valid_mode for checking - */ - connector->display_info.min_vfreq = 0; - connector->display_info.max_vfreq = 200; - connector->display_info.min_hfreq = 0; - connector->display_info.max_hfreq = 200; - if (mode_dev->panel_fixed_mode != NULL) { struct drm_display_mode *mode = drm_mode_duplicate(dev, mode_dev->panel_fixed_mode); diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c index 49550470483e..e29f3d12fd1c 100644 --- a/drivers/gpu/drm/i915/intel_lvds.c +++ b/drivers/gpu/drm/i915/intel_lvds.c @@ -1054,17 +1054,6 @@ void intel_lvds_init(struct drm_device *dev) } lvds_connector->base.edid = edid; - if (IS_ERR_OR_NULL(edid)) { - /* Didn't get an EDID, so - * Set wide sync ranges so we get all modes - * handed to valid_mode for checking - */ - connector->display_info.min_vfreq = 0; - connector->display_info.max_vfreq = 200; - connector->display_info.min_hfreq = 0; - connector->display_info.max_hfreq = 200; - } - list_for_each_entry(scan, &connector->probed_modes, head) { if (scan->type & DRM_MODE_TYPE_PREFERRED) { DRM_DEBUG_KMS("using preferred mode from EDID: "); diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h index c9b275cd4085..fa42d2071bfb 100644 --- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -94,9 +94,6 @@ struct drm_display_info { unsigned int width_mm; unsigned int height_mm; - /* Clock limits FIXME: storage format */ - unsigned int min_vfreq, max_vfreq; - unsigned int min_hfreq, max_hfreq; unsigned int pixel_clock; unsigned int bpc;