From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759588AbaIOWw2 (ORCPT ); Mon, 15 Sep 2014 18:52:28 -0400 Received: from mail-ig0-f171.google.com ([209.85.213.171]:46024 "EHLO mail-ig0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754065AbaIOWwY (ORCPT ); Mon, 15 Sep 2014 18:52:24 -0400 Date: Mon, 15 Sep 2014 23:52:20 +0100 From: Lee Jones To: Krzysztof Kozlowski Cc: Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Samuel Ortiz , linux-kernel@vger.kernel.org, linux-api@vger.kernel.org, linux-pm@vger.kernel.org, Liam Girdwood , Mark Brown , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , devicetree@vger.kernel.org, Andrew Morton Subject: Re: [PATCH v6 0/8] charger/mfd: max14577: Add support for MAX77836 Message-ID: <20140915225220.GH25162@lee--X1> References: <1410504840-23543-1-git-send-email-k.kozlowski@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1410504840-23543-1-git-send-email-k.kozlowski@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 12 Sep 2014, Krzysztof Kozlowski wrote: > This is a sixth version of patches adding support for > MAX77836 device to the max14577 drivers. > > This patchset has been on the lists for quite long time and was already > reviewed by all necessary maintainers except power supply. > I need acks from power supply subsystem (patches: 3, 5-8). Give this set a few more days for the Power guys to respond. If they don't, ping me and I will just apply the set. > Generally this should be pulled at once but some parts may be split > and pulled separately in following batches: > - Patches 1 and 2: as MFD prerequisites, > - Patches 3-5: adding actual charger code, depends on 1 and 2, > - Patch 6: fuel-gauge, independent, > - Patches 7 and 8: only documentation. > > Lee Jones said he can take the set through his tree. Still I need acks > from power subsystem. > > Everything rebased on Linus' tree (v3.17-rc2-9-g68e370289c29). > > Changes since v5 > ================ > 1. 5/8: Remove 'inline' from functions (suggested by Andrew Morton). > 2. Add Andrew Morton's acks. > > Changes since v4 > ================ > 1. Add Lee Jones' acks to patch 1 and 2 (mfd). > 2. power max17040 6/8: Remove duplicated (and not used) value of id > in array of i2c_device_id. (suggested by Lee Jones) > > Changes since v3 > ================ > 1. Split MFD changes into separate new patches: > 1/8: mfd: max14577: Add defines for MAX77836 charger > 2/8: Map charger device to its own of_node > Suggested by Lee Jones. > > Changes since v2 > ================ > 1. charger: Use sysfs instead of DTS for setting the fast charge timer. > The charger driver now selects the CONFIG_SYSFS and exports > a DEVICE_ATTR. (suggested by Mark Rutland) > 2. Add patch 6 with documentation of exported sysfs entry for fast > charge timer. > 3. charger 3/6: Add missing 'break' in switch parsing valid values > for fast charge timer. > > Changes since v1 > ================ > 1. charger 3/5: Add an error message for each unsuccessful parse of DT > property (suggested by Mark Rutland). > 2. charger 3/5: Use 'u32' type for storing values from DT (suggested > by Mark Rutland). > 3. charger 3/5: Remove an error message for memory allocation failure. > > > The patchset (first and second part of the MAX77836 drivers) has been > on the lists since January. Changelog for the first part of the drivers > (merged): > https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg628696.html > > > Best regards, > Krzysztof Kozlowski > > > Krzysztof Kozlowski (8): > mfd: max14577: Add defines for MAX77836 charger > mfd: max14577: Map charger device to its own of_node > charger: max14577: Add support for MAX77836 charger > regulator/mfd: max14577: Export symbols for calculating charger > current > charger: max14577: Configure battery-dependent settings from DTS and > sysfs > power: max17040: Add ID for MAX77836 Fuel Gauge block > devicetree: mfd: max14577: Add device tree bindings document > Documentation: charger: max14577: Document exported sysfs entry > > Documentation/ABI/testing/sysfs-class-power | 14 + > Documentation/devicetree/bindings/mfd/max14577.txt | 146 ++++++++ > drivers/mfd/max14577.c | 100 +++++- > drivers/power/Kconfig | 5 +- > drivers/power/max14577_charger.c | 370 +++++++++++++++++++-- > drivers/power/max17040_battery.c | 3 +- > drivers/regulator/max14577.c | 80 +---- > include/linux/mfd/max14577-private.h | 95 ++++-- > include/linux/mfd/max14577.h | 30 ++ > 9 files changed, 704 insertions(+), 139 deletions(-) > create mode 100644 Documentation/devicetree/bindings/mfd/max14577.txt > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog