Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753549Ab2HTLer (ORCPT ); Mon, 20 Aug 2012 07:34:47 -0400 Received: from moutng.kundenserver.de ([212.227.126.186]:65255 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751243Ab2HTLek (ORCPT ); Mon, 20 Aug 2012 07:34:40 -0400 Date: Mon, 20 Aug 2012 13:34:23 +0200 From: Thierry Reding To: Bill Huang Cc: sameo@linux.intel.com, grant.likely@secretlab.ca, rob.herring@calxeda.com, rob@landley.net, broonie@opensource.wolfsonmicro.com, ldewangan@nvidia.com, swarren@wwwdotorg.org, xxie@nvidia.com, lrg@slimlogic.co.uk, jhovold@gmail.com, kyle.manna@fuel7.com, rklein@nvidia.com, devicetree-discuss@lists.ozlabs.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/2] mfd: dt: tps6586x: Add power off control Message-ID: <20120820113423.GA5996@avionic-0098.mockup.avionic-design.de> References: <1345424876-4365-1-git-send-email-bilhuang@nvidia.com> <1345424876-4365-2-git-send-email-bilhuang@nvidia.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="bp/iNruPH9dso1Pn" Content-Disposition: inline In-Reply-To: <1345424876-4365-2-git-send-email-bilhuang@nvidia.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Provags-ID: V02:K0:ZKIEuGuUypwnL2xPS1ca3JvOa2Xe12OE8ZYIktOZN8f OD0v+bsnx3tIDMrh6frvvqH0kTymQ39OPtLJRF2OIY99yqRXer LUqolnGz2fOWwvize/8dUBUvqUsfvBiShKgCbCcxWozSACVrq1 dT716BNeE8dage20K+7dFmTqThTVDxUAUCPJ94WPZYNCo5Itbu 3vp81urIy8OQQLLjSslu6Na39nH6EDTsvZelbdGqvsPdGbX+D1 cIMtvhugd/VFERds+wEUJz0STeGj/hIcozyZj3i8a8nfPa5AwF 31JZ9fUAQH7lZm7wo5K0t0t91wL+QX8fh8/A6E5kD6Z0/qNzlJ aFXVbu/ziELrSH8bzy3zobqDOOoEWVQ46+n3uYhn4KkG4eWbIZ A+k82d0purRYM2evDfvCIjMClDvlQ85ggI= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4448 Lines: 104 --bp/iNruPH9dso1Pn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Aug 19, 2012 at 06:07:55PM -0700, Bill Huang wrote: > Add DT property "ti,system-power-controller" telling whether or not this > pmic is in charge of controlling the system power, so the power off > routine can be hooked up to system call "pm_power_off". >=20 > Based on the work by: > Dan Willemsen >=20 > Signed-off-by: Bill Huang > Tested-by: Stephen Warren > --- > .../devicetree/bindings/regulator/tps6586x.txt | 6 ++++++ > drivers/mfd/tps6586x.c | 19 ++++++++++++++= +++++ > include/linux/mfd/tps6586x.h | 1 + > 3 files changed, 26 insertions(+), 0 deletions(-) Hi, I've seen the following while trying this patch applied on top of next-2012= 0817: [ 40.581151] Power down. [ 41.583160] ------------[ cut here ]------------ [ 41.587784] WARNING: at /home/thierry.reding/src/kernel/linux-ipmp.git/d= rivers/i2c/busses/i2c-tegra.c:525 tegra_i2c_xfer+0x21c/0x29c() [ 41.599850] Modules linked in: [ 41.602927] [] (unwind_backtrace+0x0/0xf8) from [] (= warn_slowpath_common+0x4c/0x64) [ 41.612304] [] (warn_slowpath_common+0x4c/0x64) from [] (warn_slowpath_null+0x1c/0x24) [ 41.621947] [] (warn_slowpath_null+0x1c/0x24) from [= ] (tegra_i2c_xfer+0x21c/0x29c) [ 41.631244] [] (tegra_i2c_xfer+0x21c/0x29c) from [] = (__i2c_transfer+0x44/0x80) [ 41.640192] [] (__i2c_transfer+0x44/0x80) from [] (i= 2c_transfer+0x7c/0xb8) [ 41.648796] [] (i2c_transfer+0x7c/0xb8) from [] (reg= map_i2c_read+0x48/0x64) [ 41.657485] [] (regmap_i2c_read+0x48/0x64) from [] (= _regmap_raw_read+0x90/0x98) [ 41.666518] [] (_regmap_raw_read+0x90/0x98) from [] = (_regmap_read+0x50/0xa8) [ 41.675290] [] (_regmap_read+0x50/0xa8) from [] (_re= gmap_update_bits+0x24/0x64) [ 41.684322] [] (_regmap_update_bits+0x24/0x64) from [] (regmap_update_bits+0x3c/0x58) [ 41.693885] [] (regmap_update_bits+0x3c/0x58) from [= ] (tps6586x_power_off+0x18/0x38) [ 41.703362] [] (tps6586x_power_off+0x18/0x38) from [= ] (machine_power_off+0x1c/0x24) [ 41.712749] [] (machine_power_off+0x1c/0x24) from []= (sys_reboot+0x138/0x1b0) [ 41.721612] [] (sys_reboot+0x138/0x1b0) from [] (ret= _fast_syscall+0x0/0x30) [ 41.730293] ---[ end trace 9af366974fefa459 ]--- [ 41.734906] tegra-i2c tegra-i2c.3: i2c transfer timed out [ 41.740689] Kernel panic - not syncing: Attempted to kill init! exitcode= =3D0x00000000 [ 41.740689]=20 [ 41.749823] [] (unwind_backtrace+0x0/0xf8) from [] (= panic+0x8c/0x1d8) [ 41.757993] [] (panic+0x8c/0x1d8) from [] (do_exit+0= x694/0x750) [ 41.765636] [] (do_exit+0x694/0x750) from [] (do_gro= up_exit+0x3c/0xb0) [ 41.773884] [] (do_group_exit+0x3c/0xb0) from [] (__= wake_up_parent+0x0/0x18) Thierry --bp/iNruPH9dso1Pn Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (GNU/Linux) iQIcBAEBAgAGBQJQMiC/AAoJEN0jrNd/PrOhiYEQALfHXXAnQSwm5zlTloPZGPQo MnRBhNJskwHBl1Wpkn+iZvFsjvMCW6yWFsRSED9M3sH7jH8lHknKW91Uf01ZO8AR u25B+DAygmye0KVWOvV7Dp574OwMSs7op1oN/GkixVspfEoZNMaU9zOQNmsXLN4T 7KkvwMJIaFJj3RzS27YxA0h6lAcUsOLkzmY5WGZdnjbUCfr4mQ/9gYaX6NrECEFZ bSAcukhxAMuKSGdwlzOlfaXger78joount8rdIy2zL8xFtUA4FS4zrxc0ofExBH5 2nCp0+JsZjG9ty5vcuMFeiJ2tOjX37EJiZ3lEG5tjwSLSFanbMcMJBvd6lgIdydO s/p8QbeZtoMZu1jm9P/zT0Hq/RrrITEohjFl35WzFs0zDJnPJ9EmhHCBBt5H29zg kGX2m8mBATJTBjMKkSFwavJJptzfVOrslFaRLJKYOyQ6PkB62chI6PlFgMYMGbOx U7ckvOQuuir4IISoyOvrx5JZ4lsDumTpqPgY9VEy5MR0QvcqvJMX47ztC6TZWBzb WumVBOTux/gPKpeF4jM6dul1Hrb6BVgBQyP+G3AVYwdfTDCSnr2/hB3uwUR9GqDp H11Y8RjTeepZI4DnacRHX0HcGCI+z1xXX35BgLZoKSs1evXfHXazzXhM2frwv6OV FYAqfloD+nq+kDDL6Znl =XZuQ -----END PGP SIGNATURE----- --bp/iNruPH9dso1Pn-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/