* [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
@ 2016-05-11 18:00 Sam Nelson
2016-05-13 18:56 ` Denys Dmytriyenko
0 siblings, 1 reply; 9+ messages in thread
From: Sam Nelson @ 2016-05-11 18:00 UTC (permalink / raw)
To: meta-ti
Updated to new version 3.42.01.03
Added procedures and source urls for adding examples and documentation
into src package.
Added dependency on doxygen for document creation
Added big endian c66 targets
Updated uniform formatting
Signed-off-by: Sam Nelson <sam.nelson@ti.com>
---
recipes-ti/ipc/ti-ipc-rtos_git.bb | 119 ++++++++++++++++++++++++++++++++------
recipes-ti/ipc/ti-ipc.inc | 6 +-
2 files changed, 104 insertions(+), 21 deletions(-)
diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb b/recipes-ti/ipc/ti-ipc-rtos_git.bb
index 4c0fc74..8fb448f 100644
--- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
+++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
@@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
require recipes-ti/includes/ti-staging.inc
require ti-ipc.inc
-PR = "${INC_PR}.4"
+TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
+TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
+TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
+TI_IPC_EXAMPLES_GIT_BRANCH = "master"
+TI_IPC_EXAMPLES_NAME = "ipc-examples"
-DEPENDS = "ti-xdctools ti-sysbios"
+SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
+destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
+protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
+branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
+name=${TI_IPC_EXAMPLES_NAME}"
+
+TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
+TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
+TI_IPC_METADATA_GIT_PROTOCOL = "git"
+TI_IPC_METADATA_GIT_BRANCH = "master"
+TI_IPC_METADATA_NAME = "ipc-metadata"
+
+SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
+destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
+protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
+branch=${TI_IPC_METADATA_GIT_BRANCH};\
+name=${TI_IPC_METADATA_NAME}"
+
+# Corresponds to tag: 3.42.01.03
+SRCREV_ipc-examples = "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
+
+# Corresponds to tag: 3.42.01.03
+SRCREV_ipc-metadata = "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
+
+SRCREV_FORMAT = "ipc-metadata_ipc-examples"
+
+S_ipc-examples = "${WORKDIR}/git/ipc-examples"
+S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
+
+PR = "${INC_PR}.0"
+
+DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
DEPENDS_append_keystone = " ti-cgt6x-native \
gcc-arm-none-eabi-native \
"
@@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
ti-ccsv6-native \
gcc-arm-none-eabi-native \
"
+DEPENDS_append_c66x = " ti-cgt6x-native \
+"
PACKAGES =+ "${PN}-fw"
FILES_${PN}-fw = "${base_libdir}/firmware/*"
@@ -26,6 +63,7 @@ IPC_TARGETS = ""
IPC_TARGETS_omap-a15 = "\
gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
+ ti.targets.elf.C66_big_endian="${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}" \
"
@@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
IPC_TARGETS_keystone = " \
gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
+ ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
+"
+
+IPC_TARGETS_c66x = " \
+ ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
+ ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
+ ${IPC_TARGETS} \
"
+RELEASE_TYPE = "GA"
+RELEASE_SUFFIX = ""
+
+SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
+SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.', '_')}${RELEASE_SUFFIX}"
+
do_compile() {
- oe_runmake clean
- oe_runmake all
+ oe_runmake -f ipc-bios.mak clean
+ oe_runmake -f ipc-bios.mak release
+
+ cd ${S_ipc-metadata}
+ oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
+ BUILD_HOST_OS="linux" \
+ RELEASE_TYPE="${RELEASE_TYPE}"
+
+ cd ${S_ipc-examples}/src
+ oe_runmake .examples \
+ IPCTOOLS="${S_ipc-metadata}/src/etc"
+
+ if [ "${PLATFORM}" != "UNKNOWN" ]; then
+ oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
+ oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
+
+ if [ ! -z ${ALT_PLATFORM} ]; then
+ oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios" IPC_INSTALL_DIR="${S}"
+ oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux" IPC_INSTALL_DIR="${S}"
+ fi
+ fi
+
+ IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'`
+ # Copy docs and other meta files
+ install -d ${D}${IPC_INSTALL_DIR_RECIPE}
+ cp -pPrf ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d ${D}${IPC_INSTALL_DIR_RECIPE}
+
+ # Copy example folders corresponding to the platforms
+ if [ "${PLATFORM}" != "UNKNOWN" ]; then
+ install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
+ cp -pPf ${S_ipc-examples}/src/examples/*.* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
+ cp -pPf ${S_ipc-examples}/src/examples/makefile ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
+ cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
+ if [ ! -z ${ALT_PLATFORM} ]; then
+ cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
+ fi
+ find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type f | xargs -I {} rm {}
+ fi
}
do_compile_append() {
@@ -52,25 +138,22 @@ do_compile_append() {
}
do_install() {
- install -d ${D}${IPC_INSTALL_DIR_RECIPE}
- cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
-
- install -d ${D}${base_libdir}/firmware/ipc
- cp -pPrf ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
+ install -d ${D}${base_libdir}/firmware/ipc
+ cp -pPrf ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
}
ALTERNATIVE_PRIORITY = "5"
pkg_postinst_${PN}-fw_omap-a15 () {
- update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66 ${ALTERNATIVE_PRIORITY}
- update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66 ${ALTERNATIVE_PRIORITY}
- update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4 ${ALTERNATIVE_PRIORITY}
- update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4 ${ALTERNATIVE_PRIORITY}
+ update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66 ${ALTERNATIVE_PRIORITY}
+ update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66 ${ALTERNATIVE_PRIORITY}
+ update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4 ${ALTERNATIVE_PRIORITY}
+ update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4 ${ALTERNATIVE_PRIORITY}
}
pkg_postrm_${PN}-fw_omap-a15 () {
- update-alternatives --remove dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
- update-alternatives --remove dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
- update-alternatives --remove dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
- update-alternatives --remove dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
+ update-alternatives --remove dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
+ update-alternatives --remove dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
+ update-alternatives --remove dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
+ update-alternatives --remove dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
}
diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
index 69127aa..9c5dfed 100644
--- a/recipes-ti/ipc/ti-ipc.inc
+++ b/recipes-ti/ipc/ti-ipc.inc
@@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
TI_IPC_GIT_PROTOCOL = "git"
TI_IPC_GIT_BRANCH = "3.42"
-#Corresponds to 3.42.00.02
-TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
+#Corresponds to 3.42.01.03
+TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
BRANCH = "${TI_IPC_GIT_BRANCH}"
SRC_URI = "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
SRCREV = "${TI_IPC_SRCREV}"
S = "${WORKDIR}/git"
-PV = "3.42.00.02"
+PV = "3.42.01.03"
INC_PR = "r0"
PACKAGE_ARCH = "${MACHINE_ARCH}"
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-11 18:00 [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package Sam Nelson
@ 2016-05-13 18:56 ` Denys Dmytriyenko
2016-05-13 19:12 ` Stiffler, Jacob
2016-05-13 20:55 ` Nelson, Sam
0 siblings, 2 replies; 9+ messages in thread
From: Denys Dmytriyenko @ 2016-05-13 18:56 UTC (permalink / raw)
To: Sam Nelson; +Cc: meta-ti
Sam,
Is this v2? You didn't specify changes since v1, so I would have to dig out
the previous submission and check that past comments were addressed.
Jake,
Do you have any comments here?
On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> Updated to new version 3.42.01.03
> Added procedures and source urls for adding examples and documentation
> into src package.
> Added dependency on doxygen for document creation
> Added big endian c66 targets
> Updated uniform formatting
>
> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> ---
> recipes-ti/ipc/ti-ipc-rtos_git.bb | 119 ++++++++++++++++++++++++++++++++------
> recipes-ti/ipc/ti-ipc.inc | 6 +-
> 2 files changed, 104 insertions(+), 21 deletions(-)
>
> diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> index 4c0fc74..8fb448f 100644
> --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
> require recipes-ti/includes/ti-staging.inc
> require ti-ipc.inc
>
> -PR = "${INC_PR}.4"
> +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> +TI_IPC_EXAMPLES_NAME = "ipc-examples"
>
> -DEPENDS = "ti-xdctools ti-sysbios"
> +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> +name=${TI_IPC_EXAMPLES_NAME}"
> +
> +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> +TI_IPC_METADATA_GIT_BRANCH = "master"
> +TI_IPC_METADATA_NAME = "ipc-metadata"
> +
> +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> +name=${TI_IPC_METADATA_NAME}"
> +
> +# Corresponds to tag: 3.42.01.03
> +SRCREV_ipc-examples = "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> +
> +# Corresponds to tag: 3.42.01.03
> +SRCREV_ipc-metadata = "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> +
> +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> +
> +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> +
> +PR = "${INC_PR}.0"
> +
> +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> DEPENDS_append_keystone = " ti-cgt6x-native \
> gcc-arm-none-eabi-native \
> "
> @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> ti-ccsv6-native \
> gcc-arm-none-eabi-native \
> "
> +DEPENDS_append_c66x = " ti-cgt6x-native \
> +"
>
> PACKAGES =+ "${PN}-fw"
> FILES_${PN}-fw = "${base_libdir}/firmware/*"
> @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> IPC_TARGETS_omap-a15 = "\
> gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> + ti.targets.elf.C66_big_endian="${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}" \
> "
> @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> IPC_TARGETS_keystone = " \
> gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> + ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> +"
> +
> +IPC_TARGETS_c66x = " \
> + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> + ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> + ${IPC_TARGETS} \
> "
>
> +RELEASE_TYPE = "GA"
> +RELEASE_SUFFIX = ""
> +
> +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.', '_')}${RELEASE_SUFFIX}"
> +
> do_compile() {
> - oe_runmake clean
> - oe_runmake all
> + oe_runmake -f ipc-bios.mak clean
> + oe_runmake -f ipc-bios.mak release
> +
> + cd ${S_ipc-metadata}
> + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> + BUILD_HOST_OS="linux" \
> + RELEASE_TYPE="${RELEASE_TYPE}"
> +
> + cd ${S_ipc-examples}/src
> + oe_runmake .examples \
> + IPCTOOLS="${S_ipc-metadata}/src/etc"
> +
> + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> +
> + if [ ! -z ${ALT_PLATFORM} ]; then
> + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> + fi
> + fi
> +
> + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'`
> + # Copy docs and other meta files
> + install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> + cp -pPrf ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d ${D}${IPC_INSTALL_DIR_RECIPE}
> +
> + # Copy example folders corresponding to the platforms
> + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> + cp -pPf ${S_ipc-examples}/src/examples/*.* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> + cp -pPf ${S_ipc-examples}/src/examples/makefile ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> + if [ ! -z ${ALT_PLATFORM} ]; then
> + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}* ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> + fi
> + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type f | xargs -I {} rm {}
> + fi
> }
>
> do_compile_append() {
> @@ -52,25 +138,22 @@ do_compile_append() {
> }
>
> do_install() {
> - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> -
> - install -d ${D}${base_libdir}/firmware/ipc
> - cp -pPrf ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
> + install -d ${D}${base_libdir}/firmware/ipc
> + cp -pPrf ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
> }
>
> ALTERNATIVE_PRIORITY = "5"
>
> pkg_postinst_${PN}-fw_omap-a15 () {
> - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66 ${ALTERNATIVE_PRIORITY}
> - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66 ${ALTERNATIVE_PRIORITY}
> - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4 ${ALTERNATIVE_PRIORITY}
> - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4 ${ALTERNATIVE_PRIORITY}
> + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66 ${ALTERNATIVE_PRIORITY}
> + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66 ${ALTERNATIVE_PRIORITY}
> + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4 ${ALTERNATIVE_PRIORITY}
> + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4 ${ALTERNATIVE_PRIORITY}
> }
>
> pkg_postrm_${PN}-fw_omap-a15 () {
> - update-alternatives --remove dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> - update-alternatives --remove dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> - update-alternatives --remove dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> - update-alternatives --remove dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> + update-alternatives --remove dra7-dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> + update-alternatives --remove dra7-dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> + update-alternatives --remove dra7-ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> + update-alternatives --remove dra7-ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> }
> diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> index 69127aa..9c5dfed 100644
> --- a/recipes-ti/ipc/ti-ipc.inc
> +++ b/recipes-ti/ipc/ti-ipc.inc
> @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> TI_IPC_GIT_PROTOCOL = "git"
> TI_IPC_GIT_BRANCH = "3.42"
>
> -#Corresponds to 3.42.00.02
> -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> +#Corresponds to 3.42.01.03
> +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
>
> BRANCH = "${TI_IPC_GIT_BRANCH}"
> SRC_URI = "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> SRCREV = "${TI_IPC_SRCREV}"
>
> S = "${WORKDIR}/git"
> -PV = "3.42.00.02"
> +PV = "3.42.01.03"
> INC_PR = "r0"
>
> PACKAGE_ARCH = "${MACHINE_ARCH}"
> --
> 1.9.1
>
> --
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-13 18:56 ` Denys Dmytriyenko
@ 2016-05-13 19:12 ` Stiffler, Jacob
2016-05-17 18:39 ` Nelson, Sam
2016-05-13 20:55 ` Nelson, Sam
1 sibling, 1 reply; 9+ messages in thread
From: Stiffler, Jacob @ 2016-05-13 19:12 UTC (permalink / raw)
To: Dmytriyenko, Denys, Nelson, Sam; +Cc: meta-ti
> -----Original Message-----
> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> Sent: Friday, May 13, 2016 2:57 PM
> To: Nelson, Sam
> Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> documentation with the package
>
> Sam,
>
> Is this v2? You didn't specify changes since v1, so I would have to dig out
> the previous submission and check that past comments were addressed.
>
> Jake,
>
> Do you have any comments here?
>
>
> On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > Updated to new version 3.42.01.03
> > Added procedures and source urls for adding examples and
> documentation
> > into src package.
> > Added dependency on doxygen for document creation
> > Added big endian c66 targets
> > Updated uniform formatting
> >
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > ---
> > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> ++++++++++++++++++++++++++++++++------
> > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > 2 files changed, 104 insertions(+), 21 deletions(-)
> >
> > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb b/recipes-ti/ipc/ti-ipc-
> rtos_git.bb
> > index 4c0fc74..8fb448f 100644
> > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
> > require recipes-ti/includes/ti-staging.inc
> > require ti-ipc.inc
> >
> > -PR = "${INC_PR}.4"
> > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> >
> > -DEPENDS = "ti-xdctools ti-sysbios"
> > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > +name=${TI_IPC_EXAMPLES_NAME}"
> > +
> > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > +
> > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > +name=${TI_IPC_METADATA_NAME}"
> > +
> > +# Corresponds to tag: 3.42.01.03
> > +SRCREV_ipc-examples = "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > +
> > +# Corresponds to tag: 3.42.01.03
> > +SRCREV_ipc-metadata = "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > +
> > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
I'm not sure about this SRCREV_FORMAT. Should it also contain the main ipc git SRCREV? And is this needed if SRCPV is not in the PV?
> > +
> > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > +
> > +PR = "${INC_PR}.0"
> > +
> > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > DEPENDS_append_keystone = " ti-cgt6x-native \
> > gcc-arm-none-eabi-native \
> > "
> > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > ti-ccsv6-native \
> > gcc-arm-none-eabi-native \
> > "
> > +DEPENDS_append_c66x = " ti-cgt6x-native \
> > +"
These c66x machines have not yet been accepted into meta-ti.
> >
> > PACKAGES =+ "${PN}-fw"
> > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > IPC_TARGETS_omap-a15 = "\
> > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> ti.targets.elf.C66_big_endian="${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}" \
> > "
> > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> > IPC_TARGETS_keystone = " \
> > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> c6x" \
> > +"
> > +
> > +IPC_TARGETS_c66x = " \
> > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > + ${IPC_TARGETS} \
> > "
> >
> > +RELEASE_TYPE = "GA"
> > +RELEASE_SUFFIX = ""
> > +
> > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> '_')}${RELEASE_SUFFIX}"
This already specified in meta-arago's arago-source-ipk.conf?
> > +
> > do_compile() {
> > - oe_runmake clean
> > - oe_runmake all
> > + oe_runmake -f ipc-bios.mak clean
> > + oe_runmake -f ipc-bios.mak release
> > +
> > + cd ${S_ipc-metadata}
> > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > + BUILD_HOST_OS="linux" \
> > + RELEASE_TYPE="${RELEASE_TYPE}"
> > +
> > + cd ${S_ipc-examples}/src
> > + oe_runmake .examples \
> > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > +
> > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > +
> > + if [ ! -z ${ALT_PLATFORM} ]; then
> > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios"
> IPC_INSTALL_DIR="${S}"
> > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux"
> IPC_INSTALL_DIR="${S}"
> > + fi
> > + fi
> > +
> > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'`
> > + # Copy docs and other meta files
> > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > + cp -pPrf ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> ${D}${IPC_INSTALL_DIR_RECIPE}
> > +
> > + # Copy example folders corresponding to the platforms
> > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + if [ ! -z ${ALT_PLATFORM} ]; then
> > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + fi
> > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type f |
> xargs -I {} rm {}
> > + fi
Why do all of this installation in the do_compile stage? Should it be in do_install?
> > }
> >
> > do_compile_append() {
> > @@ -52,25 +138,22 @@ do_compile_append() {
> > }
> >
> > do_install() {
> > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > -
> > - install -d ${D}${base_libdir}/firmware/ipc
> > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> ${D}${base_libdir}/firmware/ipc
> > + install -d ${D}${base_libdir}/firmware/ipc
> > + cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> ${D}${base_libdir}/firmware/ipc
> > }
> >
> > ALTERNATIVE_PRIORITY = "5"
> >
> > pkg_postinst_${PN}-fw_omap-a15 () {
> > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> dsp1-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> dsp2-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> ipu1-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> ipu2-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> dsp1-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> dsp2-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> ipu1-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> ipu2-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > }
> >
> > pkg_postrm_${PN}-fw_omap-a15 () {
> > - update-alternatives --remove dra7-dsp1-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > - update-alternatives --remove dra7-dsp2-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > - update-alternatives --remove dra7-ipu1-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > - update-alternatives --remove dra7-ipu2-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > + update-alternatives --remove dra7-dsp1-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > + update-alternatives --remove dra7-dsp2-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > + update-alternatives --remove dra7-ipu1-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > + update-alternatives --remove dra7-ipu2-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > }
> > diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> > index 69127aa..9c5dfed 100644
> > --- a/recipes-ti/ipc/ti-ipc.inc
> > +++ b/recipes-ti/ipc/ti-ipc.inc
> > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > TI_IPC_GIT_PROTOCOL = "git"
> > TI_IPC_GIT_BRANCH = "3.42"
> >
> > -#Corresponds to 3.42.00.02
> > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > +#Corresponds to 3.42.01.03
> > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> >
> > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > SRC_URI =
> "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > SRCREV = "${TI_IPC_SRCREV}"
> >
> > S = "${WORKDIR}/git"
> > -PV = "3.42.00.02"
> > +PV = "3.42.01.03"
> > INC_PR = "r0"
> >
> > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > --
> > 1.9.1
> >
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
> --
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-13 18:56 ` Denys Dmytriyenko
2016-05-13 19:12 ` Stiffler, Jacob
@ 2016-05-13 20:55 ` Nelson, Sam
1 sibling, 0 replies; 9+ messages in thread
From: Nelson, Sam @ 2016-05-13 20:55 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-ti
Yes. I have addressed the couple of comments from the previous patch. I had to rebase the changes completely to the new baseline.
(That is why I did not do a v2. Sorry for the confusion)
---
Here are the changes based on previous comments
- Added SRCREV_FORMAT.
- avoided creating source package after install and added the example steps to do_compile
---
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, May 13, 2016 2:57 PM
> To: Nelson, Sam
> Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> documentation with the package
>
> Sam,
>
> Is this v2? You didn't specify changes since v1, so I would have to dig out the
> previous submission and check that past comments were addressed.
>
> Jake,
>
> Do you have any comments here?
>
>
> On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > Updated to new version 3.42.01.03
> > Added procedures and source urls for adding examples and documentation
> > into src package.
> > Added dependency on doxygen for document creation Added big endian
> c66
> > targets Updated uniform formatting
> >
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > ---
> > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> ++++++++++++++++++++++++++++++++------
> > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > 2 files changed, 104 insertions(+), 21 deletions(-)
> >
> > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > index 4c0fc74..8fb448f 100644
> > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc require
> > recipes-ti/includes/ti-staging.inc
> > require ti-ipc.inc
> >
> > -PR = "${INC_PR}.4"
> > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> >
> > -DEPENDS = "ti-xdctools ti-sysbios"
> > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > +name=${TI_IPC_EXAMPLES_NAME}"
> > +
> > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > +
> > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > +name=${TI_IPC_METADATA_NAME}"
> > +
> > +# Corresponds to tag: 3.42.01.03
> > +SRCREV_ipc-examples = "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > +
> > +# Corresponds to tag: 3.42.01.03
> > +SRCREV_ipc-metadata = "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > +
> > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> > +
> > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > +
> > +PR = "${INC_PR}.0"
> > +
> > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > DEPENDS_append_keystone = " ti-cgt6x-native \
> > gcc-arm-none-eabi-native \ "
> > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > ti-ccsv6-native \
> > gcc-arm-none-eabi-native \ "
> > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
> >
> > PACKAGES =+ "${PN}-fw"
> > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > IPC_TARGETS_omap-a15 = "\
> > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> > + ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cg
> > + t-c6x" \
> > ti.targets.arm.elf.M4="${M4_TOOLCHAIN_INSTALL_DIR}" \
> > ti.targets.arm.elf.M4F="${M4_TOOLCHAIN_INSTALL_DIR}" \ "
> > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> IPC_TARGETS_keystone =
> > " \
> > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> > +ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> c6x" \ "
> > +
> > +IPC_TARGETS_c66x = " \
> > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x" \
> > +
> > +ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > + ${IPC_TARGETS} \
> > "
> >
> > +RELEASE_TYPE = "GA"
> > +RELEASE_SUFFIX = ""
> > +
> > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.', '_')}${RELEASE_SUFFIX}"
> > +
> > do_compile() {
> > - oe_runmake clean
> > - oe_runmake all
> > + oe_runmake -f ipc-bios.mak clean
> > + oe_runmake -f ipc-bios.mak release
> > +
> > + cd ${S_ipc-metadata}
> > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > + BUILD_HOST_OS="linux" \
> > + RELEASE_TYPE="${RELEASE_TYPE}"
> > +
> > + cd ${S_ipc-examples}/src
> > + oe_runmake .examples \
> > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > +
> > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > +
> > + if [ ! -z ${ALT_PLATFORM} ]; then
> > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios"
> IPC_INSTALL_DIR="${S}"
> > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux"
> IPC_INSTALL_DIR="${S}"
> > + fi
> > + fi
> > +
> > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'` #
> > + Copy docs and other meta files install -d
> > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > + ${D}${IPC_INSTALL_DIR_RECIPE}
> > +
> > + # Copy example folders corresponding to the platforms if [
> > + "${PLATFORM}" != "UNKNOWN" ]; then
> > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + if [ ! -z ${ALT_PLATFORM} ]; then
> > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > + fi
> > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type f
> > + | xargs -I {} rm {} fi
> > }
> >
> > do_compile_append() {
> > @@ -52,25 +138,22 @@ do_compile_append() { }
> >
> > do_install() {
> > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > -
> > - install -d ${D}${base_libdir}/firmware/ipc
> > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
> > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > + ${S}/packages/ti/ipc/tests/bin/* ${D}${base_libdir}/firmware/ipc
> > }
> >
> > ALTERNATIVE_PRIORITY = "5"
> >
> > pkg_postinst_${PN}-fw_omap-a15 () {
> > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> dsp1-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> dsp2-fw.xe66 ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> ipu1-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> ipu2-fw.xem4 ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> ${ALTERNATIVE_PRIORITY}
> > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> > + dra7-dsp1-fw.xe66
> > + ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > + ${ALTERNATIVE_PRIORITY} update-alternatives --install
> > + /lib/firmware/dra7-dsp2-fw.xe66 dra7-dsp2-fw.xe66
> > + ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > + ${ALTERNATIVE_PRIORITY} update-alternatives --install
> > + /lib/firmware/dra7-ipu1-fw.xem4 dra7-ipu1-fw.xem4
> > + ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > + ${ALTERNATIVE_PRIORITY} update-alternatives --install
> > + /lib/firmware/dra7-ipu2-fw.xem4 dra7-ipu2-fw.xem4
> > + ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > + ${ALTERNATIVE_PRIORITY}
> > }
> >
> > pkg_postrm_${PN}-fw_omap-a15 () {
> > - update-alternatives --remove dra7-dsp1-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > - update-alternatives --remove dra7-dsp2-fw.xe66
> ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > - update-alternatives --remove dra7-ipu1-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > - update-alternatives --remove dra7-ipu2-fw.xem4
> ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > + update-alternatives --remove dra7-dsp1-fw.xe66
> > + ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > + update-alternatives --remove dra7-dsp2-fw.xe66
> > + ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > + update-alternatives --remove dra7-ipu1-fw.xem4
> > + ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > + update-alternatives --remove dra7-ipu2-fw.xem4
> > + ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > }
> > diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> > index 69127aa..9c5dfed 100644
> > --- a/recipes-ti/ipc/ti-ipc.inc
> > +++ b/recipes-ti/ipc/ti-ipc.inc
> > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > TI_IPC_GIT_PROTOCOL = "git"
> > TI_IPC_GIT_BRANCH = "3.42"
> >
> > -#Corresponds to 3.42.00.02
> > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > +#Corresponds to 3.42.01.03
> > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> >
> > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > SRC_URI =
> "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > SRCREV = "${TI_IPC_SRCREV}"
> >
> > S = "${WORKDIR}/git"
> > -PV = "3.42.00.02"
> > +PV = "3.42.01.03"
> > INC_PR = "r0"
> >
> > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > --
> > 1.9.1
> >
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-13 19:12 ` Stiffler, Jacob
@ 2016-05-17 18:39 ` Nelson, Sam
2016-05-17 18:56 ` Denys Dmytriyenko
0 siblings, 1 reply; 9+ messages in thread
From: Nelson, Sam @ 2016-05-17 18:39 UTC (permalink / raw)
To: Stiffler, Jacob, Dmytriyenko, Denys; +Cc: meta-ti
> -----Original Message-----
> From: Stiffler, Jacob
> Sent: Friday, May 13, 2016 3:13 PM
> To: Dmytriyenko, Denys; Nelson, Sam
> Cc: meta-ti@yoctoproject.org
> Subject: RE: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> documentation with the package
>
>
>
> > -----Original Message-----
> > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > Sent: Friday, May 13, 2016 2:57 PM
> > To: Nelson, Sam
> > Cc: meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples
> > and documentation with the package
> >
> > Sam,
> >
> > Is this v2? You didn't specify changes since v1, so I would have to
> > dig out the previous submission and check that past comments were
> addressed.
> >
> > Jake,
> >
> > Do you have any comments here?
> >
> >
> > On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > > Updated to new version 3.42.01.03
> > > Added procedures and source urls for adding examples and
> > documentation
> > > into src package.
> > > Added dependency on doxygen for document creation Added big endian
> > > c66 targets Updated uniform formatting
> > >
> > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > ---
> > > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> > ++++++++++++++++++++++++++++++++------
> > > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > > 2 files changed, 104 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > b/recipes-ti/ipc/ti-ipc-
> > rtos_git.bb
> > > index 4c0fc74..8fb448f 100644
> > > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc require
> > > recipes-ti/includes/ti-staging.inc
> > > require ti-ipc.inc
> > >
> > > -PR = "${INC_PR}.4"
> > > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> > >
> > > -DEPENDS = "ti-xdctools ti-sysbios"
> > > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > > +name=${TI_IPC_EXAMPLES_NAME}"
> > > +
> > > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > > +
> > > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > > +name=${TI_IPC_METADATA_NAME}"
> > > +
> > > +# Corresponds to tag: 3.42.01.03
> > > +SRCREV_ipc-examples =
> "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > > +
> > > +# Corresponds to tag: 3.42.01.03
> > > +SRCREV_ipc-metadata =
> "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > > +
> > > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
>
> I'm not sure about this SRCREV_FORMAT. Should it also contain the main ipc
> git SRCREV? And is this needed if SRCPV is not in the PV?
Look like there is some confusion here. I am also not clear if this is needed if SRCREV is not used in PV.
>
> > > +
> > > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > > +
> > > +PR = "${INC_PR}.0"
> > > +
> > > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > > DEPENDS_append_keystone = " ti-cgt6x-native \
> > > gcc-arm-none-eabi-native \ "
> > > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > > ti-ccsv6-native \
> > > gcc-arm-none-eabi-native \ "
> > > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
>
> These c66x machines have not yet been accepted into meta-ti.
Ok. I will remove this for now.
>
> > >
> > > PACKAGES =+ "${PN}-fw"
> > > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > > IPC_TARGETS_omap-a15 = "\
> > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > \
> > > +
> > ti.targets.elf.C66_big_endian="${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}" \ "
> > > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> IPC_TARGETS_keystone
> > > = " \
> > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > \
> > > +
> > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> > c6x" \
> > > +"
> > > +
> > > +IPC_TARGETS_c66x = " \
> > > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > +\
> > > +
> > ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > > + ${IPC_TARGETS} \
> > > "
> > >
> > > +RELEASE_TYPE = "GA"
> > > +RELEASE_SUFFIX = ""
> > > +
> > > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> > '_')}${RELEASE_SUFFIX}"
>
> This already specified in meta-arago's arago-source-ipk.conf?
I don't see anything related ti-ipc-rtos, in the master branch meta-arago: arago-source-ipk.conf.
Am I missing something?
>
> > > +
> > > do_compile() {
> > > - oe_runmake clean
> > > - oe_runmake all
> > > + oe_runmake -f ipc-bios.mak clean
> > > + oe_runmake -f ipc-bios.mak release
> > > +
> > > + cd ${S_ipc-metadata}
> > > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > > + BUILD_HOST_OS="linux" \
> > > + RELEASE_TYPE="${RELEASE_TYPE}"
> > > +
> > > + cd ${S_ipc-examples}/src
> > > + oe_runmake .examples \
> > > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > > +
> > > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > > +
> > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios"
> > IPC_INSTALL_DIR="${S}"
> > > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux"
> > IPC_INSTALL_DIR="${S}"
> > > + fi
> > > + fi
> > > +
> > > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'` #
> > > + Copy docs and other meta files install -d
> > > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > ${D}${IPC_INSTALL_DIR_RECIPE}
> > > +
> > > + # Copy example folders corresponding to the platforms if [
> > > + "${PLATFORM}" != "UNKNOWN" ]; then
> > > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > + fi
> > > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type
> > > + f |
> > xargs -I {} rm {}
> > > + fi
>
> Why do all of this installation in the do_compile stage? Should it be in
> do_install?
>
This is rtos package and everything is that is created is part of the source package.
Only what goes into file system is in the do_install step.
> > > }
> > >
> > > do_compile_append() {
> > > @@ -52,25 +138,22 @@ do_compile_append() { }
> > >
> > > do_install() {
> > > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > > -
> > > - install -d ${D}${base_libdir}/firmware/ipc
> > > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> > ${D}${base_libdir}/firmware/ipc
> > > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > > + ${S}/packages/ti/ipc/tests/bin/*
> > ${D}${base_libdir}/firmware/ipc
> > > }
> > >
> > > ALTERNATIVE_PRIORITY = "5"
> > >
> > > pkg_postinst_${PN}-fw_omap-a15 () {
> > > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> > dsp1-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > ${ALTERNATIVE_PRIORITY}
> > > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> > dsp2-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > ${ALTERNATIVE_PRIORITY}
> > > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> > ipu1-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > ${ALTERNATIVE_PRIORITY}
> > > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> > ipu2-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > ${ALTERNATIVE_PRIORITY}
> > > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> > > + dra7-
> > dsp1-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > ${ALTERNATIVE_PRIORITY}
> > > + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66
> > > + dra7-
> > dsp2-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > ${ALTERNATIVE_PRIORITY}
> > > + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4
> > > + dra7-
> > ipu1-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > ${ALTERNATIVE_PRIORITY}
> > > + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4
> > > + dra7-
> > ipu2-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > ${ALTERNATIVE_PRIORITY}
> > > }
> > >
> > > pkg_postrm_${PN}-fw_omap-a15 () {
> > > - update-alternatives --remove dra7-dsp1-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > - update-alternatives --remove dra7-dsp2-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > - update-alternatives --remove dra7-ipu1-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > - update-alternatives --remove dra7-ipu2-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > + update-alternatives --remove dra7-dsp1-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > + update-alternatives --remove dra7-dsp2-fw.xe66
> > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > + update-alternatives --remove dra7-ipu1-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > + update-alternatives --remove dra7-ipu2-fw.xem4
> > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > }
> > > diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> > > index 69127aa..9c5dfed 100644
> > > --- a/recipes-ti/ipc/ti-ipc.inc
> > > +++ b/recipes-ti/ipc/ti-ipc.inc
> > > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > > TI_IPC_GIT_PROTOCOL = "git"
> > > TI_IPC_GIT_BRANCH = "3.42"
> > >
> > > -#Corresponds to 3.42.00.02
> > > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > > +#Corresponds to 3.42.01.03
> > > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> > >
> > > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > > SRC_URI =
> >
> "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > > SRCREV = "${TI_IPC_SRCREV}"
> > >
> > > S = "${WORKDIR}/git"
> > > -PV = "3.42.00.02"
> > > +PV = "3.42.01.03"
> > > INC_PR = "r0"
> > >
> > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > --
> > > 1.9.1
> > >
> > > --
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti
> > --
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-17 18:39 ` Nelson, Sam
@ 2016-05-17 18:56 ` Denys Dmytriyenko
2016-05-17 21:32 ` Nelson, Sam
0 siblings, 1 reply; 9+ messages in thread
From: Denys Dmytriyenko @ 2016-05-17 18:56 UTC (permalink / raw)
To: Nelson, Sam; +Cc: meta-ti
On Tue, May 17, 2016 at 02:39:45PM -0400, Nelson, Sam wrote:
>
>
> > -----Original Message-----
> > From: Stiffler, Jacob
> > Sent: Friday, May 13, 2016 3:13 PM
> > To: Dmytriyenko, Denys; Nelson, Sam
> > Cc: meta-ti@yoctoproject.org
> > Subject: RE: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> > documentation with the package
> >
> >
> >
> > > -----Original Message-----
> > > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > > Sent: Friday, May 13, 2016 2:57 PM
> > > To: Nelson, Sam
> > > Cc: meta-ti@yoctoproject.org
> > > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples
> > > and documentation with the package
> > >
> > > Sam,
> > >
> > > Is this v2? You didn't specify changes since v1, so I would have to
> > > dig out the previous submission and check that past comments were
> > addressed.
> > >
> > > Jake,
> > >
> > > Do you have any comments here?
> > >
> > >
> > > On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > > > Updated to new version 3.42.01.03
> > > > Added procedures and source urls for adding examples and
> > > documentation
> > > > into src package.
> > > > Added dependency on doxygen for document creation Added big endian
> > > > c66 targets Updated uniform formatting
> > > >
> > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > ---
> > > > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> > > ++++++++++++++++++++++++++++++++------
> > > > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > > > 2 files changed, 104 insertions(+), 21 deletions(-)
> > > >
> > > > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > b/recipes-ti/ipc/ti-ipc-
> > > rtos_git.bb
> > > > index 4c0fc74..8fb448f 100644
> > > > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc require
> > > > recipes-ti/includes/ti-staging.inc
> > > > require ti-ipc.inc
> > > >
> > > > -PR = "${INC_PR}.4"
> > > > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > > > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > > > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > > > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > > > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> > > >
> > > > -DEPENDS = "ti-xdctools ti-sysbios"
> > > > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > > > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > > > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > > > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > > > +name=${TI_IPC_EXAMPLES_NAME}"
> > > > +
> > > > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > > > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > > > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > > > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > > > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > > > +
> > > > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > > > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > > > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > > > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > > > +name=${TI_IPC_METADATA_NAME}"
> > > > +
> > > > +# Corresponds to tag: 3.42.01.03
> > > > +SRCREV_ipc-examples =
> > "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > > > +
> > > > +# Corresponds to tag: 3.42.01.03
> > > > +SRCREV_ipc-metadata =
> > "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > > > +
> > > > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> >
> > I'm not sure about this SRCREV_FORMAT. Should it also contain the main ipc
> > git SRCREV? And is this needed if SRCPV is not in the PV?
>
> Look like there is some confusion here. I am also not clear if this is needed if SRCREV is not used in PV.
>
> >
> > > > +
> > > > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > > > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > > > +
> > > > +PR = "${INC_PR}.0"
> > > > +
> > > > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > > > DEPENDS_append_keystone = " ti-cgt6x-native \
> > > > gcc-arm-none-eabi-native \ "
> > > > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > > > ti-ccsv6-native \
> > > > gcc-arm-none-eabi-native \ "
> > > > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
> >
> > These c66x machines have not yet been accepted into meta-ti.
>
> Ok. I will remove this for now.
>
> >
> > > >
> > > > PACKAGES =+ "${PN}-fw"
> > > > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > > > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > > > IPC_TARGETS_omap-a15 = "\
> > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > \
> > > > +
> > > ti.targets.elf.C66_big_endian="${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}" \ "
> > > > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> > IPC_TARGETS_keystone
> > > > = " \
> > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > \
> > > > +
> > > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> > > c6x" \
> > > > +"
> > > > +
> > > > +IPC_TARGETS_c66x = " \
> > > > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > +\
> > > > +
> > > ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > > > + ${IPC_TARGETS} \
> > > > "
> > > >
> > > > +RELEASE_TYPE = "GA"
> > > > +RELEASE_SUFFIX = ""
> > > > +
> > > > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > > > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> > > '_')}${RELEASE_SUFFIX}"
> >
> > This already specified in meta-arago's arago-source-ipk.conf?
>
> I don't see anything related ti-ipc-rtos, in the master branch meta-arago: arago-source-ipk.conf.
> Am I missing something?
Yes, no SRCIPK stuff in meta-ti allowed.
> > > > +
> > > > do_compile() {
> > > > - oe_runmake clean
> > > > - oe_runmake all
> > > > + oe_runmake -f ipc-bios.mak clean
> > > > + oe_runmake -f ipc-bios.mak release
> > > > +
> > > > + cd ${S_ipc-metadata}
> > > > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > > > + BUILD_HOST_OS="linux" \
> > > > + RELEASE_TYPE="${RELEASE_TYPE}"
> > > > +
> > > > + cd ${S_ipc-examples}/src
> > > > + oe_runmake .examples \
> > > > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > > > +
> > > > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > > > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > > > +
> > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="bios"
> > > IPC_INSTALL_DIR="${S}"
> > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM} HOSTOS="linux"
> > > IPC_INSTALL_DIR="${S}"
> > > > + fi
> > > > + fi
> > > > +
> > > > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'` #
> > > > + Copy docs and other meta files install -d
> > > > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > > > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > > ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > +
> > > > + # Copy example folders corresponding to the platforms if [
> > > > + "${PLATFORM}" != "UNKNOWN" ]; then
> > > > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > > > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > + fi
> > > > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip" -type
> > > > + f |
> > > xargs -I {} rm {}
> > > > + fi
> >
> > Why do all of this installation in the do_compile stage? Should it be in
> > do_install?
> >
>
> This is rtos package and everything is that is created is part of the source package.
> Only what goes into file system is in the do_install step.
>
> > > > }
> > > >
> > > > do_compile_append() {
> > > > @@ -52,25 +138,22 @@ do_compile_append() { }
> > > >
> > > > do_install() {
> > > > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > -
> > > > - install -d ${D}${base_libdir}/firmware/ipc
> > > > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> > > ${D}${base_libdir}/firmware/ipc
> > > > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > > > + ${S}/packages/ti/ipc/tests/bin/*
> > > ${D}${base_libdir}/firmware/ipc
> > > > }
> > > >
> > > > ALTERNATIVE_PRIORITY = "5"
> > > >
> > > > pkg_postinst_${PN}-fw_omap-a15 () {
> > > > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> > > dsp1-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > ${ALTERNATIVE_PRIORITY}
> > > > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> > > dsp2-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > ${ALTERNATIVE_PRIORITY}
> > > > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> > > ipu1-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > ${ALTERNATIVE_PRIORITY}
> > > > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> > > ipu2-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > ${ALTERNATIVE_PRIORITY}
> > > > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> > > > + dra7-
> > > dsp1-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > ${ALTERNATIVE_PRIORITY}
> > > > + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66
> > > > + dra7-
> > > dsp2-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > ${ALTERNATIVE_PRIORITY}
> > > > + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4
> > > > + dra7-
> > > ipu1-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > ${ALTERNATIVE_PRIORITY}
> > > > + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4
> > > > + dra7-
> > > ipu2-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > ${ALTERNATIVE_PRIORITY}
> > > > }
> > > >
> > > > pkg_postrm_${PN}-fw_omap-a15 () {
> > > > - update-alternatives --remove dra7-dsp1-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > - update-alternatives --remove dra7-dsp2-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > - update-alternatives --remove dra7-ipu1-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > - update-alternatives --remove dra7-ipu2-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > + update-alternatives --remove dra7-dsp1-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > + update-alternatives --remove dra7-dsp2-fw.xe66
> > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > + update-alternatives --remove dra7-ipu1-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > + update-alternatives --remove dra7-ipu2-fw.xem4
> > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > }
> > > > diff --git a/recipes-ti/ipc/ti-ipc.inc b/recipes-ti/ipc/ti-ipc.inc
> > > > index 69127aa..9c5dfed 100644
> > > > --- a/recipes-ti/ipc/ti-ipc.inc
> > > > +++ b/recipes-ti/ipc/ti-ipc.inc
> > > > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > > > TI_IPC_GIT_PROTOCOL = "git"
> > > > TI_IPC_GIT_BRANCH = "3.42"
> > > >
> > > > -#Corresponds to 3.42.00.02
> > > > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > > > +#Corresponds to 3.42.01.03
> > > > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> > > >
> > > > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > > > SRC_URI =
> > >
> > "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > > > SRCREV = "${TI_IPC_SRCREV}"
> > > >
> > > > S = "${WORKDIR}/git"
> > > > -PV = "3.42.00.02"
> > > > +PV = "3.42.01.03"
> > > > INC_PR = "r0"
> > > >
> > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > > --
> > > > 1.9.1
> > > >
> > > > --
> > > > _______________________________________________
> > > > meta-ti mailing list
> > > > meta-ti@yoctoproject.org
> > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > > --
> > > _______________________________________________
> > > meta-ti mailing list
> > > meta-ti@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-17 18:56 ` Denys Dmytriyenko
@ 2016-05-17 21:32 ` Nelson, Sam
2016-05-18 2:41 ` Denys Dmytriyenko
0 siblings, 1 reply; 9+ messages in thread
From: Nelson, Sam @ 2016-05-17 21:32 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-ti
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, May 17, 2016 2:57 PM
> To: Nelson, Sam
> Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> documentation with the package
>
> On Tue, May 17, 2016 at 02:39:45PM -0400, Nelson, Sam wrote:
> >
> >
> > > -----Original Message-----
> > > From: Stiffler, Jacob
> > > Sent: Friday, May 13, 2016 3:13 PM
> > > To: Dmytriyenko, Denys; Nelson, Sam
> > > Cc: meta-ti@yoctoproject.org
> > > Subject: RE: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > examples and documentation with the package
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > > > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > > > Sent: Friday, May 13, 2016 2:57 PM
> > > > To: Nelson, Sam
> > > > Cc: meta-ti@yoctoproject.org
> > > > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > > examples and documentation with the package
> > > >
> > > > Sam,
> > > >
> > > > Is this v2? You didn't specify changes since v1, so I would have
> > > > to dig out the previous submission and check that past comments
> > > > were
> > > addressed.
> > > >
> > > > Jake,
> > > >
> > > > Do you have any comments here?
> > > >
> > > >
> > > > On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > > > > Updated to new version 3.42.01.03 Added procedures and source
> > > > > urls for adding examples and
> > > > documentation
> > > > > into src package.
> > > > > Added dependency on doxygen for document creation Added big
> > > > > endian
> > > > > c66 targets Updated uniform formatting
> > > > >
> > > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > > ---
> > > > > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> > > > ++++++++++++++++++++++++++++++++------
> > > > > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > > > > 2 files changed, 104 insertions(+), 21 deletions(-)
> > > > >
> > > > > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > b/recipes-ti/ipc/ti-ipc-
> > > > rtos_git.bb
> > > > > index 4c0fc74..8fb448f 100644
> > > > > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
> > > > > require recipes-ti/includes/ti-staging.inc
> > > > > require ti-ipc.inc
> > > > >
> > > > > -PR = "${INC_PR}.4"
> > > > > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > > > > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > > > > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > > > > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > > > > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> > > > >
> > > > > -DEPENDS = "ti-xdctools ti-sysbios"
> > > > > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > > > > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > > > > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > > > > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > > > > +name=${TI_IPC_EXAMPLES_NAME}"
> > > > > +
> > > > > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > > > > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > > > > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > > > > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > > > > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > > > > +
> > > > > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > > > > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > > > > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > > > > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > > > > +name=${TI_IPC_METADATA_NAME}"
> > > > > +
> > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-examples =
> > > "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > > > > +
> > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-metadata =
> > > "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > > > > +
> > > > > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> > >
> > > I'm not sure about this SRCREV_FORMAT. Should it also contain the
> > > main ipc git SRCREV? And is this needed if SRCPV is not in the PV?
> >
> > Look like there is some confusion here. I am also not clear if this is needed if
> SRCREV is not used in PV.
Denys, Can you clarify on whether the SRCREV_FORMAT is really needed or not?
> >
> > >
> > > > > +
> > > > > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > > > > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > > > > +
> > > > > +PR = "${INC_PR}.0"
> > > > > +
> > > > > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > > > > DEPENDS_append_keystone = " ti-cgt6x-native \
> > > > > gcc-arm-none-eabi-native \ "
> > > > > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > > > > ti-ccsv6-native \
> > > > > gcc-arm-none-eabi-native \ "
> > > > > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
> > >
> > > These c66x machines have not yet been accepted into meta-ti.
> >
> > Ok. I will remove this for now.
> >
> > >
> > > > >
> > > > > PACKAGES =+ "${PN}-fw"
> > > > > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > > > > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > > > > IPC_TARGETS_omap-a15 = "\
> > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > \
> > > > > +
> > > > ti.targets.elf.C66_big_endian="${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}" \ "
> > > > > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> > > IPC_TARGETS_keystone
> > > > > = " \
> > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > \
> > > > > +
> > > > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/
> > > > cgt-
> > > > c6x" \
> > > > > +"
> > > > > +
> > > > > +IPC_TARGETS_c66x = " \
> > > > > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > +\
> > > > > +
> > > > ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > > > > + ${IPC_TARGETS} \
> > > > > "
> > > > >
> > > > > +RELEASE_TYPE = "GA"
> > > > > +RELEASE_SUFFIX = ""
> > > > > +
> > > > > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > > > > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> > > > '_')}${RELEASE_SUFFIX}"
> > >
> > > This already specified in meta-arago's arago-source-ipk.conf?
> >
> > I don't see anything related ti-ipc-rtos, in the master branch meta-arago:
> arago-source-ipk.conf.
> > Am I missing something?
>
> Yes, no SRCIPK stuff in meta-ti allowed.
>
I see. I will remove from here and send update to meta-arago.
>
> > > > > +
> > > > > do_compile() {
> > > > > - oe_runmake clean
> > > > > - oe_runmake all
> > > > > + oe_runmake -f ipc-bios.mak clean oe_runmake -f ipc-bios.mak
> > > > > + release
> > > > > +
> > > > > + cd ${S_ipc-metadata}
> > > > > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > > > > + BUILD_HOST_OS="linux" \
> > > > > + RELEASE_TYPE="${RELEASE_TYPE}"
> > > > > +
> > > > > + cd ${S_ipc-examples}/src
> > > > > + oe_runmake .examples \
> > > > > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > > > > +
> > > > > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > > > > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > > > > +
> > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> HOSTOS="bios"
> > > > IPC_INSTALL_DIR="${S}"
> > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> HOSTOS="linux"
> > > > IPC_INSTALL_DIR="${S}"
> > > > > + fi
> > > > > + fi
> > > > > +
> > > > > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'`
> > > > > + # Copy docs and other meta files install -d
> > > > > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > > > > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > > > ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > +
> > > > > + # Copy example folders corresponding to the platforms if [
> > > > > + "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > > > > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > + fi
> > > > > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip"
> > > > > + -type f |
> > > > xargs -I {} rm {}
> > > > > + fi
> > >
> > > Why do all of this installation in the do_compile stage? Should it
> > > be in do_install?
> > >
> >
> > This is rtos package and everything is that is created is part of the source
> package.
> > Only what goes into file system is in the do_install step.
> >
> > > > > }
> > > > >
> > > > > do_compile_append() {
> > > > > @@ -52,25 +138,22 @@ do_compile_append() { }
> > > > >
> > > > > do_install() {
> > > > > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > -
> > > > > - install -d ${D}${base_libdir}/firmware/ipc
> > > > > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> > > > ${D}${base_libdir}/firmware/ipc
> > > > > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > > > > + ${S}/packages/ti/ipc/tests/bin/*
> > > > ${D}${base_libdir}/firmware/ipc
> > > > > }
> > > > >
> > > > > ALTERNATIVE_PRIORITY = "5"
> > > > >
> > > > > pkg_postinst_${PN}-fw_omap-a15 () {
> > > > > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> > > > dsp1-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> > > > dsp2-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> > > > ipu1-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> > > > ipu2-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> > > > > + dra7-
> > > > dsp1-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66
> > > > > + dra7-
> > > > dsp2-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4
> > > > > + dra7-
> > > > ipu1-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4
> > > > > + dra7-
> > > > ipu2-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > ${ALTERNATIVE_PRIORITY}
> > > > > }
> > > > >
> > > > > pkg_postrm_${PN}-fw_omap-a15 () {
> > > > > - update-alternatives --remove dra7-dsp1-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > - update-alternatives --remove dra7-dsp2-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > - update-alternatives --remove dra7-ipu1-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > - update-alternatives --remove dra7-ipu2-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > + update-alternatives --remove dra7-dsp1-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > + update-alternatives --remove dra7-dsp2-fw.xe66
> > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > + update-alternatives --remove dra7-ipu1-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > + update-alternatives --remove dra7-ipu2-fw.xem4
> > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > }
> > > > > diff --git a/recipes-ti/ipc/ti-ipc.inc
> > > > > b/recipes-ti/ipc/ti-ipc.inc index 69127aa..9c5dfed 100644
> > > > > --- a/recipes-ti/ipc/ti-ipc.inc
> > > > > +++ b/recipes-ti/ipc/ti-ipc.inc
> > > > > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > > > > TI_IPC_GIT_PROTOCOL = "git"
> > > > > TI_IPC_GIT_BRANCH = "3.42"
> > > > >
> > > > > -#Corresponds to 3.42.00.02
> > > > > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > > > > +#Corresponds to 3.42.01.03
> > > > > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> > > > >
> > > > > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > > > > SRC_URI =
> > > >
> > >
> "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > > > > SRCREV = "${TI_IPC_SRCREV}"
> > > > >
> > > > > S = "${WORKDIR}/git"
> > > > > -PV = "3.42.00.02"
> > > > > +PV = "3.42.01.03"
> > > > > INC_PR = "r0"
> > > > >
> > > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > > > --
> > > > > 1.9.1
> > > > >
> > > > > --
> > > > > _______________________________________________
> > > > > meta-ti mailing list
> > > > > meta-ti@yoctoproject.org
> > > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > > > --
> > > > _______________________________________________
> > > > meta-ti mailing list
> > > > meta-ti@yoctoproject.org
> > > > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-17 21:32 ` Nelson, Sam
@ 2016-05-18 2:41 ` Denys Dmytriyenko
2016-05-18 12:41 ` Nelson, Sam
0 siblings, 1 reply; 9+ messages in thread
From: Denys Dmytriyenko @ 2016-05-18 2:41 UTC (permalink / raw)
To: Nelson, Sam; +Cc: meta-ti
On Tue, May 17, 2016 at 05:32:57PM -0400, Nelson, Sam wrote:
>
>
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Tuesday, May 17, 2016 2:57 PM
> > To: Nelson, Sam
> > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
> > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> > documentation with the package
> >
> > On Tue, May 17, 2016 at 02:39:45PM -0400, Nelson, Sam wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Stiffler, Jacob
> > > > Sent: Friday, May 13, 2016 3:13 PM
> > > > To: Dmytriyenko, Denys; Nelson, Sam
> > > > Cc: meta-ti@yoctoproject.org
> > > > Subject: RE: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > > examples and documentation with the package
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > > > > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > > > > Sent: Friday, May 13, 2016 2:57 PM
> > > > > To: Nelson, Sam
> > > > > Cc: meta-ti@yoctoproject.org
> > > > > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > > > examples and documentation with the package
> > > > >
> > > > > Sam,
> > > > >
> > > > > Is this v2? You didn't specify changes since v1, so I would have
> > > > > to dig out the previous submission and check that past comments
> > > > > were
> > > > addressed.
> > > > >
> > > > > Jake,
> > > > >
> > > > > Do you have any comments here?
> > > > >
> > > > >
> > > > > On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > > > > > Updated to new version 3.42.01.03 Added procedures and source
> > > > > > urls for adding examples and
> > > > > documentation
> > > > > > into src package.
> > > > > > Added dependency on doxygen for document creation Added big
> > > > > > endian
> > > > > > c66 targets Updated uniform formatting
> > > > > >
> > > > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > > > ---
> > > > > > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> > > > > ++++++++++++++++++++++++++++++++------
> > > > > > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > > > > > 2 files changed, 104 insertions(+), 21 deletions(-)
> > > > > >
> > > > > > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > b/recipes-ti/ipc/ti-ipc-
> > > > > rtos_git.bb
> > > > > > index 4c0fc74..8fb448f 100644
> > > > > > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
> > > > > > require recipes-ti/includes/ti-staging.inc
> > > > > > require ti-ipc.inc
> > > > > >
> > > > > > -PR = "${INC_PR}.4"
> > > > > > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-examples.git"
> > > > > > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > > > > > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > > > > > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > > > > > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> > > > > >
> > > > > > -DEPENDS = "ti-xdctools ti-sysbios"
> > > > > > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > > > > > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > > > > > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > > > > > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > > > > > +name=${TI_IPC_EXAMPLES_NAME}"
> > > > > > +
> > > > > > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-metadata.git"
> > > > > > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > > > > > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > > > > > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > > > > > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > > > > > +
> > > > > > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > > > > > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > > > > > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > > > > > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > > > > > +name=${TI_IPC_METADATA_NAME}"
> > > > > > +
> > > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-examples =
> > > > "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > > > > > +
> > > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-metadata =
> > > > "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > > > > > +
> > > > > > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> > > >
> > > > I'm not sure about this SRCREV_FORMAT. Should it also contain the
> > > > main ipc git SRCREV? And is this needed if SRCPV is not in the PV?
> > >
> > > Look like there is some confusion here. I am also not clear if this is needed if
> > SRCREV is not used in PV.
>
> Denys, Can you clarify on whether the SRCREV_FORMAT is really needed or not?
Do you get an error about SRCREV_FORMAT required w/o it?
> > > > > > +
> > > > > > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > > > > > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > > > > > +
> > > > > > +PR = "${INC_PR}.0"
> > > > > > +
> > > > > > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > > > > > DEPENDS_append_keystone = " ti-cgt6x-native \
> > > > > > gcc-arm-none-eabi-native \ "
> > > > > > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-native \
> > > > > > ti-ccsv6-native \
> > > > > > gcc-arm-none-eabi-native \ "
> > > > > > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
> > > >
> > > > These c66x machines have not yet been accepted into meta-ti.
> > >
> > > Ok. I will remove this for now.
> > >
> > > >
> > > > > >
> > > > > > PACKAGES =+ "${PN}-fw"
> > > > > > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > > > > > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > > > > > IPC_TARGETS_omap-a15 = "\
> > > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > > \
> > > > > > +
> > > > > ti.targets.elf.C66_big_endian="${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}" \ "
> > > > > > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> > > > IPC_TARGETS_keystone
> > > > > > = " \
> > > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > > \
> > > > > > +
> > > > > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share/ti/
> > > > > cgt-
> > > > > c6x" \
> > > > > > +"
> > > > > > +
> > > > > > +IPC_TARGETS_c66x = " \
> > > > > > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-c6x"
> > > > > > +\
> > > > > > +
> > > > > ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > > > > > + ${IPC_TARGETS} \
> > > > > > "
> > > > > >
> > > > > > +RELEASE_TYPE = "GA"
> > > > > > +RELEASE_SUFFIX = ""
> > > > > > +
> > > > > > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > > > > > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> > > > > '_')}${RELEASE_SUFFIX}"
> > > >
> > > > This already specified in meta-arago's arago-source-ipk.conf?
> > >
> > > I don't see anything related ti-ipc-rtos, in the master branch meta-arago:
> > arago-source-ipk.conf.
> > > Am I missing something?
> >
> > Yes, no SRCIPK stuff in meta-ti allowed.
> >
>
> I see. I will remove from here and send update to meta-arago.
>
> >
> > > > > > +
> > > > > > do_compile() {
> > > > > > - oe_runmake clean
> > > > > > - oe_runmake all
> > > > > > + oe_runmake -f ipc-bios.mak clean oe_runmake -f ipc-bios.mak
> > > > > > + release
> > > > > > +
> > > > > > + cd ${S_ipc-metadata}
> > > > > > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > > > > > + BUILD_HOST_OS="linux" \
> > > > > > + RELEASE_TYPE="${RELEASE_TYPE}"
> > > > > > +
> > > > > > + cd ${S_ipc-examples}/src
> > > > > > + oe_runmake .examples \
> > > > > > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > > > > > +
> > > > > > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > > > > > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > > > > > +
> > > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> > HOSTOS="bios"
> > > > > IPC_INSTALL_DIR="${S}"
> > > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> > HOSTOS="linux"
> > > > > IPC_INSTALL_DIR="${S}"
> > > > > > + fi
> > > > > > + fi
> > > > > > +
> > > > > > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e 's|\.|_|g'`
> > > > > > + # Copy docs and other meta files install -d
> > > > > > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > > > > > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > > > > ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > +
> > > > > > + # Copy example folders corresponding to the platforms if [
> > > > > > + "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > > > > > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > > + cp -pPrf ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > + fi
> > > > > > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip"
> > > > > > + -type f |
> > > > > xargs -I {} rm {}
> > > > > > + fi
> > > >
> > > > Why do all of this installation in the do_compile stage? Should it
> > > > be in do_install?
> > > >
> > >
> > > This is rtos package and everything is that is created is part of the source
> > package.
> > > Only what goes into file system is in the do_install step.
> > >
> > > > > > }
> > > > > >
> > > > > > do_compile_append() {
> > > > > > @@ -52,25 +138,22 @@ do_compile_append() { }
> > > > > >
> > > > > > do_install() {
> > > > > > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > -
> > > > > > - install -d ${D}${base_libdir}/firmware/ipc
> > > > > > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> > > > > ${D}${base_libdir}/firmware/ipc
> > > > > > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > > > > > + ${S}/packages/ti/ipc/tests/bin/*
> > > > > ${D}${base_libdir}/firmware/ipc
> > > > > > }
> > > > > >
> > > > > > ALTERNATIVE_PRIORITY = "5"
> > > > > >
> > > > > > pkg_postinst_${PN}-fw_omap-a15 () {
> > > > > > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66 dra7-
> > > > > dsp1-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66 dra7-
> > > > > dsp2-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4 dra7-
> > > > > ipu1-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4 dra7-
> > > > > ipu2-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > + update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> > > > > > + dra7-
> > > > > dsp1-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > + update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66
> > > > > > + dra7-
> > > > > dsp2-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > + update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4
> > > > > > + dra7-
> > > > > ipu1-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > + update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4
> > > > > > + dra7-
> > > > > ipu2-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > }
> > > > > >
> > > > > > pkg_postrm_${PN}-fw_omap-a15 () {
> > > > > > - update-alternatives --remove dra7-dsp1-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > - update-alternatives --remove dra7-dsp2-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > - update-alternatives --remove dra7-ipu1-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > - update-alternatives --remove dra7-ipu2-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > + update-alternatives --remove dra7-dsp1-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > + update-alternatives --remove dra7-dsp2-fw.xe66
> > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > + update-alternatives --remove dra7-ipu1-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > + update-alternatives --remove dra7-ipu2-fw.xem4
> > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > }
> > > > > > diff --git a/recipes-ti/ipc/ti-ipc.inc
> > > > > > b/recipes-ti/ipc/ti-ipc.inc index 69127aa..9c5dfed 100644
> > > > > > --- a/recipes-ti/ipc/ti-ipc.inc
> > > > > > +++ b/recipes-ti/ipc/ti-ipc.inc
> > > > > > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI = "git://git.ti.com/ipc/ipcdev.git"
> > > > > > TI_IPC_GIT_PROTOCOL = "git"
> > > > > > TI_IPC_GIT_BRANCH = "3.42"
> > > > > >
> > > > > > -#Corresponds to 3.42.00.02
> > > > > > -TI_IPC_SRCREV = "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > > > > > +#Corresponds to 3.42.01.03
> > > > > > +TI_IPC_SRCREV = "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> > > > > >
> > > > > > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > > > > > SRC_URI =
> > > > >
> > > >
> > "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > > > > > SRCREV = "${TI_IPC_SRCREV}"
> > > > > >
> > > > > > S = "${WORKDIR}/git"
> > > > > > -PV = "3.42.00.02"
> > > > > > +PV = "3.42.01.03"
> > > > > > INC_PR = "r0"
> > > > > >
> > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > > > > --
> > > > > > 1.9.1
> > > > > >
> > > > > > --
> > > > > > _______________________________________________
> > > > > > meta-ti mailing list
> > > > > > meta-ti@yoctoproject.org
> > > > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > > > > --
> > > > > _______________________________________________
> > > > > meta-ti mailing list
> > > > > meta-ti@yoctoproject.org
> > > > > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package
2016-05-18 2:41 ` Denys Dmytriyenko
@ 2016-05-18 12:41 ` Nelson, Sam
0 siblings, 0 replies; 9+ messages in thread
From: Nelson, Sam @ 2016-05-18 12:41 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-ti
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Tuesday, May 17, 2016 10:41 PM
> To: Nelson, Sam
> Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include examples and
> documentation with the package
>
> On Tue, May 17, 2016 at 05:32:57PM -0400, Nelson, Sam wrote:
> >
> >
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Tuesday, May 17, 2016 2:57 PM
> > > To: Nelson, Sam
> > > Cc: Stiffler, Jacob; meta-ti@yoctoproject.org
> > > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > examples and documentation with the package
> > >
> > > On Tue, May 17, 2016 at 02:39:45PM -0400, Nelson, Sam wrote:
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: Stiffler, Jacob
> > > > > Sent: Friday, May 13, 2016 3:13 PM
> > > > > To: Dmytriyenko, Denys; Nelson, Sam
> > > > > Cc: meta-ti@yoctoproject.org
> > > > > Subject: RE: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > > > examples and documentation with the package
> > > > >
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> > > > > > bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
> > > > > > Sent: Friday, May 13, 2016 2:57 PM
> > > > > > To: Nelson, Sam
> > > > > > Cc: meta-ti@yoctoproject.org
> > > > > > Subject: Re: [meta-ti] [PATCH] ti-ipc-rtos: Update to include
> > > > > > examples and documentation with the package
> > > > > >
> > > > > > Sam,
> > > > > >
> > > > > > Is this v2? You didn't specify changes since v1, so I would
> > > > > > have to dig out the previous submission and check that past
> > > > > > comments were
> > > > > addressed.
> > > > > >
> > > > > > Jake,
> > > > > >
> > > > > > Do you have any comments here?
> > > > > >
> > > > > >
> > > > > > On Wed, May 11, 2016 at 02:00:39PM -0400, Sam Nelson wrote:
> > > > > > > Updated to new version 3.42.01.03 Added procedures and
> > > > > > > source urls for adding examples and
> > > > > > documentation
> > > > > > > into src package.
> > > > > > > Added dependency on doxygen for document creation Added big
> > > > > > > endian
> > > > > > > c66 targets Updated uniform formatting
> > > > > > >
> > > > > > > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> > > > > > > ---
> > > > > > > recipes-ti/ipc/ti-ipc-rtos_git.bb | 119
> > > > > > ++++++++++++++++++++++++++++++++------
> > > > > > > recipes-ti/ipc/ti-ipc.inc | 6 +-
> > > > > > > 2 files changed, 104 insertions(+), 21 deletions(-)
> > > > > > >
> > > > > > > diff --git a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > > b/recipes-ti/ipc/ti-ipc-
> > > > > > rtos_git.bb
> > > > > > > index 4c0fc74..8fb448f 100644
> > > > > > > --- a/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > > +++ b/recipes-ti/ipc/ti-ipc-rtos_git.bb
> > > > > > > @@ -2,9 +2,44 @@ require recipes-ti/includes/ti-paths.inc
> > > > > > > require recipes-ti/includes/ti-staging.inc
> > > > > > > require ti-ipc.inc
> > > > > > >
> > > > > > > -PR = "${INC_PR}.4"
> > > > > > > +TI_IPC_EXAMPLES_GIT_URI = "git://git.ti.com/ipc/ipc-
> examples.git"
> > > > > > > +TI_IPC_EXAMPLES_DEST_SUFFIX = "git/ipc-examples"
> > > > > > > +TI_IPC_EXAMPLES_GIT_PROTOCOL = "git"
> > > > > > > +TI_IPC_EXAMPLES_GIT_BRANCH = "master"
> > > > > > > +TI_IPC_EXAMPLES_NAME = "ipc-examples"
> > > > > > >
> > > > > > > -DEPENDS = "ti-xdctools ti-sysbios"
> > > > > > > +SRC_URI += "${TI_IPC_EXAMPLES_GIT_URI};\
> > > > > > > +destsuffix=${TI_IPC_EXAMPLES_DEST_SUFFIX};\
> > > > > > > +protocol=${TI_IPC_EXAMPLES_GIT_PROTOCOL};\
> > > > > > > +branch=${TI_IPC_EXAMPLES_GIT_BRANCH};\
> > > > > > > +name=${TI_IPC_EXAMPLES_NAME}"
> > > > > > > +
> > > > > > > +TI_IPC_METADATA_GIT_URI = "git://git.ti.com/ipc/ipc-
> metadata.git"
> > > > > > > +TI_IPC_METADATA_DEST_SUFFIX = "git/ipc-metadata"
> > > > > > > +TI_IPC_METADATA_GIT_PROTOCOL = "git"
> > > > > > > +TI_IPC_METADATA_GIT_BRANCH = "master"
> > > > > > > +TI_IPC_METADATA_NAME = "ipc-metadata"
> > > > > > > +
> > > > > > > +SRC_URI += "${TI_IPC_METADATA_GIT_URI};\
> > > > > > > +destsuffix=${TI_IPC_METADATA_DEST_SUFFIX};\
> > > > > > > +protocol=${TI_IPC_METADATA_GIT_PROTOCOL};\
> > > > > > > +branch=${TI_IPC_METADATA_GIT_BRANCH};\
> > > > > > > +name=${TI_IPC_METADATA_NAME}"
> > > > > > > +
> > > > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-examples =
> > > > > "bd4f28bd46467553e6610629ff13b4b5b3ba3af0"
> > > > > > > +
> > > > > > > +# Corresponds to tag: 3.42.01.03 SRCREV_ipc-metadata =
> > > > > "4d01fe4a7c41d2ce76a62b4325e66753d59b298c"
> > > > > > > +
> > > > > > > +SRCREV_FORMAT = "ipc-metadata_ipc-examples"
> > > > >
> > > > > I'm not sure about this SRCREV_FORMAT. Should it also contain
> > > > > the main ipc git SRCREV? And is this needed if SRCPV is not in the PV?
> > > >
> > > > Look like there is some confusion here. I am also not clear if
> > > > this is needed if
> > > SRCREV is not used in PV.
> >
> > Denys, Can you clarify on whether the SRCREV_FORMAT is really needed or
> not?
>
> Do you get an error about SRCREV_FORMAT required w/o it?
>
Not really. No errors reported without SRCREV_FORMAT.
>
> > > > > > > +
> > > > > > > +S_ipc-examples = "${WORKDIR}/git/ipc-examples"
> > > > > > > +S_ipc-metadata = "${WORKDIR}/git/ipc-metadata"
> > > > > > > +
> > > > > > > +PR = "${INC_PR}.0"
> > > > > > > +
> > > > > > > +DEPENDS = "ti-xdctools ti-sysbios doxygen-native"
> > > > > > > DEPENDS_append_keystone = " ti-cgt6x-native \
> > > > > > > gcc-arm-none-eabi-native \ "
> > > > > > > @@ -12,6 +47,8 @@ DEPENDS_append_omap-a15 = " ti-cgt6x-
> native \
> > > > > > > ti-ccsv6-native \
> > > > > > > gcc-arm-none-eabi-native \ "
> > > > > > > +DEPENDS_append_c66x = " ti-cgt6x-native \ "
> > > > >
> > > > > These c66x machines have not yet been accepted into meta-ti.
> > > >
> > > > Ok. I will remove this for now.
> > > >
> > > > >
> > > > > > >
> > > > > > > PACKAGES =+ "${PN}-fw"
> > > > > > > FILES_${PN}-fw = "${base_libdir}/firmware/*"
> > > > > > > @@ -26,6 +63,7 @@ IPC_TARGETS = ""
> > > > > > > IPC_TARGETS_omap-a15 = "\
> > > > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> c6x"
> > > > > > > \
> > > > > > > +
> > > > > > ti.targets.elf.C66_big_endian="${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}" \ "
> > > > > > > @@ -33,18 +71,66 @@ IPC_TARGETS_omap-a15 = "\
> > > > > IPC_TARGETS_keystone
> > > > > > > = " \
> > > > > > > gnu.targets.arm.A15F="${GCC_ARM_NONE_TOOLCHAIN}" \
> > > > > > > ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> c6x"
> > > > > > > \
> > > > > > > +
> > > > > > ti.targets.elf.C66_big_endian="${STAGING_DIR_NATIVE}/usr/share
> > > > > > /ti/
> > > > > > cgt-
> > > > > > c6x" \
> > > > > > > +"
> > > > > > > +
> > > > > > > +IPC_TARGETS_c66x = " \
> > > > > > > + ti.targets.elf.C66="${STAGING_DIR_NATIVE}/usr/share/ti/cgt-
> c6x"
> > > > > > > +\
> > > > > > > +
> > > > > > ti.targets.elf.C66_big_endian="${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} -f ipc-bios.mak\
> > > > > > > + ${IPC_TARGETS} \
> > > > > > > "
> > > > > > >
> > > > > > > +RELEASE_TYPE = "GA"
> > > > > > > +RELEASE_SUFFIX = ""
> > > > > > > +
> > > > > > > +SRCIPK_SRC_DIR = "${D}${IPC_INSTALL_DIR_RECIPE}"
> > > > > > > +SRCIPK_INSTALL_DIR = "ipc_${@'${PV}'.replace('.',
> > > > > > '_')}${RELEASE_SUFFIX}"
> > > > >
> > > > > This already specified in meta-arago's arago-source-ipk.conf?
> > > >
> > > > I don't see anything related ti-ipc-rtos, in the master branch meta-arago:
> > > arago-source-ipk.conf.
> > > > Am I missing something?
> > >
> > > Yes, no SRCIPK stuff in meta-ti allowed.
> > >
> >
> > I see. I will remove from here and send update to meta-arago.
> >
> > >
> > > > > > > +
> > > > > > > do_compile() {
> > > > > > > - oe_runmake clean
> > > > > > > - oe_runmake all
> > > > > > > + oe_runmake -f ipc-bios.mak clean oe_runmake -f
> > > > > > > + ipc-bios.mak release
> > > > > > > +
> > > > > > > + cd ${S_ipc-metadata}
> > > > > > > + oe_runmake .all-files IPC_INSTALL_DIR="${S}" \
> > > > > > > + BUILD_HOST_OS="linux" \
> > > > > > > + RELEASE_TYPE="${RELEASE_TYPE}"
> > > > > > > +
> > > > > > > + cd ${S_ipc-examples}/src
> > > > > > > + oe_runmake .examples \
> > > > > > > + IPCTOOLS="${S_ipc-metadata}/src/etc"
> > > > > > > +
> > > > > > > + if [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > > > + oe_runmake extract HOSTOS="bios" IPC_INSTALL_DIR="${S}"
> > > > > > > + oe_runmake extract HOSTOS="linux" IPC_INSTALL_DIR="${S}"
> > > > > > > +
> > > > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> > > HOSTOS="bios"
> > > > > > IPC_INSTALL_DIR="${S}"
> > > > > > > + oe_runmake extract PLATFORM=${ALT_PLATFORM}
> > > HOSTOS="linux"
> > > > > > IPC_INSTALL_DIR="${S}"
> > > > > > > + fi
> > > > > > > + fi
> > > > > > > +
> > > > > > > + IPC_VERSION=`echo ${PV}${RELEASE_SUFFIX} | sed -e
> > > > > > > + 's|\.|_|g'` # Copy docs and other meta files install -d
> > > > > > > + ${D}${IPC_INSTALL_DIR_RECIPE} cp -pPrf
> > > > > > > + ${S_ipc-metadata}/exports/ipc_${IPC_VERSION}/* -d
> > > > > > ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > > +
> > > > > > > + # Copy example folders corresponding to the platforms if
> > > > > > > + [ "${PLATFORM}" != "UNKNOWN" ]; then
> > > > > > > + install -d ${D}${IPC_INSTALL_DIR_RECIPE}/examples
> > > > > > > + cp -pPf ${S_ipc-examples}/src/examples/*.*
> > > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > > + cp -pPf ${S_ipc-examples}/src/examples/makefile
> > > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > > + cp -pPrf ${S_ipc-examples}/src/examples/${PLATFORM}*
> > > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > > + if [ ! -z ${ALT_PLATFORM} ]; then
> > > > > > > + cp -pPrf
> > > > > > > + ${S_ipc-examples}/src/examples/${ALT_PLATFORM}*
> > > > > > ${D}${IPC_INSTALL_DIR_RECIPE}/examples/
> > > > > > > + fi
> > > > > > > + find ${D}${IPC_INSTALL_DIR_RECIPE}/examples/ -name "*zip"
> > > > > > > + -type f |
> > > > > > xargs -I {} rm {}
> > > > > > > + fi
> > > > >
> > > > > Why do all of this installation in the do_compile stage? Should
> > > > > it be in do_install?
> > > > >
> > > >
> > > > This is rtos package and everything is that is created is part of
> > > > the source
> > > package.
> > > > Only what goes into file system is in the do_install step.
> > > >
> > > > > > > }
> > > > > > >
> > > > > > > do_compile_append() {
> > > > > > > @@ -52,25 +138,22 @@ do_compile_append() { }
> > > > > > >
> > > > > > > do_install() {
> > > > > > > - install -d ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > > - cp -pPrf ${S}/* ${D}${IPC_INSTALL_DIR_RECIPE}
> > > > > > > -
> > > > > > > - install -d ${D}${base_libdir}/firmware/ipc
> > > > > > > - cp -pPrf ${S}/packages/ti/ipc/tests/bin/*
> > > > > > ${D}${base_libdir}/firmware/ipc
> > > > > > > + install -d ${D}${base_libdir}/firmware/ipc cp -pPrf
> > > > > > > + ${S}/packages/ti/ipc/tests/bin/*
> > > > > > ${D}${base_libdir}/firmware/ipc
> > > > > > > }
> > > > > > >
> > > > > > > ALTERNATIVE_PRIORITY = "5"
> > > > > > >
> > > > > > > pkg_postinst_${PN}-fw_omap-a15 () {
> > > > > > > - update-alternatives --install /lib/firmware/dra7-dsp1-fw.xe66
> dra7-
> > > > > > dsp1-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > - update-alternatives --install /lib/firmware/dra7-dsp2-fw.xe66
> dra7-
> > > > > > dsp2-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > - update-alternatives --install /lib/firmware/dra7-ipu1-fw.xem4
> dra7-
> > > > > > ipu1-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > - update-alternatives --install /lib/firmware/dra7-ipu2-fw.xem4
> dra7-
> > > > > > ipu2-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > + update-alternatives --install
> > > > > > > + /lib/firmware/dra7-dsp1-fw.xe66
> > > > > > > + dra7-
> > > > > > dsp1-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > + update-alternatives --install
> > > > > > > + /lib/firmware/dra7-dsp2-fw.xe66
> > > > > > > + dra7-
> > > > > > dsp2-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > + update-alternatives --install
> > > > > > > + /lib/firmware/dra7-ipu1-fw.xem4
> > > > > > > + dra7-
> > > > > > ipu1-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > + update-alternatives --install
> > > > > > > + /lib/firmware/dra7-ipu2-fw.xem4
> > > > > > > + dra7-
> > > > > > ipu2-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > ${ALTERNATIVE_PRIORITY}
> > > > > > > }
> > > > > > >
> > > > > > > pkg_postrm_${PN}-fw_omap-a15 () {
> > > > > > > - update-alternatives --remove dra7-dsp1-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > > - update-alternatives --remove dra7-dsp2-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > > - update-alternatives --remove dra7-ipu1-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > > - update-alternatives --remove dra7-ipu2-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > > + update-alternatives --remove dra7-dsp1-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp1/test_omx_dsp1_vayu.xe66
> > > > > > > + update-alternatives --remove dra7-dsp2-fw.xe66
> > > > > > ipc/ti_platforms_evmDRA7XX_dsp2/test_omx_dsp2_vayu.xe66
> > > > > > > + update-alternatives --remove dra7-ipu1-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu1/test_omx_ipu1_vayu.xem4
> > > > > > > + update-alternatives --remove dra7-ipu2-fw.xem4
> > > > > > ipc/ti_platforms_evmDRA7XX_ipu2/test_omx_ipu2_vayu.xem4
> > > > > > > }
> > > > > > > diff --git a/recipes-ti/ipc/ti-ipc.inc
> > > > > > > b/recipes-ti/ipc/ti-ipc.inc index 69127aa..9c5dfed 100644
> > > > > > > --- a/recipes-ti/ipc/ti-ipc.inc
> > > > > > > +++ b/recipes-ti/ipc/ti-ipc.inc
> > > > > > > @@ -8,15 +8,15 @@ TI_IPC_GIT_URI =
> "git://git.ti.com/ipc/ipcdev.git"
> > > > > > > TI_IPC_GIT_PROTOCOL = "git"
> > > > > > > TI_IPC_GIT_BRANCH = "3.42"
> > > > > > >
> > > > > > > -#Corresponds to 3.42.00.02
> > > > > > > -TI_IPC_SRCREV =
> "a520969994378fb2629d7dfc25f306c3a711b5bc"
> > > > > > > +#Corresponds to 3.42.01.03
> > > > > > > +TI_IPC_SRCREV =
> "dc6c5082f478225b7fbce4216173e6c5b36100b4"
> > > > > > >
> > > > > > > BRANCH = "${TI_IPC_GIT_BRANCH}"
> > > > > > > SRC_URI =
> > > > > >
> > > > >
> > >
> "${TI_IPC_GIT_URI};protocol=${TI_IPC_GIT_PROTOCOL};branch=${BRANCH}"
> > > > > > > SRCREV = "${TI_IPC_SRCREV}"
> > > > > > >
> > > > > > > S = "${WORKDIR}/git"
> > > > > > > -PV = "3.42.00.02"
> > > > > > > +PV = "3.42.01.03"
> > > > > > > INC_PR = "r0"
> > > > > > >
> > > > > > > PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > > > > > --
> > > > > > > 1.9.1
> > > > > > >
> > > > > > > --
> > > > > > > _______________________________________________
> > > > > > > meta-ti mailing list
> > > > > > > meta-ti@yoctoproject.org
> > > > > > > https://lists.yoctoproject.org/listinfo/meta-ti
> > > > > > --
> > > > > > _______________________________________________
> > > > > > meta-ti mailing list
> > > > > > meta-ti@yoctoproject.org
> > > > > > https://lists.yoctoproject.org/listinfo/meta-ti
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-05-18 12:41 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-11 18:00 [PATCH] ti-ipc-rtos: Update to include examples and documentation with the package Sam Nelson
2016-05-13 18:56 ` Denys Dmytriyenko
2016-05-13 19:12 ` Stiffler, Jacob
2016-05-17 18:39 ` Nelson, Sam
2016-05-17 18:56 ` Denys Dmytriyenko
2016-05-17 21:32 ` Nelson, Sam
2016-05-18 2:41 ` Denys Dmytriyenko
2016-05-18 12:41 ` Nelson, Sam
2016-05-13 20:55 ` Nelson, Sam
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.