[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank |
Date: |
Fri, 06 Jan 2012 21:11:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111220 Thunderbird/9.0 |
Am 06.01.2012 20:10, schrieb Igor Mitsyanko:
> On 01/06/2012 10:45 PM, Peter Maydell wrote:
>> On 6 January 2012 18:37, Igor Mitsyanko<address@hidden> wrote:
>>> On 01/06/2012 12:02 AM, Mark Langsdorf wrote:
>>>> + if (!cpu_model) {
>>>> + cpu_model = "cortex-a9";
>>>> + }
>>>
>>>
>>> Google said there is only cortexA9-based Highbank SoC version, maybe you
>>> should just hardcode cpu model?
>>
>> This is just boilerplate code for any random ARM board at the moment:
>> it defaults the CPU but lets the user override. We should either make
>> a decision to do something else for all boards, or follow the usual
>> convention here; I'm happy to do the latter.
>>
>
> Are you saying that it's a mistake that we hardcoded cpu model and
> memory size for Exynos boards in our patches?
No machine should silently change the user's -cpu to something else.
Either error out or warn the user, or let them face the consequences of
their parameters themselves.
Not sure how hardcoding the cpu_model would work with CPU features,
would they be still included or stripped out before. Peter?
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank, Andreas Färber, 2012/01/06
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank, Igor Mitsyanko, 2012/01/06
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank, Igor Mitsyanko, 2012/01/07
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank, Andreas Färber, 2012/01/07
Re: [Qemu-devel] [PATCH 5/5] arm: SoC model for Calxeda Highbank, Rob Herring, 2012/01/06
[Qemu-devel] [PATCH v2 0/6] arm: add support for Calxeda Highbank SoC, Mark Langsdorf, 2012/01/09