Skip to contentSkip to conversation

Add on2-8170 kernel modules

PATCH Superseded Alexandre Belloni 2 participants 4 messages Jun 9, 2013 – Jun 10, 2013

v2 of 3 Superseded Latest is v3 →

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/3 Add on2-8170 kernel modules

---
 package/Config.in                            |  1 +
 package/on2-8170-modules/Config.in           | 10 ++++++++++
 package/on2-8170-modules/on2-8170-modules.mk | 20 ++++++++++++++++++++
 3 files changed, 31 insertions(+)
 create mode 100644 package/on2-8170-modules/Config.in
 create mode 100644 package/on2-8170-modules/on2-8170-modules.mk
package/Config.in +1 -0
index ce82e99..e915abe 100644
@@ -272,6 +272,7 @@ source "package/minicom/Config.in"
 source "package/nanocom/Config.in"
 source "package/neard/Config.in"
 source "package/ofono/Config.in"
+source "package/on2-8170-modules/Config.in"
 source "package/open2300/Config.in"
 source "package/openocd/Config.in"
 source "package/owl-linux/Config.in"
package/on2-8170-modules/Config.in new file +10 -0
new file mode 100644
index 0000000..f082c5f
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_ON2_8170_MODULES
+	bool "on2-8170 modules"
+	depends on BR2_LINUX_KERNEL
+	help
+	  Kernel modules for the Hantro x170 hardware video decompression.
+
+	  http://www.at91.com/linux4sam/bin/view/Linux4SAM/SAM9M10Gstreamer
+
+comment "on2-8170 modules requires a linux kernel to be built"
+	depends on !BR2_LINUX_KERNEL
package/on2-8170-modules/on2-8170-modules.mk new file +20 -0
new file mode 100644
index 0000000..53286fb
@@ -0,0 +1,20 @@
+################################################################################
+#
+# on2-8170-modules
+#
+################################################################################
+
+ON2_8170_MODULES_VERSION = 73b08061d30789178e692bc332b73d1d9922bf39
+ON2_8170_MODULES_SITE    = git://github.com/alexandrebelloni/on2-8170-modules.git
+
+ON2_8170_MODULES_DEPENDENCIES = linux
+
+define ON2_8170_MODULES_BUILD_CMDS
+	$(MAKE) -C $(LINUX_DIR) $(LINUX_MAKE_FLAGS) M=$(@D)
+endef
+
+define ON2_8170_MODULES_INSTALL_TARGET_CMDS
+	$(MAKE) -C $(LINUX_DIR) $(LINUX_MAKE_FLAGS) M=$(@D) modules_install
+endef
+
+$(eval $(generic-package))
Signature · 1 line
1.8.1.2
#1 lore raw

PATCH v2 2/3 Add proprietary libraries for Hantro x170 HW decoding

---
 package/multimedia/Config.in                      |  1 +
 package/multimedia/on2-8170-libs/Config.in        | 12 ++++++++++++
 package/multimedia/on2-8170-libs/on2-8170-libs.mk | 24 +++++++++++++++++++++++
 3 files changed, 37 insertions(+)
 create mode 100644 package/multimedia/on2-8170-libs/Config.in
 create mode 100644 package/multimedia/on2-8170-libs/on2-8170-libs.mk
package/multimedia/Config.in +1 -0
index dd74863..4cb81bf 100644
@@ -21,6 +21,7 @@ source "package/multimedia/mpd/Config.in"
 source "package/multimedia/mpg123/Config.in"
 source "package/multimedia/mplayer/Config.in"
 source "package/multimedia/musepack/Config.in"
+source "package/multimedia/on2-8170-libs/Config.in"
 source "package/opus-tools/Config.in"
 source "package/multimedia/pulseaudio/Config.in"
 source "package/multimedia/tidsp-binaries/Config.in"
