Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757778AbcCaXJD (ORCPT ); Thu, 31 Mar 2016 19:09:03 -0400 Received: from pandora.arm.linux.org.uk ([78.32.30.218]:45841 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754300AbcCaXJB (ORCPT ); Thu, 31 Mar 2016 19:09:01 -0400 Date: Fri, 1 Apr 2016 00:08:45 +0100 From: Russell King - ARM Linux To: Laura Abbott Cc: Josh Cartwright , punnaiah choudary kalluri , linux-mm@kvack.org, "linux-kernel@vger.kernel.org" , linux-arm-kernel , Sergey Dyasly , Arnd Bergmann Subject: Re: Issue with ioremap Message-ID: <20160331230845.GN19428@n2100.arm.linux.org.uk> References: <20160331200147.GA20530@jcartwri.amer.corp.natinst.com> <56FDAA66.2000505@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56FDAA66.2000505@redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1039 Lines: 23 On Thu, Mar 31, 2016 at 03:53:26PM -0700, Laura Abbott wrote: > (cc linux-arm) > > On 03/31/2016 01:01 PM, Josh Cartwright wrote: > >The driver _currently_ expects the virtual address to be 16M aligned, > >but is that a hard requirement? It seems possible that the driver could > >be written without this assumption, correct? > > > >This would mean that the driver would need to maintain the cs/cycles > >configuration state outside of the mapped virtual address, and then > >calculate + add the calculated offset to the base. Would that work? > >I had been meaning to give it a try, but haven't gotten around to it. > > I was curious so I took a look and this seems to be caused by The driver is most likely buggy in the way Josh has identified. The peripheral device has no clue what virtual address is used to access it, all it sees is the address on the bus. -- RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.