Merge lp:~linaro-foundations/linaro-ci/add-device-tree-build-support into lp:linaro-ci

Proposed by Avik Sil
Status: Rejected
Rejected by: Данило Шеган
Proposed branch: lp:~linaro-foundations/linaro-ci/add-device-tree-build-support
Merge into: lp:linaro-ci
Diff against target: 11 lines (+1/-0)
1 file modified
jenkins_common_lib (+1/-0)
To merge this branch: bzr merge lp:~linaro-foundations/linaro-ci/add-device-tree-build-support
Reviewer Review Type Date Requested Status
Данило Шеган (community) Needs Information
Deepti B. Kalakeri Pending
Review via email: mp+103048@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Deepti B. Kalakeri (deeptik) wrote :

On Mon, Apr 23, 2012 at 12:09 PM, Avik Sil <email address hidden> wrote:

> Avik Sil has proposed merging
> lp:~linaro-foundations/linaro-ci/add-device-tree-build-support into
> lp:linaro-ci.
>
> Requested reviews:
> Deepti B. Kalakeri (deeptik)
>
> For more details, see:
>
> https://code.launchpad.net/~linaro-foundations/linaro-ci/add-device-tree-build-support/+merge/103048<https://code.launchpad.net/%7Elinaro-foundations/linaro-ci/add-device-tree-build-support/+merge/103048>
> --
>
> https://code.launchpad.net/~linaro-foundations/linaro-ci/add-device-tree-build-support/+merge/103048<https://code.launchpad.net/%7Elinaro-foundations/linaro-ci/add-device-tree-build-support/+merge/103048>
> You are requested to review the proposed merge of
> lp:~linaro-foundations/linaro-ci/add-device-tree-build-support into
> lp:linaro-ci.
>
> === modified file 'jenkins_common_lib'
> --- jenkins_common_lib 2012-03-26 20:41:50 +0000
> +++ jenkins_common_lib 2012-04-23 06:38:20 +0000
> @@ -60,6 +60,7 @@
> yes "" | make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version"
> KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- oldconfig
> make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version"
> KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count
> uImage
> make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version"
> KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count
> modules
> + make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version"
> KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count
> dtbs
>

This is a universal script used for building all the trees. Hence, this
would fail the other builds which use this scripts if dt changes are not in
the tree.
Do you know if the dt can be built for arm-soc, linus, linux-next in the
same manner ?
You should probable enable building the device tree when for example
dt_build = 1.
You should refer to submit_job flag as to how to modify for such a flag

    make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version"
> KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi-
> KBUILD_DEBARCH=armel V=1 deb-pkg
> END=$(date +%s)
> EXECUTION_TIME_IN_SEC=$(( $END - $START ))
>
>
>

--
Thanks and Regards,
Deepti

Revision history for this message
Ricardo Salveti (rsalveti) wrote :

I don't think that will break the other builds, as make dtbs is available since 3.0. At the case where we don't have device tree support, it'll just skill as it'll not find any target.

Revision history for this message
Deepti B. Kalakeri (deeptik) wrote :

Avik, last when you mentioned there was no support from lava for testing dt.
Is this the same now ?

Revision history for this message
Данило Шеган (danilo) wrote :

This has seen no movement for a full month, so I am rejecting it. If it is still relevant, please reopen it and provide the information Deepti has asked for.

Revision history for this message
Данило Шеган (danilo) :
review: Needs Information

Unmerged revisions

60. By Avik Sil

Add device tree build support

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'jenkins_common_lib'
2--- jenkins_common_lib 2012-03-26 20:41:50 +0000
3+++ jenkins_common_lib 2012-04-23 06:38:20 +0000
4@@ -60,6 +60,7 @@
5 yes "" | make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version" KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- oldconfig
6 make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version" KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count uImage
7 make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version" KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count modules
8+ make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version" KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- -j$j_count dtbs
9 make ARCH=arm O=$pkg_dir KERNELVERSION="$kernel_version" KERNELRELEASE="$kernel_version" CROSS_COMPILE=arm-linux-gnueabi- KBUILD_DEBARCH=armel V=1 deb-pkg
10 END=$(date +%s)
11 EXECUTION_TIME_IN_SEC=$(( $END - $START ))

Subscribers

People subscribed via source and target branches