From: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
barebox@lists.infradead.org,
Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Subject: Re: [PATCH 3/5] arm: initial support for Marvell Dove SoCs
Date: Wed, 15 May 2013 08:20:03 +0200 [thread overview]
Message-ID: <51932913.90704@gmail.com> (raw)
In-Reply-To: <20130515055550.GZ32299@pengutronix.de>
On 05/15/2013 07:55 AM, Sascha Hauer wrote:
>>> Is it possible to remap the registers using the kwbimage tool? That way
>>> every code that runs would see the desired register layout.
>>
>> Although possible, I prefer a different approach:
>> - IIRC all MVEBU SoCs boot up at 0xd0000000
>> - Have a regbase variable within barebox that initially points to that
>> - Have a mvreadl/mvwritel that adds it to reg offsets
>> - Rewrite _initial_ SoC code to use mvreadl/mvwritel
>
> I applied this series, but please get this topic done before more code
> enters that has to be changed when doing the above.
Sascha,
it's not that easy as I thought. Armada 370 behaves very different
from Dove here. While Dove allows to read from unmapped adresses,
Armada 370 hangs immediately.
I will work with Thomas on this asap and find a solution, but I am
not sure if it will work without dirty tricks. Thomas has one in
mind and I tend to agree with him on this one.
Further, yesterday when I played with Armada 370 I wasn't able to
have it boot with a variable pointer but the same code boots with
a pointer that never gets modified. Haven't investigated more, but
if we remap first thing after boot, that can remain constant at the
remapped reg base.
Sebastian
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2013-05-15 6:20 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-12 13:09 [PATCH 0/5] Initial support for Marvell Dove SoCs Sebastian Hesselbarth
2013-05-12 13:09 ` [PATCH 1/5] scripts: kwbimage: add references to Marvell Dove SoC Sebastian Hesselbarth
2013-05-12 13:09 ` [PATCH 2/5] scripts: kwboot: add support for Marvell Dove Sebastian Hesselbarth
2013-05-12 16:48 ` Thomas Petazzoni
2013-05-12 13:09 ` [PATCH 3/5] arm: initial support for Marvell Dove SoCs Sebastian Hesselbarth
2013-05-12 16:49 ` Thomas Petazzoni
2013-05-12 16:51 ` Sebastian Hesselbarth
2013-05-12 17:19 ` Thomas Petazzoni
2013-05-12 17:26 ` Sebastian Hesselbarth
2013-05-13 7:58 ` Sascha Hauer
2013-05-13 9:17 ` Sebastian Hesselbarth
2013-05-13 10:57 ` Sascha Hauer
2013-05-13 13:06 ` Sebastian Hesselbarth
2013-05-13 14:11 ` Sascha Hauer
2013-05-13 14:23 ` Thomas Petazzoni
2013-05-13 15:14 ` Sebastian Hesselbarth
2013-05-13 16:12 ` Sebastian Hesselbarth
2013-05-13 16:21 ` Thomas Petazzoni
2013-05-13 16:30 ` Sebastian Hesselbarth
2013-05-13 16:34 ` Thomas Petazzoni
2013-05-13 16:48 ` Sebastian Hesselbarth
2013-05-13 17:14 ` Thomas Petazzoni
2013-05-13 17:42 ` Sebastian Hesselbarth
2013-05-15 5:55 ` Sascha Hauer
2013-05-15 6:20 ` Sebastian Hesselbarth [this message]
2013-05-15 6:39 ` Sascha Hauer
2013-05-15 6:48 ` Sebastian Hesselbarth
2013-05-15 7:29 ` Thomas Petazzoni
2013-05-15 8:03 ` Lucas Stach
2013-05-15 8:11 ` Thomas Petazzoni
2013-05-15 7:26 ` Thomas Petazzoni
2013-05-15 8:11 ` Sascha Hauer
2013-05-15 8:19 ` Thomas Petazzoni
2013-05-12 13:09 ` [PATCH 4/5] arm: add basic support for SolidRun CuBox Sebastian Hesselbarth
2013-05-12 20:28 ` [PATCH v2 " Sebastian Hesselbarth
2013-05-12 13:09 ` [PATCH 5/5] arm: mach-mvebu: rename Armada 370/XP core code Sebastian Hesselbarth
2013-05-12 16:50 ` Thomas Petazzoni
2013-05-12 16:53 ` Sebastian Hesselbarth
2013-05-12 20:29 ` [PATCH v2 " Sebastian Hesselbarth
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=51932913.90704@gmail.com \
--to=sebastian.hesselbarth@gmail.com \
--cc=barebox@lists.infradead.org \
--cc=ezequiel.garcia@free-electrons.com \
--cc=s.hauer@pengutronix.de \
--cc=thomas.petazzoni@free-electrons.com \
/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.