[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 59/68] aspeed: Add an AST2600 eval board
From: |
Cédric Le Goater |
Subject: |
Re: [PULL 59/68] aspeed: Add an AST2600 eval board |
Date: |
Wed, 16 Oct 2019 14:41:00 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 16/10/2019 14:20, Philippe Mathieu-Daudé wrote:
> Hi Peter,
>
> On 10/15/19 7:03 PM, Peter Maydell wrote:
>> On Mon, 14 Oct 2019 at 17:05, Peter Maydell <address@hidden> wrote:
>>>
>>> From: Cédric Le Goater <address@hidden>
>>>
>>> Signed-off-by: Cédric Le Goater <address@hidden>
>>> Reviewed-by: Joel Stanley <address@hidden>
>>> Message-id: address@hidden
>>> Signed-off-by: Peter Maydell <address@hidden>
>>> ---
>>> include/hw/arm/aspeed.h | 1 +
>>> hw/arm/aspeed.c | 23 +++++++++++++++++++++++
>>> 2 files changed, 24 insertions(+)
>>
>>> @@ -455,6 +467,17 @@ static const AspeedBoardConfig aspeed_boards[] = {
>>> .num_cs = 2,
>>> .i2c_init = witherspoon_bmc_i2c_init,
>>> .ram = 512 * MiB,
>>> + }, {
>>> + .name = MACHINE_TYPE_NAME("ast2600-evb"),
>>> + .desc = "Aspeed AST2600 EVB (Cortex A7)",
>>> + .soc_name = "ast2600-a0",
>>> + .hw_strap1 = AST2600_EVB_HW_STRAP1,
>>> + .hw_strap2 = AST2600_EVB_HW_STRAP2,
>>> + .fmc_model = "w25q512jv",
>>> + .spi_model = "mx66u51235f",
>>> + .num_cs = 1,
>>> + .i2c_init = ast2600_evb_i2c_init,
>>> + .ram = 2 * GiB,
>>
>> Hi. I just discovered that this makes 'make check' fail on
>> 32-bit systems, because you can't default to 2GB of RAM
>> for a board:
>>
>> (armhf)pmaydell@mustang-maydell:~/qemu$
>> ./build/all-a32/arm-softmmu/qemu-system-arm -M ast2600-evb
>> qemu-system-arm: at most 2047 MB RAM can be simulated
>>
>> It's also a pretty rudely large amount of RAM to allocate
>> by default: it caused 'make check' to fail on my OSX
>> box, which is 64-bits but doesn't have huge swathes
>> of free RAM.
>
> It is unlikely you use this board on a 32-bit system...
>
> You usually prefer to have modeled hardware matching real-life,
> what about making this board not available on 32-bit systems
> (we will soon have more boards like this):
>
> #if HOST_LONG_BITS > 32
> {
> .name = MACHINE_TYPE_NAME("ast2600-evb"),
> .desc = "Aspeed AST2600 EVB (Cortex A7)",
> ...
> },
> #endif /* HOST_LONG_BITS > 32 */
I sent a patch to lower the default RAM size to 1G but you can always
increase it on the command line.
Making the machine available seems a better choice but that's fine with
me if we prefer to restrict its use to 64bits hosts. As you wish.
C.
- [PULL 53/68] hw/gpio: Add in AST2600 specific implementation, (continued)
- [PULL 53/68] hw/gpio: Add in AST2600 specific implementation, Peter Maydell, 2019/10/14
- [PULL 55/68] aspeed/i2c: Add AST2600 support, Peter Maydell, 2019/10/14
- [PULL 54/68] aspeed/i2c: Introduce an object class per SoC, Peter Maydell, 2019/10/14
- [PULL 58/68] m25p80: Add support for w25q512jv, Peter Maydell, 2019/10/14
- [PULL 59/68] aspeed: Add an AST2600 eval board, Peter Maydell, 2019/10/14
- Re: [PULL 59/68] aspeed: Add an AST2600 eval board, Philippe Mathieu-Daudé, 2019/10/16
- Re: [PULL 59/68] aspeed: Add an AST2600 eval board,
Cédric Le Goater <=
- Re: [PULL 59/68] aspeed: Add an AST2600 eval board, Philippe Mathieu-Daudé, 2019/10/16
[PULL 56/68] aspeed: Introduce an object class per SoC, Peter Maydell, 2019/10/14
[PULL 57/68] aspeed/soc: Add AST2600 support, Peter Maydell, 2019/10/14
[PULL 60/68] aspeed: Parameterise number of MACs, Peter Maydell, 2019/10/14
[PULL 61/68] aspeed: add support for the Aspeed MII controller of the AST2600, Peter Maydell, 2019/10/14
[PULL 62/68] aspeed/soc: Add ASPEED Video stub, Peter Maydell, 2019/10/14
[PULL 63/68] hw/arm/raspi: Use the IEC binary prefix definitions, Peter Maydell, 2019/10/14
[PULL 64/68] hw/arm/bcm2835_peripherals: Improve logging, Peter Maydell, 2019/10/14
[PULL 65/68] hw/arm/bcm2835_peripherals: Name various address spaces, Peter Maydell, 2019/10/14
[PULL 66/68] hw/arm/bcm2835: Rename some definitions, Peter Maydell, 2019/10/14