Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751817AbdGaHml (ORCPT ); Mon, 31 Jul 2017 03:42:41 -0400 Received: from mail-io0-f177.google.com ([209.85.223.177]:32894 "EHLO mail-io0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751045AbdGaHmj (ORCPT ); Mon, 31 Jul 2017 03:42:39 -0400 MIME-Version: 1.0 X-Originating-IP: [194.230.159.208] In-Reply-To: <1501449098-26148-1-git-send-email-david@lechnology.com> References: <1501449098-26148-1-git-send-email-david@lechnology.com> From: Daniel Vetter Date: Mon, 31 Jul 2017 09:42:38 +0200 X-Google-Sender-Auth: 0DBSXky3gn9zcHfkIiZd20zqTbo Message-ID: Subject: Re: [RFC] drm/fb: Propagate physical display dimensions to fbdev To: David Lechner Cc: dri-devel , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2060 Lines: 55 > Sun, Jul 30, 2017 at 11:11 PM, David Lechner wrote: > fbdev has a place for the physical width and height of a display. I would > like to have this information available to userspace. This patch works for me, > but I have a strong suspicion that this is not the "right way". > > Any suggestions on how to get the proper struct drm_display_info here > rather than assuming the first connector one is valid? I don't see an obvious > way to do this. This data can also change at runtime, which means we need to reprobe it. And as you mention, we need to pick the data of the first connected connector, not the very first connector the driver has. The final connector loo in drm_setup_crtcs() would be a good place to do that I think. -Daniel > --- > drivers/gpu/drm/drm_fb_helper.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c > index 574af01..ff422da 100644 > --- a/drivers/gpu/drm/drm_fb_helper.c > +++ b/drivers/gpu/drm/drm_fb_helper.c > @@ -1768,8 +1768,14 @@ void drm_fb_helper_fill_var(struct fb_info *info, struct drm_fb_helper *fb_helpe > info->var.xoffset = 0; > info->var.yoffset = 0; > info->var.activate = FB_ACTIVATE_NOW; > - info->var.height = -1; > - info->var.width = -1; > + > + if (fb_helper->connector_count) { > + info->var.height = fb_helper->connector_info[0]->connector->display_info.width_mm; > + info->var.width = fb_helper->connector_info[0]->connector->display_info.height_mm; > + } else { > + info->var.height = -1; > + info->var.width = -1; > + } > > switch (fb->format->depth) { > case 8: > -- > 2.7.4 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch