public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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.

      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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox