From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/4] armv8: enable compilation with CONFIG_SYS_NONCACHED_MEMORY
Date: Tue, 10 Nov 2015 18:01:27 +0100 [thread overview]
Message-ID: <20151110180127.64daefe8@lilith> (raw)
In-Reply-To: <f52dcc16c4ec46af8eda07a2cab2100b@HQMAIL103.nvidia.com>
Hello Tom,
Switching to these ASAP. :)
On Tue, 10 Nov 2015 15:54:58 +0000, Tom Warren <TWarren@nvidia.com>
wrote:
> Thanks, Tom.
>
> > -----Original Message-----
> > From: Tom Rini [mailto:trini at konsulko.com]
> > Sent: Monday, November 09, 2015 4:36 PM
> > To: Tom Warren <TWarren@nvidia.com>
> > Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>; swarren at wwwdotorg.org; u-
> > boot at lists.denx.de; Simon Glass <sjg@chromium.org>; Stephen Warren
> > <swarren@nvidia.com>; Thierry Reding <treding@nvidia.com>; York Sun
> > <yorksun@freescale.com>; Michal Simek <michal.simek@xilinx.com>;
> > 'tomcwarren3959 at gmail.com' <tomcwarren3959@gmail.com>
> > Subject: Re: [U-Boot] [PATCH 1/4] armv8: enable compilation with
> > CONFIG_SYS_NONCACHED_MEMORY
> >
> > * PGP Signed by an unknown key
> >
> > 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' <albert.u.boot@aribaud.net>; 'Stephen Warren'
> > > > <swarren@wwwdotorg.org>; Tom Rini (trini at konsulko.com)
> > > > <trini@konsulko.com>
> > > > Cc: 'u-boot at lists.denx.de' <u-boot@lists.denx.de>; 'Simon Glass'
> > > > <sjg@chromium.org>; Stephen Warren <swarren@nvidia.com>; Thierry
> > > > Reding <treding@nvidia.com>; 'York Sun' <yorksun@freescale.com>;
> > 'Michal Simek'
> > > > <michal.simek@xilinx.com>; 'tomcwarren3959 at gmail.com'
> > > > <tomcwarren3959@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' <albert.u.boot@aribaud.net>; 'Stephen Warren'
> > > > > <swarren@wwwdotorg.org>; Tom Rini (trini at konsulko.com)
> > > > > <trini@konsulko.com>
> > > > > Cc: 'u-boot at lists.denx.de' <u-boot@lists.denx.de>; 'Simon Glass'
> > > > > <sjg@chromium.org>; Stephen Warren <swarren@nvidia.com>; Thierry
> > > > > Reding <treding@nvidia.com>; 'York Sun' <yorksun@freescale.com>;
> > > > > 'Michal
> > > > Simek'
> > > > > <michal.simek@xilinx.com>; 'tomcwarren3959 at gmail.com'
> > > > > <tomcwarren3959@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' <albert.u.boot@aribaud.net>; Stephen Warren
> > > > > > <swarren@wwwdotorg.org>
> > > > > > Cc: u-boot at lists.denx.de; Simon Glass <sjg@chromium.org>;
> > > > > > Stephen Warren <swarren@nvidia.com>; Thierry Reding
> > > > > > <treding@nvidia.com>; York Sun <yorksun@freescale.com>; Michal
> > > > > > Simek <michal.simek@xilinx.com>
> > > > > > 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 <swarren@wwwdotorg.org>
> > > > > > > Cc: u-boot at lists.denx.de; Simon Glass <sjg@chromium.org>; Tom
> > > > > > > Warren <TWarren@nvidia.com>; Stephen Warren
> > > > <swarren@nvidia.com>;
> > > > > > > Thierry Reding <treding@nvidia.com>; York Sun
> > > > > > > <yorksun@freescale.com>; Michal Simek
> > > > > > > <michal.simek@xilinx.com>
> > > > > > > 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
> > > > > > > <swarren@wwwdotorg.org> wrote:
> > > > > > > > On 10/05/2015 12:08 PM, Stephen Warren wrote:
> > > > > > > > > From: Stephen Warren <swarren@nvidia.com>
> > > > > > > > >
> > > > > > > > > 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
> >
> > * Unknown Key
> > * 0x56D6FECD
Amicalement,
--
Albert.
prev parent reply other threads:[~2015-11-10 17:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-05 18:08 [U-Boot] [PATCH 1/4] armv8: enable compilation with CONFIG_SYS_NONCACHED_MEMORY Stephen Warren
2015-10-05 18:09 ` [U-Boot] [PATCH 2/4] armv8: allow custom MMU setup routines on ARMv8 Stephen Warren
2015-10-05 18:09 ` [U-Boot] [PATCH 3/4] ARM: tegra: add custom MMU setup " Stephen Warren
2015-10-05 18:09 ` [U-Boot] [PATCH 4/4] ARM: tegra: enable CONFIG_SYS_NONCACHED_MEMORY everywhere Stephen Warren
2015-10-21 16:35 ` [U-Boot] [PATCH 1/4] armv8: enable compilation with CONFIG_SYS_NONCACHED_MEMORY Stephen Warren
2015-10-22 6:25 ` Albert ARIBAUD
2015-10-27 20:38 ` Tom Warren
2015-11-03 19:18 ` Tom Warren
2015-11-06 16:31 ` Tom Warren
2015-11-09 23:28 ` Tom Warren
2015-11-09 23:35 ` Tom Rini
2015-11-10 15:54 ` Tom Warren
2015-11-10 17:01 ` Albert ARIBAUD [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151110180127.64daefe8@lilith \
--to=albert.u.boot@aribaud.net \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.