[Intel-gfx] [PATCH i-g-t] Demos/Android.mk: dont build intel_sprite_on

[Daniel Vetter]

On Mon, Dec 15, 2014 at 02:51:03PM +0000, Gore, Tim wrote:
> 
> 
> > -----Original Message-----
> > From: Daniel Vetter [mailto:daniel.vetter at ffwll.ch] On Behalf Of Daniel
> > Vetter
> > Sent: Monday, December 15, 2014 2:41 PM
> > To: Wood, Thomas
> > Cc: Gore, Tim; Intel Graphics Development
> > Subject: Re: [Intel-gfx] [PATCH i-g-t] Demos/Android.mk: dont build
> > intel_sprite_on
> > 
> > On Fri, Dec 12, 2014 at 11:43:55AM +0000, Thomas Wood wrote:
> > > On 12 December 2014 at 10:18,  <tim.gore at intel.com> wrote:
> > > > From: Tim Gore <tim.gore at intel.com>
> > > >
> > > > intel_sprite_on wont build on Android, due to use of a particular
> > > > API that has changed in Gmin
> > > >
> > > > Signed-off-by: Tim Gore <tim.gore at intel.com>
> > > > ---
> > > >  demos/Android.mk | 6 ++++--
> > > >  1 file changed, 4 insertions(+), 2 deletions(-)
> > > >
> > > > diff --git a/demos/Android.mk b/demos/Android.mk index
> > > > 6227e06..be7f3c2 100644
> > > > --- a/demos/Android.mk
> > > > +++ b/demos/Android.mk
> > > > @@ -4,11 +4,13 @@ LOCAL_PATH := $(call my-dir)
> > > >
> > > >  include $(CLEAR_VARS)
> > > >
> > > > -LOCAL_SRC_FILES := intel_sprite_on.c
> > > > +# This demo wont build on android (from Gmin on).
> > > > +#LOCAL_SRC_FILES := intel_sprite_on.c
> > > > +
> > > >
> > > >  LOCAL_CFLAGS += -DHAVE_TERMIOS_H
> > > >  LOCAL_CFLAGS += -DANDROID -UNDEBUG
> > > > -LOCAL_CFLAGS += -std=c99
> > > > +LOCAL_CFLAGS += -std=gnu99
> > >
> > > I think this change should have been in your previous patch
> > > ("Android.mk: replace std=c99 with std=gnu99")?
> > 
> > btw intel_sprite_on has been contributed by your team (Armin Reese), so
> > not sure why you want to disable it. Have you checked with display folks
> > whether they don't need it?
> > -Daniel
> > 
> 
> Unfortunately this test wont build, out of the box, for Android. I can patch 
> This locally but I don't think the patch is suitable for upstream.

I'm still confused ... why does it not build on Android? I've thought we
have this now all worked out wrt igt build issues. Imo better to address
whatever's upsetting your builder than to just disable stuff.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch