From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] ARM: OMAP: Fix map_io for Amstrad E3 Date: Wed, 9 Nov 2011 16:04:22 -0800 Message-ID: <20111110000421.GU31337@atomide.com> References: <20111109231900.GS31337@atomide.com> <20111109232525.GT31337@atomide.com> <20111109233900.GP15294@n2100.arm.linux.org.uk> <20D8B951-2E77-42DF-B735-6062994D47D5@iki.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:20087 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753536Ab1KJAE3 (ORCPT ); Wed, 9 Nov 2011 19:04:29 -0500 Content-Disposition: inline In-Reply-To: <20D8B951-2E77-42DF-B735-6062994D47D5@iki.fi> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Aaro Koskinen Cc: Russell King - ARM Linux , linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org * Aaro Koskinen [111109 15:25]: > Hi, > > On 10.11.2011, at 1.39, Russell King - ARM Linux wrote: > >On Wed, Nov 09, 2011 at 03:25:25PM -0800, Tony Lindgren wrote: > >>Commit 7b88e62f5d219a86d81bdf4388012c97dc42e8f8 (ARM: OMAP1: Use > >>generic > >>map_io, init_early and init_irq) changed omap1 to use generic map_io. > >> > >>Looks like I missed one board though. Fix this by adding a custom > >>map_io for Amstrad E3. > >> > >>Reported-by: Aaro Koskinen > >>Signed-off-by: Tony Lindgren > >> > >>--- > >>Untested, does this help? > >>--- a/arch/arm/mach-omap1/board-ams-delta.c > >>+++ b/arch/arm/mach-omap1/board-ams-delta.c > >>@@ -302,8 +302,6 @@ static void __init ams_delta_init(void) > >> omap_cfg_reg(J19_1610_CAM_D6); > >> omap_cfg_reg(J18_1610_CAM_D7); > >> > >>- iotable_init(ams_delta_io_desc, ARRAY_SIZE(ams_delta_io_desc)); > >>- > > > >This is definitely wrong. Using iotable_init() after map_io has > >returned > >has (and remains) a serious bug - doing so may _appear_ to work > >but as it > >doesn't obtain its memory from the standard kernel memory > >allocators, it > >will lead to duplicate usage of that memory. > > Also, the patch did not help. It still hangs like before. Hmm that patch is needed for sure but sounds like there's also something else wrong.. Can you try to git bisect it? Tony