From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753467AbbCJPC6 (ORCPT ); Tue, 10 Mar 2015 11:02:58 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:57641 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751285AbbCJPCy (ORCPT ); Tue, 10 Mar 2015 11:02:54 -0400 From: Arnd Bergmann To: Maxime Coquelin Cc: u.kleine-koenig@pengutronix.de, afaerber@suse.de, geert@linux-m68k.org, Rob Herring , Philipp Zabel , Jonathan Corbet , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Daniel Lezcano , Thomas Gleixner , Linus Walleij , Greg Kroah-Hartman , Jiri Slaby , Andrew Morton , "David S. Miller" , Mauro Carvalho Chehab , Joe Perches , Antti Palosaari , Tejun Heo , Will Deacon , Nikolay Borisov , Rusty Russell , Kees Cook , Michal Marek , linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-gpio@vger.kernel.org, linux-serial@vger.kernel.org, linux-arch@vger.kernel.org, linux-api@vger.kernel.org Subject: Re: [PATCH v2 07/18] drivers: reset: Add STM32 reset driver Date: Tue, 10 Mar 2015 16:02:14 +0100 Message-ID: <2383270.cAJPOxrkvo@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1424455277-29983-8-git-send-email-mcoquelin.stm32@gmail.com> References: <1424455277-29983-1-git-send-email-mcoquelin.stm32@gmail.com> <1424455277-29983-8-git-send-email-mcoquelin.stm32@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:F3foWIDx07th3Lh2/WCUuDZ+pgcDJfWciSjeZ+K316x8SU6zSCv yzkMWim3dkBo3PRNcYZ+WxGZPhyiUoxPW5U7LVpATFbMBWf9Fs1WM/2o448A4FS+je5aaRN UbSx1hIikqr4R0VyL1XZdA3plgvl9E+pc2tvWkEiqPtGfL7G2i8DtS8oQgmGwg+vEtU1Z2K j7hp7dWR3RFUinbt8hALw== X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 20 February 2015 19:01:06 Maxime Coquelin wrote: > +/* AHB1 */ > +#define GPIOA_RESET 0 > +#define GPIOB_RESET 1 > +#define GPIOC_RESET 2 > +#define GPIOD_RESET 3 > +#define GPIOE_RESET 4 > +#define GPIOF_RESET 5 > +#define GPIOG_RESET 6 > +#define GPIOH_RESET 7 > +#define GPIOI_RESET 8 > +#define GPIOJ_RESET 9 > +#define GPIOK_RESET 10 > As these are just the hardware numbers, it's better to not make them part of the binding at all. Instead, just document in the binding that one is supposed to pass the hardware number as the argument. Arnd