The for loop already handles the case when KERNEL_DEVICETREE is empty.

Signed-off-by: Stefan Christ <[email protected]>
---
 meta/recipes-kernel/linux/linux-dtb.inc | 78 +++++++++++++++------------------
 1 file changed, 36 insertions(+), 42 deletions(-)

diff --git a/meta/recipes-kernel/linux/linux-dtb.inc 
b/meta/recipes-kernel/linux/linux-dtb.inc
index f44851a..5bfd33b 100644
--- a/meta/recipes-kernel/linux/linux-dtb.inc
+++ b/meta/recipes-kernel/linux/linux-dtb.inc
@@ -6,54 +6,48 @@ python __anonymous () {
 }
 
 do_compile_append() {
-       if test -n "${KERNEL_DEVICETREE}"; then
-               for DTB in ${KERNEL_DEVICETREE}; do
-                       if echo ${DTB} | grep -q '\.dts$'; then
-                               bbwarn "${DTB} contains the full path to the 
the dts file, but only the dtb name should be used."
-                               DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
-                       fi
-                       oe_runmake ${DTB}
-               done
-       fi
+       for DTB in ${KERNEL_DEVICETREE}; do
+               if echo ${DTB} | grep -q '\.dts$'; then
+                       bbwarn "${DTB} contains the full path to the the dts 
file, but only the dtb name should be used."
+                       DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
+               fi
+               oe_runmake ${DTB}
+       done
 }
 
 do_install_append() {
-       if test -n "${KERNEL_DEVICETREE}"; then
-               for DTB in ${KERNEL_DEVICETREE}; do
-                       if echo ${DTB} | grep -q '\.dts$'; then
-                               bbwarn "${DTB} contains the full path to the 
the dts file, but only the dtb name should be used."
-                               DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
-                       fi
-                       DTB_BASE_NAME=`basename ${DTB} .dtb`
-                       DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | 
sed "s/${MACHINE}/${DTB_BASE_NAME}/g"`
-                       DTB_PATH="${B}/arch/${ARCH}/boot/dts/${DTB}"
-                       if [ ! -e "${DTB_PATH}" ]; then
-                               DTB_PATH="${B}/arch/${ARCH}/boot/${DTB}"
-                       fi
-                       install -m 0644 ${DTB_PATH} 
${D}/${KERNEL_IMAGEDEST}/devicetree-${DTB_SYMLINK_NAME}.dtb
-               done
-       fi
+       for DTB in ${KERNEL_DEVICETREE}; do
+               if echo ${DTB} | grep -q '\.dts$'; then
+                       bbwarn "${DTB} contains the full path to the the dts 
file, but only the dtb name should be used."
+                       DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
+               fi
+               DTB_BASE_NAME=`basename ${DTB} .dtb`
+               DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed 
"s/${MACHINE}/${DTB_BASE_NAME}/g"`
+               DTB_PATH="${B}/arch/${ARCH}/boot/dts/${DTB}"
+               if [ ! -e "${DTB_PATH}" ]; then
+                       DTB_PATH="${B}/arch/${ARCH}/boot/${DTB}"
+               fi
+               install -m 0644 ${DTB_PATH} 
${D}/${KERNEL_IMAGEDEST}/devicetree-${DTB_SYMLINK_NAME}.dtb
+       done
 }
 
 do_deploy_append() {
-       if test -n "${KERNEL_DEVICETREE}"; then
-               for DTB in ${KERNEL_DEVICETREE}; do
-                       if echo ${DTB} | grep -q '\.dts$'; then
-                               bbwarn "${DTB} contains the full path to the 
the dts file, but only the dtb name should be used."
-                               DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
-                       fi
-                       DTB_BASE_NAME=`basename ${DTB} .dtb`
-                       DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed 
"s/${MACHINE}/${DTB_BASE_NAME}/g"`
-                       DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | 
sed "s/${MACHINE}/${DTB_BASE_NAME}/g"`
-                       DTB_PATH="${B}/arch/${ARCH}/boot/dts/${DTB}"
-                       if [ ! -e "${DTB_PATH}" ]; then
-                               DTB_PATH="${B}/arch/${ARCH}/boot/${DTB}"
-                       fi
-                       install -d ${DEPLOYDIR}
-                       install -m 0644 ${DTB_PATH} ${DEPLOYDIR}/${DTB_NAME}.dtb
-                       ln -sf ${DTB_NAME}.dtb 
${DEPLOYDIR}/${DTB_SYMLINK_NAME}.dtb
-               done
-       fi
+       for DTB in ${KERNEL_DEVICETREE}; do
+               if echo ${DTB} | grep -q '\.dts$'; then
+                       bbwarn "${DTB} contains the full path to the the dts 
file, but only the dtb name should be used."
+                       DTB=`basename ${DTB} | sed 's,\.dts$,.dtb,g'`
+               fi
+               DTB_BASE_NAME=`basename ${DTB} .dtb`
+               DTB_NAME=`echo ${KERNEL_IMAGE_BASE_NAME} | sed 
"s/${MACHINE}/${DTB_BASE_NAME}/g"`
+               DTB_SYMLINK_NAME=`echo ${KERNEL_IMAGE_SYMLINK_NAME} | sed 
"s/${MACHINE}/${DTB_BASE_NAME}/g"`
+               DTB_PATH="${B}/arch/${ARCH}/boot/dts/${DTB}"
+               if [ ! -e "${DTB_PATH}" ]; then
+                       DTB_PATH="${B}/arch/${ARCH}/boot/${DTB}"
+               fi
+               install -d ${DEPLOYDIR}
+               install -m 0644 ${DTB_PATH} ${DEPLOYDIR}/${DTB_NAME}.dtb
+               ln -sf ${DTB_NAME}.dtb ${DEPLOYDIR}/${DTB_SYMLINK_NAME}.dtb
+       done
 }
 
 pkg_postinst_kernel-devicetree () {
-- 
1.9.1

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to