From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by yocto-www.yoctoproject.org (Postfix, from userid 118) id ECE8FE00B20; Wed, 18 Nov 2015 05:38:34 -0800 (PST) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on yocto-www.yoctoproject.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 X-Spam-HAM-Report: * -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high * trust * [198.47.26.152 listed in list.dnswl.org] * -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Received: from comal.ext.ti.com (comal.ext.ti.com [198.47.26.152]) by yocto-www.yoctoproject.org (Postfix) with ESMTP id 5A47BE00B04 for ; Wed, 18 Nov 2015 05:38:32 -0800 (PST) Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id tAIDcVV0030418 for ; Wed, 18 Nov 2015 07:38:31 -0600 Received: from DLEE70.ent.ti.com (dlemailx.itg.ti.com [157.170.170.113]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id tAIDcVs6005352 for ; Wed, 18 Nov 2015 07:38:31 -0600 Received: from dflp33.itg.ti.com (10.64.6.16) by DLEE70.ent.ti.com (157.170.170.113) with Microsoft SMTP Server id 14.3.224.2; Wed, 18 Nov 2015 07:38:32 -0600 Received: from [10.218.109.201] (ileax41-snat.itg.ti.com [10.172.224.153]) by dflp33.itg.ti.com (8.14.3/8.13.8) with ESMTP id tAIDcV8i020501; Wed, 18 Nov 2015 07:38:31 -0600 Message-ID: <564C7F56.7080901@ti.com> Date: Wed, 18 Nov 2015 08:38:30 -0500 From: Jacob Stiffler User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Sam Nelson , References: <1447797979-10356-1-git-send-email-sam.nelson@ti.com> <564C7600.8050209@ti.com> In-Reply-To: <564C7600.8050209@ti.com> Subject: Re: [PATCH v4] ipc: ti-ipc, ti-ipc-rtos: Update to new version of IPC 3.41.0.3_eng X-BeenThere: meta-ti@yoctoproject.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: Usage and development list for the meta-ti layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 18 Nov 2015 13:38:35 -0000 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit On 11/18/2015 7:58 AM, Jacob Stiffler wrote: > > > On 11/17/2015 5:06 PM, Sam Nelson wrote: >> - Update to new version of IPC 3.41.0.3_eng >> - Add recipe to build ti-ipc-rtos from git repo >> - Remove K2G hack >> >> Signed-off-by: Sam Nelson >> >> --- >> Changes from the previous version of the patch >> - Removed redundant platform definitions >> --- >> --- >> recipes-ti/ipc/ti-ipc-rtos.inc | 19 ------------ >> recipes-ti/ipc/ti-ipc-rtos_3.40.01.08.bb | 9 ------ >> recipes-ti/ipc/ti-ipc-rtos_git.bb | 51 >> ++++++++++++++++++++++++++++++++ >> recipes-ti/ipc/ti-ipc.inc | 26 ++++++++++++++++ >> recipes-ti/ipc/ti-ipc_git.bb | 27 +++-------------- >> 5 files changed, 81 insertions(+), 51 deletions(-) >> delete mode 100644 recipes-ti/ipc/ti-ipc-rtos.inc > > Other layers use this file. Do we want to keep it, or should those > layers make a copy of it? > >> delete mode 100644 recipes-ti/ipc/ti-ipc-rtos_3.40.01.08.bb >> create mode 100644 recipes-ti/ipc/ti-ipc-rtos_git.bb >> create mode 100644 recipes-ti/ipc/ti-ipc.inc >> >> diff --git a/recipes-ti/ipc/ti-ipc-rtos.inc >> b/recipes-ti/ipc/ti-ipc-rtos.inc >> deleted file mode 100644 >> index d14c756..0000000 >> --- a/recipes-ti/ipc/ti-ipc-rtos.inc >> +++ /dev/null >> @@ -1,19 +0,0 @@ >> -DESCRIPTION = "TI Inter Process Communication (IPC) Mechanisms (for >> Uni- and Multi- Processor Configurations)" >> -HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:IPC" >> - >> -LICENSE = "BSD" >> - >> -require recipes-ti/includes/ti-paths.inc >> -require recipes-ti/includes/ti-staging.inc >> - >> -SRC_URI = >> "http://software-dl.ti.com/dsps/dsps_public_sw/sdo_sb/targetcontent/ipc/${PV}/exports/ipc_${PV}.zip;name=ipczip" >> - >> -S = "${WORKDIR}/ipc_${PV}" >> - >> -do_install() { >> - install -d ${D}${IPC_INSTALL_DIR_RECIPE} >> - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE} >> -} >> - >> -ALLOW_EMPTY_${PN} = "1" >> -FILES_${PN}-dev += "${IPC_INSTALL_DIR_RECIPE}" >> diff --git a/recipes-ti/ipc/ti-ipc-rtos_3.40.01.08.bb >> b/recipes-ti/ipc/ti-ipc-rtos_3.40.01.08.bb >> deleted file mode 100644 >> index 7c93695..0000000 >> --- a/recipes-ti/ipc/ti-ipc-rtos_3.40.01.08.bb >> +++ /dev/null >> @@ -1,9 +0,0 @@ >> -require ti-ipc-rtos.inc >> - >> -PV = "3_40_01_08" >> -PR = "r0" >> - >> -LIC_FILES_CHKSUM = >> "file://${S}/ipc-linux.mak;beginline=1;endline=30;md5=7b327f9b710fd7c95e545b91cec79255" >> - >> -SRC_URI[ipczip.md5sum] = "738e1d54da9fab94899ad162f1ad78de" >> -SRC_URI[ipczip.sha256sum] = >> "8cc7d40ad7eab916eae109b355dc7e7876c6ab82e3b40bf5b6bf60474312d6cf" >> diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb >> b/recipes-ti/ipc/ti-ipc-rtos_git.bb >> new file mode 100644 >> index 0000000..aaea3f7 >> --- /dev/null >> +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb >> @@ -0,0 +1,51 @@ >> +require recipes-ti/includes/ti-paths.inc >> +require recipes-ti/includes/ti-staging.inc >> +require ti-ipc.inc >> + >> +DEPENDS += "ti-cgt6x-native ti-xdctools ti-sysbios" >> + >> +PACKAGES =+ "${PN}-fw" >> +FILES_${PN}-fw = "${libdir}/firmware/*" >> +FILES_${PN}-dev += "${IPC_INSTALL_DIR_RECIPE}" >> + >> +INSANE_SKIP_${PN}-fw += "arch" >> +INSANE_SKIP_${PN}-dev += "arch" >> + >> +ALLOW_EMPTY_${PN} = "1" >> + >> +IPC_TARGETS = "" >> +IPC_TARGETS_omap-a15 = "\ >> + gnu.targets.arm.A15F="${A15_TOOLCHAIN_INSTALL_DIR}" \ >> + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \ >> + ti.targets.arm.elf.M4="${M4_TOOLCHAIN_INSTALL_DIR}" \ >> + ti.targets.arm.elf.M4F="${M4_TOOLCHAIN_INSTALL_DIR}" \ >> +" >> + > > If you are building for the baremetal A15 and M4, you need to add > gcc-arm-none-eabi-native and ti-ccsv6-native to the dependencies. > >> +IPC_TARGETS_keystone = " \ >> + gnu.targets.arm.A15F="${A15_TOOLCHAIN_INSTALL_DIR}" \ >> + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \ >> +" >> + >> +EXTRA_OEMAKE = "\ >> + PLATFORM=${PLATFORM} \ >> + XDC_INSTALL_DIR="${XDC_INSTALL_DIR}" \ >> + BIOS_INSTALL_DIR="${SYSBIOS_INSTALL_DIR}" \ >> + ${IPC_TARGETS} \ >> +" >> + >> +do_compile() { >> + oe_runmake -f ipc-bios.mak clean >> + oe_runmake -f ipc-bios.mak all I just checked with how we were doing this internally, and we were only building the libs. Do we also want to make the tests also? >> +} >> + >> +do_compile_append() { >> + sourceipk_do_create_srcipk >> +} >> + >> +do_install() { >> + install -d ${D}${IPC_INSTALL_DIR_RECIPE} >> + cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE} >> + >> + install -d ${D}${libdir}/firmware >> + find . -name "*.xe66" -type f | xargs -I {} install -m 0644 {} >> ${D}${libdir}/firmware/ >> +} >> diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc >> new file mode 100644 >> index 0000000..325fbc4 >> --- /dev/null >> +++ b/recipes-ti/ipc/ti-ipc.inc >> @@ -0,0 +1,26 @@ >> +DESCRIPTION = "TI Inter Process Communication (IPC) Mechanisms (for >> Uni- and Multi- Processor Configurations)" >> +HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:IPC" >> + >> +LICENSE = "BSD" >> +LIC_FILES_CHKSUM = >> "file://${S}/ipc-linux.mak;beginline=1;endline=30;md5=7b327f9b710fd7c95e545b91cec79255" >> + >> +BRANCH = "ipc-next" >> + >> +SRC_URI = >> "git://git.ti.com/ipc/ipcdev.git;protocol=git;branch=${BRANCH}" >> + >> +#Corresponds to 3.41.00.03_eng >> +SRCREV = "354db7d3b6aa397e6ab4b86fdbd312746fd46a07" >> + >> +S = "${WORKDIR}/git" >> +PV = "3.41.00.03" >> +PR = "r1" > > Can you use "INC_PR"? > >> + >> +PACKAGE_ARCH = "${MACHINE_ARCH}" >> + >> +PLATFORM = "UNKNOWN" >> +PLATFORM_omap5-evm = "OMAP54XX" >> +PLATFORM_dra7xx = "DRA7XX" >> +PLATFORM_k2hk-evm = "TCI6638" >> +PLATFORM_k2l-evm = "TCI6630" >> +PLATFORM_k2e-evm = "66AK2E" >> +PLATFORM_k2g-evm = "66AK2G" >> diff --git a/recipes-ti/ipc/ti-ipc_git.bb b/recipes-ti/ipc/ti-ipc_git.bb >> index a6b6a90..070247b 100644 >> --- a/recipes-ti/ipc/ti-ipc_git.bb >> +++ b/recipes-ti/ipc/ti-ipc_git.bb >> @@ -1,32 +1,13 @@ >> DESCRIPTION = "TI Inter Process Communication (IPC) Mechanisms (for >> Uni- and Multi- Processor Configurations)" >> HOMEPAGE="http://processors.wiki.ti.com/index.php/Category:IPC" >> -LICENSE = "BSD" >> -LIC_FILES_CHKSUM = >> "file://${S}/ipc-linux.mak;beginline=1;endline=30;md5=7b327f9b710fd7c95e545b91cec79255" >> +require ti-ipc.inc >> DEPENDS += "virtual/kernel" >> -PV = "3.41.00.02" >> - >> -BRANCH = "ipc-next" >> -SRC_URI = >> "git://git.ti.com/ipc/ipcdev.git;protocol=git;branch=${BRANCH} \ >> - file://tiipclad-daemon.sh \ >> - file://0001-Add-kernel-build-dir.patch \ >> +SRC_URI += "file://tiipclad-daemon.sh \ >> + file://0001-Add-kernel-build-dir.patch \ >> " >> -# Commit corresponds to 3.41.00.02-eng >> -SRCREV = "b34c8ada4ed243cd52e504bf2c6fb3451dc730f8" >> - >> -S = "${WORKDIR}/git" >> - >> -PACKAGE_ARCH = "${MACHINE_ARCH}" >> - >> -PLATFORM = "UNKNOWN" >> -PLATFORM_omap5-evm = "OMAP54XX" >> -PLATFORM_dra7xx = "DRA7XX" >> -PLATFORM_k2hk-evm = "TCI6638" >> -PLATFORM_k2l-evm = "TCI6630" >> -PLATFORM_k2e-evm = "66AK2E" >> -PLATFORM_k2g-evm = "66AK2E" >> DAEMON = "UNKNOWN" >> DAEMON_omap5-evm = "lad_omap54xx_smp" >> @@ -34,7 +15,7 @@ DAEMON_dra7xx = "lad_dra7xx" >> DAEMON_k2hk-evm = "lad_tci6638" >> DAEMON_k2l-evm = "lad_tci6630" >> DAEMON_k2e-evm = "lad_66ak2e" >> -DAEMON_k2g-evm = "lad_66ak2e" >> +DAEMON_k2g-evm = "lad_66ak2g" >> inherit autotools-brokensep pkgconfig update-rc.d >