All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nishanth Menon <nm@ti.com>
To: Nishanth Menon <menon.nishanth@gmail.com>
Cc: "Guruswamy, Senthilvadivu" <svadivu@ti.com>,
	Koen Kooi <koen@dominion.thruhere.net>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"tony@atomide.com" <tony@atomide.com>,
	"tomi.valkeinen@nokia.com" <tomi.valkeinen@nokia.com>,
	"Hiremath, Vaibhav" <hvaibhav@ti.com>,
	"linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>
Subject: Re: [PATCH] DSS2 Include VRFB into omap2-3build only
Date: Mon, 10 May 2010 13:12:27 +0000	[thread overview]
Message-ID: <4BE8063B.1080709@ti.com> (raw)
In-Reply-To: <4BE7ECE4.5070608@gmail.com>

On 05/10/2010 06:24 AM, Nishanth Menon wrote:
> On 05/10/2010 03:26 AM, Guruswamy, Senthilvadivu wrote:
>> Tiler support is not yet in OMAP builds. So multi-omap build will NOT break, but would exclude VRFB feature in the build.
> in which case, is'nt this patch redundant :) ?
>
> joking apart, would'nt a set of function pointers be better - the
> function pointers could be runtime initialized. the rotational functions
> for Tiler OR vrfb could be used?

just an addon -> I can see one more potential user of the FEATURES 
framework like dma descriptor feature.. what kind of rotator do we have? 
vrfb Vs tiler..

>
>>
>>> -----Original Message-----
>>> From: Koen Kooi [mailto:koen@dominion.thruhere.net]
>>> Sent: Monday, May 10, 2010 1:33 PM
>>> 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.
>>>
>>> So you are saying it *IS* broken when wanting vrfb on omap3
>>> and tiler on omap4, right?
>>>
>>>
>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: Koen Kooi [mailto:koen@dominion.thruhere.net]
>>>>> Sent: Monday, May 10, 2010 1:10 PM
>>>>> 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 08:48 heeft Senthilvadivu Guruswamy het
>>>>> volgende geschreven:
>>>>>
>>>>>> Exclude VRFB from OMAP4 onwards and include only for
>>>>>> OMAP2 and OMAP3 builds.  In OMAP4 VRFB HW IP is replaced
>>>>>> with a new HW IP "TILER"
>>>>>>
>>>>>> --- a/drivers/video/omap2/Kconfig
>>>>>> +++ b/drivers/video/omap2/Kconfig
>>>>>> @@ -3,6 +3,10 @@ config OMAP2_VRAM
>>>>>>
>>>>>> config OMAP2_VRFB
>>>>>> 	bool
>>>>>> +	depends on FB_OMAP2&&   (!ARCH_OMAP4)
>>>>>> +	default y if (ARCH_OMAP2 || ARCH_OMAP3)
>>>>>> +	help
>>>>>> +	  OMAP VRFB buffer support is efficient for rotation
>>>>>
>>>>> How does this work for multi-omap kernels, e.g. building a
>>>>> kernel with beagle and panda support?
>>>>>
>>>>> regards,
>>>>>
>>>>> Koen
>>>
>>> --
>> 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
>>
>
> Regards,
> Nishanth Menon
> --
> 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


WARNING: multiple messages have this Message-ID (diff)
From: Nishanth Menon <nm@ti.com>
To: Nishanth Menon <menon.nishanth@gmail.com>
Cc: "Guruswamy, Senthilvadivu" <svadivu@ti.com>,
	Koen Kooi <koen@dominion.thruhere.net>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"tony@atomide.com" <tony@atomide.com>,
	"tomi.valkeinen@nokia.com" <tomi.valkeinen@nokia.com>,
	"Hiremath, Vaibhav" <hvaibhav@ti.com>,
	"linux-fbdev@vger.kernel.org" <linux-fbdev@vger.kernel.org>
Subject: Re: [PATCH] DSS2 Include VRFB into omap2-3build only
Date: Mon, 10 May 2010 08:12:27 -0500	[thread overview]
Message-ID: <4BE8063B.1080709@ti.com> (raw)
In-Reply-To: <4BE7ECE4.5070608@gmail.com>

On 05/10/2010 06:24 AM, Nishanth Menon wrote:
> On 05/10/2010 03:26 AM, Guruswamy, Senthilvadivu wrote:
>> Tiler support is not yet in OMAP builds. So multi-omap build will NOT break, but would exclude VRFB feature in the build.
> in which case, is'nt this patch redundant :) ?
>
> joking apart, would'nt a set of function pointers be better - the
> function pointers could be runtime initialized. the rotational functions
> for Tiler OR vrfb could be used?

just an addon -> I can see one more potential user of the FEATURES 
framework like dma descriptor feature.. what kind of rotator do we have? 
vrfb Vs tiler..

>
>>
>>> -----Original Message-----
>>> From: Koen Kooi [mailto:koen@dominion.thruhere.net]
>>> Sent: Monday, May 10, 2010 1:33 PM
>>> 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.
>>>
>>> So you are saying it *IS* broken when wanting vrfb on omap3
>>> and tiler on omap4, right?
>>>
>>>
>>>
>>>>
>>>>> -----Original Message-----
>>>>> From: Koen Kooi [mailto:koen@dominion.thruhere.net]
>>>>> Sent: Monday, May 10, 2010 1:10 PM
>>>>> 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 08:48 heeft Senthilvadivu Guruswamy het
>>>>> volgende geschreven:
>>>>>
>>>>>> Exclude VRFB from OMAP4 onwards and include only for
>>>>>> OMAP2 and OMAP3 builds.  In OMAP4 VRFB HW IP is replaced
>>>>>> with a new HW IP "TILER"
>>>>>>
>>>>>> --- a/drivers/video/omap2/Kconfig
>>>>>> +++ b/drivers/video/omap2/Kconfig
>>>>>> @@ -3,6 +3,10 @@ config OMAP2_VRAM
>>>>>>
>>>>>> config OMAP2_VRFB
>>>>>> 	bool
>>>>>> +	depends on FB_OMAP2&&   (!ARCH_OMAP4)
>>>>>> +	default y if (ARCH_OMAP2 || ARCH_OMAP3)
>>>>>> +	help
>>>>>> +	  OMAP VRFB buffer support is efficient for rotation
>>>>>
>>>>> How does this work for multi-omap kernels, e.g. building a
>>>>> kernel with beagle and panda support?
>>>>>
>>>>> regards,
>>>>>
>>>>> Koen
>>>
>>> --
>> 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
>>
>
> Regards,
> Nishanth Menon
> --
> 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


  reply	other threads:[~2010-05-10 13:12 UTC|newest]

Thread overview: 15+ 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:26       ` Guruswamy, Senthilvadivu
2010-05-10  8:38         ` Guruswamy, Senthilvadivu
2010-05-10 11:24         ` Nishanth Menon
2010-05-10 11:24           ` Nishanth Menon
2010-05-10 13:12           ` Nishanth Menon [this message]
2010-05-10 13:12             ` Nishanth Menon
2010-05-10 17:14             ` Tony Lindgren
2010-05-10 17:14               ` Tony Lindgren
2010-06-18 16:57       ` Aguirre, Sergio
2010-06-21  8:38         ` Tomi Valkeinen
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=4BE8063B.1080709@ti.com \
    --to=nm@ti.com \
    --cc=hvaibhav@ti.com \
    --cc=koen@dominion.thruhere.net \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=menon.nishanth@gmail.com \
    --cc=svadivu@ti.com \
    --cc=tomi.valkeinen@nokia.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 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.