Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Add Armadeus systems APF9328 support.
@ 2011-11-17 10:21 julien.boibessot at free.fr
  2011-11-17 19:57 ` Thomas Petazzoni
  0 siblings, 1 reply; 7+ messages in thread
From: julien.boibessot at free.fr @ 2011-11-17 10:21 UTC (permalink / raw)
  To: buildroot

From: Julien Boibessot <julien.boibessot@armadeus.com>

The APF9328 is an i.MXL+FPGA based SOM (System On Module). Here only minimal support is added: booting Linux kernel (UART, Ethernet and NOR), Buildroot toolchain and JFFS2 rootfs.

Signed-off-by: Julien Boibessot <julien.boibessot@armadeus.com>
Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
---
 ...1.1-0001-fixes_arm_mach-types_for_apf9328.patch |   26 ++++++++++++++++++++
 ...002-add_missing_config_option_for_apf9328.patch |   16 ++++++++++++
 configs/armadeus_apf9328_defconfig                 |   25 +++++++++++++++++++
 3 files changed, 67 insertions(+), 0 deletions(-)
 create mode 100644 board/armadeus/apf9328/linux-3.1.1-0001-fixes_arm_mach-types_for_apf9328.patch
 create mode 100644 board/armadeus/apf9328/linux-3.1.1-0002-add_missing_config_option_for_apf9328.patch
 create mode 100644 configs/armadeus_apf9328_defconfig

diff --git a/board/armadeus/apf9328/linux-3.1.1-0001-fixes_arm_mach-types_for_apf9328.patch b/board/armadeus/apf9328/linux-3.1.1-0001-fixes_arm_mach-types_for_apf9328.patch
new file mode 100644
index 0000000..aa464a3
--- /dev/null
+++ b/board/armadeus/apf9328/linux-3.1.1-0001-fixes_arm_mach-types_for_apf9328.patch
@@ -0,0 +1,26 @@
+From 053ba505eb0d40287ac5750daf454af0aed2ce75 Mon Sep 17 00:00:00 2001
+From: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
+Date: Tue, 4 Oct 2011 14:19:58 +0200
+Subject: [PATCH 1/2] [ARM] mach-types: Re-add apf9328
+
+
+Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
+---
+ arch/arm/tools/mach-types |    1 +
+ 1 files changed, 1 insertions(+), 0 deletions(-)
+
+diff --git a/arch/arm/tools/mach-types b/arch/arm/tools/mach-types
+index fff68d0..07811a7 100644
+--- a/arch/arm/tools/mach-types
++++ b/arch/arm/tools/mach-types
+@@ -188,6 +188,7 @@ omap_2430sdp		MACH_OMAP_2430SDP	OMAP_2430SDP		900
+ davinci_evm		MACH_DAVINCI_EVM	DAVINCI_EVM		901
+ palmz72			MACH_PALMZ72		PALMZ72			904
+ nxdb500			MACH_NXDB500		NXDB500			905
++apf9328			MACH_APF9328		APF9328			906
+ palmt5			MACH_PALMT5		PALMT5			917
+ palmtc			MACH_PALMTC		PALMTC			918
+ omap_apollon		MACH_OMAP_APOLLON	OMAP_APOLLON		919
+-- 
+1.7.3.4
+
diff --git a/board/armadeus/apf9328/linux-3.1.1-0002-add_missing_config_option_for_apf9328.patch b/board/armadeus/apf9328/linux-3.1.1-0002-add_missing_config_option_for_apf9328.patch
new file mode 100644
index 0000000..312b9a0
--- /dev/null
+++ b/board/armadeus/apf9328/linux-3.1.1-0002-add_missing_config_option_for_apf9328.patch
@@ -0,0 +1,16 @@
+APF9328 needs CONFIG_MTD_CFI_INTELEXT to boot on its NOR FLASH.
+
+Signed-off-by: Julien Boibessot <julien.boibessot@armadeus.com>
+
+Index: linux-3.1.1/arch/arm/mach-imx/Kconfig
+===================================================================
+--- linux-3.1.1.orig/arch/arm/mach-imx/Kconfig	2011-11-16 14:45:55.000000000 +0100
++++ linux-3.1.1/arch/arm/mach-imx/Kconfig	2011-11-16 14:47:20.000000000 +0100
+@@ -84,6 +84,7 @@
+ 	bool "APF9328"
+ 	select SOC_IMX1
+ 	select IMX_HAVE_PLATFORM_IMX_UART
++	select MTD_CFI_INTELEXT
+ 	help
+ 	  Say Yes here if you are using the Armadeus APF9328 development board
+ 
diff --git a/configs/armadeus_apf9328_defconfig b/configs/armadeus_apf9328_defconfig
new file mode 100644
index 0000000..f600c94
--- /dev/null
+++ b/configs/armadeus_apf9328_defconfig
@@ -0,0 +1,25 @@
+BR2_arm=y
+BR2_arm920t=y
+BR2_DL_DIR="$(BASE_DIR)/../downloads"
+BR2_SOURCEFORGE_MIRROR="switch"
+BR2_JLEVEL=1
+BR2_CCACHE=y
+BR2_STRIP_sstrip=y
+BR2_GCC_VERSION_4_4_X=y
+BR2_PACKAGE_GDB=y
+BR2_PACKAGE_GDB_HOST=y
+BR2_ENABLE_LOCALE_PURGE=y
+BR2_TARGET_OPTIMIZATION="-Os -pipe"
+BR2_TOOLCHAIN_BUILDROOT_LARGEFILE=y
+BR2_TOOLCHAIN_BUILDROOT_INET_IPV6=y
+BR2_TOOLCHAIN_BUILDROOT_INET_RPC=y
+BR2_TOOLCHAIN_BUILDROOT_LOCALE=y
+BR2_TOOLCHAIN_BUILDROOT_CXX=y
+BR2_TARGET_GENERIC_HOSTNAME="apf9328"
+BR2_TARGET_GENERIC_ISSUE="Welcome to Armadeus development platform !"
+BR2_ROOTFS_DEVICE_CREATION_DYNAMIC_DEVTMPFS=y
+BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
+BR2_TARGET_ROOTFS_JFFS2=y
+BR2_LINUX_KERNEL=y
+BR2_LINUX_KERNEL_PATCH="board/armadeus/apf9328"
+BR2_LINUX_KERNEL_DEFCONFIG="mx1"
-- 
1.7.4.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] Add Armadeus systems APF9328 support.
  2011-11-17 10:21 [Buildroot] [PATCH] Add Armadeus systems APF9328 support julien.boibessot at free.fr
@ 2011-11-17 19:57 ` Thomas Petazzoni
  2011-11-18 17:06   ` Julien Boibessot
  0 siblings, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2011-11-17 19:57 UTC (permalink / raw)
  To: buildroot

Hello,

Good to see Armadeus stuff coming into mainline Buildroot!

Le Thu, 17 Nov 2011 11:21:41 +0100,
julien.boibessot at free.fr a ?crit :

> From: Julien Boibessot <julien.boibessot@armadeus.com>
> 
> The APF9328 is an i.MXL+FPGA based SOM (System On Module). Here only minimal support is added: booting Linux kernel (UART, Ethernet and NOR), Buildroot toolchain and JFFS2 rootfs.

This should be wrapped at ~80 chars.

> +BR2_DL_DIR="$(BASE_DIR)/../downloads"
> +BR2_SOURCEFORGE_MIRROR="switch"
> +BR2_JLEVEL=1
> +BR2_CCACHE=y

Those four options (or at least the first three) are very
user-specific. I don't think they should be part of a default board
configuration file.

> +BR2_LINUX_KERNEL=y
> +BR2_LINUX_KERNEL_PATCH="board/armadeus/apf9328"
> +BR2_LINUX_KERNEL_DEFCONFIG="mx1"

Since two kernel patches are needed, you should have a defconfig that
explicitly tells which kernel version it wants:

BR2_LINUX_KERNEL_CUSTOM_VERSION=y
BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1"

see configs/qemu_* for examples.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] Add Armadeus systems APF9328 support.
  2011-11-17 19:57 ` Thomas Petazzoni
