linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tom Li <tomli@tomli.me>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-kernel@vger.kernel.org,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
	Sudip Mukherjee <sudipm.mukherjee@gmail.com>,
	Teddy Wang <teddy.wang@siliconmotion.com>
Subject: Re: [PATCH 0/8] fbdev: sm712fb: implement 2D acceleration w/ cleanups.
Date: Mon, 04 Feb 2019 22:06:50 +0000	[thread overview]
Message-ID: <20190204220650.GA24673@localhost.localdomain> (raw)
In-Reply-To: <20190204092618.GM3271@phenom.ffwll.local>

> Since you care about this driver, considered converting it to a drm
> display driver? You can still have all the acceleration and stuff, the
> fbdev compat mode in drm is rather flexible.
> -Daniel

Yes, I know fbdev is now in maintenance-only mode, reimplementing it
on top of DRM is on my roadmap, including rewriting a saner version
of the modesetting code. The current version is beyond repair. I'll
start working on it once I managed to purchase the hardware for testing.

But currently my objectively is to have something usable during the
transitional period. The major user of the chipset was Yeeloong 8089D
laptop - a choice for many non-x86 and MIPS hobbyists for experimentation.
Getting the current version of the patch merged would solve the immediate
usability issues, and then trying to get some platform drivers merged.
And after completing the preliminary upstreaming process, since then,
can start working on a DRM version of the driver, and possibly eventually
remove the fbdev version of sm712fb entirely. 

The 2D acceleration implementation is strictly no more than 200 lines,
the vast majority of the code insertions are comments and documentation
changes ranting about the known issues of the driver/hardware, nothing
non-trivial is added and I think the changeset is manageable and would
not be a burden for the fbdev maintainers, and I can grarantee that I will
not add any other new features to this driver.

Cheers,
Tom Li

  reply	other threads:[~2019-02-04 22:06 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-02  6:16 [PATCH 0/8] fbdev: sm712fb: implement 2D acceleration w/ cleanups Yifeng Li
2019-02-02  6:16 ` [PATCH 1/8] fbdev: sm712fb: update copyright headers Yifeng Li
2019-02-02  6:16 ` [PATCH 2/8] fbdev: sm712fb: use type "u8" for 8-bit I/O Yifeng Li
2019-02-02  6:16 ` [PATCH 3/8] fbdev: sm712fb: add 2D-related I/O headers and functions Yifeng Li
2019-02-02  6:16 ` [PATCH 4/8] fbdev: sm712fb: support 2D acceleration on SM712 w/ Little-Endian CPU Yifeng Li
2019-02-02  6:16 ` [PATCH 5/8] fbdev: sm712fb: add 32-bit color modes, drops some other modes Yifeng Li
2019-02-02  6:16 ` [PATCH 6/8] Documentation: fb: sm712fb: add information mainly about 2D Yifeng Li
2019-02-02  6:16 ` [PATCH 7/8] fbdev: sm712fb: Kconfig: add information about docs Yifeng Li
2019-02-02  6:16 ` [PATCH 8/8] MAINTAINERS: sm712fb: list myself as one maintainer Yifeng Li
2019-02-04  9:26 ` [PATCH 0/8] fbdev: sm712fb: implement 2D acceleration w/ cleanups Daniel Vetter
2019-02-04 22:06   ` Tom Li [this message]
2019-02-05  9:02     ` Daniel Vetter

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=20190204220650.GA24673@localhost.localdomain \
    --to=tomli@tomli.me \
    --cc=b.zolnierkie@samsung.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sudipm.mukherjee@gmail.com \
    --cc=teddy.wang@siliconmotion.com \
    /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;
as well as URLs for NNTP newsgroup(s).