Hi,
On 21.09.2004 17:41, Greg KH wrote:
> On Mon, Sep 20, 2004 at 07:19:24PM +0200, Michael Hunold wrote:
>
>>
>>+ /* a ioctl like command that can be used to perform specific functions
>>+ * with the adapter.
>>+ */
>>+ int (*command)(struct i2c_adapter *adapter, unsigned int cmd, void *arg);
>
>
> Ick ick ick. We don't like ioctls for the very reason they aren't type
> safe, and you can pretty much stick anything in there you want. So
> let's not try to add the same type of interface to another subsystem.
Ok, Gerd Knorr and I have been discussing this and we have come up with
the following idea.
We like to have an completly isolated i2c adapter, where the device
driver can invite i2c drivers to connect an i2c client to. When the
connection is made, an "interface" pointer with client-specific data or
function pointers can be provided.
- i2c adapter and i2c clients register themselves as usual
- add a new NO_PROBE flag to struct i2c_adapter, so a particular adapter
is never probed by anyone
- add these two functions to struct i2c_driver:
int (*connect)(struct i2c_adapter *adapter, void *interface, struct
i2c_client **client);
int (*disconnect)(struct i2c_client *client);
- add new generic i2c functions:
struct i2c_driver* i2c_driver_get(char *name);
void i2c_driver_put(struct i2c_driver *drv);
- the dvb-ttpci driver now can do the following:
struct stv0299_interface {
struct dvb_adapter *dvb_adap;
int tuner_addr;
};
struct stv0299_interface s_if;
struct i2c_driver *drv;
struct i2c_client *clt;
request_module("stv0299");
drv = i2c_driver_get("stv0299");
// fill s_if here
drv->connect(adap, &s_if, &clt);
Now inside the "connect" function of the stv0299 demodulator i2c driver
the device is probed, registered to the adapter and the pointer to the
interface with the h/w dependend information is stored.
> thanks,
> greg k-h
The crucial point is probably the void * interface pointer, isn't it? Is
this a no-no in this situation?
The dvb-ttpci driver is inviting a very specific driver, no probing
involved. The driver interface is well-defined and will only be hidden
behind the void * pointer to avoid a functional reference to the
demodulator driver.
We have discussed simply adding a type-safe stv0299-specific connect
function as well, but that would introduce a functional dependency. The
dvb-ttpci driver can be used with about 8 different frontend drivers, so
loading the driver would cause unnecessary frontend drivers to be loaded
as well.
CU
Michael.
