[Meego-kernel] [PATCH] sst remove harmless error messages

Lu Guanqun guanqun.lu at intel.com
Mon Jul 11 07:56:59 PDT 2011


Hi Kristen,

This patch is used to fix BOO #10742.

These error message are harmless, so I change these to debug messages.

Could you review it and please kindly put it into your MRST patchset?

Thanks!
-- 
guanqun
-------------- next part --------------
sst: remove harmless error messages

People are terrified by the error messages, in fact these code paths are
harmless, therefore we'd better make these error message to be debug messages
in order to prevent people from heart attacks. :)

Signed-off-by: Lu Guanqun <guanqun.lu at intel.com>

Index: linux-2.6.37/drivers/staging/intel_sst/intel_sst_stream.c
===================================================================
--- linux-2.6.37.orig/drivers/staging/intel_sst/intel_sst_stream.c
+++ linux-2.6.37/drivers/staging/intel_sst/intel_sst_stream.c
@@ -335,7 +335,7 @@ int sst_pause_stream(int str_id)
 		}
 	} else {
 		retval = -EBADRQC;
-		pr_err("SST ERR:BADQRC for stream\n ");
+		pr_debug("SST DBG:BADQRC for stream\n ");
 	}
 
 	return retval;
@@ -395,7 +395,7 @@ int sst_resume_stream(int str_id)
 		}
 	} else {
 		retval = -EBADRQC;
-		pr_err("SST ERR: BADQRC for stream\n");
+		pr_debug("SST DBG: BADQRC for stream\n");
 	}
 
 	return retval;
@@ -470,7 +470,7 @@ int sst_drop_stream(int str_id)
 		}
 	} else {
 		retval = -EBADRQC;
-		pr_err("SST ERR:BADQRC for stream\n");
+		pr_debug("SST DBG:BADQRC for stream\n");
 	}
 	return retval;
 }
@@ -497,7 +497,7 @@ int sst_drain_stream(int str_id)
 	if (str_info->status != STREAM_RUNNING &&
 		str_info->status != STREAM_INIT &&
 		str_info->status != STREAM_PAUSED) {
-			pr_err("SST ERR: BADQRC for stream = %d\n",
+			pr_debug("SST DBG: BADQRC for stream = %d\n",
 				       str_info->status);
 			return -EBADRQC;
 	}


More information about the MeeGo-kernel mailing list