From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755153AbcIPSb6 (ORCPT ); Fri, 16 Sep 2016 14:31:58 -0400 Received: from mail-pf0-f175.google.com ([209.85.192.175]:36695 "EHLO mail-pf0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751975AbcIPSbs (ORCPT ); Fri, 16 Sep 2016 14:31:48 -0400 Date: Fri, 16 Sep 2016 11:31:45 -0700 From: Matthias Kaehlcke To: Mark Brown Cc: lgirdwood@gmail.com, Douglas Anderson , briannorris@chromium.org, javier@dowhile0.org, robh+dt@kernel.org, mark.rutland@arm.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v4 4/4] regulator: Prevent falling too fast Message-ID: <20160916183145.GF62872@google.com> References: <20160906190524.GB79728@google.com> <20160912185633.GH27946@sirena.org.uk> <20160913172140.GC62872@google.com> <20160915143945.GJ27974@sirena.org.uk> <20160915180223.GE62872@google.com> <20160916163253.GA10189@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160916163253.GA10189@sirena.org.uk> 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 El Fri, Sep 16, 2016 at 05:32:53PM +0100 Mark Brown ha dit: > On Thu, Sep 15, 2016 at 11:02:23AM -0700, Matthias Kaehlcke wrote: > > El Thu, Sep 15, 2016 at 03:39:45PM +0100 Mark Brown ha dit: > > > > The obvious question here is how the OVP hardware knows about the new > > > voltage and why we're bodging this in the regulator core rather than in > > > the OVP hardware. > > > The OVP hardware is part of the regulator and the regulator is not > > notified directly about voltage changes. The regulator transforms the > > PWM input into DC output and does the OVP internally with the limits > > described above. > > So the PWM is just configuring this external regulator chip (which > doesn't seem to be described in DT...) Exactly > and that's just incredibly bad at coping with voltage changes? Supposedly OVP is a feature of the chip, but it gets in our way on "larger" voltage changes. > It does sound rather like we ought to be representing this chip > directly in case it needs other workarounds. Ok, we'll consider this. It seems we can drop this patch since the regulator core is not the best place to address this problem. Thanks for your reviews! Matthias