Re: [Qemu-devel] [PATCH v1 1/1] target_arm: Make the reset rom_ptr a property

[Peter Crosthwaite]

On Thu, Aug 21, 2014 at 10:33 PM, Peter Maydell
<address@hidden> wrote:
> On 21 August 2014 04:37, Peter Crosthwaite <address@hidden> wrote:
>> On Thu, Aug 21, 2014 at 11:40 AM, Alistair Francis <address@hidden> wrote:
>>> On Thu, Aug 21, 2014 at 11:36 AM, Alistair Francis <address@hidden> wrote:
>>>> This allows the board to set the reset address, which is required
>>>> for some boards (the Netduino Plus 2 for example)
>
>> The change is armv7m specific right? You should mention that in the
>> commit message.
>>
>> This also obsoletes my attempt at correcting armv7m elf loading which
>> was trying to solve this issue a different way.
>
> I think the ELF loading thing is a separate issue -- if we're loading
> an ELF file we should honour its entry point address instead of
> trying to load from the vector table. (The question of what we
> do about SP is an interesting one...)
>

Well for an elf load you can grab just the SP from VTOR and not the
PC? You will still need to solve the problem of this patch however.

> PS: is the patch you refer to currently with you for rework? I don't
> see anything relevant in my to-review queue...
>

No itis dropped for the moment. I was hoping for the issue to go away
with whatever solution we come up for this.

Regards,
Peter

> thanks
> -- PMM
>