From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Mon, 19 Oct 2009 10:12:04 -0700 Subject: [PATCH 08/14] omap: Use ioremap for omap4 L4 code In-Reply-To: References: <20091016233624.2463.26653.stgit@localhost> <20091016234138.2463.38350.stgit@localhost> <20091016234622.GH12576@atomide.com> <20091017161947.GK12576@atomide.com> <20091019002416.GM12576@atomide.com> Message-ID: <20091019171204.GP12576@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Shilimkar, Santosh [091019 00:27]: > Tony, > > -----Original Message----- > > From: Tony Lindgren [mailto:tony at atomide.com] > > Sent: Monday, October 19, 2009 5:54 AM > > To: Shilimkar, Santosh > > Cc: linux-arm-kernel at lists.infradead.org; linux-omap at vger.kernel.org > > Subject: Re: [PATCH 08/14] omap: Use ioremap for omap4 L4 code > > > > * Tony Lindgren [091017 09:19]: > > > * Shilimkar, Santosh [091017 03:49]: > > > > Thanks Tony for this ioremap series!! > > > > > > > > > -----Original Message----- > > > > > From: Tony Lindgren [mailto:tony at atomide.com] > > > > > Sent: Saturday, October 17, 2009 5:16 AM > > > > > To: linux-arm-kernel at lists.infradead.org > > > > > Cc: linux-omap at vger.kernel.org; Shilimkar, Santosh > > > > > Subject: Re: [PATCH 08/14] omap: Use ioremap for omap4 L4 code > > > > > > > > > > * Tony Lindgren [091016 16:42]: > > > > > > Use ioremap for omap4 L4 code > > > > > > > > > > Santosh, can you please check and ack this patch? > > > > > > > > > > Especially some of the ioremap sizes may be wrong. Also, this one > > > > > I cannot boot test, so it's just compile tested. > > > > > > > > I have checked the patch and it looks all right except few typo. Also > > the scu_base is needed bit earlier. > > > > > > > > > > Here is the patch on top of ioremap series which should fix above > > mentioned things. With below patch + two serial related patches the kernel > > boots on OMAP4430. > > > > > > Great, will update. Thanks for taking the time to fix it! > > > > Here's this one updated with Santosh's fix merged in. > V2 has one typo which I think I also missed in my patch. > >+ /* Static mapping, never released */ > >+ gic_cpu_base_addr = ioremap(OMAP44XX_GIC_CPU_BASE, SZ_512); > >+ BUG_ON(!base); > Should be BUG_ON(!gic_cpu_base_addr); OK, here's V3 of the patch. Tony