Skip to contentSkip to conversation

cx23885: Wrong command printed in cmd_to_str()

PATCH In review Roel Kluin 2 participants 2 messages Jan 20, 2010 Media (V4L/DVB)

lore
Share & tools
lore

Roel Kluin

author
#0 lore raw

PATCH 1/1 cx23885: Wrong command printed in cmd_to_str()

The wrong command was printed for case CX2341X_ENC_SET_DNR_FILTER_MODE, and a typo in case CX2341X_ENC_SET_PCR_ID.

Signed-off-by Roel Kluin <[email protected]>

---
Or is this intended?

drivers/media/video/cx23885/cx23885-417.c +2 -2
index 88c0d24..2ab97ad 100644
@@ -681,7 +681,7 @@ static char *cmd_to_str(int cmd)
 	case CX2341X_ENC_SET_VIDEO_ID:
 		return  "SET_VIDEO_ID";
 	case CX2341X_ENC_SET_PCR_ID:
-		return  "SET_PCR_PID";
+		return  "SET_PCR_ID";
 	case CX2341X_ENC_SET_FRAME_RATE:
 		return  "SET_FRAME_RATE";
 	case CX2341X_ENC_SET_FRAME_SIZE:

@@ -693,7 +693,7 @@ static char *cmd_to_str(int cmd)
 	case CX2341X_ENC_SET_ASPECT_RATIO:
 		return  "SET_ASPECT_RATIO";
 	case CX2341X_ENC_SET_DNR_FILTER_MODE:
-		return  "SET_DNR_FILTER_PROPS";
+		return  "SET_DNR_FILTER_MODE";
 	case CX2341X_ENC_SET_DNR_FILTER_PROPS:
 		return  "SET_DNR_FILTER_PROPS";
 	case CX2341X_ENC_SET_CORING_LEVELS:
Acked-by on v1 1/1
#1 lore raw

Thanks Roel.

Acked-by Steven Toth <[email protected]>

- Steve

Signature · 2 lines
Steven Toth - Kernel Labs
http://www.kernellabs.com
Quoted 34 lines · Roel Kluin

The wrong command was printed for case CX2341X_ENC_SET_DNR_FILTER_MODE, and a typo in case CX2341X_ENC_SET_PCR_ID.

Signed-off-by Roel Kluin <[email protected]>

---
Or is this intended?

drivers/media/video/cx23885/cx23885-417.c +0 -0
index 88c0d24..2ab97ad 100644
@@ -681,7 +681,7 @@ static char *cmd_to_str(int cmd)

       case CX2341X_ENC_SET_VIDEO_ID:
               return  "SET_VIDEO_ID";
       case CX2341X_ENC_SET_PCR_ID:
-               return  "SET_PCR_PID";
+               return  "SET_PCR_ID";
       case CX2341X_ENC_SET_FRAME_RATE:
               return  "SET_FRAME_RATE";
       case CX2341X_ENC_SET_FRAME_SIZE:

drivers/media/video/cx23885/cx23885-417.c +0 -0
@@ -693,7 +693,7 @@ static char *cmd_to_str(int cmd)

       case CX2341X_ENC_SET_ASPECT_RATIO:
               return  "SET_ASPECT_RATIO";
       case CX2341X_ENC_SET_DNR_FILTER_MODE:
-               return  "SET_DNR_FILTER_PROPS";
+               return  "SET_DNR_FILTER_MODE";
       case CX2341X_ENC_SET_DNR_FILTER_PROPS:
               return  "SET_DNR_FILTER_PROPS";
       case CX2341X_ENC_SET_CORING_LEVELS:
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Signature · 2 lines
Steven Toth - Kernel Labs
http://www.kernellabs.com