lkml.org 
[lkml]   [2010]   [Sep]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ds2760_battery: add HAS_IOMEM dependency
On Thu, Sep 23, 2010 at 03:21:36PM +0200, Martin Schwidefsky wrote:
[...]
> > I think the better fix would be to turn this into 'depends on W1',
> > and remove these selects.
>
> That would work as well. Dunno what is better, W1 is a bus for a certain
> class of devices. The bus is only needed if at least one driver that uses
> it is configured. Which would argue for the select method but that has
> the dependency problem. Anyway, either method is fine with me. If you
> prefer the "depends on W1" lets do it that way.
> --
> [PATCH] ds2760_battery: fix W1 and W1_SLAVE_DS2760 dependency
>
> From: Martin Schwidefsky <schwidefsky@de.ibm.com>
>
> Fix this Kconfig warning on allmodconfig for s390:
>
> warning: (BATTERY_DS2760 && POWER_SUPPLY) selects W1 which has unmet direct dependencies (HAS_IOMEM)
>
> Cc: Anton Vorontsov <cbou@mail.ru>
> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
> ---

Applied, thanks!

> drivers/power/Kconfig | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff -urpN linux-2.6/drivers/power/Kconfig linux-2.6-patched/drivers/power/Kconfig
> --- linux-2.6/drivers/power/Kconfig 2010-09-23 15:18:00.000000000 +0200
> +++ linux-2.6-patched/drivers/power/Kconfig 2010-09-23 15:18:46.000000000 +0200
> @@ -64,8 +64,7 @@ config TEST_POWER
>
> config BATTERY_DS2760
> tristate "DS2760 battery driver (HP iPAQ & others)"
> - select W1
> - select W1_SLAVE_DS2760
> + depends on W1 && W1_SLAVE_DS2760
> help
> Say Y here to enable support for batteries with ds2760 chip.
>
>
> --
> blue skies,
> Martin.
>
> "Reality continues to ruin my life." - Calvin.

--
Anton Vorontsov
email: cbouatmailru@gmail.com
irc://irc.freenode.net/bd2


\
 
 \ /
  Last update: 2010-09-27 20:09    [W:0.269 / U:0.280 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site