From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935265Ab3FSWKn (ORCPT ); Wed, 19 Jun 2013 18:10:43 -0400 Received: from avon.wwwdotorg.org ([70.85.31.133]:49679 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935070Ab3FSWKm (ORCPT ); Wed, 19 Jun 2013 18:10:42 -0400 Message-ID: <51C22C5F.5020402@wwwdotorg.org> Date: Wed, 19 Jun 2013 16:10:39 -0600 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 MIME-Version: 1.0 To: =?UTF-8?B?SGVpa28gU3TDvGJuZXI=?= CC: Linus Walleij , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Grant Likely , rob.herring@calxeda.com, devicetree-discuss@lists.ozlabs.org, James Hogan , Laurent Pinchart Subject: Re: [PATCH 2/5] pinctrl: clarify some dt pinconfig options References: <201306141741.46077.heiko@sntech.de> <201306141742.49923.heiko@sntech.de> In-Reply-To: <201306141742.49923.heiko@sntech.de> X-Enigmail-Version: 1.4.6 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/14/2013 09:42 AM, Heiko Stübner wrote: > The bias-pull-* options use values > 0 to indicate that the pull should > be activated and optionally also indicate the strength of the pull. > Therefore use an default value of 1 for these options. > > Split the low-power-mode option into low-power-enable and -disable. > > Update the documentation to describe the param arguments better. > > Wrong default options > Reported-by: James Hogan > That blank line should be before the Reported-by not after it. > Signed-off-by: Heiko Stuebner > diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt b/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt > -low-power-mode - low power mode > +low-power-enable - enable low power mode > +low-power-disable - disable low power mode Hmmm. That's changing the binding definition. What if somebody already wrote their device tree according previous definition? It seems to be that tri-states are preferable for pinctrl DT: no entry: do nothing = 0: disable = 1: enable > +Arguments for parameters: > + > +- bias-pull-up, -down and -pin-default take as optional argument 0 to disable > + the pull, on hardware supporting it the pull strength in Ohm. bias-disable > + will also disable any active pull. Does this agree with the latest definition of the kernel-internal meaning of 0 for pull-up/down? > +- input-schmitt takes as argument the adjustable hysteresis in a > + driver-specific format > + > +- input-debounce takes the debounce time as argument or 0 to disable debouncing > + > +- power-source argument is the custom value describing the source to select > + > +- slew-rate takes as argument the target rate in a driver-specific format If those things have driver-specific (note: should be DT-binding-specific, not driver-specific) values, then I'm not convinced that having a generic parameter name for them is a good idea; it makes things look the same when they aren't. By forcing each binding to include the vendor prefix on those properties and hence define a custom property name, you're making it clear that the semantics may be different.