Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754897AbYJUSnq (ORCPT ); Tue, 21 Oct 2008 14:43:46 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752588AbYJUSnh (ORCPT ); Tue, 21 Oct 2008 14:43:37 -0400 Received: from pih-relay06.plus.net ([212.159.14.19]:54038 "EHLO pih-relay06.plus.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751903AbYJUSng (ORCPT ); Tue, 21 Oct 2008 14:43:36 -0400 Message-ID: <48FE22A2.7030604@mimc.co.uk> Date: Tue, 21 Oct 2008 19:42:42 +0100 From: Mark Jackson Organization: Mercury IMC Ltd User-Agent: Thunderbird 2.0.0.16 (X11/20080724) MIME-Version: 1.0 To: Jean Delvare CC: ben-linux@fluff.org, akpm@linux-foundation.org, i2c@lm-sensors.org, lkml Subject: Re: [i2c] [PATCH v2] Add support for Dallas DS28CM00 Unique ID chip References: <48FDED5E.5080605@mimc.co.uk> <20081021173505.2f7673e2@hyperion.delvare> In-Reply-To: <20081021173505.2f7673e2@hyperion.delvare> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Plusnet-Relay: a082ebaac809d79b052f5c4d765076bf Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1504 Lines: 44 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 >> --- >> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/