@ 2011-11-18 17:06   ` Julien Boibessot
  2011-11-18 18:20     ` Peter Korsgaard
  0 siblings, 1 reply; 7+ messages in thread
From: Julien Boibessot @ 2011-11-18 17:06 UTC (permalink / raw)
  To: buildroot

Hello,

On 17/11/2011 20:57, Thomas Petazzoni wrote:
> Good to see Armadeus stuff coming into mainline Buildroot!

Thanks for the review.
More will come as soon as BR can handle Linux patches tarball
downloading ;-)
> Le Thu, 17 Nov 2011 11:21:41 +0100,
> julien.boibessot at free.fr a ?crit :
>
>> From: Julien Boibessot <julien.boibessot@armadeus.com>
>>
>> The APF9328 is an i.MXL+FPGA based SOM (System On Module). Here only minimal support is added: booting Linux kernel (UART, Ethernet and NOR), Buildroot toolchain and JFFS2 rootfs.
> This should be wrapped at ~80 chars.
sorry
>> +BR2_DL_DIR="$(BASE_DIR)/../downloads"
>> +BR2_SOURCEFORGE_MIRROR="switch"
>> +BR2_JLEVEL=1
>> +BR2_CCACHE=y
> Those four options (or at least the first three) are very
> user-specific. I don't think they should be part of a default board
> configuration file.
ok
>> +BR2_LINUX_KERNEL=y
>> +BR2_LINUX_KERNEL_PATCH="board/armadeus/apf9328"
>> +BR2_LINUX_KERNEL_DEFCONFIG="mx1"
> Since two kernel patches are needed, you should have a defconfig that
> explicitly tells which kernel version it wants:
>
> BR2_LINUX_KERNEL_CUSTOM_VERSION=y
> BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1"

