* [PATCH] da8xx: Allow use by am33xx based devices @ 2012-10-31 15:56 ` Pantelis Antoniou 0 siblings, 0 replies; 12+ messages in thread From: Pantelis Antoniou @ 2012-10-30 18:18 UTC (permalink / raw) To: Florian Tobias Schandinat Cc: Pantelis Antoniou, linux-fbdev, linux-kernel, Koen Kooi, Matt Porter, Russ Dill, linux-omap This driver can be used for AM33xx devices, like the popular beaglebone. Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> --- drivers/video/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig index 9791d10..e7868d8 100644 --- a/drivers/video/Kconfig +++ b/drivers/video/Kconfig @@ -2202,7 +2202,7 @@ config FB_SH7760 config FB_DA8XX tristate "DA8xx/OMAP-L1xx Framebuffer support" - depends on FB && ARCH_DAVINCI_DA8XX + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) select FB_CFB_FILLRECT select FB_CFB_COPYAREA select FB_CFB_IMAGEBLIT -- 1.7.12 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] da8xx: Allow use by am33xx based devices @ 2012-10-31 15:56 ` Pantelis Antoniou 0 siblings, 0 replies; 12+ messages in thread From: Pantelis Antoniou @ 2012-10-31 15:56 UTC (permalink / raw) To: Florian Tobias Schandinat Cc: Pantelis Antoniou, linux-fbdev, linux-kernel, Koen Kooi, Matt Porter, Russ Dill, linux-omap This driver can be used for AM33xx devices, like the popular beaglebone. Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> --- drivers/video/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig index 9791d10..e7868d8 100644 --- a/drivers/video/Kconfig +++ b/drivers/video/Kconfig @@ -2202,7 +2202,7 @@ config FB_SH7760 config FB_DA8XX tristate "DA8xx/OMAP-L1xx Framebuffer support" - depends on FB && ARCH_DAVINCI_DA8XX + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) select FB_CFB_FILLRECT select FB_CFB_COPYAREA select FB_CFB_IMAGEBLIT -- 1.7.12 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices 2012-10-31 15:56 ` Pantelis Antoniou (?) @ 2012-10-31 5:23 ` Manjunathappa, Prakash -1 siblings, 0 replies; 12+ messages in thread From: Manjunathappa, Prakash @ 2012-10-31 5:23 UTC (permalink / raw) To: Pantelis Antoniou, Florian Tobias Schandinat Cc: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, Koen Kooi, Porter, Matt, Dill, Russ, linux-omap@vger.kernel.org, davinci-linux-open-source@linux.davincidsp.com Hi, On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: > This driver can be used for AM33xx devices, like the popular beaglebone. > > Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > --- > drivers/video/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > index 9791d10..e7868d8 100644 > --- a/drivers/video/Kconfig > +++ b/drivers/video/Kconfig > @@ -2202,7 +2202,7 @@ config FB_SH7760 > > config FB_DA8XX > tristate "DA8xx/OMAP-L1xx Framebuffer support" > - depends on FB && ARCH_DAVINCI_DA8XX > + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) Agreed this is present on da8xx and am33xx, but moving forward for supporting DT, we should be avoiding these dependencies. So instead change this to remove machine dependencies. Thanks, Prakash > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA > select FB_CFB_IMAGEBLIT > -- > 1.7.12 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices 2012-10-31 15:56 ` Pantelis Antoniou @ 2012-12-06 8:08 ` Manjunathappa, Prakash -1 siblings, 0 replies; 12+ messages in thread From: Manjunathappa, Prakash @ 2012-12-06 8:08 UTC (permalink / raw) To: Valkeinen, Tomi Cc: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, Koen Kooi, Porter, Matt, Dill, Russ, linux-omap@vger.kernel.org, davinci-linux-open-source@linux.davincidsp.com, FlorianSchandinat@gmx.de, Pantelis Antoniou Hi Tomi, On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: > Hi, > > On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: > > This driver can be used for AM33xx devices, like the popular beaglebone. > > > > Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > > --- > > drivers/video/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > > index 9791d10..e7868d8 100644 > > --- a/drivers/video/Kconfig > > +++ b/drivers/video/Kconfig > > @@ -2202,7 +2202,7 @@ config FB_SH7760 > > > > config FB_DA8XX > > tristate "DA8xx/OMAP-L1xx Framebuffer support" > > - depends on FB && ARCH_DAVINCI_DA8XX > > + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) > > Agreed this is present on da8xx and am33xx, but moving forward for > supporting DT, we should be avoiding these dependencies. So instead > change this to remove machine dependencies. > I could be wrong here, having dependency on platform seems to be right. Otherwise may lead to build errors for other platforms. Please ignore my comments and accept this patch. Thanks, Prakash ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices @ 2012-12-06 8:08 ` Manjunathappa, Prakash 0 siblings, 0 replies; 12+ messages in thread From: Manjunathappa, Prakash @ 2012-12-06 8:08 UTC (permalink / raw) To: Valkeinen, Tomi Cc: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, Koen Kooi, Porter, Matt, Dill, Russ, linux-omap@vger.kernel.org, davinci-linux-open-source@linux.davincidsp.com, FlorianSchandinat@gmx.de, Pantelis Antoniou Hi Tomi, On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: > Hi, > > On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: > > This driver can be used for AM33xx devices, like the popular beaglebone. > > > > Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > > --- > > drivers/video/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > > index 9791d10..e7868d8 100644 > > --- a/drivers/video/Kconfig > > +++ b/drivers/video/Kconfig > > @@ -2202,7 +2202,7 @@ config FB_SH7760 > > > > config FB_DA8XX > > tristate "DA8xx/OMAP-L1xx Framebuffer support" > > - depends on FB && ARCH_DAVINCI_DA8XX > > + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) > > Agreed this is present on da8xx and am33xx, but moving forward for > supporting DT, we should be avoiding these dependencies. So instead > change this to remove machine dependencies. > I could be wrong here, having dependency on platform seems to be right. Otherwise may lead to build errors for other platforms. Please ignore my comments and accept this patch. Thanks, Prakash ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] da8xx: Allow use by am33xx based devices 2012-12-06 8:08 ` Manjunathappa, Prakash @ 2012-12-10 9:14 ` Vaibhav Hiremath -1 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-12-10 9:02 UTC (permalink / raw) To: Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org On 12/6/2012 1:38 PM, Manjunathappa, Prakash wrote: > Hi Tomi, > > On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: >> Hi, >> >> On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: >>> This driver can be used for AM33xx devices, like the popular beaglebone. >>> >>> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> >>> --- >>> drivers/video/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig >>> index 9791d10..e7868d8 100644 >>> --- a/drivers/video/Kconfig >>> +++ b/drivers/video/Kconfig >>> @@ -2202,7 +2202,7 @@ config FB_SH7760 >>> >>> config FB_DA8XX >>> tristate "DA8xx/OMAP-L1xx Framebuffer support" >>> - depends on FB && ARCH_DAVINCI_DA8XX >>> + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) >> >> Agreed this is present on da8xx and am33xx, but moving forward for >> supporting DT, we should be avoiding these dependencies. So instead >> change this to remove machine dependencies. >> > > I could be wrong here, having dependency on platform seems to be right. > Otherwise may lead to build errors for other platforms. No, it should not result in to build error unless driver uses some platform specific api's. Thanks, Vaibhav > Please ignore my > comments and accept this patch. > > Thanks, > Prakash > _______________________________________________ > Davinci-linux-open-source mailing list > Davinci-linux-open-source@linux.davincidsp.com > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] da8xx: Allow use by am33xx based devices @ 2012-12-10 9:14 ` Vaibhav Hiremath 0 siblings, 0 replies; 12+ messages in thread From: Vaibhav Hiremath @ 2012-12-10 9:14 UTC (permalink / raw) To: Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org On 12/6/2012 1:38 PM, Manjunathappa, Prakash wrote: > Hi Tomi, > > On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: >> Hi, >> >> On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: >>> This driver can be used for AM33xx devices, like the popular beaglebone. >>> >>> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> >>> --- >>> drivers/video/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig >>> index 9791d10..e7868d8 100644 >>> --- a/drivers/video/Kconfig >>> +++ b/drivers/video/Kconfig >>> @@ -2202,7 +2202,7 @@ config FB_SH7760 >>> >>> config FB_DA8XX >>> tristate "DA8xx/OMAP-L1xx Framebuffer support" >>> - depends on FB && ARCH_DAVINCI_DA8XX >>> + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) >> >> Agreed this is present on da8xx and am33xx, but moving forward for >> supporting DT, we should be avoiding these dependencies. So instead >> change this to remove machine dependencies. >> > > I could be wrong here, having dependency on platform seems to be right. > Otherwise may lead to build errors for other platforms. No, it should not result in to build error unless driver uses some platform specific api's. Thanks, Vaibhav > Please ignore my > comments and accept this patch. > > Thanks, > Prakash > _______________________________________________ > Davinci-linux-open-source mailing list > Davinci-linux-open-source@linux.davincidsp.com > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source > ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices 2012-12-10 9:14 ` Vaibhav Hiremath (?) @ 2012-12-12 7:20 ` Manjunathappa, Prakash 2012-12-12 8:00 ` Hiremath, Vaibhav -1 siblings, 1 reply; 12+ messages in thread From: Manjunathappa, Prakash @ 2012-12-12 7:20 UTC (permalink / raw) To: Hiremath, Vaibhav Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org Hi Vaibhav, On Mon, Dec 10, 2012 at 14:32:06, Hiremath, Vaibhav wrote: > > > On 12/6/2012 1:38 PM, Manjunathappa, Prakash wrote: > > Hi Tomi, > > > > On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: > >> Hi, > >> > >> On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: > >>> This driver can be used for AM33xx devices, like the popular beaglebone. > >>> > >>> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > >>> --- > >>> drivers/video/Kconfig | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > >>> index 9791d10..e7868d8 100644 > >>> --- a/drivers/video/Kconfig > >>> +++ b/drivers/video/Kconfig > >>> @@ -2202,7 +2202,7 @@ config FB_SH7760 > >>> > >>> config FB_DA8XX > >>> tristate "DA8xx/OMAP-L1xx Framebuffer support" > >>> - depends on FB && ARCH_DAVINCI_DA8XX > >>> + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) > >> > >> Agreed this is present on da8xx and am33xx, but moving forward for > >> supporting DT, we should be avoiding these dependencies. So instead > >> change this to remove machine dependencies. > >> > > > > I could be wrong here, having dependency on platform seems to be right. > > Otherwise may lead to build errors for other platforms. > > No, it should not result in to build error unless driver uses some > platform specific api's. > Agreed, should not result in build error. But is it ok to show this option on the platforms which do not have this IP? Thanks, Prakash ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices 2012-12-12 7:20 ` Manjunathappa, Prakash @ 2012-12-12 8:00 ` Hiremath, Vaibhav 2013-01-07 12:03 ` Mohammed, Afzal 0 siblings, 1 reply; 12+ messages in thread From: Hiremath, Vaibhav @ 2012-12-12 8:00 UTC (permalink / raw) To: Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org On Wed, Dec 12, 2012 at 12:50:28, Manjunathappa, Prakash wrote: > Hi Vaibhav, > > On Mon, Dec 10, 2012 at 14:32:06, Hiremath, Vaibhav wrote: > > > > > > On 12/6/2012 1:38 PM, Manjunathappa, Prakash wrote: > > > Hi Tomi, > > > > > > On Wed, Oct 31, 2012 at 10:52:59, Manjunathappa, Prakash wrote: > > >> Hi, > > >> > > >> On Wed, Oct 31, 2012 at 21:26:08, Pantelis Antoniou wrote: > > >>> This driver can be used for AM33xx devices, like the popular beaglebone. > > >>> > > >>> Signed-off-by: Pantelis Antoniou <panto@antoniou-consulting.com> > > >>> --- > > >>> drivers/video/Kconfig | 2 +- > > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > > >>> > > >>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig > > >>> index 9791d10..e7868d8 100644 > > >>> --- a/drivers/video/Kconfig > > >>> +++ b/drivers/video/Kconfig > > >>> @@ -2202,7 +2202,7 @@ config FB_SH7760 > > >>> > > >>> config FB_DA8XX > > >>> tristate "DA8xx/OMAP-L1xx Framebuffer support" > > >>> - depends on FB && ARCH_DAVINCI_DA8XX > > >>> + depends on FB && (ARCH_DAVINCI_DA8XX || SOC_AM33XX) > > >> > > >> Agreed this is present on da8xx and am33xx, but moving forward for > > >> supporting DT, we should be avoiding these dependencies. So instead > > >> change this to remove machine dependencies. > > >> > > > > > > I could be wrong here, having dependency on platform seems to be right. > > > Otherwise may lead to build errors for other platforms. > > > > No, it should not result in to build error unless driver uses some > > platform specific api's. > > > > Agreed, should not result in build error. But is it ok to show this option > on the platforms which do not have this IP? > You can choose to put machine dependency here, as this patch is already doing it. The side-effect of this would be, list may grow and you may have to edit this file everytime. Thanks, Vaibhav ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices 2012-12-12 8:00 ` Hiremath, Vaibhav 2013-01-07 12:03 ` Mohammed, Afzal @ 2013-01-07 12:03 ` Mohammed, Afzal 0 siblings, 0 replies; 12+ messages in thread From: Mohammed, Afzal @ 2013-01-07 12:03 UTC (permalink / raw) To: Hiremath, Vaibhav, Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org SGksDQoNCk9uIFdlZCwgRGVjIDEyLCAyMDEyIGF0IDEzOjMwOjU2LCBIaXJlbWF0aCwgVmFpYmhh diB3cm90ZToNCj4gT24gV2VkLCBEZWMgMTIsIDIwMTIgYXQgMTI6NTA6MjgsIE1hbmp1bmF0aGFw cGEsIFByYWthc2ggd3JvdGU6DQoNCj4gPiBBZ3JlZWQsIHNob3VsZCBub3QgcmVzdWx0IGluIGJ1 aWxkIGVycm9yLiBCdXQgaXMgaXQgb2sgdG8gc2hvdyB0aGlzIG9wdGlvbg0KPiA+IG9uIHRoZSBw bGF0Zm9ybXMgd2hpY2ggZG8gbm90IGhhdmUgdGhpcyBJUD8NCj4gPiANCj4gDQo+IFlvdSBjYW4g Y2hvb3NlIHRvIHB1dCBtYWNoaW5lIGRlcGVuZGVuY3kgaGVyZSwgYXMgdGhpcyBwYXRjaCBpcyBh bHJlYWR5IA0KPiBkb2luZyBpdC4gVGhlIHNpZGUtZWZmZWN0IG9mIHRoaXMgd291bGQgYmUsIGxp c3QgbWF5IGdyb3cgYW5kIHlvdSBtYXkgaGF2ZSANCj4gdG8gZWRpdCB0aGlzIGZpbGUgZXZlcnl0 aW1lLg0KDQpJdCBzZWVtcyBhZGRpbmcgbWFjaGluZSBkZXBlbmRlbmN5IGlzIG1vcmUgaW4gbGlu ZSB3aXRoIHRoZQ0KYXBwcm9hY2ggYWRvcHRlZCBpbiBzaW1pbGFyIGNhc2VzLiBBcyB0aGlzIElQ IGlzIHVzZWQgb25seQ0KaW4gMiBhcmNoJ3MgYW5kIHdvdWxkIG5vdCBpbiBhbnkgaW1tZWRpYXRl IGZ1dHVyZSBnbyB3aXRoDQpvdGhlciBhcmNoJ3MsIGl0IHdvdWxkIGJlIGJldHRlciBub3QgdG8g YXNrIG90aGVyIGFyY2gncyBhDQpxdWVzdGlvbiBvbiB3aGV0aGVyIHRoaXMgZHJpdmVyIG5lZWRz IHRvIGJlIHNlbGVjdGVkIG9yIG5vdC4NCg0KUmVnYXJkcw0KQWZ6YWwNCg= ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices @ 2013-01-07 12:03 ` Mohammed, Afzal 0 siblings, 0 replies; 12+ messages in thread From: Mohammed, Afzal @ 2013-01-07 12:03 UTC (permalink / raw) To: Hiremath, Vaibhav, Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain; charset="utf-8", Size: 923 bytes --] Hi, On Wed, Dec 12, 2012 at 13:30:56, Hiremath, Vaibhav wrote: > On Wed, Dec 12, 2012 at 12:50:28, Manjunathappa, Prakash wrote: > > Agreed, should not result in build error. But is it ok to show this option > > on the platforms which do not have this IP? > > > > You can choose to put machine dependency here, as this patch is already > doing it. The side-effect of this would be, list may grow and you may have > to edit this file everytime. It seems adding machine dependency is more in line with the approach adopted in similar cases. As this IP is used only in 2 arch's and would not in any immediate future go with other arch's, it would be better not to ask other arch's a question on whether this driver needs to be selected or not. Regards Afzal ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥ ^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [PATCH] da8xx: Allow use by am33xx based devices @ 2013-01-07 12:03 ` Mohammed, Afzal 0 siblings, 0 replies; 12+ messages in thread From: Mohammed, Afzal @ 2013-01-07 12:03 UTC (permalink / raw) To: Hiremath, Vaibhav, Manjunathappa, Prakash Cc: Valkeinen, Tomi, davinci-linux-open-source@linux.davincidsp.com, Porter, Matt, linux-fbdev@vger.kernel.org, FlorianSchandinat@gmx.de, Koen Kooi, Pantelis Antoniou, linux-kernel@vger.kernel.org, Dill, Russ, linux-omap@vger.kernel.org Hi, On Wed, Dec 12, 2012 at 13:30:56, Hiremath, Vaibhav wrote: > On Wed, Dec 12, 2012 at 12:50:28, Manjunathappa, Prakash wrote: > > Agreed, should not result in build error. But is it ok to show this option > > on the platforms which do not have this IP? > > > > You can choose to put machine dependency here, as this patch is already > doing it. The side-effect of this would be, list may grow and you may have > to edit this file everytime. It seems adding machine dependency is more in line with the approach adopted in similar cases. As this IP is used only in 2 arch's and would not in any immediate future go with other arch's, it would be better not to ask other arch's a question on whether this driver needs to be selected or not. Regards Afzal ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-01-07 12:03 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-10-30 18:18 [PATCH] da8xx: Allow use by am33xx based devices Pantelis Antoniou 2012-10-31 15:56 ` Pantelis Antoniou 2012-10-31 5:23 ` Manjunathappa, Prakash 2012-12-06 8:08 ` Manjunathappa, Prakash 2012-12-06 8:08 ` Manjunathappa, Prakash 2012-12-10 9:02 ` Vaibhav Hiremath 2012-12-10 9:14 ` Vaibhav Hiremath 2012-12-12 7:20 ` Manjunathappa, Prakash 2012-12-12 8:00 ` Hiremath, Vaibhav 2013-01-07 12:03 ` Mohammed, Afzal 2013-01-07 12:03 ` Mohammed, Afzal 2013-01-07 12:03 ` Mohammed, Afzal
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.