From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 03/11] arm: omap: musb: ioremap only what's ours Date: Fri, 19 Feb 2010 16:10:28 -0800 Message-ID: <20100220001028.GJ21755@atomide.com> References: <20100219232648.10559.380.stgit@baageli.muru.com> <20100219233019.10559.71005.stgit@baageli.muru.com> <20100219234414.GH21755@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:59899 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753139Ab0BTAJm (ORCPT ); Fri, 19 Feb 2010 19:09:42 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Aguirre, Sergio" Cc: "linux-arm-kernel@lists.infradead.org" , Felipe Balbi , "linux-omap@vger.kernel.org" , "linux-usb@vger.kernel.org" * Aguirre, Sergio [100219 15:52]: > Hi Tony, > > > -----Original Message----- > > From: Tony Lindgren [mailto:tony@atomide.com] > > Sent: Friday, February 19, 2010 5:44 PM > > To: Aguirre, Sergio > > Cc: linux-arm-kernel@lists.infradead.org; Felipe Balbi; linux- > > omap@vger.kernel.org; linux-usb@vger.kernel.org > > Subject: Re: [PATCH 03/11] arm: omap: musb: ioremap only what's ours > > > > * Aguirre, Sergio [100219 15:34]: > > > Hi, > > > > > > > -----Original Message----- > > > > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap- > > > > owner@vger.kernel.org] On Behalf Of Tony Lindgren > > > > Sent: Friday, February 19, 2010 5:30 PM > > > > To: linux-arm-kernel@lists.infradead.org > > > > Cc: Felipe Balbi; linux-omap@vger.kernel.org; linux- > > usb@vger.kernel.org > > > > Subject: [PATCH 03/11] arm: omap: musb: ioremap only what's ours > > > > > > > > From: Felipe Balbi > > > > > > > > omap3430 TRM says the OTG address space is 4k, not 8k. > > > > > > > > Cc: linux-usb@vger.kernel.org > > > > Signed-off-by: Felipe Balbi > > > > Signed-off-by: Tony Lindgren > > > > --- > > > > arch/arm/mach-omap2/usb-musb.c | 2 +- > > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > > > diff --git a/arch/arm/mach-omap2/usb-musb.c b/arch/arm/mach-omap2/usb- > > > > musb.c > > > > index a80441d..ba71f76 100644 > > > > --- a/arch/arm/mach-omap2/usb-musb.c > > > > +++ b/arch/arm/mach-omap2/usb-musb.c > > > > @@ -152,7 +152,7 @@ void __init usb_musb_init(void) > > > > musb_resources[0].start = OMAP243X_HS_BASE; > > > > else > > > > musb_resources[0].start = OMAP34XX_HSUSB_OTG_BASE; > > > > - musb_resources[0].end = musb_resources[0].start + SZ_8K - 1; > > > > + musb_resources[0].end = musb_resources[0].start + SZ_4K - 1; > > > > > > I'm just curious... > > > > > > Is this valid also if cpu_is_243x() ? > > > > Yes, the other 4K is for the L4 interconnect. > > Ok... > > But then in this case, shouldn't these register boundaries come from a platform specific header file? > > That would be cleaner, I believe, and easier to maintain in the long run. > > What do you think? Yeah that will eventually get sorted out by hwmod. Tony