Skip to content

Commit

Permalink
V4L/DVB (3271): Update tuner comments
Browse files Browse the repository at this point in the history
Right now, all tuners are using the first tuner_params[]
array element for analog mode. We are now ready to begin merging
similar tuner definitions together, such that each tuner definition
will have a tuner_params struct for each available video standard.
The tuner_params[] array element will be chosen based on the video
standard in use.

Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
  • Loading branch information
Michael Krufky authored and Mauro Carvalho Chehab committed Feb 7, 2006
1 parent bd0d0f5 commit 99d33d5
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 16 deletions.
10 changes: 0 additions & 10 deletions drivers/media/video/tuner-simple.c
Original file line number Diff line number Diff line change
Expand Up @@ -79,16 +79,6 @@ MODULE_PARM_DESC(offset,"Allows to specify an offset for tuner");
#define TUNER_PLL_LOCKED 0x40
#define TUNER_STEREO_MK3 0x04

/* FIXME:
* Right now, all tuners are using the first tuner_params[] array element
* for analog mode. In the future, we will be merging similar tuner
* definitions together, such that each tuner definition will have a
* tuner_params struct for each available video standard. At that point,
* the tuner_params[] array element will be chosen based on the video
* standard in use.
*
*/

/* ---------------------------------------------------------------------- */

static int tuner_getstatus(struct i2c_client *c)
Expand Down
15 changes: 9 additions & 6 deletions drivers/media/video/tuner-types.c
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,16 @@
* Each tuner_params array may contain one or more elements, one
* for each video standard.
*
* FIXME: Some tuner_range definitions are duplicated, and
* should be eliminated.
* FIXME: tuner_params struct contains an element, tda988x. We must
* set this for all tuners that contain a tda988x chip, and then we
* can remove this setting from the various card structs.
*
* FIXME: tunertype struct contains an element, has_tda988x.
* We must set this for all tunertypes that contain a tda988x
* chip, and then we can remove this setting from the various
* card structs.
* FIXME: Right now, all tuners are using the first tuner_params[]
* array element for analog mode. In the future, we will be merging
* similar tuner definitions together, such that each tuner definition
* will have a tuner_params struct for each available video standard.
* At that point, the tuner_params[] array element will be chosen
* based on the video standard in use.
*/

/* 0-9 */
Expand Down

0 comments on commit 99d33d5

Please sign in to comment.