Re: About the radio-si470x driver for I2C interface

[Joonyoung Shim <jy0922.shim@samsung.com>]

> Hello
> 
> On Tue, Mar 10, 2009 at 3:20 AM, Mauro Carvalho Chehab
> <mchehab@infradead.org> wrote:
>> On Mon, 9 Mar 2009 23:33:38 +0100
>> Tobias Lorenz <tobias.lorenz@gmx.net> wrote:
>>
>>> Hi,
>>>
>>>> The proper way is to break radio-si470x into two parts:
>>>>
>>>>     - A i2c adapter driver (similar to what we have on cx88-i2c, for
>>>>       example, plus the radio part of cx88-video);
>>>>     - A radio driver (similar to tea5767.c).
>>>>
>>>> This way, the i2c driver can be used on designs that use a different i2c adapter.
>>> yes, this is why I already capsulated most of the USB functionality into own functions. I awaited that somewhen the si470x is used in the "usual" way by i2c.
>>>
>>> I'm not sure, if we should split the driver into three files (generic/common, usb, i2c) or just implement the new functionality within the same file using macros/defines.
>> It is better to split. It will provide more flexibility.
> 
> Tobias, Joonyoung
> 
> Is there any success on this ?

Hi,

I have tried, but could not do it yet.

Like radio-tea5764.c, radio-si470x driver for I2C interface also was
implemented as the I2C driver, but it has many redundant functions with
the existing radio-si470x for USB interface, so I tried to join two
interfaces but it was a difficult work. I have never seen the linux device
driver probing more than one interface optionally.

-- Joonyoung Shim

> 
> --
> Best regards, Klimov Alexey
> 
> 
>