From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [RFC] Port TI DSP BRIDGE for a new dedicated branch in linux-omap Date: Mon, 18 Aug 2008 10:56:24 +0300 Message-ID: <20080818075624.GB12243@atomide.com> References: <94a0d4530808151409p309cf9e2sf1ab3274baac32fb@mail.gmail.com> <94a0d4530808160257r77341fafn5f2b8404a8510203@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-bos.mailhop.org ([63.208.196.178]:52436 "EHLO mho-01-bos.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751068AbYHRH4a (ORCPT ); Mon, 18 Aug 2008 03:56:30 -0400 Content-Disposition: inline In-Reply-To: <94a0d4530808160257r77341fafn5f2b8404a8510203@mail.gmail.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Felipe Contreras Cc: "Pasam, Vijay" , "Woodruff, Richard" , Hiroshi DOYU , linux-omap@vger.kernel.org, soni.trilok@gmail.com, "Kanigeri, Hari" , "Ramirez Luna, Omar" , "Gupta, Ramesh" , felipe.contreras@nokia.com Hi, * Felipe Contreras [080816 12:58]: > Hi Vijay, > > On Sat, Aug 16, 2008 at 12:59 AM, Pasam, Vijay wrote: > >> I'm not familiar with checkpatch, but I guess the purpose is > >> not to highlight functional issues. > >> > >> However, there are functional issues, but it makes sense to > >> cleanup the code first: there's no point in analyzing code > >> that is never used. > > > > There are about 9 errors with the latest set of patches. These are all > > false positives - not really errors. Majority of warnings also fall > > under this category. > > I'm not talking about issues returned by checkpatch, but issues > visible to the human observer. And the two pieces of code that need to be fixed for Linux ASAP for DSP are: - External device MMU hardware - Mailbox handling This code needs to be shared for dsp[bios|bridge|gateway|link]. And it can be also used for any other devices and coprocessors needing MMU handling. So far the sanest solution seems to be in: arch/arm/*omap*/mmu.[ch] arch/arm/*omap*/mailbox.[ch] Is there any reason why this code cannot be used for dspbridge? BTW, the MMU code needs to be fixed so that the code shared with ioremap() is moved to ioremap(). Basically we need to enhance ioremap() in a way where it can support external MMU hardware. Otherwise the maintenance will be a nightmare. This was pointed out by Russell King a while back when we tried to get the MMU code integrated to the mainline kernel. Regards, Tony