Re: [PATCH 3/5] tm6000: add audio mode parameter

[Stefan Ringel <stefan.ringel@arcor.de>]

Am 03.05.2011 16:52, schrieb Stefan Ringel:
> Am 03.05.2011 16:42, schrieb Mauro Carvalho Chehab:
>> Em 03-05-2011 11:25, Stefan Ringel arcor escreveu:
>>> Am 03.05.2011 12:07, schrieb Mauro Carvalho Chehab:
>>>> Em 20-04-2011 05:36, Stefan Ringel escreveu:
>>>>> Am 19.04.2011 23:26, schrieb Mauro Carvalho Chehab:
>>>>>> Em 04-04-2011 17:18, stefan.ringel@arcor.de escreveu:
>>>>>>> From: Stefan Ringel<stefan.ringel@arcor.de>
>>>>>>>
>>>>>>> add audio mode parameter
>>>>>> Why we need a parameter for it? It should be determined based on
>>>>>> the standard.
>>>>>>
>>>>> tm6010 has a sif decoder, and I think if auto detect doesn't work, 
>>>>> use can set the audio standard, which it has in your region. Or 
>>>>> it's better if users can see image but can hear audio?
>>>> I did some tests with SIF and MTS here. None of them were capable 
>>>> of working with BTSC signals with
>>>> my devices. Adding a parameter won't help it at all. What we need 
>>>> to do is to fix the audio
>>>> decoding.
>>>>
>>> In the next patch I will send it. A preview I have send to be test (
>>> https://patchwork.kernel.org/patch/722021/ ).
>> I tested your preview. Didn't make any difference.
> Has you test with all setting variants? (BG_A2 works auto and A2 audio 
> mode).
I use mplayer, and have in the first use the wrong parameter. Now I know 
the right parameter for mplayer:

mplayer -vc rawyuy2 -ac pcm -ao alsa:device=hw=0.0 -tv 
driver=v4l2:device=/dev/video2:input=0:outfmt=yv12:freq=210.25:normid=3:alsa:adevice=hw.2,0:amode=1:immediatemode=0 
-fs tv://
>>>>>>> Signed-off-by: Stefan Ringel<stefan.ringel@arcor.de>
>>>>>>> ---
>>>>>>>    drivers/staging/tm6000/tm6000-stds.c |    5 +++++
>>>>>>>    1 files changed, 5 insertions(+), 0 deletions(-)
>>>>>>>
>>>>>>> diff --git a/drivers/staging/tm6000/tm6000-stds.c 
>>>>>>> b/drivers/staging/tm6000/tm6000-stds.c
>>>>>>> index da3e51b..a9e1921 100644
>>>>>>> --- a/drivers/staging/tm6000/tm6000-stds.c
>>>>>>> +++ b/drivers/staging/tm6000/tm6000-stds.c
>>>>>>> @@ -22,12 +22,17 @@
>>>>>>>    #include "tm6000.h"
>>>>>>>    #include "tm6000-regs.h"
>>>>>>>
>>>>>>> +static unsigned int tm6010_a_mode;
>>>>>>> +module_param(tm6010_a_mode, int, 0644);
>>>>>>> +MODULE_PARM_DESC(tm6010_a_mode, "set sif audio mode (tm6010 
>>>>>>> only)");
>>>>>>> +
>>>>>>>    struct tm6000_reg_settings {
>>>>>>>        unsigned char req;
>>>>>>>        unsigned char reg;
>>>>>>>        unsigned char value;
>>>>>>>    };
>>>>>>>
>>>>>>> +/* must be updated */
>>>>>>>    enum tm6000_audio_std {
>>>>>>>        BG_NICAM,
>>>>>>>        BTSC,
>>>>> -- 
>>>>> To unsubscribe from this list: send the line "unsubscribe 
>>>>> linux-media" in
>>>>> the body of a message to majordomo@vger.kernel.org
>>>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> -- 
>> To unsubscribe from this list: send the line "unsubscribe 
>> linux-media" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html