All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Daniel Vetter <daniel@ffwll.ch>,
	Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	Linux Fbdev development list <linux-fbdev@vger.kernel.org>,
	Teddy Wang <teddy.wang@siliconmotion.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Sudip Mukherjee <sudipm.mukherjee@gmail.com>,
	Arnaud Patard <arnaud.patard@rtp-net.org>
Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers
Date: Thu, 08 Dec 2016 21:34:37 +0000	[thread overview]
Message-ID: <1481232877.26959.52.camel@kernel.crashing.org> (raw)
In-Reply-To: <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local>

On Thu, 2016-12-08 at 16:21 +0100, Daniel Vetter wrote:
> Yeah, small drivers like these we have piles now, things exploded a lot
> after atomic landed two years ago. And they seem to shrink with every
> release a bit more (since lots more drivers gives you lots more insight
> into what other refactorings would make sense). Those we have a big pile
> of, and nowadays (at least with developers expirienced with upstream, but
> not necessarily with drm) it takes but a few weeks from initial submission
> to getting them merged.
> 
> What we don't yet have a nice tidy example driver of is the even simpler
> "dumb framebuffer behind a slow bus with explicit/manual upload", for like
> small i2c/spi panels (and conceptually also usb, even though there bw and
> panel size are a bit scaled up). We've gained some really nice helpers for
> this this year, and there's 3 drivers in-flight to make use of it. But
> since that's right now just a hobbyist effort it's moving a bit slower
> (and I was mistaken a few weeks back where I assumed that one of them
> landed already).

