[S390] dasd: message cleanup

Moved some Messages into s390 debug feature and changed remaining
messages to use the dev_xxx and pr_xxx macros.

Signed-off-by: Stefan Haberland <stefan.haberland@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

1 files changed, 19 insertions, 19 deletions

diff --git a/drivers/s390/block/dasd_devmap.c b/drivers/s390/block/dasd_devmap.c
index da231a787cee..e77666c8e6c0 100644
--- a/drivers/s390/block/dasd_devmap.c
+++ b/drivers/s390/block/dasd_devmap.c
@@ -13,6 +13,8 @@
  *
  */
 
+#define KMSG_COMPONENT "dasd"
+
 #include <linux/ctype.h>
 #include <linux/init.h>
 #include <linux/module.h>
@@ -127,6 +129,7 @@ __setup ("dasd=", dasd_call_setup);
  * Read a device busid/devno from a string.
  */
 static int
+
 dasd_busid(char **str, int *id0, int *id1, int *devno)
 {
         int val, old_style;
@@ -134,8 +137,7 @@ dasd_busid(char **str, int *id0, int *id1, int *devno)
         /* Interpret ipldev busid */
         if (strncmp(DASD_IPLDEV, *str, strlen(DASD_IPLDEV)) == 0) {
                 if (ipl_info.type != IPL_TYPE_CCW) {
-                        MESSAGE(KERN_ERR, "%s", "ipl device is not a ccw "
-                                "device");
+                        pr_err("The IPL device is not a CCW device\n");
                         return -EINVAL;
                 }
                 *id0 = 0;
@@ -211,9 +213,8 @@ dasd_feature_list(char *str, char **endp)
                 else if (len == 8 && !strncmp(str, "failfast", 8))
                         features |= DASD_FEATURE_FAILFAST;
                 else {
-                        MESSAGE(KERN_WARNING,
-                                "unsupported feature: %*s, "
-                                "ignoring setting", len, str);
+                        pr_warning("%*s is not a supported device option\n",
+                                   len, str);
                         rc = -EINVAL;
                 }
                 str += len;
@@ -222,8 +223,8 @@ dasd_feature_list(char *str, char **endp)
                 str++;
         }
         if (*str != ')') {
-                MESSAGE(KERN_WARNING, "%s",
-                        "missing ')' in dasd parameter string\n");
+                pr_warning("A closing parenthesis ')' is missing in the "
+                           "dasd= parameter\n");
                 rc = -EINVAL;
         } else
                 str++;
@@ -255,28 +256,27 @@ dasd_parse_keyword( char *parsestring ) {
         }
         if (strncmp("autodetect", parsestring, length) == 0) {
                 dasd_autodetect = 1;
-                MESSAGE (KERN_INFO, "%s",
-                         "turning to autodetection mode");
+                pr_info("The autodetection mode has been activated\n");
                 return residual_str;
         }
         if (strncmp("probeonly", parsestring, length) == 0) {
                 dasd_probeonly = 1;
-                MESSAGE(KERN_INFO, "%s",
-                        "turning to probeonly mode");
+                pr_info("The probeonly mode has been activated\n");
                 return residual_str;
         }
         if (strncmp("nopav", parsestring, length) == 0) {
                 if (MACHINE_IS_VM)
-                        MESSAGE(KERN_INFO, "%s", "'nopav' not supported on VM");
+                        pr_info("'nopav' is not supported on z/VM\n");
                 else {
                         dasd_nopav = 1;
-                        MESSAGE(KERN_INFO, "%s", "disable PAV mode");
+                        pr_info("PAV support has be deactivated\n");
                 }
                 return residual_str;
         }
         if (strncmp("nofcx", parsestring, length) == 0) {
                 dasd_nofcx = 1;
-                MESSAGE(KERN_INFO, "%s", "disable High Performance Ficon");
+                pr_info("High Performance FICON support has been "
+                        "deactivated\n");
                 return residual_str;
         }
         if (strncmp("fixedbuffers", parsestring, length) == 0) {
@@ -287,10 +287,10 @@ dasd_parse_keyword( char *parsestring ) {
                                           PAGE_SIZE, SLAB_CACHE_DMA,
                                           NULL);
                 if (!dasd_page_cache)
-                        MESSAGE(KERN_WARNING, "%s", "Failed to create slab, "
+                        DBF_EVENT(DBF_WARNING, "%s", "Failed to create slab, "
                                 "fixed buffer mode disabled.");
                 else
-                        MESSAGE (KERN_INFO, "%s",
+                        DBF_EVENT(DBF_INFO, "%s",
                                  "turning on fixed buffer mode");
                 return residual_str;
         }
@@ -328,7 +328,7 @@ dasd_parse_range( char *parsestring ) {
             (from_id0 != to_id0 || from_id1 != to_id1 || from > to))
                 rc = -EINVAL;
         if (rc) {
-                MESSAGE(KERN_ERR, "Invalid device range %s", parsestring);
+                pr_err("%s is not a valid device range\n", parsestring);
                 return ERR_PTR(rc);
         }
         features = dasd_feature_list(str, &str);
@@ -347,8 +347,8 @@ dasd_parse_range( char *parsestring ) {
                 return str + 1;
         if (*str == '\0')
                 return str;
-        MESSAGE(KERN_WARNING,
-                "junk at end of dasd parameter string: %s\n", str);
+        pr_warning("The dasd= parameter value %s has an invalid ending\n",
+                   str);
         return ERR_PTR(-EINVAL);
 }