ok but if I set BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1.1", it is not
saved by "make savedefconfig". Seems like there is a "bug" in BR when
custom kernel version == default one

Regards,
Julien

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] Add Armadeus systems APF9328 support.
  2011-11-18 17:06   ` Julien Boibessot
@ 2011-11-18 18:20     ` Peter Korsgaard
  2011-11-18 23:06       ` Arnout Vandecappelle
  0 siblings, 1 reply; 7+ messages in thread
From: Peter Korsgaard @ 2011-11-18 18:20 UTC (permalink / raw)
  To: buildroot

>>>>> "Julien" == Julien Boibessot <julien.boibessot@free.fr> writes:

Hi,

 >> Since two kernel patches are needed, you should have a defconfig that
 >> explicitly tells which kernel version it wants:
 >> 
 >> BR2_LINUX_KERNEL_CUSTOM_VERSION=y
 >> BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1"

 Julien> ok but if I set BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1.1",
 Julien> it is not saved by "make savedefconfig". Seems like there is a
 Julien> "bug" in BR when custom kernel version == default one

Well, it's how savedefconfig behaves in the kernel. Just edit the file
manually afterwards - That way it will still work when we change the
default kernel version to be 3.2.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] Add Armadeus systems APF9328 support.
  2011-11-18 18:20     ` Peter Korsgaard
@ 2011-11-18 23:06       ` Arnout Vandecappelle
  2011-11-18 23:08         ` [Buildroot] [PATCH] linux: remove default for custom kernel version Arnout Vandecappelle
  0 siblings, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2011-11-18 23:06 UTC (permalink / raw)
  To: buildroot

On Friday 18 November 2011 18:20:27 Peter Korsgaard wrote:
> >>>>> "Julien" == Julien Boibessot <julien.boibessot@free.fr> writes:
> 
> Hi,
> 
>  >> Since two kernel patches are needed, you should have a defconfig that
>  >> explicitly tells which kernel version it wants:
>  >> 
>  >> BR2_LINUX_KERNEL_CUSTOM_VERSION=y
>  >> BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1"
> 
>  Julien> ok but if I set BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.1.1",
>  Julien> it is not saved by "make savedefconfig". Seems like there is a
>  Julien> "bug" in BR when custom kernel version == default one
> 
> Well, it's how savedefconfig behaves in the kernel. Just edit the file
> manually afterwards - That way it will still work when we change the
> default kernel version to be 3.2.

 In fact, that default for the custom kernel version is redundant.  We
already have a default kernel version, there's no point in having it twice.

 Patch follows.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] linux: remove default for custom kernel version
  2011-11-18 23:06       ` Arnout Vandecappelle
@ 2011-11-18 23:08         ` Arnout Vandecappelle
  2012-01-16 15:26           ` Peter Korsgaard
  0 siblings, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2011-11-18 23:08 UTC (permalink / raw)
  To: buildroot

From: "Arnout Vandecappelle (Essensium/Mind)" <arnout@mind.be>

There already is a default kernel version, so it makes little sense to have
a default custom kernel version as well.  This default breaks the 'make
savedefconfig' if we want to force a specific version.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 linux/Config.in |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/linux/Config.in b/linux/Config.in
index c304d86..b0d90c4 100644
--- a/linux/Config.in
+++ b/linux/Config.in
@@ -58,7 +58,6 @@ endchoice
 config BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE
 	string "Kernel version"
 	depends on BR2_LINUX_KERNEL_CUSTOM_VERSION
-	default "3.1.1"
 
 config BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION
 	string "URL of custom kernel tarball"
-- 
1.7.7.2

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [Buildroot] [PATCH] linux: remove default for custom kernel version
  2011-11-18 23:08         ` [Buildroot] [PATCH] linux: remove default for custom kernel version Arnout Vandecappelle
@ 2012-01-16 15:26           ` Peter Korsgaard
  0 siblings, 0 replies; 7+ messages in thread
From: Peter Korsgaard @ 2012-01-16 15:26 UTC (permalink / raw)
  To: buildroot

>>>>> "Arnout" == Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> writes:

 Arnout> From: "Arnout Vandecappelle (Essensium/Mind)" <arnout@mind.be>
 Arnout> There already is a default kernel version, so it makes little sense to have
 Arnout> a default custom kernel version as well.  This default breaks the 'make
 Arnout> savedefconfig' if we want to force a specific version.

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2012-01-16 15:26 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-17 10:21 [Buildroot] [PATCH] Add Armadeus systems APF9328 support julien.boibessot at free.fr
2011-11-17 19:57 ` Thomas Petazzoni
2011-11-18 17:06   ` Julien Boibessot
2011-11-18 18:20     ` Peter Korsgaard
2011-11-18 23:06       ` Arnout Vandecappelle
2011-11-18 23:08         ` [Buildroot] [PATCH] linux: remove default for custom kernel version Arnout Vandecappelle
2012-01-16 15:26           ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox