[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Eric Jarrige]

Elftosb is a Freescale toolset to build binary image files (like U-Boot.sb)
to be used with CPUs imx23, imx28, stp37xx

Signed-off-by: Eric Jarrige <eric.jarrige@armadeus.org>
---
 package/elftosb/elftosb.mk                         |   28 ++++++++++++++++++++
 .../elftosb/elftosb10.12.01-fixes_includes.patch   |   13 +++++++++
 2 files changed, 41 insertions(+), 0 deletions(-)
 create mode 100644 package/elftosb/elftosb.mk
 create mode 100644 package/elftosb/elftosb10.12.01-fixes_includes.patch

diff --git a/package/elftosb/elftosb.mk b/package/elftosb/elftosb.mk
new file mode 100644
index 0000000..75b95a5
--- /dev/null
+++ b/package/elftosb/elftosb.mk
@@ -0,0 +1,28 @@
+#############################################################
+#
+# elftosb
+#
+#############################################################
+ELFTOSB_VERSION = 10.12.01
+ELFTOSB_SOURCE = elftosb-$(ELFTOSB_VERSION).tar.gz
+ELFTOSB_SITE = http://repository.timesys.com/buildsources/e/elftosb/elftosb-$(ELFTOSB_VERSION)
+
+define HOST_ELFTOSB_BUILD_CMDS
+	$(HOST_MAKE_ENV) $(MAKE) -C $(@D) all
+endef
+
+define HOST_ELFTOSB_INSTALL_CMDS
+	$(INSTALL) -D $(@D)/bld/linux/elftosb $(HOST_DIR)/usr/bin/elftosb
+	$(INSTALL) -D $(@D)/bld/linux/keygen $(HOST_DIR)/usr/bin/keygen
+	$(INSTALL) -D $(@D)/bld/linux/sbtool $(HOST_DIR)/usr/bin/sbtool
+endef
+
+define HOST_ELFTOSB_CLEAN_CMDS
+	rm -rf $(@D)/bld/linux
+	rm -f $(HOST_DIR)/usr/bin/elftosb
+	rm -f $(HOST_DIR)/usr/bin/keygen
+	rm -f $(HOST_DIR)/usr/bin/sbtool
+endef
+
+$(eval $(call GENTARGETS,host))
+
diff --git a/package/elftosb/elftosb10.12.01-fixes_includes.patch b/package/elftosb/elftosb10.12.01-fixes_includes.patch
new file mode 100644
index 0000000..5e31233
--- /dev/null
+++ b/package/elftosb/elftosb10.12.01-fixes_includes.patch
@@ -0,0 +1,13 @@
+Index: host-elftosb-10.12.01/common/stdafx.h
+===================================================================
+--- host-elftosb-10.12.01.orig/common/stdafx.h	2012-01-30 12:43:35.000000000 +0100
++++ host-elftosb-10.12.01/common/stdafx.h	2012-01-30 12:43:54.000000000 +0100
+@@ -27,7 +27,7 @@
+ // For Linux systems only, types.h only defines the signed
+ // integer types.  This is not professional code.
+ // Update: They are defined in the header files in the more recent version of redhat enterprise gcc.
+-#include "/usr/include/sys/types.h"
++#include <sys/types.h>
+ #include <stdint.h>
+ //typedef unsigned long uint32_t;
+ //typedef unsigned short uint16_t;
-- 
1.7.2.5

[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Thomas Petazzoni]

Hello Eric,

Thanks for this patch. See comments below.

Le Mon, 12 Mar 2012 14:55:07 +0100,
Eric Jarrige <eric.jarrige@armadeus.org> a ?crit :

> diff --git a/package/elftosb/elftosb10.12.01-fixes_includes.patch b/package/elftosb/elftosb10.12.01-fixes_includes.patch
> new file mode 100644
> index 0000000..5e31233
> --- /dev/null
> +++ b/package/elftosb/elftosb10.12.01-fixes_includes.patch

This patch should include a description and a Signed-off-by line.

Moreover, who is using this host-elftosb package at the moment? Do you
intend to send some further package to the linux package to make use of
it? Or should this package be presented to the user in the "Host
utilities" submenu?

Best regards,

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

[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Eric Jarrige]

Hi Thomas,

On 12 mars 2012, at 18:32, Thomas Petazzoni wrote:

> Hello Eric,
> 
> Thanks for this patch. See comments below.
> 
> Le Mon, 12 Mar 2012 14:55:07 +0100,
> Eric Jarrige <eric.jarrige@armadeus.org> a ?crit :
> 
>> diff --git a/package/elftosb/elftosb10.12.01-fixes_includes.patch b/package/elftosb/elftosb10.12.01-fixes_includes.patch
>> new file mode 100644
>> index 0000000..5e31233
>> --- /dev/null
>> +++ b/package/elftosb/elftosb10.12.01-fixes_includes.patch
> 
> This patch should include a description and a Signed-off-by line.

Correct, I will fix it.

> 
> 
> Moreover, who is using this host-elftosb package at the moment? Do you
> intend to send some further package to the linux package to make use of
> it? Or should this package be presented to the user in the "Host
> utilities" submenu?

I have the plan to send another patch to support the u-boot.sb binary file
format that will depend on this host tool. 
The new apf28 board will also use this .sb binary file format and the elftosb
tool. It also possible to package the linux kernel with this tool for some CPUs
but for the time being that is not in my plan.
I do not know if this can be useful to have it in a user submenu.

Thanks for your response.

Best regards,
Eric


 

[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Thomas Petazzoni]

Le Tue, 13 Mar 2012 00:13:53 +0100,
Eric Jarrige <eric.jarrige@armadeus.org> a ?crit :

> I have the plan to send another patch to support the u-boot.sb binary file
> format that will depend on this host tool. 

Ok, good. Would it be possible to post the patch adding the elftosb
tool together with a patch adding this u-boot.sb support? Of course,
this should be two separate patches, but sent together.

> The new apf28 board will also use this .sb binary file format and the elftosb
> tool. It also possible to package the linux kernel with this tool for some CPUs
> but for the time being that is not in my plan.
> I do not know if this can be useful to have it in a user submenu.

As long as it is used somewhere as a dependency for something, then
it's fine.

Thanks,

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

[Buildroot] [PATCH] Add (Freescale) elftosb host package

[Eric Jarrige]

Hi Thomas,

On 13 mars 2012, at 07:53, Thomas Petazzoni wrote:

> Ok, good. Would it be possible to post the patch adding the elftosb
> tool together with a patch adding this u-boot.sb support? Of course,
> this should be two separate patches, but sent together.
> 

Ok I add the u-boot.sb patch to my next submit.

Best regards,
Eric