linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@nokia.com>
To: "ext Aguirre, Sergio" <saaguirre@ti.com>
Cc: Koen Kooi <koen@dominion.thruhere.net>,
	"Guruswamy, Senthilvadivu" <svadivu@ti.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"linux-fbdev-devel@lists.sourceforge.net"
	<linux-fbdev-devel@lists.sourceforge.net>,
	"tony@atomide.com" <tony@atomide.com>,
	"Hiremath, Vaibhav" <hvaibhav@ti.com>
Subject: RE: [PATCH] DSS2 Include VRFB into omap2-3build only
Date: Mon, 21 Jun 2010 11:38:26 +0300	[thread overview]
Message-ID: <1277109506.2526.19.camel@tubuntu.research.nokia.com> (raw)
In-Reply-To: <A24693684029E5489D1D202277BE894456043018@dlee02.ent.ti.com>

On Fri, 2010-06-18 at 18:57 +0200, ext Aguirre, Sergio wrote:
> Senthil,
> 
> > -----Original Message-----
> > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> > owner@vger.kernel.org] On Behalf Of Koen Kooi
> > Sent: Monday, May 10, 2010 3:03 AM
> > To: Guruswamy, Senthilvadivu
> > Cc: linux-omap@vger.kernel.org; linux-fbdev-devel@lists.sourceforge.net;
> > tony@atomide.com; tomi.valkeinen@nokia.com; Hiremath, Vaibhav
> > Subject: Re: [PATCH] DSS2 Include VRFB into omap2-3build only
> > 
> > 
> > Op 10 mei 2010, om 10:01 heeft Guruswamy, Senthilvadivu het volgende
> > geschreven:
> > 
> > > Building a multi-omap kernel is not impacted as long as the display is
> > not choosen in the build.  Usually display is chosen from the board file.
> 
> Solving this problem involves doing these selections in run time, NOT in build time.
> 
> One of the solutions for this is to provide all this selections as part
> of some platform driver data.
> 
> It _should_ be possible to build DSS2, VRFB AND Tiler code at the same time
> for a multi-omap build, and be executed selectively, depending on your
> silicon detection and board code execution.
> 
> If you need to deselect it to succeed, then it's broken for multi-omap
> builds. That's the core problem and it hasn't been addressed until date.

I think the v3 patch set does it correctly. It allows VRFB to be built
only if ARCH_OMAP2 || ARCH_OMAP3, but does not turn VRFB off on
ARCH_OMAP4.

 Tomi



  reply	other threads:[~2010-06-21  8:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-10  6:48 [PATCH] DSS2 Include VRFB into omap2-3build only Senthilvadivu Guruswamy
2010-05-10  7:39 ` Koen Kooi
2010-05-10  8:01   ` Guruswamy, Senthilvadivu
2010-05-10  8:03     ` Koen Kooi
2010-05-10  8:38       ` Guruswamy, Senthilvadivu
2010-05-10 11:24         ` Nishanth Menon
2010-05-10 13:12           ` Nishanth Menon
2010-05-10 17:14             ` Tony Lindgren
2010-06-18 16:57       ` Aguirre, Sergio
2010-06-21  8:38         ` Tomi Valkeinen [this message]
2010-06-21  8:50           ` Guruswamy, Senthilvadivu

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=1277109506.2526.19.camel@tubuntu.research.nokia.com \
    --to=tomi.valkeinen@nokia.com \
    --cc=hvaibhav@ti.com \
    --cc=koen@dominion.thruhere.net \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    --cc=linux-omap@vger.kernel.org \
    --cc=saaguirre@ti.com \
    --cc=svadivu@ti.com \
    --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;
as well as URLs for NNTP newsgroup(s).