package/multimedia/on2-8170-libs/Config.in new file +12 -0
new file mode 100644
index 0000000..09ed808
@@ -0,0 +1,12 @@
+config BR2_PACKAGE_ON2_8170_LIBS
+	#This is a binary only package which has been compiled for glibc
+	depends on BR2_arm926t
+	depends on (BR2_TOOLCHAIN_EXTERNAL_GLIBC || \
+		BR2_TOOLCHAIN_CTNG_eglibc || \
+		BR2_TOOLCHAIN_CTNG_glibc)
+	bool "on2-8170-libs"
+	help
+	  Libraries for Hantro X170 video decoder
+
+	  http://www.at91.com/linux4sam/bin/view/Linux4SAM/SAM9M10Gstreamer
+
package/multimedia/on2-8170-libs/on2-8170-libs.mk new file +24 -0
new file mode 100644
index 0000000..e04551d
@@ -0,0 +1,24 @@
+################################################################################
+#
+# on2-8170-libs
+#
+################################################################################
+
+ON2_8170_LIBS_VERSION = 1.0
+ON2_8170_LIBS_SITE = ftp://ftp.linux4sam.org/pub/demo/linux4sam_1.9/codec/
+
+ON2_8170_LICENSE = PROPRIETARY
+# No license file is included in the archive
+
+ON2_8170_LIBS_INSTALL_STAGING = YES
+
+define ON2_8170_LIBS_INSTALL_STAGING_CMDS
+	cp -dpf $(@D)/*.a $(@D)/*.so $(STAGING_DIR)/usr/lib
+	cp -dpf $(@D)/*.h $(STAGING_DIR)/usr/include
+endef
+
+define ON2_8170_LIBS_INSTALL_TARGET_CMDS
+	cp -dpf $(@D)/*.so $(TARGET_DIR)/usr/lib
+endef
+
+$(eval $(generic-package))
Signature · 1 line
1.8.1.2
#2 lore raw

PATCH v2 3/3 Add gstreamer plugin for Hantro x170

In reply to Alexandre Belloni
---
 package/multimedia/Config.in                       |  1 +
 package/multimedia/gst-plugin-x170/Config.in       | 14 +++++++++++
 .../gst-plugin-x170-01-correct-CFLAGS.patch        | 28 ++++++++++++++++++++++
 .../multimedia/gst-plugin-x170/gst-plugin-x170.mk  | 18 ++++++++++++++
 4 files changed, 61 insertions(+)
 create mode 100644 package/multimedia/gst-plugin-x170/Config.in
 create mode 100644 package/multimedia/gst-plugin-x170/gst-plugin-x170-01-correct-CFLAGS.patch
 create mode 100644 package/multimedia/gst-plugin-x170/gst-plugin-x170.mk
package/multimedia/Config.in +1 -0
index 4cb81bf..77c0a22 100644
@@ -14,6 +14,7 @@ source "package/multimedia/gst-plugins-base/Config.in"
 source "package/multimedia/gst-plugins-good/Config.in"
 source "package/multimedia/gst-plugins-bad/Config.in"
 source "package/multimedia/gst-plugins-ugly/Config.in"
+source "package/multimedia/gst-plugin-x170/Config.in"
 source "package/multimedia/lame/Config.in"
 source "package/multimedia/libvpx/Config.in"
 source "package/multimedia/madplay/Config.in"
package/multimedia/gst-plugin-x170/Config.in new file +14 -0
new file mode 100644
index 0000000..0787013
@@ -0,0 +1,14 @@
+config BR2_PACKAGE_GST_PLUGIN_X170
+	bool "gst-plugin-x170"
+	depends on BR2_PACKAGE_GSTREAMER
+	depends on BR2_arm926t
+	depends on (BR2_TOOLCHAIN_EXTERNAL_GLIBC || \
+		BR2_TOOLCHAIN_CTNG_eglibc || \
+		BR2_TOOLCHAIN_CTNG_glibc)
+	select BR2_PACKAGE_ON2_8170_LIBS
+	help
+	  GStreamer plug-in to use the Hantro X170 video decoder present on
+	  ATMEL AT91SAM9M10 SoC.
+
+	  http://www.at91.com/linux4sam/bin/view/Linux4SAM/SAM9M10Gstreamer
+
package/multimedia/gst-plugin-x170/gst-plugin-x170-01-correct-CFLAGS.patch new file +28 -0
new file mode 100644
index 0000000..9c32e46
@@ -0,0 +1,28 @@
+The configure script is correctly getting the CFLAGS needed to compile a plugin
+for gstreamer and storing them in GST_BASE_CFLAGS but the Makefiles are never
+making use of those.
+
+We actually have to use AM_CPPFLAGS as AM_CFLAGS is used everywhere but on the
+real compiling rule...
+
+Signed-off-by: Alexandre Belloni <[email protected]>
+---
+ src/Makefile.am | 2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/src/Makefile.am b/src/Makefile.am
+index 8cb51d1..6af5d91 100644
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -5,6 +5,8 @@
+ ##############################################################################
+ plugin_LTLIBRARIES = libgstx170.la
+ 
++AM_CPPFLAGS = @GST_BASE_CFLAGS@
++
+ ##############################################################################
+ # for the next set of variables, rename the prefix if you renamed the .la,   #
+ #  e.g. libgstplugin_la_SOURCES => libmysomething_la_SOURCES                 #
+-- 
+1.8.1.2
+
package/multimedia/gst-plugin-x170/gst-plugin-x170.mk new file +18 -0
new file mode 100644
index 0000000..a6143ed
@@ -0,0 +1,18 @@
+################################################################################
+#
+# gst-plugins-x170
+#
+################################################################################
+
+GST_PLUGIN_X170_VERSION = 1.0
+GST_PLUGIN_X170_SITE = ftp://ftp.linux4sam.org/pub/demo/linux4sam_1.9/codec/
+
+GST_PLUGIN_X170_LICENSE = BSD-1c
+#A license file is included but it is just a placeholder
+
+# There is no generated configure script in the tarball.
+GST_PLUGIN_X170_AUTORECONF = YES
+GST_PLUGIN_X170_AUTORECONF_OPT = -Im4/
+GST_PLUGIN_X170_DEPENDENCIES = gstreamer libglib2 on2-8170-libs
+
+$(eval $(autotools-package))
Signature · 1 line
1.8.1.2
#3 lore raw

PATCH v2 1/3 Add on2-8170 kernel modules

In reply to Alexandre Belloni

package/on2-8170-modules/Config.in line 3

Quoted 9 lines · Alexandre Belloni
package/on2-8170-modules/Config.in new file +3 -0
new file mode 100644
index 0000000..f082c5f
@@ -0,0 +1,10 @@
+config BR2_PACKAGE_ON2_8170_MODULES
+	bool "on2-8170 modules"
+	depends on BR2_LINUX_KERNEL

We usually add a "depends on BR2_arm" or something similar for such
SoC-specific packages.

I'm not sure though if everyone agrees that this is a good ide.

Quoted 13 lines
package/on2-8170-modules/Config.in new file +7 -0
+	help
+	  Kernel modules for the Hantro x170 hardware video decompression.
+
+	http://www.at91.com/linux4sam/bin/view/Linux4SAM/SAM9M10Gstreamer
+
+comment "on2-8170 modules requires a linux kernel to be built"
+	depends on !BR2_LINUX_KERNEL
package/on2-8170-modules/on2-8170-modules.mk new file +0 -0
new file mode 100644
index 0000000..53286fb

package/on2-8170-modules/on2-8170-modules.mk line 8

Quoted 8 lines
package/on2-8170-modules/on2-8170-modules.mk new file +8 -0
@@ -0,0 +1,20 @@
+################################################################################
+#
+# on2-8170-modules
+#
+################################################################################
+
+ON2_8170_MODULES_VERSION = 73b08061d30789178e692bc332b73d1d9922bf39
+ON2_8170_MODULES_SITE    = git://github.com/alexandrebelloni/on2-8170-modules.git

Can you use a tarball, cfr.
docs/manual/adding-packages-tips.txt:[[github-download-url]] ?

Could you add license info?

Regards,
Arnout

Quoted 12 lines
package/on2-8170-modules/on2-8170-modules.mk new file +12 -0
+
+ON2_8170_MODULES_DEPENDENCIES = linux
+
+define ON2_8170_MODULES_BUILD_CMDS
+	$(MAKE) -C $(LINUX_DIR) $(LINUX_MAKE_FLAGS) M=$(@D)
+endef
+
+define ON2_8170_MODULES_INSTALL_TARGET_CMDS
+	$(MAKE) -C $(LINUX_DIR) $(LINUX_MAKE_FLAGS) M=$(@D) modules_install
+endef
+
+$(eval $(generic-package))
Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
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:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F