dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Archit Taneja <architt@codeaurora.org>
To: dri-devel@lists.freedesktop.org, daniel@ffwll.ch
Subject: [PATCH v3 00/24] rm: fb emulation: Step 1: Create new drm_fb_helper wrapper funcs
Date: Fri, 31 Jul 2015 16:21:38 +0530	[thread overview]
Message-ID: <1438339922-6805-2-git-send-email-architt@codeaurora.org> (raw)
In-Reply-To: <1438339922-6805-1-git-send-email-architt@codeaurora.org>

DRM drivers using drm_fb_helpers still call some fbdev core functions.
This makes the driver depend on CONFIG_FB, resulting in complicated
Kconfig options, and preventing us from creating a top level drm config
option to enable/disable FBDEV emulation.

Create new drm_fb_helper functions that replace these fbdev functions.

In most cases, the new helper funcs simply wrap around the original fbdev
functions. For a few (like framebufer_alloc), we actually do some work
that is currently redundant across multiple drivers.

With these patches, the drivers don't call any fbdev functions directly.
They are now called through functions in drm_fb_helper.c. We will later
create a fbdev emulation config option to stub out the fb helpers.

The only exception is vmwgfx driver. This doesn't use drm_fb_helper. It
creates a fb device how a driver in drivers/video/fbdev would. Maybe this
needs to be converted to use drm_fb_helpers.

For more info, have a look at the threads:
http://lists.freedesktop.org/archives/dri-devel/2015-March/078729.html
http://lists.freedesktop.org/archives/dri-devel/2015-March/078975.html

changes in v3:
- Removed drm_fb_helper for remove_conflicting_framebuffers.
- Fixed kerneldoc header errors
- Some fixes pointed out on the mailing list

changes in v2:
- Incorporated comments
- Misc fixes for build wannings/errors
- resend version has v2 changes in changelog, 'compile tested only' is
  removed.

