From mboxrd@z Thu Jan 1 00:00:00 1970 From: marek.vasut@gmail.com (Marek Vasut) Date: Wed, 10 Mar 2010 16:20:16 +0100 Subject: [PATCH] Voipac PXA270 In-Reply-To: <20100310133529.GK28972@buzzloop.caiaq.de> References: <201003100247.27059.marek.vasut@gmail.com> <201003101237.26819.marek.vasut@gmail.com> <20100310133529.GK28972@buzzloop.caiaq.de> Message-ID: <201003101620.17501.marek.vasut@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dne St 10. b?ezna 2010 14:35:29 Daniel Mack napsal(a): > On Wed, Mar 10, 2010 at 12:37:26PM +0100, Marek Vasut wrote: > > Dne St 10. b?ezna 2010 07:18:40 Mike Rapoport napsal(a): > > > Do these defines really deserve dedicated .h file? Why cannot they live > > > in the arch/arm/mach-pxa/vpac270.c? > > > > I'm not moving this in. I dont want to polute the .c file with stuff that > > should obviously go into the header file. Besides there'll be more GPIOs > > defined and that'd make the .c file less readable, which I want to avoid. > > The question is - are they shared? If they're not and will never be used > by anything else then this .c file, I'd also vote for not having an > extra header file for them. > > Daniel > Yes, they are used in the PCMCIA driver. And I'd rather avoid having GPIO definitions scattered across multiple files.