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 ? -- 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
