From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 9 Nov 2015 18:35:59 -0500 Subject: [U-Boot] [PATCH 1/4] armv8: enable compilation with CONFIG_SYS_NONCACHED_MEMORY In-Reply-To: References: <1444068542-23191-1-git-send-email-swarren@wwwdotorg.org> <5627BEEF.5060307@wwwdotorg.org> <20151022082531.1f08e541@lilith> Message-ID: <20151109233559.GN8499@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Nov 09, 2015 at 11:28:27PM +0000, Tom Warren wrote: > TomR, > > Any chance of you taking a look at these ARM MMU patches and either > taking them in to u-boot/master or ACKing them so I can take them in > via the Tegra repo? So I know Albert was tacking a PR today and then ran into a bug with his Thumb-1 patchset. If they aren't in that PR then yes, I'll review/ack them but I suspect / hope Albert is taking a look and got side-tracked by the bug he found. > > Thanks, > > TomW > > > -----Original Message----- > > From: Tom Warren > > Sent: Friday, November 06, 2015 9:32 AM > > To: 'Albert ARIBAUD' ; 'Stephen Warren' > > ; Tom Rini (trini at konsulko.com) > > > > Cc: 'u-boot at lists.denx.de' ; 'Simon Glass' > > ; Stephen Warren ; Thierry Reding > > ; 'York Sun' ; 'Michal Simek' > > ; 'tomcwarren3959 at gmail.com' > > > > Subject: RE: [U-Boot] [PATCH 1/4] armv8: enable compilation with > > CONFIG_SYS_NONCACHED_MEMORY > > > > Albert, > > > > Ping! > > > > > -----Original Message----- > > > From: Tom Warren > > > Sent: Tuesday, November 03, 2015 12:19 PM > > > To: 'Albert ARIBAUD' ; 'Stephen Warren' > > > ; Tom Rini (trini at konsulko.com) > > > > > > Cc: 'u-boot at lists.denx.de' ; 'Simon Glass' > > > ; Stephen Warren ; Thierry > > > Reding ; 'York Sun' ; 'Michal > > Simek' > > > ; 'tomcwarren3959 at gmail.com' > > > > > > Subject: RE: [U-Boot] [PATCH 1/4] armv8: enable compilation with > > > CONFIG_SYS_NONCACHED_MEMORY > > > > > > Albert, > > > > > > > -----Original Message----- > > > > From: Tom Warren > > > > Sent: Tuesday, October 27, 2015 1:39 PM > > > > To: 'Albert ARIBAUD' ; Stephen Warren > > > > > > > > Cc: u-boot at lists.denx.de; Simon Glass ; Stephen > > > > Warren ; Thierry Reding ; > > > > York Sun ; Michal Simek > > > > > > > > Subject: RE: [U-Boot] [PATCH 1/4] armv8: enable compilation with > > > > CONFIG_SYS_NONCACHED_MEMORY > > > > > > > > Albert, > > > > > > > > > -----Original Message----- > > > > > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net] > > > > > Sent: Wednesday, October 21, 2015 11:26 PM > > > > > To: Stephen Warren > > > > > Cc: u-boot at lists.denx.de; Simon Glass ; Tom > > > > > Warren ; Stephen Warren > > ; > > > > > Thierry Reding ; York Sun > > > > > ; Michal Simek > > > > > Subject: Re: [U-Boot] [PATCH 1/4] armv8: enable compilation with > > > > > CONFIG_SYS_NONCACHED_MEMORY > > > > > > > > > > Hello Stephen, > > > > > > > > > > On Wed, 21 Oct 2015 10:35:59 -0600, Stephen Warren > > > > > wrote: > > > > > > On 10/05/2015 12:08 PM, Stephen Warren wrote: > > > > > > > From: Stephen Warren > > > > > > > > > > > > > > The implementation of noncached_init() uses define > > > > MMU_SECTION_SIZE. > > > > > > > Define this on ARM64. > > > > > > > > > > > > > > Move the prototype of noncached_{init,alloc}() to a location > > > > > > > that doesn't depend on !defined(CONFIG_ARM64). > > > > > > > > > > > > > > Note that noncached_init() calls > > > > > > > mmu_set_region_dcache_behaviour() which relies on something > > > > > > > having set up translation tables with 2MB block > > > > > size. > > > > > > > The core ARMv8 MMU setup code does not do this by default, but > > > > > > > currently relies on SoC specific MMU setup code. Be aware of > > > > > > > this before enabling this feature on your platform! > > > > > > > > > > > > Albert, it looks like I forgot to add you to the CC on this email. > > > > > > Sorry! Could you please take a look at this series and tell me > > > > > > what you think? If you need a resend just let me know. > > > > > > > > > > > > Also note that this series is a dependency for PCIe support on > > > > > > 64-bit Tegra systems; I guess either Tom Warren would be looking > > > > > > for an ack so he can apply the series to the Tegra tree, or > > > > > > perhaps if you just apply the whole series, he can base his > > > > > > branch on the branch where you apply > > > > > this. > > > > > > > > > > I'll have a look at it in the coming days. > > > > Please let me know when this series is in arm/master (or > > > > u-boot/master), as I need it to base Stephen's PCI/PCIe/XUSB patches > > > > on before I can send a PR for u-boot-tegra/master. Thanks! > > > I need this patchset in before I can add Stephen's PCIe/XUSB patchsets > > > to Tegra U-Boot. > > > > > > If you can Ack them, I can take them in via the Tegra repo, and send > > > them upstream to TomR on my next PR. Or if you can let me know an ETA > > > for getting them in to the ARM/master repo, I can base my schedule off > > > of that. But I'm blocked until the ARMv8 MMU changes are in somewhere, > > > and I'd like to clear this from my plate. > > > > > > Thanks, > > > > > > Tom > > > -- > > > nvpublic > > > > > > > > Tom > > -- > > nvpublic > > > > > > > > > > Amicalement, > > > > > -- > > > > > Albert. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: