RE: [PATCH 03/11] arm: omap: musb: ioremap only what's ours

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Tony,

> -----Original Message-----
> From: Tony Lindgren [mailto:tony@xxxxxxxxxxx]
> Sent: Friday, February 19, 2010 5:44 PM
> To: Aguirre, Sergio
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Felipe Balbi; linux-
> omap@xxxxxxxxxxxxxxx; linux-usb@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 03/11] arm: omap: musb: ioremap only what's ours
> 
> * Aguirre, Sergio <saaguirre@xxxxxx> [100219 15:34]:
> > Hi,
> >
> > > -----Original Message-----
> > > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-
> > > owner@xxxxxxxxxxxxxxx] On Behalf Of Tony Lindgren
> > > Sent: Friday, February 19, 2010 5:30 PM
> > > To: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> > > Cc: Felipe Balbi; linux-omap@xxxxxxxxxxxxxxx; linux-
> usb@xxxxxxxxxxxxxxx
> > > Subject: [PATCH 03/11] arm: omap: musb: ioremap only what's ours
> > >
> > > From: Felipe Balbi <felipe.balbi@xxxxxxxxx>
> > >
> > > omap3430 TRM says the OTG address space is 4k, not 8k.
> > >
> > > Cc: linux-usb@xxxxxxxxxxxxxxx
> > > Signed-off-by: Felipe Balbi <felipe.balbi@xxxxxxxxx>
> > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> > > ---
> > >  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?

Regards,
Sergio
> 
> Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux