[PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Richard Purdie]

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/recipes-graphics/mesa/mesa-dri.inc b/meta/recipes-graphics/mesa/mesa-dri.inc
index 603438e..795144a 100644
--- a/meta/recipes-graphics/mesa/mesa-dri.inc
+++ b/meta/recipes-graphics/mesa/mesa-dri.inc
@@ -4,7 +4,11 @@ LIB_DEPS += "libdrm expat"
 # most of our targets do not have DRI so will use mesa-xlib
 DEFAULT_PREFERENCE = "-1"
 
-EXTRA_OECONF += "--with-driver=dri --disable-egl --disable-gallium --without-gallium-drivers"
+DRIDRIVERS = "swrast"
+DRIDRIVERS_x86 = "swrast,i915,i965"
+DRIDRIVERS_x86-64 = "swrast,i915,i965"
+
+EXTRA_OECONF += "--with-driver=dri --disable-egl --disable-gallium --without-gallium-drivers --with-dri-drivers=${DRIDRIVERS}"
 
 python populate_packages_prepend() {
 	import os.path
diff --git a/meta/recipes-graphics/mesa/mesa-dri_7.11.bb b/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
index 6a4f9a9..5d25127 100644
--- a/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
+++ b/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
@@ -2,7 +2,3 @@ include mesa-common.inc
 include mesa-${PV}.inc
 include mesa-dri.inc
 PR = "${INC_PR}.0"
-
-EXTRA_OECONF += "--with-dri-drivers=swrast,i915,i965"
-
-COMPATIBLE_HOST = '(i.86.*-linux|x86_64.*-linux)'
diff --git a/meta/recipes-graphics/mesa/mesa-dri_git.bb b/meta/recipes-graphics/mesa/mesa-dri_git.bb
index 15cde60..a0e1945 100644
--- a/meta/recipes-graphics/mesa/mesa-dri_git.bb
+++ b/meta/recipes-graphics/mesa/mesa-dri_git.bb
@@ -7,5 +7,3 @@ DEFAULT_PREFERENCE = "-2"
 
 PR = "${INC_PR}.0"
 
-COMPATIBLE_HOST = '(i.86|x86_64).*-linux'
-

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Otavio Salvador]

+1

Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
override it.

On Fri, Oct 14, 2011 at 11:42, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
> diff --git a/meta/recipes-graphics/mesa/mesa-dri.inc b/meta/recipes-graphics/mesa/mesa-dri.inc
> index 603438e..795144a 100644
> --- a/meta/recipes-graphics/mesa/mesa-dri.inc
> +++ b/meta/recipes-graphics/mesa/mesa-dri.inc
> @@ -4,7 +4,11 @@ LIB_DEPS += "libdrm expat"
>  # most of our targets do not have DRI so will use mesa-xlib
>  DEFAULT_PREFERENCE = "-1"
>
> -EXTRA_OECONF += "--with-driver=dri --disable-egl --disable-gallium --without-gallium-drivers"
> +DRIDRIVERS = "swrast"
> +DRIDRIVERS_x86 = "swrast,i915,i965"
> +DRIDRIVERS_x86-64 = "swrast,i915,i965"
> +
> +EXTRA_OECONF += "--with-driver=dri --disable-egl --disable-gallium --without-gallium-drivers --with-dri-drivers=${DRIDRIVERS}"
>
>  python populate_packages_prepend() {
>        import os.path
> diff --git a/meta/recipes-graphics/mesa/mesa-dri_7.11.bb b/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
> index 6a4f9a9..5d25127 100644
> --- a/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
> +++ b/meta/recipes-graphics/mesa/mesa-dri_7.11.bb
> @@ -2,7 +2,3 @@ include mesa-common.inc
>  include mesa-${PV}.inc
>  include mesa-dri.inc
>  PR = "${INC_PR}.0"
> -
> -EXTRA_OECONF += "--with-dri-drivers=swrast,i915,i965"
> -
> -COMPATIBLE_HOST = '(i.86.*-linux|x86_64.*-linux)'
> diff --git a/meta/recipes-graphics/mesa/mesa-dri_git.bb b/meta/recipes-graphics/mesa/mesa-dri_git.bb
> index 15cde60..a0e1945 100644
> --- a/meta/recipes-graphics/mesa/mesa-dri_git.bb
> +++ b/meta/recipes-graphics/mesa/mesa-dri_git.bb
> @@ -7,5 +7,3 @@ DEFAULT_PREFERENCE = "-2"
>
>  PR = "${INC_PR}.0"
>
> -COMPATIBLE_HOST = '(i.86|x86_64).*-linux'
> -
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>



-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Richard Purdie]

On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> +1
> 
> Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> override it.

I really wouldn't recommend overriding this on a per machine basis, it
needs to be on a per arch basis. This is because the recipe is not
machine specific (nor should it be).

Configuration therefore falls to the distro, not machine.

Cheers,

Richard

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 797 bytes --]

On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
> On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> > +1
> > 
> > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> > override it.
> 
> I really wouldn't recommend overriding this on a per machine basis, it
> needs to be on a per arch basis. This is because the recipe is not
> machine specific (nor should it be).
> 
> Configuration therefore falls to the distro, not machine.

Why not make it machine specific only when machine provides own module
(like the case with glamo on om-gta02)?

Or recipe cannot change PACKAGE_ARCH in some special cases (like
$MACHINE in path to some file in SRC_URI) anymore?

Regards,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Koen Kooi]

Op 14 okt. 2011, om 17:30 heeft Martin Jansa het volgende geschreven:

> On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
>> On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
>>> +1
>>> 
>>> Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
>>> override it.
>> 
>> I really wouldn't recommend overriding this on a per machine basis, it
>> needs to be on a per arch basis. This is because the recipe is not
>> machine specific (nor should it be).
>> 
>> Configuration therefore falls to the distro, not machine.
> 
> Why not make it machine specific only when machine provides own module
> (like the case with glamo on om-gta02)?
> 
> Or recipe cannot change PACKAGE_ARCH in some special cases (like
> $MACHINE in path to some file in SRC_URI) anymore?

Kinda like the wpa-supplicant and mplayer trick in OE classic?


> 
> Regards,
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Richard Purdie]

On Fri, 2011-10-14 at 17:30 +0200, Martin Jansa wrote:
> On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
> > On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> > > +1
> > > 
> > > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> > > override it.
> > 
> > I really wouldn't recommend overriding this on a per machine basis, it
> > needs to be on a per arch basis. This is because the recipe is not
> > machine specific (nor should it be).
> > 
> > Configuration therefore falls to the distro, not machine.
> 
> Why not make it machine specific only when machine provides own module
> (like the case with glamo on om-gta02)?
> 
> Or recipe cannot change PACKAGE_ARCH in some special cases (like
> $MACHINE in path to some file in SRC_URI) anymore?

It works just fine but its not nice practise in my opinion for a library
like this and I don't see there is any need in this case. Certainly I
don't see it as something OE-Core should be recommending.

Cheers,

Richard

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]

On Fri, Oct 14, 2011 at 04:55:50PM +0100, Richard Purdie wrote:
> On Fri, 2011-10-14 at 17:30 +0200, Martin Jansa wrote:
> > On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
> > > On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> > > > +1
> > > > 
> > > > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> > > > override it.
> > > 
> > > I really wouldn't recommend overriding this on a per machine basis, it
> > > needs to be on a per arch basis. This is because the recipe is not
> > > machine specific (nor should it be).
> > > 
> > > Configuration therefore falls to the distro, not machine.
> > 
> > Why not make it machine specific only when machine provides own module
> > (like the case with glamo on om-gta02)?
> > 
> > Or recipe cannot change PACKAGE_ARCH in some special cases (like
> > $MACHINE in path to some file in SRC_URI) anymore?
> 
> It works just fine but its not nice practise in my opinion for a library
> like this and I don't see there is any need in this case. Certainly I
> don't see it as something OE-Core should be recommending.

So can I send patches adding my glamo.patch to libdrm and mesa-dri so we
can add glamo to 
DRIDRIVERS_armv4t ?

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Richard Purdie]

On Fri, 2011-10-14 at 18:05 +0200, Martin Jansa wrote:
> On Fri, Oct 14, 2011 at 04:55:50PM +0100, Richard Purdie wrote:
> > On Fri, 2011-10-14 at 17:30 +0200, Martin Jansa wrote:
> > > On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
> > > > On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> > > > > +1
> > > > > 
> > > > > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> > > > > override it.
> > > > 
> > > > I really wouldn't recommend overriding this on a per machine basis, it
> > > > needs to be on a per arch basis. This is because the recipe is not
> > > > machine specific (nor should it be).
> > > > 
> > > > Configuration therefore falls to the distro, not machine.
> > > 
> > > Why not make it machine specific only when machine provides own module
> > > (like the case with glamo on om-gta02)?
> > > 
> > > Or recipe cannot change PACKAGE_ARCH in some special cases (like
> > > $MACHINE in path to some file in SRC_URI) anymore?
> > 
> > It works just fine but its not nice practise in my opinion for a library
> > like this and I don't see there is any need in this case. Certainly I
> > don't see it as something OE-Core should be recommending.
> 
> So can I send patches adding my glamo.patch to libdrm and mesa-dri so we
> can add glamo to 
> DRIDRIVERS_armv4t ?

