[meta-ti] [PATCH] boot-monitor: add K2L and K2E boot monitor build support

[Denys Dmytriyenko]

On Wed, May 14, 2014 at 02:58:51PM -0400, Hao Zhang wrote:
> On 5/14/2014 2:40 PM, Denys Dmytriyenko wrote:
> > On Wed, May 14, 2014 at 02:29:49PM -0400, hzhang at ti.com wrote:
> >> From: Hao Zhang <hzhang at ti.com>
> >>
> >> ---
> >>  recipes-bsp/boot-monitor/boot-monitor_git.bb |   18 +++++++++++++-----
> >>  1 file changed, 13 insertions(+), 5 deletions(-)
> >>
> >> diff --git a/recipes-bsp/boot-monitor/boot-monitor_git.bb b/recipes-bsp/boot-monitor/boot-monitor_git.bb
> >> index 7c5e9cf..2c3aa43 100644
> >> --- a/recipes-bsp/boot-monitor/boot-monitor_git.bb
> >> +++ b/recipes-bsp/boot-monitor/boot-monitor_git.bb
> >> @@ -3,8 +3,12 @@ EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX}"
> >>  
> >>  LICENSE = "BSD"
> >>  
> >> -BOOT_MONITOR_BINARY ?= "skern.bin"
> >> -BOOT_MONITOR_IMAGE  ?= "skern-${MACHINE}.bin"
> >> +BOOT_MONITOR_BINARY_K2HK ?= "skern-k2hk.bin"
> >> +BOOT_MONITOR_IMAGE_k2HK  ?= "skern-k2hk-${MACHINE}.bin"
> >> +BOOT_MONITOR_BINARY_K2L ?= "skern-k2l.bin"
> >> +BOOT_MONITOR_IMAGE_k2L  ?= "skern-k2l-${MACHINE}.bin"
> >> +BOOT_MONITOR_BINARY_K2E ?= "skern-k2e.bin"
> >> +BOOT_MONITOR_IMAGE_k2E  ?= "skern-k2e-${MACHINE}.bin"
> > 
> > Have you tested it? You mix case in the above variables - one has K2HK 
> > uppercase, another has k2HK lowercase. Same for k2l...
> > 
> > 
> 
> Yes, I tested it. I will correct the lowercase error and re-submit the
> patch.

It shouldn't have worked, as you use incorrect variables below in do_deploy.


> >>  BOOT_MONITOR_MAKE_TARGET  ?= "all"
> >>  
> >>  LIC_FILES_CHKSUM = "file://COPYING;md5=25fe219a6febf6e5bb45beda1b2eb315"
> >> @@ -14,7 +18,7 @@ COMPATIBLE_MACHINE = "keystone-evm"
> >>  SRC_URI = "git://git.ti.com/keystone-linux/boot-monitor.git;protocol=git;branch=${BRANCH}"
> >>  
> >>  PV = "2.0"
> >> -PR = "r0+gitr${SRCPV}"
> >> +PR = "r1+gitr${SRCPV}"
> >>  
> >>  BRANCH = "master"
> >>  
> >> @@ -27,7 +31,9 @@ do_compile () {
> >>  	unset LDFLAGS
> >>  	unset CFLAGS
> >>  	unset CPPFLAGS
> >> -	oe_runmake ${BOOT_MONITOR_MAKE_TARGET}
> >> +	oe_runmake k2hk
> >> +	oe_runmake k2l
> >> +	oe_runmake k2e
> >>  }
> >>  
> >>  inherit deploy
> >> @@ -36,5 +42,7 @@ addtask deploy before do_build after do_compile
> >>  
> >>  do_deploy () {
> >>  	install -d ${DEPLOYDIR}
> >> -	install ${S}/${BOOT_MONITOR_BINARY} ${DEPLOYDIR}/${BOOT_MONITOR_IMAGE}
> >> +	install ${S}/${BOOT_MONITOR_BINARY_K2HK} ${DEPLOYDIR}/${BOOT_MONITOR_IMAGE_K2HK}
> >> +	install ${S}/${BOOT_MONITOR_BINARY_K2L}	${DEPLOYDIR}/${BOOT_MONITOR_IMAGE_K2L}
> >> +	install ${S}/${BOOT_MONITOR_BINARY_K2E}	${DEPLOYDIR}/${BOOT_MONITOR_IMAGE_K2E}
> >>  }
> >> -- 
> >> 1.7.9.5
> >>
> >> -- 
> >> _______________________________________________
> >> meta-ti mailing list
> >> meta-ti at yoctoproject.org
> >> https://lists.yoctoproject.org/listinfo/meta-ti
>