Re: [PATCH 0/1] Add support for TEA5761 (from linux-omap)

[mkrufky@linuxtv.org]

Mauro Carvalho Chehab wrote:
> Hi Eduardo,
>
> On Wed, 4 Jun 2008 10:25:23 -0400
> "Eduardo Valentin" <edubezval@gmail.com> wrote:
>
>   
>> Hi Mauro and Michael,
>>
>> Thanks for pointing that there were a duplicated work. If there is any
>> update on the current driver, I'll contact you. I'm not the author of
>> this driver, but I'm interested in some points here.
>>
>> This chip is used on n800 FM radio. That's why this version came from
>> linux-omap.
>> Anyway, one quest that came from my mind, taking a brief look into
>> this two drivers,
>> I see they use different interfaces to register a FM radio driver, and
>> more they are located
>> under different places inside the tree. So, what is more recommended
>> for FM radio drivers?
>> being under drivers/media/radio/ or under drivers/media/common/tunners/ ?
>> What is the API more recommended dvb_tuner_ops or video_device ? I
>> wonder also what current applications are using.
>>     
>
> Good point. some radio tuners are used inside video boards. Before, this
were
> located inside drivers/media/video. Now, they are at common/tuners. This
seems
> to be a better place.
>
> It should be noticed that tea5761 is an I2C device. So, you'll probably
need a
> counterpart module, at media/radio, that will provice I2C access methods
needed
> on N800.
>   
Basically, what you will have to do is create a n800 driver under 
media/radio.  This n800 driver will provide the userspace interface via 
v4l2, and it will use internal tuner API to interface to the tea5761 module.

When all is said and done, most likely all of the tea5761-specific code 
would be removed from the driver that you submit -- the remaining code 
would merely handle glue between userspace and internal tuner api.

Regards,

Mike

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list