Subject: Re: [PATCH v2 4/4] Freescale enhanced Local Bus Controller FCM NAND support.

From: Arnd Bergmann <[email-masked]>
Date: Fri, 14 Dec 2007 22:40:48 +0100

On Friday 14 December 2007, Anton Vorontsov wrote:
> > >Maybe this desires its own header?
> > 
> > It can be factored out if anything else ever uses it.
> 
> It's just confusing to parse lbc-specific and nand-specific code
> placed in the same file.

In that case, you could argue for splitting the implementation into
two C files. I don't see any reason to move stuff into a header
when it's not an interface between two parts of the code, but
only describes a hardware interface used by a single driver.

	Arnd <><