Archit Taneja (24):
  drm/fb_helper: Add drm_fb_helper functions to manage fb_info creation
  drm/fb_helper: Create a wrapper for unlink_framebuffer
  drm/fb_helper: Create wrappers for fb_sys_read/write funcs
  drm/fb_helper: Create wrappers for blit, copyarea and fillrect funcs
  drm/fb_helper: Create a wrapper for fb_set_suspend
  drm/cirrus: Use new drm_fb_helper functions
  drm/rockchip: Use new drm_fb_helper functions
  drm/armada: Use new drm_fb_helper functions
  drm/ast: Use new drm_fb_helper functions
  drm/omap: Use new drm_fb_helper functions
  drm/tegra: Use new drm_fb_helper functions
  drm/msm: Use new drm_fb_helper functions
  drm/exynos: Use new drm_fb_helper functions
  drm/gma500: Use new drm_fb_helper functions
  drm/mgag200: Use new drm_fb_helper functions
  drm/radeon: Use new drm_fb_helper functions
  drm/qxl: Use new drm_fb_helper functions
  drm/i915: Use new drm_fb_helper functions
  drm/nouveau: Use new drm_fb_helper functions
  drm/udl: Use new drm_fb_helper functions
  drm/bochs: Use new drm_fb_helper functions
  drm/amdgpu: Use new drm_fb_helper functions
  drm/virtio: Use new drm_fb_helper functions
  drm/fb_cma_helper: Use new drm_fb_helper functions

 drivers/gpu/drm/Kconfig                       |   7 +
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c        |  45 ++----
 drivers/gpu/drm/armada/armada_fbdev.c         |  33 ++--
 drivers/gpu/drm/ast/ast_fb.c                  |  48 ++----
 drivers/gpu/drm/bochs/bochs_drv.c             |   4 +-
 drivers/gpu/drm/bochs/bochs_fbdev.c           |  36 ++---
 drivers/gpu/drm/cirrus/cirrus_drv.c           |   4 +-
 drivers/gpu/drm/cirrus/cirrus_fbdev.c         |  41 +----
 drivers/gpu/drm/drm_fb_cma_helper.c           |  45 ++----
 drivers/gpu/drm/drm_fb_helper.c               | 223 ++++++++++++++++++++++++++
 drivers/gpu/drm/exynos/exynos_drm_fbdev.c     |  47 ++----
 drivers/gpu/drm/gma500/accel_2d.c             |   6 +-
 drivers/gpu/drm/gma500/framebuffer.c          |  48 ++----
 drivers/gpu/drm/i915/intel_fbdev.c            |  40 ++---
 drivers/gpu/drm/mgag200/mgag200_fb.c          |  39 +----
 drivers/gpu/drm/msm/msm_fbdev.c               |  34 ++--
 drivers/gpu/drm/nouveau/nouveau_fbcon.c       |  39 ++---
 drivers/gpu/drm/omapdrm/omap_fbdev.c          |  38 ++---
 drivers/gpu/drm/qxl/qxl_fb.c                  |  40 ++---
 drivers/gpu/drm/radeon/radeon_fb.c            |  42 ++---
 drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c |  47 ++----
 drivers/gpu/drm/tegra/fb.c                    |  35 ++--
 drivers/gpu/drm/udl/udl_fb.c                  |  41 ++---
 drivers/gpu/drm/virtio/virtgpu_fb.c           |  32 ++--
 include/drm/drm_fb_helper.h                   |  27 ++++
 25 files changed, 485 insertions(+), 556 deletions(-)

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-07-31 10:52 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1437548401-10125-1-git-send-email-architt@codeaurora.or>
2015-07-22  9:27 ` [PATCH RESEND v2 00/25] drm: fb emulation: Step 1: Create new drm_fb_helper wrapper funcs Archit Taneja
2015-07-22  9:27   ` [PATCH RESEND v2 01/25] drm/fb_helper: Add drm_fb_helper functions to manage fb_info creation Archit Taneja
2015-07-22  9:27   ` [PATCH RESEND v2 02/25] drm/fb_helper: Create a wrapper for unlink_framebuffer Archit Taneja
2015-07-22  9:27   ` [PATCH RESEND v2 03/25] drm/fb_helper: Create wrappers for fb_sys_read/write funcs Archit Taneja
2015-07-22  9:27   ` [PATCH RESEND v2 04/25] drm/fb_helper: Create wrappers for blit, copyarea and fillrect funcs Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 05/25] drm/fb_helper: Create a wrapper for fb_set_suspend Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 06/25] drm/fb_helper: Create a wrapper for remove_conflicting_framebuffers Archit Taneja
2015-07-28 18:46     ` Daniel Vetter
2015-07-28 18:49       ` Daniel Vetter
2015-07-29  4:20         ` Archit Taneja
2015-07-29  6:30           ` Daniel Vetter
2015-07-29  4:19       ` Archit Taneja
2015-07-29  6:30         ` Daniel Vetter
2015-07-22  9:28   ` [PATCH RESEND v2 07/25] drm/cirrus: Use new drm_fb_helper functions Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 08/25] drm/rockchip: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 09/25] drm/armada: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 10/25] drm/ast: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 11/25] drm/omap: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 12/25] drm/tegra: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 13/25] drm/msm: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 14/25] drm/exynos: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 15/25] drm/gma500: " Archit Taneja
2015-07-22 20:59     ` Patrik Jakobsson
2015-07-22  9:28   ` [PATCH RESEND v2 16/25] drm/mgag200: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 17/25] drm/radeon: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 18/25] drm/qxl: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 19/25] drm/i915: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 20/25] drm/nouveau: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 21/25] drm/udl: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 22/25] drm/bochs: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 23/25] drm/amdgpu: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 24/25] drm/virtio: " Archit Taneja
2015-07-22  9:28   ` [PATCH RESEND v2 25/25] drm/fb_cma_helper: " Archit Taneja
2015-07-31 10:51   ` [PATCH v3 00/24] drm: fb emulation: Step 1: Create new drm_fb_helper wrapper funcs Archit Taneja
2015-07-31 10:51     ` Archit Taneja [this message]
2015-07-31 10:51     ` [PATCH v3 01/24] drm/fb_helper: Add drm_fb_helper functions to manage fb_info creation Archit Taneja
2015-07-31 10:51     ` [PATCH v3 02/24] drm/fb_helper: Create a wrapper for unlink_framebuffer Archit Taneja
2015-07-31 10:51     ` [PATCH v3 03/24] drm/fb_helper: Create wrappers for fb_sys_read/write funcs Archit Taneja
2015-07-31 10:51     ` [PATCH v3 04/24] drm/fb_helper: Create wrappers for blit, copyarea and fillrect funcs Archit Taneja
2015-07-31 10:51     ` [PATCH v3 05/24] drm/fb_helper: Create a wrapper for fb_set_suspend Archit Taneja
2015-07-31 10:51     ` [PATCH v3 06/24] drm/cirrus: Use new drm_fb_helper functions Archit Taneja
2015-07-31 10:51     ` [PATCH v3 07/24] drm/rockchip: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 08/24] drm/armada: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 09/24] drm/ast: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 10/24] drm/omap: " Archit Taneja
2015-08-01  9:55       ` Laurent Pinchart
2015-08-20 10:38       ` Tomi Valkeinen
2015-07-31 10:51     ` [PATCH v3 11/24] drm/tegra: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 12/24] drm/msm: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 13/24] drm/exynos: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 14/24] drm/gma500: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 15/24] drm/mgag200: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 16/24] drm/radeon: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 17/24] drm/qxl: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 18/24] drm/i915: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 19/24] drm/nouveau: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 20/24] drm/udl: " Archit Taneja
2015-07-31 10:51     ` [PATCH v3 21/24] drm/bochs: " Archit Taneja
2015-07-31 10:52     ` [PATCH v3 22/24] drm/amdgpu: " Archit Taneja
2015-07-31 10:52     ` [PATCH v3 23/24] drm/virtio: " Archit Taneja
2015-07-31 10:52     ` [PATCH v3 24/24] drm/fb_cma_helper: " Archit Taneja

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=1438339922-6805-2-git-send-email-architt@codeaurora.org \
    --to=architt@codeaurora.org \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox