From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966317AbcIWPOa (ORCPT ); Fri, 23 Sep 2016 11:14:30 -0400 Received: from mail-oi0-f67.google.com ([209.85.218.67]:35483 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965725AbcIWPO2 (ORCPT ); Fri, 23 Sep 2016 11:14:28 -0400 Date: Fri, 23 Sep 2016 10:14:26 -0500 From: Rob Herring To: Matthias Kaehlcke Cc: Mark Brown , lgirdwood@gmail.com, Douglas Anderson , briannorris@chromium.org, javier@dowhile0.org, mark.rutland@arm.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v5 5/6] regulator: core: Add support for a fixed delay after voltage changes Message-ID: <20160923151426.GA13456@rob-hp-laptop> References: <1473871930-99603-1-git-send-email-mka@chromium.org> <1473871930-99603-5-git-send-email-mka@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1473871930-99603-5-git-send-email-mka@chromium.org> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 14, 2016 at 09:52:09AM -0700, Matthias Kaehlcke wrote: > The target voltage is not necessarily reached inmediately after > requesting a regulator to change the voltage. In some cases the > ramp_delay can be used to calculate the stabilisation time, in others > there is no direct relationship between the delta in the voltage and > the stabilisation time. This change introduces the device tree properties > "regulator-settle-time-up-us"/"regulator-settle-time-down-us" which > allow to specify a fixed delay after a voltage increase or decrease. > > Signed-off-by: Matthias Kaehlcke > --- > Changes in v5: > - Added support for delay on voltage decreases > - Don't skip set_voltage_time op if no settle time and ramp delay > > .../devicetree/bindings/regulator/regulator.txt | 4 ++++ Acked-by: Rob Herring > drivers/regulator/core.c | 18 +++++++++++------- > drivers/regulator/of_regulator.c | 8 ++++++++ > include/linux/regulator/machine.h | 4 ++++ > 4 files changed, 27 insertions(+), 7 deletions(-)