[Buildroot] [PATCH v2] Adding support for Ariag25 and Terra board from Acmesystems

Angelo Compagnucci angelo.compagnucci at gmail.com
Wed May 28 12:51:17 UTC 2014


Hi Arnout,

2014-05-28 12:30 GMT+02:00 Arnout Vandecappelle <arnout at mind.be>:
> On 28/05/14 11:41, Angelo Compagnucci wrote:

>  No, just mention in the README or in the commit message where it comes from then.

Ok. I'll add a README.

>  The config you provide doesn't show that because it doesn't have an ext4 rootfs.

Yes, but I can simply untar the rootfs in a EXT4 root partition,
removing EXT4 will make the system unbootable without a plausible
reason. Moreover this setting is inside the producer provided config,
so I think it should stay.
A user that wants a really tailored rootfs for is own needs should
tweak the kernel configuration anyway. I think it's better to have a
sensible default, and ext4 seems sensible to me! Anyway, I'll remove
it if this prevents the patch been accepted.

>>>  Since you create just a root filesystem and a kernel, not even a bootloader,
>>> I'd expect a README file with an explanation how you get this stuff on your device.
>>
>> Ok, I will add the at91  bootloader.
>
>  And a readme about how to put it on the board. You can make it very short and
> just refer to a website - on condition that the instructions on that website are
> correct for the images generated by buildroot.

Ok. Producer website has very detailed explanations that work for sure
with buildroot.

>  We want a user to be able to boot a board with buildroot by doing:
>
> - make ariag25_defconfig
> - follow the instructions in board/acmesystems/ariag25/readme.txt

Ok, will be done!


>  That's perfect - but then you should refer to the examples on the website, and
> how to install them on the system. Perhaps even create a package for them
> (though that's probably not very useful).

I think a package it's not that much useful, they are simply examples
(like how to blink an led). So I will mention them in the README.

>> iputils
>> contains basic useful commands as ping, ntp is ntp,
>
>  Those are also in busybox.

Ok, catched. I'll replace packages with busybox ones.

>> and OPKG makes
>> possible to add other pre-compiled software from
>> http://feeds.angstrom-distribution.org/
>
>  Have you tried that? I doubt that angstrom uses uClibc toolchains...

Yeah, you are right ... I've tried in the past, but used eGlibc, with
default toolchain setting it will not work. I'll remove it.

Thank you for your great support!

Angelo



More information about the buildroot mailing list