Patchwork linux-dtb: add dtb into deploy cache package

login
register
mail settings
Submitter Zhenhua Luo
Date Dec. 22, 2011, 10:23 a.m.
Message ID <1324549428-22664-1-git-send-email-b19537@freescale.com>
Download mbox | patch
Permalink /patch/17457/
State Superseded
Headers show

Comments

Zhenhua Luo - Dec. 22, 2011, 10:23 a.m.
From: Zhenhua Luo <b19537@freescale.com>

    > current dtb is not included in the deploy cache package, dtb is missing
      when build kernel from caches

Signed-off-by: Zhenhua Luo <b19537@freescale.com>
---
 meta/recipes-kernel/linux/linux-dtb.inc |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
Saul Wold - Dec. 22, 2011, 9:55 p.m.
On 12/22/2011 02:23 AM, b19537@freescale.com wrote:
> From: Zhenhua Luo<b19537@freescale.com>
>
>      >  current dtb is not included in the deploy cache package, dtb is missing
>        when build kernel from caches
>
> Signed-off-by: Zhenhua Luo<b19537@freescale.com>
> ---
>   meta/recipes-kernel/linux/linux-dtb.inc |    6 ++++++
>   1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc
> index 7ec7584..f6f5f81 100644
> --- a/meta/recipes-kernel/linux/linux-dtb.inc
> +++ b/meta/recipes-kernel/linux/linux-dtb.inc
> @@ -25,3 +25,9 @@ do_install_append() {
>       fi
>   }
>
> +do_deploy_append() {
> +    install -d ${DEPLOYDIR}
> +    install -m 0644 ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGE_BASE_NAME}.dtb ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb
> +    cd ${DEPLOYDIR}
> +    ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb ${KERNEL_IMAGE_SYMLINK_NAME}.dtb
> +}
What machines did you test this on?  Is it possible that some kernels do 
not generate .dtb files?

Sau!
Zhenhua Luo - Dec. 23, 2011, 2:52 a.m.
Hello Saul, 

> -----Original Message-----
> From: Saul Wold [mailto:sgw@linux.intel.com]
> Sent: Friday, December 23, 2011 5:56 AM
> 
> > +do_deploy_append() {
> > +    install -d ${DEPLOYDIR}
> > +    install -m 0644 ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGE_BASE_NAME}.dtb
> ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb
> > +    cd ${DEPLOYDIR}
> > +    ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb
> > +${KERNEL_IMAGE_SYMLINK_NAME}.dtb }
> What machines did you test this on?  Is it possible that some kernels do not
> generate .dtb files?
[Luo Zhenhua-B19537] I verified on Freescale powerpc platforms, this will cause error on targets which don't need dtb. I will post a new patch to fix the issue.  


Best Regards,

Zhenhua

Patch

diff --git a/meta/recipes-kernel/linux/linux-dtb.inc b/meta/recipes-kernel/linux/linux-dtb.inc
index 7ec7584..f6f5f81 100644
--- a/meta/recipes-kernel/linux/linux-dtb.inc
+++ b/meta/recipes-kernel/linux/linux-dtb.inc
@@ -25,3 +25,9 @@  do_install_append() {
     fi
 }
 
+do_deploy_append() {
+    install -d ${DEPLOYDIR}
+    install -m 0644 ${DEPLOY_DIR_IMAGE}/${KERNEL_IMAGE_BASE_NAME}.dtb ${DEPLOYDIR}/${KERNEL_IMAGE_BASE_NAME}.dtb
+    cd ${DEPLOYDIR}
+    ln -sf ${KERNEL_IMAGE_BASE_NAME}.dtb ${KERNEL_IMAGE_SYMLINK_NAME}.dtb
+}