linux-kernel - Re: [RFC] drm/fb: Propagate physical display dimensions to fbdev

Open Source and information security mailing list archives

Message-ID: <CAKMK7uGJPuCqQ0BkJki7e9-g71zrvCSEB5oeCo0KGBkRyvJpHw@mail.gmail.com>
Date:   Mon, 31 Jul 2017 09:42:38 +0200
From:   Daniel Vetter <daniel@...ll.ch>
To:     David Lechner <david@...hnology.com>
Cc:     dri-devel <dri-devel@...ts.freedesktop.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [RFC] drm/fb: Propagate physical display dimensions to fbdev

> Sun, Jul 30, 2017 at 11:11 PM, David Lechner <david@...hnology.com> 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@...ts.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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives