Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751695AbbEZTmG (ORCPT ); Tue, 26 May 2015 15:42:06 -0400 Received: from avon.wwwdotorg.org ([70.85.31.133]:36250 "EHLO avon.wwwdotorg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751282AbbEZTmC (ORCPT ); Tue, 26 May 2015 15:42:02 -0400 Message-ID: <5564CC84.1030700@wwwdotorg.org> Date: Tue, 26 May 2015 13:41:56 -0600 From: Stephen Warren User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Tomeu Vizoso , Linus Walleij CC: linux-arm-kernel@lists.infradead.org, =?windows-1252?Q?St=E9phane_M?= =?windows-1252?Q?archesin?= , Thierry Reding , Dmitry Torokhov , Alexander Holler , Grant Likely , Rob Herring , Mark Rutland , Pawel Moll , Ian Campbell , Kumar Gala , Russell King , Alexandre Courbot , devicetree@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 02/21] ARM: tegra: Add gpio-ranges property References: <1432565608-26036-1-git-send-email-tomeu.vizoso@collabora.com> <1432565608-26036-3-git-send-email-tomeu.vizoso@collabora.com> In-Reply-To: <1432565608-26036-3-git-send-email-tomeu.vizoso@collabora.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1910 Lines: 43 On 05/25/2015 08:53 AM, Tomeu Vizoso wrote: > Specify how the GPIOs map to the pins in T124, so the dependency is > explicit. > > Signed-off-by: Tomeu Vizoso > --- > arch/arm/boot/dts/tegra124.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi > index 13cc7ca..5d1d35f 100644 > --- a/arch/arm/boot/dts/tegra124.dtsi > +++ b/arch/arm/boot/dts/tegra124.dtsi > @@ -254,6 +254,7 @@ > gpio-controller; > #interrupt-cells = <2>; > interrupt-controller; > + gpio-ranges = <&pinmux 0 0 250>; We should be consistent between SoCs. Why not make the same change for all Tegra SoCs? I think this change will cause the GPIO subsystem to call into the pinctrl subsystem and create/add/register a new GPIO<->pinctrl range structure. The pinctrl driver already does this, so I think we'll end up with two duplicate entries in the pinctrl device's gpio_ranges list. This probably won't cause a problem, but I wanted to make sure you'd thought about it to make sure. Right now, I think we get lucky and pinctrl ends up probing first (or at least very early) anyway. Somewhat related to this series, I wonder if we shouldn't add pinctrl client properties to every node in the Tegra DT that describes a controller that makes use of external pins that are affected by the pinmux. Such a change would guarantee this desired probing order. In order to preserve the "program the entire pinmux at once" semantics, these new pinctrl client properties would all need to reference empty states, yet would still need to exist to represent the dependency. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/