linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Anton Vorontsov <anton@enomsg.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	David Woodhouse <dwmw2@infradead.org>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	linux-kernel@vger.kernel.org
Subject: [GIT PULL] battery-2.6.git
Date: Sat, 25 May 2013 15:23:54 -0700	[thread overview]
Message-ID: <20130525222354.GA29486@teo> (raw)

Hello Linus,

Please pull battery-2.6 git tree to get a few one-liners: wrong kfree
usage fix, module alias fixup and kconfig adjustments. The patches really
small, so for convenience I inline them here.

Thanks!

Anton

The following changes since commit 6b170807cd5cac8dc6353f47a88ccb14bbf76c4f:

  pm2301-charger: Fix suspend/resume (2013-04-16 19:00:01 -0700)

are available in the git repository at:

  git://git.infradead.org/battery-2.6.git tags/for-v3.10-fixes

for you to fetch changes up to dccab6092d3c25bf943d12fb658e63fd88bf8b4a:

  pm2301_charger: Fix module alias prefix (2013-05-10 11:57:14 -0700)

----------------------------------------------------------------
Axel Lin (2):
      wm831x_backup: Fix wrong kfree call for devdata->backup.name
      pm2301_charger: Fix module alias prefix

Randy Dunlap (1):
      lp8788-charger: Fix kconfig dependency

Xiong Zhou (1):
      bq27x00: Fix I2C dependency in KConfig

 drivers/power/Kconfig          | 2 ++
 drivers/power/pm2301_charger.c | 2 +-
 drivers/power/wm831x_backup.c  | 1 -
 3 files changed, 3 insertions(+), 2 deletions(-)


commit dccab6092d3c25bf943d12fb658e63fd88bf8b4a
Author: Axel Lin <axel.lin@ingics.com>
Date:   Sat May 4 18:51:09 2013 +0800

    pm2301_charger: Fix module alias prefix
    
    This driver is a i2c driver, use "i2c" rather than "platform" prefix for
    module alias.
    
    Signed-off-by: Axel Lin <axel.lin@ingics.com>
    Signed-off-by: Anton Vorontsov <anton@enomsg.org>

diff --git a/drivers/power/pm2301_charger.c b/drivers/power/pm2301_charger.c
index f123f3c..e9c6ba0 100644
--- a/drivers/power/pm2301_charger.c
+++ b/drivers/power/pm2301_charger.c
@@ -1269,5 +1269,5 @@ module_exit(pm2xxx_charger_exit);
 
 MODULE_LICENSE("GPL v2");
 MODULE_AUTHOR("Rajkumar kasirajan, Olivier Launay");
-MODULE_ALIAS("platform:pm2xxx-charger");
+MODULE_ALIAS("i2c:pm2xxx-charger");
 MODULE_DESCRIPTION("PM2xxx charger management driver");

commit c909fc8573af3cff9184551e79cf37784b5ddc24
Author: Axel Lin <axel.lin@ingics.com>
Date:   Sat May 4 13:57:55 2013 +0800

    wm831x_backup: Fix wrong kfree call for devdata->backup.name
    
    devdata->backup.name points to devdata->name, the memory for devdata->name
    is part of struct wm831x_backup. Thus remove kfree call for
    devdata->backup.name.
    
    Signed-off-by: Axel Lin <axel.lin@ingics.com>
    Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
    Signed-off-by: Anton Vorontsov <anton@enomsg.org>

diff --git a/drivers/power/wm831x_backup.c b/drivers/power/wm831x_backup.c
index 58cbb00..56fb509 100644
--- a/drivers/power/wm831x_backup.c
+++ b/drivers/power/wm831x_backup.c
@@ -207,7 +207,6 @@ static int wm831x_backup_remove(struct platform_device *pdev)
 	struct wm831x_backup *devdata = platform_get_drvdata(pdev);
 
 	power_supply_unregister(&devdata->backup);
-	kfree(devdata->backup.name);
 
 	return 0;
 }

commit a2d0dbb4b55681874c5f288538ae55ae69baeaff
Author: Xiong Zhou <jencce.kernel@gmail.com>
Date:   Tue May 7 10:15:56 2013 +0800

    bq27x00: Fix I2C dependency in KConfig
    
    This patch fixes build failure(randconfig) of next-20130501. When config
    I2C as m, BATTERY_BQ27x00 as y, here comes the failure. The driver depends
    on I2C only if I2C is not disabled, as Lars commented. Last version of
    this patch make the driver depend on I2C unconditionally.
    
    Failure message:
    drivers/built-in.o: In function `bq27x00_read_i2c':
    bq27x00_battery.c:(.text+0x1082a7): undefined reference to `i2c_transfer'
    drivers/built-in.o: In function `bq27x00_battery_init':
    bq27x00_battery.c:(.init.text+0x6085): undefined reference to `i2c_register_driver'
    bq27x00_battery.c:(.init.text+0x60c7): undefined reference to `i2c_del_driver'
    drivers/built-in.o: In function `bq27x00_battery_exit':
    bq27x00_battery.c:(.exit.text+0xbf0): undefined reference to `i2c_del_driver'
    make: *** [vmlinux] Error 1
    
    Signed-off-by: Xiong Zhou <jencce.kernel@gmail.com>
    Cc: Lars-Peter Clausen <lars@metafoo.de>
    Signed-off-by: Anton Vorontsov <anton@enomsg.org>

diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig
index 814bcb9..674e633 100644
--- a/drivers/power/Kconfig
+++ b/drivers/power/Kconfig
@@ -152,6 +152,7 @@ config BATTERY_SBS
 
 config BATTERY_BQ27x00
 	tristate "BQ27x00 battery driver"
+	depends on I2C || I2C=n
 	help
 	  Say Y here to enable support for batteries with BQ27x00 (I2C/HDQ) chips.
 

commit 237a1b01fdb29df6a28d50d6dbe7a988c4fb3625
Author: Randy Dunlap <rdunlap@infradead.org>
Date:   Wed May 1 12:18:43 2013 -0700

    lp8788-charger: Fix kconfig dependency
    
    Fix build errors in lp8788-charger by making it depend on IIO.
    Fixes errors when CONFIG_IIO=m and CHARGER_LP8788=y.
    
    lp8788-charger.c:(.text+0x2146b5): undefined reference to `iio_channel_get'
    lp8788-charger.c:(.text+0x2146ce): undefined reference to `iio_channel_get'
    lp8788-charger.c:(.text+0x214a86): undefined reference to `iio_read_channel_processed'
    lp8788-charger.c:(.text+0x214b51): undefined reference to `iio_read_channel_processed'
    lp8788-charger.c:(.text+0x214c30): undefined reference to `iio_read_channel_processed'
    lp8788-charger.c:(.text+0x214d93): undefined reference to `iio_channel_release'
    lp8788-charger.c:(.text+0x214dac): undefined reference to `iio_channel_release'
    
    Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
    Acked-by: Milo Kim <milo.kim@ti.com>
    Signed-off-by: Anton Vorontsov <anton@enomsg.org>

diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig
index 339f802..814bcb9 100644
--- a/drivers/power/Kconfig
+++ b/drivers/power/Kconfig
@@ -284,6 +284,7 @@ config CHARGER_LP8788
 	tristate "TI LP8788 charger driver"
 	depends on MFD_LP8788
 	depends on LP8788_ADC
+	depends on IIO
 	help
 	  Say Y to enable support for the LP8788 linear charger.
 

             reply	other threads:[~2013-05-25 23:12 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-25 22:23 Anton Vorontsov [this message]
2013-05-25 23:23 ` [GIT PULL] battery-2.6.git David Woodhouse
2013-05-25 22:39   ` Anton Vorontsov
  -- strict thread matches above, loose matches on Subject: below --
2014-01-21 17:54 Dmitry Eremin-Solenikov
2014-01-03  9:27 Anton Vorontsov
2013-11-18  2:08 Anton Vorontsov
2013-09-11  5:24 Anton Vorontsov
2013-07-09  1:39 Anton Vorontsov
2013-04-30  5:14 Anton Vorontsov
2013-02-19  7:09 Anton Vorontsov
2012-12-12 10:22 Anton Vorontsov
2012-10-05  3:04 Anton Vorontsov
2012-07-31 13:05 Anton Vorontsov
2012-05-31  1:08 Anton Vorontsov
2012-05-31  1:17 ` Stephen Rothwell
2012-05-31 19:13 ` Linus Torvalds
2012-03-30 20:50 Anton Vorontsov
2012-01-11  0:31 Anton Vorontsov
2011-07-31 10:40 Anton Vorontsov
2011-05-27 10:54 Anton Vorontsov
2011-05-27 11:10 ` David Woodhouse
2011-05-27 12:45   ` Anton Vorontsov
2011-03-24 14:58 Anton Vorontsov
2011-01-14 15:43 Anton Vorontsov
2010-10-26 14:41 Anton Vorontsov
2010-08-11  5:17 Anton Vorontsov
2010-05-24 22:28 [Regression] Commit "power_supply: Use attribute groups" breaks KDE battery monitor on openSUSE 11.3 M6 Rafael J. Wysocki
2010-05-25  0:20 ` Rafael J. Wysocki
2010-05-25  0:47   ` Daniel Mack
2010-05-25  0:55     ` Rafael J. Wysocki
2010-05-25  7:08       ` Daniel Mack
2010-05-25 10:27         ` [GIT PULL] battery-2.6.git Anton Vorontsov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130525222354.GA29486@teo \
    --to=anton@enomsg.org \
    --cc=akpm@linux-foundation.org \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sfr@canb.auug.org.au \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).