[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170725134827.wxlkoebcwh5ycnk3@earth>
Date: Tue, 25 Jul 2017 15:48:27 +0200
From: Sebastian Reichel <sebastian.reichel@...labora.co.uk>
To: Evgeniy Polyakov <zbr@...emap.net>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx
from w1 to power/supply
Hi Greg & Evgeniy,
Here is a signed immutable branch for Andrew's patch, which moves
bq27000 w1 driver to the power-supply subsystem. I guess git will
figure everything out without this, but better safe than sorry :)
The following changes since commit 520eccdfe187591a51ea9ab4c1a024ae4d0f68d9:
Linux 4.13-rc2 (2017-07-23 16:15:17 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-power-supply.git tags/ib-w1-bq27000-4.14
for you to fetch changes up to 55a9db679183bcf85a6e5c44a4f92f158bb6f03d:
power: supply: move HDQ interface for bq27xxx from w1 to power/supply (2017-07-25 15:17:39 +0200)
----------------------------------------------------------------
immutable branch moving bq2700 driver from w1 to power-supply
----------------------------------------------------------------
Andrew F. Davis (1):
power: supply: move HDQ interface for bq27xxx from w1 to power/supply
drivers/power/supply/Kconfig | 9 +++++++++
drivers/power/supply/Makefile | 1 +
.../slaves/w1_bq27000.c => power/supply/bq27xxx_battery_hdq.c} | 2 --
drivers/w1/slaves/Kconfig | 6 ------
drivers/w1/slaves/Makefile | 1 -
5 files changed, 10 insertions(+), 9 deletions(-)
rename drivers/{w1/slaves/w1_bq27000.c => power/supply/bq27xxx_battery_hdq.c} (98%)
-- Sebastian
On Tue, Jul 25, 2017 at 03:30:22PM +0200, Sebastian Reichel wrote:
> Hi,
>
> On Wed, Jul 19, 2017 at 12:04:06PM -0500, Andrew F. Davis wrote:
> > The HDQ interface driver should be in this folder just like the I2C
> > interface driver. Move this driver out of drivers/w1/slave and into
> > drivers/power/supply.
> >
> > Signed-off-by: Andrew F. Davis <afd@...com>
> > Acked-by: Pali Rohár <pali.rohar@...il.com>
> > Acked-by: Sebastian Reichel <sre@...nel.org>
>
> Thanks, queued to power-supply's for-next branch. I prepared a
> immutable branch/tag for the w1 tree and will send a pull request
> for that in a minute.
>
> -- Sebastian
>
> > drivers/power/supply/Kconfig | 9 +++++++++
> > drivers/power/supply/Makefile | 1 +
> > .../slaves/w1_bq27000.c => power/supply/bq27xxx_battery_hdq.c} | 2 --
> > drivers/w1/slaves/Kconfig | 6 ------
> > drivers/w1/slaves/Makefile | 1 -
> > 5 files changed, 10 insertions(+), 9 deletions(-)
> > rename drivers/{w1/slaves/w1_bq27000.c => power/supply/bq27xxx_battery_hdq.c} (98%)
> >
> > diff --git a/drivers/power/supply/Kconfig b/drivers/power/supply/Kconfig
> > index 969f5005669c..af6397224a82 100644
> > --- a/drivers/power/supply/Kconfig
> > +++ b/drivers/power/supply/Kconfig
> > @@ -198,6 +198,15 @@ config BATTERY_BQ27XXX_I2C
> > Say Y here to enable support for batteries with BQ27xxx chips
> > connected over an I2C bus.
> >
> > +config BATTERY_BQ27XXX_HDQ
> > + tristate "BQ27xxx HDQ support"
> > + depends on BATTERY_BQ27XXX
> > + depends on W1
> > + default y
> > + help
> > + Say Y here to enable support for batteries with BQ27xxx chips
> > + connected over an HDQ bus.
> > +
> > config BATTERY_BQ27XXX_DT_UPDATES_NVM
> > bool "BQ27xxx support for update of NVM/flash data memory"
> > depends on BATTERY_BQ27XXX_I2C
> > diff --git a/drivers/power/supply/Makefile b/drivers/power/supply/Makefile
> > index a41f40957847..946a5e6be860 100644
> > --- a/drivers/power/supply/Makefile
> > +++ b/drivers/power/supply/Makefile
> > @@ -38,6 +38,7 @@ obj-$(CONFIG_BATTERY_SBS) += sbs-battery.o
> > obj-$(CONFIG_CHARGER_SBS) += sbs-charger.o
> > obj-$(CONFIG_BATTERY_BQ27XXX) += bq27xxx_battery.o
> > obj-$(CONFIG_BATTERY_BQ27XXX_I2C) += bq27xxx_battery_i2c.o
> > +obj-$(CONFIG_BATTERY_BQ27XXX_HDQ) += bq27xxx_battery_hdq.o
> > obj-$(CONFIG_BATTERY_DA9030) += da9030_battery.o
> > obj-$(CONFIG_BATTERY_DA9052) += da9052-battery.o
> > obj-$(CONFIG_CHARGER_DA9150) += da9150-charger.o
> > diff --git a/drivers/w1/slaves/w1_bq27000.c b/drivers/power/supply/bq27xxx_battery_hdq.c
> > similarity index 98%
> > rename from drivers/w1/slaves/w1_bq27000.c
> > rename to drivers/power/supply/bq27xxx_battery_hdq.c
> > index 8046ac45381a..f4df67eb9d2c 100644
> > --- a/drivers/w1/slaves/w1_bq27000.c
> > +++ b/drivers/power/supply/bq27xxx_battery_hdq.c
> > @@ -1,6 +1,4 @@
> > /*
> > - * drivers/w1/slaves/w1_bq27000.c
> > - *
> > * Copyright (C) 2007 Texas Instruments, Inc.
> > *
> > * This file is licensed under the terms of the GNU General Public License
> > diff --git a/drivers/w1/slaves/Kconfig b/drivers/w1/slaves/Kconfig
> > index fb68465908f2..dc4437683956 100644
> > --- a/drivers/w1/slaves/Kconfig
> > +++ b/drivers/w1/slaves/Kconfig
> > @@ -140,10 +140,4 @@ config W1_SLAVE_DS28E04
> >
> > If you are unsure, say N.
> >
> > -config W1_SLAVE_BQ27000
> > - tristate "BQ27000 slave support"
> > - help
> > - Say Y here if you want to use a hdq
> > - bq27000 slave support.
> > -
> > endmenu
> > diff --git a/drivers/w1/slaves/Makefile b/drivers/w1/slaves/Makefile
> > index 54c63e420302..e59441a5e157 100644
> > --- a/drivers/w1/slaves/Makefile
> > +++ b/drivers/w1/slaves/Makefile
> > @@ -15,5 +15,4 @@ obj-$(CONFIG_W1_SLAVE_DS2438) += w1_ds2438.o
> > obj-$(CONFIG_W1_SLAVE_DS2760) += w1_ds2760.o
> > obj-$(CONFIG_W1_SLAVE_DS2780) += w1_ds2780.o
> > obj-$(CONFIG_W1_SLAVE_DS2781) += w1_ds2781.o
> > -obj-$(CONFIG_W1_SLAVE_BQ27000) += w1_bq27000.o
> > obj-$(CONFIG_W1_SLAVE_DS28E04) += w1_ds28e04.o
> > --
> > 2.13.0
> >
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists