Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 99d33d50 authored by Michael Krufky's avatar Michael Krufky Committed by Mauro Carvalho Chehab
Browse files

V4L/DVB (3271): Update tuner comments 



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: default avatarMichael Krufky <mkrufky@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@infradead.org>
parent bd0d0f59

drivers/media/video/tuner-simple.c

+0 −10
Original line number Diff line number Diff line
@@ -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)

drivers/media/video/tuner-types.c

+9 −6
Original line number Diff line number Diff line
@@ -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 */