Re: [linux-dvb] Kworld DVB-T 210 - dvb tuning problem

[timf <timf@iinet.net.au>]

Hartmut Hackmann wrote:
> Hi, Tim
> <snip>
>>>
>>> I think i know now how your card needs to be configured.
>>> Something that seems not to be really clear for you is the 
>>> configuration
>>> of the so-called AGC (automatic gain control) of the tuner. This
>>> needs to be switched according to the function the tuner is used for:
>>> analog TV -> FM Radio -> DVB-T.
>>> This is done with the 4052 analog mux and requires 2 control bits -> 
>>> the GPIOs.
>>> GPIO 21 is used to switch between analog TV and FM radio.
>>> Typically GPIO21=0 -> analog TV, GPIO21=1 -> FM Radio.
>>> In your case, GPIO22 is used to switch between DVB-T and analog 
>>> while most cards
>>> use GPIO1 of the tda10046 for this.
>>>
>>> You should need to make chages only in saa7134-dvb, dvb_init()
>>> - remove the old
>>>     case SAA7134_BOARD_KWORLD_DVBT_210: (plus code)
>>> -and add it to
>>>     case SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS:
>>>
>>> That should do it (though we should add a .antenna switch to the 
>>> configuration).
>>>
>>> Best regards
>>>   Hartmut
>>>
>> Hi Hartmut,
>>
>>
>> OK, so instead of this:
>> ...
>> /* ------------------------------------------------------------------
>> * special case: this card uses saa713x GPIO22 for the mode switch
>> */
>>
>> static int ads_duo_tuner_init(struct dvb_frontend *fe)
>> {
>>    struct saa7134_dev *dev = fe->dvb->priv;
>>    philips_tda827x_tuner_init(fe);
>>    /* route TDA8275a AGC input to the channel decoder */
>>    saa7134_set_gpio(dev, 22, 1);
>>    return 0;
>> }
>>
>> static int ads_duo_tuner_sleep(struct dvb_frontend *fe)
>> {
>>    struct saa7134_dev *dev = fe->dvb->priv;
>>    /* route TDA8275a AGC input to the analog IF chip*/
>>    saa7134_set_gpio(dev, 22, 0);
>>    philips_tda827x_tuner_sleep(fe);
>>    return 0;
>> }
>>
>> static struct tda827x_config ads_duo_cfg = {
>>    .tuner_callback = saa7134_tuner_callback,
>>    .init = ads_duo_tuner_init,
>>    .sleep = ads_duo_tuner_sleep,
>>    .config = 0
>> };
>>
>> static struct tda1004x_config ads_tech_duo_config = {
>>    .demod_address = 0x08,
>>    .invert        = 1,
>>    .invert_oclk   = 0,
>>    .xtal_freq     = TDA10046_XTAL_16M,
>>    .agc_config    = TDA10046_AGC_TDA827X,
>>    .gpio_config   = TDA10046_GP00_I,
>>    .if_freq       = TDA10046_FREQ_045,
>>    .tuner_address = 0x61,
>>    .request_firmware = philips_tda1004x_request_firmware
>> };
>>
>> static int kw210_tuner_init(struct dvb_frontend *fe)
>> {
>>    struct saa7134_dev *dev = fe->dvb->priv;
>>    philips_tda827x_tuner_init(fe);
>>    /* route TDA8275a AGC input to the channel decoder */
>>    saa7134_set_gpio(dev, 22, 1);
>>    return 0;
>> }
>>
>> static int kw210_tuner_sleep(struct dvb_frontend *fe)
>> {
>>    struct saa7134_dev *dev = fe->dvb->priv;
>>    /* route TDA8275a AGC input to the analog IF chip*/
>>    saa7134_set_gpio(dev, 22, 0);
>>    philips_tda827x_tuner_sleep(fe);
>>    return 0;
>> }
>>
>> static struct tda827x_config kw210_cfg = {
>>    .tuner_callback = saa7134_tuner_callback,
>>    .init = kw210_tuner_init,
>>    .sleep = kw210_tuner_sleep,
>>    .config = 0
>> };
>>
>> static struct tda1004x_config kworld_dvb_t_210_config = {
>>    .demod_address = 0x08,
>>    .invert        = 1,
>>    .invert_oclk   = 0,
>>    .xtal_freq     = TDA10046_XTAL_16M,
>>    .agc_config    = TDA10046_AGC_TDA827X,
>>    .gpio_config   = TDA10046_GP11_I,
>>    .if_freq       = TDA10046_FREQ_045,
>>    .tuner_address = 0x61,
>>    .antenna_switch= 1,
>>    .request_firmware = philips_tda1004x_request_firmware
>> };
>> -------------------
>> /* ==================================================================
>> * Core code
>> */
>>
>> static int dvb_init(struct saa7134_dev *dev)
>> {
>> ...
>>    case SAA7134_BOARD_KWORLD_DVBT_210:
>>        dev->dvb.frontend = dvb_attach(tda10046_attach, 
>> &kworld_dvb_t_210_config, &dev->i2c_adap);
>>        if (dev->dvb.frontend) {
>>            if (dvb_attach(tda827x_attach,dev->dvb.frontend,
>>                   kworld_dvb_t_210_config.tuner_address, &dev->i2c_adap,
>>                                &kw210_cfg) == NULL) {
>>                wprintk("no tda827x tuner found at addr: %02x\n",
>>                    kworld_dvb_t_210_config.tuner_address);
>>            }
>>        }
>>        break;
>> ...
>>    case SAA7134_BOARD_ADS_DUO_CARDBUS_PTV331:
>>    case SAA7134_BOARD_FLYDVBT_HYBRID_CARDBUS:
>>        dev->dvb.frontend = dvb_attach(tda10046_attach,
>>                           &ads_tech_duo_config,
>>                           &dev->i2c_adap);
>>        if (dev->dvb.frontend) {
>>            if (dvb_attach(tda827x_attach,dev->dvb.frontend,
>>                   ads_tech_duo_config.tuner_address, &dev->i2c_adap,
>>                                &ads_duo_cfg) == NULL) {
>>                wprintk("no tda827x tuner found at addr: %02x\n",
>>                    ads_tech_duo_config.tuner_address);
>>            }
>>        }
>>        break;
>> ...
>> <snip>
> Ok , the antenna_switch is clear. But the
> .gpio_config makes no sense. The card either uses either GPIO22 or
> the tda10046 for mode switching but not both.
> What happens if you use the "bad" gpio_config?
> Please switch to tzap for testing. This command line program clearly
> reports the status of the channel decoder, including signal level.
>
> Hartmut
>

Hi Hartmut,


1) With    .gpio_config   = TDA10046_GP11_I,

