Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-08-27 Thread Evgeniy Polyakov 
Hi everyone

Sorry for long delay

25.07.2017, 16:48, "Sebastian Reichel" :
> 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)

Greg, should it be pulled into your tree or submitted as a separate pull 
request or as set of patches?
Either way looks good to me, please get it into your tree.

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-08-27 Thread Evgeniy Polyakov 
Hi everyone

Sorry for long delay

25.07.2017, 16:48, "Sebastian Reichel" :
> 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)

Greg, should it be pulled into your tree or submitted as a separate pull 
request or as set of patches?
Either way looks good to me, please get it into your tree.

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-25 Thread Sebastian Reichel 
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 
> > Acked-by: Pali Rohár 
> > Acked-by: Sebastian Reichel 
> 
> 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

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-25 Thread Sebastian Reichel 
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 
> > Acked-by: Pali Rohár 
> > Acked-by: Sebastian Reichel 
> 
> 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
> > -

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-25 Thread Sebastian Reichel 
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 
> Acked-by: Pali Rohár 
> Acked-by: Sebastian Reichel 

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
> 


signature.asc
Description: PGP signature

Re: [PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-25 Thread Sebastian Reichel 
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 
> Acked-by: Pali Rohár 
> Acked-by: Sebastian Reichel 

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
> 


signature.asc
Description: PGP signature

[PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-19 Thread Andrew F. Davis 
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 
Acked-by: Pali Rohár 
Acked-by: Sebastian Reichel 
---
 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

[PATCH v2 1/2] power: supply: move HDQ interface for bq27xxx from w1 to power/supply

2017-07-19 Thread Andrew F. Davis 
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 
Acked-by: Pali Rohár 
Acked-by: Sebastian Reichel 
---
 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