All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: dri-devel@lists.freedesktop.org
Cc: Kevin Brace <kevinbrace@bracecomputerlab.com>,
	Sam Ravnborg <sam@ravnborg.org>,
	Javier Martinez Canillas <javierm@redhat.com>,
	Thomas Zimmermann <tzimmermann@suse.de>
Subject: [PATCH v2 0/11] drm/via: Make via a single file driver
Date: Wed, 13 Jul 2022 19:01:49 +0200	[thread overview]
Message-ID: <20220713170202.1798216-1-sam@ravnborg.org> (raw)

We have an upcoming openchrome driver for VIA where some
of the files conflicts with the existing via driver.

It is not acceptable to just delete the existing DRI1
based driver as there are most likely users out there,
so a different approach was required.

It was disccussed what to do and the least invasive
solution was to keep the DRI1 driver in the current
directory as a single file.

Thomas Zimmermann already posted a patch to do the
same but it attemped to have a single driver
for the DRI1 and the upcoming new driver.

This patchset embeds the files one by one to make the
diffs remotely readable and end up with an independent
DRI1 driver.

The driver was built tested for each step.

v2:
  - Drop the rename of the driver - keep the name via.
    We can name the new driver viakms or openchrome so
    there is no confusion in userspace if the old or
    the new driver is used.
  - Add a few patches to make via_3d_reg more readable,
    which has the nice side-effect that it is now checkpatch
    clean.
  - Added Kevin as cc: on all patches

This set of patches should make it simpler to add the new
openchrome driver, and I am happy to assist if there are
open questions how to do it.

Note: The patches has seen zero run-time testing - I only
know they builds in my setup (for several archs).

	Sam

Sam Ravnborg (13):
      drm/via: Rename via_drv to via_dri1
      drm/via: Embed via_dma in via_dri1
      drm/via: Embed via_map in via_dri1
      drm/via: Embed via_mm in via_dri1
      drm/via: Embed via_video in via_dri1
      drm/via: Embed via_irq in via_dri1
      drm/via: Embed via_dmablit in via_dri1
      drm/via: Embed via_verifier in via_dri1
      drm/via: Embed via_drv.h in via_dri1
      drm/via: Update to the latest via_3d_reg header
      drm/via: Use SPDX tag for MIT license in via_3d_reg header
      drm/via: Make macros readable in the via_3d_reg header
      drm/via: Fix style issues in via_3d_reg header

 drivers/gpu/drm/via/Makefile       |    2 +-
 drivers/gpu/drm/via/via_3d_reg.h   |  349 ++--
 drivers/gpu/drm/via/via_dma.c      |  744 --------
 drivers/gpu/drm/via/via_dmablit.c  |  807 --------
 drivers/gpu/drm/via/via_dmablit.h  |  140 --
 drivers/gpu/drm/via/via_dri1.c     | 3630 ++++++++++++++++++++++++++++++++++++
 drivers/gpu/drm/via/via_drv.c      |  124 --
 drivers/gpu/drm/via/via_drv.h      |  229 ---
 drivers/gpu/drm/via/via_irq.c      |  388 ----
 drivers/gpu/drm/via/via_map.c      |  132 --
 drivers/gpu/drm/via/via_mm.c       |  241 ---
 drivers/gpu/drm/via/via_verifier.c | 1110 -----------
 drivers/gpu/drm/via/via_verifier.h |   62 -
 drivers/gpu/drm/via/via_video.c    |   94 -
 14 files changed, 3866 insertions(+), 4186 deletions(-)



             reply	other threads:[~2022-07-13 17:02 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-13 17:01 Sam Ravnborg [this message]
2022-07-13 17:01 ` [PATCH v2 01/13] drm/via: Rename via_drv to via_dri1 Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 02/13] drm/via: Embed via_dma in via_dri1 Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 03/13] drm/via: Embed via_map " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 04/13] drm/via: Embed via_mm " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 05/13] drm/via: Embed via_video " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 06/13] drm/via: Embed via_irq " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 07/13] drm/via: Embed via_dmablit " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 08/13] drm/via: Embed via_verifier " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 09/13] drm/via: Embed via_drv.h " Sam Ravnborg
2022-07-13 17:01 ` [PATCH v2 10/13] drm/via: Update to the latest via_3d_reg header Sam Ravnborg
2022-07-13 17:02 ` [PATCH v2 11/13] drm/via: Use SPDX tag for MIT license in " Sam Ravnborg
2022-07-13 17:02 ` [PATCH v2 12/13] drm/via: Make macros readable in the " Sam Ravnborg
2022-07-13 17:02 ` [PATCH v2 13/13] drm/via: Fix style issues in " Sam Ravnborg
2022-07-14  8:15 ` [PATCH v2 0/11] drm/via: Make via a single file driver 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=20220713170202.1798216-1-sam@ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=javierm@redhat.com \
    --cc=kevinbrace@bracecomputerlab.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.