root@ubuntu:/home/timf# tzap -r "TEN HD"
using '/dev/dvb/adapter0/frontend0' and '/dev/dvb/adapter0/demux0'
tuning to 788500000 Hz
video pid 0x0202, audio pid 0x0000
status 00 | signal 9d9d | snr 3333 | ber 0001fffe | unc 0000004d |
status 1f | signal 9d9d | snr fdfd | ber 000001b8 | unc ffffffff | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fdfd | ber 000001de | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fefe | ber 0000017c | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fefe | ber 00000182 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fefe | ber 000001a4 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9b9b | snr fefe | ber 00000202 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9b9b | snr fefe | ber 0000021c | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9b9b | snr fefe | ber 00000226 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fefe | ber 00000234 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9b9b | snr fefe | ber 00000290 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9b9b | snr fefe | ber 00000240 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 9c9c | snr fefe | ber 000001fa | unc 00000000 | 
FE_HAS_LOCK

Analog-tv also scans fine.

2) With
//    .gpio_config   = TDA10046_GP11_I,
    .gpio_config   = TDA10046_GP00_I,

timf@ubuntu:~$ sudo -s -H
[sudo] password for timf:
root@ubuntu:/home/timf# tzap -r "TEN HD"
using '/dev/dvb/adapter0/frontend0' and '/dev/dvb/adapter0/demux0'
tuning to 788500000 Hz
video pid 0x0202, audio pid 0x0000
status 00 | signal 0000 | snr 9494 | ber 0001fffe | unc 00000000 |
status 1f | signal 0000 | snr ecec | ber 00019622 | unc ffffffff | 
FE_HAS_LOCK
status 1f | signal 0000 | snr f8f8 | ber 00014eac | unc ffffffff | 
FE_HAS_LOCK
status 00 | signal 0000 | snr 5e5e | ber 0001fffe | unc 00000000 |
status 1f | signal 0000 | snr fcfc | ber 00002bcc | unc ffffffff | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fdfd | ber 000018a2 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fdfd | ber 00000e74 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fdfd | ber 00000912 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fefe | ber 000006b6 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fefe | ber 0000068e | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fdfd | ber 000005c2 | unc 00000000 | 
FE_HAS_LOCK
status 1f | signal 0000 | snr fefe | ber 00000584 | unc 00000000 | 
FE_HAS_LOCK

