* [PATCH i-g-t] Android.mk: Disable tools that do not build for android @ 2015-06-15 13:04 Derek Morton 2015-06-15 13:08 ` Thomas Wood 0 siblings, 1 reply; 7+ messages in thread From: Derek Morton @ 2015-06-15 13:04 UTC (permalink / raw) To: intel-gfx; +Cc: thomas.wood Disable the tools / demo code that do not currently build for android until they can be fixed. Affected tools / demos intel_reg intel_display_crc intel_sprite_on Signed-off-by: Derek Morton <derek.j.morton@intel.com> --- Android.mk | 2 +- tools/Android.mk | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/Android.mk b/Android.mk index 1ab3e64..681d114 100644 --- a/Android.mk +++ b/Android.mk @@ -1,2 +1,2 @@ -include $(call all-named-subdir-makefiles, lib tests tools benchmarks demos) +include $(call all-named-subdir-makefiles, lib tests tools benchmarks) diff --git a/tools/Android.mk b/tools/Android.mk index 39f4512..fd2800e 100644 --- a/tools/Android.mk +++ b/tools/Android.mk @@ -36,7 +36,9 @@ endef #================# skip_tools_list := \ + intel_display_crc \ intel_framebuffer_dump \ + intel_reg \ intel_reg_dumper \ intel_vga_read \ intel_vga_write -- 1.9.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:04 [PATCH i-g-t] Android.mk: Disable tools that do not build for android Derek Morton @ 2015-06-15 13:08 ` Thomas Wood 2015-06-15 13:19 ` Morton, Derek J 0 siblings, 1 reply; 7+ messages in thread From: Thomas Wood @ 2015-06-15 13:08 UTC (permalink / raw) To: Derek Morton; +Cc: Intel Graphics Development On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: > Disable the tools / demo code that do not currently build > for android until they can be fixed. What needs to be fixed? > > Affected tools / demos > intel_reg > intel_display_crc > intel_sprite_on > > Signed-off-by: Derek Morton <derek.j.morton@intel.com> > --- > Android.mk | 2 +- > tools/Android.mk | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Android.mk b/Android.mk > index 1ab3e64..681d114 100644 > --- a/Android.mk > +++ b/Android.mk > @@ -1,2 +1,2 @@ > -include $(call all-named-subdir-makefiles, lib tests tools benchmarks demos) > +include $(call all-named-subdir-makefiles, lib tests tools benchmarks) > > diff --git a/tools/Android.mk b/tools/Android.mk > index 39f4512..fd2800e 100644 > --- a/tools/Android.mk > +++ b/tools/Android.mk > @@ -36,7 +36,9 @@ endef > #================# > > skip_tools_list := \ > + intel_display_crc \ > intel_framebuffer_dump \ > + intel_reg \ > intel_reg_dumper \ > intel_vga_read \ > intel_vga_write > -- > 1.9.1 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:08 ` Thomas Wood @ 2015-06-15 13:19 ` Morton, Derek J 2015-06-15 13:23 ` Jani Nikula 2015-06-15 13:42 ` Daniel Vetter 0 siblings, 2 replies; 7+ messages in thread From: Morton, Derek J @ 2015-06-15 13:19 UTC (permalink / raw) To: Wood, Thomas; +Cc: Intel Graphics Development > >-----Original Message----- >From: Thomas Wood [mailto:thomas.wood@intel.com] >Sent: Monday, June 15, 2015 2:09 PM >To: Morton, Derek J >Cc: Intel Graphics Development; Gore, Tim >Subject: Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android > >On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: >> Disable the tools / demo code that do not currently build for android >> until they can be fixed. > >What needs to be fixed? > Intel_reg - Relies on a header file not provided by the Android compiler Intel_display_crc - Cairo dependency Intel_sprite_on - An API difference in the Android i915 driver. //Derek > >> >> Affected tools / demos >> intel_reg >> intel_display_crc >> intel_sprite_on >> >> Signed-off-by: Derek Morton <derek.j.morton@intel.com> >> --- >> Android.mk | 2 +- >> tools/Android.mk | 2 ++ >> 2 files changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Android.mk b/Android.mk >> index 1ab3e64..681d114 100644 >> --- a/Android.mk >> +++ b/Android.mk >> @@ -1,2 +1,2 @@ >> -include $(call all-named-subdir-makefiles, lib tests tools benchmarks >> demos) >> +include $(call all-named-subdir-makefiles, lib tests tools >> +benchmarks) >> >> diff --git a/tools/Android.mk b/tools/Android.mk index >> 39f4512..fd2800e 100644 >> --- a/tools/Android.mk >> +++ b/tools/Android.mk >> @@ -36,7 +36,9 @@ endef >> #================# >> >> skip_tools_list := \ >> + intel_display_crc \ >> intel_framebuffer_dump \ >> + intel_reg \ >> intel_reg_dumper \ >> intel_vga_read \ >> intel_vga_write >> -- >> 1.9.1 >> > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:19 ` Morton, Derek J @ 2015-06-15 13:23 ` Jani Nikula 2015-06-15 13:38 ` Morton, Derek J 2015-06-15 13:42 ` Daniel Vetter 1 sibling, 1 reply; 7+ messages in thread From: Jani Nikula @ 2015-06-15 13:23 UTC (permalink / raw) To: Morton, Derek J, Wood, Thomas; +Cc: Intel Graphics Development On Mon, 15 Jun 2015, "Morton, Derek J" <derek.j.morton@intel.com> wrote: >> >>-----Original Message----- >>From: Thomas Wood [mailto:thomas.wood@intel.com] >>Sent: Monday, June 15, 2015 2:09 PM >>To: Morton, Derek J >>Cc: Intel Graphics Development; Gore, Tim >>Subject: Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android >> >>On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: >>> Disable the tools / demo code that do not currently build for android >>> until they can be fixed. >> >>What needs to be fixed? >> > > Intel_reg - Relies on a header file not provided by the Android compiler ...which one? > Intel_display_crc - Cairo dependency > Intel_sprite_on - An API difference in the Android i915 driver. > > //Derek > >> >>> >>> Affected tools / demos >>> intel_reg >>> intel_display_crc >>> intel_sprite_on >>> >>> Signed-off-by: Derek Morton <derek.j.morton@intel.com> >>> --- >>> Android.mk | 2 +- >>> tools/Android.mk | 2 ++ >>> 2 files changed, 3 insertions(+), 1 deletion(-) >>> >>> diff --git a/Android.mk b/Android.mk >>> index 1ab3e64..681d114 100644 >>> --- a/Android.mk >>> +++ b/Android.mk >>> @@ -1,2 +1,2 @@ >>> -include $(call all-named-subdir-makefiles, lib tests tools benchmarks >>> demos) >>> +include $(call all-named-subdir-makefiles, lib tests tools >>> +benchmarks) >>> >>> diff --git a/tools/Android.mk b/tools/Android.mk index >>> 39f4512..fd2800e 100644 >>> --- a/tools/Android.mk >>> +++ b/tools/Android.mk >>> @@ -36,7 +36,9 @@ endef >>> #================# >>> >>> skip_tools_list := \ >>> + intel_display_crc \ >>> intel_framebuffer_dump \ >>> + intel_reg \ >>> intel_reg_dumper \ >>> intel_vga_read \ >>> intel_vga_write >>> -- >>> 1.9.1 >>> >> > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:23 ` Jani Nikula @ 2015-06-15 13:38 ` Morton, Derek J 0 siblings, 0 replies; 7+ messages in thread From: Morton, Derek J @ 2015-06-15 13:38 UTC (permalink / raw) To: Jani Nikula, Wood, Thomas; +Cc: Intel Graphics Development > > >-----Original Message----- >From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >Sent: Monday, June 15, 2015 2:24 PM >To: Morton, Derek J; Wood, Thomas >Cc: Intel Graphics Development >Subject: Re: [Intel-gfx] [PATCH i-g-t] Android.mk: Disable tools that do not build for android > >On Mon, 15 Jun 2015, "Morton, Derek J" <derek.j.morton@intel.com> wrote: >>> >>>-----Original Message----- >>>From: Thomas Wood [mailto:thomas.wood@intel.com] >>>Sent: Monday, June 15, 2015 2:09 PM >>>To: Morton, Derek J >>>Cc: Intel Graphics Development; Gore, Tim >>>Subject: Re: [PATCH i-g-t] Android.mk: Disable tools that do not build >>>for android >>> >>>On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: >>>> Disable the tools / demo code that do not currently build for >>>> android until they can be fixed. >>> >>>What needs to be fixed? >>> >> >> Intel_reg - Relies on a header file not provided by the Android >> compiler > >...which one? sys/io.h Tim Gore has looked into this issue. The complexity in fixing is more with the licencing than technical. //Derek > >> Intel_display_crc - Cairo dependency >> Intel_sprite_on - An API difference in the Android i915 driver. >> >> //Derek >> >>> >>>> >>>> Affected tools / demos >>>> intel_reg >>>> intel_display_crc >>>> intel_sprite_on >>>> >>>> Signed-off-by: Derek Morton <derek.j.morton@intel.com> >>>> --- >>>> Android.mk | 2 +- >>>> tools/Android.mk | 2 ++ >>>> 2 files changed, 3 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/Android.mk b/Android.mk index 1ab3e64..681d114 100644 >>>> --- a/Android.mk >>>> +++ b/Android.mk >>>> @@ -1,2 +1,2 @@ >>>> -include $(call all-named-subdir-makefiles, lib tests tools >>>> benchmarks >>>> demos) >>>> +include $(call all-named-subdir-makefiles, lib tests tools >>>> +benchmarks) >>>> >>>> diff --git a/tools/Android.mk b/tools/Android.mk index >>>> 39f4512..fd2800e 100644 >>>> --- a/tools/Android.mk >>>> +++ b/tools/Android.mk >>>> @@ -36,7 +36,9 @@ endef >>>> #================# >>>> >>>> skip_tools_list := \ >>>> + intel_display_crc \ >>>> intel_framebuffer_dump \ >>>> + intel_reg \ >>>> intel_reg_dumper \ >>>> intel_vga_read \ >>>> intel_vga_write >>>> -- >>>> 1.9.1 >>>> >>> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx > >-- >Jani Nikula, Intel Open Source Technology Center > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:19 ` Morton, Derek J 2015-06-15 13:23 ` Jani Nikula @ 2015-06-15 13:42 ` Daniel Vetter 2015-06-15 14:08 ` Morton, Derek J 1 sibling, 1 reply; 7+ messages in thread From: Daniel Vetter @ 2015-06-15 13:42 UTC (permalink / raw) To: Morton, Derek J; +Cc: Intel Graphics Development, Wood, Thomas On Mon, Jun 15, 2015 at 01:19:16PM +0000, Morton, Derek J wrote: > > > >-----Original Message----- > >From: Thomas Wood [mailto:thomas.wood@intel.com] > >Sent: Monday, June 15, 2015 2:09 PM > >To: Morton, Derek J > >Cc: Intel Graphics Development; Gore, Tim > >Subject: Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android > > > >On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: > >> Disable the tools / demo code that do not currently build for android > >> until they can be fixed. > > > >What needs to be fixed? > > > > Intel_reg - Relies on a header file not provided by the Android compiler > Intel_display_crc - Cairo dependency Cairo should be available by now on android, at least on some builds. The test at least go to great pains to build if cairo is around. Imo we should adopt it. > Intel_sprite_on - An API difference in the Android i915 driver. This tool is only around for the benefit of the android display folks. If you dont need it then it's time to remove it. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH i-g-t] Android.mk: Disable tools that do not build for android 2015-06-15 13:42 ` Daniel Vetter @ 2015-06-15 14:08 ` Morton, Derek J 0 siblings, 0 replies; 7+ messages in thread From: Morton, Derek J @ 2015-06-15 14:08 UTC (permalink / raw) To: Daniel Vetter; +Cc: Intel Graphics Development, Wood, Thomas > > >-----Original Message----- >From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel Vetter >Sent: Monday, June 15, 2015 2:43 PM >To: Morton, Derek J >Cc: Wood, Thomas; Intel Graphics Development >Subject: Re: [Intel-gfx] [PATCH i-g-t] Android.mk: Disable tools that do not build for android > >On Mon, Jun 15, 2015 at 01:19:16PM +0000, Morton, Derek J wrote: >> > >> >-----Original Message----- >> >From: Thomas Wood [mailto:thomas.wood@intel.com] >> >Sent: Monday, June 15, 2015 2:09 PM >> >To: Morton, Derek J >> >Cc: Intel Graphics Development; Gore, Tim >> >Subject: Re: [PATCH i-g-t] Android.mk: Disable tools that do not >> >build for android >> > >> >On 15 June 2015 at 14:04, Derek Morton <derek.j.morton@intel.com> wrote: >> >> Disable the tools / demo code that do not currently build for >> >> android until they can be fixed. >> > >> >What needs to be fixed? >> > >> >> Intel_reg - Relies on a header file not provided by the Android >> compiler Intel_display_crc - Cairo dependency > >Cairo should be available by now on android, at least on some builds. The test at least go to great pains to build if cairo is around. Imo we should adopt it. Cairo is not present by default on the Gmin build. There is a CAIRO_DEPENDANT flag for building tests. I will update the patch so the tool gets built conditionally with the same flag. > >> Intel_sprite_on - An API difference in the Android i915 driver. > >This tool is only around for the benefit of the android display folks. If you dont need it then it's time to remove it. >-Daniel My aim is to get the vanilla freedesktop code to build on android. Currently intel_sprit_on requires a local patch to build so needs to be disabled by default until that is resolved. //Derek >-- >Daniel Vetter >Software Engineer, Intel Corporation >http://blog.ffwll.ch > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-06-15 14:08 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-06-15 13:04 [PATCH i-g-t] Android.mk: Disable tools that do not build for android Derek Morton 2015-06-15 13:08 ` Thomas Wood 2015-06-15 13:19 ` Morton, Derek J 2015-06-15 13:23 ` Jani Nikula 2015-06-15 13:38 ` Morton, Derek J 2015-06-15 13:42 ` Daniel Vetter 2015-06-15 14:08 ` Morton, Derek J
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox