[Buildroot] [PATCH v2 2/3] configs: add defconfig for Telit EVK-PRO3

Fabio Porcedda fabio.porcedda at gmail.com
Wed May 1 22:14:49 UTC 2013


On Wed, May 1, 2013 at 11:13 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Dear Fabio Porcedda,
>

Hi Thomas,
thanks for reviewing.

> On Wed, 17 Apr 2013 09:41:42 +0200, Fabio Porcedda wrote:
>> For more info, please find board/telit/evk-pro3/readme.txt
>>
>> Signed-off-by: Fabio Porcedda <fabio.porcedda at gmail.com>
>> ---
>>  ...04.0-0001-watchdog-add-keep-alive-support.patch |  88 ++++++++
>>  ...02-watchdog-add-at91sam9-watchdog-support.patch | 228 +++++++++++++++++++++
>>  ...0003-at91sam9260-9g20-add-wathdog-support.patch |  40 ++++
>>  ...0004-at91sam9260-9g20-fix-wathdog-support.patch |  26 +++
>>  ...13.04.0-0005-watchdog-enable-for-evk-pro3.patch |  26 +++
>>  ...t91-at91_dt_defconfig-update-for-evk-pro3.patch |  44 ++++
>
> That's quite a number of patches. I suppose they are all already
> upstream, correct? If so, then maybe using a Git version would allow to
> get rid of those patches in Buildroot, and then you could later on
> submit a patch to update Barebox to the next stable version that
> contains those patches? Maybe even Barebox 2013.05 will be released
> soon with those patches?

Unfortunately they are not upstream, the original author is still working on it.
If possible I will try to help the original author, but i don't know
when those patches will be
included upstream.
Anyway If using a git version is the preferred solution,
I will create and use a git repostiory on github.com.

>> diff --git a/configs/telit_evk_pro3_defconfig b/configs/telit_evk_pro3_defconfig
>> new file mode 100644
>> index 0000000..f7df1e4
>> --- /dev/null
>> +++ b/configs/telit_evk_pro3_defconfig
>> @@ -0,0 +1,16 @@
>> +BR2_arm=y
>> +BR2_PREFER_STATIC_LIB=y
>
> This choice shouldn't be part of the defconfig.

Ok.

>> +BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_DEVTMPFS=y
>
> This one generally isn't either, but I think we can be flexible on this
> one.
>
>> +BR2_PACKAGE_BUSYBOX_WATCHDOG=y
>> +BR2_TARGET_ROOTFS_UBIFS=y
>> +BR2_TARGET_ROOTFS_UBI=y
>> +BR2_TARGET_BAREBOX=y
>> +BR2_TARGET_BAREBOX_CUSTOM_VERSION=y
>> +BR2_TARGET_BAREBOX_CUSTOM_VERSION_VALUE="2013.04.0"
>> +BR2_TARGET_BAREBOX_CUSTOM_PATCH_DIR="board/telit/evk-pro3"
>> +BR2_TARGET_BAREBOX_BOARD_DEFCONFIG="telit_evk_pro3"
>> +BR2_LINUX_KERNEL=y
>> +BR2_LINUX_KERNEL_PATCH="board/telit/evk-pro3"
>> +BR2_LINUX_KERNEL_DEFCONFIG="at91_dt"
>> +BR2_LINUX_KERNEL_APPENDED_ZIMAGE=y
>> +BR2_LINUX_KERNEL_INTREE_DTS_NAME="evk-pro3"
>
> Thanks,
>
> Thomas
> --
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com


Best regards
--
Fabio Porcedda



More information about the buildroot mailing list