* linux-dtb: add multi-dtb build support
@ 2012-06-26 3:53 b19537
2012-06-28 17:12 ` Saul Wold
2012-07-16 22:48 ` Saul Wold
0 siblings, 2 replies; 14+ messages in thread
From: b19537 @ 2012-06-26 3:53 UTC (permalink / raw)
To: openembedded-core
From: Zhenhua Luo <b19537@freescale.com>
including following enhancement:
* support multi-dtb build
* skip dtb build and install when KERNEL_DEVICETREE is empty
* print a warning message when specified dts file is not available
Signed-off-by: Zhenhua Luo <b19537@freescale.com>
---
meta/recipes-kernel/linux/linux-dtb.inc | 36 +++++++++++++++++++++++++------
1 files changed, 29 insertions(+), 7 deletions(-)
diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc
index 9188cee..8aa1458 100644
--- a/meta/recipes-kernel/linux/linux-dtb.inc
+++ b/meta/recipes-kernel/linux/linux-dtb.inc
@@ -15,13 +15,35 @@ python __anonymous () {
do_install_append() {
if test -n "${KERNEL_DEVICETREE}"; then
- dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree ${KERNEL_DEVICETREE}
- install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION}
- install -d ${DEPLOYDIR}
- install -m 0644 devicetree ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb
- cd ${DEPLOYDIR}
- rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb
- ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb ${KERNEL_IMAGE_SYMLINK_NAME}.dtb
+ for DTS_FILE in ${KERNEL_DEVICETREE}; do
+ if [ ! -f ${DTS_FILE} ]; then
+ echo "Warning: ${DTS_FILE} is not available!"
+ continue
+ fi
+ DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'`
+ DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"`
+ DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"`
+ dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o ${DTS_BASE_NAME} ${DTS_FILE}
+ install -m 0644 ${DTS_BASE_NAME} ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb
+ done
fi
}
+do_deploy_append() {
+ if test -n "${KERNEL_DEVICETREE}"; then
+ for DTS_FILE in ${KERNEL_DEVICETREE}; do
+ if [ ! -f ${DTS_FILE} ]; then
+ echo "Warning: ${DTS_FILE} is not available!"
+ continue
+ fi
+ DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'`
+ DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"`
+ DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"`
+ install -d ${DEPLOYDIR}
+ install -m 0644 ${B}/${DTS_BASE_NAME} ${DEPLOYDIR}/${DTB_NAME}.dtb
+ cd ${DEPLOYDIR}
+ ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb
+ cd -
+ done
+ fi
+}
--
1.7.0.4
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: linux-dtb: add multi-dtb build support 2012-06-26 3:53 linux-dtb: add multi-dtb build support b19537 @ 2012-06-28 17:12 ` Saul Wold 2012-07-05 18:29 ` McClintock Matthew-B29882 2012-07-16 22:48 ` Saul Wold 1 sibling, 1 reply; 14+ messages in thread From: Saul Wold @ 2012-06-28 17:12 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On 06/25/2012 08:53 PM, b19537@freescale.com wrote: > From: Zhenhua Luo<b19537@freescale.com> > > including following enhancement: > * support multi-dtb build > * skip dtb build and install when KERNEL_DEVICETREE is empty > * print a warning message when specified dts file is not available > > Signed-off-by: Zhenhua Luo<b19537@freescale.com> > --- > meta/recipes-kernel/linux/linux-dtb.inc | 36 +++++++++++++++++++++++++------ > 1 files changed, 29 insertions(+), 7 deletions(-) > > diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc > index 9188cee..8aa1458 100644 > --- a/meta/recipes-kernel/linux/linux-dtb.inc > +++ b/meta/recipes-kernel/linux/linux-dtb.inc > @@ -15,13 +15,35 @@ python __anonymous () { > > do_install_append() { > if test -n "${KERNEL_DEVICETREE}"; then > - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree ${KERNEL_DEVICETREE} > - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} > - install -d ${DEPLOYDIR} > - install -m 0644 devicetree ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb > - cd ${DEPLOYDIR} > - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > + for DTS_FILE in ${KERNEL_DEVICETREE}; do > + if [ ! -f ${DTS_FILE} ]; then > + echo "Warning: ${DTS_FILE} is not available!" > + continue > + fi > + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'` > + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o ${DTS_BASE_NAME} ${DTS_FILE} > + install -m 0644 ${DTS_BASE_NAME} ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb > + done > fi > } > > +do_deploy_append() { > + if test -n "${KERNEL_DEVICETREE}"; then > + for DTS_FILE in ${KERNEL_DEVICETREE}; do > + if [ ! -f ${DTS_FILE} ]; then > + echo "Warning: ${DTS_FILE} is not available!" > + continue > + fi > + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'` > + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + install -d ${DEPLOYDIR} > + install -m 0644 ${B}/${DTS_BASE_NAME} ${DEPLOYDIR}/${DTB_NAME}.dtb > + cd ${DEPLOYDIR} > + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb > + cd - > + done > + fi > +} Merged into OE-Core Thanks Sau! ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-06-28 17:12 ` Saul Wold @ 2012-07-05 18:29 ` McClintock Matthew-B29882 2012-07-05 19:38 ` Scott Garman 2012-07-05 19:59 ` Scott Garman 0 siblings, 2 replies; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-05 18:29 UTC (permalink / raw) To: Patches and discussions about the oe-core layer, Scott Garman, Luo Zhenhua-B19537 Can we add this to denzil branch? (post 1.2.1 is fine) - I realize this is somewhat a new feature though but it would be helpful to have it in denzil. -M On Thu, Jun 28, 2012 at 12:12 PM, Saul Wold <sgw@linux.intel.com> wrote: > On 06/25/2012 08:53 PM, b19537@freescale.com wrote: >> >> From: Zhenhua Luo<b19537@freescale.com> >> >> including following enhancement: >> * support multi-dtb build >> * skip dtb build and install when KERNEL_DEVICETREE is empty >> * print a warning message when specified dts file is not available >> >> Signed-off-by: Zhenhua Luo<b19537@freescale.com> >> --- >> meta/recipes-kernel/linux/linux-dtb.inc | 36 >> +++++++++++++++++++++++++------ >> 1 files changed, 29 insertions(+), 7 deletions(-) >> >> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc >> b/meta/recipes-kernel/linux/linux-dtb.inc >> index 9188cee..8aa1458 100644 >> --- a/meta/recipes-kernel/linux/linux-dtb.inc >> +++ b/meta/recipes-kernel/linux/linux-dtb.inc >> @@ -15,13 +15,35 @@ python __anonymous () { >> >> do_install_append() { >> if test -n "${KERNEL_DEVICETREE}"; then >> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree >> ${KERNEL_DEVICETREE} >> - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} >> - install -d ${DEPLOYDIR} >> - install -m 0644 devicetree >> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb >> - cd ${DEPLOYDIR} >> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb >> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >> + if [ ! -f ${DTS_FILE} ]; then >> + echo "Warning: ${DTS_FILE} is not available!" >> + continue >> + fi >> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >> $1}'` >> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o >> ${DTS_BASE_NAME} ${DTS_FILE} >> + install -m 0644 ${DTS_BASE_NAME} >> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb >> + done >> fi >> } >> >> +do_deploy_append() { >> + if test -n "${KERNEL_DEVICETREE}"; then >> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >> + if [ ! -f ${DTS_FILE} ]; then >> + echo "Warning: ${DTS_FILE} is not available!" >> + continue >> + fi >> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >> $1}'` >> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + install -d ${DEPLOYDIR} >> + install -m 0644 ${B}/${DTS_BASE_NAME} >> ${DEPLOYDIR}/${DTB_NAME}.dtb >> + cd ${DEPLOYDIR} >> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb >> + cd - >> + done >> + fi >> +} > > > Merged into OE-Core > > Thanks > Sau! > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 18:29 ` McClintock Matthew-B29882 @ 2012-07-05 19:38 ` Scott Garman 2012-07-05 19:59 ` Scott Garman 1 sibling, 0 replies; 14+ messages in thread From: Scott Garman @ 2012-07-05 19:38 UTC (permalink / raw) To: McClintock Matthew-B29882 Cc: Luo Zhenhua-B19537, Patches and discussions about the oe-core layer On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: > Can we add this to denzil branch? (post 1.2.1 is fine) - I realize > this is somewhat a new feature though but it would be helpful to have > it in denzil. I've made a note of this, and the commit appears to apply cleanly. So I should be able to take it for 1.2.2. Scott -- Scott Garman Embedded Linux Engineer - Yocto Project Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 18:29 ` McClintock Matthew-B29882 2012-07-05 19:38 ` Scott Garman @ 2012-07-05 19:59 ` Scott Garman 2012-07-05 20:52 ` McClintock Matthew-B29882 1 sibling, 1 reply; 14+ messages in thread From: Scott Garman @ 2012-07-05 19:59 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: > Can we add this to denzil branch? (post 1.2.1 is fine) - I realize > this is somewhat a new feature though but it would be helpful to have > it in denzil. Ok, I've now created a contrib branch to start tracking changes for 1.2.2, and have included this commit: http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 The usual caveats apply - I can rebase this branch at any time, etc. Scott > > -M > > On Thu, Jun 28, 2012 at 12:12 PM, Saul Wold <sgw@linux.intel.com> wrote: >> On 06/25/2012 08:53 PM, b19537@freescale.com wrote: >>> >>> From: Zhenhua Luo<b19537@freescale.com> >>> >>> including following enhancement: >>> * support multi-dtb build >>> * skip dtb build and install when KERNEL_DEVICETREE is empty >>> * print a warning message when specified dts file is not available >>> >>> Signed-off-by: Zhenhua Luo<b19537@freescale.com> >>> --- >>> meta/recipes-kernel/linux/linux-dtb.inc | 36 >>> +++++++++++++++++++++++++------ >>> 1 files changed, 29 insertions(+), 7 deletions(-) >>> >>> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc >>> b/meta/recipes-kernel/linux/linux-dtb.inc >>> index 9188cee..8aa1458 100644 >>> --- a/meta/recipes-kernel/linux/linux-dtb.inc >>> +++ b/meta/recipes-kernel/linux/linux-dtb.inc >>> @@ -15,13 +15,35 @@ python __anonymous () { >>> >>> do_install_append() { >>> if test -n "${KERNEL_DEVICETREE}"; then >>> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree >>> ${KERNEL_DEVICETREE} >>> - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} >>> - install -d ${DEPLOYDIR} >>> - install -m 0644 devicetree >>> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb >>> - cd ${DEPLOYDIR} >>> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb >>> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>> + if [ ! -f ${DTS_FILE} ]; then >>> + echo "Warning: ${DTS_FILE} is not available!" >>> + continue >>> + fi >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>> $1}'` >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o >>> ${DTS_BASE_NAME} ${DTS_FILE} >>> + install -m 0644 ${DTS_BASE_NAME} >>> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb >>> + done >>> fi >>> } >>> >>> +do_deploy_append() { >>> + if test -n "${KERNEL_DEVICETREE}"; then >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>> + if [ ! -f ${DTS_FILE} ]; then >>> + echo "Warning: ${DTS_FILE} is not available!" >>> + continue >>> + fi >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>> $1}'` >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + install -d ${DEPLOYDIR} >>> + install -m 0644 ${B}/${DTS_BASE_NAME} >>> ${DEPLOYDIR}/${DTB_NAME}.dtb >>> + cd ${DEPLOYDIR} >>> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb >>> + cd - >>> + done >>> + fi >>> +} >> >> >> Merged into OE-Core >> >> Thanks >> Sau! >> >> >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core -- Scott Garman Embedded Linux Engineer - Yocto Project Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 19:59 ` Scott Garman @ 2012-07-05 20:52 ` McClintock Matthew-B29882 2012-07-05 21:33 ` Scott Garman 0 siblings, 1 reply; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-05 20:52 UTC (permalink / raw) To: Patches and discussions about the oe-core layer, Scott Garman, Khem Raj On Thu, Jul 5, 2012 at 2:59 PM, Scott Garman <scott.a.garman@intel.com> wrote: > On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: >> >> Can we add this to denzil branch? (post 1.2.1 is fine) - I realize >> this is somewhat a new feature though but it would be helpful to have >> it in denzil. > > > Ok, I've now created a contrib branch to start tracking changes for 1.2.2, > and have included this commit: > > http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 > > The usual caveats apply - I can rebase this branch at any time, etc. Are you missing a patch on this series? ChecksumError class does not exist... commit 2f009e5b? -M > > Scott > > >> >> -M >> >> On Thu, Jun 28, 2012 at 12:12 PM, Saul Wold <sgw@linux.intel.com> wrote: >>> >>> On 06/25/2012 08:53 PM, b19537@freescale.com wrote: >>>> >>>> >>>> From: Zhenhua Luo<b19537@freescale.com> >>>> >>>> including following enhancement: >>>> * support multi-dtb build >>>> * skip dtb build and install when KERNEL_DEVICETREE is empty >>>> * print a warning message when specified dts file is not available >>>> >>>> Signed-off-by: Zhenhua Luo<b19537@freescale.com> >>>> --- >>>> meta/recipes-kernel/linux/linux-dtb.inc | 36 >>>> +++++++++++++++++++++++++------ >>>> 1 files changed, 29 insertions(+), 7 deletions(-) >>>> >>>> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc >>>> b/meta/recipes-kernel/linux/linux-dtb.inc >>>> index 9188cee..8aa1458 100644 >>>> --- a/meta/recipes-kernel/linux/linux-dtb.inc >>>> +++ b/meta/recipes-kernel/linux/linux-dtb.inc >>>> @@ -15,13 +15,35 @@ python __anonymous () { >>>> >>>> do_install_append() { >>>> if test -n "${KERNEL_DEVICETREE}"; then >>>> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree >>>> ${KERNEL_DEVICETREE} >>>> - install -m 0644 devicetree >>>> ${D}/boot/devicetree-${KERNEL_VERSION} >>>> - install -d ${DEPLOYDIR} >>>> - install -m 0644 devicetree >>>> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb >>>> - cd ${DEPLOYDIR} >>>> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>>> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb >>>> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>>> + if [ ! -f ${DTS_FILE} ]; then >>>> + echo "Warning: ${DTS_FILE} is not available!" >>>> + continue >>>> + fi >>>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>>> $1}'` >>>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>>> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o >>>> ${DTS_BASE_NAME} ${DTS_FILE} >>>> + install -m 0644 ${DTS_BASE_NAME} >>>> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb >>>> + done >>>> fi >>>> } >>>> >>>> +do_deploy_append() { >>>> + if test -n "${KERNEL_DEVICETREE}"; then >>>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>>> + if [ ! -f ${DTS_FILE} ]; then >>>> + echo "Warning: ${DTS_FILE} is not available!" >>>> + continue >>>> + fi >>>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>>> $1}'` >>>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>>> + install -d ${DEPLOYDIR} >>>> + install -m 0644 ${B}/${DTS_BASE_NAME} >>>> ${DEPLOYDIR}/${DTB_NAME}.dtb >>>> + cd ${DEPLOYDIR} >>>> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb >>>> + cd - >>>> + done >>>> + fi >>>> +} >>> >>> >>> >>> Merged into OE-Core >>> >>> Thanks >>> Sau! >>> >>> >>> _______________________________________________ >>> Openembedded-core mailing list >>> Openembedded-core@lists.openembedded.org >>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core > > > > -- > Scott Garman > Embedded Linux Engineer - Yocto Project > Intel Open Source Technology Center > > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 20:52 ` McClintock Matthew-B29882 @ 2012-07-05 21:33 ` Scott Garman 2012-07-05 21:37 ` McClintock Matthew-B29882 0 siblings, 1 reply; 14+ messages in thread From: Scott Garman @ 2012-07-05 21:33 UTC (permalink / raw) To: openembedded-core On 07/05/2012 01:52 PM, McClintock Matthew-B29882 wrote: > On Thu, Jul 5, 2012 at 2:59 PM, Scott Garman <scott.a.garman@intel.com> wrote: >> On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: >>> >>> Can we add this to denzil branch? (post 1.2.1 is fine) - I realize >>> this is somewhat a new feature though but it would be helpful to have >>> it in denzil. >> >> >> Ok, I've now created a contrib branch to start tracking changes for 1.2.2, >> and have included this commit: >> >> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 >> >> The usual caveats apply - I can rebase this branch at any time, etc. > > Are you missing a patch on this series? ChecksumError class does not exist... > > commit 2f009e5b? What is the title of that commit? I'm not seeing it. Thanks, Scott -- Scott Garman Embedded Linux Engineer - Yocto Project Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 21:33 ` Scott Garman @ 2012-07-05 21:37 ` McClintock Matthew-B29882 2012-07-05 22:07 ` Scott Garman 0 siblings, 1 reply; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-05 21:37 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On Thu, Jul 5, 2012 at 4:33 PM, Scott Garman <scott.a.garman@intel.com> wrote: > On 07/05/2012 01:52 PM, McClintock Matthew-B29882 wrote: >> >> On Thu, Jul 5, 2012 at 2:59 PM, Scott Garman <scott.a.garman@intel.com> >> wrote: >>> >>> On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: >>>> >>>> >>>> Can we add this to denzil branch? (post 1.2.1 is fine) - I realize >>>> this is somewhat a new feature though but it would be helpful to have >>>> it in denzil. >>> >>> >>> >>> Ok, I've now created a contrib branch to start tracking changes for >>> 1.2.2, >>> and have included this commit: >>> >>> >>> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 >>> >>> The usual caveats apply - I can rebase this branch at any time, etc. >> >> >> Are you missing a patch on this series? ChecksumError class does not >> exist... >> >> commit 2f009e5b? > > > What is the title of that commit? I'm not seeing it. $ git show 2f009e5b commit 2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e Author: Richard Purdie <richard.purdie@linuxfoundation.org> Date: Sun May 13 12:00:45 2012 +0000 bitbake/fetch2: Improve visibility of checksum warnings when fetching from mirrors When fetching from mirrors, checksum errors would get buried in the logs. This raises their profile so a warning is logged on the console when fetcher checksum issues are encountered, even if other attempts are made to get the file (which may or may not have the same issue). (Bitbake rev: d43fafd7f01b5534499b45213197d8ccececdbc4) Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> -M > > Thanks, > > Scott > > > -- > Scott Garman > Embedded Linux Engineer - Yocto Project > Intel Open Source Technology Center > > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 21:37 ` McClintock Matthew-B29882 @ 2012-07-05 22:07 ` Scott Garman 2012-07-20 14:43 ` McClintock Matthew-B29882 0 siblings, 1 reply; 14+ messages in thread From: Scott Garman @ 2012-07-05 22:07 UTC (permalink / raw) To: openembedded-core On 07/05/2012 02:37 PM, McClintock Matthew-B29882 wrote: > On Thu, Jul 5, 2012 at 4:33 PM, Scott Garman <scott.a.garman@intel.com> wrote: >> On 07/05/2012 01:52 PM, McClintock Matthew-B29882 wrote: >>> >>> On Thu, Jul 5, 2012 at 2:59 PM, Scott Garman <scott.a.garman@intel.com> >>> wrote: >>>> >>>> On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: >>>>> >>>>> >>>>> Can we add this to denzil branch? (post 1.2.1 is fine) - I realize >>>>> this is somewhat a new feature though but it would be helpful to have >>>>> it in denzil. >>>> >>>> >>>> >>>> Ok, I've now created a contrib branch to start tracking changes for >>>> 1.2.2, >>>> and have included this commit: >>>> >>>> >>>> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 >>>> >>>> The usual caveats apply - I can rebase this branch at any time, etc. >>> >>> >>> Are you missing a patch on this series? ChecksumError class does not >>> exist... >>> >>> commit 2f009e5b? >> >> >> What is the title of that commit? I'm not seeing it. > > $ git show 2f009e5b > commit 2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e > Author: Richard Purdie <richard.purdie@linuxfoundation.org> > Date: Sun May 13 12:00:45 2012 +0000 > > bitbake/fetch2: Improve visibility of checksum warnings when > fetching from mirrors > > When fetching from mirrors, checksum errors would get buried in the > logs. This raises their profile so a warning is logged on the console > when fetcher checksum issues are encountered, even if other attempts > are made to get the file (which may or may not have the same issue). > > (Bitbake rev: d43fafd7f01b5534499b45213197d8ccececdbc4) > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> Ok, this applies then to the poky-based tree, not the oe-core one. I just tried merging this in but I get conflicts with one of Khem's patches. I'm leaving imminently for vacation so I will take a closer look at this early next week. Scott -- Scott Garman Embedded Linux Engineer - Yocto Project Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-05 22:07 ` Scott Garman @ 2012-07-20 14:43 ` McClintock Matthew-B29882 0 siblings, 0 replies; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-20 14:43 UTC (permalink / raw) To: Patches and discussions about the oe-core layer, Scott Garman, Khem Raj, Liu Ting-B28495 On Thu, Jul 5, 2012 at 5:07 PM, Scott Garman <scott.a.garman@intel.com> wrote: > On 07/05/2012 02:37 PM, McClintock Matthew-B29882 wrote: >> >> On Thu, Jul 5, 2012 at 4:33 PM, Scott Garman <scott.a.garman@intel.com> >> wrote: >>> >>> On 07/05/2012 01:52 PM, McClintock Matthew-B29882 wrote: >>>> >>>> >>>> On Thu, Jul 5, 2012 at 2:59 PM, Scott Garman <scott.a.garman@intel.com> >>>> wrote: >>>>> >>>>> >>>>> On 07/05/2012 11:29 AM, McClintock Matthew-B29882 wrote: >>>>>> >>>>>> >>>>>> >>>>>> Can we add this to denzil branch? (post 1.2.1 is fine) - I realize >>>>>> this is somewhat a new feature though but it would be helpful to have >>>>>> it in denzil. >>>>> >>>>> >>>>> >>>>> >>>>> Ok, I've now created a contrib branch to start tracking changes for >>>>> 1.2.2, >>>>> and have included this commit: >>>>> >>>>> >>>>> >>>>> http://git.openembedded.org/openembedded-core-contrib/log/?h=sgarman/denzil-next-1.2.2 >>>>> >>>>> The usual caveats apply - I can rebase this branch at any time, etc. >>>> >>>> >>>> >>>> Are you missing a patch on this series? ChecksumError class does not >>>> exist... >>>> >>>> commit 2f009e5b? >>> >>> >>> >>> What is the title of that commit? I'm not seeing it. >> >> >> $ git show 2f009e5b >> commit 2f009e5b1e4e0a4148ab7d5c60bf0e7a193faa2e >> Author: Richard Purdie <richard.purdie@linuxfoundation.org> >> Date: Sun May 13 12:00:45 2012 +0000 >> >> bitbake/fetch2: Improve visibility of checksum warnings when >> fetching from mirrors >> >> When fetching from mirrors, checksum errors would get buried in the >> logs. This raises their profile so a warning is logged on the console >> when fetcher checksum issues are encountered, even if other attempts >> are made to get the file (which may or may not have the same issue). >> >> (Bitbake rev: d43fafd7f01b5534499b45213197d8ccececdbc4) >> >> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> > > > Ok, this applies then to the poky-based tree, not the oe-core one. > > I just tried merging this in but I get conflicts with one of Khem's patches. > I'm leaving imminently for vacation so I will take a closer look at this > early next week. Scott, Any update on the missing ChecksumError class? -M > > > Scott > > -- > Scott Garman > Embedded Linux Engineer - Yocto Project > Intel Open Source Technology Center > > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-06-26 3:53 linux-dtb: add multi-dtb build support b19537 2012-06-28 17:12 ` Saul Wold @ 2012-07-16 22:48 ` Saul Wold 2012-07-16 22:50 ` McClintock Matthew-B29882 1 sibling, 1 reply; 14+ messages in thread From: Saul Wold @ 2012-07-16 22:48 UTC (permalink / raw) To: Patches and discussions about the oe-core layer On 06/25/2012 08:53 PM, b19537@freescale.com wrote: > From: Zhenhua Luo <b19537@freescale.com> > > including following enhancement: > * support multi-dtb build > * skip dtb build and install when KERNEL_DEVICETREE is empty > * print a warning message when specified dts file is not available > > Signed-off-by: Zhenhua Luo <b19537@freescale.com> > --- > meta/recipes-kernel/linux/linux-dtb.inc | 36 +++++++++++++++++++++++++------ > 1 files changed, 29 insertions(+), 7 deletions(-) > This patch appears to rename the .dtb file different than it did before this change. Is that correct? Seem to change it from: uImage-mpc8315e-rdb.dtb to uImage-mpc8315erdb.dtb While the .bin file is still: uImage-mpc8315e-rdb.bin Inquiring minds?? This was filed as bug 2769 https://bugzilla.yoctoproject.org/show_bug.cgi?id=2769 Thanks Sau! > diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc > index 9188cee..8aa1458 100644 > --- a/meta/recipes-kernel/linux/linux-dtb.inc > +++ b/meta/recipes-kernel/linux/linux-dtb.inc > @@ -15,13 +15,35 @@ python __anonymous () { > > do_install_append() { > if test -n "${KERNEL_DEVICETREE}"; then > - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree ${KERNEL_DEVICETREE} > - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} > - install -d ${DEPLOYDIR} > - install -m 0644 devicetree ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb > - cd ${DEPLOYDIR} > - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > + for DTS_FILE in ${KERNEL_DEVICETREE}; do > + if [ ! -f ${DTS_FILE} ]; then > + echo "Warning: ${DTS_FILE} is not available!" > + continue > + fi > + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'` > + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o ${DTS_BASE_NAME} ${DTS_FILE} > + install -m 0644 ${DTS_BASE_NAME} ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb > + done > fi > } > > +do_deploy_append() { > + if test -n "${KERNEL_DEVICETREE}"; then > + for DTS_FILE in ${KERNEL_DEVICETREE}; do > + if [ ! -f ${DTS_FILE} ]; then > + echo "Warning: ${DTS_FILE} is not available!" > + continue > + fi > + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print $1}'` > + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed "s/${MACHINE}/${DTS_BASE_NAME}/g"` > + install -d ${DEPLOYDIR} > + install -m 0644 ${B}/${DTS_BASE_NAME} ${DEPLOYDIR}/${DTB_NAME}.dtb > + cd ${DEPLOYDIR} > + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb > + cd - > + done > + fi > +} > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-16 22:48 ` Saul Wold @ 2012-07-16 22:50 ` McClintock Matthew-B29882 2012-07-20 14:42 ` McClintock Matthew-B29882 0 siblings, 1 reply; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-16 22:50 UTC (permalink / raw) To: Patches and discussions about the oe-core layer, Luo Zhenhua-B19537 Adding Zhenhua so he sees the email for sure ;) -M On Mon, Jul 16, 2012 at 5:48 PM, Saul Wold <sgw@linux.intel.com> wrote: > On 06/25/2012 08:53 PM, b19537@freescale.com wrote: >> >> From: Zhenhua Luo <b19537@freescale.com> >> >> including following enhancement: >> * support multi-dtb build >> * skip dtb build and install when KERNEL_DEVICETREE is empty >> * print a warning message when specified dts file is not available >> >> Signed-off-by: Zhenhua Luo <b19537@freescale.com> >> --- >> meta/recipes-kernel/linux/linux-dtb.inc | 36 >> +++++++++++++++++++++++++------ >> 1 files changed, 29 insertions(+), 7 deletions(-) >> > This patch appears to rename the .dtb file different than it did before this > change. Is that correct? > > Seem to change it from: > > uImage-mpc8315e-rdb.dtb > > to > > uImage-mpc8315erdb.dtb > > While the .bin file is still: uImage-mpc8315e-rdb.bin > > Inquiring minds?? This was filed as bug 2769 > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=2769 > > Thanks > Sau! > > > >> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc >> b/meta/recipes-kernel/linux/linux-dtb.inc >> index 9188cee..8aa1458 100644 >> --- a/meta/recipes-kernel/linux/linux-dtb.inc >> +++ b/meta/recipes-kernel/linux/linux-dtb.inc >> @@ -15,13 +15,35 @@ python __anonymous () { >> >> do_install_append() { >> if test -n "${KERNEL_DEVICETREE}"; then >> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree >> ${KERNEL_DEVICETREE} >> - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} >> - install -d ${DEPLOYDIR} >> - install -m 0644 devicetree >> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb >> - cd ${DEPLOYDIR} >> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb >> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >> + if [ ! -f ${DTS_FILE} ]; then >> + echo "Warning: ${DTS_FILE} is not available!" >> + continue >> + fi >> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >> $1}'` >> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o >> ${DTS_BASE_NAME} ${DTS_FILE} >> + install -m 0644 ${DTS_BASE_NAME} >> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb >> + done >> fi >> } >> >> +do_deploy_append() { >> + if test -n "${KERNEL_DEVICETREE}"; then >> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >> + if [ ! -f ${DTS_FILE} ]; then >> + echo "Warning: ${DTS_FILE} is not available!" >> + continue >> + fi >> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >> $1}'` >> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >> + install -d ${DEPLOYDIR} >> + install -m 0644 ${B}/${DTS_BASE_NAME} >> ${DEPLOYDIR}/${DTB_NAME}.dtb >> + cd ${DEPLOYDIR} >> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb >> + cd - >> + done >> + fi >> +} >> > > > _______________________________________________ > Openembedded-core mailing list > Openembedded-core@lists.openembedded.org > http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-16 22:50 ` McClintock Matthew-B29882 @ 2012-07-20 14:42 ` McClintock Matthew-B29882 2012-08-20 6:10 ` Luo Zhenhua-B19537 0 siblings, 1 reply; 14+ messages in thread From: McClintock Matthew-B29882 @ 2012-07-20 14:42 UTC (permalink / raw) To: Patches and discussions about the oe-core layer, Luo Zhenhua-B19537 Zhenhua, Any update here? -M On Mon, Jul 16, 2012 at 5:50 PM, Matthew McClintock <msm@freescale.com> wrote: > Adding Zhenhua so he sees the email for sure ;) > > -M > > On Mon, Jul 16, 2012 at 5:48 PM, Saul Wold <sgw@linux.intel.com> wrote: >> On 06/25/2012 08:53 PM, b19537@freescale.com wrote: >>> >>> From: Zhenhua Luo <b19537@freescale.com> >>> >>> including following enhancement: >>> * support multi-dtb build >>> * skip dtb build and install when KERNEL_DEVICETREE is empty >>> * print a warning message when specified dts file is not available >>> >>> Signed-off-by: Zhenhua Luo <b19537@freescale.com> >>> --- >>> meta/recipes-kernel/linux/linux-dtb.inc | 36 >>> +++++++++++++++++++++++++------ >>> 1 files changed, 29 insertions(+), 7 deletions(-) >>> >> This patch appears to rename the .dtb file different than it did before this >> change. Is that correct? >> >> Seem to change it from: >> >> uImage-mpc8315e-rdb.dtb >> >> to >> >> uImage-mpc8315erdb.dtb >> >> While the .bin file is still: uImage-mpc8315e-rdb.bin >> >> Inquiring minds?? This was filed as bug 2769 >> >> https://bugzilla.yoctoproject.org/show_bug.cgi?id=2769 >> >> Thanks >> Sau! >> >> >> >>> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc >>> b/meta/recipes-kernel/linux/linux-dtb.inc >>> index 9188cee..8aa1458 100644 >>> --- a/meta/recipes-kernel/linux/linux-dtb.inc >>> +++ b/meta/recipes-kernel/linux/linux-dtb.inc >>> @@ -15,13 +15,35 @@ python __anonymous () { >>> >>> do_install_append() { >>> if test -n "${KERNEL_DEVICETREE}"; then >>> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree >>> ${KERNEL_DEVICETREE} >>> - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} >>> - install -d ${DEPLOYDIR} >>> - install -m 0644 devicetree >>> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb >>> - cd ${DEPLOYDIR} >>> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb >>> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>> + if [ ! -f ${DTS_FILE} ]; then >>> + echo "Warning: ${DTS_FILE} is not available!" >>> + continue >>> + fi >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>> $1}'` >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o >>> ${DTS_BASE_NAME} ${DTS_FILE} >>> + install -m 0644 ${DTS_BASE_NAME} >>> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb >>> + done >>> fi >>> } >>> >>> +do_deploy_append() { >>> + if test -n "${KERNEL_DEVICETREE}"; then >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do >>> + if [ ! -f ${DTS_FILE} ]; then >>> + echo "Warning: ${DTS_FILE} is not available!" >>> + continue >>> + fi >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." '{print >>> $1}'` >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` >>> + install -d ${DEPLOYDIR} >>> + install -m 0644 ${B}/${DTS_BASE_NAME} >>> ${DEPLOYDIR}/${DTB_NAME}.dtb >>> + cd ${DEPLOYDIR} >>> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb >>> + cd - >>> + done >>> + fi >>> +} >>> >> >> >> _______________________________________________ >> Openembedded-core mailing list >> Openembedded-core@lists.openembedded.org >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: linux-dtb: add multi-dtb build support 2012-07-20 14:42 ` McClintock Matthew-B29882 @ 2012-08-20 6:10 ` Luo Zhenhua-B19537 0 siblings, 0 replies; 14+ messages in thread From: Luo Zhenhua-B19537 @ 2012-08-20 6:10 UTC (permalink / raw) To: McClintock Matthew-B29882, Patches and discussions about the oe-core layer Sorry for the late response. I think this issue(2769) has been closed. Doesn't it make sense to generate the dtb with the same name with corresponding dts? Best Regards, Zhenhua > -----Original Message----- > From: McClintock Matthew-B29882 > Sent: Friday, July 20, 2012 10:42 PM > To: Patches and discussions about the oe-core layer; Luo Zhenhua-B19537 > Subject: Re: [OE-core] [oe-core] linux-dtb: add multi-dtb build support > > Zhenhua, > > Any update here? > > -M > > On Mon, Jul 16, 2012 at 5:50 PM, Matthew McClintock <msm@freescale.com> wrote: > > Adding Zhenhua so he sees the email for sure ;) > > > > -M > > > > On Mon, Jul 16, 2012 at 5:48 PM, Saul Wold <sgw@linux.intel.com> wrote: > >> On 06/25/2012 08:53 PM, b19537@freescale.com wrote: > >>> > >>> From: Zhenhua Luo <b19537@freescale.com> > >>> > >>> including following enhancement: > >>> * support multi-dtb build > >>> * skip dtb build and install when KERNEL_DEVICETREE is empty > >>> * print a warning message when specified dts file is not > >>> available > >>> > >>> Signed-off-by: Zhenhua Luo <b19537@freescale.com> > >>> --- > >>> meta/recipes-kernel/linux/linux-dtb.inc | 36 > >>> +++++++++++++++++++++++++------ > >>> 1 files changed, 29 insertions(+), 7 deletions(-) > >>> > >> This patch appears to rename the .dtb file different than it did > >> before this change. Is that correct? > >> > >> Seem to change it from: > >> > >> uImage-mpc8315e-rdb.dtb > >> > >> to > >> > >> uImage-mpc8315erdb.dtb > >> > >> While the .bin file is still: uImage-mpc8315e-rdb.bin > >> > >> Inquiring minds?? This was filed as bug 2769 > >> > >> https://bugzilla.yoctoproject.org/show_bug.cgi?id=2769 > >> > >> Thanks > >> Sau! > >> > >> > >> > >>> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc > >>> b/meta/recipes-kernel/linux/linux-dtb.inc > >>> index 9188cee..8aa1458 100644 > >>> --- a/meta/recipes-kernel/linux/linux-dtb.inc > >>> +++ b/meta/recipes-kernel/linux/linux-dtb.inc > >>> @@ -15,13 +15,35 @@ python __anonymous () { > >>> > >>> do_install_append() { > >>> if test -n "${KERNEL_DEVICETREE}"; then > >>> - dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o devicetree > >>> ${KERNEL_DEVICETREE} > >>> - install -m 0644 devicetree ${D}/boot/devicetree-${KERNEL_VERSION} > >>> - install -d ${DEPLOYDIR} > >>> - install -m 0644 devicetree > >>> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb > >>> - cd ${DEPLOYDIR} > >>> - rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > >>> - ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb > >>> ${KERNEL_IMAGE_SYMLINK_NAME}.dtb > >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do > >>> + if [ ! -f ${DTS_FILE} ]; then > >>> + echo "Warning: ${DTS_FILE} is not available!" > >>> + continue > >>> + fi > >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." > >>> + '{print > >>> $1}'` > >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed > >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` > >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | > >>> + sed > >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` > >>> + dtc -I dts -O dtb ${KERNEL_DEVICETREE_FLAGS} -o > >>> ${DTS_BASE_NAME} ${DTS_FILE} > >>> + install -m 0644 ${DTS_BASE_NAME} > >>> ${D}/boot/devicetree-${DTB_SYMLINK_NAME}.dtb > >>> + done > >>> fi > >>> } > >>> > >>> +do_deploy_append() { > >>> + if test -n "${KERNEL_DEVICETREE}"; then > >>> + for DTS_FILE in ${KERNEL_DEVICETREE}; do > >>> + if [ ! -f ${DTS_FILE} ]; then > >>> + echo "Warning: ${DTS_FILE} is not available!" > >>> + continue > >>> + fi > >>> + DTS_BASE_NAME=`basename ${DTS_FILE} | awk -F "." > >>> +'{print > >>> $1}'` > >>> + DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed > >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` > >>> + DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | > >>> + sed > >>> "s/${MACHINE}/${DTS_BASE_NAME}/g"` > >>> + install -d ${DEPLOYDIR} > >>> + install -m 0644 ${B}/${DTS_BASE_NAME} > >>> ${DEPLOYDIR}/${DTB_NAME}.dtb > >>> + cd ${DEPLOYDIR} > >>> + ln -sf ${DTB_NAME}.dtb ${DTB_SYMLINK_NAME}.dtb > >>> + cd - > >>> + done > >>> + fi > >>> +} > >>> > >> > >> > >> _______________________________________________ > >> Openembedded-core mailing list > >> Openembedded-core@lists.openembedded.org > >> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2012-08-20 7:07 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-06-26 3:53 linux-dtb: add multi-dtb build support b19537 2012-06-28 17:12 ` Saul Wold 2012-07-05 18:29 ` McClintock Matthew-B29882 2012-07-05 19:38 ` Scott Garman 2012-07-05 19:59 ` Scott Garman 2012-07-05 20:52 ` McClintock Matthew-B29882 2012-07-05 21:33 ` Scott Garman 2012-07-05 21:37 ` McClintock Matthew-B29882 2012-07-05 22:07 ` Scott Garman 2012-07-20 14:43 ` McClintock Matthew-B29882 2012-07-16 22:48 ` Saul Wold 2012-07-16 22:50 ` McClintock Matthew-B29882 2012-07-20 14:42 ` McClintock Matthew-B29882 2012-08-20 6:10 ` Luo Zhenhua-B19537
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox