linux-kernel - Re: [PATCH v2] power: bq20z75: change to platform driver

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>] [day] [month] [year] [list]

Date:	Mon, 28 Feb 2011 17:04:46 +0300
From:	Anton Vorontsov <cbouatmailru@...il.com>
To:	Rhyland Klein <rklein@...dia.com>
Cc:	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] power: bq20z75: change to platform driver

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@...dia.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@...dia.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@...il.com
--
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/