| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-id: <1397112697.6717.2.camel@AMDC1943> Date: Thu, 10 Apr 2014 08:51:37 +0200 From: Krzysztof Kozlowski <k.kozlowski@...sung.com> To: Chanwoo Choi <cw00.choi@...sung.com> Cc: Samuel Ortiz <sameo@...ux.intel.com>, Lee Jones <lee.jones@...aro.org>, Mark Brown <broonie@...nel.org>, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, Dmitry Eremin-Solenikov <dbaryshkov@...il.com>, David Woodhouse <dwmw2@...radead.org>, Marek Szyprowski <m.szyprowski@...sung.com>, Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>, Kyungmin Park <kyungmin.park@...sung.com>, Tomasz Figa <t.figa@...sung.com> Subject: Re: [PATCH v4 00/16] mfd: max14577: Add support for MAX77836 On czw, 2014-04-10 at 13:28 +0900, Chanwoo Choi wrote: > Hi Krzysztof, > > Did you finish this patchset for applying git repository? > I can't find this patchset on mainline kernel. > > If this patchset isn't applied on mainline, I'd like you to finish this patch > because I want to maintain up to date patch for max77836/max14577 on mainline. Hi, The patches didn't get to mainline. I couldn't obtain all needed acks. I prepared another version of the patchset (with less cross-subsystem renames) which I plan to send after 3.15 merge window. Best regards, Krzysztof > Thanks, > Chanwoo Choi > > On 02/24/2014 07:10 PM, Krzysztof Kozlowski wrote: > > Hi, > > > > > > This is fourth version of patchset adding support for MAX77836 device > > to the max14577 driver. > > > > Lee Jones suggested that after obtaining acks from all maintainers he can > > merged these patches to his tree. These patches touch different subsystems > > (charger/power, regulator, extcon and mfd) so this would be probably the best > > way to merge everything. I would be glad if I could get missing acks > > (or comments). > > > > > > Changes since v3 > > ================ > > 1. Applied minor fixes (pointed by Lee Jones). > > 2. Added one ACK (Lee Jones) and Review-by (Tomasz Figa). > > 3. Patch 14/charger: Minor change in parsing EOC value from DTS. > > 4. Rebased on next-20140224. > > > > Changes since v2 > > ================ > > 1. Added ACK-s. > > 2. Applied minor checkpatch fixes (pointed by Lee Jones). > > 3. Rebased on next-20140217. > > > > Changes since v1 > > ================ > > 1. Added ACK-s, reviews and tested-by tags. > > 2. Removed applied patches (they were merged to the linux-next tree). > > 3. Applied comments from review (Lee Jones) to 5/15 (detection of device type) > > and 8/15 (add max77836 support to max14577). > > 4. Rebased on next tree. > > 5. Added patch 13 and 14 (pointed by Jenny Tc): > > - regulator/mfd: max14577: Export symbols for calculating charger current > > - charger: max14577: Configure battery-dependent settings from DTS > > 6. Updated bindings documentation with new charger bindings. > > Previous thread: > > - [PATCH 00/18] mfd: max14577: Add support for MAX77836 > > http://thread.gmane.org/gmane.linux.kernel/1636654/focus=1636674 > > > > > > Description > > =========== > > The MAX77836 is almost the same as MAX14577. Basically it is an extended > > version of MAX14577. The chipsets have same MUIC component so the extcon, > > charger and regulators require only little adjustments. More changes were > > needed in the charger (because it uses lower charging current) and regulator > > drivers (because mentioned different charing current and additional > > regulators). The MAX77836 has also PMIC and Fuel Gauge (which is the same > > as MAX17040/17048 Fuel Gauge). > > > > The MAX77836 uses three I2C slave addresses and has additional interrupts > > (related to PMIC and Fuel Gauge). It has also Interrupt Source register, > > just like MAX77686 and MAX77693. > > > > The patchset is organized in following way: > > 1. Patches from 1 to 7 prepare the driver for supporting MAX77836. They > > mostly rename symbols and add additional logic related to different > > device types. > > 2. Patches from 8 to 12 add support for MAX77836. > > 3. Patches 13 and 14 remove hard-coded charger configuration with DTS. They > > were added after review on mailing lists. > > 4. Patch 15 adds bindings documentation, which was not merged previously. > > 5. Patch 16 adds SUSPEND mode to the regulator driver. > > > > > > The patchset is based on linux-next tree and depends on patches from previous > > release which are already merged into next. > > Testing was done on 3.10. > > > > > > TODO: Merge max14577, max77686 and max77693 into one common driver. They are > > very similar. > > > > > > Best regards, > > Krzysztof Kozlowski > > > > > > Chanwoo Choi (1): > > extcon: max14577: Change extcon name instead of static name according > > to device type > > > > Krzysztof Kozlowski (15): > > mfd: max14577: Rename and add MAX14577 symbols to prepare for > > max77836 > > mfd: max14577: Rename state container to maxim_core > > mfd: max14577: Add "muic" suffix to regmap and irq_chip > > mfd: max14577: Add detection of device type > > extcon: max14577: Add max14577 prefix to muic_irqs > > extcon: max14577: Choose muic_irqs according to device type > > mfd: max77836: Add max77836 support to max14577 driver > > extcon: max14577: Add support for max77836 > > regulator: max14577: Add support for max77836 regulators > > charger: max14577: Add support for MAX77836 charger > > power: max17040: Add ID for max77836 Fuel Gauge block > > regulator/mfd: max14577: Export symbols for calculating charger > > current > > charger: max14577: Configure battery-dependent settings from DTS > > mfd: max14577: Add device tree bindings document > > regulator: max14577: Implement SUSPEND mode for MAX77836 LDO-s > > > > Documentation/devicetree/bindings/mfd/max14577.txt | 149 +++++ > > drivers/extcon/extcon-max14577.c | 274 +++++---- > > drivers/mfd/max14577.c | 475 +++++++++++++--- > > drivers/power/max14577_charger.c | 361 +++++++++--- > > drivers/power/max17040_battery.c | 1 + > > drivers/regulator/max14577.c | 334 ++++++++--- > > include/linux/mfd/max14577-private.h | 586 ++++++++++++-------- > > include/linux/mfd/max14577.h | 50 +- > > 8 files changed, 1697 insertions(+), 533 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/mfd/max14577.txt > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists