diff mbox series

[U-Boot] Travis-CI: Switch back to using the top of tree dtc

Message ID 1505402973-24263-1-git-send-email-trini@konsulko.com
State Accepted
Commit ae1c0a38c8b163d685121e3109161da7ac1caacd
Delegated to: Simon Glass
Headers show
Series [U-Boot] Travis-CI: Switch back to using the top of tree dtc | expand

Commit Message

Tom Rini Sept. 14, 2017, 3:29 p.m. UTC
In a0f3e3df4adc we switched to using the Ubuntu-provided dtc as travis
was having a problem with the number of warnings that were generated by
the newer dtc.  This is no longer a concern as we now have the same
logic as Linux to enable/disable additional more stringent warnings.  Go
back to building dtc from source.

Signed-off-by: Tom Rini <trini@konsulko.com>
---
 .travis.yml | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Simon Glass Sept. 15, 2017, 11:16 a.m. UTC | #1
On 14 September 2017 at 09:29, Tom Rini <trini@konsulko.com> wrote:
>
> In a0f3e3df4adc we switched to using the Ubuntu-provided dtc as travis
> was having a problem with the number of warnings that were generated by
> the newer dtc.  This is no longer a concern as we now have the same
> logic as Linux to enable/disable additional more stringent warnings.  Go
> back to building dtc from source.
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  .travis.yml | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Tested on travis-ci:
Tested-by: Simon Glass <sjg@chromium.org>
Simon Glass Sept. 15, 2017, 7:19 p.m. UTC | #2
On 14 September 2017 at 09:29, Tom Rini <trini@konsulko.com> wrote:
>
> In a0f3e3df4adc we switched to using the Ubuntu-provided dtc as travis
> was having a problem with the number of warnings that were generated by
> the newer dtc.  This is no longer a concern as we now have the same
> logic as Linux to enable/disable additional more stringent warnings.  Go
> back to building dtc from source.
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  .travis.yml | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Tested on travis-ci:
Tested-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-fdt thanks!
diff mbox series

Patch

diff --git a/.travis.yml b/.travis.yml
index 4b7f8dd8e4de..b81d733695e1 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -26,12 +26,11 @@  addons:
     - grub-efi-ia32-bin
     - rpm2cpio
     - wget
-    - device-tree-compiler
 
 install:
  # install latest device tree compiler
- #- git clone --depth=1 git://git.kernel.org/pub/scm/utils/dtc/dtc.git /tmp/dtc
- #- make -j4 -C /tmp/dtc
+ - git clone --depth=1 git://git.kernel.org/pub/scm/utils/dtc/dtc.git /tmp/dtc
+ - make -j4 -C /tmp/dtc
  # Clone uboot-test-hooks
  - git clone --depth=1 git://github.com/swarren/uboot-test-hooks.git /tmp/uboot-test-hooks
  - ln -s travis-ci /tmp/uboot-test-hooks/bin/`hostname`