public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@nokia.com>
To: ext Paul Walmsley <paul@pwsan.com>
Cc: ext Sergey Lapin <slapinid@gmail.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: dadd2bb931a08a4b6b17f9e82d9bbe7bedebbc98 breaks omapfb (old non-dss2)
Date: Thu, 07 Jan 2010 13:11:13 +0200	[thread overview]
Message-ID: <1262862673.2127.8.camel@tubuntu.research.nokia.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1001051030580.2083@utopia.booyaka.com>

On Tue, 2010-01-05 at 18:32 +0100, ext Paul Walmsley wrote:
> Hi Tomi,
> 
> On Tue, 15 Dec 2009, Tomi Valkeinen wrote:
> 
> > On Tue, 2009-12-15 at 10:58 +0100, ext Sergey Lapin wrote:
> > > dadd2bb931a08a4b6b17f9e82d9bbe7bedebbc98
> > > OMAP: OMAPFB: add omapdss device
> > > 
> > >     The upcoming new display subsystem driver is divided to two devices,
> > >     omapdss and omapfb, of which omapdss handles the actual hardware.
> > > 
> > >     This patch adds a dummy omapdss platform device for the current omapfb
> > >     driver, which is then used to get the clocks. This will make it possible
> > >     for the current and the new display drivers to co-exist.
> > > 
> > >     Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
> > >     Acked-by: Tony Lindgren <tony@atomide.com>
> > > 
> > > breaks old omapfb.
> > 
> > I didn't look at this further, but I quickly tested with OMAP3 SDP
> > board, reverting the patch that makes SDP use DSS2, and it seems to work
> > fine with the old omapfb.
> > 
> > Perhaps you have more kernel debug options enabled, and then it
> > complains about missing release function. I'll look at it tomorrow.
> 
> This also affects omap_2430sdp_defconfig on 2430SDP:

I pushed patches fixing thisto master branch in my tree:
http://gitorious.org/linux-omap-dss2/linux

Can you try them out on 2320sdp? The commits are
3a2cbca3a3703ecd77c6fe633543ea0926b95564 and
c47f085ecf6d024c8850177694c702d814563580

 Tomi



      reply	other threads:[~2010-01-07 11:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-15  9:58 dadd2bb931a08a4b6b17f9e82d9bbe7bedebbc98 breaks omapfb (old non-dss2) Sergey Lapin
2009-12-15 15:22 ` Tomi Valkeinen
2009-12-15 16:01   ` Sergey Lapin
2009-12-16 11:21     ` Tomi Valkeinen
2009-12-17 14:51       ` Sergey Lapin
2010-01-07 11:04         ` Tomi Valkeinen
2010-01-07 11:46           ` Tomi Valkeinen
2009-12-15 19:50   ` Sergey Lapin
2010-01-05 17:32   ` Paul Walmsley
2010-01-07 11:11     ` Tomi Valkeinen [this message]

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=1262862673.2127.8.camel@tubuntu.research.nokia.com \
    --to=tomi.valkeinen@nokia.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=paul@pwsan.com \
    --cc=slapinid@gmail.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