linux-kernel - Re: [i2c] [PATCH v2] Add support for Dallas DS28CM00 Unique ID chip

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20081021211219.18fb7d84@hyperion.delvare>
Date:	Tue, 21 Oct 2008 21:12:19 +0200
From:	Jean Delvare <khali@...ux-fr.org>
To:	Mark Jackson <mpfj@...c.co.uk>
Cc:	ben-linux@...ff.org, akpm@...ux-foundation.org, i2c@...sensors.org,
	lkml <linux-kernel@...r.kernel.org>
Subject: Re: [i2c] [PATCH v2] Add support for Dallas DS28CM00 Unique ID chip

Hi Mark,

On Tue, 21 Oct 2008 19:42:42 +0100, Mark Jackson wrote:
> 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 <mpfj@...c.co.uk> 
> >> ---
> >>  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 ?

Depends of what the device does.
* Hardware monitoring chip drivers go to drivers/hwmon
* RTC chip drivers go to drivers/rtc
* I/O expander drivers go to drivers/gpio
If you can't find a suitable directory under drivers, either create a
new one if that makes sense, or put your driver under drivers/misc.

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

As of 2.6.27-git10: all drivers under drivers/i2c/chips, all i2c
drivers under drivers/gpio, all i2c drivers under drivers/hwmon, all
i2c drivers under drivers/rtc, and more.

You may want to take a look at Documentation/i2c/upgrading-clients too.


-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/