No, what I mean is if the layer containing that machine appends those
patches for arm in general (or armv4t), you can then enable the dri
drivers for armv4t in general to.

It means you would have to keep the layer enabled whenever generating
armv4t feeds but I think that is ok as long as you know about it?

Cheers,

Richard

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Khem Raj]

On Fri, Oct 14, 2011 at 3:15 PM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Fri, 2011-10-14 at 18:05 +0200, Martin Jansa wrote:
>> On Fri, Oct 14, 2011 at 04:55:50PM +0100, Richard Purdie wrote:
>> > On Fri, 2011-10-14 at 17:30 +0200, Martin Jansa wrote:
>> > > On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
>> > > > On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
>> > > > > +1
>> > > > >
>> > > > > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
>> > > > > override it.
>> > > >
>> > > > I really wouldn't recommend overriding this on a per machine basis, it
>> > > > needs to be on a per arch basis. This is because the recipe is not
>> > > > machine specific (nor should it be).
>> > > >
>> > > > Configuration therefore falls to the distro, not machine.
>> > >
>> > > Why not make it machine specific only when machine provides own module
>> > > (like the case with glamo on om-gta02)?
>> > >
>> > > Or recipe cannot change PACKAGE_ARCH in some special cases (like
>> > > $MACHINE in path to some file in SRC_URI) anymore?
>> >
>> > It works just fine but its not nice practise in my opinion for a library
>> > like this and I don't see there is any need in this case. Certainly I
>> > don't see it as something OE-Core should be recommending.
>>
>> So can I send patches adding my glamo.patch to libdrm and mesa-dri so we
>> can add glamo to
>> DRIDRIVERS_armv4t ?
>
> No, what I mean is if the layer containing that machine appends those
> patches for arm in general (or armv4t), you can then enable the dri
> drivers for armv4t in general to.

but there can be more than 1 armv4t machines in different layers.

>
> It means you would have to keep the layer enabled whenever generating
> armv4t feeds but I think that is ok as long as you know about it?
>
> Cheers,
>
> Richard
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

Re: [PATCH] mesa-dri: Enable swrast only by default and intel drivers only on IA platform

[Richard Purdie]

On Fri, 2011-10-14 at 15:53 -0700, Khem Raj wrote:
> On Fri, Oct 14, 2011 at 3:15 PM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > On Fri, 2011-10-14 at 18:05 +0200, Martin Jansa wrote:
> >> On Fri, Oct 14, 2011 at 04:55:50PM +0100, Richard Purdie wrote:
> >> > On Fri, 2011-10-14 at 17:30 +0200, Martin Jansa wrote:
> >> > > On Fri, Oct 14, 2011 at 04:25:16PM +0100, Richard Purdie wrote:
> >> > > > On Fri, 2011-10-14 at 11:46 -0300, Otavio Salvador wrote:
> >> > > > > +1
> >> > > > >
> >> > > > > Just another thing, I'd prefer to have DRIDRIVERS as ?= so machine can
> >> > > > > override it.
> >> > > >
> >> > > > I really wouldn't recommend overriding this on a per machine basis, it
> >> > > > needs to be on a per arch basis. This is because the recipe is not
> >> > > > machine specific (nor should it be).
> >> > > >
> >> > > > Configuration therefore falls to the distro, not machine.
> >> > >
> >> > > Why not make it machine specific only when machine provides own module
> >> > > (like the case with glamo on om-gta02)?
> >> > >
> >> > > Or recipe cannot change PACKAGE_ARCH in some special cases (like
> >> > > $MACHINE in path to some file in SRC_URI) anymore?
> >> >
> >> > It works just fine but its not nice practise in my opinion for a library
> >> > like this and I don't see there is any need in this case. Certainly I
> >> > don't see it as something OE-Core should be recommending.
> >>
> >> So can I send patches adding my glamo.patch to libdrm and mesa-dri so we
> >> can add glamo to
> >> DRIDRIVERS_armv4t ?
> >
> > No, what I mean is if the layer containing that machine appends those
> > patches for arm in general (or armv4t), you can then enable the dri
> > drivers for armv4t in general to.
> 
> but there can be more than 1 armv4t machines in different layers.

Yes, I know. We established earlier that this will compile on other arm
machines. Obviously those machines would not include the module in their
rootfs, gta02 would. Building it shouldn't make much difference to them.

Cheers,

Richard