[PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[Marek Belisko]

bq2700 slave driver depends on bq27x00-battery driver thus adding this
dependency to config. When bq2700 is build as module bq27x00-battery
will be loaded automatically.

Signed-off-by: Marek Belisko <marek@goldelico.com>
---
 drivers/w1/slaves/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/w1/slaves/Kconfig b/drivers/w1/slaves/Kconfig
index cfe74d0..15c230a 100644
--- a/drivers/w1/slaves/Kconfig
+++ b/drivers/w1/slaves/Kconfig
@@ -128,6 +128,7 @@ config W1_SLAVE_DS28E04
 
 config W1_SLAVE_BQ27000
 	tristate "BQ27000 slave support"
+	depends on BATTERY_BQ27X00_PLATFORM
 	help
 	  Say Y here if you want to use a hdq
 	  bq27000 slave support.
-- 
1.9.1

Re: [PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[Evgeniy Polyakov]

HI Marek

28.04.2015, 23:25, "Marek Belisko" <marek@goldelico.com>:
> bq2700 slave driver depends on bq27x00-battery driver thus adding this
> dependency to config. When bq2700 is build as module bq27x00-battery
> will be loaded automatically.
>
> Signed-off-by: Marek Belisko <marek@goldelico.com>

Patch looks good to me, but you should add appropriate maintainer to copy,
since I know nothing about bq27000 device.

Re: [PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[Belisko Marek]

+CC Sebastian

On Thu, Apr 30, 2015 at 3:30 PM, Evgeniy Polyakov <zbr@ioremap.net> wrote:
> HI Marek
>
> 28.04.2015, 23:25, "Marek Belisko" <marek@goldelico.com>:
>> bq2700 slave driver depends on bq27x00-battery driver thus adding this
>> dependency to config. When bq2700 is build as module bq27x00-battery
>> will be loaded automatically.
>>
>> Signed-off-by: Marek Belisko <marek@goldelico.com>
>
> Patch looks good to me, but you should add appropriate maintainer to copy,
> since I know nothing about bq27000 device.

Sebastian could you please add your Ack to this change or comment if
you disagree? Thanks.

BR,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com

Re: [PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[Belisko Marek]

Sebastian ping. Thanks.

On Thu, Apr 30, 2015 at 3:37 PM, Belisko Marek <marek.belisko@gmail.com> wrote:
> +CC Sebastian
>
> On Thu, Apr 30, 2015 at 3:30 PM, Evgeniy Polyakov <zbr@ioremap.net> wrote:
>> HI Marek
>>
>> 28.04.2015, 23:25, "Marek Belisko" <marek@goldelico.com>:
>>> bq2700 slave driver depends on bq27x00-battery driver thus adding this
>>> dependency to config. When bq2700 is build as module bq27x00-battery
>>> will be loaded automatically.
>>>
>>> Signed-off-by: Marek Belisko <marek@goldelico.com>
>>
>> Patch looks good to me, but you should add appropriate maintainer to copy,
>> since I know nothing about bq27000 device.
>
> Sebastian could you please add your Ack to this change or comment if
> you disagree? Thanks.
>
> BR,
>
> marek
>
> --
> as simple and primitive as possible
> -------------------------------------------------
> Marek Belisko - OPEN-NANDRA
> Freelance Developer
>
> Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
> Tel: +421 915 052 184
> skype: marekwhite
> twitter: #opennandra
> web: http://open-nandra.com

BR,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com

Re: [PATCH] w1: slaves: Add dependency for W1_SLAVE_BQ27000

[Sebastian Reichel]

[-- Attachment #1: Type: text/plain, Size: 985 bytes --]

Hi,

On Fri, May 15, 2015 at 09:57:38PM +0200, Belisko Marek wrote:
> 
> On Thu, Apr 30, 2015 at 3:37 PM, Belisko Marek <marek.belisko@gmail.com> wrote:
> > +CC Sebastian
> >
> > On Thu, Apr 30, 2015 at 3:30 PM, Evgeniy Polyakov <zbr@ioremap.net> wrote:
> >> HI Marek
> >>
> >> 28.04.2015, 23:25, "Marek Belisko" <marek@goldelico.com>:
> >>> bq2700 slave driver depends on bq27x00-battery driver thus adding this
> >>> dependency to config. When bq2700 is build as module bq27x00-battery
> >>> will be loaded automatically.
> >>>
> >>> Signed-off-by: Marek Belisko <marek@goldelico.com>
> >>
> >> Patch looks good to me, but you should add appropriate maintainer to copy,
> >> since I know nothing about bq27000 device.
> >
> > Sebastian could you please add your Ack to this change or comment if
> > you disagree? Thanks.
>
> Sebastian ping. Thanks.

Sorry, did not noticed this thread so far:

Acked-By: Sebastian Reichel <sre@kernel.org>

-- Sebastian

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]