obmc_boot_test to only get FFDC on request

Previously, any obmc_boot_test test failure would cause FFDC collection.
This is redundant when it is called from within another robot test that is
handling FFDC on failure.

obmc_boot_test will no longer dump FFDC due to a test failure.

Change-Id: I3c97452a48f8c911a87f275b44845e2c0e6fc9e9
Signed-off-by: Michael Walsh <micwalsh@us.ibm.com>
diff --git a/lib/obmc_boot_test.py b/lib/obmc_boot_test.py
index d7c5e36..8acc4e4 100755
--- a/lib/obmc_boot_test.py
+++ b/lib/obmc_boot_test.py
@@ -945,7 +945,7 @@
     rc, shell_rc, failed_plug_in_name = grpi.rprocess_plug_in_packages(
         call_point='ffdc_check', shell_rc=dump_ffdc_rc(),
         stop_on_plug_in_failure=1, stop_on_non_zero_rc=1)
-    if boot_status != "PASS" or ffdc_check == "All" or\
+    if ffdc_check == "All" or\
        shell_rc == dump_ffdc_rc():
         status, ret_values = grk.run_key_u("my_ffdc", ignore=1)
         if status != 'PASS':