From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olof Johansson Subject: Re: [PATCH v3 00/13] arm/tegra: Initialize GPIO & pinmux from DT Date: Thu, 25 Aug 2011 22:04:31 -0700 Message-ID: References: <1314315824-9687-1-git-send-email-swarren@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <1314315824-9687-1-git-send-email-swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Grant Likely , Colin Cross , Erik Gilling , Russell King , Arnd Bergmann , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Belisko Marek , Jamie Iles , Shawn Guo , Sergei Shtylyov , Linus Walleij List-Id: linux-tegra@vger.kernel.org On Thu, Aug 25, 2011 at 4:43 PM, Stephen Warren wr= ote: > v3: > * Build on Jamie Iles' patches to split some of the DT parsing into > =A0common code. > * Renamed some properties to remove "nvidia," from name; they can be > =A0generic. > * Support pins property, to apply DT nodes to multiple pins/groups > * Some minor error-message changes driven by some of the above. > * Removed requirement to #ifdef CONFIG_OF > * Removed AUXDATA table entries in mach-tegra/board-dt.c > > TODO: > * Convert from using strings to using integers for the function and p= in > =A0names. This relies on the dtc patches I posted yesterday, but shou= ld > =A0be pretty simple once they're in. > > Jamie Iles (1): > =A0of: add a generic pinmux helper > > Stephen Warren (12): > =A0arm/tegra: Prep boards for gpio/pinmux conversion to pdevs > =A0docs/dt: Document nvidia,tegra20-gpio's nvidia,enabled-gpios prope= rty > =A0arm/dt: Tegra: Add nvidia,enabled-gpios property to GPIO controlle= r > =A0docs/dt: Document nvidia,tegra20-pinmux binding > =A0arm/dt: Tegra: Add pinmux node > =A0dt: add empty for_each_child_of_node, of_find_property > =A0gpio/tegra: Convert to a platform device > =A0gpio/tegra: Add device tree support > =A0arm/tegra: Convert pinmux driver to a platform device > =A0of: add property iteration helpers > =A0arm/tegra: Add device tree support to pinmux driver > =A0arm/tegra: board-dt: Remove dependency on non-dt pinmux functions This is looking pretty sane to me. Grant, how do you prefer to handle this, since it spans of/dt, gpio and tegra code? I can start a topic branch for Arnd to pull in for the tegra parts, would you prefer picking up the of helper functions on your own or is an ack from you and merge through the arm-soc tree easier? -Olof From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752056Ab1HZFEe (ORCPT ); Fri, 26 Aug 2011 01:04:34 -0400 Received: from mail-gw0-f46.google.com ([74.125.83.46]:33294 "EHLO mail-gw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751195Ab1HZFEc convert rfc822-to-8bit (ORCPT ); Fri, 26 Aug 2011 01:04:32 -0400 MIME-Version: 1.0 X-Originating-IP: [173.13.129.225] In-Reply-To: <1314315824-9687-1-git-send-email-swarren@nvidia.com> References: <1314315824-9687-1-git-send-email-swarren@nvidia.com> Date: Thu, 25 Aug 2011 22:04:31 -0700 Message-ID: Subject: Re: [PATCH v3 00/13] arm/tegra: Initialize GPIO & pinmux from DT From: Olof Johansson To: Stephen Warren Cc: Grant Likely , Colin Cross , Erik Gilling , Russell King , Arnd Bergmann , devicetree-discuss@lists.ozlabs.org, linux-tegra@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Belisko Marek , Jamie Iles , Shawn Guo , Sergei Shtylyov , Linus Walleij Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 25, 2011 at 4:43 PM, Stephen Warren wrote: > v3: > * Build on Jamie Iles' patches to split some of the DT parsing into >  common code. > * Renamed some properties to remove "nvidia," from name; they can be >  generic. > * Support pins property, to apply DT nodes to multiple pins/groups > * Some minor error-message changes driven by some of the above. > * Removed requirement to #ifdef CONFIG_OF > * Removed AUXDATA table entries in mach-tegra/board-dt.c > > TODO: > * Convert from using strings to using integers for the function and pin >  names. This relies on the dtc patches I posted yesterday, but should >  be pretty simple once they're in. > > Jamie Iles (1): >  of: add a generic pinmux helper > > Stephen Warren (12): >  arm/tegra: Prep boards for gpio/pinmux conversion to pdevs >  docs/dt: Document nvidia,tegra20-gpio's nvidia,enabled-gpios property >  arm/dt: Tegra: Add nvidia,enabled-gpios property to GPIO controller >  docs/dt: Document nvidia,tegra20-pinmux binding >  arm/dt: Tegra: Add pinmux node >  dt: add empty for_each_child_of_node, of_find_property >  gpio/tegra: Convert to a platform device >  gpio/tegra: Add device tree support >  arm/tegra: Convert pinmux driver to a platform device >  of: add property iteration helpers >  arm/tegra: Add device tree support to pinmux driver >  arm/tegra: board-dt: Remove dependency on non-dt pinmux functions This is looking pretty sane to me. Grant, how do you prefer to handle this, since it spans of/dt, gpio and tegra code? I can start a topic branch for Arnd to pull in for the tegra parts, would you prefer picking up the of helper functions on your own or is an ack from you and merge through the arm-soc tree easier? -Olof From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Thu, 25 Aug 2011 22:04:31 -0700 Subject: [PATCH v3 00/13] arm/tegra: Initialize GPIO & pinmux from DT In-Reply-To: <1314315824-9687-1-git-send-email-swarren@nvidia.com> References: <1314315824-9687-1-git-send-email-swarren@nvidia.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Aug 25, 2011 at 4:43 PM, Stephen Warren wrote: > v3: > * Build on Jamie Iles' patches to split some of the DT parsing into > ?common code. > * Renamed some properties to remove "nvidia," from name; they can be > ?generic. > * Support pins property, to apply DT nodes to multiple pins/groups > * Some minor error-message changes driven by some of the above. > * Removed requirement to #ifdef CONFIG_OF > * Removed AUXDATA table entries in mach-tegra/board-dt.c > > TODO: > * Convert from using strings to using integers for the function and pin > ?names. This relies on the dtc patches I posted yesterday, but should > ?be pretty simple once they're in. > > Jamie Iles (1): > ?of: add a generic pinmux helper > > Stephen Warren (12): > ?arm/tegra: Prep boards for gpio/pinmux conversion to pdevs > ?docs/dt: Document nvidia,tegra20-gpio's nvidia,enabled-gpios property > ?arm/dt: Tegra: Add nvidia,enabled-gpios property to GPIO controller > ?docs/dt: Document nvidia,tegra20-pinmux binding > ?arm/dt: Tegra: Add pinmux node > ?dt: add empty for_each_child_of_node, of_find_property > ?gpio/tegra: Convert to a platform device > ?gpio/tegra: Add device tree support > ?arm/tegra: Convert pinmux driver to a platform device > ?of: add property iteration helpers > ?arm/tegra: Add device tree support to pinmux driver > ?arm/tegra: board-dt: Remove dependency on non-dt pinmux functions This is looking pretty sane to me. Grant, how do you prefer to handle this, since it spans of/dt, gpio and tegra code? I can start a topic branch for Arnd to pull in for the tegra parts, would you prefer picking up the of helper functions on your own or is an ack from you and merge through the arm-soc tree easier? -Olof