* 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-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-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-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