> Hello > > On Tue, Mar 10, 2009 at 3:20 AM, Mauro Carvalho Chehab > <mchehab@xxxxxxxxxxxxx> wrote: >> On Mon, 9 Mar 2009 23:33:38 +0100 >> Tobias Lorenz <tobias.lorenz@xxxxxxx> 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 > > > -- 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
