All of lore.kernel.org
 help / color / mirror / Atom feed
From: KONRAD Frederic <frederic.konrad@adacore.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "QEMU Developers" <qemu-devel@nongnu.org>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	alistair@alistair23.me
Subject: Re: [Qemu-devel] [PATCH v1 2/3] arm: fix the armv7m reset state
Date: Mon, 3 Jul 2017 11:04:28 +0200	[thread overview]
Message-ID: <bd674de7-5bcb-8767-9ce4-0832485e6ca0@adacore.com> (raw)
In-Reply-To: <CAFEAcA-C3zJ6yqsbMm8a0XM-i9Ew_OtukjGgCR5aRKzF5-0bFw@mail.gmail.com>



On 07/03/2017 10:51 AM, Peter Maydell wrote:
> On 3 July 2017 at 08:31, KONRAD Frederic <frederic.konrad@adacore.com> wrote:
>> On 06/30/2017 11:06 AM, Peter Maydell wrote:
>>> On 30 June 2017 at 09:24, KONRAD Frederic <frederic.konrad@adacore.com>
>>> wrote:
>>>> If I choose (b) I won't be able to load it to SRAM and it is
>>>> basically the same result I'll need to move or modify the config.
>>>
>>>
>>> I don't understand this, though. Option (b) is probably painful
>>> to implement (I don't have a good idea of how to do it) but
>>> it ought to mean that the ELF files that work on the board
>>> also work for QEMU (regardless of how the board model
>>> implemented the aliased flash).
>>>
>>
>> Yes that's exactly what I want.
>>
>> Basically the 0x00000000 alias can point to the SRAM or the ROM
>> during the reset depending on some boot config. The ELF is
>> directly loaded in the ROM or in the SRAM and my patch allows to
>> fetch the two first words in the reset handler to make it work
>> for any boot config.
> 
> Yes, but it only works if you implemented it that way
> round, and not for board implementations which put the
> real device at 0 and the alias at high memory. I'd like a fix
> which deals with all of this, not just with the particular
> arrangement your board implementation has.

Ok got it, I'll check if I can do something clean which can
handle both ways.

Fred

> 
> thanks
> -- PMM
> 

  reply	other threads:[~2017-07-03  9:04 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-29  9:28 [Qemu-devel] [PATCH v1 0/3] Some armv7m fixes KONRAD Frederic
2017-06-29  9:28 ` [Qemu-devel] [PATCH v1 1/3] add memory_region_get_offset_within_address_space KONRAD Frederic
2017-06-29  9:28 ` [Qemu-devel] [PATCH v1 2/3] arm: fix the armv7m reset state KONRAD Frederic
2017-06-29 15:14   ` Peter Maydell
2017-06-29 16:41     ` KONRAD Frederic
2017-06-29 16:45       ` Peter Maydell
2017-06-30  8:24         ` KONRAD Frederic
2017-06-30  9:06           ` Peter Maydell
2017-07-03  7:31             ` KONRAD Frederic
2017-07-03  8:51               ` Peter Maydell
2017-07-03  9:04                 ` KONRAD Frederic [this message]
2017-06-29  9:28 ` [Qemu-devel] [PATCH v1 3/3] armv7m_systick: abort instead of locking on a bad rate KONRAD Frederic
2017-06-29 12:35   ` Philippe Mathieu-Daudé
2017-06-29 12:43     ` Peter Maydell
2017-06-29 12:48       ` KONRAD Frederic
2017-06-29 13:02       ` Philippe Mathieu-Daudé
2017-06-29 13:17         ` KONRAD Frederic
2017-07-06 23:49 ` [Qemu-devel] [PATCH v1 0/3] Some armv7m fixes no-reply
2017-07-07  0:03   ` Fam Zheng

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=bd674de7-5bcb-8767-9ce4-0832485e6ca0@adacore.com \
    --to=frederic.konrad@adacore.com \
    --cc=alex.bennee@linaro.org \
    --cc=alistair@alistair23.me \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.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 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.