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

[Hao Zhang <hzhang@ti.com>]

On 5/14/2014 2:40 PM, Denys Dmytriyenko wrote:
> On Wed, May 14, 2014 at 02:29:49PM -0400, hzhang@ti.com wrote:
>> From: Hao Zhang <hzhang@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.

>>  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@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti