Re: [PATCH v2 00/10] add linux driver for chip TLG2300

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Huang Shijie wrote:
> 
>>>> Instead of a country code, the driver should use the V4L2_STD_
>>>> macros to
>>>>
>>>>        
>>> If we are in the radio mode, I do not have any video standard, how can I
>>> choose
>>> the right audio setting in this situation?
>>>      
>> In the case of radio, the frequency ranges are controlled via the tuner
>>    
> 
> Do you mean that the frequency range can be used to set the pre-emphasis?
> I am not sure about this.

No, I don't meant that. 

The differences of FM radio standards are basically the preemphasis and the
frequency ranges.

For frequency ranges, V4L2_TUNER_RADIO allows specifying the maximum/minimum values.

For preemphasis, you should implement V4L2_CID_TUNE_PREEMPHASIS ctrl. This
CTRL has 3 states:

        static const char *tune_preemphasis[] = {
                "No preemphasis",
                "50 useconds",
                "75 useconds",
                NULL,
        };

At v4l2-common.c, there are some functions that helps to implement it
at the driver, like:
	v4l2_ctrl_get_menu, v4l2_ctrl_get_name and v4l2_ctrl_query_fill.

Take a look at si4713-i2c.c for an example on how to use it.

Ah, please submit those changes as another series of patches. This helps me
to not needing to review the entire changeset again.

Cheers,
Mauro