From: Sam Ravnborg <sam@ravnborg.org>
To: Thomas Zimmermann <tzimmermann@suse.de>
Cc: airlied@linux.ie, hdegoede@redhat.com, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 0/3] drm/vboxvideo: Use generic fbdev and framebuffer
Date: Sat, 12 Oct 2019 14:36:37 +0200 [thread overview]
Message-ID: <20191012123636.GA25302@ravnborg.org> (raw)
In-Reply-To: <20191011134808.3955-1-tzimmermann@suse.de>
Hi Thomas.
On Fri, Oct 11, 2019 at 03:48:05PM +0200, Thomas Zimmermann wrote:
> The vboxvideo driver provides its own implementation for fbdev emulation
> and framebuffers. Both can be replaced by DRM's generic code.
>
> All patches have been tested on VirtualBox 6.0.12.
>
> Thomas Zimmermann (3):
> drm/vboxvideo: Switch to generic fbdev emulation
> drm/vboxvideo: Switch to drm_atomic_helper_dirty_fb()
> drm/vboxvideo: Replace struct vram_framebuffer with generic
> implemenation
>
> drivers/gpu/drm/vboxvideo/Makefile | 2 +-
> drivers/gpu/drm/vboxvideo/vbox_drv.c | 14 +--
> drivers/gpu/drm/vboxvideo/vbox_drv.h | 25 -----
> drivers/gpu/drm/vboxvideo/vbox_fb.c | 149 --------------------------
> drivers/gpu/drm/vboxvideo/vbox_main.c | 119 +-------------------
> drivers/gpu/drm/vboxvideo/vbox_mode.c | 86 +++++++--------
> 6 files changed, 49 insertions(+), 346 deletions(-)
> delete mode 100644 drivers/gpu/drm/vboxvideo/vbox_fb.c
Nice work. Browsed through the changes and enjoyed all the deleted code.
You can add my:
Acked-by: Sam Ravnborg <sam@ravnborg.org>
to the whole series.
Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2019-10-12 12:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-11 13:48 [PATCH 0/3] drm/vboxvideo: Use generic fbdev and framebuffer Thomas Zimmermann
2019-10-11 13:48 ` [PATCH 1/3] drm/vboxvideo: Switch to generic fbdev emulation Thomas Zimmermann
2019-10-11 13:48 ` [PATCH 2/3] drm/vboxvideo: Switch to drm_atomic_helper_dirty_fb() Thomas Zimmermann
2019-10-11 13:48 ` [PATCH 3/3] drm/vboxvideo: Replace struct vram_framebuffer with generic implemenation Thomas Zimmermann
2019-10-12 12:36 ` Sam Ravnborg [this message]
2019-10-14 14:35 ` [PATCH 0/3] drm/vboxvideo: Use generic fbdev and framebuffer Thomas Zimmermann
2019-10-28 11:26 ` Hans de Goede
2019-10-28 11:34 ` Thomas Zimmermann
2019-10-28 12:06 ` Hans de Goede
2019-10-28 12:24 ` Thomas Zimmermann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191012123636.GA25302@ravnborg.org \
--to=sam@ravnborg.org \
--cc=airlied@linux.ie \
--cc=dri-devel@lists.freedesktop.org \
--cc=hdegoede@redhat.com \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.