[Buildroot] [PATCH 1/2] openpowerlink: use git repository instead of zip archive.

[Buildroot] [PATCH 1/2] openpowerlink: use git repository instead of zip archive.

[Romain Naour]

Since commit 6c5c08b854e4490697076ae3c5a9c587d8672c63,
openpowerlink package is rebuilt at every make call because
.stamp_downloaded is missing in the build directory.

The culprit is OPENPOWERLINK_EXTRACT_CMDS that remove and
replace the build directory.

unzip extract a directory "openPOWERLINK-V1.08.4" and
Buildroot expect a directory "openpowerlink-V1.08.4"

It is easier to use git repository instead of zip archive and
avoids rename the directory openPOWERLINK to openpowerlink.

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
 package/openpowerlink/openpowerlink.mk | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/package/openpowerlink/openpowerlink.mk b/package/openpowerlink/openpowerlink.mk
index 37a06e5..0aae544 100644
--- a/package/openpowerlink/openpowerlink.mk
+++ b/package/openpowerlink/openpowerlink.mk
@@ -5,9 +5,8 @@
 ################################################################################
 
 OPENPOWERLINK_VERSION = V1.08.4
-OPENPOWERLINK_SOURCE = openPOWERLINK-$(OPENPOWERLINK_VERSION).zip
-OPENPOWERLINK_SITE = http://downloads.sourceforge.net/project/openpowerlink/openPOWERLINK/V1.8.4
-
+OPENPOWERLINK_SITE = http://git.code.sf.net/p/openpowerlink/code
+OPENPOWERLINK_SITE_METHOD = git
 OPENPOWERLINK_LICENSE = BSD-2c, GPLv2
 OPENPOWERLINK_LICENSE_FILES = license.txt
 OPENPOWERLINK_INSTALL_STAGING = YES
@@ -89,11 +88,4 @@ else
 OPENPOWERLINK_CONF_OPT += -DCFG_X86_DEMO_LINUX_KERNEL=OFF
 endif
 
-define OPENPOWERLINK_EXTRACT_CMDS
-	$(RM) -rf $(OPENPOWERLINK_DIR)
-	unzip -q -d $(BUILD_DIR)/ $(DL_DIR)/$(OPENPOWERLINK_SOURCE)
-	test -d $(OPENPOWERLINK_DIR) || \
-		mv $(BUILD_DIR)/$(subst .zip,,$(OPENPOWERLINK_SOURCE)) $(OPENPOWERLINK_DIR)
-endef
-
 $(eval $(cmake-package))
-- 
1.9.0

[Buildroot] [PATCH 2/2] openpowerlink: add Intel I210 chip support

[Romain Naour]

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
 package/openpowerlink/Config.in        | 3 +++
 package/openpowerlink/openpowerlink.mk | 6 ++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/package/openpowerlink/Config.in b/package/openpowerlink/Config.in
index 8573961..d87f8be 100644
--- a/package/openpowerlink/Config.in
+++ b/package/openpowerlink/Config.in
@@ -60,6 +60,9 @@ config BR2_PACKAGE_OPENPOWERLINK_82573
 config BR2_PACKAGE_OPENPOWERLINK_8255x
 	bool "Intel 8255x"
 
+config BR2_PACKAGE_OPENPOWERLINK_I210
+	bool "Intel I210"
+
 config BR2_PACKAGE_OPENPOWERLINK_RTL8139
 	bool "Realtek RTL-8139"
 
diff --git a/package/openpowerlink/openpowerlink.mk b/package/openpowerlink/openpowerlink.mk
index 0aae544..152f014 100644
--- a/package/openpowerlink/openpowerlink.mk
+++ b/package/openpowerlink/openpowerlink.mk
@@ -51,10 +51,12 @@ endif
 
 ifeq ($(BR2_PACKAGE_OPENPOWERLINK_82573),y)
 OPENPOWERLINK_CONF_OPT += -DCFG_POWERLINK_EDRV=82573
-else ifeq ($(BR2_PACKAGE_OPENPOWERLINK_RTL8139),y)
-OPENPOWERLINK_CONF_OPT += -DCFG_POWERLINK_EDRV=8139
 else ifeq ($(BR2_PACKAGE_OPENPOWERLINK_8255x),y)
 OPENPOWERLINK_CONF_OPT += -DCFG_POWERLINK_EDRV=8255x
+else ifeq ($(BR2_PACKAGE_OPENPOWERLINK_I210),y)
+OPENPOWERLINK_CONF_OPT += -DCFG_POWERLINK_EDRV=I210
+else ifeq ($(BR2_PACKAGE_OPENPOWERLINK_RTL8139),y)
+OPENPOWERLINK_CONF_OPT += -DCFG_POWERLINK_EDRV=8139
 endif
 
 ifeq ($(BR2_PACKAGE_OPENPOWERLINK_MN),y)
-- 
1.9.0

[Buildroot] [PATCH 1/2] openpowerlink: use git repository instead of zip archive.

[Peter Korsgaard]

>>>>> "Romain" == Romain Naour <romain.naour@openwide.fr> writes:

 > Since commit 6c5c08b854e4490697076ae3c5a9c587d8672c63,
 > openpowerlink package is rebuilt at every make call because
 > .stamp_downloaded is missing in the build directory.

 > The culprit is OPENPOWERLINK_EXTRACT_CMDS that remove and
 > replace the build directory.

 > unzip extract a directory "openPOWERLINK-V1.08.4" and
 > Buildroot expect a directory "openpowerlink-V1.08.4"

 > It is easier to use git repository instead of zip archive and
 > avoids rename the directory openPOWERLINK to openpowerlink.

 > Signed-off-by: Romain Naour <romain.naour@openwide.fr>

Committed both, thanks. The 2nd patch is strictly seen a feature patch,
but it's pretty trivial so it looks safe for 2014.05.

-- 
Bye, Peter Korsgaard