* [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
@ 2016-03-31 9:37 ` Yuqing Zhu
2016-03-31 15:05 ` Otavio Salvador
2016-03-31 15:14 ` Carlos Rafael Giani
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 2/5] libfslparser: " Yuqing Zhu
` (4 subsequent siblings)
5 siblings, 2 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-03-31 9:37 UTC (permalink / raw)
To: meta-freescale; +Cc: Yuqing Zhu
--Change the recipe name from "libfslcodec" to "imx-codec"
--[aac_dec] refine the code for pure c build
aac decoder c build need float point support, which use some
float point function.
Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
---
.../{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} | 10 +++++-----
recipes-multimedia/imx-codec/imx-codec_4.0.9.bb | 10 ++++++++++
recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb | 9 ---------
3 files changed, 15 insertions(+), 14 deletions(-)
rename recipes-multimedia/{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} (91%)
create mode 100644 recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
delete mode 100644 recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
diff --git a/recipes-multimedia/libfslcodec/libfslcodec.inc b/recipes-multimedia/imx-codec/imx-codec.inc
similarity index 91%
rename from recipes-multimedia/libfslcodec/libfslcodec.inc
rename to recipes-multimedia/imx-codec/imx-codec.inc
index 2f0bc44..b2aca48 100644
--- a/recipes-multimedia/libfslcodec/libfslcodec.inc
+++ b/recipes-multimedia/imx-codec/imx-codec.inc
@@ -1,4 +1,4 @@
-# Copyright (C) 2012-2014 Freescale Semiconductor
+# Copyright (C) 2012-2016 Freescale Semiconductor
# Released under the MIT license (see COPYING.MIT for the terms)
DESCRIPTION = "Freescale Multimedia codec libs"
LICENSE = "Proprietary"
@@ -41,7 +41,7 @@ python __set_insane_skip() {
# FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
# the source we cannot fix it. Disable the insane check for now.
- if p == 'libfslcodec-test-bin':
+ if p == 'imx-codec-test-bin':
# FIXME: includes the DUT .so files so we need to deploy those
d.setVar("INSANE_SKIP_%s" % p, "ldflags textrel libdir")
else:
@@ -54,7 +54,7 @@ python __split_libfslcodec_plugins() {
codecdir = bb.data.expand('${libdir}', d)
do_split_packages(d, codecdir, '^lib_([^_]*).*_arm.*_elinux\.so\..*',
aux_files_pattern='${libdir}/imx-mm/audio-codec/wrap/lib_%sd_wrap_arm*_elinux.so.*',
- output_pattern='libfslcodec-%s',
+ output_pattern='imx-codec-%s',
description='Freescale i.MX Codec (%s)',
extra_depends='')
}
@@ -63,9 +63,9 @@ python __set_metapkg_rdepends() {
# Allow addition of all codecs in a image; useful specially for
# debugging.
codec_pkgs = oe.utils.packages_filter_out_system(d)
- codec_pkgs = filter(lambda x: x not in ['libfslcodec-test-bin', 'libfslcodec-test-source'],
+ codec_pkgs = filter(lambda x: x not in ['imx-codec-test-bin', 'imx-codec-test-source'],
codec_pkgs)
- d.appendVar('RDEPENDS_libfslcodec-meta', ' ' + ' '.join(codec_pkgs))
+ d.appendVar('RDEPENDS_imx-codec-meta', ' ' + ' '.join(codec_pkgs))
}
PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
diff --git a/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
new file mode 100644
index 0000000..e7bdfc8
--- /dev/null
+++ b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
@@ -0,0 +1,10 @@
+# Copyright (C) 2013-2016 Freescale Semiconductor
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+require imx-codec.inc
+LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
+
+SRC_URI[md5sum] = "bcf54698af6f52958b36e34dcbd9fcd1"
+SRC_URI[sha256sum] = "62bda182143cde9743cba0eb2a1c315a53cb434294605742f9fc364219958888"
+
+COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
diff --git a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb b/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
deleted file mode 100644
index 1b32f90..0000000
--- a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (C) 2013-2015 Freescale Semiconductor
-# Released under the MIT license (see COPYING.MIT for the terms)
-
-require libfslcodec.inc
-
-SRC_URI[md5sum] = "462fb27b20cd7df56a49b9e7d6b7c8b4"
-SRC_URI[sha256sum] = "544ffc5989bce18ca50c7a826cc03370b1cf8455335e4291ef0c0779b4c33de0"
-
-COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
--
1.9.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9 Yuqing Zhu
@ 2016-03-31 15:05 ` Otavio Salvador
2016-04-01 9:00 ` Yuqing Zhu
2016-03-31 15:14 ` Carlos Rafael Giani
1 sibling, 1 reply; 15+ messages in thread
From: Otavio Salvador @ 2016-03-31 15:05 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: meta-freescale@yoctoproject.org
On Thu, Mar 31, 2016 at 6:37 AM, Yuqing Zhu <carol.zhu@nxp.com> wrote:
> --Change the recipe name from "libfslcodec" to "imx-codec"
>
> --[aac_dec] refine the code for pure c build
> aac decoder c build need float point support, which use some
> float point function.
>
> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
Split this change in two patches:
- One to do the rename;
- One for the upgrade;
The rename one needs to also fix the upgrade path.
It needs to take care of runtime packages and as well as the depends
(PROVIDES and RPROVIDES/RCONFLICTS/RREPLACES).
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 15:05 ` Otavio Salvador
@ 2016-04-01 9:00 ` Yuqing Zhu
2016-04-01 11:19 ` Otavio Salvador
0 siblings, 1 reply; 15+ messages in thread
From: Yuqing Zhu @ 2016-04-01 9:00 UTC (permalink / raw)
To: Otavio Salvador; +Cc: meta-freescale@yoctoproject.org
Hi Otavio,
For now, the gst1.0-fsl-plugin 4.0.8 has some build errors.
It seems that the gstreamer has been upgraded to 1.6, but gst1.0 v4.0.8 is for gstreamer 1.4.5, which cause the build error.
And renaming first still can't pass the compiling.
How about this:
Upgrade to v 4.0.9( just create new recipe but not delete the old ones), make the gst1.0 pass the compiling firstly,
and then delete the old recipes? Which can avoid building error of any recipe's using old name.
B.R.
Yuqing Zhu
-----Original Message-----
From: Otavio Salvador [mailto:otavio.salvador@ossystems.com.br]
Sent: Thursday, March 31, 2016 11:05 PM
To: Yuqing Zhu <carol.zhu@nxp.com>
Cc: meta-freescale@yoctoproject.org
Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
On Thu, Mar 31, 2016 at 6:37 AM, Yuqing Zhu <carol.zhu@nxp.com> wrote:
> --Change the recipe name from "libfslcodec" to "imx-codec"
>
> --[aac_dec] refine the code for pure c build
> aac decoder c build need float point support, which use some
> float point function.
>
> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
Split this change in two patches:
- One to do the rename;
- One for the upgrade;
The rename one needs to also fix the upgrade path.
It needs to take care of runtime packages and as well as the depends (PROVIDES and RPROVIDES/RCONFLICTS/RREPLACES).
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-04-01 9:00 ` Yuqing Zhu
@ 2016-04-01 11:19 ` Otavio Salvador
0 siblings, 0 replies; 15+ messages in thread
From: Otavio Salvador @ 2016-04-01 11:19 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: meta-freescale@yoctoproject.org
Hello Zhu,
On Fri, Apr 1, 2016 at 6:00 AM, Yuqing Zhu <carol.zhu@nxp.com> wrote:
> For now, the gst1.0-fsl-plugin 4.0.8 has some build errors.
> It seems that the gstreamer has been upgraded to 1.6, but gst1.0 v4.0.8 is for gstreamer 1.4.5, which cause the build error.
> And renaming first still can't pass the compiling.
gst1.0-fsl-plugin is the last patch of the serie so it will use the
renamed dependencies.
> How about this:
> Upgrade to v 4.0.9( just create new recipe but not delete the old ones), make the gst1.0 pass the compiling firstly,
> and then delete the old recipes? Which can avoid building error of any recipe's using old name.
Does not work. The imx-codec replaces the libfslcodec and as such it
needs to be done when upgrading.
We can do it at same commit, no problem, but it needs to:
- PROVIDES old recipe name
- have code the generate the RREPLACE/RPROVIDES/RCONFLICTS of every binary.
So for imx-codec, the former package generated:
libfslcodec-test-bin_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [0]
libfslcodec-png_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [1]
libfslcodec-g.723.1_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [2]
libfslcodec-jpeg_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [3]
libfslcodec-h264_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [4]
libfslcodec-bmp_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [5]
libfslcodec-g.729ab_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [6]
libfslcodec-gif_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [7]
libfslcodec-sbc_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [8]
libfslcodec-test-source_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [9]
libfslcodec_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [10]
libfslcodec-g.726_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [11]
libfslcodec-doc_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [12]
libfslcodec-meta_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [13]
libfslcodec-nb_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [14]
libfslcodec-flac_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [15]
libfslcodec-g.711_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [16]
libfslcodec-dev_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [17]
libfslcodec-wb_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [18]
libfslcodec-mpeg4asp_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [19]
libfslcodec-dbg_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [20]
libfslcodec-aac_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [21]
libfslcodec-mp3_4.0.7-r0.3_cortexa9hf-vfp-neon-mx6qdl.ipk [22]
and each counterpart needs to handle the upgrade. For example,
imx-codec-aac needs to have propre RREPLACE/RPROVIDES/RCONFLICTS for
libfslcodec-aac.
In the former __split_libfslcodec_plugins function, you can handle the
plugins subpackages, easily.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9 Yuqing Zhu
2016-03-31 15:05 ` Otavio Salvador
@ 2016-03-31 15:14 ` Carlos Rafael Giani
2016-03-31 16:22 ` Otavio Salvador
2016-04-06 8:36 ` Yuqing Zhu
1 sibling, 2 replies; 15+ messages in thread
From: Carlos Rafael Giani @ 2016-03-31 15:14 UTC (permalink / raw)
To: meta-freescale
I have a suggestion for this recipe.
The uniaudio codecs require knowledge about the SONAME of the plugin.
Without this SONAME, accessing these plugins is not possible. For this
reason, imx-gst1.0-plugin installs a file in /etc/ that contains a list
of these plugins.
(gstreamer-imx scans for plugins and extracts the SONAME at build time
instead.)
It would make more sense to add such a file to the libfslcodec package.
This list would contain a list of each uniaudio plugin along with its
SONAME. This way, other packages such as imx-gst1.0-plugin and
gstreamer-imx do not have to scan the plugins on their own for the
SONAME - they can just rely on this list file to exist in the rootfs.
On 03/31/2016 11:37 AM, Yuqing Zhu wrote:
> --Change the recipe name from "libfslcodec" to "imx-codec"
>
> --[aac_dec] refine the code for pure c build
> aac decoder c build need float point support, which use some
> float point function.
>
> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
> ---
> .../{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} | 10 +++++-----
> recipes-multimedia/imx-codec/imx-codec_4.0.9.bb | 10 ++++++++++
> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb | 9 ---------
> 3 files changed, 15 insertions(+), 14 deletions(-)
> rename recipes-multimedia/{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} (91%)
> create mode 100644 recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> delete mode 100644 recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>
> diff --git a/recipes-multimedia/libfslcodec/libfslcodec.inc b/recipes-multimedia/imx-codec/imx-codec.inc
> similarity index 91%
> rename from recipes-multimedia/libfslcodec/libfslcodec.inc
> rename to recipes-multimedia/imx-codec/imx-codec.inc
> index 2f0bc44..b2aca48 100644
> --- a/recipes-multimedia/libfslcodec/libfslcodec.inc
> +++ b/recipes-multimedia/imx-codec/imx-codec.inc
> @@ -1,4 +1,4 @@
> -# Copyright (C) 2012-2014 Freescale Semiconductor
> +# Copyright (C) 2012-2016 Freescale Semiconductor
> # Released under the MIT license (see COPYING.MIT for the terms)
> DESCRIPTION = "Freescale Multimedia codec libs"
> LICENSE = "Proprietary"
> @@ -41,7 +41,7 @@ python __set_insane_skip() {
>
> # FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
> # the source we cannot fix it. Disable the insane check for now.
> - if p == 'libfslcodec-test-bin':
> + if p == 'imx-codec-test-bin':
> # FIXME: includes the DUT .so files so we need to deploy those
> d.setVar("INSANE_SKIP_%s" % p, "ldflags textrel libdir")
> else:
> @@ -54,7 +54,7 @@ python __split_libfslcodec_plugins() {
> codecdir = bb.data.expand('${libdir}', d)
> do_split_packages(d, codecdir, '^lib_([^_]*).*_arm.*_elinux\.so\..*',
> aux_files_pattern='${libdir}/imx-mm/audio-codec/wrap/lib_%sd_wrap_arm*_elinux.so.*',
> - output_pattern='libfslcodec-%s',
> + output_pattern='imx-codec-%s',
> description='Freescale i.MX Codec (%s)',
> extra_depends='')
> }
> @@ -63,9 +63,9 @@ python __set_metapkg_rdepends() {
> # Allow addition of all codecs in a image; useful specially for
> # debugging.
> codec_pkgs = oe.utils.packages_filter_out_system(d)
> - codec_pkgs = filter(lambda x: x not in ['libfslcodec-test-bin', 'libfslcodec-test-source'],
> + codec_pkgs = filter(lambda x: x not in ['imx-codec-test-bin', 'imx-codec-test-source'],
> codec_pkgs)
> - d.appendVar('RDEPENDS_libfslcodec-meta', ' ' + ' '.join(codec_pkgs))
> + d.appendVar('RDEPENDS_imx-codec-meta', ' ' + ' '.join(codec_pkgs))
> }
>
> PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
> diff --git a/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> new file mode 100644
> index 0000000..e7bdfc8
> --- /dev/null
> +++ b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> @@ -0,0 +1,10 @@
> +# Copyright (C) 2013-2016 Freescale Semiconductor
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +
> +require imx-codec.inc
> +LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
> +
> +SRC_URI[md5sum] = "bcf54698af6f52958b36e34dcbd9fcd1"
> +SRC_URI[sha256sum] = "62bda182143cde9743cba0eb2a1c315a53cb434294605742f9fc364219958888"
> +
> +COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
> diff --git a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb b/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
> deleted file mode 100644
> index 1b32f90..0000000
> --- a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -# Copyright (C) 2013-2015 Freescale Semiconductor
> -# Released under the MIT license (see COPYING.MIT for the terms)
> -
> -require libfslcodec.inc
> -
> -SRC_URI[md5sum] = "462fb27b20cd7df56a49b9e7d6b7c8b4"
> -SRC_URI[sha256sum] = "544ffc5989bce18ca50c7a826cc03370b1cf8455335e4291ef0c0779b4c33de0"
> -
> -COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 15:14 ` Carlos Rafael Giani
@ 2016-03-31 16:22 ` Otavio Salvador
2016-04-06 8:36 ` Yuqing Zhu
1 sibling, 0 replies; 15+ messages in thread
From: Otavio Salvador @ 2016-03-31 16:22 UTC (permalink / raw)
To: Carlos Rafael Giani; +Cc: meta-freescale@yoctoproject.org
On Thu, Mar 31, 2016 at 12:14 PM, Carlos Rafael Giani
<dv@pseudoterminal.org> wrote:
> I have a suggestion for this recipe.
>
> The uniaudio codecs require knowledge about the SONAME of the plugin.
> Without this SONAME, accessing these plugins is not possible. For this
> reason, imx-gst1.0-plugin installs a file in /etc/ that contains a list of
> these plugins.
> (gstreamer-imx scans for plugins and extracts the SONAME at build time
> instead.)
>
> It would make more sense to add such a file to the libfslcodec package. This
> list would contain a list of each uniaudio plugin along with its SONAME.
> This way, other packages such as imx-gst1.0-plugin and gstreamer-imx do not
> have to scan the plugins on their own for the SONAME - they can just rely on
> this list file to exist in the rootfs.
I agree with the general idea however this needs to be done for next
imx-codec release as this must be included in the build process; to be
honest, the backward compatibility will be a headache as the content
user may, or may not, have it at disposal.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-03-31 15:14 ` Carlos Rafael Giani
2016-03-31 16:22 ` Otavio Salvador
@ 2016-04-06 8:36 ` Yuqing Zhu
2016-04-06 8:43 ` Carlos Rafael Giani
1 sibling, 1 reply; 15+ messages in thread
From: Yuqing Zhu @ 2016-04-06 8:36 UTC (permalink / raw)
To: Carlos Rafael Giani, meta-freescale@yoctoproject.org
Hi Carlos,
Thanks for your feedback.
But it's not reasonable to add this file to imx-codec, it's gstreamer plugin specific file, imx-codec is a codec general package.
User doesn't need to know SONAME, they just link the lib_codec.so in the makefile.
B.R.
Yuqing Zhu
-----Original Message-----
From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-bounces@yoctoproject.org] On Behalf Of Carlos Rafael Giani
Sent: Thursday, March 31, 2016 11:14 PM
To: meta-freescale@yoctoproject.org
Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
I have a suggestion for this recipe.
The uniaudio codecs require knowledge about the SONAME of the plugin.
Without this SONAME, accessing these plugins is not possible. For this reason, imx-gst1.0-plugin installs a file in /etc/ that contains a list of these plugins.
(gstreamer-imx scans for plugins and extracts the SONAME at build time
instead.)
It would make more sense to add such a file to the libfslcodec package.
This list would contain a list of each uniaudio plugin along with its SONAME. This way, other packages such as imx-gst1.0-plugin and gstreamer-imx do not have to scan the plugins on their own for the SONAME - they can just rely on this list file to exist in the rootfs.
On 03/31/2016 11:37 AM, Yuqing Zhu wrote:
> --Change the recipe name from "libfslcodec" to "imx-codec"
>
> --[aac_dec] refine the code for pure c build
> aac decoder c build need float point support, which use some
> float point function.
>
> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
> ---
> .../{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} | 10 +++++-----
> recipes-multimedia/imx-codec/imx-codec_4.0.9.bb | 10 ++++++++++
> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb | 9 ---------
> 3 files changed, 15 insertions(+), 14 deletions(-)
> rename recipes-multimedia/{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} (91%)
> create mode 100644 recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> delete mode 100644
> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>
> diff --git a/recipes-multimedia/libfslcodec/libfslcodec.inc
> b/recipes-multimedia/imx-codec/imx-codec.inc
> similarity index 91%
> rename from recipes-multimedia/libfslcodec/libfslcodec.inc
> rename to recipes-multimedia/imx-codec/imx-codec.inc
> index 2f0bc44..b2aca48 100644
> --- a/recipes-multimedia/libfslcodec/libfslcodec.inc
> +++ b/recipes-multimedia/imx-codec/imx-codec.inc
> @@ -1,4 +1,4 @@
> -# Copyright (C) 2012-2014 Freescale Semiconductor
> +# Copyright (C) 2012-2016 Freescale Semiconductor
> # Released under the MIT license (see COPYING.MIT for the terms)
> DESCRIPTION = "Freescale Multimedia codec libs"
> LICENSE = "Proprietary"
> @@ -41,7 +41,7 @@ python __set_insane_skip() {
>
> # FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
> # the source we cannot fix it. Disable the insane check for now.
> - if p == 'libfslcodec-test-bin':
> + if p == 'imx-codec-test-bin':
> # FIXME: includes the DUT .so files so we need to deploy those
> d.setVar("INSANE_SKIP_%s" % p, "ldflags textrel libdir")
> else:
> @@ -54,7 +54,7 @@ python __split_libfslcodec_plugins() {
> codecdir = bb.data.expand('${libdir}', d)
> do_split_packages(d, codecdir, '^lib_([^_]*).*_arm.*_elinux\.so\..*',
> aux_files_pattern='${libdir}/imx-mm/audio-codec/wrap/lib_%sd_wrap_arm*_elinux.so.*',
> - output_pattern='libfslcodec-%s',
> + output_pattern='imx-codec-%s',
> description='Freescale i.MX Codec (%s)',
> extra_depends='')
> }
> @@ -63,9 +63,9 @@ python __set_metapkg_rdepends() {
> # Allow addition of all codecs in a image; useful specially for
> # debugging.
> codec_pkgs = oe.utils.packages_filter_out_system(d)
> - codec_pkgs = filter(lambda x: x not in ['libfslcodec-test-bin', 'libfslcodec-test-source'],
> + codec_pkgs = filter(lambda x: x not in ['imx-codec-test-bin',
> + 'imx-codec-test-source'],
> codec_pkgs)
> - d.appendVar('RDEPENDS_libfslcodec-meta', ' ' + ' '.join(codec_pkgs))
> + d.appendVar('RDEPENDS_imx-codec-meta', ' ' + '
> + '.join(codec_pkgs))
> }
>
> PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
> diff --git a/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> new file mode 100644
> index 0000000..e7bdfc8
> --- /dev/null
> +++ b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
> @@ -0,0 +1,10 @@
> +# Copyright (C) 2013-2016 Freescale Semiconductor # Released under
> +the MIT license (see COPYING.MIT for the terms)
> +
> +require imx-codec.inc
> +LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
> +
> +SRC_URI[md5sum] = "bcf54698af6f52958b36e34dcbd9fcd1"
> +SRC_URI[sha256sum] = "62bda182143cde9743cba0eb2a1c315a53cb434294605742f9fc364219958888"
> +
> +COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
> diff --git a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
> b/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
> deleted file mode 100644
> index 1b32f90..0000000
> --- a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
> +++ /dev/null
> @@ -1,9 +0,0 @@
> -# Copyright (C) 2013-2015 Freescale Semiconductor -# Released under
> the MIT license (see COPYING.MIT for the terms)
> -
> -require libfslcodec.inc
> -
> -SRC_URI[md5sum] = "462fb27b20cd7df56a49b9e7d6b7c8b4"
> -SRC_URI[sha256sum] = "544ffc5989bce18ca50c7a826cc03370b1cf8455335e4291ef0c0779b4c33de0"
> -
> -COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
--
_______________________________________________
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-04-06 8:36 ` Yuqing Zhu
@ 2016-04-06 8:43 ` Carlos Rafael Giani
2016-04-13 7:50 ` Yuqing Zhu
0 siblings, 1 reply; 15+ messages in thread
From: Carlos Rafael Giani @ 2016-04-06 8:43 UTC (permalink / raw)
To: Yuqing Zhu, meta-freescale@yoctoproject.org
Well, users do need to know the SONAME if they are loaded with dlopen()
(and this is how their documentations suggest it should be done.)
See the beepdec registry in imx-gst1.0-plugin for an example. Also, the
configure.ac in imx-gst1.0-plugin runs objdump on the plugins to get the
SONAME.
I do not see how just linking to uniaudio codecs could work. The way
their API is designed requires you to explicitely query for a
"UniACodecQueryInterface" symbol via dlsym().
Carlos
On 2016-04-06 10:36, Yuqing Zhu wrote:
> Hi Carlos,
>
> Thanks for your feedback.
> But it's not reasonable to add this file to imx-codec, it's gstreamer plugin specific file, imx-codec is a codec general package.
> User doesn't need to know SONAME, they just link the lib_codec.so in the makefile.
>
>
>
> B.R.
> Yuqing Zhu
>
> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-bounces@yoctoproject.org] On Behalf Of Carlos Rafael Giani
> Sent: Thursday, March 31, 2016 11:14 PM
> To: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
>
> I have a suggestion for this recipe.
>
> The uniaudio codecs require knowledge about the SONAME of the plugin.
> Without this SONAME, accessing these plugins is not possible. For this reason, imx-gst1.0-plugin installs a file in /etc/ that contains a list of these plugins.
> (gstreamer-imx scans for plugins and extracts the SONAME at build time
> instead.)
>
> It would make more sense to add such a file to the libfslcodec package.
> This list would contain a list of each uniaudio plugin along with its SONAME. This way, other packages such as imx-gst1.0-plugin and gstreamer-imx do not have to scan the plugins on their own for the SONAME - they can just rely on this list file to exist in the rootfs.
>
>
> On 03/31/2016 11:37 AM, Yuqing Zhu wrote:
>> --Change the recipe name from "libfslcodec" to "imx-codec"
>>
>> --[aac_dec] refine the code for pure c build
>> aac decoder c build need float point support, which use some
>> float point function.
>>
>> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
>> ---
>> .../{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} | 10 +++++-----
>> recipes-multimedia/imx-codec/imx-codec_4.0.9.bb | 10 ++++++++++
>> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb | 9 ---------
>> 3 files changed, 15 insertions(+), 14 deletions(-)
>> rename recipes-multimedia/{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} (91%)
>> create mode 100644 recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> delete mode 100644
>> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>>
>> diff --git a/recipes-multimedia/libfslcodec/libfslcodec.inc
>> b/recipes-multimedia/imx-codec/imx-codec.inc
>> similarity index 91%
>> rename from recipes-multimedia/libfslcodec/libfslcodec.inc
>> rename to recipes-multimedia/imx-codec/imx-codec.inc
>> index 2f0bc44..b2aca48 100644
>> --- a/recipes-multimedia/libfslcodec/libfslcodec.inc
>> +++ b/recipes-multimedia/imx-codec/imx-codec.inc
>> @@ -1,4 +1,4 @@
>> -# Copyright (C) 2012-2014 Freescale Semiconductor
>> +# Copyright (C) 2012-2016 Freescale Semiconductor
>> # Released under the MIT license (see COPYING.MIT for the terms)
>> DESCRIPTION = "Freescale Multimedia codec libs"
>> LICENSE = "Proprietary"
>> @@ -41,7 +41,7 @@ python __set_insane_skip() {
>>
>> # FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
>> # the source we cannot fix it. Disable the insane check for now.
>> - if p == 'libfslcodec-test-bin':
>> + if p == 'imx-codec-test-bin':
>> # FIXME: includes the DUT .so files so we need to deploy those
>> d.setVar("INSANE_SKIP_%s" % p, "ldflags textrel libdir")
>> else:
>> @@ -54,7 +54,7 @@ python __split_libfslcodec_plugins() {
>> codecdir = bb.data.expand('${libdir}', d)
>> do_split_packages(d, codecdir, '^lib_([^_]*).*_arm.*_elinux\.so\..*',
>> aux_files_pattern='${libdir}/imx-mm/audio-codec/wrap/lib_%sd_wrap_arm*_elinux.so.*',
>> - output_pattern='libfslcodec-%s',
>> + output_pattern='imx-codec-%s',
>> description='Freescale i.MX Codec (%s)',
>> extra_depends='')
>> }
>> @@ -63,9 +63,9 @@ python __set_metapkg_rdepends() {
>> # Allow addition of all codecs in a image; useful specially for
>> # debugging.
>> codec_pkgs = oe.utils.packages_filter_out_system(d)
>> - codec_pkgs = filter(lambda x: x not in ['libfslcodec-test-bin', 'libfslcodec-test-source'],
>> + codec_pkgs = filter(lambda x: x not in ['imx-codec-test-bin',
>> + 'imx-codec-test-source'],
>> codec_pkgs)
>> - d.appendVar('RDEPENDS_libfslcodec-meta', ' ' + ' '.join(codec_pkgs))
>> + d.appendVar('RDEPENDS_imx-codec-meta', ' ' + '
>> + '.join(codec_pkgs))
>> }
>>
>> PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
>> diff --git a/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> new file mode 100644
>> index 0000000..e7bdfc8
>> --- /dev/null
>> +++ b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> @@ -0,0 +1,10 @@
>> +# Copyright (C) 2013-2016 Freescale Semiconductor # Released under
>> +the MIT license (see COPYING.MIT for the terms)
>> +
>> +require imx-codec.inc
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
>> +
>> +SRC_URI[md5sum] = "bcf54698af6f52958b36e34dcbd9fcd1"
>> +SRC_URI[sha256sum] = "62bda182143cde9743cba0eb2a1c315a53cb434294605742f9fc364219958888"
>> +
>> +COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
>> diff --git a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> b/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> deleted file mode 100644
>> index 1b32f90..0000000
>> --- a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> +++ /dev/null
>> @@ -1,9 +0,0 @@
>> -# Copyright (C) 2013-2015 Freescale Semiconductor -# Released under
>> the MIT license (see COPYING.MIT for the terms)
>> -
>> -require libfslcodec.inc
>> -
>> -SRC_URI[md5sum] = "462fb27b20cd7df56a49b9e7d6b7c8b4"
>> -SRC_URI[sha256sum] = "544ffc5989bce18ca50c7a826cc03370b1cf8455335e4291ef0c0779b4c33de0"
>> -
>> -COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale
^ permalink raw reply [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
2016-04-06 8:43 ` Carlos Rafael Giani
@ 2016-04-13 7:50 ` Yuqing Zhu
0 siblings, 0 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-04-13 7:50 UTC (permalink / raw)
To: Carlos Rafael Giani, meta-freescale@yoctoproject.org
Hi Carlos,
Actually, when make install the libfslcodec package, it will create a symbol link xxx.so and it will point to the library with .so.x (the soname) in the rootfs
So if users need to get the soname, when the package installed into rootfs, they can find the soname from the library name.
B.R.
Yuqing Zhu
-----Original Message-----
From: Carlos Rafael Giani [mailto:dv@pseudoterminal.org]
Sent: Wednesday, April 06, 2016 4:44 PM
To: Yuqing Zhu <carol.zhu@nxp.com>; meta-freescale@yoctoproject.org
Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9
Well, users do need to know the SONAME if they are loaded with dlopen() (and this is how their documentations suggest it should be done.) See the beepdec registry in imx-gst1.0-plugin for an example. Also, the configure.ac in imx-gst1.0-plugin runs objdump on the plugins to get the SONAME.
I do not see how just linking to uniaudio codecs could work. The way their API is designed requires you to explicitely query for a "UniACodecQueryInterface" symbol via dlsym().
Carlos
On 2016-04-06 10:36, Yuqing Zhu wrote:
> Hi Carlos,
>
> Thanks for your feedback.
> But it's not reasonable to add this file to imx-codec, it's gstreamer plugin specific file, imx-codec is a codec general package.
> User doesn't need to know SONAME, they just link the lib_codec.so in the makefile.
>
>
>
> B.R.
> Yuqing Zhu
>
> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org
> [mailto:meta-freescale-bounces@yoctoproject.org] On Behalf Of Carlos
> Rafael Giani
> Sent: Thursday, March 31, 2016 11:14 PM
> To: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 1/5] libfslcodec:
> Upgrade to v4.0.9
>
> I have a suggestion for this recipe.
>
> The uniaudio codecs require knowledge about the SONAME of the plugin.
> Without this SONAME, accessing these plugins is not possible. For this reason, imx-gst1.0-plugin installs a file in /etc/ that contains a list of these plugins.
> (gstreamer-imx scans for plugins and extracts the SONAME at build time
> instead.)
>
> It would make more sense to add such a file to the libfslcodec package.
> This list would contain a list of each uniaudio plugin along with its SONAME. This way, other packages such as imx-gst1.0-plugin and gstreamer-imx do not have to scan the plugins on their own for the SONAME - they can just rely on this list file to exist in the rootfs.
>
>
> On 03/31/2016 11:37 AM, Yuqing Zhu wrote:
>> --Change the recipe name from "libfslcodec" to "imx-codec"
>>
>> --[aac_dec] refine the code for pure c build
>> aac decoder c build need float point support, which use some
>> float point function.
>>
>> Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
>> ---
>> .../{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} | 10 +++++-----
>> recipes-multimedia/imx-codec/imx-codec_4.0.9.bb | 10 ++++++++++
>> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb | 9 ---------
>> 3 files changed, 15 insertions(+), 14 deletions(-)
>> rename recipes-multimedia/{libfslcodec/libfslcodec.inc => imx-codec/imx-codec.inc} (91%)
>> create mode 100644 recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> delete mode 100644
>> recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>>
>> diff --git a/recipes-multimedia/libfslcodec/libfslcodec.inc
>> b/recipes-multimedia/imx-codec/imx-codec.inc
>> similarity index 91%
>> rename from recipes-multimedia/libfslcodec/libfslcodec.inc
>> rename to recipes-multimedia/imx-codec/imx-codec.inc
>> index 2f0bc44..b2aca48 100644
>> --- a/recipes-multimedia/libfslcodec/libfslcodec.inc
>> +++ b/recipes-multimedia/imx-codec/imx-codec.inc
>> @@ -1,4 +1,4 @@
>> -# Copyright (C) 2012-2014 Freescale Semiconductor
>> +# Copyright (C) 2012-2016 Freescale Semiconductor
>> # Released under the MIT license (see COPYING.MIT for the terms)
>> DESCRIPTION = "Freescale Multimedia codec libs"
>> LICENSE = "Proprietary"
>> @@ -41,7 +41,7 @@ python __set_insane_skip() {
>>
>> # FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
>> # the source we cannot fix it. Disable the insane check for now.
>> - if p == 'libfslcodec-test-bin':
>> + if p == 'imx-codec-test-bin':
>> # FIXME: includes the DUT .so files so we need to deploy those
>> d.setVar("INSANE_SKIP_%s" % p, "ldflags textrel libdir")
>> else:
>> @@ -54,7 +54,7 @@ python __split_libfslcodec_plugins() {
>> codecdir = bb.data.expand('${libdir}', d)
>> do_split_packages(d, codecdir, '^lib_([^_]*).*_arm.*_elinux\.so\..*',
>> aux_files_pattern='${libdir}/imx-mm/audio-codec/wrap/lib_%sd_wrap_arm*_elinux.so.*',
>> - output_pattern='libfslcodec-%s',
>> + output_pattern='imx-codec-%s',
>> description='Freescale i.MX Codec (%s)',
>> extra_depends='')
>> }
>> @@ -63,9 +63,9 @@ python __set_metapkg_rdepends() {
>> # Allow addition of all codecs in a image; useful specially for
>> # debugging.
>> codec_pkgs = oe.utils.packages_filter_out_system(d)
>> - codec_pkgs = filter(lambda x: x not in ['libfslcodec-test-bin', 'libfslcodec-test-source'],
>> + codec_pkgs = filter(lambda x: x not in ['imx-codec-test-bin',
>> + 'imx-codec-test-source'],
>> codec_pkgs)
>> - d.appendVar('RDEPENDS_libfslcodec-meta', ' ' + ' '.join(codec_pkgs))
>> + d.appendVar('RDEPENDS_imx-codec-meta', ' ' + '
>> + '.join(codec_pkgs))
>> }
>>
>> PACKAGESPLITFUNCS =+ "__split_libfslcodec_plugins __set_metapkg_rdepends"
>> diff --git a/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> new file mode 100644
>> index 0000000..e7bdfc8
>> --- /dev/null
>> +++ b/recipes-multimedia/imx-codec/imx-codec_4.0.9.bb
>> @@ -0,0 +1,10 @@
>> +# Copyright (C) 2013-2016 Freescale Semiconductor # Released under
>> +the MIT license (see COPYING.MIT for the terms)
>> +
>> +require imx-codec.inc
>> +LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
>> +
>> +SRC_URI[md5sum] = "bcf54698af6f52958b36e34dcbd9fcd1"
>> +SRC_URI[sha256sum] = "62bda182143cde9743cba0eb2a1c315a53cb434294605742f9fc364219958888"
>> +
>> +COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
>> diff --git a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> b/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> deleted file mode 100644
>> index 1b32f90..0000000
>> --- a/recipes-multimedia/libfslcodec/libfslcodec_4.0.8.bb
>> +++ /dev/null
>> @@ -1,9 +0,0 @@
>> -# Copyright (C) 2013-2015 Freescale Semiconductor -# Released under
>> the MIT license (see COPYING.MIT for the terms)
>> -
>> -require libfslcodec.inc
>> -
>> -SRC_URI[md5sum] = "462fb27b20cd7df56a49b9e7d6b7c8b4"
>> -SRC_URI[sha256sum] = "544ffc5989bce18ca50c7a826cc03370b1cf8455335e4291ef0c0779b4c33de0"
>> -
>> -COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale
^ permalink raw reply [flat|nested] 15+ messages in thread
* [meta-fsl-arm][PATCH 2/5] libfslparser: Upgrade to v4.0.9
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9 Yuqing Zhu
@ 2016-03-31 9:37 ` Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 3/5] libfslvpuwrap: Upgrade to v1.0.65 Yuqing Zhu
` (3 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-03-31 9:37 UTC (permalink / raw)
To: meta-freescale; +Cc: Yuqing Zhu
--Change the recipe name from "libfslparser" to "imx-parser"
--Add aac latm parser in parser_libs, support aac latm in mpeg2 parser
Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
---
.../libfslparser.inc => imx-parser/imx-parser.inc} | 2 +-
recipes-multimedia/imx-parser/imx-parser_4.0.9.bb | 10 ++++++++++
recipes-multimedia/libfslparser/libfslparser_4.0.8.bb | 9 ---------
3 files changed, 11 insertions(+), 10 deletions(-)
rename recipes-multimedia/{libfslparser/libfslparser.inc => imx-parser/imx-parser.inc} (96%)
create mode 100644 recipes-multimedia/imx-parser/imx-parser_4.0.9.bb
delete mode 100644 recipes-multimedia/libfslparser/libfslparser_4.0.8.bb
diff --git a/recipes-multimedia/libfslparser/libfslparser.inc b/recipes-multimedia/imx-parser/imx-parser.inc
similarity index 96%
rename from recipes-multimedia/libfslparser/libfslparser.inc
rename to recipes-multimedia/imx-parser/imx-parser.inc
index b63aad8..1bc443a 100644
--- a/recipes-multimedia/libfslparser/libfslparser.inc
+++ b/recipes-multimedia/imx-parser/imx-parser.inc
@@ -1,5 +1,5 @@
# Copyright (C) 2012-2014 O.S. Systems Software LTDA.
-# Copyright (C) 2012-2013 Freescale Semiconductor
+# Copyright (C) 2012-2016 Freescale Semiconductor
# Released under the MIT license (see COPYING.MIT for the terms)
DESCRIPTION = "Freescale Multimedia parser libs"
LICENSE = "Proprietary"
diff --git a/recipes-multimedia/imx-parser/imx-parser_4.0.9.bb b/recipes-multimedia/imx-parser/imx-parser_4.0.9.bb
new file mode 100644
index 0000000..f7880f7
--- /dev/null
+++ b/recipes-multimedia/imx-parser/imx-parser_4.0.9.bb
@@ -0,0 +1,10 @@
+# Copyright (C) 2013-2016 Freescale Semiconductor
+# Released under the MIT license (see COPYING.MIT for the terms)
+
+require imx-parser.inc
+LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
+
+SRC_URI[md5sum] = "184687baede96fe3919196c3f91d907c"
+SRC_URI[sha256sum] = "1bec0101843db2e95d61e3dbe9e35172d874e54be19cc5d8bf72a735d5e3236f"
+
+COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
diff --git a/recipes-multimedia/libfslparser/libfslparser_4.0.8.bb b/recipes-multimedia/libfslparser/libfslparser_4.0.8.bb
deleted file mode 100644
index 06657fd..0000000
--- a/recipes-multimedia/libfslparser/libfslparser_4.0.8.bb
+++ /dev/null
@@ -1,9 +0,0 @@
-# Copyright (C) 2013-2015 Freescale Semiconductor
-# Released under the MIT license (see COPYING.MIT for the terms)
-
-require libfslparser.inc
-
-SRC_URI[md5sum] = "9538df4dc416e12ff13fdf12ddbf3684"
-SRC_URI[sha256sum] = "a04621783c84a9776216caff9563c7f840fddd584b0b2d27738d6ca6d2c77f32"
-
-COMPATIBLE_MACHINE = "(mx6|mx6ul|mx7)"
--
1.9.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [meta-fsl-arm][PATCH 3/5] libfslvpuwrap: Upgrade to v1.0.65
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 1/5] libfslcodec: Upgrade to v4.0.9 Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 2/5] libfslparser: " Yuqing Zhu
@ 2016-03-31 9:37 ` Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 4/5] fsl-alsa-plugins: Upgrade to v1.0.26 Yuqing Zhu
` (2 subsequent siblings)
5 siblings, 0 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-03-31 9:37 UTC (permalink / raw)
To: meta-freescale; +Cc: Yuqing Zhu
--Change the recipe name from "libfslvpuwrap" to "imx-vpuwrap".
--Update 'nLastFrameEndPosPhy' in seqinit stage to avoid feed
the same SPS to vpu repeatedly.
--Fix vpu VC1 decoder hang when decoding some wrong data.
Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
---
.../libfslvpuwrap_1.0.62.bb => imx-vpuwrap/imx-vpuwrap_1.0.65.bb} | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
rename recipes-multimedia/{libfslvpuwrap/libfslvpuwrap_1.0.62.bb => imx-vpuwrap/imx-vpuwrap_1.0.65.bb} (63%)
diff --git a/recipes-multimedia/libfslvpuwrap/libfslvpuwrap_1.0.62.bb b/recipes-multimedia/imx-vpuwrap/imx-vpuwrap_1.0.65.bb
similarity index 63%
rename from recipes-multimedia/libfslvpuwrap/libfslvpuwrap_1.0.62.bb
rename to recipes-multimedia/imx-vpuwrap/imx-vpuwrap_1.0.65.bb
index 363184b..435fc6b 100644
--- a/recipes-multimedia/libfslvpuwrap/libfslvpuwrap_1.0.62.bb
+++ b/recipes-multimedia/imx-vpuwrap/imx-vpuwrap_1.0.65.bb
@@ -1,16 +1,16 @@
-# Copyright (C) 2013-2015 Freescale Semiconductor
+# Copyright (C) 2013-2016 Freescale Semiconductor
# Released under the MIT license (see COPYING.MIT for the terms)
DESCRIPTION = "Freescale Multimedia VPU wrapper"
DEPENDS = "imx-vpu"
LICENSE = "Proprietary"
SECTION = "multimedia"
-LIC_FILES_CHKSUM = "file://COPYING;md5=3880bb9c943b135a30fad5e8aabd3ee9"
+LIC_FILES_CHKSUM = "file://COPYING;md5=8cf95184c220e247b9917e7244124c5a"
SRC_URI = "${FSL_MIRROR}/${PN}-${PV}.bin;fsl-eula=true"
S = "${WORKDIR}/${PN}-${PV}"
-SRC_URI[md5sum] = "6497758e27f72992a86ac918ce12db56"
-SRC_URI[sha256sum] = "0717faccb5413dc95ce9ad919400095ecf722320478c924366793a6cc450ae16"
+SRC_URI[md5sum] = "0bbcd6b9c9906fb85f98160eba94827b"
+SRC_URI[sha256sum] = "ddeacb7d9cb26c8864374df6d3f090aad47cd2ae409c2b9c817b773447252d61"
inherit fsl-eula-unpack autotools pkgconfig
--
1.9.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [meta-fsl-arm][PATCH 4/5] fsl-alsa-plugins: Upgrade to v1.0.26
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
` (2 preceding siblings ...)
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 3/5] libfslvpuwrap: Upgrade to v1.0.65 Yuqing Zhu
@ 2016-03-31 9:37 ` Yuqing Zhu
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 5/5] gst1.0-fsl-plugin: Upgrade to v4.0.9 Yuqing Zhu
2016-03-31 15:06 ` [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Otavio Salvador
5 siblings, 0 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-03-31 9:37 UTC (permalink / raw)
To: meta-freescale; +Cc: Yuqing Zhu
--Change the recipe name from "fsl-alsa-plugins" to "imx-alsa-plugins"
--Remove 0001-asrc_pair-update-output-buffer-size.patch as it has been
integrated into the package.
Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
---
.../0001-asrc_pair-update-output-buffer-size.patch | 47 ----------------------
...lugins_1.0.25.bb => imx-alsa-plugins_1.0.26.bb} | 9 ++---
2 files changed, 3 insertions(+), 53 deletions(-)
delete mode 100644 recipes-multimedia/alsa/fsl-alsa-plugins/0001-asrc_pair-update-output-buffer-size.patch
rename recipes-multimedia/alsa/{fsl-alsa-plugins_1.0.25.bb => imx-alsa-plugins_1.0.26.bb} (68%)
diff --git a/recipes-multimedia/alsa/fsl-alsa-plugins/0001-asrc_pair-update-output-buffer-size.patch b/recipes-multimedia/alsa/fsl-alsa-plugins/0001-asrc_pair-update-output-buffer-size.patch
deleted file mode 100644
index d76c74d..0000000
--- a/recipes-multimedia/alsa/fsl-alsa-plugins/0001-asrc_pair-update-output-buffer-size.patch
+++ /dev/null
@@ -1,47 +0,0 @@
-From 9acab46fe307ec71d4a4dbe447e356f90b6a4a09 Mon Sep 17 00:00:00 2001
-From: Shengjiu Wang <shengjiu.wang@freescale.com>
-Date: Fri, 12 Dec 2014 14:58:06 +0800
-Subject: [PATCH] [asrc_pair] update output buffer size
-
-When input size larger than DMA_MAX_BYTES the output size should be updated.
-Otherwise the asrc have will not have enough data, then it will be timeout.
-
-Upstream Status: Inappropriate [platform specific]
-
-Signed-off-by: Shengjiu Wang <shengjiu.wang@freescale.com>
----
- asrc/asrc_pair.c | 11 ++++++++---
- 1 file changed, 8 insertions(+), 3 deletions(-)
-
-diff --git a/asrc/asrc_pair.c b/asrc/asrc_pair.c
-index e3df027..e74e8b1 100644
---- a/asrc/asrc_pair.c
-+++ b/asrc/asrc_pair.c
-@@ -139,7 +139,7 @@ asrc_pair *asrc_pair_create(unsigned int channels, ssize_t in_period_frames,
- config.dma_buffer_size = dma_buffer_size;
- config.input_sample_rate = in_rate;
- config.output_sample_rate = out_rate;
-- config.buffer_num = 1;
-+ config.buffer_num = buf_num;
- config.input_word_width = ASRC_WIDTH_16_BIT;
- config.output_word_width = ASRC_WIDTH_16_BIT;
- config.inclk = INCLK_NONE;
-@@ -303,8 +303,13 @@ void asrc_pair_convert_s16(asrc_pair *pair, const int16_t *src, unsigned int src
-
- while (src_left > 0)
- {
-- in_len = src_left > pair->buf_size ? pair->buf_size : src_left;
-- out_len = dst_left;
-+ if (src_left > pair->buf_size) {
-+ in_len = pair->buf_size;
-+ out_len = dst_left * in_len/src_left;
-+ } else {
-+ in_len = src_left;
-+ out_len = dst_left;
-+ }
-
- buf_info.input_buffer_vaddr = s;
- buf_info.input_buffer_length = in_len;
---
-1.9.1
-
diff --git a/recipes-multimedia/alsa/fsl-alsa-plugins_1.0.25.bb b/recipes-multimedia/alsa/imx-alsa-plugins_1.0.26.bb
similarity index 68%
rename from recipes-multimedia/alsa/fsl-alsa-plugins_1.0.25.bb
rename to recipes-multimedia/alsa/imx-alsa-plugins_1.0.26.bb
index 2959c7a..a59ad0b 100644
--- a/recipes-multimedia/alsa/fsl-alsa-plugins_1.0.25.bb
+++ b/recipes-multimedia/alsa/imx-alsa-plugins_1.0.26.bb
@@ -1,4 +1,4 @@
-# Copyright (C) 2013 Freescale Semiconductor
+# Copyright (C) 2013-2016 Freescale Semiconductor
# Released under the MIT license (see COPYING.MIT for the terms)
DESCRIPTION = "Freescale alsa-lib plugins"
@@ -14,12 +14,9 @@ LIC_FILES_CHKSUM = "file://COPYING.GPL;md5=94d55d512a9ba36caa9b7df079bae19f"
inherit autotools pkgconfig
SRC_URI = "${FSL_MIRROR}/${PN}-${PV}.tar.gz"
-SRC_URI[md5sum] = "b1ca7a250a8cd5da07062081b30b4118"
-SRC_URI[sha256sum] = "902df92255d755e8eb08b3c3db0c7b9d70d26d9659b219373bee425ffdc34245"
+SRC_URI[md5sum] = "8fe4bcfddeca82dd01a9c4c6ce9471df"
+SRC_URI[sha256sum] = "0a7e8d90fdde2f6780605ecfee674e6e13523915a7bcd64078bc507ac1a8deb8"
-SRC_URI_append_mx6 = " file://0001-asrc_pair-update-output-buffer-size.patch"
-SRC_URI_append_mx6ul = " file://0001-asrc_pair-update-output-buffer-size.patch"
-SRC_URI_append_mx7 = " file://0001-asrc_pair-update-output-buffer-size.patch"
INCLUDE_DIR = "-I${STAGING_KERNEL_DIR}/include/uapi -I${STAGING_KERNEL_DIR}/include"
--
1.9.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* [meta-fsl-arm][PATCH 5/5] gst1.0-fsl-plugin: Upgrade to v4.0.9
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
` (3 preceding siblings ...)
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 4/5] fsl-alsa-plugins: Upgrade to v1.0.26 Yuqing Zhu
@ 2016-03-31 9:37 ` Yuqing Zhu
2016-03-31 15:06 ` [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Otavio Salvador
5 siblings, 0 replies; 15+ messages in thread
From: Yuqing Zhu @ 2016-03-31 9:37 UTC (permalink / raw)
To: meta-freescale; +Cc: Yuqing Zhu
--Change the recipe name from "gst1.0-fsl-plugin" to "imx-gst1.0-plugin"
--Remove dependency on gstreamer1.0-plugins-bad
There is a patch in gstreamer1.0-plugins-bad which intall the header file of
gstaggregator and gstvideoaggregator.
so we can use them outside of gstreamer-plugins-bad package.
Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
---
...-plugin_4.0.8.bb => imx-gst1.0-plugin_4.0.9.bb} | 26 +++++++++++-----------
1 file changed, 13 insertions(+), 13 deletions(-)
rename recipes-multimedia/gstreamer/{gst1.0-fsl-plugin_4.0.8.bb => imx-gst1.0-plugin_4.0.9.bb} (75%)
diff --git a/recipes-multimedia/gstreamer/gst1.0-fsl-plugin_4.0.8.bb b/recipes-multimedia/gstreamer/imx-gst1.0-plugin_4.0.9.bb
similarity index 75%
rename from recipes-multimedia/gstreamer/gst1.0-fsl-plugin_4.0.8.bb
rename to recipes-multimedia/gstreamer/imx-gst1.0-plugin_4.0.9.bb
index 8de4889..336c798 100644
--- a/recipes-multimedia/gstreamer/gst1.0-fsl-plugin_4.0.8.bb
+++ b/recipes-multimedia/gstreamer/imx-gst1.0-plugin_4.0.9.bb
@@ -1,4 +1,4 @@
-# Copyright (C) 2014,2015 Freescale Semiconductor
+# Copyright (C) 2014,2016 Freescale Semiconductor
# Copyright (C) 2012-2015 O.S. Systems Software LTDA.
# Released under the MIT license (see COPYING.MIT for the terms)
@@ -6,9 +6,9 @@ DESCRIPTION = "Gstreamer freescale plugins"
LICENSE = "GPLv2 & LGPLv2 & LGPLv2.1"
SECTION = "multimedia"
-DEPENDS = "libfslcodec libfslparser virtual/kernel gstreamer1.0 gstreamer1.0-plugins-base gstreamer1.0-plugins-bad"
-DEPENDS_append_mx6q = " imx-lib imx-vpu libfslvpuwrap"
-DEPENDS_append_mx6dl = " imx-lib imx-vpu libfslvpuwrap"
+DEPENDS = "imx-codec imx-parser virtual/kernel gstreamer1.0 gstreamer1.0-plugins-base"
+DEPENDS_append_mx6q = " imx-lib imx-vpu imx-vpuwrap"
+DEPENDS_append_mx6dl = " imx-lib imx-vpu imx-vpuwrap"
DEPENDS_append_mx6sl = " imx-lib"
DEPENDS_append_mx6sx = " imx-lib"
DEPENDS_append_mx6ul = " imx-lib"
@@ -19,13 +19,13 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
file://COPYING-LGPL-2.1;md5=fbc093901857fcd118f065f900982c24"
SRC_URI = " \
- ${FSL_MIRROR}/gst1.0-fsl-plugins-${PV}.tar.gz \
+ ${FSL_MIRROR}/imx-gst1.0-plugin-${PV}.tar.gz \
"
-SRC_URI[md5sum] = "0669eeea4e37203e2e654a00ded35ba2"
-SRC_URI[sha256sum] = "43c86f88d3c7b1da8d38815eaec297a9b8652e1fb770ff89c26d617f63ea068b"
+SRC_URI[md5sum] = "56fc7d4fcbad0ba14739b668793fcf2c"
+SRC_URI[sha256sum] = "4ab74e3660a56c5a79c714b820bb1ce8dd1db5d66c92529b5378bb211cf25896"
-S = "${WORKDIR}/gst1.0-fsl-plugins-${PV}"
+S = "${WORKDIR}/imx-gst1.0-plugin-${PV}"
inherit autotools pkgconfig
@@ -47,8 +47,8 @@ EXTRA_OECONF = "PLATFORM=${PLATFORM} \
PACKAGES =+ "${PN}-gplay ${PN}-libgplaycore ${PN}-libgstfsl ${PN}-grecorder ${PN}-librecorder-engine ${PN}-libplayengine"
# Add codec list that the beep plugin run-time depended
-BEEP_RDEPENDS = "libfslcodec-aac libfslcodec-mp3 libfslcodec-oggvorbis"
-RDEPENDS_${PN} += "libfslparser ${BEEP_RDEPENDS} gstreamer1.0-plugins-good-id3demux "
+BEEP_RDEPENDS = "imx-codec-aac imx-codec-mp3 imx-codec-oggvorbis"
+RDEPENDS_${PN} += "imx-parser ${BEEP_RDEPENDS} gstreamer1.0-plugins-good-id3demux "
PACKAGECONFIG ?= ""
PACKAGECONFIG_mx6 = "overlaysink"
@@ -56,10 +56,10 @@ PACKAGECONFIG_mx6 = "overlaysink"
# FIXME: Add all features
# feature from excluded mm packages
-PACKAGECONFIG[ac3] += ",,libfslac3codec,libfslac3codec"
+PACKAGECONFIG[ac3] += ",,imx-ac3codec,imx-ac3codec"
# feature from special mm packages
-PACKAGECONFIG[aacp] += ",,libfslaacpcodec,libfslaacpcodec"
-MSDEPENDS = "libfslmsparser libfslmscodec"
+PACKAGECONFIG[aacp] += ",,imx-aacpcodec,imx-aacpcodec"
+MSDEPENDS = "imx-msparser imx-mscodec"
PACKAGECONFIG[wma10dec] += ",,${MSDEPENDS},${MSDEPENDS}"
PACKAGECONFIG[wma8enc] += "--enable-wma8enc,--disable-wma8enc,${MSDEPENDS},${MSDEPENDS}"
OVDEPENDS = "virtual/libg2d"
--
1.9.1
^ permalink raw reply related [flat|nested] 15+ messages in thread* Re: [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release
2016-03-31 9:37 [meta-fsl-arm][PATCH 0/5] Upgrade i.MX multimedia to jethro_4.1.15-1.0.0_ga release Yuqing Zhu
` (4 preceding siblings ...)
2016-03-31 9:37 ` [meta-fsl-arm][PATCH 5/5] gst1.0-fsl-plugin: Upgrade to v4.0.9 Yuqing Zhu
@ 2016-03-31 15:06 ` Otavio Salvador
5 siblings, 0 replies; 15+ messages in thread
From: Otavio Salvador @ 2016-03-31 15:06 UTC (permalink / raw)
To: Yuqing Zhu; +Cc: meta-freescale@yoctoproject.org
On Thu, Mar 31, 2016 at 6:37 AM, Yuqing Zhu <carol.zhu@nxp.com> wrote:
> --Upgrade libfslcodec to v4.0.9
> --Upgrade libfslparser to v4.0.9
> --Upgrade gst1.0-fsl-plugin to v4.0.9
> --Upgrade libfslvpuwrap to v1.0.65
> --Upgrade fsl-alsa-plugins to v1.0.26
I reviewed the first patch but the very same comment applies for all
them. So please rework the patchset and resend.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
^ permalink raw reply [flat|nested] 15+ messages in thread