However, analog-tv fails to scan anything -> "no signal"

3) antenna_switch

static int philips_tda827x_tuner_init(struct dvb_frontend *fe)
{
    struct saa7134_dev *dev = fe->dvb->priv;
    struct tda1004x_state *state = fe->demodulator_priv;

    switch (state->config->antenna_switch) {
    case 0: break;
    case 1:    dprintk("setting GPIO21 to 0 (TV antenna?)\n");
        saa7134_set_gpio(dev, 21, 0);
        break;
    case 2: dprintk("setting GPIO21 to 1 (Radio antenna?)\n");
        saa7134_set_gpio(dev, 21, 1);
        break;
    }
    return 0;
}

static int philips_tda827x_tuner_sleep(struct dvb_frontend *fe)
{
    struct saa7134_dev *dev = fe->dvb->priv;
    struct tda1004x_state *state = fe->demodulator_priv;

    switch (state->config->antenna_switch) {
    case 0: break;
    case 1: dprintk("setting GPIO21 to 1 (Radio antenna?)\n");
        saa7134_set_gpio(dev, 21, 1);
        break;
    case 2:    dprintk("setting GPIO21 to 0 (TV antenna?)\n");
        saa7134_set_gpio(dev, 21, 0);
        break;
    }
    return 0;
}

If I put .antenna_switch = 1 or 2, in my struct, dvb-t fails to scan.
I can't put antenna_switch in my struct, can I, as I am using gpio22?
So I took it out, and in saa7134-cards.c, changed .radio to this:

    [SAA7134_BOARD_KWORLD_DVBT_210] = {
        .name           = "KWorld DVB-T 210",
        .audio_clock    = 0x00187de7,
        .tuner_type     = TUNER_PHILIPS_TDA8290,
        .radio_type     = UNSET,
        .tuner_addr    = ADDR_UNSET,
        .radio_addr    = ADDR_UNSET,
        .mpeg           = SAA7134_MPEG_DVB,
        .gpiomask    = 0 << 21,
        .inputs = {{
            .name   = name_tv,
            .vmux   = 1,
            .amux   = TV,
            .tv     = 1,
        },{
            .name   = name_comp1,
            .vmux   = 3,
            .amux   = LINE1,
        },{
            .name   = name_svideo,
            .vmux   = 8,
            .amux   = LINE1,
        }},
        .radio = {
            .name   = name_radio,
            .amux   = TV,
            .gpio    = 0x00000000,
        },
    },
I still can't tune to a radio cahnnel.
I'm not sure how to switch to radio, I'm not even sure which gpio it's on.
I assume it's on gpio21, but I've tried various configs - none work.

Regards,
Tim

_______________________________________________
linux-dvb mailing list
linux-dvb@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb