diff mbox

[1/6] Add config file for the snowball board

Message ID 1351500073-4351-2-git-send-email-gregory.hermant@calao-systems.com
State Superseded
Headers show

Commit Message

Gregory Hermant Oct. 29, 2012, 8:41 a.m. UTC
Signed-off-by: Gregory Hermant <gregory.hermant@calao-systems.com>
---
 configs/calao_snowball_defconfig |   52 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 52 insertions(+)
 create mode 100644 configs/calao_snowball_defconfig

Comments

Arnout Vandecappelle Oct. 30, 2012, 10:09 p.m. UTC | #1
On 10/29/12 09:41, Gregory Hermant wrote:
>
> Signed-off-by: Gregory Hermant<gregory.hermant@calao-systems.com>
> ---
>   configs/calao_snowball_defconfig |   52 ++++++++++++++++++++++++++++++++++++++
>   1 file changed, 52 insertions(+)
>   create mode 100644 configs/calao_snowball_defconfig
>
> diff --git a/configs/calao_snowball_defconfig b/configs/calao_snowball_defconfig
> new file mode 100644
> index 0000000..68f0a51
> --- /dev/null
> +++ b/configs/calao_snowball_defconfig
> @@ -0,0 +1,52 @@
> +BR2_arm=y
> +BR2_cortex_a9=y
> +BR2_TOOLCHAIN_EXTERNAL=y

  We don't use external toolchains for the defconfigs, unless the internal
one doesn't work for some reason (e.g. microblaze).

> +BR2_TARGET_GENERIC_GETTY_PORT="ttyAMA2"
> +BR2_PACKAGE_BUSYBOX_CONFIG="board/calao/snowball/busybox-1.20.x.config"

  This file should be created in the same patch, otherwise this defconfig
isn't working.

  Also, is it really necessary to have a custom busybox config?

> +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
> +BR2_PACKAGE_BZIP2=y
> +BR2_PACKAGE_FBSET=y
> +BR2_PACKAGE_FBTERM=y
> +BR2_PACKAGE_FBV=y
> +BR2_PACKAGE_QT=y
> +BR2_PACKAGE_QT_DEMOS=y
> +BR2_PACKAGE_QT_LICENSE_APPROVED=y
> +BR2_PACKAGE_QT_MOUSE_PC=y
> +BR2_PACKAGE_QT_MOUSE_LINUXTP=y
> +BR2_PACKAGE_QT_MOUSE_LINUXINPUT=y
> +BR2_PACKAGE_QT_MOUSE_TSLIB=y
> +BR2_PACKAGE_MINICOM=y
> +BR2_PACKAGE_LUA=y
> +BR2_PACKAGE_OPENSSL=y
> +BR2_PACKAGE_SQLITE=y
> +BR2_PACKAGE_LIBELF=y
> +BR2_PACKAGE_PCRE=y
> +BR2_PACKAGE_LIBXML2=y

  We don't select any packages in a defconfig, unless they're required
for booting correctly.  We certainly don't want to build Qt!

> +BR2_PACKAGE_HDMISERVICE=y
> +BR2_PACKAGE_SNOWBALL_FIRMWARE=y
> +BR2_PACKAGE_SNOWBALL_INIT=y

  These packages don't exist yet; the patches adding them should come
before this patch.

  Note that for these, it could be relevant to include them in the
defconfig - they're not strictly required for booting, but I guess
you want them in practice.

> +BR2_PACKAGE_BLUEZ_UTILS=y
> +BR2_PACKAGE_IW=y
> +BR2_PACKAGE_WGET=y
> +BR2_PACKAGE_WIRELESS_TOOLS=y
> +BR2_PACKAGE_WIRELESS_TOOLS_LIB=y
> +BR2_PACKAGE_WPA_SUPPLICANT=y
> +BR2_PACKAGE_WPA_SUPPLICANT_EAP=y
> +BR2_PACKAGE_WPA_SUPPLICANT_CLI=y
> +BR2_PACKAGE_WPA_SUPPLICANT_PASSPHRASE=y
> +BR2_PACKAGE_WPA_SUPPLICANT_AP_SUPPORT=y
> +BR2_PACKAGE_WPA_SUPPLICANT_WPS=y
> +BR2_TARGET_ROOTFS_TAR_GZIP=y
> +BR2_TARGET_UBOOT=y
> +BR2_TARGET_UBOOT_BOARDNAME="u8500_snowball"
> +BR2_TARGET_UBOOT_CUSTOM_GIT=y
> +BR2_TARGET_UBOOT_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/boot/u-boot-ux500.git"
> +BR2_TARGET_UBOOT_CUSTOM_GIT_VERSION="master"
> +BR2_LINUX_KERNEL=y
> +BR2_LINUX_KERNEL_CUSTOM_GIT=y
> +BR2_LINUX_KERNEL_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/kernel/igloo-kernel.git"
> +BR2_LINUX_KERNEL_CUSTOM_GIT_VERSION="stable-linux-ux500-3.4"
> +BR2_LINUX_KERNEL_PATCH="board/calao/snowball/linux-3.4.patch"

  This patch is more than a month old - how come it's not yet accepted
upstream?

> +BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
> +BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/calao/snowball/linux-3.4.config"

  Any reason why the u8500_defconfig isn't appropriate?

> +BR2_LINUX_KERNEL_INSTALL_TARGET=y

  Why?


  Regards,
  Arnout
Gregory Hermant Nov. 8, 2012, 8:28 a.m. UTC | #2
Le 30/10/2012 23:09, Arnout Vandecappelle a écrit :
> On 10/29/12 09:41, Gregory Hermant wrote:
>>
>> Signed-off-by: Gregory Hermant<gregory.hermant@calao-systems.com>
>> ---
>>   configs/calao_snowball_defconfig |   52 ++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 52 insertions(+)
>>   create mode 100644 configs/calao_snowball_defconfig
>>
>> diff --git a/configs/calao_snowball_defconfig b/configs/calao_snowball_defconfig
>> new file mode 100644
>> index 0000000..68f0a51
>> --- /dev/null
>> +++ b/configs/calao_snowball_defconfig
>> @@ -0,0 +1,52 @@
>> +BR2_arm=y
>> +BR2_cortex_a9=y
>> +BR2_TOOLCHAIN_EXTERNAL=y
> 
>  We don't use external toolchains for the defconfigs, unless the internal
> one doesn't work for some reason (e.g. microblaze).
> 
Until now the ubuntu images available on the igloo community website were
built from the linaro toolchain that's why i used the external linaro toolchain.
I will change the config for the buildroot one.

>> +BR2_TARGET_GENERIC_GETTY_PORT="ttyAMA2"
>> +BR2_PACKAGE_BUSYBOX_CONFIG="board/calao/snowball/busybox-1.20.x.config"
> 
>  This file should be created in the same patch, otherwise this defconfig
> isn't working.
> 
>  Also, is it really necessary to have a custom busybox config?
>
I need to enable the mdev loading firmware support.
I will change the snowball defconfig file in order to use mdev.

>> +BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
>> +BR2_PACKAGE_BZIP2=y
>> +BR2_PACKAGE_FBSET=y
>> +BR2_PACKAGE_FBTERM=y
>> +BR2_PACKAGE_FBV=y
>> +BR2_PACKAGE_QT=y
>> +BR2_PACKAGE_QT_DEMOS=y
>> +BR2_PACKAGE_QT_LICENSE_APPROVED=y
>> +BR2_PACKAGE_QT_MOUSE_PC=y
>> +BR2_PACKAGE_QT_MOUSE_LINUXTP=y
>> +BR2_PACKAGE_QT_MOUSE_LINUXINPUT=y
>> +BR2_PACKAGE_QT_MOUSE_TSLIB=y
>> +BR2_PACKAGE_MINICOM=y
>> +BR2_PACKAGE_LUA=y
>> +BR2_PACKAGE_OPENSSL=y
>> +BR2_PACKAGE_SQLITE=y
>> +BR2_PACKAGE_LIBELF=y
>> +BR2_PACKAGE_PCRE=y
>> +BR2_PACKAGE_LIBXML2=y
> 
>  We don't select any packages in a defconfig, unless they're required
> for booting correctly.  We certainly don't want to build Qt!

