[PATCH] v4l: print warning if pal= or secam= argument is unrecognized

- print warning if pal= or secam= argument is unrecognized

Signed-off-by: Philip Rowlands <phr@doc.ic.ac.uk>
Signed-off-by: Michael Krufky <mkrufky@m1k.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

2 files changed, 25 insertions, 0 deletions

diff --git a/drivers/media/video/tda9887.c b/drivers/media/video/tda9887.c
index 79e0bd1aa70f..0456dda2624d 100644
--- a/drivers/media/video/tda9887.c
+++ b/drivers/media/video/tda9887.c
@@ -23,6 +23,7 @@
       TDA9887 (world), TDA9885 (USA)
       Note: OP2 of tda988x must be set to 1, else MT2032 is disabled!
    - KNC One TV-Station RDS (saa7134)
+   - Hauppauge PVR-150/500 (possibly more)
 */
 
 
@@ -519,6 +520,12 @@ static int tda9887_fixup_std(struct tda9887 *t)
                         dprintk(PREFIX "insmod fixup: PAL => PAL-DK\n");
                         t->std = V4L2_STD_PAL_DK;
                         break;
+                case '-':
+                        /* default parameter, do nothing */
+                        break;
+                default:
+                        printk(PREFIX "pal= argument not recognised\n");
+                        break;
                 }
         }
         if ((t->std & V4L2_STD_SECAM) == V4L2_STD_SECAM) {
@@ -535,6 +542,12 @@ static int tda9887_fixup_std(struct tda9887 *t)
                         dprintk(PREFIX "insmod fixup: SECAM => SECAM-L\n");
                         t->std = V4L2_STD_SECAM_L;
                         break;
+                case '-':
+                        /* default parameter, do nothing */
+                        break;
+                default:
+                        printk(PREFIX "secam= argument not recognised\n");
+                        break;
                 }
         }
         return 0;
diff --git a/drivers/media/video/tuner-core.c b/drivers/media/video/tuner-core.c
index afc96bbb1c11..05572020af4d 100644
--- a/drivers/media/video/tuner-core.c
+++ b/drivers/media/video/tuner-core.c
@@ -281,6 +281,12 @@ static int tuner_fixup_std(struct tuner *t)
                         tuner_dbg ("insmod fixup: PAL => PAL-N\n");
                         t->std = V4L2_STD_PAL_N;
                         break;
+                case '-':
+                        /* default parameter, do nothing */
+                        break;
+                default:
+                        tuner_warn ("pal= argument not recognised\n");
+                        break;
                 }
         }
         if ((t->std & V4L2_STD_SECAM) == V4L2_STD_SECAM) {
@@ -297,6 +303,12 @@ static int tuner_fixup_std(struct tuner *t)
                         tuner_dbg ("insmod fixup: SECAM => SECAM-L\n");
                         t->std = V4L2_STD_SECAM_L;
                         break;
+                case '-':
+                        /* default parameter, do nothing */
+                        break;
+                default:
+                        tuner_warn ("secam= argument not recognised\n");
+                        break;
                 }
         }