From 72e1ad81152372f444133cca3686da1cf436ebbc Mon Sep 17 00:00:00 2001 From: David Gibson Date: Fri, 9 Dec 2016 13:35:34 +1100 Subject: tests: Make overlay/plugin tests unconditional When overlay apply supprt was added to libfdt the testcases included some which could only be executed with the (then) out of tree dtc with overlay output support. So, the test script automatically skipped those tests if it wasn't available. Now that the overlay support is merged into dtc mainline there's no reason to keep this logic. Instead run all the overlay tests unconditionally. Signed-off-by: David Gibson --- tests/run_tests.sh | 71 ++++++++++++++++++++++++++---------------------------- 1 file changed, 34 insertions(+), 37 deletions(-) diff --git a/tests/run_tests.sh b/tests/run_tests.sh index b8e7d44..a1baf7a 100755 --- a/tests/run_tests.sh +++ b/tests/run_tests.sh @@ -170,47 +170,44 @@ BAD_FIXUP_TREES="bad_index \ path_prop" overlay_tests () { - # Overlay tests that don't require overlay support in dtc + # Overlay tests for libfdt alone run_dtc_test -I dts -O dtb -o overlay_base_no_symbols.test.dtb overlay_base.dts run_dtc_test -I dts -O dtb -o overlay_overlay_no_symbols.test.dtb overlay_overlay_nodtc.dts run_test overlay overlay_base_no_symbols.test.dtb overlay_overlay_no_symbols.test.dtb - # Overlay tests that requires overlay support in dtc - echo "/dts-v1/; / {};" | $DTC -@ > /dev/null 2>&1 - if [ $? -eq 0 ]; then - run_dtc_test -@ -I dts -O dtb -o overlay_base_with_symbols.test.dtb overlay_base.dts - run_dtc_test -@ -I dts -O dtb -o overlay_overlay_with_symbols.test.dtb overlay_overlay_dtc.dts - run_test overlay overlay_base_with_symbols.test.dtb overlay_overlay_with_symbols.test.dtb - - # Test symbols/fixups existence - run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__symbols__" - run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__fixups__" - run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__local_fixups__" - - # test plugin source to dtb and back - run_dtc_test -@ -I dtb -O dts -o overlay_overlay_dtc.test.dts overlay_overlay_with_symbols.test.dtb - run_dtc_test -@ -I dts -O dtb -o overlay_overlay_with_symbols.test.test.dtb overlay_overlay_dtc.test.dts - run_test dtbs_equal_ordered overlay_overlay_with_symbols.test.dtb overlay_overlay_with_symbols.test.test.dtb - - # test plugin auto-generation without using -@ - run_dtc_test -I dts -O dtb -o overlay_overlay_with_symbols_auto.test.dtb overlay_overlay_dtc.dts - run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__symbols__" - run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__fixups__" - run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__local_fixups__" - - # Test suppression of fixups - run_dtc_test -@ -I dts -O dtb -o overlay_base_with_symbols_no_fixups.test.dtb overlay_base_fixups.dts - run_test check_path overlay_base_with_symbols_no_fixups.test.dtb exists "/__symbols__" - run_test check_path overlay_base_with_symbols_no_fixups.test.dtb not-exists "/__fixups__" - run_test check_path overlay_base_with_symbols_no_fixups.test.dtb not-exists "/__local_fixups__" - - # Test generation of aliases insted of symbols - run_dtc_test -A -I dts -O dtb -o overlay_overlay_with_aliases.dtb overlay_overlay_dtc.dts - run_test check_path overlay_overlay_with_aliases.dtb exists "/aliases" - run_test check_path overlay_overlay_with_aliases.dtb exists "/__symbols__" - run_test check_path overlay_overlay_with_aliases.dtb exists "/__fixups__" - run_test check_path overlay_overlay_with_aliases.dtb exists "/__local_fixups__" - fi + # Overlay tests for dtc + run_dtc_test -@ -I dts -O dtb -o overlay_base_with_symbols.test.dtb overlay_base.dts + run_dtc_test -@ -I dts -O dtb -o overlay_overlay_with_symbols.test.dtb overlay_overlay_dtc.dts + run_test overlay overlay_base_with_symbols.test.dtb overlay_overlay_with_symbols.test.dtb + + # Test symbols/fixups existence + run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__symbols__" + run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__fixups__" + run_test check_path overlay_overlay_with_symbols.test.dtb exists "/__local_fixups__" + + # test plugin source to dtb and back + run_dtc_test -@ -I dtb -O dts -o overlay_overlay_dtc.test.dts overlay_overlay_with_symbols.test.dtb + run_dtc_test -@ -I dts -O dtb -o overlay_overlay_with_symbols.test.test.dtb overlay_overlay_dtc.test.dts + run_test dtbs_equal_ordered overlay_overlay_with_symbols.test.dtb overlay_overlay_with_symbols.test.test.dtb + + # test plugin auto-generation without using -@ + run_dtc_test -I dts -O dtb -o overlay_overlay_with_symbols_auto.test.dtb overlay_overlay_dtc.dts + run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__symbols__" + run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__fixups__" + run_test check_path overlay_overlay_with_symbols_auto.test.dtb exists "/__local_fixups__" + + # Test suppression of fixups + run_dtc_test -@ -I dts -O dtb -o overlay_base_with_symbols_no_fixups.test.dtb overlay_base_fixups.dts + run_test check_path overlay_base_with_symbols_no_fixups.test.dtb exists "/__symbols__" + run_test check_path overlay_base_with_symbols_no_fixups.test.dtb not-exists "/__fixups__" + run_test check_path overlay_base_with_symbols_no_fixups.test.dtb not-exists "/__local_fixups__" + + # Test generation of aliases insted of symbols + run_dtc_test -A -I dts -O dtb -o overlay_overlay_with_aliases.dtb overlay_overlay_dtc.dts + run_test check_path overlay_overlay_with_aliases.dtb exists "/aliases" + run_test check_path overlay_overlay_with_aliases.dtb exists "/__symbols__" + run_test check_path overlay_overlay_with_aliases.dtb exists "/__fixups__" + run_test check_path overlay_overlay_with_aliases.dtb exists "/__local_fixups__" # Bad fixup tests for test in $BAD_FIXUP_TREES; do -- cgit v1.1