From: Tomi Valkeinen <tomi.valkeinen@nokia.com>
To: "ext Måns Rullgård" <mans@mansr.com>
Cc: linux-omap@vger.kernel.org, video4linux-list@redhat.com
Subject: Re: [PREVIEW] New display subsystem for OMAP2/3
Date: Mon, 15 Sep 2008 15:05:34 +0300 [thread overview]
Message-ID: <1221480334.6312.54.camel@tubuntu> (raw)
In-Reply-To: <yw1xbpyr3egz.fsf@thrashbarg.mansr.com>
On Sat, 2008-09-13 at 22:47 +0100, ext Måns Rullgård wrote:
> Koen Kooi <k.kooi@student.utwente.nl> writes:
>
> > Op 12 sep 2008, om 17:29 heeft Daniel Stone het volgende geschreven:
> >
> >> On Fri, Sep 12, 2008 at 07:59:44PM +0530, ext Shah, Hardik wrote:
> >>> It's time to re-design DSS frame buffer driver for the OMAP2/3.
> >>> Current frame buffer driver is not covering the most of the
> >>> functionality of the OMAP2/3 DSS Hardware like multiple outputs and
> >>> multiple overlay managers supported by OMAP2/3 class of SoC. Again
> >>> there is no V4L2 interface exposed by the DSS drivers for
> >>> controlling the video pipelines of the DSS which is highly
> >>> desirable feature as the video pipelines of the DSS hardware is a
> >>> natural fit to the V4L2 architecture.
> >>
> >> If you want to use v4l for video output, don't let me stop you, but I
> >> don't see that it has much actual wide use beyond TI PowerPoint
> >> presentations about their graphical architecture.
> >
> > That was my thought as well, but I've encountered at least 2 products
> > this weekend at IBC using the v4l way on omap3. One of the engineers
> > was complaining about the lack of synchronous updates if you move
> > various videoplanes around (think resizing video windows) which makes
> > the video picture end up outside your nice cairo-drawn borders. So
> > yes, it is getting used outside of TI :)
>
> I'm thinking the best solution is probably to have a low-level
> internal driver providing access to various planes, exposing as much
> functionality as is practical. Various user-space interfaces, such as
> fbdev and v4l, could then be implemented on top of this with very
> little code. If I've understood things correctly, this is essentially
> what the patch in this thread is doing. This approach should let the
> TI people and Koen's mythical friends from IBC use the v4l2 interface,
> while still allowing the less masochistic among us to use a simpler
> interface.
Yes, that was my idea while implementing the driver.
> What I don't like about the patch posted is its size. I'm sure the
> transition could be done in a sequence of smaller patches. At the
> very least, it should be possible to move existing functionality to
> the new architecture, then add the new parts afterwards. I also see
> little value in keeping the old model around, as is done in the patch.
I don't like the size either. However, I have no idea how the old driver
could be transformed to include this functionality with a reasonable
effort. The implementations are quite different.
Any suggestions how I could approach this task? Only thing that comes to
my mind is that there are very similar low level functions in both DSS1
and DSS2 (for dispc and rfbi), that I could remove from the old place
and move to arch/arm/plat-omap/dss/, but that doesn't take us very far.
I wanted to keep the old driver because it works and contains drivers
for many displays. At some point my driver could of course include
these, but it may take time. Also, the old driver supports OMAP1, mine
doesn't.
Tomi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2008-09-15 12:07 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-09-11 14:55 [PREVIEW] New display subsystem for OMAP2/3 Tomi Valkeinen
2008-09-12 14:29 ` Shah, Hardik
2008-09-12 15:29 ` Daniel Stone
2008-09-13 20:27 ` Koen Kooi
2008-09-13 21:47 ` Måns Rullgård
2008-09-15 8:40 ` Shah, Hardik
2008-09-15 12:05 ` Tomi Valkeinen [this message]
2008-09-15 19:27 ` Måns Rullgård
2008-09-17 7:32 ` Tomi Valkeinen
2008-09-17 8:47 ` Måns Rullgård
2008-09-15 11:07 ` Tomi Valkeinen
2008-09-15 11:52 ` Shah, Hardik
2008-09-15 12:36 ` Tomi Valkeinen
2008-09-15 12:04 ` Daniel Stone
2008-09-15 13:32 ` Hiremath, Vaibhav
2008-09-15 14:38 ` Tomi Valkeinen
2008-09-15 16:20 ` Hiremath, Vaibhav
-- strict thread matches above, loose matches on Subject: below --
2008-10-01 10:51 Hiremath, Vaibhav
2008-10-02 8:24 ` Tomi Valkeinen
2008-10-03 12:47 ` Hiremath, Vaibhav
2008-10-03 13:25 ` Tomi Valkeinen
2008-10-03 13:34 ` Tony Lindgren
2008-10-03 14:16 ` Hiremath, Vaibhav
2008-10-24 9:50 ` Shah, Hardik
2008-10-03 14:03 ` Hans Verkuil
2008-10-03 14:19 ` Tomi Valkeinen
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=1221480334.6312.54.camel@tubuntu \
--to=tomi.valkeinen@nokia.com \
--cc=linux-omap@vger.kernel.org \
--cc=mans@mansr.com \
--cc=video4linux-list@redhat.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