* [PATCH] boot-monitor: add K2L and K2E boot monitor build support @ 2014-05-14 18:29 hzhang 2014-05-14 18:40 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: hzhang @ 2014-05-14 18:29 UTC (permalink / raw) To: meta-ti 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" 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 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 18:29 [PATCH] boot-monitor: add K2L and K2E boot monitor build support hzhang @ 2014-05-14 18:40 ` Denys Dmytriyenko 2014-05-14 18:58 ` Hao Zhang 0 siblings, 1 reply; 22+ messages in thread From: Denys Dmytriyenko @ 2014-05-14 18:40 UTC (permalink / raw) To: hzhang; +Cc: meta-ti 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... > 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 ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 18:40 ` Denys Dmytriyenko @ 2014-05-14 18:58 ` Hao Zhang 2014-05-14 19:01 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Hao Zhang @ 2014-05-14 18:58 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: meta-ti 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 ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 18:58 ` Hao Zhang @ 2014-05-14 19:01 ` Denys Dmytriyenko 2014-05-14 19:55 ` Maupin, Chase 0 siblings, 1 reply; 22+ messages in thread From: Denys Dmytriyenko @ 2014-05-14 19:01 UTC (permalink / raw) To: Hao Zhang; +Cc: meta-ti 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@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. 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@yoctoproject.org > >> https://lists.yoctoproject.org/listinfo/meta-ti > ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 19:01 ` Denys Dmytriyenko @ 2014-05-14 19:55 ` Maupin, Chase 2014-05-14 20:38 ` Hao Zhang 0 siblings, 1 reply; 22+ messages in thread From: Maupin, Chase @ 2014-05-14 19:55 UTC (permalink / raw) To: Dmytriyenko, Denys, Zhang, Hao; +Cc: meta-ti@yoctoproject.org >-----Original Message----- >From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- >bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys >Sent: Wednesday, May 14, 2014 2:01 PM >To: Zhang, Hao >Cc: meta-ti@yoctoproject.org >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >monitor build support > >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@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" Maybe a silly question but do you need to package all of these in a single recipe or do you want to package K2HK and K2L and K2E in separate packages? >> > >> > 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 If you really want to package all of these in a single recipe why not use a for loop? The naming, etc is formulaic. You could just have a list of "k2hk k2l k2e" and do the make for that list as well as your do_deploy stuff below. >> >> } >> >> >> >> 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 >> >-- >_______________________________________________ >meta-ti mailing list >meta-ti@yoctoproject.org >https://lists.yoctoproject.org/listinfo/meta-ti ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 19:55 ` Maupin, Chase @ 2014-05-14 20:38 ` Hao Zhang 2014-05-15 14:21 ` Maupin, Chase 0 siblings, 1 reply; 22+ messages in thread From: Hao Zhang @ 2014-05-14 20:38 UTC (permalink / raw) To: Maupin, Chase; +Cc: meta-ti@yoctoproject.org On 5/14/2014 3:55 PM, Maupin, Chase wrote: >> -----Original Message----- >> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- >> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys >> Sent: Wednesday, May 14, 2014 2:01 PM >> To: Zhang, Hao >> Cc: meta-ti@yoctoproject.org >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >> monitor build support >> >> 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@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" > > Maybe a silly question but do you need to package all of these in a single recipe or do you want to package K2HK and K2L and K2E in separate packages? > >>>> >>>> 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. >> >> For some reason, it still works, anyway I fixed the lowercase errors >>>>> 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 > > If you really want to package all of these in a single recipe why not use a for loop? The naming, etc is formulaic. You could just have a list of "k2hk k2l k2e" and do the make for that list as well as your do_deploy stuff below. > Will use for loop and re-sumbit the patch. >>>>> } >>>>> >>>>> 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 >>> >> -- >> _______________________________________________ >> meta-ti mailing list >> meta-ti@yoctoproject.org >> https://lists.yoctoproject.org/listinfo/meta-ti ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-14 20:38 ` Hao Zhang @ 2014-05-15 14:21 ` Maupin, Chase 2014-05-15 14:41 ` Hao Zhang 0 siblings, 1 reply; 22+ messages in thread From: Maupin, Chase @ 2014-05-15 14:21 UTC (permalink / raw) To: Zhang, Hao; +Cc: meta-ti@yoctoproject.org >-----Original Message----- >From: Zhang, Hao >Sent: Wednesday, May 14, 2014 3:38 PM >To: Maupin, Chase >Cc: Dmytriyenko, Denys; meta-ti@yoctoproject.org >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >monitor build support > >On 5/14/2014 3:55 PM, Maupin, Chase wrote: >>> -----Original Message----- >>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- >>> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys >>> Sent: Wednesday, May 14, 2014 2:01 PM >>> To: Zhang, Hao >>> Cc: meta-ti@yoctoproject.org >>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >boot >>> monitor build support >>> >>> 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@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" >> >> Maybe a silly question but do you need to package all of these >in a single recipe or do you want to package K2HK and K2L and K2E >in separate packages? >> >>>>> >>>>> 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. >>> >>> > >For some reason, it still works, anyway I fixed the lowercase >errors > >>>>>> 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 >> >> If you really want to package all of these in a single recipe >why not use a for loop? The naming, etc is formulaic. You could >just have a list of "k2hk k2l k2e" and do the make for that list >as well as your do_deploy stuff below. >> > >Will use for loop and re-sumbit the patch. Can you clarify if you really want all 3 devices installed all the time or do you really want a recipe that installs the boot monitor per device? I know you don't currently have 3 machine types so maybe that is what is feeding your issue here, but my question is whether you need to have separate builds per device. > >>>>>> } >>>>>> >>>>>> 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 >>>> >>> -- >>> _______________________________________________ >>> meta-ti mailing list >>> meta-ti@yoctoproject.org >>> https://lists.yoctoproject.org/listinfo/meta-ti ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 14:21 ` Maupin, Chase @ 2014-05-15 14:41 ` Hao Zhang 2014-05-15 14:54 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Hao Zhang @ 2014-05-15 14:41 UTC (permalink / raw) To: Maupin, Chase; +Cc: meta-ti@yoctoproject.org On 5/15/2014 10:21 AM, Maupin, Chase wrote: >> -----Original Message----- >> From: Zhang, Hao >> Sent: Wednesday, May 14, 2014 3:38 PM >> To: Maupin, Chase >> Cc: Dmytriyenko, Denys; meta-ti@yoctoproject.org >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >> monitor build support >> >> On 5/14/2014 3:55 PM, Maupin, Chase wrote: >>>> -----Original Message----- >>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- >>>> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys >>>> Sent: Wednesday, May 14, 2014 2:01 PM >>>> To: Zhang, Hao >>>> Cc: meta-ti@yoctoproject.org >>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >> boot >>>> monitor build support >>>> >>>> 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@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" >>> >>> Maybe a silly question but do you need to package all of these >> in a single recipe or do you want to package K2HK and K2L and K2E >> in separate packages? >>> >>>>>> >>>>>> 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. >>>> >>>> >> >> For some reason, it still works, anyway I fixed the lowercase >> errors >> >>>>>>> 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 >>> >>> If you really want to package all of these in a single recipe >> why not use a for loop? The naming, etc is formulaic. You could >> just have a list of "k2hk k2l k2e" and do the make for that list >> as well as your do_deploy stuff below. >>> >> >> Will use for loop and re-sumbit the patch. > > Can you clarify if you really want all 3 devices installed all the time or do you really want a recipe that installs the boot monitor per device? I know you don't currently have 3 machine types so maybe that is what is feeding your issue here, but my question is whether you need to have separate builds per device. > I want all the 3 boot monitors built and installed all the time in one recipe, since MCSDK 3.1 supports all the 3 Keystone II devices in the same release package. This applies to the U-boot (3 U-boot build for all the 3 Keystone II devices) and Linux kernel DTB. >> >>>>>>> } >>>>>>> >>>>>>> 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 >>>>> >>>> -- >>>> _______________________________________________ >>>> meta-ti mailing list >>>> meta-ti@yoctoproject.org >>>> https://lists.yoctoproject.org/listinfo/meta-ti > ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 14:41 ` Hao Zhang @ 2014-05-15 14:54 ` Denys Dmytriyenko 2014-05-15 15:07 ` Hao Zhang 2014-05-15 15:20 ` Tom Rini 0 siblings, 2 replies; 22+ messages in thread From: Denys Dmytriyenko @ 2014-05-15 14:54 UTC (permalink / raw) To: Hao Zhang; +Cc: trini, meta-ti@yoctoproject.org On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: > On 5/15/2014 10:21 AM, Maupin, Chase wrote: > >> -----Original Message----- > >> From: Zhang, Hao > >> Sent: Wednesday, May 14, 2014 3:38 PM > >> To: Maupin, Chase > >> Cc: Dmytriyenko, Denys; meta-ti@yoctoproject.org > >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > >> monitor build support > >> > >> On 5/14/2014 3:55 PM, Maupin, Chase wrote: > >>>> -----Original Message----- > >>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- > >>>> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys > >>>> Sent: Wednesday, May 14, 2014 2:01 PM > >>>> To: Zhang, Hao > >>>> Cc: meta-ti@yoctoproject.org > >>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E > >> boot > >>>> monitor build support > >>>> > >>>> 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@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" > >>> > >>> Maybe a silly question but do you need to package all of these > >> in a single recipe or do you want to package K2HK and K2L and K2E > >> in separate packages? > >>> > >>>>>> > >>>>>> 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. > >>>> > >>>> > >> > >> For some reason, it still works, anyway I fixed the lowercase > >> errors > >> > >>>>>>> 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 > >>> > >>> If you really want to package all of these in a single recipe > >> why not use a for loop? The naming, etc is formulaic. You could > >> just have a list of "k2hk k2l k2e" and do the make for that list > >> as well as your do_deploy stuff below. > >>> > >> > >> Will use for loop and re-sumbit the patch. > > > > Can you clarify if you really want all 3 devices installed all the time or > > do you really want a recipe that installs the boot monitor per device? I > > know you don't currently have 3 machine types so maybe that is what is > > feeding your issue here, but my question is whether you need to have > > separate builds per device. > > > > I want all the 3 boot monitors built and installed all the time in one > recipe, since MCSDK 3.1 supports all the 3 Keystone II devices in the > same release package. This applies to the U-boot (3 U-boot build for all > the 3 Keystone II devices) and Linux kernel DTB. Linux kernel has support for board variations through DTBs, obviously. As of U-boot, in Sitara world we had to manage board variations by detecting the board at runtime. So, the same single binary would work on AM335x-EVM, AM335x-SK, BeagleBone White and BeagleBone Black. I would recommend you working with Tom Rini and doing it similarly, so you don't have to build 3 different binaries for 3 slightly different Keystone baords... > >> > >>>>>>> } > >>>>>>> > >>>>>>> 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 > >>>>> > >>>> -- > >>>> _______________________________________________ > >>>> meta-ti mailing list > >>>> meta-ti@yoctoproject.org > >>>> https://lists.yoctoproject.org/listinfo/meta-ti > > > ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 14:54 ` Denys Dmytriyenko @ 2014-05-15 15:07 ` Hao Zhang 2014-05-15 15:38 ` Santosh Shilimkar 2014-05-15 15:20 ` Tom Rini 1 sibling, 1 reply; 22+ messages in thread From: Hao Zhang @ 2014-05-15 15:07 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: trini, meta-ti@yoctoproject.org, Santosh Shilimkar On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: > On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: >> On 5/15/2014 10:21 AM, Maupin, Chase wrote: >>>> -----Original Message----- >>>> From: Zhang, Hao >>>> Sent: Wednesday, May 14, 2014 3:38 PM >>>> To: Maupin, Chase >>>> Cc: Dmytriyenko, Denys; meta-ti@yoctoproject.org >>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >>>> monitor build support >>>> >>>> On 5/14/2014 3:55 PM, Maupin, Chase wrote: >>>>>> -----Original Message----- >>>>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- >>>>>> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys >>>>>> Sent: Wednesday, May 14, 2014 2:01 PM >>>>>> To: Zhang, Hao >>>>>> Cc: meta-ti@yoctoproject.org >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >>>> boot >>>>>> monitor build support >>>>>> >>>>>> 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@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" >>>>> >>>>> Maybe a silly question but do you need to package all of these >>>> in a single recipe or do you want to package K2HK and K2L and K2E >>>> in separate packages? >>>>> >>>>>>>> >>>>>>>> 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. >>>>>> >>>>>> >>>> >>>> For some reason, it still works, anyway I fixed the lowercase >>>> errors >>>> >>>>>>>>> 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 >>>>> >>>>> If you really want to package all of these in a single recipe >>>> why not use a for loop? The naming, etc is formulaic. You could >>>> just have a list of "k2hk k2l k2e" and do the make for that list >>>> as well as your do_deploy stuff below. >>>>> >>>> >>>> Will use for loop and re-sumbit the patch. >>> >>> Can you clarify if you really want all 3 devices installed all the time or >>> do you really want a recipe that installs the boot monitor per device? I >>> know you don't currently have 3 machine types so maybe that is what is >>> feeding your issue here, but my question is whether you need to have >>> separate builds per device. >>> >> >> I want all the 3 boot monitors built and installed all the time in one >> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices in the >> same release package. This applies to the U-boot (3 U-boot build for all >> the 3 Keystone II devices) and Linux kernel DTB. > > Linux kernel has support for board variations through DTBs, obviously. > > As of U-boot, in Sitara world we had to manage board variations by detecting > the board at runtime. So, the same single binary would work on AM335x-EVM, > AM335x-SK, BeagleBone White and BeagleBone Black. > > I would recommend you working with Tom Rini and doing it similarly, so you > don't have to build 3 different binaries for 3 slightly different Keystone > baords... > > Keystone Linux team is working on the up-streaming of U-boot which may include a single binary of U-boot for all the 3 different boards, but for MCSDK 3.1 release, we will still use 3 binaries. I am looping Santosh who is the Keystone Linux team manager .. >>>> >>>>>>>>> } >>>>>>>>> >>>>>>>>> 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 >>>>>>> >>>>>> -- >>>>>> _______________________________________________ >>>>>> meta-ti mailing list >>>>>> meta-ti@yoctoproject.org >>>>>> https://lists.yoctoproject.org/listinfo/meta-ti >>> >> ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 15:07 ` Hao Zhang @ 2014-05-15 15:38 ` Santosh Shilimkar 2014-05-15 15:56 ` Maupin, Chase 0 siblings, 1 reply; 22+ messages in thread From: Santosh Shilimkar @ 2014-05-15 15:38 UTC (permalink / raw) To: Hao Zhang, Denys Dmytriyenko; +Cc: trini, meta-ti@yoctoproject.org On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: > On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: [..] >>>> Can you clarify if you really want all 3 devices installed all the time or >>>> do you really want a recipe that installs the boot monitor per device? I >>>> know you don't currently have 3 machine types so maybe that is what is >>>> feeding your issue here, but my question is whether you need to have >>>> separate builds per device. >>>> >>> >>> I want all the 3 boot monitors built and installed all the time in one >>> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices in the >>> same release package. This applies to the U-boot (3 U-boot build for all >>> the 3 Keystone II devices) and Linux kernel DTB. >> >> Linux kernel has support for board variations through DTBs, obviously. >> >> As of U-boot, in Sitara world we had to manage board variations by detecting >> the board at runtime. So, the same single binary would work on AM335x-EVM, >> AM335x-SK, BeagleBone White and BeagleBone Black. >> >> I would recommend you working with Tom Rini and doing it similarly, so you >> don't have to build 3 different binaries for 3 slightly different Keystone >> baords... >> >> Three boars for same SOC is different than 3 different SOCs with their own boards. We need to support different u-boot configs for that. And upstream of the patches work is already in progress with Tom reviewing the patches. Regards, Santosh ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 15:38 ` Santosh Shilimkar @ 2014-05-15 15:56 ` Maupin, Chase 2014-05-15 16:06 ` Santosh Shilimkar 0 siblings, 1 reply; 22+ messages in thread From: Maupin, Chase @ 2014-05-15 15:56 UTC (permalink / raw) To: Shilimkar, Santosh, Zhang, Hao, Dmytriyenko, Denys Cc: Rini, Tom, meta-ti@yoctoproject.org >-----Original Message----- >From: Shilimkar, Santosh >Sent: Thursday, May 15, 2014 10:39 AM >To: Zhang, Hao; Dmytriyenko, Denys >Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >monitor build support > >On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: > >[..] > >>>>> Can you clarify if you really want all 3 devices installed >all the time or >>>>> do you really want a recipe that installs the boot monitor >per device? I >>>>> know you don't currently have 3 machine types so maybe that >is what is >>>>> feeding your issue here, but my question is whether you need >to have >>>>> separate builds per device. >>>>> >>>> >>>> I want all the 3 boot monitors built and installed all the >time in one >>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices >in the >>>> same release package. This applies to the U-boot (3 U-boot >build for all >>>> the 3 Keystone II devices) and Linux kernel DTB. >>> >>> Linux kernel has support for board variations through DTBs, >obviously. >>> >>> As of U-boot, in Sitara world we had to manage board variations >by detecting >>> the board at runtime. So, the same single binary would work on >AM335x-EVM, >>> AM335x-SK, BeagleBone White and BeagleBone Black. >>> >>> I would recommend you working with Tom Rini and doing it >similarly, so you >>> don't have to build 3 different binaries for 3 slightly >different Keystone >>> baords... >>> >>> >Three boars for same SOC is different than 3 different SOCs with >their >own boards. We need to support different u-boot configs for that. >And >upstream of the patches work is already in progress with Tom >reviewing >the patches. So which one is it? Is this a case of three boards for a single SoC or 3 SoCs with their own boards? > >Regards, >Santosh > ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 15:56 ` Maupin, Chase @ 2014-05-15 16:06 ` Santosh Shilimkar 2014-05-15 16:11 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Santosh Shilimkar @ 2014-05-15 16:06 UTC (permalink / raw) To: Maupin, Chase, Zhang, Hao, Dmytriyenko, Denys Cc: Rini, Tom, meta-ti@yoctoproject.org On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: >> -----Original Message----- >> From: Shilimkar, Santosh >> Sent: Thursday, May 15, 2014 10:39 AM >> To: Zhang, Hao; Dmytriyenko, Denys >> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >> monitor build support >> >> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: >> >> [..] >> >>>>>> Can you clarify if you really want all 3 devices installed >> all the time or >>>>>> do you really want a recipe that installs the boot monitor >> per device? I >>>>>> know you don't currently have 3 machine types so maybe that >> is what is >>>>>> feeding your issue here, but my question is whether you need >> to have >>>>>> separate builds per device. >>>>>> >>>>> >>>>> I want all the 3 boot monitors built and installed all the >> time in one >>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices >> in the >>>>> same release package. This applies to the U-boot (3 U-boot >> build for all >>>>> the 3 Keystone II devices) and Linux kernel DTB. >>>> >>>> Linux kernel has support for board variations through DTBs, >> obviously. >>>> >>>> As of U-boot, in Sitara world we had to manage board variations >> by detecting >>>> the board at runtime. So, the same single binary would work on >> AM335x-EVM, >>>> AM335x-SK, BeagleBone White and BeagleBone Black. >>>> >>>> I would recommend you working with Tom Rini and doing it >> similarly, so you >>>> don't have to build 3 different binaries for 3 slightly >> different Keystone >>>> baords... >>>> >>>> >> Three boars for same SOC is different than 3 different SOCs with >> their >> own boards. We need to support different u-boot configs for that. >> And >> upstream of the patches work is already in progress with Tom >> reviewing >> the patches. > > So which one is it? Is this a case of three boards for a single SoC or 3 SoCs with their own boards? > I was just saying you AM example was multiple board for 1 SOC. What Hao is talking '3 SOCs with their own boards. Regards, Santosh ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:06 ` Santosh Shilimkar @ 2014-05-15 16:11 ` Denys Dmytriyenko 2014-05-15 16:14 ` Santosh Shilimkar 0 siblings, 1 reply; 22+ messages in thread From: Denys Dmytriyenko @ 2014-05-15 16:11 UTC (permalink / raw) To: Santosh Shilimkar; +Cc: Rini, Tom, meta-ti@yoctoproject.org On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar wrote: > On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: > >> -----Original Message----- > >> From: Shilimkar, Santosh > >> Sent: Thursday, May 15, 2014 10:39 AM > >> To: Zhang, Hao; Dmytriyenko, Denys > >> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org > >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > >> monitor build support > >> > >> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: > >>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: > >>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: > >> > >> [..] > >> > >>>>>> Can you clarify if you really want all 3 devices installed > >> all the time or > >>>>>> do you really want a recipe that installs the boot monitor > >> per device? I > >>>>>> know you don't currently have 3 machine types so maybe that > >> is what is > >>>>>> feeding your issue here, but my question is whether you need > >> to have > >>>>>> separate builds per device. > >>>>>> > >>>>> > >>>>> I want all the 3 boot monitors built and installed all the > >> time in one > >>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices > >> in the > >>>>> same release package. This applies to the U-boot (3 U-boot > >> build for all > >>>>> the 3 Keystone II devices) and Linux kernel DTB. > >>>> > >>>> Linux kernel has support for board variations through DTBs, > >> obviously. > >>>> > >>>> As of U-boot, in Sitara world we had to manage board variations > >> by detecting > >>>> the board at runtime. So, the same single binary would work on > >> AM335x-EVM, > >>>> AM335x-SK, BeagleBone White and BeagleBone Black. > >>>> > >>>> I would recommend you working with Tom Rini and doing it > >> similarly, so you > >>>> don't have to build 3 different binaries for 3 slightly > >> different Keystone > >>>> baords... > >>>> > >>>> > >> Three boars for same SOC is different than 3 different SOCs with > >> their > >> own boards. We need to support different u-boot configs for that. > >> And > >> upstream of the patches work is already in progress with Tom > >> reviewing > >> the patches. > > > > So which one is it? Is this a case of three boards for a single SoC or 3 SoCs with their own boards? > > > I was just saying you AM example was multiple board for 1 SOC. What Hao is talking > '3 SOCs with their own boards. If those are 3 different SOCs (not just spins or diff part #s), then we should consider creating 3 different OE machine configs. -- Denys ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:11 ` Denys Dmytriyenko @ 2014-05-15 16:14 ` Santosh Shilimkar 2014-05-15 16:22 ` Maupin, Chase 0 siblings, 1 reply; 22+ messages in thread From: Santosh Shilimkar @ 2014-05-15 16:14 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: Rini, Tom, meta-ti@yoctoproject.org On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: > On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar wrote: >> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: >>>> -----Original Message----- >>>> From: Shilimkar, Santosh >>>> Sent: Thursday, May 15, 2014 10:39 AM >>>> To: Zhang, Hao; Dmytriyenko, Denys >>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >>>> monitor build support >>>> >>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: >>>> >>>> [..] >>>> >>>>>>>> Can you clarify if you really want all 3 devices installed >>>> all the time or >>>>>>>> do you really want a recipe that installs the boot monitor >>>> per device? I >>>>>>>> know you don't currently have 3 machine types so maybe that >>>> is what is >>>>>>>> feeding your issue here, but my question is whether you need >>>> to have >>>>>>>> separate builds per device. >>>>>>>> >>>>>>> >>>>>>> I want all the 3 boot monitors built and installed all the >>>> time in one >>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II devices >>>> in the >>>>>>> same release package. This applies to the U-boot (3 U-boot >>>> build for all >>>>>>> the 3 Keystone II devices) and Linux kernel DTB. >>>>>> >>>>>> Linux kernel has support for board variations through DTBs, >>>> obviously. >>>>>> >>>>>> As of U-boot, in Sitara world we had to manage board variations >>>> by detecting >>>>>> the board at runtime. So, the same single binary would work on >>>> AM335x-EVM, >>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. >>>>>> >>>>>> I would recommend you working with Tom Rini and doing it >>>> similarly, so you >>>>>> don't have to build 3 different binaries for 3 slightly >>>> different Keystone >>>>>> baords... >>>>>> >>>>>> >>>> Three boars for same SOC is different than 3 different SOCs with >>>> their >>>> own boards. We need to support different u-boot configs for that. >>>> And >>>> upstream of the patches work is already in progress with Tom >>>> reviewing >>>> the patches. >>> >>> So which one is it? Is this a case of three boards for a single SoC or 3 SoCs with their own boards? >>> >> I was just saying you AM example was multiple board for 1 SOC. What Hao is talking >> '3 SOCs with their own boards. > > If those are 3 different SOCs (not just spins or diff part #s), then we should > consider creating 3 different OE machine configs. > yes they are 3 different SOCs with different capabilities ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:14 ` Santosh Shilimkar @ 2014-05-15 16:22 ` Maupin, Chase 2014-05-15 16:26 ` Hao Zhang 0 siblings, 1 reply; 22+ messages in thread From: Maupin, Chase @ 2014-05-15 16:22 UTC (permalink / raw) To: Shilimkar, Santosh, Dmytriyenko, Denys Cc: Rini, Tom, meta-ti@yoctoproject.org >-----Original Message----- >From: Shilimkar, Santosh >Sent: Thursday, May 15, 2014 11:14 AM >To: Dmytriyenko, Denys >Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta-ti@yoctoproject.org >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >monitor build support > >On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: >> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar >wrote: >>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: >>>>> -----Original Message----- >>>>> From: Shilimkar, Santosh >>>>> Sent: Thursday, May 15, 2014 10:39 AM >>>>> To: Zhang, Hao; Dmytriyenko, Denys >>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >boot >>>>> monitor build support >>>>> >>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: >>>>> >>>>> [..] >>>>> >>>>>>>>> Can you clarify if you really want all 3 devices >installed >>>>> all the time or >>>>>>>>> do you really want a recipe that installs the boot >monitor >>>>> per device? I >>>>>>>>> know you don't currently have 3 machine types so maybe >that >>>>> is what is >>>>>>>>> feeding your issue here, but my question is whether you >need >>>>> to have >>>>>>>>> separate builds per device. >>>>>>>>> >>>>>>>> >>>>>>>> I want all the 3 boot monitors built and installed all the >>>>> time in one >>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II >devices >>>>> in the >>>>>>>> same release package. This applies to the U-boot (3 U-boot >>>>> build for all >>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. >>>>>>> >>>>>>> Linux kernel has support for board variations through DTBs, >>>>> obviously. >>>>>>> >>>>>>> As of U-boot, in Sitara world we had to manage board >variations >>>>> by detecting >>>>>>> the board at runtime. So, the same single binary would work >on >>>>> AM335x-EVM, >>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. >>>>>>> >>>>>>> I would recommend you working with Tom Rini and doing it >>>>> similarly, so you >>>>>>> don't have to build 3 different binaries for 3 slightly >>>>> different Keystone >>>>>>> baords... >>>>>>> >>>>>>> >>>>> Three boars for same SOC is different than 3 different SOCs >with >>>>> their >>>>> own boards. We need to support different u-boot configs for >that. >>>>> And >>>>> upstream of the patches work is already in progress with Tom >>>>> reviewing >>>>> the patches. >>>> >>>> So which one is it? Is this a case of three boards for a >single SoC or 3 SoCs with their own boards? >>>> >>> I was just saying you AM example was multiple board for 1 SOC. >What Hao is talking >>> '3 SOCs with their own boards. >> >> If those are 3 different SOCs (not just spins or diff part #s), >then we should >> consider creating 3 different OE machine configs. >> >yes they are 3 different SOCs with different capabilities Then Denys is right. We should have 3 different OE machine configs which all share an SOC_FAMILY of "keystone". That way they can re-use as much as possible, but unique differences such as the bootloader, example apps, etc can be easily handled. ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:22 ` Maupin, Chase @ 2014-05-15 16:26 ` Hao Zhang 2014-05-15 16:33 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Hao Zhang @ 2014-05-15 16:26 UTC (permalink / raw) To: Maupin, Chase; +Cc: Rini, Tom, meta-ti@yoctoproject.org, Shilimkar, Santosh On 5/15/2014 12:22 PM, Maupin, Chase wrote: >> -----Original Message----- >> From: Shilimkar, Santosh >> Sent: Thursday, May 15, 2014 11:14 AM >> To: Dmytriyenko, Denys >> Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta-ti@yoctoproject.org >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >> monitor build support >> >> On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: >>> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar >> wrote: >>>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: >>>>>> -----Original Message----- >>>>>> From: Shilimkar, Santosh >>>>>> Sent: Thursday, May 15, 2014 10:39 AM >>>>>> To: Zhang, Hao; Dmytriyenko, Denys >>>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >> boot >>>>>> monitor build support >>>>>> >>>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >>>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >>>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: >>>>>> >>>>>> [..] >>>>>> >>>>>>>>>> Can you clarify if you really want all 3 devices >> installed >>>>>> all the time or >>>>>>>>>> do you really want a recipe that installs the boot >> monitor >>>>>> per device? I >>>>>>>>>> know you don't currently have 3 machine types so maybe >> that >>>>>> is what is >>>>>>>>>> feeding your issue here, but my question is whether you >> need >>>>>> to have >>>>>>>>>> separate builds per device. >>>>>>>>>> >>>>>>>>> >>>>>>>>> I want all the 3 boot monitors built and installed all the >>>>>> time in one >>>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II >> devices >>>>>> in the >>>>>>>>> same release package. This applies to the U-boot (3 U-boot >>>>>> build for all >>>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. >>>>>>>> >>>>>>>> Linux kernel has support for board variations through DTBs, >>>>>> obviously. >>>>>>>> >>>>>>>> As of U-boot, in Sitara world we had to manage board >> variations >>>>>> by detecting >>>>>>>> the board at runtime. So, the same single binary would work >> on >>>>>> AM335x-EVM, >>>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. >>>>>>>> >>>>>>>> I would recommend you working with Tom Rini and doing it >>>>>> similarly, so you >>>>>>>> don't have to build 3 different binaries for 3 slightly >>>>>> different Keystone >>>>>>>> baords... >>>>>>>> >>>>>>>> >>>>>> Three boars for same SOC is different than 3 different SOCs >> with >>>>>> their >>>>>> own boards. We need to support different u-boot configs for >> that. >>>>>> And >>>>>> upstream of the patches work is already in progress with Tom >>>>>> reviewing >>>>>> the patches. >>>>> >>>>> So which one is it? Is this a case of three boards for a >> single SoC or 3 SoCs with their own boards? >>>>> >>>> I was just saying you AM example was multiple board for 1 SOC. >> What Hao is talking >>>> '3 SOCs with their own boards. >>> >>> If those are 3 different SOCs (not just spins or diff part #s), >> then we should >>> consider creating 3 different OE machine configs. >>> >> yes they are 3 different SOCs with different capabilities > > Then Denys is right. We should have 3 different OE machine configs which all share an SOC_FAMILY of "keystone". That way they can re-use as much as possible, but unique differences such as the bootloader, example apps, etc can be easily handled. > Can you show me an example how to do that? ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:26 ` Hao Zhang @ 2014-05-15 16:33 ` Denys Dmytriyenko 2014-06-04 13:48 ` Maupin, Chase 0 siblings, 1 reply; 22+ messages in thread From: Denys Dmytriyenko @ 2014-05-15 16:33 UTC (permalink / raw) To: Hao Zhang; +Cc: Rini, Tom, meta-ti@yoctoproject.org, Shilimkar, Santosh On Thu, May 15, 2014 at 12:26:13PM -0400, Hao Zhang wrote: > On 5/15/2014 12:22 PM, Maupin, Chase wrote: > >> -----Original Message----- > >> From: Shilimkar, Santosh > >> Sent: Thursday, May 15, 2014 11:14 AM > >> To: Dmytriyenko, Denys > >> Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta-ti@yoctoproject.org > >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > >> monitor build support > >> > >> On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: > >>> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar > >> wrote: > >>>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: > >>>>>> -----Original Message----- > >>>>>> From: Shilimkar, Santosh > >>>>>> Sent: Thursday, May 15, 2014 10:39 AM > >>>>>> To: Zhang, Hao; Dmytriyenko, Denys > >>>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org > >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E > >> boot > >>>>>> monitor build support > >>>>>> > >>>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: > >>>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: > >>>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: > >>>>>> > >>>>>> [..] > >>>>>> > >>>>>>>>>> Can you clarify if you really want all 3 devices > >> installed > >>>>>> all the time or > >>>>>>>>>> do you really want a recipe that installs the boot > >> monitor > >>>>>> per device? I > >>>>>>>>>> know you don't currently have 3 machine types so maybe > >> that > >>>>>> is what is > >>>>>>>>>> feeding your issue here, but my question is whether you > >> need > >>>>>> to have > >>>>>>>>>> separate builds per device. > >>>>>>>>>> > >>>>>>>>> > >>>>>>>>> I want all the 3 boot monitors built and installed all the > >>>>>> time in one > >>>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II > >> devices > >>>>>> in the > >>>>>>>>> same release package. This applies to the U-boot (3 U-boot > >>>>>> build for all > >>>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. > >>>>>>>> > >>>>>>>> Linux kernel has support for board variations through DTBs, > >>>>>> obviously. > >>>>>>>> > >>>>>>>> As of U-boot, in Sitara world we had to manage board > >> variations > >>>>>> by detecting > >>>>>>>> the board at runtime. So, the same single binary would work > >> on > >>>>>> AM335x-EVM, > >>>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. > >>>>>>>> > >>>>>>>> I would recommend you working with Tom Rini and doing it > >>>>>> similarly, so you > >>>>>>>> don't have to build 3 different binaries for 3 slightly > >>>>>> different Keystone > >>>>>>>> baords... > >>>>>>>> > >>>>>>>> > >>>>>> Three boars for same SOC is different than 3 different SOCs > >> with > >>>>>> their > >>>>>> own boards. We need to support different u-boot configs for > >> that. > >>>>>> And > >>>>>> upstream of the patches work is already in progress with Tom > >>>>>> reviewing > >>>>>> the patches. > >>>>> > >>>>> So which one is it? Is this a case of three boards for a > >> single SoC or 3 SoCs with their own boards? > >>>>> > >>>> I was just saying you AM example was multiple board for 1 SOC. > >> What Hao is talking > >>>> '3 SOCs with their own boards. > >>> > >>> If those are 3 different SOCs (not just spins or diff part #s), > >> then we should > >>> consider creating 3 different OE machine configs. > >>> > >> yes they are 3 different SOCs with different capabilities > > > > Then Denys is right. We should have 3 different OE machine configs which all share an SOC_FAMILY of "keystone". That way they can re-use as much as possible, but unique differences such as the bootloader, example apps, etc can be easily handled. > > > > Can you show me an example how to do that? meta-ti layer, conf/machine for machine configs and conf/machine/include for SOC configs. examples would be: - am335x-evm.conf and beaglebone.conf both use ti33x.inc SOC definition - dra7xx-evm.conf and omap5-evm.conf both use omap-a15.inc SOC -- Denys ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 16:33 ` Denys Dmytriyenko @ 2014-06-04 13:48 ` Maupin, Chase 2014-06-04 15:04 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Maupin, Chase @ 2014-06-04 13:48 UTC (permalink / raw) To: Dmytriyenko, Denys, Zhang, Hao Cc: Rini, Tom, meta-ti@yoctoproject.org, Shilimkar, Santosh >-----Original Message----- >From: Dmytriyenko, Denys >Sent: Thursday, May 15, 2014 11:33 AM >To: Zhang, Hao >Cc: Maupin, Chase; Shilimkar, Santosh; Rini, Tom; meta- >ti@yoctoproject.org >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot >monitor build support > >On Thu, May 15, 2014 at 12:26:13PM -0400, Hao Zhang wrote: >> On 5/15/2014 12:22 PM, Maupin, Chase wrote: >> >> -----Original Message----- >> >> From: Shilimkar, Santosh >> >> Sent: Thursday, May 15, 2014 11:14 AM >> >> To: Dmytriyenko, Denys >> >> Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta- >ti@yoctoproject.org >> >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E >boot >> >> monitor build support >> >> >> >> On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: >> >>> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar >> >> wrote: >> >>>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: >> >>>>>> -----Original Message----- >> >>>>>> From: Shilimkar, Santosh >> >>>>>> Sent: Thursday, May 15, 2014 10:39 AM >> >>>>>> To: Zhang, Hao; Dmytriyenko, Denys >> >>>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org >> >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and >K2E >> >> boot >> >>>>>> monitor build support >> >>>>>> >> >>>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: >> >>>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: >> >>>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang >wrote: >> >>>>>> >> >>>>>> [..] >> >>>>>> >> >>>>>>>>>> Can you clarify if you really want all 3 devices >> >> installed >> >>>>>> all the time or >> >>>>>>>>>> do you really want a recipe that installs the boot >> >> monitor >> >>>>>> per device? I >> >>>>>>>>>> know you don't currently have 3 machine types so >maybe >> >> that >> >>>>>> is what is >> >>>>>>>>>> feeding your issue here, but my question is whether >you >> >> need >> >>>>>> to have >> >>>>>>>>>> separate builds per device. >> >>>>>>>>>> >> >>>>>>>>> >> >>>>>>>>> I want all the 3 boot monitors built and installed all >the >> >>>>>> time in one >> >>>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II >> >> devices >> >>>>>> in the >> >>>>>>>>> same release package. This applies to the U-boot (3 U- >boot >> >>>>>> build for all >> >>>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. >> >>>>>>>> >> >>>>>>>> Linux kernel has support for board variations through >DTBs, >> >>>>>> obviously. >> >>>>>>>> >> >>>>>>>> As of U-boot, in Sitara world we had to manage board >> >> variations >> >>>>>> by detecting >> >>>>>>>> the board at runtime. So, the same single binary would >work >> >> on >> >>>>>> AM335x-EVM, >> >>>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. >> >>>>>>>> >> >>>>>>>> I would recommend you working with Tom Rini and doing >it >> >>>>>> similarly, so you >> >>>>>>>> don't have to build 3 different binaries for 3 slightly >> >>>>>> different Keystone >> >>>>>>>> baords... >> >>>>>>>> >> >>>>>>>> >> >>>>>> Three boars for same SOC is different than 3 different >SOCs >> >> with >> >>>>>> their >> >>>>>> own boards. We need to support different u-boot configs >for >> >> that. >> >>>>>> And >> >>>>>> upstream of the patches work is already in progress with >Tom >> >>>>>> reviewing >> >>>>>> the patches. >> >>>>> >> >>>>> So which one is it? Is this a case of three boards for a >> >> single SoC or 3 SoCs with their own boards? >> >>>>> >> >>>> I was just saying you AM example was multiple board for 1 >SOC. >> >> What Hao is talking >> >>>> '3 SOCs with their own boards. >> >>> >> >>> If those are 3 different SOCs (not just spins or diff part >#s), >> >> then we should >> >>> consider creating 3 different OE machine configs. >> >>> >> >> yes they are 3 different SOCs with different capabilities >> > >> > Then Denys is right. We should have 3 different OE machine >configs which all share an SOC_FAMILY of "keystone". That way >they can re-use as much as possible, but unique differences such >as the bootloader, example apps, etc can be easily handled. >> > >> >> Can you show me an example how to do that? > >meta-ti layer, conf/machine for machine configs and >conf/machine/include for >SOC configs. > >examples would be: > >- am335x-evm.conf and beaglebone.conf both use ti33x.inc SOC >definition >- dra7xx-evm.conf and omap5-evm.conf both use omap-a15.inc SOC Denys, Is there something outstanding here? Does this patch need to be revamped now that we have individual machine types? > >-- >Denys ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-06-04 13:48 ` Maupin, Chase @ 2014-06-04 15:04 ` Denys Dmytriyenko 2014-06-05 15:12 ` Denys Dmytriyenko 0 siblings, 1 reply; 22+ messages in thread From: Denys Dmytriyenko @ 2014-06-04 15:04 UTC (permalink / raw) To: Maupin, Chase; +Cc: Rini, Tom, meta-ti@yoctoproject.org, Shilimkar, Santosh On Wed, Jun 04, 2014 at 09:48:57AM -0400, Maupin, Chase wrote: > >-----Original Message----- > >From: Dmytriyenko, Denys > >Sent: Thursday, May 15, 2014 11:33 AM > >To: Zhang, Hao > >Cc: Maupin, Chase; Shilimkar, Santosh; Rini, Tom; meta- > >ti@yoctoproject.org > >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > >monitor build support > > > >On Thu, May 15, 2014 at 12:26:13PM -0400, Hao Zhang wrote: > >> On 5/15/2014 12:22 PM, Maupin, Chase wrote: > >> >> -----Original Message----- > >> >> From: Shilimkar, Santosh > >> >> Sent: Thursday, May 15, 2014 11:14 AM > >> >> To: Dmytriyenko, Denys > >> >> Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta- > >ti@yoctoproject.org > >> >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E > >boot > >> >> monitor build support > >> >> > >> >> On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: > >> >>> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar > >> >> wrote: > >> >>>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: > >> >>>>>> -----Original Message----- > >> >>>>>> From: Shilimkar, Santosh > >> >>>>>> Sent: Thursday, May 15, 2014 10:39 AM > >> >>>>>> To: Zhang, Hao; Dmytriyenko, Denys > >> >>>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org > >> >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and > >K2E > >> >> boot > >> >>>>>> monitor build support > >> >>>>>> > >> >>>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: > >> >>>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: > >> >>>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang > >wrote: > >> >>>>>> > >> >>>>>> [..] > >> >>>>>> > >> >>>>>>>>>> Can you clarify if you really want all 3 devices > >> >> installed > >> >>>>>> all the time or > >> >>>>>>>>>> do you really want a recipe that installs the boot > >> >> monitor > >> >>>>>> per device? I > >> >>>>>>>>>> know you don't currently have 3 machine types so > >maybe > >> >> that > >> >>>>>> is what is > >> >>>>>>>>>> feeding your issue here, but my question is whether > >you > >> >> need > >> >>>>>> to have > >> >>>>>>>>>> separate builds per device. > >> >>>>>>>>>> > >> >>>>>>>>> > >> >>>>>>>>> I want all the 3 boot monitors built and installed all > >the > >> >>>>>> time in one > >> >>>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II > >> >> devices > >> >>>>>> in the > >> >>>>>>>>> same release package. This applies to the U-boot (3 U- > >boot > >> >>>>>> build for all > >> >>>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. > >> >>>>>>>> > >> >>>>>>>> Linux kernel has support for board variations through > >DTBs, > >> >>>>>> obviously. > >> >>>>>>>> > >> >>>>>>>> As of U-boot, in Sitara world we had to manage board > >> >> variations > >> >>>>>> by detecting > >> >>>>>>>> the board at runtime. So, the same single binary would > >work > >> >> on > >> >>>>>> AM335x-EVM, > >> >>>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. > >> >>>>>>>> > >> >>>>>>>> I would recommend you working with Tom Rini and doing > >it > >> >>>>>> similarly, so you > >> >>>>>>>> don't have to build 3 different binaries for 3 slightly > >> >>>>>> different Keystone > >> >>>>>>>> baords... > >> >>>>>>>> > >> >>>>>>>> > >> >>>>>> Three boars for same SOC is different than 3 different > >SOCs > >> >> with > >> >>>>>> their > >> >>>>>> own boards. We need to support different u-boot configs > >for > >> >> that. > >> >>>>>> And > >> >>>>>> upstream of the patches work is already in progress with > >Tom > >> >>>>>> reviewing > >> >>>>>> the patches. > >> >>>>> > >> >>>>> So which one is it? Is this a case of three boards for a > >> >> single SoC or 3 SoCs with their own boards? > >> >>>>> > >> >>>> I was just saying you AM example was multiple board for 1 > >SOC. > >> >> What Hao is talking > >> >>>> '3 SOCs with their own boards. > >> >>> > >> >>> If those are 3 different SOCs (not just spins or diff part > >#s), > >> >> then we should > >> >>> consider creating 3 different OE machine configs. > >> >>> > >> >> yes they are 3 different SOCs with different capabilities > >> > > >> > Then Denys is right. We should have 3 different OE machine > >configs which all share an SOC_FAMILY of "keystone". That way > >they can re-use as much as possible, but unique differences such > >as the bootloader, example apps, etc can be easily handled. > >> > > >> > >> Can you show me an example how to do that? > > > >meta-ti layer, conf/machine for machine configs and > >conf/machine/include for > >SOC configs. > > > >examples would be: > > > >- am335x-evm.conf and beaglebone.conf both use ti33x.inc SOC > >definition > >- dra7xx-evm.conf and omap5-evm.conf both use omap-a15.inc SOC > > Denys, > > Is there something outstanding here? Does this patch need to be revamped > now that we have individual machine types? Yes, indeed. This recipe should build boot-monitor for current ${MACHINE} and not all 3 of them together. Similar to kernel and u-boot. -- Denys ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-06-04 15:04 ` Denys Dmytriyenko @ 2014-06-05 15:12 ` Denys Dmytriyenko 0 siblings, 0 replies; 22+ messages in thread From: Denys Dmytriyenko @ 2014-06-05 15:12 UTC (permalink / raw) To: Maupin, Chase; +Cc: Rini, Tom, meta-ti@yoctoproject.org, Shilimkar, Santosh Hao, Do you have any questions about the changes needed here? Will you be sending the updated patch? Please let me know if you need any additional help. Thanks. -- Denys On Wed, Jun 04, 2014 at 11:04:52AM -0400, Denys Dmytriyenko wrote: > On Wed, Jun 04, 2014 at 09:48:57AM -0400, Maupin, Chase wrote: > > >-----Original Message----- > > >From: Dmytriyenko, Denys > > >Sent: Thursday, May 15, 2014 11:33 AM > > >To: Zhang, Hao > > >Cc: Maupin, Chase; Shilimkar, Santosh; Rini, Tom; meta- > > >ti@yoctoproject.org > > >Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > > >monitor build support > > > > > >On Thu, May 15, 2014 at 12:26:13PM -0400, Hao Zhang wrote: > > >> On 5/15/2014 12:22 PM, Maupin, Chase wrote: > > >> >> -----Original Message----- > > >> >> From: Shilimkar, Santosh > > >> >> Sent: Thursday, May 15, 2014 11:14 AM > > >> >> To: Dmytriyenko, Denys > > >> >> Cc: Maupin, Chase; Zhang, Hao; Rini, Tom; meta- > > >ti@yoctoproject.org > > >> >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E > > >boot > > >> >> monitor build support > > >> >> > > >> >> On Thursday 15 May 2014 12:11 PM, Denys Dmytriyenko wrote: > > >> >>> On Thu, May 15, 2014 at 12:06:15PM -0400, Santosh Shilimkar > > >> >> wrote: > > >> >>>> On Thursday 15 May 2014 11:56 AM, Maupin, Chase wrote: > > >> >>>>>> -----Original Message----- > > >> >>>>>> From: Shilimkar, Santosh > > >> >>>>>> Sent: Thursday, May 15, 2014 10:39 AM > > >> >>>>>> To: Zhang, Hao; Dmytriyenko, Denys > > >> >>>>>> Cc: Maupin, Chase; Rini, Tom; meta-ti@yoctoproject.org > > >> >>>>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and > > >K2E > > >> >> boot > > >> >>>>>> monitor build support > > >> >>>>>> > > >> >>>>>> On Thursday 15 May 2014 11:07 AM, Hao Zhang wrote: > > >> >>>>>>> On 5/15/2014 10:54 AM, Denys Dmytriyenko wrote: > > >> >>>>>>>> On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang > > >wrote: > > >> >>>>>> > > >> >>>>>> [..] > > >> >>>>>> > > >> >>>>>>>>>> Can you clarify if you really want all 3 devices > > >> >> installed > > >> >>>>>> all the time or > > >> >>>>>>>>>> do you really want a recipe that installs the boot > > >> >> monitor > > >> >>>>>> per device? I > > >> >>>>>>>>>> know you don't currently have 3 machine types so > > >maybe > > >> >> that > > >> >>>>>> is what is > > >> >>>>>>>>>> feeding your issue here, but my question is whether > > >you > > >> >> need > > >> >>>>>> to have > > >> >>>>>>>>>> separate builds per device. > > >> >>>>>>>>>> > > >> >>>>>>>>> > > >> >>>>>>>>> I want all the 3 boot monitors built and installed all > > >the > > >> >>>>>> time in one > > >> >>>>>>>>> recipe, since MCSDK 3.1 supports all the 3 Keystone II > > >> >> devices > > >> >>>>>> in the > > >> >>>>>>>>> same release package. This applies to the U-boot (3 U- > > >boot > > >> >>>>>> build for all > > >> >>>>>>>>> the 3 Keystone II devices) and Linux kernel DTB. > > >> >>>>>>>> > > >> >>>>>>>> Linux kernel has support for board variations through > > >DTBs, > > >> >>>>>> obviously. > > >> >>>>>>>> > > >> >>>>>>>> As of U-boot, in Sitara world we had to manage board > > >> >> variations > > >> >>>>>> by detecting > > >> >>>>>>>> the board at runtime. So, the same single binary would > > >work > > >> >> on > > >> >>>>>> AM335x-EVM, > > >> >>>>>>>> AM335x-SK, BeagleBone White and BeagleBone Black. > > >> >>>>>>>> > > >> >>>>>>>> I would recommend you working with Tom Rini and doing > > >it > > >> >>>>>> similarly, so you > > >> >>>>>>>> don't have to build 3 different binaries for 3 slightly > > >> >>>>>> different Keystone > > >> >>>>>>>> baords... > > >> >>>>>>>> > > >> >>>>>>>> > > >> >>>>>> Three boars for same SOC is different than 3 different > > >SOCs > > >> >> with > > >> >>>>>> their > > >> >>>>>> own boards. We need to support different u-boot configs > > >for > > >> >> that. > > >> >>>>>> And > > >> >>>>>> upstream of the patches work is already in progress with > > >Tom > > >> >>>>>> reviewing > > >> >>>>>> the patches. > > >> >>>>> > > >> >>>>> So which one is it? Is this a case of three boards for a > > >> >> single SoC or 3 SoCs with their own boards? > > >> >>>>> > > >> >>>> I was just saying you AM example was multiple board for 1 > > >SOC. > > >> >> What Hao is talking > > >> >>>> '3 SOCs with their own boards. > > >> >>> > > >> >>> If those are 3 different SOCs (not just spins or diff part > > >#s), > > >> >> then we should > > >> >>> consider creating 3 different OE machine configs. > > >> >>> > > >> >> yes they are 3 different SOCs with different capabilities > > >> > > > >> > Then Denys is right. We should have 3 different OE machine > > >configs which all share an SOC_FAMILY of "keystone". That way > > >they can re-use as much as possible, but unique differences such > > >as the bootloader, example apps, etc can be easily handled. > > >> > > > >> > > >> Can you show me an example how to do that? > > > > > >meta-ti layer, conf/machine for machine configs and > > >conf/machine/include for > > >SOC configs. > > > > > >examples would be: > > > > > >- am335x-evm.conf and beaglebone.conf both use ti33x.inc SOC > > >definition > > >- dra7xx-evm.conf and omap5-evm.conf both use omap-a15.inc SOC > > > > Denys, > > > > Is there something outstanding here? Does this patch need to be revamped > > now that we have individual machine types? > > Yes, indeed. This recipe should build boot-monitor for current ${MACHINE} and > not all 3 of them together. Similar to kernel and u-boot. > > -- > Denys > -- > _______________________________________________ > meta-ti mailing list > meta-ti@yoctoproject.org > https://lists.yoctoproject.org/listinfo/meta-ti ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] boot-monitor: add K2L and K2E boot monitor build support 2014-05-15 14:54 ` Denys Dmytriyenko 2014-05-15 15:07 ` Hao Zhang @ 2014-05-15 15:20 ` Tom Rini 1 sibling, 0 replies; 22+ messages in thread From: Tom Rini @ 2014-05-15 15:20 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: meta-ti@yoctoproject.org On Thu, May 15, 2014 at 10:54:32AM -0400, Denys Dmytriyenko wrote: > On Thu, May 15, 2014 at 10:41:52AM -0400, Hao Zhang wrote: > > On 5/15/2014 10:21 AM, Maupin, Chase wrote: > > >> -----Original Message----- > > >> From: Zhang, Hao > > >> Sent: Wednesday, May 14, 2014 3:38 PM > > >> To: Maupin, Chase > > >> Cc: Dmytriyenko, Denys; meta-ti@yoctoproject.org > > >> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E boot > > >> monitor build support > > >> > > >> On 5/14/2014 3:55 PM, Maupin, Chase wrote: > > >>>> -----Original Message----- > > >>>> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti- > > >>>> bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys > > >>>> Sent: Wednesday, May 14, 2014 2:01 PM > > >>>> To: Zhang, Hao > > >>>> Cc: meta-ti@yoctoproject.org > > >>>> Subject: Re: [meta-ti] [PATCH] boot-monitor: add K2L and K2E > > >> boot > > >>>> monitor build support > > >>>> > > >>>> 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@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" > > >>> > > >>> Maybe a silly question but do you need to package all of these > > >> in a single recipe or do you want to package K2HK and K2L and K2E > > >> in separate packages? > > >>> > > >>>>>> > > >>>>>> 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. > > >>>> > > >>>> > > >> > > >> For some reason, it still works, anyway I fixed the lowercase > > >> errors > > >> > > >>>>>>> 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 > > >>> > > >>> If you really want to package all of these in a single recipe > > >> why not use a for loop? The naming, etc is formulaic. You could > > >> just have a list of "k2hk k2l k2e" and do the make for that list > > >> as well as your do_deploy stuff below. > > >>> > > >> > > >> Will use for loop and re-sumbit the patch. > > > > > > Can you clarify if you really want all 3 devices installed all the time or > > > do you really want a recipe that installs the boot monitor per device? I > > > know you don't currently have 3 machine types so maybe that is what is > > > feeding your issue here, but my question is whether you need to have > > > separate builds per device. > > > > > > > I want all the 3 boot monitors built and installed all the time in one > > recipe, since MCSDK 3.1 supports all the 3 Keystone II devices in the > > same release package. This applies to the U-boot (3 U-boot build for all > > the 3 Keystone II devices) and Linux kernel DTB. > > Linux kernel has support for board variations through DTBs, obviously. > > As of U-boot, in Sitara world we had to manage board variations by detecting > the board at runtime. So, the same single binary would work on AM335x-EVM, > AM335x-SK, BeagleBone White and BeagleBone Black. Note that this relies on a reliable way to tell what board we're on at run-time. AM335x TI influenced boards have an i2c EEPROM with a defined layout and a board name field. Siemens does something similar (but different layout) with their am335x HW as well. Beagleboard also supports similar due to intentional GPIO setups, iirc. If Keystone II platforms have some way to tell at run-time, then we can do this there too, otherwise it's 3 binaries. -- Tom ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2014-06-05 15:13 UTC | newest] Thread overview: 22+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-05-14 18:29 [PATCH] boot-monitor: add K2L and K2E boot monitor build support hzhang 2014-05-14 18:40 ` Denys Dmytriyenko 2014-05-14 18:58 ` Hao Zhang 2014-05-14 19:01 ` Denys Dmytriyenko 2014-05-14 19:55 ` Maupin, Chase 2014-05-14 20:38 ` Hao Zhang 2014-05-15 14:21 ` Maupin, Chase 2014-05-15 14:41 ` Hao Zhang 2014-05-15 14:54 ` Denys Dmytriyenko 2014-05-15 15:07 ` Hao Zhang 2014-05-15 15:38 ` Santosh Shilimkar 2014-05-15 15:56 ` Maupin, Chase 2014-05-15 16:06 ` Santosh Shilimkar 2014-05-15 16:11 ` Denys Dmytriyenko 2014-05-15 16:14 ` Santosh Shilimkar 2014-05-15 16:22 ` Maupin, Chase 2014-05-15 16:26 ` Hao Zhang 2014-05-15 16:33 ` Denys Dmytriyenko 2014-06-04 13:48 ` Maupin, Chase 2014-06-04 15:04 ` Denys Dmytriyenko 2014-06-05 15:12 ` Denys Dmytriyenko 2014-05-15 15:20 ` Tom Rini
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.