From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933477AbcLTSIX (ORCPT ); Tue, 20 Dec 2016 13:08:23 -0500 Received: from mail-ve1eur01on0124.outbound.protection.outlook.com ([104.47.1.124]:38144 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752210AbcLTSIT (ORCPT ); Tue, 20 Dec 2016 13:08:19 -0500 X-Greylist: delayed 10822 seconds by postgrey-1.27 at vger.kernel.org; Tue, 20 Dec 2016 13:08:19 EST Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=peda@axentia.se; From: Peter Rosin To: linux-kernel@vger.kernel.org Cc: Peter Rosin , Sebastian Reichel , linux-pm@vger.kernel.org Subject: [PATCH 0/3] power: supply: bq24735: handle AC adapter absence Date: Tue, 20 Dec 2016 12:32:59 +0100 Message-Id: <1482233582-29770-1-git-send-email-peda@axentia.se> X-Mailer: git-send-email 2.1.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [217.210.101.82] X-ClientProxiedBy: DB5PR0101CA0002.eurprd01.prod.exchangelabs.com (10.165.200.140) To DB6PR0201MB2312.eurprd02.prod.outlook.com (10.169.222.151) X-MS-Office365-Filtering-Correlation-Id: 7148c38d-bd59-4c39-1bbb-08d428cbfe1e X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:DB6PR0201MB2312; X-Microsoft-Exchange-Diagnostics: 1;DB6PR0201MB2312;3:SpKShLv2iNf1grPBfuQreNf+HQjA1hCJ2NmsWPRAsunMCNVQpE0/Hi9yRFjxIwS6LO/YwklgE7qnFGxbklrRyt2cRDE6J7/Oix6WyDHYPTiDaTkEDV5LrluaJJMib0gSVlGKVQ7pzFd58ycxD3SWaXFuyHtGb/rq/dhjFhb9UwnY8SUlo7nZYyf79uOGRcR38ih0rr7KIVNenwR0FQOLekare9oogYyG+YrBdUWUQXPXAg8rDMglvJoRv0qe8yWZ6cV7Phjxe/vWSq2K6aYZkQ==;25:Dz4nvVtGjLmjIx0H3qQnJp1je+XJlVWk6O9TfwS12d11K5wbJUTjb3L8Iuj5X9rqxux1bDk828vloX30Ne1bdq9R4n88cGpIz1Y56y57jVk67cI0vbaGgejL7/c3N5+yD1Vi54oaNcyeVeMD1Db4lA/Zgyf9PejXajvdgzMpXZvK5R5fwc+zeX1V1W4AdXu71F0e9vnPApKx3BEF1ZtQySfGHero6IkOrUy97xUCaSmxDORWrQRvNdztQKifq+ARe8RHi/+PhSUnJyLmBDL4xYfm7qyqPQyLuk9M7zJwKRKvECVTYpvbVGAkyN5g4xKyvVOrtlPv9z8NBwIwN0F7v8cc2VYRAlOwwIgxzy13NUfZcIaAVCHd59cu3RKq7VxET0ov79xvBKJjSMQx2muGmiysr9/vxneGvW3nlUOsYJeYtgE1GIurxJ8T5ycwDPlaQlvQn7QrguOW78zOCDMCcA== X-Microsoft-Exchange-Diagnostics: 1;DB6PR0201MB2312;31:K5XED9hf3u4UiLztMKpXq507OqyIRNMK3gtbmjxm0nnNcrFpm7Zrw5Z1cQKjjV6SgA0e+p42LYAT0DO79tRLiGUPTAkCxfxJ6/vbGkXlnGKA/1doiFY39A2j7/vGj/Ucr6WSr1R1KV2EQE/NAlNDZCADZSehxlZ5jF3lyc1jwuUGK8dG/pvu8Exs6VGQTfSkg5TINVydfyP6MMsThnTD5MSkJLGIpQNYzXPZVf9N4WH1CcOcCkIe0Av2qYrRCNu8lP5Naf+5wJuQ28YL63Ukhg== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(211171220733660); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6041248)(20161123562025)(20161123564025)(2016111802025)(20161123560025)(20161123555025)(6072148)(6043046);SRVR:DB6PR0201MB2312;BCL:0;PCL:0;RULEID:;SRVR:DB6PR0201MB2312; X-Microsoft-Exchange-Diagnostics: 1;DB6PR0201MB2312;4:o1LUTO5pBfj3KObJL4iDgcr5SjGmbm7mVCfOtHvF0z57f56y1JD/e5PNapx/h9mtviwVhP7cjbnnPycsigjdZlbd+mP+b2GeoODvJWLtI70YwMW+ISpXeJHRgP77nkukvlIKh31J2P0rHh0dluv+RsOr5T09NNw2r4y2bYcKnxZ2HJE/p85XsTp+tHca6LsdEsVaTzcP4dSE9vwjQcaeda8h1Ug6vP8TMU8m51C9u3y17phcQLpVovA6wOuqGa8HEVMgxkuVsZXWhewqwlqzJ43eatwdnAskGwShyODOBL4OGM/PSFNfHm9ZBkMXLYY0Mqk7KAnMJ7ow1wRQE4GyXIQqViFLMg/1YdpCRrgRUPVEcb8xQEcdqB8YuaTm8/hfg+F+zgBQTI3YziQ1ceBY36Ya5KLFBkgbj53vyrIMw7Wyfohrn+VE/YNZ34yRrTPQm3N/fV6k9cMrIHUNDJ/W046sS3yP269X4cqrBGnarbSaL1U3L4K4EeBpjuJHNkb+Mkn+GnXtwODt6KarU5u7LZpBocuUDARYUEkCElxnwmjvk+BKi/f8jhXfeTHRP5Y/jrXmTkmJGqC0ppWPI9L27/foMfMv3/yZzQv6C5l1s6SEdOd9Z2cPZLituJGY6JjlnwqPqBdcr8CtX6R9gffaBFhHXZOY89B2H8X3l0tpDRk= X-Forefront-PRVS: 0162ACCC24 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6069001)(6009001)(7916002)(39830400002)(39450400003)(39410400002)(189002)(199003)(6916009)(86362001)(2351001)(97736004)(6486002)(2906002)(6506006)(101416001)(6512006)(2361001)(92566002)(7736002)(25786008)(305945005)(50226002)(68736007)(38730400001)(5890100001)(106356001)(8676002)(42186005)(81166006)(48376002)(50466002)(74482002)(36756003)(5660300001)(4326007)(33646002)(81156014)(50986999)(66066001)(3846002)(189998001)(6666003)(110136003)(5003940100001)(6116002)(47776003)(105586002)(42262002);DIR:OUT;SFP:1102;SCL:1;SRVR:DB6PR0201MB2312;H:localhost.localdomain;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DB6PR0201MB2312;23:U2eltqRlpRScafR1YKhOH9fuLsCQIKoamPkR322?= =?us-ascii?Q?oD3ZjVYKmB8PMmyfLugyw4RxDlzjgRLsGgWBl4O1HaP8xm0eeD7J960KN+YL?= =?us-ascii?Q?nDzYIH2VL6s2A7vNeppr7LOfjsOhvRW2p+cK30UpsgHX+R4q/w4QZDO000Jj?= =?us-ascii?Q?YfdqQYZcXaUEBHHGBGmY0v6cBiUUZQ9wPWzMRrTm2N66sP5yBk1hnbd2DK2m?= =?us-ascii?Q?cA6bXeKmHHEgp1jg0w/U5Z8umi2QXhv2/I2H+dS7e1R0wzA9u0Ot9XnSDWg4?= =?us-ascii?Q?NnLVV/WP4Cf/kERnCBcCaQglTMDbdfg4ix6Mwoap4xZh1aeMNwg1CM3atot6?= =?us-ascii?Q?rvXvUiKa3yshahxrJ9jj0Qa2YmcfTdlQAsf5wo5ZoCBwNG96iEPlcH+l/NHc?= =?us-ascii?Q?zThra2yn2eceAL7Cew0RnaWsOoXGHmsEHLVPyTMSW+UTV02JbHJJ0DNmtYiR?= =?us-ascii?Q?vYymhpSuNrlWnPnOhXCBkHnI+Zw6r8lP+DuN8WHIwDd+agBvE6mcxEe4ijC6?= =?us-ascii?Q?Bv4d/+NBtQc98B/QMHweYrAiG9CQe5v8kuERNSFokT3sZsM2b6HNqntl1S03?= =?us-ascii?Q?h+qDEAP1yEjAcQ5KhmIQE/XKHfgbkXgpFwM9IYNsmVpmgoUStOrX92/yDeAl?= =?us-ascii?Q?dEKrkP+Q2bxfjqIGQZoJS/M40u2PQ7xTv6+0AGvTyNcRPRbt9znd85n3HsoL?= =?us-ascii?Q?WdWqnc8y5ONKEiNg1eMTs3iirfQe51fxjBu2QaYcaOuzbWl/eJFrd2bsJsXC?= =?us-ascii?Q?ajX440RRJPiT257HFR3mYgvVJjHwO2tBjBX0vELgUbfaM+pWrsYVJoiwXZBN?= =?us-ascii?Q?M6iskj+1oBClGrO75gItt81sJCJ/26jFjrDQ7krYn/mYOc6Ccjvy2LWWBIrO?= =?us-ascii?Q?Z4uldtmneg8wnZTdHkH0yqk3wZsxQTsyW5W5KZKe4tvu3hAR0EyDL6JmZEd8?= =?us-ascii?Q?y3J/xtZpwF7o0Hz2mHNt2uSvoYJS0yHf3Y2DHIQczGLzaEIVKw3u4xiVf42J?= =?us-ascii?Q?DjVL++RoxHBOR44JOFSAUhB2T/bmP3L/ZXIDnVYgPI4Bxw1KkVbOgOwJH39Y?= =?us-ascii?Q?Zj5IdD0xZgxPdOsP5RKhVqT9jD2G9LaVDHf8BXZaxcw1CZ2gi94wfzQ3g5gy?= =?us-ascii?Q?OC1RyopMxRFU4bUcQH5Q2gtC+P8Sl+hSmh0tRBVRtChljbLaLQucmFubsCgJ?= =?us-ascii?Q?oHyCucoRchpQll1i3LN3az58/HkSrDDF3KLeb?= X-Microsoft-Exchange-Diagnostics: 1;DB6PR0201MB2312;6:DOFA64gyt3LJCfOK0uA7gPJpmurc6SXVhZm8L0afVZloVS/zZ44cfZ47AK4zqEii/Wd34MaysbQ6A4kz0N3UjBQLLp983MFiueJG9OuapN9SFsn9u4vCaTIR1CcnwHcfxcb3Xd7iEDy/aR46tSBtm4VtlEpz2kQlvqCOqnwJ72MkFC016XWBVZeUYybKU1kEw6SFv5jELM+uVZlqMVEJHL5ATE4+plgkOwFpZi7KOFHDvvqzMNq3E4KsjhKPQxvmpHkyZAwyIHDkQsQ6Y0Shhe1X6wNGunvOKKfwmQXwFuioNaBXiqoRWRgWhRmJ3C8zE79MYS6ZhxaV59CQM6Q7mSCOH2HANwwIhH1iWnTqJPUGixWUhor81VRUra9LwRpEJxl3dXJGhRDG7SkAx5uO8RFiSUqmAt+bo0v/Sxll9yK0Fg/m+X6BNspNe8cBiYgK;5:zNqgkH43wThbsZQ32yo/PTCHeTp7WRqkYfS2JWWBOM/1i43h7QVN72L+s1+hCWWzm1WrzGMj7JRY8kL88HoaUAJ6auPCKeROxACIWUWA+CCqSoME0vHuiGE+ToRmU/juXvt3BkGuBNFugeL3tGZVRw==;24:VujYajKxyzQvel96miAJJH9oTW8jEx1anpWJRKtDF4FJ2aMt4cOZfMPD0MaQ4GJjlYc6egkoqTpTJxf0DB8KC+Rms7lSVm/CBjKOtmKzuUM= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DB6PR0201MB2312;7:D3Jv6dn76BBo98RhN7oKc/zzpqqTgEKCb9WN1X1lxx4sZUlu4Q/QhC+nRVv+kG031Hx1BizmbVJDDA42EEwHqjgM4dokSchbk+TRXcsspvpDIy92JFaHGo+Bh15BDH1PqZtmdsghoQO5HkEHoqdvWBKtHcf1vFXyfRGizr1y/Jck3/uA47+JcQqQsVnnK8u2L51G36fSX4pwD+KZ+SwG1t4PNZ8UNICpp47E69520tKoU0iKmzWj3RF74VwR9FXTZQr+vlcNQ37qY+ICU5Ec3xuGZXqfs7ZiTNhx8cd17NLVULCvRWrFQNak7hEz3YIZU+b1QoRwAYKgBf+Pi4Ig04U3S8rWZoXna5u9giSO7dcLszf06nFO8g1wXjYAg3/ipQpP08AZRfZ9AAJxEdnbd5t8FM8gUnyIstFBdYI3LgTPO30vELKojQWt8k3nyKPUtF87UwZGJ8oA2pDgjoYGNA== X-OriginatorOrg: axentia.se X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Dec 2016 11:33:16.6475 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0201MB2312 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi! There is provision in the driver for handling the case where the chargers are not responding to i2c requests when the AC adapter is absent. I think this has been used by some Tegra boards that leave the charger handling to some other component and that only wants the kernel to be aware of the charging status for reporting (or whatever). However, in my case, the kernel should handle configuration and enable/disable the charging, but this is not working very well when the charger disappears (and is reset) on AC adapter absence. This series fixes the issues I have found in this area so that I can boot without AC adapter and have the correct charging config every time I do attach the AC adapter. Cheers, Peter Peter Rosin (3): power: supply: bq24735: move down bq24735_{en,dis}able_charging power: supply: bq24735: configure the charger as part of enabling it power: supply: bq24735: always check for AC adapter presence in probe drivers/power/supply/bq24735-charger.c | 56 ++++++++++++++++------------------ 1 file changed, 26 insertions(+), 30 deletions(-) -- 2.1.4