[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] aspeed: Add an AST2600 eval board
From: |
Peter Maydell |
Subject: |
Re: [PATCH] aspeed: Add an AST2600 eval board |
Date: |
Thu, 17 Oct 2019 14:54:34 +0100 |
On Thu, 17 Oct 2019 at 07:33, Joel Stanley <address@hidden> wrote:
>
> On Wed, 16 Oct 2019 at 09:08, Cédric Le Goater <address@hidden> wrote:
> >
> > Define the board with 1 GiB of RAM but some boards can have up to 2
> > GiB.
> >
> > Signed-off-by: Cédric Le Goater <address@hidden>
> > Reviewed-by: Joel Stanley <address@hidden>
> > ---
> >
> > Changes since AST2600 patchset:
> >
> > - lowered the RAM size to 1 GiB as it was breaking the tests on some
> > hosts.
>
> Peter,
>
> After chatting with Cédric I agree we should merge this patch.
>
> As it turns out the EVBs have differing amounts of RAM; his has 1GB
> while mine has 2GB. So we are not being inaccurate by setting 1GB as
> the default here.
That's convenient, means we don't have to figure out how to
special-case the test infrastructure for it :-)
Applied to target-arm.next, thanks.
-- PMM