From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: "Michael Büsch" <m@bues.ch>, lo-ml <linux-omap@vger.kernel.org>
Cc: Tony Lindgren <tony@atomide.com>
Subject: N8x0 display support (was: Re: The old omapfb support)
Date: Fri, 29 Apr 2011 16:33:39 +0300 [thread overview]
Message-ID: <1304084019.5605.75.camel@deskari> (raw)
In-Reply-To: <1303217114.27432.9.camel@maggie>
On Tue, 2011-04-19 at 14:45 +0200, Michael Büsch wrote:
> On Tue, 2011-04-19 at 15:41 +0300, Tomi Valkeinen wrote:
> > On Tue, 2011-04-19 at 14:34 +0200, Michael Büsch wrote:
> > > On Tue, 2011-04-19 at 15:30 +0300, Tony Lindgren wrote:
> > > > > But this again reminded me of the mess of having two display drivers,
> > > > > the old omapfb and the new DSS2. Many of the OMAP2 boards using the old
> > > > > driver should be quite easy to port to DSS2, with the exception of N800.
> > > > > DSS2 doesn't support OMAP1, so there's not much that can be done with
> > > > > those boards currently.
> > >
> > > > Yeh we can just make old omapfb depends on ARCH_OMAP1.
> > >
> > > As he said, the old omapfb code is used on n8x0, which is OMAP2.
> >
> > But as I also said, the old OMAP2 panel drivers can be ported to the new
> > DSS driver. I just mentioned N800 because the panel driver for N800 is
> > rather difficult to port and will require more work than the other OMAP2
> > panel drivers.
>
> So if you first port the stuff and then add the depends-on OMAP1, I'm
> fine with it.
I've now got a beta version of N800 display support for DSS2 ready, and
I can get a picture on the screen.
However, it's quite difficult to port all of the blizzard.c code, as the
new DSS2 doesn't support some of the weird things there. Also, I don't
have N800 schematics and Blizzard documentation, so properly porting
everything needs reverse engineering the code.
There is some PLL code in the blizzard.c, purpose of which I don't
understand. The new driver seems to work fine without the PLL code.
Then blizzard seems to support YUV420 mode, and currently there's no
support for controller specific modes in the DSS2, so I've left that
out.
Blizzard driver also contains auto-update code, which adds a
considerable amount of code to it. I don't want to add that code to the
new driver, because my opinion is that manual update displays should be
used like manual update displays. N800's user space should handle it
properly, as far as I know, but it is not possible to use the
framebuffer console with only manual update mode.
I experimented a bit with FB framework's deferred IO support, which
indeed allows us to use fb console quite easily. However, a proper
support for deferred IO would need more code and careful thinking how to
do it.
So, my question is: what are the uses of N800's display with the
mainline kernel? Are we happy with just having basic display
functionality, or do people need all the features in the older blizzard
driver? If latter, I wonder if there's somebody who wants to develop
this further?
If somebody wants to give it a try/look, my N800 development (based
on .39-rc3) branch can be found from:
git://gitorious.org/linux-omap-dss2/linux.git n800
I'll of course send proper patches when thing are more finalized.
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:[~2011-04-29 13:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-19 12:20 The old omapfb support Tomi Valkeinen
2011-04-19 12:30 ` Tony Lindgren
2011-04-19 12:34 ` Michael Büsch
2011-04-19 12:41 ` Tomi Valkeinen
2011-04-19 12:45 ` Michael Büsch
2011-04-20 8:06 ` Tomi Valkeinen
2011-04-20 11:18 ` Michael Büsch
2011-04-20 11:31 ` Tomi Valkeinen
2011-04-20 11:35 ` Michael Büsch
2011-04-29 13:33 ` Tomi Valkeinen [this message]
2011-04-29 14:08 ` N8x0 display support (was: Re: The old omapfb support) Jarkko Nikula
2011-04-29 21:04 ` Michael Büsch
2011-04-29 21:05 ` Michael Büsch
2011-04-30 6:01 ` 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=1304084019.5605.75.camel@deskari \
--to=tomi.valkeinen@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=m@bues.ch \
--cc=tony@atomide.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