From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Mundt Date: Tue, 18 May 2010 10:15:00 +0000 Subject: Re: [GIT PULL] sh updates for 2.6.35-rc1 Message-Id: <20100518101500.GA3156@linux-sh.org> List-Id: References: <20100518092532.GA3050@linux-sh.org> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jaswinder Singh Rajput Cc: Linus Torvalds , linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org On Tue, May 18, 2010 at 03:15:31PM +0530, Jaswinder Singh Rajput wrote: > I have few doubts : > > 1. Which other architectures are using : > include/linux/sh_clk.h, include/linux/sh_dma.h and include/linux/sh_intc.h > ARM for starters, and there are likely to be others in the future, too. Grepping would have made this pretty apparent. > 2. If you think, in future some architecture will going to use these > files, then do you think sh_*.h name is appropriate. > Yes, given that they're all SH IP blocks. Although if there's many more of them then of course putting them in their own subdirectory is an option, too. > 3. Can we move : > include/linux/sh_clk.h -> drivers/sh/sh_clk.h > include/linux/sh_dma.h -> drivers/dma/sh_dma.h > include/linux/sh_intc.h -> drivers/sh/sh_intc.h > So that if someone want to use these file they can use it from here. > No. The alternative is creating a shared architecture directory, which doesn't really scale well given how these blocks can be arbitrarily reused across different architectures -- but it's still something we might have to look at depending on what else pops up that can't be cleanly shared through the current scheme. You're of course welcome to dig up the discussions in the archives when the ARM SH-Mobile code was introduced in the first place.