* [PATCH 0/8] Upstreaming the Android build and misc fixes
@ 2014-07-27 18:25 Emil Velikov
2014-07-27 18:25 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
` (9 more replies)
0 siblings, 10 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Hello list,
Recently I've had a go at the Anroid builds and I felt ... inspired that
there are (at least) two downstream repositories that have the relevant
Android build, yet all of them use 6+month old libdrm.
Making even builds a pain in the neck :'(
Are there any objections if we get the android build upstream ? AFAICS
it's nicely isolated from everything else + I've managed to reuse all
the source/headers lists.
Note that the series lacks a couple of patches from the downstream
repos, yet adds support for radeon, nouveau and freedreno :)
The missing fixes are - s/mmap/mmap64/, dma-bufs support + other intel
specific "hacks". If people are happy with the series then we can take
a look at the final bits.
Cheers,
Emil
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCH 1/8] all: include config.h only when available and use its defines
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
` (8 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
... rather than explicitly redefining HAVE_STDINT_H and _GNU_SOURCE.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
intel/test_decode.c | 5 +++--
libkms/api.c | 2 ++
libkms/dumb.c | 4 +++-
libkms/exynos.c | 4 +++-
libkms/intel.c | 4 +++-
libkms/linux.c | 2 ++
libkms/nouveau.c | 4 +++-
libkms/radeon.c | 4 +++-
libkms/vmwgfx.c | 4 +++-
tests/drmstat.c | 2 ++
tests/modetest/buffers.c | 2 ++
tests/modetest/cursor.c | 2 ++
tests/modetest/modetest.c | 2 ++
tests/vbltest/vbltest.c | 2 ++
14 files changed, 35 insertions(+), 8 deletions(-)
diff --git a/intel/test_decode.c b/intel/test_decode.c
index b710f34..bef9d99 100644
--- a/intel/test_decode.c
+++ b/intel/test_decode.c
@@ -21,7 +21,9 @@
* IN THE SOFTWARE.
*/
-#define _GNU_SOURCE
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#include <string.h>
#include <stdlib.h>
@@ -33,7 +35,6 @@
#include <sys/mman.h>
#include <err.h>
-#include "config.h"
#include "intel_bufmgr.h"
#include "intel_chipset.h"
diff --git a/libkms/api.c b/libkms/api.c
index 5befaa0..b512c42 100644
--- a/libkms/api.c
+++ b/libkms/api.c
@@ -26,7 +26,9 @@
**************************************************************************/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <errno.h>
#include <stdlib.h>
#include <string.h>
diff --git a/libkms/dumb.c b/libkms/dumb.c
index 440efb3..794282f 100644
--- a/libkms/dumb.c
+++ b/libkms/dumb.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/exynos.c b/libkms/exynos.c
index 93e36a1..243915b 100644
--- a/libkms/exynos.c
+++ b/libkms/exynos.c
@@ -11,7 +11,9 @@
* option) any later version.
*/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/intel.c b/libkms/intel.c
index abae452..92f1cf2 100644
--- a/libkms/intel.c
+++ b/libkms/intel.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/linux.c b/libkms/linux.c
index 9b4f29e..17e1d58 100644
--- a/libkms/linux.c
+++ b/libkms/linux.c
@@ -30,7 +30,9 @@
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/libkms/nouveau.c b/libkms/nouveau.c
index 608092f..2de827d 100644
--- a/libkms/nouveau.c
+++ b/libkms/nouveau.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/radeon.c b/libkms/radeon.c
index f5e382a..29375c4 100644
--- a/libkms/radeon.c
+++ b/libkms/radeon.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/vmwgfx.c b/libkms/vmwgfx.c
index d594b3b..598f383 100644
--- a/libkms/vmwgfx.c
+++ b/libkms/vmwgfx.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/tests/drmstat.c b/tests/drmstat.c
index c51cbc6..5935d07 100644
--- a/tests/drmstat.c
+++ b/tests/drmstat.c
@@ -28,7 +28,9 @@
*
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
index 8206ce3..29b520d 100644
--- a/tests/modetest/buffers.c
+++ b/tests/modetest/buffers.c
@@ -24,7 +24,9 @@
* IN THE SOFTWARE.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <errno.h>
diff --git a/tests/modetest/cursor.c b/tests/modetest/cursor.c
index 7077f20..60f240a 100644
--- a/tests/modetest/cursor.c
+++ b/tests/modetest/cursor.c
@@ -22,7 +22,9 @@
* IN THE SOFTWARE.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <errno.h>
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 7d436b5..92efb82 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetest/modetest.c
@@ -37,7 +37,9 @@
* TODO: use cairo to write the mode info on the selected output once
* the mode has been programmed, along with possible test patterns.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <ctype.h>
diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
index 2a09d28..50e29dc 100644
--- a/tests/vbltest/vbltest.c
+++ b/tests/vbltest/vbltest.c
@@ -37,7 +37,9 @@
* TODO: use cairo to write the mode info on the selected output once
* the mode has been programmed, along with possible test patterns.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <stdio.h>
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
2014-07-27 18:25 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
` (7 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
configure.ac has AC_SYS_LARGEFILE which provides the define and/or
approapriate magic when required.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
libkms/dumb.c | 1 -
libkms/exynos.c | 1 -
libkms/intel.c | 1 -
libkms/nouveau.c | 1 -
libkms/radeon.c | 1 -
libkms/vmwgfx.c | 1 -
6 files changed, 6 deletions(-)
diff --git a/libkms/dumb.c b/libkms/dumb.c
index 794282f..5702543 100644
--- a/libkms/dumb.c
+++ b/libkms/dumb.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/exynos.c b/libkms/exynos.c
index 243915b..92e329c 100644
--- a/libkms/exynos.c
+++ b/libkms/exynos.c
@@ -14,7 +14,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/intel.c b/libkms/intel.c
index 92f1cf2..b006ea4 100644
--- a/libkms/intel.c
+++ b/libkms/intel.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/nouveau.c b/libkms/nouveau.c
index 2de827d..15c012e 100644
--- a/libkms/nouveau.c
+++ b/libkms/nouveau.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/radeon.c b/libkms/radeon.c
index 29375c4..938321b 100644
--- a/libkms/radeon.c
+++ b/libkms/radeon.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/vmwgfx.c b/libkms/vmwgfx.c
index 598f383..08163a1 100644
--- a/libkms/vmwgfx.c
+++ b/libkms/vmwgfx.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdlib.h>
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
2014-07-27 18:25 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
2014-07-27 18:25 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
` (6 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Will be used to consolidate the required sources lists as well as the
install-able headers. This is turn will help us to avoid the
duplication with the upcoming Android build support.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Makefile.am | 13 ++++---------
Makefile.sources | 12 ++++++++++++
freedreno/Makefile.am | 24 +++---------------------
freedreno/Makefile.sources | 24 ++++++++++++++++++++++++
include/drm/Makefile.am | 20 ++++----------------
include/drm/Makefile.sources | 18 ++++++++++++++++++
intel/Makefile.am | 16 ++++------------
intel/Makefile.sources | 14 ++++++++++++++
nouveau/Makefile.am | 11 ++++-------
nouveau/Makefile.sources | 9 +++++++++
radeon/Makefile.am | 22 ++++------------------
radeon/Makefile.sources | 19 +++++++++++++++++++
12 files changed, 119 insertions(+), 83 deletions(-)
create mode 100644 Makefile.sources
create mode 100644 freedreno/Makefile.sources
create mode 100644 include/drm/Makefile.sources
create mode 100644 intel/Makefile.sources
create mode 100644 nouveau/Makefile.sources
create mode 100644 radeon/Makefile.sources
diff --git a/Makefile.am b/Makefile.am
index 826c30d..16704df 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -18,6 +18,8 @@
# IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
# CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+include Makefile.sources
+
ACLOCAL_AMFLAGS = -I m4 ${ACLOCAL_FLAGS}
pkgconfigdir = @pkgconfigdir@
@@ -62,17 +64,10 @@ libdrm_la_CPPFLAGS = -I$(top_srcdir)/include/drm
AM_CFLAGS = \
$(VALGRIND_CFLAGS)
-libdrm_la_SOURCES = \
- xf86drm.c \
- xf86drmHash.c \
- xf86drmRandom.c \
- xf86drmSL.c \
- xf86drmMode.c \
- xf86atomic.h \
- libdrm_lists.h
+libdrm_la_SOURCES = $(LIBDRM_FILES)
libdrmincludedir = ${includedir}
-libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
+libdrminclude_HEADERS = $(LIBDRM_HEADERS)
EXTRA_DIST = libdrm.pc.in include/drm/*
diff --git a/Makefile.sources b/Makefile.sources
new file mode 100644
index 0000000..fbf9635
--- /dev/null
+++ b/Makefile.sources
@@ -0,0 +1,12 @@
+LIBDRM_FILES := \
+ xf86drm.c \
+ xf86drmHash.c \
+ xf86drmRandom.c \
+ xf86drmSL.c \
+ xf86drmMode.c \
+ xf86atomic.h \
+ libdrm_lists.h
+
+LIBDRM_HEADERS := \
+ xf86drm.h \
+ xf86drmMode.h
diff --git a/freedreno/Makefile.am b/freedreno/Makefile.am
index 7903e5b..55de79d 100644
--- a/freedreno/Makefile.am
+++ b/freedreno/Makefile.am
@@ -1,4 +1,5 @@
AUTOMAKE_OPTIONS=subdir-objects
+include Makefile.sources
AM_CFLAGS = \
$(WARN_CFLAGS) \
@@ -12,29 +13,10 @@ libdrm_freedreno_ladir = $(libdir)
libdrm_freedreno_la_LDFLAGS = -version-number 1:0:0 -no-undefined
libdrm_freedreno_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_freedreno_la_SOURCES = \
- freedreno_device.c \
- freedreno_pipe.c \
- freedreno_priv.h \
- freedreno_ringbuffer.c \
- freedreno_bo.c \
- kgsl/kgsl_bo.c \
- kgsl/kgsl_device.c \
- kgsl/kgsl_drm.h \
- kgsl/kgsl_pipe.c \
- kgsl/kgsl_priv.h \
- kgsl/kgsl_ringbuffer.c \
- kgsl/msm_kgsl.h \
- msm/msm_bo.c \
- msm/msm_device.c \
- msm/msm_drm.h \
- msm/msm_pipe.c \
- msm/msm_priv.h \
- msm/msm_ringbuffer.c \
- list.h
+libdrm_freedreno_la_SOURCES = $(LIBDRM_FREEDRENO_FILES)
libdrm_freedrenocommonincludedir = ${includedir}/freedreno
-libdrm_freedrenocommoninclude_HEADERS = freedreno_drmif.h freedreno_ringbuffer.h
+libdrm_freedrenocommoninclude_HEADERS = $(LIBDRM_FREEDRENO_HEADERS)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_freedreno.pc
diff --git a/freedreno/Makefile.sources b/freedreno/Makefile.sources
new file mode 100644
index 0000000..f78cd12
--- /dev/null
+++ b/freedreno/Makefile.sources
@@ -0,0 +1,24 @@
+LIBDRM_FREEDRENO_FILES := \
+ freedreno_device.c \
+ freedreno_pipe.c \
+ freedreno_priv.h \
+ freedreno_ringbuffer.c \
+ freedreno_bo.c \
+ kgsl/kgsl_bo.c \
+ kgsl/kgsl_device.c \
+ kgsl/kgsl_drm.h \
+ kgsl/kgsl_pipe.c \
+ kgsl/kgsl_priv.h \
+ kgsl/kgsl_ringbuffer.c \
+ kgsl/msm_kgsl.h \
+ msm/msm_bo.c \
+ msm/msm_device.c \
+ msm/msm_drm.h \
+ msm/msm_pipe.c \
+ msm/msm_priv.h \
+ msm/msm_ringbuffer.c \
+ list.h
+
+LIBDRM_FREEDRENO_HEADERS := \
+ freedreno_drmif.h \
+ freedreno_ringbuffer.h
diff --git a/include/drm/Makefile.am b/include/drm/Makefile.am
index 2bc34d2..83b1eae 100644
--- a/include/drm/Makefile.am
+++ b/include/drm/Makefile.am
@@ -22,23 +22,11 @@
# however, r300 and via need their reg headers installed in order to build.
# better solutions are welcome.
+include Makefile.sources
+
klibdrmincludedir = ${includedir}/libdrm
-klibdrminclude_HEADERS = \
- drm.h \
- drm_mode.h \
- drm_fourcc.h \
- drm_sarea.h \
- i915_drm.h \
- mga_drm.h \
- nouveau_drm.h \
- r128_drm.h \
- radeon_drm.h \
- savage_drm.h \
- sis_drm.h \
- via_drm.h \
- mach64_drm.h \
- qxl_drm.h
+klibdrminclude_HEADERS = $(LIBDRM_INCLUDE_HEADERS)
if HAVE_VMWGFX
-klibdrminclude_HEADERS += vmwgfx_drm.h
+klibdrminclude_HEADERS += $(LIBDRM_INCLUDE_VMWGFX_HEADERS)
endif
diff --git a/include/drm/Makefile.sources b/include/drm/Makefile.sources
new file mode 100644
index 0000000..474d4c1
--- /dev/null
+++ b/include/drm/Makefile.sources
@@ -0,0 +1,18 @@
+LIBDRM_INCLUDE_HEADERS := \
+ drm.h \
+ drm_mode.h \
+ drm_fourcc.h \
+ drm_sarea.h \
+ i915_drm.h \
+ mga_drm.h \
+ nouveau_drm.h \
+ r128_drm.h \
+ radeon_drm.h \
+ savage_drm.h \
+ sis_drm.h \
+ via_drm.h \
+ mach64_drm.h \
+ qxl_drm.h
+
+LIBDRM_INCLUDE_VMWGFX_HEADERS := \
+ vmwgfx_drm.h
diff --git a/intel/Makefile.am b/intel/Makefile.am
index f49b099..49b41a8 100644
--- a/intel/Makefile.am
+++ b/intel/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Eric Anholt <eric@anholt.net>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -39,22 +41,12 @@ libdrm_intel_la_LIBADD = ../libdrm.la \
@PCIACCESS_LIBS@ \
@CLOCK_LIB@
-libdrm_intel_la_SOURCES = \
- intel_bufmgr.c \
- intel_bufmgr_priv.h \
- intel_bufmgr_fake.c \
- intel_bufmgr_gem.c \
- intel_decode.c \
- intel_chipset.h \
- mm.c \
- mm.h
+libdrm_intel_la_SOURCES = $(LIBDRM_INTEL_FILES)
intel_bufmgr_gem_o_CFLAGS = $(AM_CFLAGS) -c99
libdrm_intelincludedir = ${includedir}/libdrm
-libdrm_intelinclude_HEADERS = intel_bufmgr.h \
- intel_aub.h \
- intel_debug.h
+libdrm_intelinclude_HEADERS = $(LIBDRM_INTEL_HEADERS)
# This may be interesting even outside of "make check", due to the -dump option.
noinst_PROGRAMS = test_decode
diff --git a/intel/Makefile.sources b/intel/Makefile.sources
new file mode 100644
index 0000000..f1ae351
--- /dev/null
+++ b/intel/Makefile.sources
@@ -0,0 +1,14 @@
+LIBDRM_INTEL_FILES := \
+ intel_bufmgr.c \
+ intel_bufmgr_priv.h \
+ intel_bufmgr_fake.c \
+ intel_bufmgr_gem.c \
+ intel_decode.c \
+ intel_chipset.h \
+ mm.c \
+ mm.h
+
+LIBDRM_INTEL_HEADERS := \
+ intel_bufmgr.h \
+ intel_aub.h \
+ intel_debug.h
diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
index 206e892..47c1eb2 100644
--- a/nouveau/Makefile.am
+++ b/nouveau/Makefile.am
@@ -1,3 +1,5 @@
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -11,15 +13,10 @@ libdrm_nouveau_ladir = $(libdir)
libdrm_nouveau_la_LDFLAGS = -version-number 2:0:0 -no-undefined
libdrm_nouveau_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_nouveau_la_SOURCES = nouveau.c \
- pushbuf.c \
- bufctx.c \
- abi16.c \
- private.h
-
+libdrm_nouveau_la_SOURCES = $(LIBDRM_NOUVEAU_FILES)
libdrm_nouveauincludedir = ${includedir}/libdrm
-libdrm_nouveauinclude_HEADERS = nouveau.h
+libdrm_nouveauinclude_HEADERS = $(LIBDRM_NOUVEAU_HEADERS)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_nouveau.pc
diff --git a/nouveau/Makefile.sources b/nouveau/Makefile.sources
new file mode 100644
index 0000000..220d44b
--- /dev/null
+++ b/nouveau/Makefile.sources
@@ -0,0 +1,9 @@
+LIBDRM_NOUVEAU_FILES := \
+ nouveau.c \
+ pushbuf.c \
+ bufctx.c \
+ abi16.c \
+ private.h
+
+LIBDRM_NOUVEAU_HEADERS := \
+ nouveau.h
diff --git a/radeon/Makefile.am b/radeon/Makefile.am
index a8cd100..a99a625 100644
--- a/radeon/Makefile.am
+++ b/radeon/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Jérôme Glisse <glisse@freedesktop.org>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -34,26 +36,10 @@ libdrm_radeon_ladir = $(libdir)
libdrm_radeon_la_LDFLAGS = -version-number 1:0:1 -no-undefined
libdrm_radeon_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_radeon_la_SOURCES = \
- radeon_bo_gem.c \
- radeon_cs_gem.c \
- radeon_cs_space.c \
- radeon_bo.c \
- radeon_cs.c \
- radeon_surface.c \
- bof.c \
- bof.h
+libdrm_radeon_la_SOURCES = $(LIBDRM_RADEON_FILES)
libdrm_radeonincludedir = ${includedir}/libdrm
-libdrm_radeoninclude_HEADERS = \
- radeon_bo.h \
- radeon_cs.h \
- radeon_surface.h \
- radeon_bo_gem.h \
- radeon_cs_gem.h \
- radeon_bo_int.h \
- radeon_cs_int.h \
- r600_pci_ids.h
+libdrm_radeoninclude_HEADERS = $(LIBDRM_RADEON_HEADERS)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_radeon.pc
diff --git a/radeon/Makefile.sources b/radeon/Makefile.sources
new file mode 100644
index 0000000..ff0e1f6
--- /dev/null
+++ b/radeon/Makefile.sources
@@ -0,0 +1,19 @@
+LIBDRM_RADEON_FILES := \
+ radeon_bo_gem.c \
+ radeon_cs_gem.c \
+ radeon_cs_space.c \
+ radeon_bo.c \
+ radeon_cs.c \
+ radeon_surface.c \
+ bof.c \
+ bof.h
+
+LIBDRM_RADEON_HEADERS := \
+ radeon_bo.h \
+ radeon_cs.h \
+ radeon_surface.h \
+ radeon_bo_gem.h \
+ radeon_cs_gem.h \
+ radeon_bo_int.h \
+ radeon_cs_int.h \
+ r600_pci_ids.h
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 4/8] libdrm,intel: Add Android build
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (2 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
` (5 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Contains the following patches squashed in:
commit f340a8b9f2b84d5762553bef046914e0bde20795
Author: Chad Versace <chad.versace@linux.intel.com>
Date: Wed, 21 Dec 2011 11:43:57 -0800
libdrm,intel: Add Android makefiles (v2)
This enables libdrm.so and libdrm_intel.so to build on Android
IceCreamSandwich.
v2: Link libdrm_intel to libpciaccess.
Change-Id: Ie5ed4bc0e6b4f9f819e3ec44488e385c35e97128
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
commit 8fb3f42389dea34218ed1fe59550ec2abb4d6953
Author: Andrew Boie <andrew.p.boie@intel.com>
Date: Wed, 26 Sep 2012 13:32:05 -0700
libdrm, libdrm_intel: Skip driver name checks
These libraries have 'optional' tags, which means they won't get
built unless something else depends on them or they are added to
PRODUCT_PACKAGES. There's no need for additional filtering.
Change-Id: I5d90969f38671f8144c0dc27d47144b3f09a15ce
Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>
---
Android.mk | 45 +++++++++++++++++++++++++++++++++++++++++++++
intel/Android.mk | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 95 insertions(+)
create mode 100644 Android.mk
create mode 100644 intel/Android.mk
diff --git a/Android.mk b/Android.mk
new file mode 100644
index 0000000..afe59ce
--- /dev/null
+++ b/Android.mk
@@ -0,0 +1,45 @@
+#
+# Copyright © 2011 Intel Corporation
+#
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice (including the next
+# paragraph) shall be included in all copies or substantial portions of the
+# Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+# IN THE SOFTWARE.
+#
+
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+LIBDRM_TOP := $(LOCAL_PATH)
+
+# Import variables LIBDRM_FILES, LIBDRM_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SRC_FILES := $(LIBDRM_FILES)
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+include $(BUILD_SHARED_LIBRARY)
+
+include $(LOCAL_PATH)/intel/Android.mk
diff --git a/intel/Android.mk b/intel/Android.mk
new file mode 100644
index 0000000..5c48a95
--- /dev/null
+++ b/intel/Android.mk
@@ -0,0 +1,50 @@
+#
+# Copyright © 2011 Intel Corporation
+#
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice (including the next
+# paragraph) shall be included in all copies or substantial portions of the
+# Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+# IN THE SOFTWARE.
+#
+
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_INTEL_FILES, LIBDRM_INTEL_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_intel
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_INTEL_FILES)
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/intel \
+ $(LIBDRM_TOP)/include/drm \
+ external/libpciaccess/include
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm \
+ libpciaccess
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 5/8] libdrm,intel: rework android header handling
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (3 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
` (4 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Contains the following patches squashed in:
commit 99247a5bd724ddcf0f06a5518baad207c53f1e2b
Author: Haitao Huang <haitao.huang@intel.com>
Date: Fri, 27 Apr 2012 13:20:53 -0500
Android.mk: use LOCAL_COPY_HEADERS to export headers.
Export necessary header files used by other components for
Android, such as libva intel-driver, gralloc, hwcomposer, etc.
Change-Id: I2feabf6941379ef4d756e942f30eba059de641f1
Signed-off-by: Haitao Huang <haitao.huang@intel.com>
[chad: Fixed inconsistent indentation.]
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
commit 7d0b528cb69995d7ea4e29b2daa1e3b28a362f42
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date: Sun, 27 Jul 2014 18:22:41 +0100
android: reuse headers lists, separate libdrm from intel headers
Rather than having a duplicate copy of the headers list(s),
reuse the existing one(s). Distinguish that the intel headers
should be copied when libdrm_intel is used.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
commit 361de3ba4cadd5357596d1537bb3f216d281532b
Author: Piotr Luc <piotr.luc@intel.com>
Date: Fri, 14 Jun 2013 13:00:39 +0200
Export include dir from libdrm
BZ: 116218
Google introduced new method of specifying include path(s)
between modules. This allows a module to include header from a
library without directly specifyining by includer the path where
headers are located.
The method requires from library that holds headers to export
include path(s) in LOCAL_EXPORT_C_INCLUDE_DIRS variable.
These exported include path(s) are automatically added to
include path(s) of modules that have name of the library in the
LOCAL_SHARED_LIBRARIES or LOCAL_STATIC_LIBRARIES list.
This change sets LOCAL_EXPORT_C_INCLUDE_DIRS to folders that
contain headers file that used by other modules in order to
export these paths.
Change-Id: Id1ac885b31ef2efe194e0289fbcaecd9eb533df0
Signed-off-by: Piotr Luc <piotr.luc@intel.com>
Reviewed-on: http://android.intel.com:8080/113562
Reviewed-by: cactus <cactus@intel.com>
Reviewed-by: Luc, Piotr <Piotr.Luc@intel.com>
Reviewed-by: Purushothaman, Vijay A <vijay.a.purushothaman@intel.com>
Reviewed-by: Stimson, Dale B <dale.b.stimson@intel.com>
Tested-by: Stimson, Dale B <dale.b.stimson@intel.com>
Reviewed-by: buildbot <buildbot@intel.com>
Tested-by: buildbot <buildbot@intel.com>
commit 2bf22fcbd4cbb9e7c7764d5eff0bb4e75ab1a005
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date: 27 Jul 2014 18:27:21 +0100
android: Separate libdrm and intel LOCAL_EXPORT_C_INCLUDE_DIRS
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 13 ++++++++++++-
intel/Android.mk | 5 +++++
2 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/Android.mk b/Android.mk
index afe59ce..ec340b4 100644
--- a/Android.mk
+++ b/Android.mk
@@ -1,5 +1,5 @@
#
-# Copyright © 2011 Intel Corporation
+# Copyright © 2011-2012 Intel Corporation
#
# Permission is hereby granted, free of charge, to any person obtaining a
# copy of this software and associated documentation files (the "Software"),
@@ -28,11 +28,16 @@ LIBDRM_TOP := $(LOCAL_PATH)
# Import variables LIBDRM_FILES, LIBDRM_HEADERS
include $(LOCAL_PATH)/Makefile.sources
+# Import variables LIBDRM_INCLUDE_HEADERS, LIBDRM_INCLUDE_VMWGFX_HEADERS
+include $(LOCAL_PATH)/include/drm/Makefile.sources
LOCAL_MODULE := libdrm
LOCAL_MODULE_TAGS := optional
LOCAL_SRC_FILES := $(LIBDRM_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH) \
+ $(LOCAL_PATH)/include/drm
LOCAL_C_INCLUDES := \
$(LIBDRM_TOP)/include/drm
@@ -40,6 +45,12 @@ LOCAL_C_INCLUDES := \
LOCAL_CFLAGS := \
-DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+LOCAL_COPY_HEADERS := \
+ $(LIBDRM_HEADERS) \
+ $(addprefix include/drm/,$(LIBDRM_INCLUDE_HEADERS)) \
+ $(addprefix include/drm/,$(LIBDRM_INCLUDE_VMWGFX_HEADERS))
+
+LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
include $(LOCAL_PATH)/intel/Android.mk
diff --git a/intel/Android.mk b/intel/Android.mk
index 5c48a95..8f6d6e3 100644
--- a/intel/Android.mk
+++ b/intel/Android.mk
@@ -33,6 +33,8 @@ LOCAL_MODULE_TAGS := optional
LOCAL_SHARED_LIBRARIES := libdrm
LOCAL_SRC_FILES := $(LIBDRM_INTEL_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/intel
LOCAL_C_INCLUDES := \
$(LIBDRM_TOP) \
@@ -43,6 +45,9 @@ LOCAL_C_INCLUDES := \
LOCAL_CFLAGS := \
-DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+LOCAL_COPY_HEADERS := $(LIBDRM_INTEL_HEADERS)
+LOCAL_COPY_HEADERS_TO := libdrm
+
LOCAL_SHARED_LIBRARIES := \
libdrm \
libpciaccess
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 6/8] radeon: add Android build support
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (4 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 7/8] nouveau: " Emil Velikov
` (3 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 1 +
radeon/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 31 insertions(+)
create mode 100644 radeon/Android.mk
diff --git a/Android.mk b/Android.mk
index ec340b4..b3fbb5b 100644
--- a/Android.mk
+++ b/Android.mk
@@ -54,3 +54,4 @@ LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
include $(LOCAL_PATH)/intel/Android.mk
+include $(LOCAL_PATH)/radeon/Android.mk
diff --git a/radeon/Android.mk b/radeon/Android.mk
new file mode 100644
index 0000000..8c267df
--- /dev/null
+++ b/radeon/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_RADEON_FILES, LIBDRM_RADEON_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_radeon
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_RADEON_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/radeon
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/radeon \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_RADEON_HEADERS)
+LOCAL_COPY_HEADERS_TO := libdrm
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 7/8] nouveau: add Android build support
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (5 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-27 18:25 ` [PATCH 8/8] freedreno: " Emil Velikov
` (2 subsequent siblings)
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 1 +
nouveau/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 31 insertions(+)
create mode 100644 nouveau/Android.mk
diff --git a/Android.mk b/Android.mk
index b3fbb5b..eb54bf7 100644
--- a/Android.mk
+++ b/Android.mk
@@ -54,4 +54,5 @@ LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
include $(LOCAL_PATH)/intel/Android.mk
+include $(LOCAL_PATH)/nouveau/Android.mk
include $(LOCAL_PATH)/radeon/Android.mk
diff --git a/nouveau/Android.mk b/nouveau/Android.mk
new file mode 100644
index 0000000..28350bd
--- /dev/null
+++ b/nouveau/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_NOUVEAU_FILES, LIBDRM_NOUVEAU_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_nouveau
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_NOUVEAU_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/nouveau
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/nouveau \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_NOUVEAU_HEADERS)
+LOCAL_COPY_HEADERS_TO := libdrm
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 8/8] freedreno: add Android build support
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (6 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 7/8] nouveau: " Emil Velikov
@ 2014-07-27 18:25 ` Emil Velikov
2014-07-28 2:48 ` [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
9 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-27 18:25 UTC (permalink / raw)
To: dri-devel; +Cc: freedreno, emil.l.velikov
Cc: freedreno@lists.freedesktop.org
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 1 +
freedreno/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 31 insertions(+)
create mode 100644 freedreno/Android.mk
diff --git a/Android.mk b/Android.mk
index eb54bf7..b3bf411 100644
--- a/Android.mk
+++ b/Android.mk
@@ -53,6 +53,7 @@ LOCAL_COPY_HEADERS := \
LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
+include $(LOCAL_PATH)/freedreno/Android.mk
include $(LOCAL_PATH)/intel/Android.mk
include $(LOCAL_PATH)/nouveau/Android.mk
include $(LOCAL_PATH)/radeon/Android.mk
diff --git a/freedreno/Android.mk b/freedreno/Android.mk
new file mode 100644
index 0000000..e91b623
--- /dev/null
+++ b/freedreno/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_FREEDRENO_FILES, LIBDRM_FREEDRENO_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_freedreno
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_FREEDRENO_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/freedreno
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/freedreno \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_FREEDRENO_HEADERS)
+LOCAL_COPY_HEADERS_TO := libdrm
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (7 preceding siblings ...)
2014-07-27 18:25 ` [PATCH 8/8] freedreno: " Emil Velikov
@ 2014-07-28 2:48 ` Emil Velikov
2014-07-28 7:07 ` Daniel Vetter
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
9 siblings, 1 reply; 31+ messages in thread
From: Emil Velikov @ 2014-07-28 2:48 UTC (permalink / raw)
To: dri-devel@lists.freedesktop.org; +Cc: emil.l.velikov
A few updates:
- Naming the headers lists *_HEADERS caused autohell to hate us. Renamed to
*_H_FILES
- Including the platform Android.mk files individually is not the right way
to do. One needs to construct an array/list of Android.mks and include it.
- The series including the above fixes can be found in branch fixes+android
over at https://github.com/evelikov/libdrm.
-Emil
On 27/07/14 19:25, Emil Velikov wrote:
> Hello list,
>
> Recently I've had a go at the Anroid builds and I felt ... inspired that
> there are (at least) two downstream repositories that have the relevant
> Android build, yet all of them use 6+month old libdrm.
> Making even builds a pain in the neck :'(
>
> Are there any objections if we get the android build upstream ? AFAICS
> it's nicely isolated from everything else + I've managed to reuse all
> the source/headers lists.
>
> Note that the series lacks a couple of patches from the downstream
> repos, yet adds support for radeon, nouveau and freedreno :)
>
> The missing fixes are - s/mmap/mmap64/, dma-bufs support + other intel
> specific "hacks". If people are happy with the series then we can take
> a look at the final bits.
>
>
> Cheers,
> Emil
>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-28 2:48 ` [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
@ 2014-07-28 7:07 ` Daniel Vetter
2014-07-28 12:01 ` Emil Velikov
0 siblings, 1 reply; 31+ messages in thread
From: Daniel Vetter @ 2014-07-28 7:07 UTC (permalink / raw)
To: Emil Velikov, Gore, Tim; +Cc: dri-devel@lists.freedesktop.org
On Mon, Jul 28, 2014 at 03:48:53AM +0100, Emil Velikov wrote:
> A few updates:
>
> - Naming the headers lists *_HEADERS caused autohell to hate us. Renamed to
> *_H_FILES
> - Including the platform Android.mk files individually is not the right way
> to do. One needs to construct an array/list of Android.mks and include it.
>
> - The series including the above fixes can be found in branch fixes+android
> over at https://github.com/evelikov/libdrm.
Adding Tim Gore who's working on Android.mk support for i-g-t from our
side and probably knows whom to poke for the intel side of things for
libdrm Android ports.
-Daniel
>
> -Emil
>
> On 27/07/14 19:25, Emil Velikov wrote:
> > Hello list,
> >
> > Recently I've had a go at the Anroid builds and I felt ... inspired that
> > there are (at least) two downstream repositories that have the relevant
> > Android build, yet all of them use 6+month old libdrm.
> > Making even builds a pain in the neck :'(
> >
> > Are there any objections if we get the android build upstream ? AFAICS
> > it's nicely isolated from everything else + I've managed to reuse all
> > the source/headers lists.
> >
> > Note that the series lacks a couple of patches from the downstream
> > repos, yet adds support for radeon, nouveau and freedreno :)
> >
> > The missing fixes are - s/mmap/mmap64/, dma-bufs support + other intel
> > specific "hacks". If people are happy with the series then we can take
> > a look at the final bits.
> >
> >
> > Cheers,
> > Emil
> >
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-28 7:07 ` Daniel Vetter
@ 2014-07-28 12:01 ` Emil Velikov
2014-07-28 15:22 ` Daniel Vetter
0 siblings, 1 reply; 31+ messages in thread
From: Emil Velikov @ 2014-07-28 12:01 UTC (permalink / raw)
To: Daniel Vetter, Gore, Tim; +Cc: emil.l.velikov, dri-devel@lists.freedesktop.org
On 28/07/14 08:07, Daniel Vetter wrote:
> On Mon, Jul 28, 2014 at 03:48:53AM +0100, Emil Velikov wrote:
>> A few updates:
>>
>> - Naming the headers lists *_HEADERS caused autohell to hate us. Renamed to
>> *_H_FILES
>> - Including the platform Android.mk files individually is not the right way
>> to do. One needs to construct an array/list of Android.mks and include it.
>>
>> - The series including the above fixes can be found in branch fixes+android
>> over at https://github.com/evelikov/libdrm.
>
> Adding Tim Gore who's working on Android.mk support for i-g-t from our
> side and probably knows whom to poke for the intel side of things for
> libdrm Android ports.
> -Daniel
>
Thank you Daniel,
In case it was not clear enough, some of these patches are taken from
android-ia/external/drm. The very same are written by Intel employees AFAICT
:) Would be great to hear if anyone is against the idea of getting Android.mks
in the canonical repo.
-Emil
>>
>> -Emil
>>
>> On 27/07/14 19:25, Emil Velikov wrote:
>>> Hello list,
>>>
>>> Recently I've had a go at the Anroid builds and I felt ... inspired that
>>> there are (at least) two downstream repositories that have the relevant
>>> Android build, yet all of them use 6+month old libdrm.
>>> Making even builds a pain in the neck :'(
>>>
>>> Are there any objections if we get the android build upstream ? AFAICS
>>> it's nicely isolated from everything else + I've managed to reuse all
>>> the source/headers lists.
>>>
>>> Note that the series lacks a couple of patches from the downstream
>>> repos, yet adds support for radeon, nouveau and freedreno :)
>>>
>>> The missing fixes are - s/mmap/mmap64/, dma-bufs support + other intel
>>> specific "hacks". If people are happy with the series then we can take
>>> a look at the final bits.
>>>
>>>
>>> Cheers,
>>> Emil
>>>
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-28 12:01 ` Emil Velikov
@ 2014-07-28 15:22 ` Daniel Vetter
2014-07-29 16:14 ` Gore, Tim
0 siblings, 1 reply; 31+ messages in thread
From: Daniel Vetter @ 2014-07-28 15:22 UTC (permalink / raw)
To: Emil Velikov; +Cc: dri-devel@lists.freedesktop.org
On Mon, Jul 28, 2014 at 01:01:19PM +0100, Emil Velikov wrote:
> On 28/07/14 08:07, Daniel Vetter wrote:
> > On Mon, Jul 28, 2014 at 03:48:53AM +0100, Emil Velikov wrote:
> >> A few updates:
> >>
> >> - Naming the headers lists *_HEADERS caused autohell to hate us. Renamed to
> >> *_H_FILES
> >> - Including the platform Android.mk files individually is not the right way
> >> to do. One needs to construct an array/list of Android.mks and include it.
> >>
> >> - The series including the above fixes can be found in branch fixes+android
> >> over at https://github.com/evelikov/libdrm.
> >
> > Adding Tim Gore who's working on Android.mk support for i-g-t from our
> > side and probably knows whom to poke for the intel side of things for
> > libdrm Android ports.
> > -Daniel
> >
> Thank you Daniel,
>
> In case it was not clear enough, some of these patches are taken from
> android-ia/external/drm. The very same are written by Intel employees AFAICT
> :) Would be great to hear if anyone is against the idea of getting Android.mks
> in the canonical repo.
Oh, that's kinda why I want to drag the relevant people in from Intel's
side. Responsibility for Android builds have shifted around a bit the past
few years and Intel is big, so I'm trying to get hold off the right
person. No success thus far :(
But personally I want this, just need to make sure that our own Android
guys see it and can start to help out. Occasionally it takes a while until
they dare to walk out of their hidings ;-)
-Daniel
>
>
> -Emil
>
> >>
> >> -Emil
> >>
> >> On 27/07/14 19:25, Emil Velikov wrote:
> >>> Hello list,
> >>>
> >>> Recently I've had a go at the Anroid builds and I felt ... inspired that
> >>> there are (at least) two downstream repositories that have the relevant
> >>> Android build, yet all of them use 6+month old libdrm.
> >>> Making even builds a pain in the neck :'(
> >>>
> >>> Are there any objections if we get the android build upstream ? AFAICS
> >>> it's nicely isolated from everything else + I've managed to reuse all
> >>> the source/headers lists.
> >>>
> >>> Note that the series lacks a couple of patches from the downstream
> >>> repos, yet adds support for radeon, nouveau and freedreno :)
> >>>
> >>> The missing fixes are - s/mmap/mmap64/, dma-bufs support + other intel
> >>> specific "hacks". If people are happy with the series then we can take
> >>> a look at the final bits.
> >>>
> >>>
> >>> Cheers,
> >>> Emil
> >>>
> >>
> >> _______________________________________________
> >> dri-devel mailing list
> >> dri-devel@lists.freedesktop.org
> >> http://lists.freedesktop.org/mailman/listinfo/dri-devel
> >
>
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-28 15:22 ` Daniel Vetter
@ 2014-07-29 16:14 ` Gore, Tim
2014-07-29 17:10 ` Emil Velikov
0 siblings, 1 reply; 31+ messages in thread
From: Gore, Tim @ 2014-07-29 16:14 UTC (permalink / raw)
To: Daniel Vetter, Emil Velikov; +Cc: dri-devel@lists.freedesktop.org
> -----Original Message-----
> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel
> Vetter
> Sent: Monday, July 28, 2014 4:22 PM
> To: Emil Velikov
> Cc: Daniel Vetter; Gore, Tim; dri-devel@lists.freedesktop.org
> Subject: Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
>
> On Mon, Jul 28, 2014 at 01:01:19PM +0100, Emil Velikov wrote:
> > On 28/07/14 08:07, Daniel Vetter wrote:
> > > On Mon, Jul 28, 2014 at 03:48:53AM +0100, Emil Velikov wrote:
> > >> A few updates:
> > >>
> > >> - Naming the headers lists *_HEADERS caused autohell to hate us.
> > >> Renamed to *_H_FILES
> > >> - Including the platform Android.mk files individually is not the
> > >> right way to do. One needs to construct an array/list of Android.mks and
> include it.
> > >>
> > >> - The series including the above fixes can be found in branch
> > >> fixes+android over at https://github.com/evelikov/libdrm.
> > >
> > > Adding Tim Gore who's working on Android.mk support for i-g-t from
> > > our side and probably knows whom to poke for the intel side of
> > > things for libdrm Android ports.
> > > -Daniel
> > >
> > Thank you Daniel,
> >
> > In case it was not clear enough, some of these patches are taken from
> > android-ia/external/drm. The very same are written by Intel employees
> > AFAICT
> > :) Would be great to hear if anyone is against the idea of getting
> > Android.mks in the canonical repo.
>
> Oh, that's kinda why I want to drag the relevant people in from Intel's side.
> Responsibility for Android builds have shifted around a bit the past few years
> and Intel is big, so I'm trying to get hold off the right person. No success thus
> far :(
>
> But personally I want this, just need to make sure that our own Android guys
> see it and can start to help out. Occasionally it takes a while until they dare to
> walk out of their hidings ;-) -Daniel
On the whole these look fine. Jon Bloomfield seemed happy with the overall idea.
3 comments.
1) Patch 3 didn't apply cleanly, I assume because it
was based on a different branch (ie not master), but the difference was trivial.
2) The Android makefiles as they are will not build within the android tree. I am
Trying to get them to work at the moment.
3) Depending on which Android tree you have, the resulting libdrm may or may not
Work in there. I don't think the latest intel android tree is compatible with the
Upstream libdrm.
Tim
> >
> >
> > -Emil
> >
> > >>
> > >> -Emil
> > >>
> > >> On 27/07/14 19:25, Emil Velikov wrote:
> > >>> Hello list,
> > >>>
> > >>> Recently I've had a go at the Anroid builds and I felt ...
> > >>> inspired that there are (at least) two downstream repositories
> > >>> that have the relevant Android build, yet all of them use 6+month old
> libdrm.
> > >>> Making even builds a pain in the neck :'(
> > >>>
> > >>> Are there any objections if we get the android build upstream ?
> > >>> AFAICS it's nicely isolated from everything else + I've managed to
> > >>> reuse all the source/headers lists.
> > >>>
> > >>> Note that the series lacks a couple of patches from the downstream
> > >>> repos, yet adds support for radeon, nouveau and freedreno :)
> > >>>
> > >>> The missing fixes are - s/mmap/mmap64/, dma-bufs support + other
> > >>> intel specific "hacks". If people are happy with the series then
> > >>> we can take a look at the final bits.
> > >>>
> > >>>
> > >>> Cheers,
> > >>> Emil
> > >>>
> > >>
> > >> _______________________________________________
> > >> dri-devel mailing list
> > >> dri-devel@lists.freedesktop.org
> > >> http://lists.freedesktop.org/mailman/listinfo/dri-devel
> > >
> >
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [PATCH 0/8] Upstreaming the Android build and misc fixes
@ 2014-07-29 16:43 Gore, Tim
0 siblings, 0 replies; 31+ messages in thread
From: Gore, Tim @ 2014-07-29 16:43 UTC (permalink / raw)
To: Emil Velikov (emil.l.velikov@gmail.com); +Cc: dri-devel@lists.freedesktop.org
Hi Emil, these seemed to work ok, with two fixes.
1) Patch 3 didn't apply cleanly to the master branch due to a difference in the
Author email address for a couple of files.
2) The Android.mk at the top level, at the end where you include the sub makefiles,
You need to use LIBDRM_TOP instead of LOCAL_PATH. (Patches 4,6,7,8) So
include $(LOCAL_PATH)/.....
Becomes
include $(LIBDRM_TOP)/....
this is because the included Android.mk files modify LOCAL_PATH
TIm
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
2014-07-29 16:14 ` Gore, Tim
@ 2014-07-29 17:10 ` Emil Velikov
0 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:10 UTC (permalink / raw)
To: Gore, Tim, Daniel Vetter; +Cc: emil.l.velikov, dri-devel@lists.freedesktop.org
On 29/07/14 17:14, Gore, Tim wrote:
>
>
>> -----Original Message-----
>> From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel
>> Vetter
>> Sent: Monday, July 28, 2014 4:22 PM
>> To: Emil Velikov
>> Cc: Daniel Vetter; Gore, Tim; dri-devel@lists.freedesktop.org
>> Subject: Re: [PATCH 0/8] Upstreaming the Android build and misc fixes
>>
>> On Mon, Jul 28, 2014 at 01:01:19PM +0100, Emil Velikov wrote:
>>> On 28/07/14 08:07, Daniel Vetter wrote:
>>>> On Mon, Jul 28, 2014 at 03:48:53AM +0100, Emil Velikov wrote:
>>>>> A few updates:
>>>>>
>>>>> - Naming the headers lists *_HEADERS caused autohell to hate us.
>>>>> Renamed to *_H_FILES
>>>>> - Including the platform Android.mk files individually is not the
>>>>> right way to do. One needs to construct an array/list of Android.mks and
>> include it.
>>>>>
>>>>> - The series including the above fixes can be found in branch
>>>>> fixes+android over at https://github.com/evelikov/libdrm.
>>>>
>>>> Adding Tim Gore who's working on Android.mk support for i-g-t from
>>>> our side and probably knows whom to poke for the intel side of
>>>> things for libdrm Android ports.
>>>> -Daniel
>>>>
>>> Thank you Daniel,
>>>
>>> In case it was not clear enough, some of these patches are taken from
>>> android-ia/external/drm. The very same are written by Intel employees
>>> AFAICT
>>> :) Would be great to hear if anyone is against the idea of getting
>>> Android.mks in the canonical repo.
>>
>> Oh, that's kinda why I want to drag the relevant people in from Intel's side.
>> Responsibility for Android builds have shifted around a bit the past few years
>> and Intel is big, so I'm trying to get hold off the right person. No success thus
>> far :(
>>
>> But personally I want this, just need to make sure that our own Android guys
>> see it and can start to help out. Occasionally it takes a while until they dare to
>> walk out of their hidings ;-) -Daniel
>
> On the whole these look fine. Jon Bloomfield seemed happy with the overall idea.
> 3 comments.
> 1) Patch 3 didn't apply cleanly, I assume because it
> was based on a different branch (ie not master), but the difference was trivial.
> 2) The Android makefiles as they are will not build within the android tree. I am
> Trying to get them to work at the moment.
The Android issue you've mentioned in the other email is fixed in the branch
mentioned above + I have squashed additional fixes over at fixes+android-v2 in
https://github.com/evelikov/libdrm. Will post these in just a moment.
> 3) Depending on which Android tree you have, the resulting libdrm may or may not
> Work in there. I don't think the latest intel android tree is compatible with the
> Upstream libdrm.
>
Not sure what is the exact meaning of "compatible" is in this case. IMHO Intel
(if interested) can easily rebase their repo on top of upstream (as these
land) thus keep a smaller (if any) delta and ease development :)
-Emil
P.S. Who is Jon Bloomfield ?
> Tim
>
>>>
>>>
>>> -Emil
>>>
>>>>>
>>>>> -Emil
>>>>>
>>>>> On 27/07/14 19:25, Emil Velikov wrote:
>>>>>> Hello list,
>>>>>>
>>>>>> Recently I've had a go at the Anroid builds and I felt ...
>>>>>> inspired that there are (at least) two downstream repositories
>>>>>> that have the relevant Android build, yet all of them use 6+month old
>> libdrm.
>>>>>> Making even builds a pain in the neck :'(
>>>>>>
>>>>>> Are there any objections if we get the android build upstream ?
>>>>>> AFAICS it's nicely isolated from everything else + I've managed to
>>>>>> reuse all the source/headers lists.
>>>>>>
>>>>>> Note that the series lacks a couple of patches from the downstream
>>>>>> repos, yet adds support for radeon, nouveau and freedreno :)
>>>>>>
>>>>>> The missing fixes are - s/mmap/mmap64/, dma-bufs support + other
>>>>>> intel specific "hacks". If people are happy with the series then
>>>>>> we can take a look at the final bits.
>>>>>>
>>>>>>
>>>>>> Cheers,
>>>>>> Emil
>>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> dri-devel mailing list
>>>>> dri-devel@lists.freedesktop.org
>>>>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>>>>
>>>
>>
>> --
>> Daniel Vetter
>> Software Engineer, Intel Corporation
>> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
` (8 preceding siblings ...)
2014-07-28 2:48 ` [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
` (8 more replies)
9 siblings, 9 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel
Changes since the orignal posting:
- Rebased on top of master.
- Used _H_FILES for header lists (_HEADERS is a no-go with autotools)
- Install the freedreno headers to {include_dir}/freedreno, similar to
the automake builds.
- Correctly include $(hw)/Android.mk
The series is also available at the fixes+android-v2 branch in
https://github.com/evelikov/libdrm.
-Emil
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCH 1/8] all: include config.h only when available and use its defines
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
` (7 subsequent siblings)
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
... rather than explicitly redefining HAVE_STDINT_H and _GNU_SOURCE.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
intel/test_decode.c | 5 +++--
libkms/api.c | 2 ++
libkms/dumb.c | 4 +++-
libkms/exynos.c | 4 +++-
libkms/intel.c | 4 +++-
libkms/linux.c | 2 ++
libkms/nouveau.c | 4 +++-
libkms/radeon.c | 4 +++-
libkms/vmwgfx.c | 4 +++-
tests/drmstat.c | 2 ++
tests/modetest/buffers.c | 2 ++
tests/modetest/cursor.c | 2 ++
tests/modetest/modetest.c | 2 ++
tests/vbltest/vbltest.c | 2 ++
14 files changed, 35 insertions(+), 8 deletions(-)
diff --git a/intel/test_decode.c b/intel/test_decode.c
index b710f34..bef9d99 100644
--- a/intel/test_decode.c
+++ b/intel/test_decode.c
@@ -21,7 +21,9 @@
* IN THE SOFTWARE.
*/
-#define _GNU_SOURCE
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#include <string.h>
#include <stdlib.h>
@@ -33,7 +35,6 @@
#include <sys/mman.h>
#include <err.h>
-#include "config.h"
#include "intel_bufmgr.h"
#include "intel_chipset.h"
diff --git a/libkms/api.c b/libkms/api.c
index 5befaa0..b512c42 100644
--- a/libkms/api.c
+++ b/libkms/api.c
@@ -26,7 +26,9 @@
**************************************************************************/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <errno.h>
#include <stdlib.h>
#include <string.h>
diff --git a/libkms/dumb.c b/libkms/dumb.c
index 440efb3..794282f 100644
--- a/libkms/dumb.c
+++ b/libkms/dumb.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/exynos.c b/libkms/exynos.c
index 93e36a1..243915b 100644
--- a/libkms/exynos.c
+++ b/libkms/exynos.c
@@ -11,7 +11,9 @@
* option) any later version.
*/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/intel.c b/libkms/intel.c
index abae452..92f1cf2 100644
--- a/libkms/intel.c
+++ b/libkms/intel.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/linux.c b/libkms/linux.c
index 9b4f29e..17e1d58 100644
--- a/libkms/linux.c
+++ b/libkms/linux.c
@@ -30,7 +30,9 @@
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <errno.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/libkms/nouveau.c b/libkms/nouveau.c
index 608092f..2de827d 100644
--- a/libkms/nouveau.c
+++ b/libkms/nouveau.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/radeon.c b/libkms/radeon.c
index f5e382a..29375c4 100644
--- a/libkms/radeon.c
+++ b/libkms/radeon.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/libkms/vmwgfx.c b/libkms/vmwgfx.c
index d594b3b..598f383 100644
--- a/libkms/vmwgfx.c
+++ b/libkms/vmwgfx.c
@@ -26,7 +26,9 @@
**************************************************************************/
-#define HAVE_STDINT_H
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
#define _FILE_OFFSET_BITS 64
#include <errno.h>
diff --git a/tests/drmstat.c b/tests/drmstat.c
index c51cbc6..5935d07 100644
--- a/tests/drmstat.c
+++ b/tests/drmstat.c
@@ -28,7 +28,9 @@
*
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/modetest/buffers.c b/tests/modetest/buffers.c
index 8206ce3..29b520d 100644
--- a/tests/modetest/buffers.c
+++ b/tests/modetest/buffers.c
@@ -24,7 +24,9 @@
* IN THE SOFTWARE.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <errno.h>
diff --git a/tests/modetest/cursor.c b/tests/modetest/cursor.c
index 7077f20..60f240a 100644
--- a/tests/modetest/cursor.c
+++ b/tests/modetest/cursor.c
@@ -22,7 +22,9 @@
* IN THE SOFTWARE.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <errno.h>
diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
index 7d436b5..92efb82 100644
--- a/tests/modetest/modetest.c
+++ b/tests/modetest/modetest.c
@@ -37,7 +37,9 @@
* TODO: use cairo to write the mode info on the selected output once
* the mode has been programmed, along with possible test patterns.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <ctype.h>
diff --git a/tests/vbltest/vbltest.c b/tests/vbltest/vbltest.c
index 2a09d28..50e29dc 100644
--- a/tests/vbltest/vbltest.c
+++ b/tests/vbltest/vbltest.c
@@ -37,7 +37,9 @@
* TODO: use cairo to write the mode info on the selected output once
* the mode has been programmed, along with possible test patterns.
*/
+#ifdef HAVE_CONFIG_H
#include "config.h"
+#endif
#include <assert.h>
#include <stdio.h>
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
2014-07-29 17:27 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
` (6 subsequent siblings)
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
configure.ac has AC_SYS_LARGEFILE which provides the define and/or
approapriate magic when required.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
libkms/dumb.c | 1 -
libkms/exynos.c | 1 -
libkms/intel.c | 1 -
libkms/nouveau.c | 1 -
libkms/radeon.c | 1 -
libkms/vmwgfx.c | 1 -
6 files changed, 6 deletions(-)
diff --git a/libkms/dumb.c b/libkms/dumb.c
index 794282f..5702543 100644
--- a/libkms/dumb.c
+++ b/libkms/dumb.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/exynos.c b/libkms/exynos.c
index 243915b..92e329c 100644
--- a/libkms/exynos.c
+++ b/libkms/exynos.c
@@ -14,7 +14,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/intel.c b/libkms/intel.c
index 92f1cf2..b006ea4 100644
--- a/libkms/intel.c
+++ b/libkms/intel.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/nouveau.c b/libkms/nouveau.c
index 2de827d..15c012e 100644
--- a/libkms/nouveau.c
+++ b/libkms/nouveau.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/radeon.c b/libkms/radeon.c
index 29375c4..938321b 100644
--- a/libkms/radeon.c
+++ b/libkms/radeon.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdio.h>
diff --git a/libkms/vmwgfx.c b/libkms/vmwgfx.c
index 598f383..08163a1 100644
--- a/libkms/vmwgfx.c
+++ b/libkms/vmwgfx.c
@@ -29,7 +29,6 @@
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
-#define _FILE_OFFSET_BITS 64
#include <errno.h>
#include <stdlib.h>
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
2014-07-29 17:27 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
2014-07-29 17:27 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-08-24 18:28 ` [PATCHv3 " Emil Velikov
2014-07-29 17:27 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
` (5 subsequent siblings)
8 siblings, 1 reply; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
Will be used to consolidate the required sources lists as well as the
install-able headers. This is turn will help us to avoid the
duplication with the upcoming Android build support.
v2 Rename the headers variable to *_H_FILES.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Makefile.am | 13 ++++---------
Makefile.sources | 12 ++++++++++++
freedreno/Makefile.am | 24 +++---------------------
freedreno/Makefile.sources | 24 ++++++++++++++++++++++++
include/drm/Makefile.am | 20 ++++----------------
include/drm/Makefile.sources | 18 ++++++++++++++++++
intel/Makefile.am | 16 ++++------------
intel/Makefile.sources | 14 ++++++++++++++
nouveau/Makefile.am | 11 ++++-------
nouveau/Makefile.sources | 9 +++++++++
radeon/Makefile.am | 22 ++++------------------
radeon/Makefile.sources | 19 +++++++++++++++++++
12 files changed, 119 insertions(+), 83 deletions(-)
create mode 100644 Makefile.sources
create mode 100644 freedreno/Makefile.sources
create mode 100644 include/drm/Makefile.sources
create mode 100644 intel/Makefile.sources
create mode 100644 nouveau/Makefile.sources
create mode 100644 radeon/Makefile.sources
diff --git a/Makefile.am b/Makefile.am
index 826c30d..fab2a9a 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -18,6 +18,8 @@
# IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
# CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+include Makefile.sources
+
ACLOCAL_AMFLAGS = -I m4 ${ACLOCAL_FLAGS}
pkgconfigdir = @pkgconfigdir@
@@ -62,17 +64,10 @@ libdrm_la_CPPFLAGS = -I$(top_srcdir)/include/drm
AM_CFLAGS = \
$(VALGRIND_CFLAGS)
-libdrm_la_SOURCES = \
- xf86drm.c \
- xf86drmHash.c \
- xf86drmRandom.c \
- xf86drmSL.c \
- xf86drmMode.c \
- xf86atomic.h \
- libdrm_lists.h
+libdrm_la_SOURCES = $(LIBDRM_FILES)
libdrmincludedir = ${includedir}
-libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
+libdrminclude_HEADERS = $(LIBDRM_H_FILES)
EXTRA_DIST = libdrm.pc.in include/drm/*
diff --git a/Makefile.sources b/Makefile.sources
new file mode 100644
index 0000000..15e38b5
--- /dev/null
+++ b/Makefile.sources
@@ -0,0 +1,12 @@
+LIBDRM_FILES := \
+ xf86drm.c \
+ xf86drmHash.c \
+ xf86drmRandom.c \
+ xf86drmSL.c \
+ xf86drmMode.c \
+ xf86atomic.h \
+ libdrm_lists.h
+
+LIBDRM_H_FILES := \
+ xf86drm.h \
+ xf86drmMode.h
diff --git a/freedreno/Makefile.am b/freedreno/Makefile.am
index 7903e5b..3a6f7f5 100644
--- a/freedreno/Makefile.am
+++ b/freedreno/Makefile.am
@@ -1,4 +1,5 @@
AUTOMAKE_OPTIONS=subdir-objects
+include Makefile.sources
AM_CFLAGS = \
$(WARN_CFLAGS) \
@@ -12,29 +13,10 @@ libdrm_freedreno_ladir = $(libdir)
libdrm_freedreno_la_LDFLAGS = -version-number 1:0:0 -no-undefined
libdrm_freedreno_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_freedreno_la_SOURCES = \
- freedreno_device.c \
- freedreno_pipe.c \
- freedreno_priv.h \
- freedreno_ringbuffer.c \
- freedreno_bo.c \
- kgsl/kgsl_bo.c \
- kgsl/kgsl_device.c \
- kgsl/kgsl_drm.h \
- kgsl/kgsl_pipe.c \
- kgsl/kgsl_priv.h \
- kgsl/kgsl_ringbuffer.c \
- kgsl/msm_kgsl.h \
- msm/msm_bo.c \
- msm/msm_device.c \
- msm/msm_drm.h \
- msm/msm_pipe.c \
- msm/msm_priv.h \
- msm/msm_ringbuffer.c \
- list.h
+libdrm_freedreno_la_SOURCES = $(LIBDRM_FREEDRENO_FILES)
libdrm_freedrenocommonincludedir = ${includedir}/freedreno
-libdrm_freedrenocommoninclude_HEADERS = freedreno_drmif.h freedreno_ringbuffer.h
+libdrm_freedrenocommoninclude_HEADERS = $(LIBDRM_FREEDRENO_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_freedreno.pc
diff --git a/freedreno/Makefile.sources b/freedreno/Makefile.sources
new file mode 100644
index 0000000..91020df
--- /dev/null
+++ b/freedreno/Makefile.sources
@@ -0,0 +1,24 @@
+LIBDRM_FREEDRENO_FILES := \
+ freedreno_device.c \
+ freedreno_pipe.c \
+ freedreno_priv.h \
+ freedreno_ringbuffer.c \
+ freedreno_bo.c \
+ kgsl/kgsl_bo.c \
+ kgsl/kgsl_device.c \
+ kgsl/kgsl_drm.h \
+ kgsl/kgsl_pipe.c \
+ kgsl/kgsl_priv.h \
+ kgsl/kgsl_ringbuffer.c \
+ kgsl/msm_kgsl.h \
+ msm/msm_bo.c \
+ msm/msm_device.c \
+ msm/msm_drm.h \
+ msm/msm_pipe.c \
+ msm/msm_priv.h \
+ msm/msm_ringbuffer.c \
+ list.h
+
+LIBDRM_FREEDRENO_H_FILES := \
+ freedreno_drmif.h \
+ freedreno_ringbuffer.h
diff --git a/include/drm/Makefile.am b/include/drm/Makefile.am
index 2bc34d2..7a246ae 100644
--- a/include/drm/Makefile.am
+++ b/include/drm/Makefile.am
@@ -22,23 +22,11 @@
# however, r300 and via need their reg headers installed in order to build.
# better solutions are welcome.
+include Makefile.sources
+
klibdrmincludedir = ${includedir}/libdrm
-klibdrminclude_HEADERS = \
- drm.h \
- drm_mode.h \
- drm_fourcc.h \
- drm_sarea.h \
- i915_drm.h \
- mga_drm.h \
- nouveau_drm.h \
- r128_drm.h \
- radeon_drm.h \
- savage_drm.h \
- sis_drm.h \
- via_drm.h \
- mach64_drm.h \
- qxl_drm.h
+klibdrminclude_HEADERS = $(LIBDRM_INCLUDE_H_FILES)
if HAVE_VMWGFX
-klibdrminclude_HEADERS += vmwgfx_drm.h
+klibdrminclude_HEADERS += $(LIBDRM_INCLUDE_VMWGFX_H_FILES)
endif
diff --git a/include/drm/Makefile.sources b/include/drm/Makefile.sources
new file mode 100644
index 0000000..8e1b585
--- /dev/null
+++ b/include/drm/Makefile.sources
@@ -0,0 +1,18 @@
+LIBDRM_INCLUDE_H_FILES := \
+ drm.h \
+ drm_mode.h \
+ drm_fourcc.h \
+ drm_sarea.h \
+ i915_drm.h \
+ mga_drm.h \
+ nouveau_drm.h \
+ r128_drm.h \
+ radeon_drm.h \
+ savage_drm.h \
+ sis_drm.h \
+ via_drm.h \
+ mach64_drm.h \
+ qxl_drm.h
+
+LIBDRM_INCLUDE_VMWGFX_H_FILES := \
+ vmwgfx_drm.h
diff --git a/intel/Makefile.am b/intel/Makefile.am
index f49b099..9c85c47 100644
--- a/intel/Makefile.am
+++ b/intel/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Eric Anholt <eric@anholt.net>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -39,22 +41,12 @@ libdrm_intel_la_LIBADD = ../libdrm.la \
@PCIACCESS_LIBS@ \
@CLOCK_LIB@
-libdrm_intel_la_SOURCES = \
- intel_bufmgr.c \
- intel_bufmgr_priv.h \
- intel_bufmgr_fake.c \
- intel_bufmgr_gem.c \
- intel_decode.c \
- intel_chipset.h \
- mm.c \
- mm.h
+libdrm_intel_la_SOURCES = $(LIBDRM_INTEL_FILES)
intel_bufmgr_gem_o_CFLAGS = $(AM_CFLAGS) -c99
libdrm_intelincludedir = ${includedir}/libdrm
-libdrm_intelinclude_HEADERS = intel_bufmgr.h \
- intel_aub.h \
- intel_debug.h
+libdrm_intelinclude_HEADERS = $(LIBDRM_INTEL_H_FILES)
# This may be interesting even outside of "make check", due to the -dump option.
noinst_PROGRAMS = test_decode
diff --git a/intel/Makefile.sources b/intel/Makefile.sources
new file mode 100644
index 0000000..7b2272c
--- /dev/null
+++ b/intel/Makefile.sources
@@ -0,0 +1,14 @@
+LIBDRM_INTEL_FILES := \
+ intel_bufmgr.c \
+ intel_bufmgr_priv.h \
+ intel_bufmgr_fake.c \
+ intel_bufmgr_gem.c \
+ intel_decode.c \
+ intel_chipset.h \
+ mm.c \
+ mm.h
+
+LIBDRM_INTEL_H_FILES := \
+ intel_bufmgr.h \
+ intel_aub.h \
+ intel_debug.h
diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
index 206e892..00fa690 100644
--- a/nouveau/Makefile.am
+++ b/nouveau/Makefile.am
@@ -1,3 +1,5 @@
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -11,15 +13,10 @@ libdrm_nouveau_ladir = $(libdir)
libdrm_nouveau_la_LDFLAGS = -version-number 2:0:0 -no-undefined
libdrm_nouveau_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_nouveau_la_SOURCES = nouveau.c \
- pushbuf.c \
- bufctx.c \
- abi16.c \
- private.h
-
+libdrm_nouveau_la_SOURCES = $(LIBDRM_NOUVEAU_FILES)
libdrm_nouveauincludedir = ${includedir}/libdrm
-libdrm_nouveauinclude_HEADERS = nouveau.h
+libdrm_nouveauinclude_HEADERS = $(LIBDRM_NOUVEAU_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_nouveau.pc
diff --git a/nouveau/Makefile.sources b/nouveau/Makefile.sources
new file mode 100644
index 0000000..89f2a2b
--- /dev/null
+++ b/nouveau/Makefile.sources
@@ -0,0 +1,9 @@
+LIBDRM_NOUVEAU_FILES := \
+ nouveau.c \
+ pushbuf.c \
+ bufctx.c \
+ abi16.c \
+ private.h
+
+LIBDRM_NOUVEAU_H_FILES := \
+ nouveau.h
diff --git a/radeon/Makefile.am b/radeon/Makefile.am
index a8cd100..ae913c0 100644
--- a/radeon/Makefile.am
+++ b/radeon/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Jérôme Glisse <glisse@freedesktop.org>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
-I$(top_srcdir) \
@@ -34,26 +36,10 @@ libdrm_radeon_ladir = $(libdir)
libdrm_radeon_la_LDFLAGS = -version-number 1:0:1 -no-undefined
libdrm_radeon_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_radeon_la_SOURCES = \
- radeon_bo_gem.c \
- radeon_cs_gem.c \
- radeon_cs_space.c \
- radeon_bo.c \
- radeon_cs.c \
- radeon_surface.c \
- bof.c \
- bof.h
+libdrm_radeon_la_SOURCES = $(LIBDRM_RADEON_FILES)
libdrm_radeonincludedir = ${includedir}/libdrm
-libdrm_radeoninclude_HEADERS = \
- radeon_bo.h \
- radeon_cs.h \
- radeon_surface.h \
- radeon_bo_gem.h \
- radeon_cs_gem.h \
- radeon_bo_int.h \
- radeon_cs_int.h \
- r600_pci_ids.h
+libdrm_radeoninclude_HEADERS = $(LIBDRM_RADEON_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_radeon.pc
diff --git a/radeon/Makefile.sources b/radeon/Makefile.sources
new file mode 100644
index 0000000..a17701a
--- /dev/null
+++ b/radeon/Makefile.sources
@@ -0,0 +1,19 @@
+LIBDRM_RADEON_FILES := \
+ radeon_bo_gem.c \
+ radeon_cs_gem.c \
+ radeon_cs_space.c \
+ radeon_bo.c \
+ radeon_cs.c \
+ radeon_surface.c \
+ bof.c \
+ bof.h
+
+LIBDRM_RADEON_H_FILES := \
+ radeon_bo.h \
+ radeon_cs.h \
+ radeon_surface.h \
+ radeon_bo_gem.h \
+ radeon_cs_gem.h \
+ radeon_bo_int.h \
+ radeon_cs_int.h \
+ r600_pci_ids.h
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 4/8] libdrm,intel: Add Android build
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (2 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
` (4 subsequent siblings)
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
Contains the following patches squashed in:
commit f340a8b9f2b84d5762553bef046914e0bde20795
Author: Chad Versace <chad.versace@linux.intel.com>
Date: Wed, 21 Dec 2011 11:43:57 -0800
libdrm,intel: Add Android makefiles (v2)
This enables libdrm.so and libdrm_intel.so to build on Android
IceCreamSandwich.
v2: Link libdrm_intel to libpciaccess.
Change-Id: Ie5ed4bc0e6b4f9f819e3ec44488e385c35e97128
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
commit 8fb3f42389dea34218ed1fe59550ec2abb4d6953
Author: Andrew Boie <andrew.p.boie@intel.com>
Date: Wed, 26 Sep 2012 13:32:05 -0700
libdrm, libdrm_intel: Skip driver name checks
These libraries have 'optional' tags, which means they won't get
built unless something else depends on them or they are added to
PRODUCT_PACKAGES. There's no need for additional filtering.
Change-Id: I5d90969f38671f8144c0dc27d47144b3f09a15ce
Signed-off-by: Andrew Boie <andrew.p.boie@intel.com>
---
Android.mk | 45 +++++++++++++++++++++++++++++++++++++++++++++
intel/Android.mk | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 95 insertions(+)
create mode 100644 Android.mk
create mode 100644 intel/Android.mk
diff --git a/Android.mk b/Android.mk
new file mode 100644
index 0000000..afe59ce
--- /dev/null
+++ b/Android.mk
@@ -0,0 +1,45 @@
+#
+# Copyright © 2011 Intel Corporation
+#
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice (including the next
+# paragraph) shall be included in all copies or substantial portions of the
+# Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+# IN THE SOFTWARE.
+#
+
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+LIBDRM_TOP := $(LOCAL_PATH)
+
+# Import variables LIBDRM_FILES, LIBDRM_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SRC_FILES := $(LIBDRM_FILES)
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+include $(BUILD_SHARED_LIBRARY)
+
+include $(LOCAL_PATH)/intel/Android.mk
diff --git a/intel/Android.mk b/intel/Android.mk
new file mode 100644
index 0000000..5c48a95
--- /dev/null
+++ b/intel/Android.mk
@@ -0,0 +1,50 @@
+#
+# Copyright © 2011 Intel Corporation
+#
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice (including the next
+# paragraph) shall be included in all copies or substantial portions of the
+# Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+# IN THE SOFTWARE.
+#
+
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_INTEL_FILES, LIBDRM_INTEL_HEADERS
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_intel
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_INTEL_FILES)
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/intel \
+ $(LIBDRM_TOP)/include/drm \
+ external/libpciaccess/include
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm \
+ libpciaccess
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 5/8] libdrm,intel: rework android header handling
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (3 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
` (3 subsequent siblings)
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
Contains the following patches squashed in:
commit 99247a5bd724ddcf0f06a5518baad207c53f1e2b
Author: Haitao Huang <haitao.huang@intel.com>
Date: Fri, 27 Apr 2012 13:20:53 -0500
Android.mk: use LOCAL_COPY_HEADERS to export headers.
Export necessary header files used by other components for
Android, such as libva intel-driver, gralloc, hwcomposer, etc.
Change-Id: I2feabf6941379ef4d756e942f30eba059de641f1
Signed-off-by: Haitao Huang <haitao.huang@intel.com>
[chad: Fixed inconsistent indentation.]
Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
commit 7d0b528cb69995d7ea4e29b2daa1e3b28a362f42
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date: Sun, 27 Jul 2014 18:22:41 +0100
android: reuse headers lists, separate libdrm from intel headers
Rather than having a duplicate copy of the headers list(s),
reuse the existing one(s). Distinguish that the intel headers
should be copied when libdrm_intel is used.
v2 Rename the headers variable(s) to *_H_FILES.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
commit 361de3ba4cadd5357596d1537bb3f216d281532b
Author: Piotr Luc <piotr.luc@intel.com>
Date: Fri, 14 Jun 2013 13:00:39 +0200
Export include dir from libdrm
BZ: 116218
Google introduced new method of specifying include path(s)
between modules. This allows a module to include header from a
library without directly specifyining by includer the path where
headers are located.
The method requires from library that holds headers to export
include path(s) in LOCAL_EXPORT_C_INCLUDE_DIRS variable.
These exported include path(s) are automatically added to
include path(s) of modules that have name of the library in the
LOCAL_SHARED_LIBRARIES or LOCAL_STATIC_LIBRARIES list.
This change sets LOCAL_EXPORT_C_INCLUDE_DIRS to folders that
contain headers file that used by other modules in order to
export these paths.
Change-Id: Id1ac885b31ef2efe194e0289fbcaecd9eb533df0
Signed-off-by: Piotr Luc <piotr.luc@intel.com>
Reviewed-on: http://android.intel.com:8080/113562
Reviewed-by: cactus <cactus@intel.com>
Reviewed-by: Luc, Piotr <Piotr.Luc@intel.com>
Reviewed-by: Purushothaman, Vijay A <vijay.a.purushothaman@intel.com>
Reviewed-by: Stimson, Dale B <dale.b.stimson@intel.com>
Tested-by: Stimson, Dale B <dale.b.stimson@intel.com>
Reviewed-by: buildbot <buildbot@intel.com>
Tested-by: buildbot <buildbot@intel.com>
commit 2bf22fcbd4cbb9e7c7764d5eff0bb4e75ab1a005
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date: 27 Jul 2014 18:27:21 +0100
android: Separate libdrm and intel LOCAL_EXPORT_C_INCLUDE_DIRS
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 15 +++++++++++++--
intel/Android.mk | 7 ++++++-
2 files changed, 19 insertions(+), 3 deletions(-)
diff --git a/Android.mk b/Android.mk
index afe59ce..795e554 100644
--- a/Android.mk
+++ b/Android.mk
@@ -1,5 +1,5 @@
#
-# Copyright © 2011 Intel Corporation
+# Copyright © 2011-2012 Intel Corporation
#
# Permission is hereby granted, free of charge, to any person obtaining a
# copy of this software and associated documentation files (the "Software"),
@@ -26,13 +26,18 @@ include $(CLEAR_VARS)
LIBDRM_TOP := $(LOCAL_PATH)
-# Import variables LIBDRM_FILES, LIBDRM_HEADERS
+# Import variables LIBDRM_FILES, LIBDRM_H_FILES
include $(LOCAL_PATH)/Makefile.sources
+# Import variables LIBDRM_INCLUDE_H_FILES, LIBDRM_INCLUDE_VMWGFX_H_FILES
+include $(LOCAL_PATH)/include/drm/Makefile.sources
LOCAL_MODULE := libdrm
LOCAL_MODULE_TAGS := optional
LOCAL_SRC_FILES := $(LIBDRM_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH) \
+ $(LOCAL_PATH)/include/drm
LOCAL_C_INCLUDES := \
$(LIBDRM_TOP)/include/drm
@@ -40,6 +45,12 @@ LOCAL_C_INCLUDES := \
LOCAL_CFLAGS := \
-DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+LOCAL_COPY_HEADERS := \
+ $(LIBDRM_H_FILES) \
+ $(addprefix include/drm/,$(LIBDRM_INCLUDE_H_FILES)) \
+ $(addprefix include/drm/,$(LIBDRM_INCLUDE_VMWGFX_H_FILES))
+
+LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
include $(LOCAL_PATH)/intel/Android.mk
diff --git a/intel/Android.mk b/intel/Android.mk
index 5c48a95..f2f21b9 100644
--- a/intel/Android.mk
+++ b/intel/Android.mk
@@ -24,7 +24,7 @@
LOCAL_PATH := $(call my-dir)
include $(CLEAR_VARS)
-# Import variables LIBDRM_INTEL_FILES, LIBDRM_INTEL_HEADERS
+# Import variables LIBDRM_INTEL_FILES, LIBDRM_INTEL_H_FILES
include $(LOCAL_PATH)/Makefile.sources
LOCAL_MODULE := libdrm_intel
@@ -33,6 +33,8 @@ LOCAL_MODULE_TAGS := optional
LOCAL_SHARED_LIBRARIES := libdrm
LOCAL_SRC_FILES := $(LIBDRM_INTEL_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/intel
LOCAL_C_INCLUDES := \
$(LIBDRM_TOP) \
@@ -43,6 +45,9 @@ LOCAL_C_INCLUDES := \
LOCAL_CFLAGS := \
-DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+LOCAL_COPY_HEADERS := $(LIBDRM_INTEL_H_FILES)
+LOCAL_COPY_HEADERS_TO := libdrm
+
LOCAL_SHARED_LIBRARIES := \
libdrm \
libpciaccess
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 6/8] radeon: add Android build support
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (4 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 7/8] nouveau: " Emil Velikov
` (2 subsequent siblings)
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
v2 Rename the headers variable(s) to *_H_FILES.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 7 ++++++-
radeon/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 36 insertions(+), 1 deletion(-)
create mode 100644 radeon/Android.mk
diff --git a/Android.mk b/Android.mk
index 795e554..3f43625 100644
--- a/Android.mk
+++ b/Android.mk
@@ -53,4 +53,9 @@ LOCAL_COPY_HEADERS := \
LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
-include $(LOCAL_PATH)/intel/Android.mk
+SUBDIRS := \
+ intel \
+ radeon
+
+mkfiles := $(patsubst %,$(LIBDRM_TOP)/%/Android.mk,$(SUBDIRS))
+include $(mkfiles)
diff --git a/radeon/Android.mk b/radeon/Android.mk
new file mode 100644
index 0000000..9cba546
--- /dev/null
+++ b/radeon/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_RADEON_FILES, LIBDRM_RADEON_H_FILES
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_radeon
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_RADEON_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/radeon
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/radeon \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_RADEON_H_FILES)
+LOCAL_COPY_HEADERS_TO := libdrm
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 7/8] nouveau: add Android build support
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (5 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-29 17:27 ` [PATCH 8/8] freedreno: " Emil Velikov
2014-07-31 15:33 ` [PATCHv2 0/8] Upstreaming the Android build and misc fixes Gore, Tim
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
v2 Rename the headers variable(s) to *_H_FILES.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 1 +
nouveau/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 31 insertions(+)
create mode 100644 nouveau/Android.mk
diff --git a/Android.mk b/Android.mk
index 3f43625..d040d4f 100644
--- a/Android.mk
+++ b/Android.mk
@@ -55,6 +55,7 @@ include $(BUILD_SHARED_LIBRARY)
SUBDIRS := \
intel \
+ nouveau \
radeon
mkfiles := $(patsubst %,$(LIBDRM_TOP)/%/Android.mk,$(SUBDIRS))
diff --git a/nouveau/Android.mk b/nouveau/Android.mk
new file mode 100644
index 0000000..734fc21
--- /dev/null
+++ b/nouveau/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_NOUVEAU_FILES, LIBDRM_NOUVEAU_H_FILES
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_nouveau
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_NOUVEAU_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/nouveau
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/nouveau \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_NOUVEAU_H_FILES)
+LOCAL_COPY_HEADERS_TO := libdrm
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [PATCH 8/8] freedreno: add Android build support
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (6 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 7/8] nouveau: " Emil Velikov
@ 2014-07-29 17:27 ` Emil Velikov
2014-07-31 15:33 ` [PATCHv2 0/8] Upstreaming the Android build and misc fixes Gore, Tim
8 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-07-29 17:27 UTC (permalink / raw)
To: dri-devel; +Cc: Emil Velikov
v2 Rename the headers variable(s) to *_H_FILES.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Android.mk | 1 +
freedreno/Android.mk | 30 ++++++++++++++++++++++++++++++
2 files changed, 31 insertions(+)
create mode 100644 freedreno/Android.mk
diff --git a/Android.mk b/Android.mk
index d040d4f..bb49b0b 100644
--- a/Android.mk
+++ b/Android.mk
@@ -54,6 +54,7 @@ LOCAL_COPY_HEADERS_TO := libdrm
include $(BUILD_SHARED_LIBRARY)
SUBDIRS := \
+ freedreno \
intel \
nouveau \
radeon
diff --git a/freedreno/Android.mk b/freedreno/Android.mk
new file mode 100644
index 0000000..243a1e2
--- /dev/null
+++ b/freedreno/Android.mk
@@ -0,0 +1,30 @@
+LOCAL_PATH := $(call my-dir)
+include $(CLEAR_VARS)
+
+# Import variables LIBDRM_FREEDRENO_FILES, LIBDRM_FREEDRENO_H_FILES
+include $(LOCAL_PATH)/Makefile.sources
+
+LOCAL_MODULE := libdrm_freedreno
+LOCAL_MODULE_TAGS := optional
+
+LOCAL_SHARED_LIBRARIES := libdrm
+
+LOCAL_SRC_FILES := $(LIBDRM_FREEDRENO_FILES)
+LOCAL_EXPORT_C_INCLUDE_DIRS += \
+ $(LOCAL_PATH)/freedreno
+
+LOCAL_C_INCLUDES := \
+ $(LIBDRM_TOP) \
+ $(LIBDRM_TOP)/freedreno \
+ $(LIBDRM_TOP)/include/drm
+
+LOCAL_CFLAGS := \
+ -DHAVE_LIBDRM_ATOMIC_PRIMITIVES=1
+
+LOCAL_COPY_HEADERS := $(LIBDRM_FREEDRENO_H_FILES)
+LOCAL_COPY_HEADERS_TO := freedreno
+
+LOCAL_SHARED_LIBRARIES := \
+ libdrm
+
+include $(BUILD_SHARED_LIBRARY)
--
2.0.2
^ permalink raw reply related [flat|nested] 31+ messages in thread
* RE: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
` (7 preceding siblings ...)
2014-07-29 17:27 ` [PATCH 8/8] freedreno: " Emil Velikov
@ 2014-07-31 15:33 ` Gore, Tim
2014-07-31 18:32 ` Emil Velikov
8 siblings, 1 reply; 31+ messages in thread
From: Gore, Tim @ 2014-07-31 15:33 UTC (permalink / raw)
To: Emil Velikov, dri-devel@lists.freedesktop.org
Ok, I still had to manually apply some of patch 3/8 as it was corrupted
And would not apply, but after this I was able to build within one of
My android trees. I have not tested the resulting libraries yet, but the
Overall build process seems ok.
Tim
> -----Original Message-----
> From: Emil Velikov [mailto:emil.l.velikov@gmail.com]
> Sent: Tuesday, July 29, 2014 6:27 PM
> To: dri-devel@lists.freedesktop.org
> Cc: Gore, Tim
> Subject: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
>
> Changes since the orignal posting:
> - Rebased on top of master.
> - Used _H_FILES for header lists (_HEADERS is a no-go with autotools)
> - Install the freedreno headers to {include_dir}/freedreno, similar to the
> automake builds.
> - Correctly include $(hw)/Android.mk
>
> The series is also available at the fixes+android-v2 branch in
> https://github.com/evelikov/libdrm.
>
> -Emil
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-07-31 15:33 ` [PATCHv2 0/8] Upstreaming the Android build and misc fixes Gore, Tim
@ 2014-07-31 18:32 ` Emil Velikov
2014-08-24 18:39 ` Emil Velikov
0 siblings, 1 reply; 31+ messages in thread
From: Emil Velikov @ 2014-07-31 18:32 UTC (permalink / raw)
To: Gore, Tim, dri-devel@lists.freedesktop.org; +Cc: emil.l.velikov
Strange, I've explicitly made sure that it's rebased on top of libdrm/master.
I will give it another try in a moment.
FWIW, for Intel at least a mmap/mmap64 would be needed due to the bionic
shortage of their original implementation.
As mentioned earlier, I'm seeking for a few acked/reviewed-by and the rest of
the fixes will follow in due time. Can I consider this acked-by ?
Thanks for having a look
-Emil
On 31/07/14 16:33, Gore, Tim wrote:
> Ok, I still had to manually apply some of patch 3/8 as it was corrupted
> And would not apply, but after this I was able to build within one of
> My android trees. I have not tested the resulting libraries yet, but the
> Overall build process seems ok.
>
> Tim
>
>> -----Original Message-----
>> From: Emil Velikov [mailto:emil.l.velikov@gmail.com]
>> Sent: Tuesday, July 29, 2014 6:27 PM
>> To: dri-devel@lists.freedesktop.org
>> Cc: Gore, Tim
>> Subject: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
>>
>> Changes since the orignal posting:
>> - Rebased on top of master.
>> - Used _H_FILES for header lists (_HEADERS is a no-go with autotools)
>> - Install the freedreno headers to {include_dir}/freedreno, similar to the
>> automake builds.
>> - Correctly include $(hw)/Android.mk
>>
>> The series is also available at the fixes+android-v2 branch in
>> https://github.com/evelikov/libdrm.
>>
>> -Emil
>
^ permalink raw reply [flat|nested] 31+ messages in thread
* [PATCHv3 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources
2014-07-29 17:27 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
@ 2014-08-24 18:28 ` Emil Velikov
0 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-08-24 18:28 UTC (permalink / raw)
To: dri-devel; +Cc: emil.l.velikov
Will be used to consolidate the required sources lists as well as the
install-able headers. This is turn will help us to avoid the
duplication with the upcoming Android build support.
v2: Rename the headers variable to *_H_FILES.
v3: Rebase on top of symbol visibility patches.
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
---
Makefile.am | 14 ++++----------
Makefile.sources | 13 +++++++++++++
freedreno/Makefile.am | 24 +++---------------------
freedreno/Makefile.sources | 24 ++++++++++++++++++++++++
include/drm/Makefile.am | 20 ++++----------------
include/drm/Makefile.sources | 18 ++++++++++++++++++
intel/Makefile.am | 16 ++++------------
intel/Makefile.sources | 14 ++++++++++++++
nouveau/Makefile.am | 11 ++++-------
nouveau/Makefile.sources | 9 +++++++++
radeon/Makefile.am | 22 ++++------------------
radeon/Makefile.sources | 19 +++++++++++++++++++
12 files changed, 120 insertions(+), 84 deletions(-)
create mode 100644 Makefile.sources
create mode 100644 freedreno/Makefile.sources
create mode 100644 include/drm/Makefile.sources
create mode 100644 intel/Makefile.sources
create mode 100644 nouveau/Makefile.sources
create mode 100644 radeon/Makefile.sources
diff --git a/Makefile.am b/Makefile.am
index 65680da..fab2a9a 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -18,6 +18,8 @@
# IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
# CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+include Makefile.sources
+
ACLOCAL_AMFLAGS = -I m4 ${ACLOCAL_FLAGS}
pkgconfigdir = @pkgconfigdir@
@@ -62,18 +64,10 @@ libdrm_la_CPPFLAGS = -I$(top_srcdir)/include/drm
AM_CFLAGS = \
$(VALGRIND_CFLAGS)
-libdrm_la_SOURCES = \
- xf86drm.c \
- xf86drmHash.c \
- xf86drmRandom.c \
- xf86drmSL.c \
- xf86drmMode.c \
- xf86atomic.h \
- libdrm.h \
- libdrm_lists.h
+libdrm_la_SOURCES = $(LIBDRM_FILES)
libdrmincludedir = ${includedir}
-libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
+libdrminclude_HEADERS = $(LIBDRM_H_FILES)
EXTRA_DIST = libdrm.pc.in include/drm/*
diff --git a/Makefile.sources b/Makefile.sources
new file mode 100644
index 0000000..d078ca9
--- /dev/null
+++ b/Makefile.sources
@@ -0,0 +1,13 @@
+LIBDRM_FILES := \
+ xf86drm.c \
+ xf86drmHash.c \
+ xf86drmRandom.c \
+ xf86drmSL.c \
+ xf86drmMode.c \
+ xf86atomic.h \
+ libdrm.h \
+ libdrm_lists.h
+
+LIBDRM_H_FILES := \
+ xf86drm.h \
+ xf86drmMode.h
diff --git a/freedreno/Makefile.am b/freedreno/Makefile.am
index 49471e9..0c7db81 100644
--- a/freedreno/Makefile.am
+++ b/freedreno/Makefile.am
@@ -1,4 +1,5 @@
AUTOMAKE_OPTIONS=subdir-objects
+include Makefile.sources
AM_CFLAGS = \
$(WARN_CFLAGS) \
@@ -13,29 +14,10 @@ libdrm_freedreno_ladir = $(libdir)
libdrm_freedreno_la_LDFLAGS = -version-number 1:0:0 -no-undefined
libdrm_freedreno_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_freedreno_la_SOURCES = \
- freedreno_device.c \
- freedreno_pipe.c \
- freedreno_priv.h \
- freedreno_ringbuffer.c \
- freedreno_bo.c \
- kgsl/kgsl_bo.c \
- kgsl/kgsl_device.c \
- kgsl/kgsl_drm.h \
- kgsl/kgsl_pipe.c \
- kgsl/kgsl_priv.h \
- kgsl/kgsl_ringbuffer.c \
- kgsl/msm_kgsl.h \
- msm/msm_bo.c \
- msm/msm_device.c \
- msm/msm_drm.h \
- msm/msm_pipe.c \
- msm/msm_priv.h \
- msm/msm_ringbuffer.c \
- list.h
+libdrm_freedreno_la_SOURCES = $(LIBDRM_FREEDRENO_FILES)
libdrm_freedrenocommonincludedir = ${includedir}/freedreno
-libdrm_freedrenocommoninclude_HEADERS = freedreno_drmif.h freedreno_ringbuffer.h
+libdrm_freedrenocommoninclude_HEADERS = $(LIBDRM_FREEDRENO_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_freedreno.pc
diff --git a/freedreno/Makefile.sources b/freedreno/Makefile.sources
new file mode 100644
index 0000000..91020df
--- /dev/null
+++ b/freedreno/Makefile.sources
@@ -0,0 +1,24 @@
+LIBDRM_FREEDRENO_FILES := \
+ freedreno_device.c \
+ freedreno_pipe.c \
+ freedreno_priv.h \
+ freedreno_ringbuffer.c \
+ freedreno_bo.c \
+ kgsl/kgsl_bo.c \
+ kgsl/kgsl_device.c \
+ kgsl/kgsl_drm.h \
+ kgsl/kgsl_pipe.c \
+ kgsl/kgsl_priv.h \
+ kgsl/kgsl_ringbuffer.c \
+ kgsl/msm_kgsl.h \
+ msm/msm_bo.c \
+ msm/msm_device.c \
+ msm/msm_drm.h \
+ msm/msm_pipe.c \
+ msm/msm_priv.h \
+ msm/msm_ringbuffer.c \
+ list.h
+
+LIBDRM_FREEDRENO_H_FILES := \
+ freedreno_drmif.h \
+ freedreno_ringbuffer.h
diff --git a/include/drm/Makefile.am b/include/drm/Makefile.am
index 2bc34d2..7a246ae 100644
--- a/include/drm/Makefile.am
+++ b/include/drm/Makefile.am
@@ -22,23 +22,11 @@
# however, r300 and via need their reg headers installed in order to build.
# better solutions are welcome.
+include Makefile.sources
+
klibdrmincludedir = ${includedir}/libdrm
-klibdrminclude_HEADERS = \
- drm.h \
- drm_mode.h \
- drm_fourcc.h \
- drm_sarea.h \
- i915_drm.h \
- mga_drm.h \
- nouveau_drm.h \
- r128_drm.h \
- radeon_drm.h \
- savage_drm.h \
- sis_drm.h \
- via_drm.h \
- mach64_drm.h \
- qxl_drm.h
+klibdrminclude_HEADERS = $(LIBDRM_INCLUDE_H_FILES)
if HAVE_VMWGFX
-klibdrminclude_HEADERS += vmwgfx_drm.h
+klibdrminclude_HEADERS += $(LIBDRM_INCLUDE_VMWGFX_H_FILES)
endif
diff --git a/include/drm/Makefile.sources b/include/drm/Makefile.sources
new file mode 100644
index 0000000..8e1b585
--- /dev/null
+++ b/include/drm/Makefile.sources
@@ -0,0 +1,18 @@
+LIBDRM_INCLUDE_H_FILES := \
+ drm.h \
+ drm_mode.h \
+ drm_fourcc.h \
+ drm_sarea.h \
+ i915_drm.h \
+ mga_drm.h \
+ nouveau_drm.h \
+ r128_drm.h \
+ radeon_drm.h \
+ savage_drm.h \
+ sis_drm.h \
+ via_drm.h \
+ mach64_drm.h \
+ qxl_drm.h
+
+LIBDRM_INCLUDE_VMWGFX_H_FILES := \
+ vmwgfx_drm.h
diff --git a/intel/Makefile.am b/intel/Makefile.am
index f734b0b..846695d 100644
--- a/intel/Makefile.am
+++ b/intel/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Eric Anholt <eric@anholt.net>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
$(VISIBILITY_CFLAGS) \
@@ -40,22 +42,12 @@ libdrm_intel_la_LIBADD = ../libdrm.la \
@PCIACCESS_LIBS@ \
@CLOCK_LIB@
-libdrm_intel_la_SOURCES = \
- intel_bufmgr.c \
- intel_bufmgr_priv.h \
- intel_bufmgr_fake.c \
- intel_bufmgr_gem.c \
- intel_decode.c \
- intel_chipset.h \
- mm.c \
- mm.h
+libdrm_intel_la_SOURCES = $(LIBDRM_INTEL_FILES)
intel_bufmgr_gem_o_CFLAGS = $(AM_CFLAGS) -c99
libdrm_intelincludedir = ${includedir}/libdrm
-libdrm_intelinclude_HEADERS = intel_bufmgr.h \
- intel_aub.h \
- intel_debug.h
+libdrm_intelinclude_HEADERS = $(LIBDRM_INTEL_H_FILES)
# This may be interesting even outside of "make check", due to the -dump option.
noinst_PROGRAMS = test_decode
diff --git a/intel/Makefile.sources b/intel/Makefile.sources
new file mode 100644
index 0000000..7b2272c
--- /dev/null
+++ b/intel/Makefile.sources
@@ -0,0 +1,14 @@
+LIBDRM_INTEL_FILES := \
+ intel_bufmgr.c \
+ intel_bufmgr_priv.h \
+ intel_bufmgr_fake.c \
+ intel_bufmgr_gem.c \
+ intel_decode.c \
+ intel_chipset.h \
+ mm.c \
+ mm.h
+
+LIBDRM_INTEL_H_FILES := \
+ intel_bufmgr.h \
+ intel_aub.h \
+ intel_debug.h
diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
index 73cff9f..a7df1ab 100644
--- a/nouveau/Makefile.am
+++ b/nouveau/Makefile.am
@@ -1,3 +1,5 @@
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
$(VISIBILITY_CFLAGS) \
@@ -12,15 +14,10 @@ libdrm_nouveau_ladir = $(libdir)
libdrm_nouveau_la_LDFLAGS = -version-number 2:0:0 -no-undefined
libdrm_nouveau_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_nouveau_la_SOURCES = nouveau.c \
- pushbuf.c \
- bufctx.c \
- abi16.c \
- private.h
-
+libdrm_nouveau_la_SOURCES = $(LIBDRM_NOUVEAU_FILES)
libdrm_nouveauincludedir = ${includedir}/libdrm
-libdrm_nouveauinclude_HEADERS = nouveau.h
+libdrm_nouveauinclude_HEADERS = $(LIBDRM_NOUVEAU_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_nouveau.pc
diff --git a/nouveau/Makefile.sources b/nouveau/Makefile.sources
new file mode 100644
index 0000000..89f2a2b
--- /dev/null
+++ b/nouveau/Makefile.sources
@@ -0,0 +1,9 @@
+LIBDRM_NOUVEAU_FILES := \
+ nouveau.c \
+ pushbuf.c \
+ bufctx.c \
+ abi16.c \
+ private.h
+
+LIBDRM_NOUVEAU_H_FILES := \
+ nouveau.h
diff --git a/radeon/Makefile.am b/radeon/Makefile.am
index c969573..aa66ca3 100644
--- a/radeon/Makefile.am
+++ b/radeon/Makefile.am
@@ -22,6 +22,8 @@
# Authors:
# Jérôme Glisse <glisse@freedesktop.org>
+include Makefile.sources
+
AM_CFLAGS = \
$(WARN_CFLAGS) \
$(VISIBILITY_CFLAGS) \
@@ -35,26 +37,10 @@ libdrm_radeon_ladir = $(libdir)
libdrm_radeon_la_LDFLAGS = -version-number 1:0:1 -no-undefined
libdrm_radeon_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@
-libdrm_radeon_la_SOURCES = \
- radeon_bo_gem.c \
- radeon_cs_gem.c \
- radeon_cs_space.c \
- radeon_bo.c \
- radeon_cs.c \
- radeon_surface.c \
- bof.c \
- bof.h
+libdrm_radeon_la_SOURCES = $(LIBDRM_RADEON_FILES)
libdrm_radeonincludedir = ${includedir}/libdrm
-libdrm_radeoninclude_HEADERS = \
- radeon_bo.h \
- radeon_cs.h \
- radeon_surface.h \
- radeon_bo_gem.h \
- radeon_cs_gem.h \
- radeon_bo_int.h \
- radeon_cs_int.h \
- r600_pci_ids.h
+libdrm_radeoninclude_HEADERS = $(LIBDRM_RADEON_H_FILES)
pkgconfigdir = @pkgconfigdir@
pkgconfig_DATA = libdrm_radeon.pc
diff --git a/radeon/Makefile.sources b/radeon/Makefile.sources
new file mode 100644
index 0000000..a17701a
--- /dev/null
+++ b/radeon/Makefile.sources
@@ -0,0 +1,19 @@
+LIBDRM_RADEON_FILES := \
+ radeon_bo_gem.c \
+ radeon_cs_gem.c \
+ radeon_cs_space.c \
+ radeon_bo.c \
+ radeon_cs.c \
+ radeon_surface.c \
+ bof.c \
+ bof.h
+
+LIBDRM_RADEON_H_FILES := \
+ radeon_bo.h \
+ radeon_cs.h \
+ radeon_surface.h \
+ radeon_bo_gem.h \
+ radeon_cs_gem.h \
+ radeon_bo_int.h \
+ radeon_cs_int.h \
+ r600_pci_ids.h
--
2.0.2
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 31+ messages in thread
* Re: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-07-31 18:32 ` Emil Velikov
@ 2014-08-24 18:39 ` Emil Velikov
2014-08-29 9:44 ` Gore, Tim
0 siblings, 1 reply; 31+ messages in thread
From: Emil Velikov @ 2014-08-24 18:39 UTC (permalink / raw)
To: Gore, Tim, dri-devel@lists.freedesktop.org; +Cc: emil.l.velikov
Hi Tim,
Gave the series another try, and there was some minor issues caused by commit
5d8357976a8 (configure: Support symbol visibility when available). Did not see
any of the originally mentioned issues. Are you sure that you've tried it on
top of fd.o/drm/master ?
Do you have any comments, would like to do any testing (note it will not work
due to the mmap/mmap64 bionic issue, to be addressed next) or should I go
ahead and commit the series ?
Thanks
Emil
The series:
http://lists.freedesktop.org/archives/dri-devel/2014-July/064952.html
Patchv3 3/8:
http://lists.freedesktop.org/archives/dri-devel/2014-August/066748.html
On 31/07/14 19:32, Emil Velikov wrote:
> Strange, I've explicitly made sure that it's rebased on top of libdrm/master.
> I will give it another try in a moment.
>
> FWIW, for Intel at least a mmap/mmap64 would be needed due to the bionic
> shortage of their original implementation.
>
> As mentioned earlier, I'm seeking for a few acked/reviewed-by and the rest of
> the fixes will follow in due time. Can I consider this acked-by ?
>
> Thanks for having a look
> -Emil
>
> On 31/07/14 16:33, Gore, Tim wrote:
>> Ok, I still had to manually apply some of patch 3/8 as it was corrupted
>> And would not apply, but after this I was able to build within one of
>> My android trees. I have not tested the resulting libraries yet, but the
>> Overall build process seems ok.
>>
>> Tim
>>
>>> -----Original Message-----
>>> From: Emil Velikov [mailto:emil.l.velikov@gmail.com]
>>> Sent: Tuesday, July 29, 2014 6:27 PM
>>> To: dri-devel@lists.freedesktop.org
>>> Cc: Gore, Tim
>>> Subject: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
>>>
>>> Changes since the orignal posting:
>>> - Rebased on top of master.
>>> - Used _H_FILES for header lists (_HEADERS is a no-go with autotools)
>>> - Install the freedreno headers to {include_dir}/freedreno, similar to the
>>> automake builds.
>>> - Correctly include $(hw)/Android.mk
>>>
>>> The series is also available at the fixes+android-v2 branch in
>>> https://github.com/evelikov/libdrm.
>>>
>>> -Emil
>>
>
^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-08-24 18:39 ` Emil Velikov
@ 2014-08-29 9:44 ` Gore, Tim
2014-08-29 10:02 ` Emil Velikov
0 siblings, 1 reply; 31+ messages in thread
From: Gore, Tim @ 2014-08-29 9:44 UTC (permalink / raw)
To: Emil Velikov, dri-devel@lists.freedesktop.org
Hi Emil, sorry for the delay.
I still see the same patch error for patch 3/8. The patch diff
for intel/makefile.am has as part of its context:
# Eric Anholt <eric at anholt.net>
Whereas the source file has
# Eric Anholt <eric@anholt.net>
So the patch fails. Its trivial of course. Perhaps there is some automatic
Substitution getting done by some tool. Maybe git-am will automatically
Fix this, not sure, I don't have a workflow for git-am.
My source comes from
Git://anongit.freedesktop.org/git/mesa/drm
Remote =origin
Merge = refs/heads/master
Other than that, as I say the patches look fine, and I have successfully built
Libdrm in an Android tree using these patches and "mm".
I am out of the office from 12:00 until Thurs 4th Sept now.
Tim
> -----Original Message-----
> From: Emil Velikov [mailto:emil.l.velikov@gmail.com]
> Sent: Sunday, August 24, 2014 7:40 PM
> To: Gore, Tim; dri-devel@lists.freedesktop.org
> Cc: emil.l.velikov@gmail.com; Daniel Vetter (daniel@ffwll.ch)
> Subject: Re: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
>
> Hi Tim,
>
> Gave the series another try, and there was some minor issues caused by
> commit
> 5d8357976a8 (configure: Support symbol visibility when available). Did not
> see any of the originally mentioned issues. Are you sure that you've tried it
> on top of fd.o/drm/master ?
>
> Do you have any comments, would like to do any testing (note it will not
> work due to the mmap/mmap64 bionic issue, to be addressed next) or
> should I go ahead and commit the series ?
>
> Thanks
> Emil
>
> The series:
> http://lists.freedesktop.org/archives/dri-devel/2014-July/064952.html
>
> Patchv3 3/8:
> http://lists.freedesktop.org/archives/dri-devel/2014-August/066748.html
>
>
> On 31/07/14 19:32, Emil Velikov wrote:
> > Strange, I've explicitly made sure that it's rebased on top of libdrm/master.
> > I will give it another try in a moment.
> >
> > FWIW, for Intel at least a mmap/mmap64 would be needed due to the
> > bionic shortage of their original implementation.
> >
> > As mentioned earlier, I'm seeking for a few acked/reviewed-by and the
> > rest of the fixes will follow in due time. Can I consider this acked-by ?
> >
> > Thanks for having a look
> > -Emil
> >
> > On 31/07/14 16:33, Gore, Tim wrote:
> >> Ok, I still had to manually apply some of patch 3/8 as it was
> >> corrupted And would not apply, but after this I was able to build
> >> within one of My android trees. I have not tested the resulting
> >> libraries yet, but the Overall build process seems ok.
> >>
> >> Tim
> >>
> >>> -----Original Message-----
> >>> From: Emil Velikov [mailto:emil.l.velikov@gmail.com]
> >>> Sent: Tuesday, July 29, 2014 6:27 PM
> >>> To: dri-devel@lists.freedesktop.org
> >>> Cc: Gore, Tim
> >>> Subject: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
> >>>
> >>> Changes since the orignal posting:
> >>> - Rebased on top of master.
> >>> - Used _H_FILES for header lists (_HEADERS is a no-go with
> >>> autotools)
> >>> - Install the freedreno headers to {include_dir}/freedreno, similar
> >>> to the automake builds.
> >>> - Correctly include $(hw)/Android.mk
> >>>
> >>> The series is also available at the fixes+android-v2 branch in
> >>> https://github.com/evelikov/libdrm.
> >>>
> >>> -Emil
> >>
> >
^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [PATCHv2 0/8] Upstreaming the Android build and misc fixes
2014-08-29 9:44 ` Gore, Tim
@ 2014-08-29 10:02 ` Emil Velikov
0 siblings, 0 replies; 31+ messages in thread
From: Emil Velikov @ 2014-08-29 10:02 UTC (permalink / raw)
To: Gore, Tim, dri-devel@lists.freedesktop.org; +Cc: emil.l.velikov
Hi Tim,
On 29/08/14 10:44, Gore, Tim wrote:
> Hi Emil, sorry for the delay.
> I still see the same patch error for patch 3/8. The patch diff
> for intel/makefile.am has as part of its context:
>
> # Eric Anholt <eric at anholt.net>
>
> Whereas the source file has
>
> # Eric Anholt <eric@anholt.net>
>
> So the patch fails. Its trivial of course. Perhaps there is some automatic
> Substitution getting done by some tool.
The issue you're seeing here is (afaik) an anti-spam feature. The ML
software/parser automatically substitutes/obscures email addresses.
For any future case, I would recommend adding a remote branch rather than
picking patches from mailling-lists - not all of them play nice :\
> Maybe git-am will automatically
> Fix this, not sure, I don't have a workflow for git-am.
> My source comes from
> Git://anongit.freedesktop.org/git/mesa/drm
> Remote =origin
> Merge = refs/heads/master
>
> Other than that, as I say the patches look fine, and I have successfully built
> Libdrm in an Android tree using these patches and "mm".
> I am out of the office from 12:00 until Thurs 4th Sept now.
>
Nice. They build fine on my end as well. I've even had a couple of people test
them (with a mmap patch on top) alongside mesa 10.2 :)
I was hoping for an acked-by/reviewed-by, but it seems like I'll push the
series as is.
Thanks
Emil
> Tim
>
^ permalink raw reply [flat|nested] 31+ messages in thread
end of thread, other threads:[~2014-08-29 10:03 UTC | newest]
Thread overview: 31+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-27 18:25 [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
2014-07-27 18:25 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
2014-07-27 18:25 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
2014-07-27 18:25 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
2014-07-27 18:25 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
2014-07-27 18:25 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
2014-07-27 18:25 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
2014-07-27 18:25 ` [PATCH 7/8] nouveau: " Emil Velikov
2014-07-27 18:25 ` [PATCH 8/8] freedreno: " Emil Velikov
2014-07-28 2:48 ` [PATCH 0/8] Upstreaming the Android build and misc fixes Emil Velikov
2014-07-28 7:07 ` Daniel Vetter
2014-07-28 12:01 ` Emil Velikov
2014-07-28 15:22 ` Daniel Vetter
2014-07-29 16:14 ` Gore, Tim
2014-07-29 17:10 ` Emil Velikov
2014-07-29 17:27 ` [PATCHv2 " Emil Velikov
2014-07-29 17:27 ` [PATCH 1/8] all: include config.h only when available and use its defines Emil Velikov
2014-07-29 17:27 ` [PATCH 2/8] libkms: remove explicit define _FILE_OFFSET_BITS 64 Emil Velikov
2014-07-29 17:27 ` [PATCH 3/8] libdrm, freedreno, intel, nouveau, radeon: add Makefile.sources Emil Velikov
2014-08-24 18:28 ` [PATCHv3 " Emil Velikov
2014-07-29 17:27 ` [PATCH 4/8] libdrm,intel: Add Android build Emil Velikov
2014-07-29 17:27 ` [PATCH 5/8] libdrm,intel: rework android header handling Emil Velikov
2014-07-29 17:27 ` [PATCH 6/8] radeon: add Android build support Emil Velikov
2014-07-29 17:27 ` [PATCH 7/8] nouveau: " Emil Velikov
2014-07-29 17:27 ` [PATCH 8/8] freedreno: " Emil Velikov
2014-07-31 15:33 ` [PATCHv2 0/8] Upstreaming the Android build and misc fixes Gore, Tim
2014-07-31 18:32 ` Emil Velikov
2014-08-24 18:39 ` Emil Velikov
2014-08-29 9:44 ` Gore, Tim
2014-08-29 10:02 ` Emil Velikov
-- strict thread matches above, loose matches on Subject: below --
2014-07-29 16:43 [PATCH " Gore, Tim
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).