From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] ARM: zynq: Move early printk virtual address to vmalloc area
Date: Tue, 16 Feb 2016 12:57:30 +0100 [thread overview]
Message-ID: <4209295.YSuxZXZMPH@wuerfel> (raw)
In-Reply-To: <67c9013fc61f53515934b54847a05df8b8e1504a.1455552315.git.michal.simek@xilinx.com>
On Monday 15 February 2016 17:05:17 Michal Simek wrote:
> The patch
> "ARM: 8432/1: move VMALLOC_END from 0xff000000 to 0xff800000"
> (sha1: 6ff0966052c46efb53980b8a1add2e7b49c9f560)
> has moved also start of VMALLOC area because size didn't change.
> That's why origin location of vmalloc was
> vmalloc : 0xf0000000 - 0xff000000 ( 240 MB)
> and now is
> vmalloc : 0xf0800000 - 0xff800000 ( 240 MB)
>
> That's why uart virtual addresses need to be changed to reflect this new
> memory setup. Starting address should be vmalloc start address.
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
All three patches look good to me,
Acked-by: Arnd Bergmann <arnd@arndb.de>
I would like to hear back from Nico on this though: is this an expected
outcome of your patch, or is it something you had not considered at
all?
Do we need to audit all platforms for other instances of the same
problem? I see for instance
static struct map_desc jornada720_io_desc[] __initdata = {
{ /* Epson registers */
.virtual = 0xf0000000,
.pfn = __phys_to_pfn(EPSONREGSTART),
.length = EPSONREGLEN,
.type = MT_DEVICE
}, { /* Epson frame buffer */
in arch/arm/mach-sa1100/jornada720.c. Could this have the same problem?
I would guess that only machines with 768MB of RAM (or more) can run
into the bug.
Arnd
prev parent reply other threads:[~2016-02-16 11:57 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-15 16:05 [PATCH 1/3] ARM: zynq: Move early printk virtual address to vmalloc area Michal Simek
2016-02-15 16:05 ` [PATCH 2/3] serial: xuartps: Enable OF earlycon support Michal Simek
2016-02-16 11:54 ` Arnd Bergmann
2016-02-16 16:33 ` Peter Hurley
2016-02-18 8:01 ` Michal Simek
2016-02-18 17:20 ` Peter Hurley
2016-02-15 16:05 ` [PATCH 3/3] ARM: zynq: Use earlycon instead of earlyprintk Michal Simek
2016-02-16 11:57 ` Arnd Bergmann [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=4209295.YSuxZXZMPH@wuerfel \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.org \
/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