I thought it could be interesting for the end user to have QT built-in in the default rootfs.
I will remove QT from the snowball config file.
> 
>> +BR2_PACKAGE_HDMISERVICE=y
>> +BR2_PACKAGE_SNOWBALL_FIRMWARE=y
>> +BR2_PACKAGE_SNOWBALL_INIT=y
> 
>  These packages don't exist yet; the patches adding them should come
> before this patch.
> 
>  Note that for these, it could be relevant to include them in the
> defconfig - they're not strictly required for booting, but I guess
> you want them in practice.
> 
>> +BR2_PACKAGE_BLUEZ_UTILS=y
>> +BR2_PACKAGE_IW=y
>> +BR2_PACKAGE_WGET=y
>> +BR2_PACKAGE_WIRELESS_TOOLS=y
>> +BR2_PACKAGE_WIRELESS_TOOLS_LIB=y
>> +BR2_PACKAGE_WPA_SUPPLICANT=y
>> +BR2_PACKAGE_WPA_SUPPLICANT_EAP=y
>> +BR2_PACKAGE_WPA_SUPPLICANT_CLI=y
>> +BR2_PACKAGE_WPA_SUPPLICANT_PASSPHRASE=y
>> +BR2_PACKAGE_WPA_SUPPLICANT_AP_SUPPORT=y
>> +BR2_PACKAGE_WPA_SUPPLICANT_WPS=y
>> +BR2_TARGET_ROOTFS_TAR_GZIP=y
>> +BR2_TARGET_UBOOT=y
>> +BR2_TARGET_UBOOT_BOARDNAME="u8500_snowball"
>> +BR2_TARGET_UBOOT_CUSTOM_GIT=y
>> +BR2_TARGET_UBOOT_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/boot/u-boot-ux500.git"
>> +BR2_TARGET_UBOOT_CUSTOM_GIT_VERSION="master"
>> +BR2_LINUX_KERNEL=y
>> +BR2_LINUX_KERNEL_CUSTOM_GIT=y
>> +BR2_LINUX_KERNEL_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/kernel/igloo-kernel.git"
>> +BR2_LINUX_KERNEL_CUSTOM_GIT_VERSION="stable-linux-ux500-3.4"
>> +BR2_LINUX_KERNEL_PATCH="board/calao/snowball/linux-3.4.patch"
> 
>  This patch is more than a month old - how come it's not yet accepted
> upstream?

I don't know exactly what is the status of the snowball support in the mainline kernel.
As i prefer to use the latest stable release available from igloo, i need to add this patch.

> 
>> +BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
>> +BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/calao/snowball/linux-3.4.config"
> 
>  Any reason why the u8500_defconfig isn't appropriate?
> 
You are right i can use the default defconfig.
I had modified the config file for QT (input devices enabled) but anyway the one attached in the patch serie
didn't match my test config.

>> +BR2_LINUX_KERNEL_INSTALL_TARGET=y
> 
>  Why?
> 
To load the kernel from the rootfs partition hence the boot partition
could be removed.

> 
>  Regards,
>  Arnout

Thanks for your feedback.

BR,
gregory
Arnout Vandecappelle Nov. 9, 2012, 1:51 a.m. UTC | #3
On 11/08/12 09:28, Grégory Hermant wrote:
> Le 30/10/2012 23:09, Arnout Vandecappelle a écrit :
>> On 10/29/12 09:41, Gregory Hermant wrote:
[snip]
>>> +BR2_TARGET_GENERIC_GETTY_PORT="ttyAMA2"
>>> +BR2_PACKAGE_BUSYBOX_CONFIG="board/calao/snowball/busybox-1.20.x.config"
>>
>>   This file should be created in the same patch, otherwise this defconfig
>> isn't working.
>>
>>   Also, is it really necessary to have a custom busybox config?
>>
> I need to enable the mdev loading firmware support.
> I will change the snowball defconfig file in order to use mdev.

  Yes, if you select BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_MDEV then mdev
