From: mike@compulab.co.il (Mike Rapoport) Date: Wed, 04 May 2011 09:38:33 +0300 Subject: [PATCH 3/4] omap: move detection of NAND CS to common-board-devices In-Reply-To: <44342B2F-34C0-490C-91B8-96A9EA683F9B@linuxhacker.ru> References: <724FD998-C5DC-4C36-911B-52037F6D4323@linuxhacker.ru> <44342B2F-34C0-490C-91B8-96A9EA683F9B@linuxhacker.ru> Message-ID: <4DC0F469.9030406@compulab.co.il> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 05/04/11 07:10, Oleg Drokin wrote: > Ok, so here's a simple patch to save everyone trouble, I guess. > > Though on the other hand I can imagine that perhaps including this generic common-board-devices.c > might not be desirable for people that don't use anything from that file. Since the common-board-devices.c has TWL initialization I doubt there would a board that does not use it at all... > Would it be a better idea to split it to a file-per-feature? Splitting the common-board-devices into a file-per-feature will diminish its added value, IMO. We can either continue to use #ifdef CONFIG_SOMETHING in both common-board-devices.[ch] as your fix proposes or just drop #ifdefs and inlines from the header. Tony, what is your preference? -- Sincerely yours, Mike.