DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=date:from:to:cc:subject:message-id:references:mime-version
         :content-type:content-disposition:in-reply-to:user-agent;
        b=GOBpEww/DnGXQrSv2yGa6KMS3xgy8Z0Pauc+a9myaW7HRtcEYQChJI9kOUcEIitczh
         aboQMH/ZwKnxuhbxvbeIAPaqksiGP9PmyK6t1QXSZS+0FEuYrAn0Q6zPYNKwa3BfTQSV
         oh+uiiHlE1dTpo1eGQWl9fWNREtZOVmfNmrC8=
Date: Mon, 28 Feb 2011 17:04:46 +0300
From: Anton Vorontsov <cbouatmailru@gmail.com>
To: Rhyland Klein <rklein@nvidia.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] power: bq20z75: change to platform driver
Message-ID: <20110228140446.GA28312@oksana.dev.rtsoft.ru>
References: <1298591301-29999-1-git-send-email-rklein@nvidia.com>
 <1298594977.30301.1.camel@rklein-linux2>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <1298594977.30301.1.camel@rklein-linux2>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1502
Lines: 44

On Thu, Feb 24, 2011 at 04:49:37PM -0800, Rhyland Klein wrote:
> On Thu, 2011-02-24 at 15:48 -0800, Rhyland Klein wrote:
> > From: Rhyland Klein <rklein@nvidia.com>
> > 
> > This change switches the bq20z75 i2c driver into a platform driver
> > and adds support for an optional battery detect gpio. Now the driver also
> > signals when the battery is inserted or removed.
> > 
> > I also added a retry mechanism so that for boards where there may be
> > intermittent i2c issues, retries can be used to avoid spurious failures.
> > 
> > Signed-off-by: Rhyland Klein <rklein@nvidia.com>
> > ---
[...]
> Anton, are you going to have a chance to review this? I have one more
> patch to submit but I was waiting for this to go in first.

Sorry for the delay.

The main problem with this patch is that it does too many things.
It should be separated into three patches:

1. Convert the driver into a platform driver;
2. Add support for detect gpio;
3. Add retry machanism.

But still, why are you converting the i2c driver into a platform driver?
To pass the GPIOs via platform data? You can do this with I2C driver as
well:

struct i2c_board_info {
        ....
        void            *platform_data;

Thanks,

-- 
Anton Vorontsov
Email: cbouatmailru@gmail.com
--
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/