will automatically be selected in busybox.

[snip]
>>> +BR2_LINUX_KERNEL_INSTALL_TARGET=y
>>
>>   Why?
>>
> To load the kernel from the rootfs partition hence the boot partition
> could be removed.

  Yes, that was actually obvious, I was stupid to ask it.


  Regards,
  Arnout
diff mbox

Patch

diff --git a/configs/calao_snowball_defconfig b/configs/calao_snowball_defconfig
new file mode 100644
index 0000000..68f0a51
--- /dev/null
+++ b/configs/calao_snowball_defconfig
@@ -0,0 +1,52 @@ 
+BR2_arm=y
+BR2_cortex_a9=y
+BR2_TOOLCHAIN_EXTERNAL=y
+BR2_TARGET_GENERIC_GETTY_PORT="ttyAMA2"
+BR2_PACKAGE_BUSYBOX_CONFIG="board/calao/snowball/busybox-1.20.x.config"
+BR2_PACKAGE_BUSYBOX_SHOW_OTHERS=y
+BR2_PACKAGE_BZIP2=y
+BR2_PACKAGE_FBSET=y
+BR2_PACKAGE_FBTERM=y
+BR2_PACKAGE_FBV=y
+BR2_PACKAGE_QT=y
+BR2_PACKAGE_QT_DEMOS=y
+BR2_PACKAGE_QT_LICENSE_APPROVED=y
+BR2_PACKAGE_QT_MOUSE_PC=y
+BR2_PACKAGE_QT_MOUSE_LINUXTP=y
+BR2_PACKAGE_QT_MOUSE_LINUXINPUT=y
+BR2_PACKAGE_QT_MOUSE_TSLIB=y
+BR2_PACKAGE_MINICOM=y
+BR2_PACKAGE_LUA=y
+BR2_PACKAGE_OPENSSL=y
+BR2_PACKAGE_SQLITE=y
+BR2_PACKAGE_LIBELF=y
+BR2_PACKAGE_PCRE=y
+BR2_PACKAGE_LIBXML2=y
+BR2_PACKAGE_HDMISERVICE=y
+BR2_PACKAGE_SNOWBALL_FIRMWARE=y
+BR2_PACKAGE_SNOWBALL_INIT=y
+BR2_PACKAGE_BLUEZ_UTILS=y
+BR2_PACKAGE_IW=y
+BR2_PACKAGE_WGET=y
+BR2_PACKAGE_WIRELESS_TOOLS=y
+BR2_PACKAGE_WIRELESS_TOOLS_LIB=y
+BR2_PACKAGE_WPA_SUPPLICANT=y
+BR2_PACKAGE_WPA_SUPPLICANT_EAP=y
+BR2_PACKAGE_WPA_SUPPLICANT_CLI=y
+BR2_PACKAGE_WPA_SUPPLICANT_PASSPHRASE=y
+BR2_PACKAGE_WPA_SUPPLICANT_AP_SUPPORT=y
+BR2_PACKAGE_WPA_SUPPLICANT_WPS=y
+BR2_TARGET_ROOTFS_TAR_GZIP=y
+BR2_TARGET_UBOOT=y
+BR2_TARGET_UBOOT_BOARDNAME="u8500_snowball"
+BR2_TARGET_UBOOT_CUSTOM_GIT=y
+BR2_TARGET_UBOOT_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/boot/u-boot-ux500.git"
+BR2_TARGET_UBOOT_CUSTOM_GIT_VERSION="master"
+BR2_LINUX_KERNEL=y
+BR2_LINUX_KERNEL_CUSTOM_GIT=y
+BR2_LINUX_KERNEL_CUSTOM_GIT_REPO_URL="git://igloocommunity.org/git/kernel/igloo-kernel.git"
+BR2_LINUX_KERNEL_CUSTOM_GIT_VERSION="stable-linux-ux500-3.4"
+BR2_LINUX_KERNEL_PATCH="board/calao/snowball/linux-3.4.patch"
+BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
+BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/calao/snowball/linux-3.4.config"
+BR2_LINUX_KERNEL_INSTALL_TARGET=y