[PATCH] SAA5246A Teletext driver also works with SAA5281

[Michael Geng]

Hello,

the SAA5246A Teletext processor driver also works for the
SAA5281 (I am the author of this driver).

This is not surprising because according to the data sheet of 
the SAA5281 it is compatible to the SAA5246A. But now I have 
tested this with a Siemens Multimedia Extension Board (MXB).

The following patch only changes comments in 
/drivers/media/video/Kconfig and in the sources of the driver.

I also added MODULE_AUTHOR and MODULE_DESCRIPTION macros which
were missing up to now.

Michael

diff -u -N linux-2.6.5/drivers/media/video/Kconfig linux/drivers/media/video/Kconfig
--- linux-2.6.5/drivers/media/video/Kconfig	Sun Apr  4 05:38:16 2004
+++ linux/drivers/media/video/Kconfig	Mon Apr  5 21:25:31 2004
@@ -112,11 +112,11 @@
 	  It is also available as a module (cpia_usb).
 
 config VIDEO_SAA5246A
-	tristate "SAA5246A Teletext processor"
+	tristate "SAA5246A, SAA5281 Teletext processor"
 	depends on VIDEO_DEV && I2C
 	help
-	  Support for I2C bus based teletext using the SAA5246A chip. Useful
-	  only if you live in Europe.
+	  Support for I2C bus based teletext using the SAA5246A or SAA5281 
+	  chip. Useful only if you live in Europe.
 
 	  To compile this driver as a module, choose M here: the
 	  module will be called saa5246a.
Common subdirectories: linux-2.6.5/drivers/media/video/cx88 and linux/drivers/media/video/cx88
diff -u -N linux-2.6.5/drivers/media/video/saa5246a.c linux/drivers/media/video/saa5246a.c
--- linux-2.6.5/drivers/media/video/saa5246a.c	Sun Apr  4 05:36:58 2004
+++ linux/drivers/media/video/saa5246a.c	Mon Apr  5 21:20:40 2004
@@ -1,9 +1,10 @@
 /*
- * Driver for the SAA5246A videotext decoder chip from Philips.
+ * Driver for the SAA5246A or SAA5281 Teletext (=Videotext) decoder chips from 
+ * Philips.
  *
- * Only capturing of videotext pages is tested. The SAA5246A chip also has
- * a TV output but my hardware doesn't use it. For this reason this driver
- * does not support changing any TV display settings.
+ * Only capturing of Teletext pages is tested. The chip also has a TV output 
+ * but my hardware doesn't use it. For this reason this driver does not support 
+ * changing any TV display settings.
  *
  * Copyright (C) 2004 Michael Geng <linux@xxxxxxxxxxxxxx>
  *
@@ -47,6 +48,10 @@
 #include <linux/videodev.h>
 #include "saa5246a.h"
 
+MODULE_AUTHOR("Michael Geng <linux@xxxxxxxxxxxxxx>");
+MODULE_DESCRIPTION("Philips SAA5246A, SAA5281 Teletext decoder driver");
+MODULE_LICENSE("GPL");
+
 struct saa5246a_device
 {
 	u8     pgbuf[NUM_DAUS][VTX_VIRTUALSIZE];
@@ -764,8 +769,8 @@
 
 static int __init init_saa_5246a (void)
 {
-	printk(KERN_INFO "SAA5246A driver (" IF_NAME
-		" interface) for VideoText version %d.%d\n",
+	printk(KERN_INFO 
+		"SAA5246A (or compatible) Teletext decoder driver version %d.%d\n",
 		MAJOR_VERSION, MINOR_VERSION);
 	return i2c_add_driver(&i2c_driver_videotext);
 }
@@ -796,5 +801,3 @@
 	.release  = video_device_release,
 	.minor    = -1,
 };
-
-MODULE_LICENSE("GPL");
diff -u -N linux-2.6.5/drivers/media/video/saa5246a.h linux/drivers/media/video/saa5246a.h
--- linux-2.6.5/drivers/media/video/saa5246a.h	Sun Apr  4 05:37:39 2004
+++ linux/drivers/media/video/saa5246a.h	Mon Apr  5 21:20:30 2004
@@ -1,5 +1,7 @@
 /*
-   Driver for the SAA5246A videotext decoder chip from Philips.
+   Driver for the SAA5246A or SAA5281 Teletext (=Videotext) decoder chips from 
+   Philips.
+
    Copyright (C) 2004 Michael Geng (linux@xxxxxxxxxxxxxx)
 
    This program is free software; you can redistribute it and/or modify
Common subdirectories: linux-2.6.5/drivers/media/video/saa7134 and linux/drivers/media/video/saa7134
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/