From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752580AbcF2IUI (ORCPT ); Wed, 29 Jun 2016 04:20:08 -0400 Received: from mail.kmu-office.ch ([178.209.48.109]:47903 "EHLO mail.kmu-office.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751438AbcF2IUA (ORCPT ); Wed, 29 Jun 2016 04:20:00 -0400 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Date: Wed, 29 Jun 2016 01:15:36 -0700 From: Stefan Agner To: Lee Jones Cc: lgirdwood@gmail.com, broonie@kernel.org, khilman@baylibre.com, carlo@caione.org, b.galvani@gmail.com, max.oss.09@gmail.com, marcel@ziswiler.com, linux@armlinux.org.uk, galak@codeaurora.org, ijc+devicetree@hellion.org.uk, mark.rutland@arm.com, pawel.moll@arm.com, robh+dt@kernel.org, linux@roeck-us.net, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org Subject: Re: [PATCH v5 0/5] regulator: add Ricoh RN5T567 PMIC support In-Reply-To: <20160629081208.GI29166@dell> References: <20160629055322.23038-1-stefan@agner.ch> <20160629081208.GI29166@dell> Message-ID: User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016-06-29 01:12, Lee Jones wrote: > On Tue, 28 Jun 2016, Stefan Agner wrote: > >> This patchset adds RN5T567 PMIC support which is used on the >> Toradex Colibri iMX7S/iMX7D modules. The existing RN5T618 is from >> the same family, hence this patchset uses the same driver and adds >> variant support. >> >> The Colibris currently do not use the PMIC's power off capabilities, >> as do the current users of that PMIC driver. Therefore this patchset >> also makes the use of the system-power-controller property mandatory >> if the power off capabilties are required. >> >> Changes since v4: >> - Spell check in patch 5 and added Acks >> >> Changes since v3: >> - Use RN5T618_REPCNT_REPWRON macro when writing RN5T618_REPCNT register >> >> Changes since v2: >> - Elevate restart priority to 192 to make sure PMIC takes presedence >> over SoC level restart capabilities (e.g. watchdog) >> - Print a warning if poweroff callback is already assigned >> - Reorder of_id and variant assignment >> - Lower delay to 1ms and justify why it is required >> - Fix DCDC count and a typos in device tree bindings >> >> Changes since v1: >> - Removed obsolete include >> >> Stefan Agner (5): >> ARM: dts: meson: minix-neo-x8: define PMIC as power controller >> mfd: add Ricoh RN5T567 PMIC support >> regulator: rn5t618: add RN5T567 PMIC support >> mfd: rn5t618: register power off callback optionally >> mfd: rn5t618: register restart handler >> >> Documentation/devicetree/bindings/mfd/rn5t618.txt | 19 +++--- >> arch/arm/boot/dts/meson8-minix-neo-x8.dts | 1 + >> drivers/mfd/Kconfig | 7 ++- >> drivers/mfd/rn5t618.c | 70 +++++++++++++++++++---- >> drivers/regulator/Kconfig | 5 +- >> drivers/regulator/rn5t618-regulator.c | 40 +++++++++++-- >> include/linux/mfd/rn5t618.h | 13 +++++ >> 7 files changed, 126 insertions(+), 29 deletions(-) > > Looks like you have all your Acks now. How do you want to play this? > Would you like me to take the set through MFD and provide the other > Maintainers with a branch to pull from, or did you have other ideas? I did not thought about that really, I guess you have a usual way for mixed regulator/mfd patchsets? Taking it through MFD sounds good to me... -- Stefan