s390/ipl: remove reipl_method and dump_method

reipl_method and dump_method have been used in addition to reipl_type
and dump_type, because a single reipl_type could be achieved with
multiple reipl_method (same for dump_type/method). After dropping
non-diag308_set based reipl methods, there is a single method per
reipl_type/dump_type and reipl_method and dump_method could be simply
removed.

Reviewed-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

1 files changed, 11 insertions, 46 deletions

diff --git a/arch/s390/kernel/ipl.c b/arch/s390/kernel/ipl.c
index 75dd08547c31..4296d7e61fb6 100644
--- a/arch/s390/kernel/ipl.c
+++ b/arch/s390/kernel/ipl.c
@@ -117,27 +117,12 @@ static char *dump_type_str(enum dump_type type)
         }
 }
 
-enum ipl_method {
-        REIPL_METHOD_CCW_DIAG,
-        REIPL_METHOD_FCP_DIAG,
-        REIPL_METHOD_FCP_DUMP,
-        REIPL_METHOD_NSS_DIAG,
-        REIPL_METHOD_DEFAULT,
-};
-
-enum dump_method {
-        DUMP_METHOD_NONE,
-        DUMP_METHOD_CCW_DIAG,
-        DUMP_METHOD_FCP_DIAG,
-};
-
 static int ipl_block_valid;
 static struct ipl_parameter_block ipl_block;
 
 static int reipl_capabilities = IPL_TYPE_UNKNOWN;
 
 static enum ipl_type reipl_type = IPL_TYPE_UNKNOWN;
-static enum ipl_method reipl_method = REIPL_METHOD_DEFAULT;
 static struct ipl_parameter_block *reipl_block_fcp;
 static struct ipl_parameter_block *reipl_block_ccw;
 static struct ipl_parameter_block *reipl_block_nss;
@@ -145,7 +130,6 @@ static struct ipl_parameter_block *reipl_block_actual;
 
 static int dump_capabilities = DUMP_TYPE_NONE;
 static enum dump_type dump_type = DUMP_TYPE_NONE;
-static enum dump_method dump_method = DUMP_METHOD_NONE;
 static struct ipl_parameter_block *dump_block_fcp;
 static struct ipl_parameter_block *dump_block_ccw;
 
@@ -928,25 +912,16 @@ static int reipl_set_type(enum ipl_type type)
 
         switch(type) {
         case IPL_TYPE_CCW:
-                reipl_method = REIPL_METHOD_CCW_DIAG;
                 reipl_block_actual = reipl_block_ccw;
                 break;
         case IPL_TYPE_FCP:
-                reipl_method = REIPL_METHOD_FCP_DIAG;
                 reipl_block_actual = reipl_block_fcp;
                 break;
-        case IPL_TYPE_FCP_DUMP:
-                reipl_method = REIPL_METHOD_FCP_DUMP;
-                break;
         case IPL_TYPE_NSS:
-                reipl_method = REIPL_METHOD_NSS_DIAG;
                 reipl_block_actual = reipl_block_nss;
                 break;
-        case IPL_TYPE_UNKNOWN:
-                reipl_method = REIPL_METHOD_DEFAULT;
-                break;
         default:
-                BUG();
+                break;
         }
         reipl_type = type;
         return 0;
@@ -981,23 +956,23 @@ static struct kset *reipl_fcp_kset;
 
 static void __reipl_run(void *unused)
 {
-        switch (reipl_method) {
-        case REIPL_METHOD_CCW_DIAG:
+        switch (reipl_type) {
+        case IPL_TYPE_CCW:
                 diag308(DIAG308_SET, reipl_block_ccw);
                 diag308(DIAG308_LOAD_CLEAR, NULL);
                 break;
-        case REIPL_METHOD_FCP_DIAG:
+        case IPL_TYPE_FCP:
                 diag308(DIAG308_SET, reipl_block_fcp);
                 diag308(DIAG308_LOAD_CLEAR, NULL);
                 break;
-        case REIPL_METHOD_NSS_DIAG:
+        case IPL_TYPE_NSS:
                 diag308(DIAG308_SET, reipl_block_nss);
                 diag308(DIAG308_LOAD_CLEAR, NULL);
                 break;
-        case REIPL_METHOD_DEFAULT:
+        case IPL_TYPE_UNKNOWN:
                 diag308(DIAG308_LOAD_CLEAR, NULL);
                 break;
-        case REIPL_METHOD_FCP_DUMP:
+        case IPL_TYPE_FCP_DUMP:
                 break;
         }
         disabled_wait((unsigned long) __builtin_return_address(0));
@@ -1230,16 +1205,6 @@ static int dump_set_type(enum dump_type type)
 {
         if (!(dump_capabilities & type))
                 return -EINVAL;
-        switch (type) {
-        case DUMP_TYPE_CCW:
-                dump_method = DUMP_METHOD_CCW_DIAG;
-                break;
-        case DUMP_TYPE_FCP:
-                dump_method = DUMP_METHOD_FCP_DIAG;
-                break;
-        default:
-                dump_method = DUMP_METHOD_NONE;
-        }
         dump_type = type;
         return 0;
 }
@@ -1282,11 +1247,11 @@ static void diag308_dump(void *dump_block)
 
 static void __dump_run(void *unused)
 {
-        switch (dump_method) {
-        case DUMP_METHOD_CCW_DIAG:
+        switch (dump_type) {
+        case DUMP_TYPE_CCW:
                 diag308_dump(dump_block_ccw);
                 break;
-        case DUMP_METHOD_FCP_DIAG:
+        case DUMP_TYPE_FCP:
                 diag308_dump(dump_block_fcp);
                 break;
         default:
@@ -1296,7 +1261,7 @@ static void __dump_run(void *unused)
 
 static void dump_run(struct shutdown_trigger *trigger)
 {
-        if (dump_method == DUMP_METHOD_NONE)
+        if (dump_type == DUMP_TYPE_NONE)
                 return;
         smp_send_stop();
         smp_call_ipl_cpu(__dump_run, NULL);