>>
>> I've tried to slove the problem by looking in the source code of the
>> driver and the datasheets. Of course, no luck to get the stereo radio
>> work.
>> I would like Gred to give me some hint then I can have some try... if
>> you haven't enough of time...
>> I still don't understand where is the problem when everything seem
>> follow the datasheet!
>
> No, the current msp3400.c does not even try to decode stereo fm radio,
> essentially is sets both channels to 10.7 MHZ, whereas the 2nd should
> be at 10.738.
>
> So I still fail to understand why this patch was reported by the tester
> as No-Fix:
>
> --- bttv-0.7.104/driver/msp3400.c-orig Thu Feb 13 23:15:45 2003
> +++ bttv-0.7.104/driver/msp3400.c Thu Feb 13 23:21:43 2003
> @@ -274,8 +274,8 @@
>
> /* FM Radio */
> { { -8, -8, 4, 6, 78, 107 }, { -8, -8, 4, 6, 78, 107 },
> - MSP_CARRIER(10.7), MSP_CARRIER(10.7),
> - 0x00d0, 0x0480, 0x0020, 0x3000 },
> + MSP_CARRIER(10.7), MSP_CARRIER(10.738),
> + 0x00d0, 0x0480, 0x0020, 0x3002 },
>
> /* Terrestial FM-mono + FM-stereo */
> { { 3, 18, 27, 48, 66, 72 }, { 3, 18, 27, 48, 66, 72 },
> @@ -1201,7 +1201,7 @@
> short programming enables carrier mute */
> msp3400c_setmode(client,MSP_MODE_FM_RADIO);
> msp3400c_setcarrier(client, MSP_CARRIER(10.7),
> - MSP_CARRIER(10.7));
> + MSP_CARRIER(10.738));
> /* scart routing */
> msp3400c_set_scart(client,SCART_IN2,0);
> #if 0
> @@ -1492,7 +1492,7 @@
> } else {
> /* set msp3400 to FM radio mode */
> msp3400c_setmode(client,MSP_MODE_FM_RADIO);
> - msp3400c_setcarrier(client,
> MSP_CARRIER(10.7),MSP_CARRIER(10.7)); +
> msp3400c_setcarrier(client, MSP_CARRIER(10.7),MSP_CARRIER(10.738));
> msp3400c_setvolume(client,msp->muted,msp->left,msp->right);> }
> if (msp->active)
>
For me, the patch fail to fix the stereo radio problem.
> Anyway you could implement FM radio as recommended in
> "MSP 34xyG, MSP 44xyG Recommendations for FM Radio Application with TV
> Tuner", ("mspg_fm_radio_application_2an.pdf"). This uses ASS mode
> (automatic sound select) which is available with newer msp chips.
>
I have tried to use the ASS mode.. but nothing help too...
Last time, I suspect the problem is releated to tda9887. However, nothing
could help after reading the data sheet and modify the source code...
Chris