What I find usually confusing is the interaction with the TTM and
overall fb memory management, when trying to plumb in simple 2d accel
to speed up fbcon mostly (but I don't mind making it available to user
space via ioctls, though that's not a priority).

As I mentioned earlier, probably 1 or 2 years ago, Dave made the
argument that shadowing through memory was necessary and precluded 2D
accel, though I don't fully remember the root of the argument. If that
is indeed not the case, then my main objection is lifted.

Cheers,
Ben.


WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Daniel Vetter <daniel@ffwll.ch>,
	Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	Linux Fbdev development list <linux-fbdev@vger.kernel.org>,
	Teddy Wang <teddy.wang@siliconmotion.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	DRI Development <dri-devel@lists.freedesktop.org>,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Sudip Mukherjee <sudipm.mukherjee@gmail.com>,
	Arnaud Patard <arnaud.patard@rtp-net.org>
Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers
Date: Fri, 09 Dec 2016 08:34:37 +1100	[thread overview]
Message-ID: <1481232877.26959.52.camel@kernel.crashing.org> (raw)
In-Reply-To: <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local>

On Thu, 2016-12-08 at 16:21 +0100, Daniel Vetter wrote:
> Yeah, small drivers like these we have piles now, things exploded a lot
> after atomic landed two years ago. And they seem to shrink with every
> release a bit more (since lots more drivers gives you lots more insight
> into what other refactorings would make sense). Those we have a big pile
> of, and nowadays (at least with developers expirienced with upstream, but
> not necessarily with drm) it takes but a few weeks from initial submission
> to getting them merged.
> 
> What we don't yet have a nice tidy example driver of is the even simpler
> "dumb framebuffer behind a slow bus with explicit/manual upload", for like
> small i2c/spi panels (and conceptually also usb, even though there bw and
> panel size are a bit scaled up). We've gained some really nice helpers for
> this this year, and there's 3 drivers in-flight to make use of it. But
> since that's right now just a hobbyist effort it's moving a bit slower
> (and I was mistaken a few weeks back where I assumed that one of them
> landed already).

What I find usually confusing is the interaction with the TTM and
overall fb memory management, when trying to plumb in simple 2d accel
to speed up fbcon mostly (but I don't mind making it available to user
space via ioctls, though that's not a priority).

As I mentioned earlier, probably 1 or 2 years ago, Dave made the
argument that shadowing through memory was necessary and precluded 2D
accel, though I don't fully remember the root of the argument. If that
is indeed not the case, then my main objection is lifted.

Cheers,
Ben.

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

WARNING: multiple messages have this Message-ID (diff)
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Daniel Vetter <daniel@ffwll.ch>,
	Geert Uytterhoeven <geert@linux-m68k.org>
Cc: "Thomas Petazzoni" <thomas.petazzoni@free-electrons.com>,
	"Tomi Valkeinen" <tomi.valkeinen@ti.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Noralf Trønnes" <noralf@tronnes.org>,
	"Sudip Mukherjee" <sudipm.mukherjee@gmail.com>,
	"Teddy Wang" <teddy.wang@siliconmotion.com>,
	"Arnaud Patard" <arnaud.patard@rtp-net.org>,
	"DRI Development" <dri-devel@lists.freedesktop.org>,
	"Linux Fbdev development list" <linux-fbdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 0/3] staging: remove fbdev drivers
Date: Fri, 09 Dec 2016 08:34:37 +1100	[thread overview]
Message-ID: <1481232877.26959.52.camel@kernel.crashing.org> (raw)
In-Reply-To: <20161208152134.wnv4j4i6m5xpoycp@phenom.ffwll.local>

On Thu, 2016-12-08 at 16:21 +0100, Daniel Vetter wrote:
> Yeah, small drivers like these we have piles now, things exploded a lot
> after atomic landed two years ago. And they seem to shrink with every
> release a bit more (since lots more drivers gives you lots more insight
> into what other refactorings would make sense). Those we have a big pile
> of, and nowadays (at least with developers expirienced with upstream, but
> not necessarily with drm) it takes but a few weeks from initial submission
> to getting them merged.
> 
> What we don't yet have a nice tidy example driver of is the even simpler
> "dumb framebuffer behind a slow bus with explicit/manual upload", for like
> small i2c/spi panels (and conceptually also usb, even though there bw and
> panel size are a bit scaled up). We've gained some really nice helpers for
> this this year, and there's 3 drivers in-flight to make use of it. But
> since that's right now just a hobbyist effort it's moving a bit slower
> (and I was mistaken a few weeks back where I assumed that one of them
> landed already).

What I find usually confusing is the interaction with the TTM and
overall fb memory management, when trying to plumb in simple 2d accel
to speed up fbcon mostly (but I don't mind making it available to user
space via ioctls, though that's not a priority).

As I mentioned earlier, probably 1 or 2 years ago, Dave made the
argument that shadowing through memory was necessary and precluded 2D
accel, though I don't fully remember the root of the argument. If that
is indeed not the case, then my main objection is lifted.

Cheers,
Ben.

  reply	other threads:[~2016-12-08 21:34 UTC|newest]

Thread overview: 154+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-23  8:03 [RFC PATCH 0/3] staging: remove fbdev drivers Tomi Valkeinen
2016-11-23  8:03 ` Tomi Valkeinen
2016-11-23  8:03 ` Tomi Valkeinen
2016-11-23  8:03 ` [RFC PATCH 1/3] staging: remove xgifb Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23  8:03 ` [RFC PATCH 2/3] staging: remove sm750fb Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23  8:03 ` [RFC PATCH 3/3] staging: remove fbtft Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23  8:03   ` Tomi Valkeinen
2016-11-23 17:26   ` Noralf Trønnes
2016-11-23 17:26     ` Noralf Trønnes
2016-11-23 17:26     ` Noralf Trønnes
2016-11-24  8:36     ` Tomi Valkeinen
2016-11-24  8:36       ` Tomi Valkeinen
2016-11-24  8:36       ` Tomi Valkeinen
2016-11-23 20:12   ` Drew Fustini
2016-11-23 20:12     ` Drew Fustini
2016-11-23  8:19 ` [RFC PATCH 0/3] staging: remove fbdev drivers Daniel Vetter
2016-11-23  8:19   ` Daniel Vetter
2016-11-23  8:19   ` Daniel Vetter
2016-11-23  8:21   ` Tomi Valkeinen
2016-11-23  8:21     ` Tomi Valkeinen
2016-11-23  8:25   ` Geert Uytterhoeven
2016-11-23  8:25     ` Geert Uytterhoeven
2016-11-23  8:25     ` Geert Uytterhoeven
2016-11-23  8:45     ` Daniel Vetter
2016-11-23  8:45       ` Daniel Vetter
2016-11-23  8:45       ` Daniel Vetter
2016-11-23  8:52 ` Greg Kroah-Hartman
2016-11-23  8:52   ` Greg Kroah-Hartman
2016-11-23  8:52   ` Greg Kroah-Hartman
2016-11-23  9:12   ` Tomi Valkeinen
2016-11-23  9:12     ` Tomi Valkeinen
2016-11-23  9:12     ` Tomi Valkeinen
2016-11-23  9:49     ` Greg Kroah-Hartman
2016-11-23  9:49       ` Greg Kroah-Hartman
2016-11-23  9:49       ` Greg Kroah-Hartman
2016-11-23 10:05     ` Thomas Petazzoni
2016-11-23 10:05       ` Thomas Petazzoni
2016-12-22 20:36       ` Andy Shevchenko
2016-12-22 20:36         ` Andy Shevchenko
2016-12-22 20:36         ` Andy Shevchenko
2016-12-08 22:00     ` Sudip Mukherjee
2016-12-08 22:00       ` Sudip Mukherjee
2016-12-08 22:00       ` Sudip Mukherjee
2016-12-08  1:01 ` Benjamin Herrenschmidt
2016-12-08  1:01   ` Benjamin Herrenschmidt
2016-12-08  8:01   ` Tomi Valkeinen
2016-12-08  8:01     ` Tomi Valkeinen
2016-12-08  8:01     ` Tomi Valkeinen
2016-12-08 21:23     ` Benjamin Herrenschmidt
2016-12-08 21:23       ` Benjamin Herrenschmidt
2016-12-08 21:23       ` Benjamin Herrenschmidt
2016-12-08 21:43       ` Benjamin Herrenschmidt
2016-12-08 21:43         ` Benjamin Herrenschmidt
2016-12-08 21:43         ` Benjamin Herrenschmidt
2016-12-09  8:13         ` Daniel Vetter
2016-12-09  8:13           ` Daniel Vetter
2016-12-09  8:13           ` Daniel Vetter
2016-12-13  7:36       ` Gerd Hoffmann
2016-12-13  7:36         ` Gerd Hoffmann
2016-12-08 10:10   ` Daniel Vetter
2016-12-08 10:10     ` Daniel Vetter
2016-12-08 10:10     ` Daniel Vetter
2016-12-08 12:15     ` Geert Uytterhoeven
2016-12-08 12:15       ` Geert Uytterhoeven
2016-12-08 12:15       ` Geert Uytterhoeven
2016-12-08 14:02       ` Daniel Vetter
2016-12-08 14:02         ` Daniel Vetter
2016-12-08 14:02         ` Daniel Vetter
2016-12-08 14:22         ` Geert Uytterhoeven
2016-12-08 14:22           ` Geert Uytterhoeven
2016-12-08 14:37           ` Thomas Petazzoni
2016-12-08 14:37             ` Thomas Petazzoni
2016-12-08 14:44             ` Geert Uytterhoeven
2016-12-08 14:44               ` Geert Uytterhoeven
2016-12-08 14:44               ` Geert Uytterhoeven
2016-12-08 15:21               ` Daniel Vetter
2016-12-08 15:21                 ` Daniel Vetter
2016-12-08 15:21                 ` Daniel Vetter
2016-12-08 21:34                 ` Benjamin Herrenschmidt [this message]
2016-12-08 21:34                   ` Benjamin Herrenschmidt
2016-12-08 21:34                   ` Benjamin Herrenschmidt
2016-12-08 21:57                   ` Benjamin Herrenschmidt
2016-12-08 21:57                     ` Benjamin Herrenschmidt
2016-12-08 21:57                     ` Benjamin Herrenschmidt
2016-12-09  8:34                     ` Daniel Vetter
2016-12-09  8:34                       ` Daniel Vetter
2016-12-09  8:34                       ` Daniel Vetter
2016-12-09  8:41                       ` Daniel Vetter
2016-12-09  8:41                         ` Daniel Vetter
2016-12-09  8:41                         ` Daniel Vetter
2016-12-09 11:48                         ` Benjamin Herrenschmidt
2016-12-09 11:48                           ` Benjamin Herrenschmidt
2016-12-09 11:48                           ` Benjamin Herrenschmidt
2016-12-09 13:35                           ` Daniel Vetter
2016-12-09 13:35                             ` Daniel Vetter
2016-12-09 13:35                             ` Daniel Vetter
2016-12-09 20:27                             ` Benjamin Herrenschmidt
2016-12-09 20:27                               ` Benjamin Herrenschmidt
2016-12-09 20:27                               ` Benjamin Herrenschmidt
2016-12-13  7:18                               ` Michel Dänzer
2016-12-13  7:18                                 ` Michel Dänzer
2016-12-13  7:18                                 ` Michel Dänzer
2016-12-09 11:44                       ` Benjamin Herrenschmidt
2016-12-09 11:44                         ` Benjamin Herrenschmidt
2016-12-09 11:44                         ` Benjamin Herrenschmidt
2016-12-09 12:33                         ` Geert Uytterhoeven
2016-12-09 12:33                           ` Geert Uytterhoeven
2016-12-09 12:33                           ` Geert Uytterhoeven
2016-12-09 13:19                         ` Lucas Stach
2016-12-09 13:19                           ` Lucas Stach
2016-12-09 13:19                           ` Lucas Stach
2016-12-09 13:33                         ` Daniel Vetter
2016-12-09 13:33                           ` Daniel Vetter
2016-12-09 13:33                           ` Daniel Vetter
2016-12-09 13:57                           ` David Herrmann
2016-12-09 13:57                             ` David Herrmann
2016-12-09 13:57                             ` David Herrmann
2016-12-09 14:04                             ` Daniel Vetter
2016-12-09 14:04                               ` Daniel Vetter
2016-12-09 14:04                               ` Daniel Vetter
2016-12-09 20:29                             ` Benjamin Herrenschmidt
2016-12-09 20:29                               ` Benjamin Herrenschmidt
2016-12-09 20:29                               ` Benjamin Herrenschmidt
2016-12-09  8:30                 ` Daniel Vetter
2016-12-09  8:30                   ` Daniel Vetter
2016-12-09  8:30                   ` Daniel Vetter
2016-12-08 14:59           ` Jani Nikula
2016-12-08 14:59             ` Jani Nikula
2016-12-08 14:59             ` Jani Nikula
2016-12-08 14:22         ` Daniel Vetter
2016-12-08 14:22           ` Daniel Vetter
2016-12-08 14:22           ` Daniel Vetter
2016-12-08 21:28     ` Benjamin Herrenschmidt
2016-12-08 21:28       ` Benjamin Herrenschmidt
2016-12-08 21:28       ` Benjamin Herrenschmidt
2016-12-09  0:08       ` Dave Airlie
2016-12-09  0:08         ` Dave Airlie
2016-12-09  0:08         ` Dave Airlie
2016-12-09  8:04         ` Geert Uytterhoeven
2016-12-09  8:04           ` Geert Uytterhoeven
2016-12-09 11:40         ` Benjamin Herrenschmidt
2016-12-09 11:40           ` Benjamin Herrenschmidt
2016-12-09 11:40           ` Benjamin Herrenschmidt
2016-12-13  7:33         ` Gerd Hoffmann
2016-12-13  7:33           ` Gerd Hoffmann
2016-12-13  7:33           ` Gerd Hoffmann
2016-12-13 15:17     ` Laurent Pinchart
2016-12-13 15:17       ` Laurent Pinchart
2016-12-13 15:17       ` Laurent Pinchart

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=1481232877.26959.52.camel@kernel.crashing.org \
    --to=benh@kernel.crashing.org \
    --cc=arnaud.patard@rtp-net.org \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=geert@linux-m68k.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sudipm.mukherjee@gmail.com \
    --cc=teddy.wang@siliconmotion.com \
    --cc=thomas.petazzoni@free-electrons.com \
    --cc=tomi.valkeinen@ti.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 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.