From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 1/8] regulator: bd71837: Disable voltage monitoring for LDO3/4 Date: Wed, 12 Sep 2018 09:42:51 +0100 Message-ID: <20180912084251.GO4185@dell> References: <20180911124029.GO4185@dell> <20180912074745.GD2381@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <20180912074745.GD2381@localhost.localdomain> Sender: linux-kernel-owner@vger.kernel.org To: Matti Vaittinen Cc: robh+dt@kernel.org, mark.rutland@arm.com, lgirdwood@gmail.com, broonie@kernel.org, mazziesaccount@gmail.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, heikki.haikola@fi.rohmeurope.com, mikko.mutanen@fi.rohmeurope.com List-Id: devicetree@vger.kernel.org On Wed, 12 Sep 2018, Matti Vaittinen wrote: > On Tue, Sep 11, 2018 at 01:40:29PM +0100, Lee Jones wrote: > > On Wed, 29 Aug 2018, Matti Vaittinen wrote: > > > > > There is a HW quirk in BD71837. The shutdown sequence timings for > > > bucks/LDOs which are enabled via register interface are changed. > > > At PMIC poweroff the voltage for BUCK6/7 is cut immediately at the > > > beginning of shut-down sequence. This causes LDO5/6 voltage > > > monitoring to detect under voltage and force PMIC to emergency > > > state instead of poweroff. Disable voltage monitoring for LDO5 and > > > LDO6 at probe to avoid this. > > > > > > Signed-off-by: Matti Vaittinen > > > --- > > > drivers/regulator/bd71837-regulator.c | 19 +++++++++++++++++++ > > > > > include/linux/mfd/rohm-bd718x7.h | 33 ++++++++++++++++++++++++++++++--- > > > > For my own reference: > > Acked-for-MFD-by: Lee Jones > > I m not sure if it is relevant but I think this was already applied to > Mark's regulator tree. Where is Mark's acceptance email? -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog