Skip to contentSkip to conversation

Add on2-8170 kernel modules

PATCH Superseded Alexandre Belloni 2 participants 5 messages Jun 9, 2013

v1 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Alexandre Belloni

author
#0 lore raw

PATCH 1/3 Add on2-8170 kernel modules

Signed-off-by Alexandre Belloni <[email protected]>
---
 package/Config.in                            |  1 +
 package/on2-8170-modules/Config.in           | 11 +++++++++++
 package/on2-8170-modules/on2-8170-modules.mk | 20 ++++++++++++++++++++
 3 files changed, 32 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 +11 -0
new file mode 100644
index 0000000..b11d69e
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_ON2_8170_MODULES
+	bool "on2-8170 modules"
+	depends on BR2_LINUX_KERNEL
+	help
+	  Kernel modules for the LTTng 2.0 kernel tracing
+	  infrastructure.
+
+	  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..87dc8f2
@@ -0,0 +1,20 @@
+################################################################################
+#
+# on2-8170-modules
+#
+################################################################################
+
+ON2_8170_MODULES_VERSION = master
+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
author
#1 lore raw

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

Signed-off-by Alexandre Belloni <[email protected]>
---
 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
author
#2 lore raw

PATCH 3/3 Add gstreamer plugin for Hantro x170

In reply to Alexandre Belloni
Signed-off-by Alexandre Belloni <[email protected]>
---
 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 1/3 Add on2-8170 kernel modules

In reply to Alexandre Belloni

Hi Alexandre,

Quoted 27 lines · Alexandre Belloni
Signed-off-by Alexandre Belloni <[email protected]>
---
 package/Config.in                            |  1 +
 package/on2-8170-modules/Config.in           | 11 +++++++++++
 package/on2-8170-modules/on2-8170-modules.mk | 20 ++++++++++++++++++++
 3 files changed, 32 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 +0 -0
new file mode 100644
index 0000000..b11d69e

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

Quoted 6 lines
package/on2-8170-modules/Config.in new file +6 -0
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_ON2_8170_MODULES
+	bool "on2-8170 modules"
+	depends on BR2_LINUX_KERNEL
+	help
+	  Kernel modules for the LTTng 2.0 kernel tracing
+	  infrastructure.

Are you sure?

Quoted 10 lines
package/on2-8170-modules/Config.in new file +4 -0
+	  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..87dc8f2

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

Quoted 7 lines
package/on2-8170-modules/on2-8170-modules.mk new file +7 -0
@@ -0,0 +1,20 @@
+################################################################################
+#
+# on2-8170-modules
+#
+################################################################################
+
+ON2_8170_MODULES_VERSION = master

We usually use a specific commit id for consistent results, since maser changes when commits are added.

baruch

Quoted 13 lines
package/on2-8170-modules/on2-8170-modules.mk new file +13 -0
+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 · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
author
#4 lore raw

PATCH 1/3 Add on2-8170 kernel modules

Hi,

Right, two things I w anted to change before sending ;)

v2 is incoming.

Quoted 73 lines · Baruch Siach

Hi Alexandre,

Quoted 27 lines · Alexandre Belloni
Signed-off-by Alexandre Belloni <[email protected]>
---
 package/Config.in                            |  1 +
 package/on2-8170-modules/Config.in           | 11 +++++++++++
 package/on2-8170-modules/on2-8170-modules.mk | 20 ++++++++++++++++++++
 3 files changed, 32 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 +0 -0
new file mode 100644
index 0000000..b11d69e

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

Quoted 6 lines
package/on2-8170-modules/Config.in new file +6 -0
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_ON2_8170_MODULES
+	bool "on2-8170 modules"
+	depends on BR2_LINUX_KERNEL
+	help
+	  Kernel modules for the LTTng 2.0 kernel tracing
+	  infrastructure.

Are you sure?

Quoted 10 lines
package/on2-8170-modules/Config.in new file +4 -0
+	  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..87dc8f2

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

Quoted 7 lines
package/on2-8170-modules/on2-8170-modules.mk new file +7 -0
@@ -0,0 +1,20 @@
+################################################################################
+#
+# on2-8170-modules
+#
+################################################################################
+
+ON2_8170_MODULES_VERSION = master

We usually use a specific commit id for consistent results, since maser changes when commits are added.

baruch

Quoted 13 lines
package/on2-8170-modules/on2-8170-modules.mk new file +13 -0
+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 · 3 lines
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com