Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752867AbcCGQe3 (ORCPT ); Mon, 7 Mar 2016 11:34:29 -0500 Received: from mail-vk0-f50.google.com ([209.85.213.50]:34641 "EHLO mail-vk0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752656AbcCGQeV (ORCPT ); Mon, 7 Mar 2016 11:34:21 -0500 MIME-Version: 1.0 In-Reply-To: References: <20151110173416.GB21727@ulmo> <20160125170855.GA10182@ulmo> <20160203145337.GD9650@ulmo> <20160222175929.GA23899@ulmo> <20160223143859.GA27656@ulmo> <20160223181448.GA14754@ulmo.nvidia.com> Date: Mon, 7 Mar 2016 08:34:19 -0800 Message-ID: Subject: Re: [PATCH v3 00/12] pwm: add support for atomic update From: Doug Anderson To: Thierry Reding Cc: Boris Brezillon , =?UTF-8?Q?Heiko_St=C3=BCbner?= , linux-pwm , Mark Brown , Liam Girdwood , Jingoo Han , Lee Jones , linux-fbdev@vger.kernel.org, Bryan Wu , Richard Purdie , Jacek Anaszewski , linux-leds@vger.kernel.org, Maxime Ripard , linux-sunxi@googlegroups.com, "open list:ARM/Rockchip SoC..." , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , Daniel Mack , Haojian Zhuang , Robert Jarzmik , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Olof Johansson Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 943 Lines: 29 Thierry, On Thu, Feb 25, 2016 at 3:14 PM, Doug Anderson wrote: > So just to summarize: > > * Add pwm_get_state(), pwm_apply_state(), pwm_get_args(). > pwm_get_state() initially returns 0 for duty cycle if driver doesn't > support readout. > > * Re-implement pwm_get_period() (and maybe other similar functions) > atop pwm_get_state() as you describe earlier in the thread. > > * Document pwm_get_period() (and maybe other similar functions) as deprecated. > > * Fix drivers for all current 2 users of PWM regulator to support > hardware readout. > > * Update PWM regulator as you described earlier in the thread (Feb 23). > > * If PWM regulator is ever used on a new board whose PWM doesn't > support hardware readout, the voltage will change at probe time. > > > Did I get all that right? Thanks! Can you provide a "yes, you got that right" or a "no, you didn't understand"? That will unblock Boris, I think. -Doug