Re: [Sound-open-firmware] [PATCH] build: add library build support for host platform

On Fri, 2018-01-12 at 10:56 -0800, Ranjani Sridharan wrote:

This patch provides library build support for host platform architecture. It enables creating separate libraries for each SOF audio component.

Signed-off-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com

README | 7 + build-all.sh | 6 + configure.ac | 3 + src/Makefile.am | 11 + src/arch/host/include/Makefile.am | 1 + src/arch/host/include/arch/Makefile.am | 8 + src/audio/Makefile.am | 1005 +++++++++++++++++++++++++++++- src/include/reef/Makefile.am | 4 +- src/include/reef/audio/Makefile.am | 8 +- src/include/uapi/Makefile.am | 2 +- src/ipc/Makefile.am | 28 +- src/library/include/platform/Makefile.am | 12 + src/math/Makefile.am | 22 +- 13 files changed, 1096 insertions(+), 21 deletions(-) create mode 100644 src/arch/host/include/Makefile.am create mode 100644 src/arch/host/include/arch/Makefile.am create mode 100644 src/library/include/platform/Makefile.am

Doesn't apply.

diff --git a/README b/README index 9ca5b12..463fc42 100644 --- a/README +++ b/README @@ -13,6 +13,13 @@ Cherrytrail :-

./configure --with-arch=xtensa --with-platform=cherrytrail --with-root-dir=$PWD/../xtensa-root/xtensa-byt-elf --host=xtensa-byt-elf

+Library for Host Platform :- +If building library for host platform, run the following configure. Please modify +the --prefix option to choose the directory for installing the library files and +headers

+./configure --with-arch=host --enable-library=yes --host=x86_64-unknown-linux-gnu --prefix=$pwd/../host-root/

3. make

4. make bin

diff --git a/build-all.sh b/build-all.sh index 2e015f6..a0ce575 100755 --- a/build-all.sh +++ b/build-all.sh @@ -9,6 +9,12 @@ pwd=`pwd`

rm -fr src/arch/xtensa/*.ri

+# build library for host platform architecture +./configure --with-arch=host --enable-library=yes +--host=x86_64-unknown-linux-gnu --prefix=$pwd/../host-root/ +make +make install

# Build for Baytrail ./configure --with-arch=xtensa --with-platform=baytrail --with-root-dir=$pwd/../xtensa-root/xtensa-byt-elf --host=xtensa-byt-elf make clean diff --git a/configure.ac b/configure.ac index 00c377f..6c559fc 100644 --- a/configure.ac +++ b/configure.ac @@ -161,6 +161,9 @@ AC_CONFIG_FILES([ src/arch/xtensa/include/xtensa/config/Makefile src/arch/xtensa/hal/Makefile src/arch/xtensa/xtos/Makefile

•   src/arch/host/Makefile
  

•   src/arch/host/include/Makefile
  

•   src/arch/host/include/arch/Makefile
    src/audio/Makefile
     src/math/Makefile
    src/drivers/Makefile
  

diff --git a/src/Makefile.am b/src/Makefile.am index fb82330..8944a8d 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -1 +1,12 @@ SUBDIRS = include init math audio platform tasks drivers ipc lib arch

+export COMMON_INCDIR = \

•   $(REEF_INCDIR) \
  

•   $(ARCH_INCDIR) \
  

•   $(PLATFORM_INCDIR)
  

+if BUILD_LIB +SUBDIRS = ipc math audio arch include library +else +SUBDIRS = init math audio platform tasks drivers ipc lib arch include +endif diff --git a/src/arch/host/include/Makefile.am b/src/arch/host/include/Makefile.am new file mode 100644 index 0000000..f0ac9b7 --- /dev/null +++ b/src/arch/host/include/Makefile.am @@ -0,0 +1 @@ +SUBDIRS = arch diff --git a/src/arch/host/include/arch/Makefile.am b/src/arch/host/include/arch/Makefile.am new file mode 100644 index 0000000..6dddb35 --- /dev/null +++ b/src/arch/host/include/arch/Makefile.am @@ -0,0 +1,8 @@ +includedir = $(prefix)/include/sof/arch

+include_HEADERS = \

•   cache.h \
  

•   interrupt.h \
  

•   reef.h \
  

•   spinlock.h \
  

•   timer.h
  

I don't see these headers in the patch ?

Liam