Commit 163c9bca authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] tuner.h: rename TUNER_PAD_IF_OUTPUT to TUNER_PAD_OUTPUT

The output of a tuner is not only IF frequencies. They may also
output audio on some of its pins, and may even be a zero-IF tuner,
with outputs a baseband. So, rename the PAD name to make it
clearer and add a proper documentation about that at tuner.h.

No functional changes.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 8dfbcc43
...@@ -661,7 +661,7 @@ int dvb_create_media_graph(struct dvb_adapter *adap, ...@@ -661,7 +661,7 @@ int dvb_create_media_graph(struct dvb_adapter *adap,
if (ntuner && ndemod) { if (ntuner && ndemod) {
ret = media_create_pad_links(mdev, ret = media_create_pad_links(mdev,
MEDIA_ENT_F_TUNER, MEDIA_ENT_F_TUNER,
tuner, TUNER_PAD_IF_OUTPUT, tuner, TUNER_PAD_OUTPUT,
MEDIA_ENT_F_DTV_DEMOD, MEDIA_ENT_F_DTV_DEMOD,
demod, 0, MEDIA_LNK_FL_ENABLED, demod, 0, MEDIA_LNK_FL_ENABLED,
false); false);
......
...@@ -276,7 +276,7 @@ static int au0828_create_media_graph(struct au0828_dev *dev) ...@@ -276,7 +276,7 @@ static int au0828_create_media_graph(struct au0828_dev *dev)
return -EINVAL; return -EINVAL;
if (tuner) { if (tuner) {
ret = media_create_pad_link(tuner, TUNER_PAD_IF_OUTPUT, ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT,
decoder, 0, decoder, 0,
MEDIA_LNK_FL_ENABLED); MEDIA_LNK_FL_ENABLED);
if (ret) if (ret)
......
...@@ -1259,7 +1259,7 @@ static int cx231xx_create_media_graph(struct cx231xx *dev) ...@@ -1259,7 +1259,7 @@ static int cx231xx_create_media_graph(struct cx231xx *dev)
return 0; return 0;
if (tuner) { if (tuner) {
ret = media_create_pad_link(tuner, TUNER_PAD_IF_OUTPUT, decoder, 0, ret = media_create_pad_link(tuner, TUNER_PAD_OUTPUT, decoder, 0,
MEDIA_LNK_FL_ENABLED); MEDIA_LNK_FL_ENABLED);
if (ret < 0) if (ret < 0)
return ret; return ret;
......
...@@ -888,7 +888,7 @@ static int mxl111sf_attach_tuner(struct dvb_usb_adapter *adap) ...@@ -888,7 +888,7 @@ static int mxl111sf_attach_tuner(struct dvb_usb_adapter *adap)
state->tuner.function = MEDIA_ENT_F_TUNER; state->tuner.function = MEDIA_ENT_F_TUNER;
state->tuner.name = "mxl111sf tuner"; state->tuner.name = "mxl111sf tuner";
state->tuner_pads[TUNER_PAD_RF_INPUT].flags = MEDIA_PAD_FL_SINK; state->tuner_pads[TUNER_PAD_RF_INPUT].flags = MEDIA_PAD_FL_SINK;
state->tuner_pads[TUNER_PAD_IF_OUTPUT].flags = MEDIA_PAD_FL_SOURCE; state->tuner_pads[TUNER_PAD_OUTPUT].flags = MEDIA_PAD_FL_SOURCE;
ret = media_entity_pads_init(&state->tuner, ret = media_entity_pads_init(&state->tuner,
TUNER_NUM_PADS, state->tuner_pads); TUNER_NUM_PADS, state->tuner_pads);
......
...@@ -697,7 +697,7 @@ static int tuner_probe(struct i2c_client *client, ...@@ -697,7 +697,7 @@ static int tuner_probe(struct i2c_client *client,
register_client: register_client:
#if defined(CONFIG_MEDIA_CONTROLLER) #if defined(CONFIG_MEDIA_CONTROLLER)
t->pad[TUNER_PAD_RF_INPUT].flags = MEDIA_PAD_FL_SINK; t->pad[TUNER_PAD_RF_INPUT].flags = MEDIA_PAD_FL_SINK;
t->pad[TUNER_PAD_IF_OUTPUT].flags = MEDIA_PAD_FL_SOURCE; t->pad[TUNER_PAD_OUTPUT].flags = MEDIA_PAD_FL_SOURCE;
t->sd.entity.function = MEDIA_ENT_F_TUNER; t->sd.entity.function = MEDIA_ENT_F_TUNER;
t->sd.entity.name = t->name; t->sd.entity.name = t->name;
......
...@@ -21,11 +21,26 @@ ...@@ -21,11 +21,26 @@
#include <linux/videodev2.h> #include <linux/videodev2.h>
/* Tuner PADs */ /**
/* FIXME: is this the right place for it? */ * enum tuner_pad_index - tuner pad index
*
* @TUNER_PAD_RF_INPUT: Radiofrequency (RF) sink pad, usually linked to a
* RF connector entity.
* @TUNER_PAD_OUTPUT: Tuner output pad. This is actually more complex than
* a single pad output, as, in addition to luminance and
* chrominance IF a tuner may have internally an
* audio decoder (like xc3028) or it may produce an audio
* IF that will be used by an audio decoder like msp34xx.
* It may also have an IF-PLL demodulator on it, like
* tuners with tda9887. Yet, currently, we don't need to
* represent all the dirty details, as this is transparent
* for the V4L2 API usage. So, let's represent all kinds
* of different outputs as a single source pad.
* @TUNER_NUM_PADS: Number of pads of the tuner.
*/
enum tuner_pad_index { enum tuner_pad_index {
TUNER_PAD_RF_INPUT, TUNER_PAD_RF_INPUT,
TUNER_PAD_IF_OUTPUT, TUNER_PAD_OUTPUT,
TUNER_NUM_PADS TUNER_NUM_PADS
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment