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

Signed-off-by: Philip Rowlands <[EMAIL PROTECTED]>
Signed-off-by: Michael Krufky <[EMAIL PROTECTED]>

 linux/drivers/media/video/tda9887.c    |   13 +++++++++++++
 linux/drivers/media/video/tuner-core.c |   12 ++++++++++++
 2 files changed, 25 insertions(+)

diff -u /tmp/dst.1007 linux/drivers/media/video/tuner-core.c
--- /tmp/dst.1007       2005-09-05 11:43:13.000000000 -0300
+++ linux/drivers/media/video/tuner-core.c      2005-09-05 11:43:13.000000000 
-0300
@@ -281,6 +281,12 @@
                        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 @@
                        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;
                }
        }
 
diff -u /tmp/dst.1007 linux/drivers/media/video/tda9887.c
--- /tmp/dst.1007       2005-09-05 11:43:13.000000000 -0300
+++ linux/drivers/media/video/tda9887.c 2005-09-05 11:43:13.000000000 -0300
@@ -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 @@
                        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 @@
                        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;

Reply via email to