[Buildroot] [PATCH] board/acmesystems/arietta: new board

Gustavo Zacarias gustavo at zacarias.com.ar
Mon Jun 29 22:25:47 UTC 2015



On 17/06/15 07:32, Angelo Compagnucci wrote:

> This patch adds support to the Arietta board from Acmesystems.
> This board comes in two memory sizes (128m and 256m) hence the
> reason for two different configs.
>
> Signed-off-by: Angelo Compagnucci <angelo.compagnucci at gmail.com>

Hi Angelo.
A few nits:

>   board/acmesystems/arietta/linux-4.0.config |  99 ++++++++++++++++

Any particular reason you need a custom config instead of being able to 
use at91_dt (like on the Aria sample i sent)?
I know the Arietta lacks an ethernet PHY, but the DTS should take care 
of that.

> +BR2_TARGET_AT91BOOTSTRAP3_CUSTOM_REPO_VERSION="acme"

Also the "acme" tag seems to be the "master" of customization (i.e. a 
moving target), could you switch to a commit hash?
Otherwise it looks OK without having hardware to test and you can have 
my Acked-by if you address these concerns.
Thanks.
Regards.


More information about the buildroot mailing list