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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux