Hello,

I am having the same kind of troubles (can't tune and mt_set_frequency 
-121 errors) since 2.6.10 (it was working in 2.6.9) on amd64.
this patch did not help sadely.

log files attached if you have time to look at them :) (the non-working 
one being 2.6.11-rc3+your patch)

Cheers,
Mik

Gerd Knorr wrote:
>>>>mt2032_set_if_freq failed with -121
>>
>>OK here you go.
> 
> 
> Thanks, seems to be a initialization order bug which changes the default
> state of the tda9887 output ports.  The patch below should fix that.
> 
>   Gerd
> 
> diff -u linux-2.6.11/drivers/media/video/tda9887.c linux/drivers/media/video/tda9887.c
> --- linux-2.6.11/drivers/media/video/tda9887.c	2005-01-25 11:57:21.000000000 +0100
> +++ linux/drivers/media/video/tda9887.c	2005-02-03 12:26:16.246003021 +0100
> @@ -305,9 +305,9 @@
>  	printk("  B5   force mute audio: %s\n",
>  	       (buf[1] & 0x20) ? "yes" : "no");
>  	printk("  B6   output port 1   : %s\n",
> -	       (buf[1] & 0x40) ? "high" : "low");
> +	       (buf[1] & 0x40) ? "high (inactive)" : "low (active)");
>  	printk("  B7   output port 2   : %s\n",
> -	       (buf[1] & 0x80) ? "high" : "low");
> +	       (buf[1] & 0x80) ? "high (inactive)" : "low (active)");
>  
>  	printk(PREFIX "write: byte C 0x%02x\n",buf[2]);
>  	printk("  C0-4 top adjustment  : %s dB\n", adjust[buf[2] & 0x1f]);
> @@ -545,9 +545,9 @@
>  	int rc;
>  
>  	memset(buf,0,sizeof(buf));
> +	tda9887_set_tvnorm(t,buf);
>  	buf[1] |= cOutputPort1Inactive;
>  	buf[1] |= cOutputPort2Inactive;
> -	tda9887_set_tvnorm(t,buf);
>  	if (UNSET != t->pinnacle_id) {
>  		tda9887_set_pinnacle(t,buf);
>  	}
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/