Jean Delvare wrote:
> On Tue, 21 Oct 2008 15:55:26 +0100, Mark Jackson wrote:
>> This patch adds support for the (I2C based) DS28CM00 ID chip.
>> (v2 after running through checkpatch)
>>
>> Signed-off-by: M.Jackson <[EMAIL PROTECTED]> 
>> ---
>>  drivers/i2c/chips/Kconfig    |    7 ++
>>  drivers/i2c/chips/Makefile   |    1 +
>>  drivers/i2c/chips/ds28cm00.c |  166 
>> ++++++++++++++++++++++++++++++++++++++++++
>>  3 files changed, 174 insertions(+), 0 deletions(-)
> 
> Nack. No new drivers under drivers/i2c/chips please, it's going away
> soon.

Okay ... where should any new drivers be located ?

<...snip...>

>> +static int ds28cm00_attach_adapter(struct i2c_adapter *adapter);
>> +static int ds28cm00_detect(struct i2c_adapter *adapter, int address, int 
>> kind);
>> +static int ds28cm00_detach_client(struct i2c_client *client);
>> +
>> +/* This is the driver that will be inserted */
>> +static struct i2c_driver ds28cm00_driver = {
>> +    .driver = {
>> +            .name   = "ds28cm00",
>> +    },
>> +    .attach_adapter = ds28cm00_attach_adapter,
>> +    .detach_client  = ds28cm00_detach_client,
>> +};
> 
> attach_adapter and detach_client are deprecated. Please use probe and
> remove instead.

Are there any existing chip drivers that use this new method ?

Thanks
Mark

_______________________________________________
i2c mailing list
i2c@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/i2c

Reply via email to