* [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates
@ 2013-09-25 19:07 Eric Nelson
2013-09-25 19:07 ` [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) Eric Nelson
` (3 more replies)
0 siblings, 4 replies; 17+ messages in thread
From: Eric Nelson @ 2013-09-25 19:07 UTC (permalink / raw)
To: buildroot
This patch series updates U-Boot and the Linux kernel for the
Boundary Devices Nitrogen6x board to the latest production-ready
releases and also updates Freescale-provided packages to the
matching versions to maintain ABI compatibility.
Eric Nelson (4):
nitrogen6x: bump U-Boot to latest production version (2013.07)
nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions
nitrogen6x: bump kernel to latest version (3.0.35-4.1.0)
i.MX6: Update Freescale hardware-acceleration bits to latest versions
board/boundarydevices/nitrogen6x/6q_bootscript | Bin 455 -> 0 bytes
board/boundarydevices/nitrogen6x/6q_upgrade | Bin 1587 -> 0 bytes
board/boundarydevices/nitrogen6x/6x_bootscript | Bin 0 -> 1653 bytes
board/boundarydevices/nitrogen6x/6x_upgrade | Bin 0 -> 1744 bytes
board/boundarydevices/nitrogen6x/post-build.sh | 14 +++++++------
configs/nitrogen6x_defconfig | 4 ++--
package/freescale-imx/freescale-imx.mk | 4 ++--
.../gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk | 8 +++++---
package/freescale-imx/imx-lib/imx-lib.mk | 11 ++++++++++-
.../gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk | 4 ++--
package/libfslcodec/libfslcodec.mk | 4 ++--
package/libfslparser/libfslparser.mk | 8 +++++---
package/libfslvpuwrap/libfslvpuwrap.mk | 22 +++++++++++++++++++--
13 files changed, 56 insertions(+), 23 deletions(-)
delete mode 100644 board/boundarydevices/nitrogen6x/6q_bootscript
delete mode 100644 board/boundarydevices/nitrogen6x/6q_upgrade
create mode 100644 board/boundarydevices/nitrogen6x/6x_bootscript
create mode 100644 board/boundarydevices/nitrogen6x/6x_upgrade
--
1.8.1.2
^ permalink raw reply [flat|nested] 17+ messages in thread* [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) 2013-09-25 19:07 [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates Eric Nelson @ 2013-09-25 19:07 ` Eric Nelson 2013-09-26 19:19 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions Eric Nelson ` (2 subsequent siblings) 3 siblings, 1 reply; 17+ messages in thread From: Eric Nelson @ 2013-09-25 19:07 UTC (permalink / raw) To: buildroot Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> --- configs/nitrogen6x_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/nitrogen6x_defconfig b/configs/nitrogen6x_defconfig index cdd8000..4c4bd1b 100644 --- a/configs/nitrogen6x_defconfig +++ b/configs/nitrogen6x_defconfig @@ -15,7 +15,7 @@ BR2_TARGET_ROOTFS_EXT2_2r1=y BR2_TARGET_UBOOT=y BR2_TARGET_UBOOT_BOARDNAME="nitrogen6q" BR2_TARGET_UBOOT_CUSTOM_TARBALL=y -BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="http://github.com/boundarydevices/u-boot-imx6/tarball/7176d07/u-boot-7176d07.tar.gz" +BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="http://github.com/boundarydevices/u-boot-imx6/tarball/4034391/u-boot-4034391.tar.gz" # kernel BR2_LINUX_KERNEL=y -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) 2013-09-25 19:07 ` [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) Eric Nelson @ 2013-09-26 19:19 ` Peter Korsgaard 0 siblings, 0 replies; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 19:19 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Eric> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> Committed, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-25 19:07 [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates Eric Nelson 2013-09-25 19:07 ` [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) Eric Nelson @ 2013-09-25 19:07 ` Eric Nelson 2013-09-26 14:37 ` Peter Korsgaard 2013-09-26 19:20 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) Eric Nelson 2013-09-25 19:07 ` [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions Eric Nelson 3 siblings, 2 replies; 17+ messages in thread From: Eric Nelson @ 2013-09-25 19:07 UTC (permalink / raw) To: buildroot Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> --- board/boundarydevices/nitrogen6x/6q_bootscript | Bin 455 -> 0 bytes board/boundarydevices/nitrogen6x/6q_upgrade | Bin 1587 -> 0 bytes board/boundarydevices/nitrogen6x/6x_bootscript | Bin 0 -> 1653 bytes board/boundarydevices/nitrogen6x/6x_upgrade | Bin 0 -> 1744 bytes board/boundarydevices/nitrogen6x/post-build.sh | 14 ++++++++------ 5 files changed, 8 insertions(+), 6 deletions(-) delete mode 100644 board/boundarydevices/nitrogen6x/6q_bootscript delete mode 100644 board/boundarydevices/nitrogen6x/6q_upgrade create mode 100644 board/boundarydevices/nitrogen6x/6x_bootscript create mode 100644 board/boundarydevices/nitrogen6x/6x_upgrade diff --git a/board/boundarydevices/nitrogen6x/6q_bootscript b/board/boundarydevices/nitrogen6x/6q_bootscript deleted file mode 100644 index e10cfeb0b742f401e36f0f242ca72e45fab26944..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 455 zcma)&!AiqG5QbB+9`Xj>h7#%}CA*=<(z;+3DS`p<P`xI*n|8@=wrpaol;Y8Y58>5^ z(RVTlQ9(qU%MAa_d_OY>-TgVceOq1)j$bXS^JKO0{J#6r-PyBZtqaJh5v$^}^&@v4 zYa!<a)U%OU<luZ&RA;%CoD{`_T2t&|eCQ(trHZ7~lnfCH!BlI+W4!c~qOnZT3OCw) zBnoH&v5c~mAeh*d5zorsyh_2iCY4PRqj7~JOQwX2{SjIp`6xUIkSh{$Jv;S>|D)iB zA}MpvE~nGM&3ra(2|*D2fk<M-74b)*i&1Dhn<U>#9WkiR182iTo^Hn&FdCscz^lLI n-Cd|?!J*%8PE+6Q32?g9&^tFq8>m4dTR_T9#wCb)sb||C*>#HV diff --git a/board/boundarydevices/nitrogen6x/6q_upgrade b/board/boundarydevices/nitrogen6x/6q_upgrade deleted file mode 100644 index 2f20efa2e85c761b09da7ae5a935589280dae5eb..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 1587 zcmb_c!H&}~5Ur33YQ=>kx6x9GQ>!#AEU-634;)ykT><CBo-|gIII`V#w`dRi1s}ti zpW(<4Ft(F6?IwFb_>jiwy!XcQ#`C=Y^vlo3Pqy>^k4L>;|4*-4-`Bss_8&a##Zns3 zOo`Iee;xhyf7=hpQVAz4MYe>`<9CEQf%BnAVDv4~x0#GM<T;}q7kYX7b_&xla0^yr zt1*d10q0;+EQa8gXH-bYQ6e(bVv7*p7}Ss`nZwn4V3Fch#sf%9Cr~OGBTOMYtN1I@ z0R?$V7s26b{sAsOeca<6izjia!Syse^*IKO6b_Q~%Jd{0%gad&7F8)X2=*svhg%+> z-Po_Hh_FVbg)B7WkzwgkvJRDLtO{VayXDy~5P40tDi<o^*ur5!K7Otw1?50yOasu! zWWjX@B9RIT8~Fxa!x_ASmv9O%Af)04WF?nvX}SEI*2wzZ8KN`9l9+09cjvM%ToyPS z>>WiFX;CaX@}#D)0Z3T|*UnI4b?9ecHWIYKQiGAyI=IHdK&%8?5?@n-n#L9;zgK3X zgdv$m2Q>SpNk_Jd<ffbKU)!)Q(|>5knrf5M<!dzOQ9JPoDAZ`LyS02ktMV=k?5?uQ zzjfJ-9Zg*4;G6T(&H`6C!6HMqf_zm{b=YpPiD*;2SItY!EHNEyY$pNP1hd)v?y#uS zyDM|}rA_cq$BVdOYiA`YKjT_ecihLcO-fM+Bce=f+mcTN-GDvX<1S?FtjMDULLyfM N2P$%7(313U_!p)>ak>Bi diff --git a/board/boundarydevices/nitrogen6x/6x_bootscript b/board/boundarydevices/nitrogen6x/6x_bootscript new file mode 100644 index 0000000000000000000000000000000000000000..dc9eea481c03145d16959e5ebcc47120ff3cc2c3 GIT binary patch literal 1653 zcmdT^!EVz)5RED|Qv3sm+sLwt1dg(H(-1YXMA|~B)LeQf+^V%bPF7j(8hc3{Mfnas zf+HWq5AYGJ9Xn}4p+dR9Ue at -^zMc0x^IjZ2zy0##+xX`E<@ZLTdD5uXv#<8S;X$+E zOQ|3WQkJMXw6izc#fJ<PiWwB?G#zJvg8Cfq?kQ7uu?*3ZQDjs}0&^8e0c2npEP_cG zvk(;wKuu5>tz9T<4LZ|#(u?yT@~uUMvOEoO<`o#W<Mv7Z+9jhmJz}%SySY4byKB72 zk1+<b+&s5GG6$<ro}^G$WYrI^z%h_#XtXexNHE*w0wLl3#pnuRDVUO}hKQKv;0B<J z0JmUMyd<gg5lBwDdy(RyZ(m=WwLe}CD>}Vi&uHmO6lMxcVnQ1~L0xpMUoGGkVTY82 zSDdyptVNl*v~`Zr(uB(JU&`_>Ma%-q5ki(FoX%@)wiP+<@2!YK+-}|@5C8JND@$GM zpH=DZsY(#q=hwqw`@`*j&?$XF@@}8(C~`kxx_4H6eXRRh;JUAM$Aoe{#%nWp$`-WY zz!M{`<DnF2)W-A(wpGr?FA3)6cI{-D<mf3i*UseFj&tG?0<)NME(5QtI~A%NpHngQ z2&CF23uiP-|67sHsG=r#U|Ne%%@bUjznzo^mlsH0N1IP43#p at zRIXKdsQCz^&5<|` zd_E;fc}~y0HiB%)B@Lmjp?F}~bb0E4L-dc+3pxd6=krxe#|TG9rRB$)u&$tV%(rPO TQz+1w7{e4(fgD(I$TWTea_I@( literal 0 HcmV?d00001 diff --git a/board/boundarydevices/nitrogen6x/6x_upgrade b/board/boundarydevices/nitrogen6x/6x_upgrade new file mode 100644 index 0000000000000000000000000000000000000000..cd26b616b0024424764e95f4d54d917c626d36d4 GIT binary patch literal 1744 zcmb_dO^?$s5Ur55Qu7bEjh0F)AzG!SD<paYD#2lwsw+4kE@?bztu}UKyI;%pz>OmZ zeh>eEpM=CX&R4tXN(g*toOI^BmwDsK#o6PJgReh-OkO|z-DotwH?no{>+PqrN6p4m zN&`AnJTduSN7v>z`#_^Xu?B6JTu}ll9xR`&XMu;|T*7%c$8Z53I<E-s9EM&NE~AKc z;BN=M@9_xQUn0F<%7D?1nO%Z=yWx7Vf7ydx-%l5;I#yMe$6L6&`@0+=lYlWUW-tUZ zN8v%boJfV1IO0px{1#!lGpHd_GKQNs5C)1_<t~A=>If1gr)V`)_n*C1dEbNni%fMj ztnxV1(IQ- at A_Zk|Gkyn?_g5ulO6+)3q5ihlr?Sz4c2>ty`@!2e$@ELvVVrcQRPdsh zRhZ({>(ecc)7Wt<GZfaSG?zj{9GGx^BstAmYRoEUZ;R#7t`S)dwMu3xU|54;lU)2* zNeYUA3%I0K3#DNCL_#DL2p2ej=g@~98P8;$mQ++q&=!Tyh0<c9mxzglsl#AWpw}wB zL=|XiwiDv0Yit05PljBkP7Yo66EGV|hY1r6MpB>Q8ij$W;^Bhusua{Nwixk)G#kYY z(Uh*&bW9bL4CT at 8G~~avVP4Ds(2hCOW=^NJ(ws-_#JZr6CJh;3-EYy#eC+%7j$l&j za=4=_o at n1Z%(dmjj*ql*JPZt-2Xa?Qb;5j`8}Yg(dXUXgjVv&|QpZjLvI$0`@%>?u z%ezl=_ at s{TkjIgv-vT?GQMs{L%UWO`!&V`=;D){s{I*Vf#OZXE*)H!|=AQC6m?1=R NB^XeVE3=uX{{YL7tK|Ry literal 0 HcmV?d00001 diff --git a/board/boundarydevices/nitrogen6x/post-build.sh b/board/boundarydevices/nitrogen6x/post-build.sh index 2f3c464..1dc5643 100755 --- a/board/boundarydevices/nitrogen6x/post-build.sh +++ b/board/boundarydevices/nitrogen6x/post-build.sh @@ -1,18 +1,20 @@ #!/bin/sh # post-build fixups -# for furthe details, see -# http://boundarydevices.com/u-boot-conventions-for-i-mx6-nitrogen6x-and-sabrelite/ +# for further details, see +# +# http://boundarydevices.com/u-boot-on-i-mx6/ +# TARGET_DIR=$1 IMAGES_DIR=$1/../images BOARD_DIR="$(dirname $0)" # bd u-boot looks for bootscript here -cp $BOARD_DIR/6q_bootscript $TARGET_DIR +cp $BOARD_DIR/6x_bootscript $TARGET_DIR # u-boot / update script for bd upgradeu command -if [ -e $IMAGES_DIR/u-boot.bin ]; +if [ -e $IMAGES_DIR/u-boot.imx ]; then - cp $IMAGES_DIR/u-boot.bin $TARGET_DIR - cp $BOARD_DIR/6q_upgrade $TARGET_DIR + cp $IMAGES_DIR/u-boot.imx $TARGET_DIR + cp $BOARD_DIR/6x_upgrade $TARGET_DIR fi -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-25 19:07 ` [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions Eric Nelson @ 2013-09-26 14:37 ` Peter Korsgaard 2013-09-26 15:13 ` Eric Nelson 2013-09-26 19:20 ` Peter Korsgaard 1 sibling, 1 reply; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 14:37 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Hi Eric, Eric> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> Eric> +++ b/board/boundarydevices/nitrogen6x/post-build.sh Eric> @@ -1,18 +1,20 @@ Eric> #!/bin/sh Eric> # post-build fixups Eric> -# for furthe details, see Eric> -# http://boundarydevices.com/u-boot-conventions-for-i-mx6-nitrogen6x-and-sabrelite/ Eric> +# for further details, see Eric> +# Eric> +# http://boundarydevices.com/u-boot-on-i-mx6/ Eric> +# Eric> TARGET_DIR=$1 Eric> IMAGES_DIR=$1/../images Eric> BOARD_DIR="$(dirname $0)" Eric> # bd u-boot looks for bootscript here Eric> -cp $BOARD_DIR/6q_bootscript $TARGET_DIR Eric> +cp $BOARD_DIR/6x_bootscript $TARGET_DIR Eric> # u-boot / update script for bd upgradeu command Eric> -if [ -e $IMAGES_DIR/u-boot.bin ]; Eric> +if [ -e $IMAGES_DIR/u-boot.imx ]; Eric> then Eric> - cp $IMAGES_DIR/u-boot.bin $TARGET_DIR Eric> - cp $BOARD_DIR/6q_upgrade $TARGET_DIR Eric> + cp $IMAGES_DIR/u-boot.imx $TARGET_DIR Eric> + cp $BOARD_DIR/6x_upgrade $TARGET_DIR Hmm, how is that supposed to work? I don't know the details of the imx6 boot sequence, but: - We don't currently have an option in the u-boot package to install u-boot.imx to IMAGES_DIR/, so this will never run - Comparing 6q_upgrade and 6x_upgrade I see we used to write the raw u-boot.bin to offset 0 in the spi flash, and now are writing u-boot.imx (which presumably is u-boot.bin with a freescale header) to offset 1K. I see that the .imx file is 4K bigger than the .bin file, so that's presumably the size of the freescale header. I'll go and add a option for u-boot.imx to the u-boot package, but would like to hear if 6x_upgrade is doing the right thing here. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-26 14:37 ` Peter Korsgaard @ 2013-09-26 15:13 ` Eric Nelson 2013-09-26 19:10 ` Peter Korsgaard 0 siblings, 1 reply; 17+ messages in thread From: Eric Nelson @ 2013-09-26 15:13 UTC (permalink / raw) To: buildroot Hi Peter, On 09/26/2013 07:37 AM, Peter Korsgaard wrote: >>>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: > > Hi Eric, > > Eric> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> > > Eric> +++ b/board/boundarydevices/nitrogen6x/post-build.sh > Eric> @@ -1,18 +1,20 @@ > Eric> #!/bin/sh > Eric> # post-build fixups > Eric> -# for furthe details, see > Eric> -# http://boundarydevices.com/u-boot-conventions-for-i-mx6-nitrogen6x-and-sabrelite/ > Eric> +# for further details, see > Eric> +# > Eric> +# http://boundarydevices.com/u-boot-on-i-mx6/ > Eric> +# > > Eric> TARGET_DIR=$1 > Eric> IMAGES_DIR=$1/../images > Eric> BOARD_DIR="$(dirname $0)" > > Eric> # bd u-boot looks for bootscript here > Eric> -cp $BOARD_DIR/6q_bootscript $TARGET_DIR > Eric> +cp $BOARD_DIR/6x_bootscript $TARGET_DIR > > Eric> # u-boot / update script for bd upgradeu command > Eric> -if [ -e $IMAGES_DIR/u-boot.bin ]; > Eric> +if [ -e $IMAGES_DIR/u-boot.imx ]; > Eric> then > Eric> - cp $IMAGES_DIR/u-boot.bin $TARGET_DIR > Eric> - cp $BOARD_DIR/6q_upgrade $TARGET_DIR > Eric> + cp $IMAGES_DIR/u-boot.imx $TARGET_DIR > Eric> + cp $BOARD_DIR/6x_upgrade $TARGET_DIR > > Hmm, how is that supposed to work? I don't know the details of the imx6 > boot sequence, but: > > - We don't currently have an option in the u-boot package to install > u-boot.imx to IMAGES_DIR/, so this will never run > > - Comparing 6q_upgrade and 6x_upgrade I see we used to write the raw > u-boot.bin to offset 0 in the spi flash, and now are writing > u-boot.imx (which presumably is u-boot.bin with a freescale header) to > offset 1K. I see that the .imx file is 4K bigger than the .bin > file, so that's presumably the size of the freescale header. > The switch from "u-boot.bin" to "u-boot.imx" occurred when we switched from using a Freescale-derived U-Boot 2009.08 to main-line U-Boot sources (currently 2013.07). Details in this post: http://boundarydevices.com/switching-u-boot-versions-on-i-mx6/ > I'll go and add a option for u-boot.imx to the u-boot package, but would > like to hear if 6x_upgrade is doing the right thing here. > A short description of the change is that Freescale's build process generated u-boot.bin with 512 bytes of padding at the beginning, while main-line U-Boot generates u-boot.imx without the padding. There are also a handful of discrepancies in the syntax for various commands between the two versions, especially in the areas of SPI NOR (sf probe command), which prompted us to switch the name of the upgrade script (6q_upgrade == old/6x_upgrade == new) when switching to main-line based code. Note that unlike many ARM-based boards out there, our devices boot to SPI-NOR, so building U-Boot is a convenience. It's not needed to build an SD card image and the choice of when to upgrade should largely be based on the need for features or bug fixes and not tied to each userspace build. The 6x_bootscript is required though, and provides the distribution- specifics of how/where to load kernel, DTS file, ram disk, et cetera. Regards, Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-26 15:13 ` Eric Nelson @ 2013-09-26 19:10 ` Peter Korsgaard 2013-09-26 20:20 ` Eric Nelson 0 siblings, 1 reply; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 19:10 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Hi Eric, >> - Comparing 6q_upgrade and 6x_upgrade I see we used to write the raw >> u-boot.bin to offset 0 in the spi flash, and now are writing >> u-boot.imx (which presumably is u-boot.bin with a freescale header) to >> offset 1K. I see that the .imx file is 4K bigger than the .bin >> file, so that's presumably the size of the freescale header. >> Eric> The switch from "u-boot.bin" to "u-boot.imx" occurred when we Eric> switched from using a Freescale-derived U-Boot 2009.08 to Eric> main-line U-Boot sources (currently 2013.07). Ahh, I guess the confusion comes from the fact that I missed the change to u-boot.imx when I updated the u-boot version back in April: http://git.buildroot.net/buildroot/commit/configs/nitrogen6x_defconfig?id=184850a4239c967742 Eric> Details in this post: Eric> http://boundarydevices.com/switching-u-boot-versions-on-i-mx6/ >> I'll go and add a option for u-boot.imx to the u-boot package, but would >> like to hear if 6x_upgrade is doing the right thing here. >> Eric> A short description of the change is that Freescale's build process Eric> generated u-boot.bin with 512 bytes of padding at the beginning, Eric> while main-line U-Boot generates u-boot.imx without the padding. I guess you mean 1K of padding to match the new script? hexdump -C u-boot.bin|head /tmp/boundarydevices-u-boot-2009-08-1f7edab 00000000 c6 01 00 ea 00 00 00 00 00 00 00 00 00 00 00 00 |................| 00000010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| * 00000400 d1 00 20 40 20 07 80 27 00 00 00 00 2c 04 80 27 |.. @ ..'....,..'| 00000410 20 04 80 27 00 04 80 27 00 00 00 00 00 00 00 00 | ..'...'........| 00000420 00 00 80 27 74 30 03 00 00 00 00 00 d2 02 f0 40 |...'t0.........@| 00000430 cc 02 ec 04 02 0e 05 a8 00 00 00 30 02 0e 05 b0 |...........0....| 00000440 00 00 00 30 02 0e 05 24 00 00 00 30 02 0e 05 1c |...0...$...0....| 00000450 00 00 00 30 02 0e 05 18 00 00 00 30 02 0e 05 0c |...0.......0....| 00000460 00 00 00 30 02 0e 05 b8 00 00 00 30 02 0e 05 c0 |...0.......0....| Eric> There are also a handful of discrepancies in the syntax for Eric> various commands between the two versions, especially in the Eric> areas of SPI NOR (sf probe command), which prompted us to switch Eric> the name of the upgrade script (6q_upgrade == old/6x_upgrade == new) Eric> when switching to main-line based code. Eric> Note that unlike many ARM-based boards out there, our devices boot Eric> to SPI-NOR, so building U-Boot is a convenience. It's not needed Eric> to build an SD card image and the choice of when to upgrade should Eric> largely be based on the need for features or bug fixes and not Eric> tied to each userspace build. Yes, I know. My nitrogen6x board still runs the original 2009.08 bootloader it was shipped with (which has problems using ext2 rev0 filesystems among others), so I should probably upgrade ;) Thanks for the info. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-26 19:10 ` Peter Korsgaard @ 2013-09-26 20:20 ` Eric Nelson 0 siblings, 0 replies; 17+ messages in thread From: Eric Nelson @ 2013-09-26 20:20 UTC (permalink / raw) To: buildroot Hi Peter, On 09/26/2013 12:10 PM, Peter Korsgaard wrote: >>>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: > > Hi Eric, > > >> - Comparing 6q_upgrade and 6x_upgrade I see we used to write the raw > >> u-boot.bin to offset 0 in the spi flash, and now are writing > >> u-boot.imx (which presumably is u-boot.bin with a freescale header) to > >> offset 1K. I see that the .imx file is 4K bigger than the .bin > >> file, so that's presumably the size of the freescale header. > >> > Eric> The switch from "u-boot.bin" to "u-boot.imx" occurred when we > Eric> switched from using a Freescale-derived U-Boot 2009.08 to > Eric> main-line U-Boot sources (currently 2013.07). > > Ahh, I guess the confusion comes from the fact that I missed the change > to u-boot.imx when I updated the u-boot version back in April: > > http://git.buildroot.net/buildroot/commit/configs/nitrogen6x_defconfig?id=184850a4239c967742 > I'll try to post updates to the ML with each quarterly release. The next upgrade will be 2013-10, that I hope enables USB OTG support. > > Eric> Details in this post: > Eric> http://boundarydevices.com/switching-u-boot-versions-on-i-mx6/ > > >> I'll go and add a option for u-boot.imx to the u-boot package, but would > >> like to hear if 6x_upgrade is doing the right thing here. > >> > Eric> A short description of the change is that Freescale's build process > Eric> generated u-boot.bin with 512 bytes of padding at the beginning, > Eric> while main-line U-Boot generates u-boot.imx without the padding. > > I guess you mean 1K of padding to match the new script? > > hexdump -C u-boot.bin|head /tmp/boundarydevices-u-boot-2009-08-1f7edab > 00000000 c6 01 00 ea 00 00 00 00 00 00 00 00 00 00 00 00 |................| > 00000010 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 |................| > * > 00000400 d1 00 20 40 20 07 80 27 00 00 00 00 2c 04 80 27 |.. @ ..'....,..'| > 00000410 20 04 80 27 00 04 80 27 00 00 00 00 00 00 00 00 | ..'...'........| > 00000420 00 00 80 27 74 30 03 00 00 00 00 00 d2 02 f0 40 |...'t0.........@| > 00000430 cc 02 ec 04 02 0e 05 a8 00 00 00 30 02 0e 05 b0 |...........0....| > 00000440 00 00 00 30 02 0e 05 24 00 00 00 30 02 0e 05 1c |...0...$...0....| > 00000450 00 00 00 30 02 0e 05 18 00 00 00 30 02 0e 05 0c |...0.......0....| > 00000460 00 00 00 30 02 0e 05 b8 00 00 00 30 02 0e 05 c0 |...0.......0....| > Yep. 1k (0x400) > > Eric> There are also a handful of discrepancies in the syntax for > Eric> various commands between the two versions, especially in the > Eric> areas of SPI NOR (sf probe command), which prompted us to switch > Eric> the name of the upgrade script (6q_upgrade == old/6x_upgrade == new) > Eric> when switching to main-line based code. > > Eric> Note that unlike many ARM-based boards out there, our devices boot > Eric> to SPI-NOR, so building U-Boot is a convenience. It's not needed > Eric> to build an SD card image and the choice of when to upgrade should > Eric> largely be based on the need for features or bug fixes and not > Eric> tied to each userspace build. > > Yes, I know. My nitrogen6x board still runs the original 2009.08 > bootloader it was shipped with (which has problems using ext2 rev0 > filesystems among others), so I should probably upgrade ;) > Upgrading is definitely recommended. The other notable enhancements are display support and support for SATA. Regards, Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions 2013-09-25 19:07 ` [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions Eric Nelson 2013-09-26 14:37 ` Peter Korsgaard @ 2013-09-26 19:20 ` Peter Korsgaard 1 sibling, 0 replies; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 19:20 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Eric> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> Committed, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) 2013-09-25 19:07 [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates Eric Nelson 2013-09-25 19:07 ` [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) Eric Nelson 2013-09-25 19:07 ` [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions Eric Nelson @ 2013-09-25 19:07 ` Eric Nelson 2013-09-26 19:20 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions Eric Nelson 3 siblings, 1 reply; 17+ messages in thread From: Eric Nelson @ 2013-09-25 19:07 UTC (permalink / raw) To: buildroot Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> --- configs/nitrogen6x_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/nitrogen6x_defconfig b/configs/nitrogen6x_defconfig index 4c4bd1b..0357e6d 100644 --- a/configs/nitrogen6x_defconfig +++ b/configs/nitrogen6x_defconfig @@ -20,6 +20,6 @@ BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="http://github.com/boundarydevices/u-bo # kernel BR2_LINUX_KERNEL=y BR2_LINUX_KERNEL_CUSTOM_TARBALL=y -BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="http://github.com/boundarydevices/linux-imx6/tarball/2dc5560/linux-imx6-2dc5560.tar.gz" +BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="http://github.com/boundarydevices/linux-imx6/tarball/5809938/linux-imx6-5809938.tar.gz BR2_LINUX_KERNEL_DEFCONFIG="nitrogen6x" BR2_LINUX_KERNEL_INSTALL_TARGET=y -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) 2013-09-25 19:07 ` [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) Eric Nelson @ 2013-09-26 19:20 ` Peter Korsgaard 0 siblings, 0 replies; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 19:20 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Eric> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> Committed, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-09-25 19:07 [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates Eric Nelson ` (2 preceding siblings ...) 2013-09-25 19:07 ` [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) Eric Nelson @ 2013-09-25 19:07 ` Eric Nelson 2013-09-26 19:36 ` Peter Korsgaard 3 siblings, 1 reply; 17+ messages in thread From: Eric Nelson @ 2013-09-25 19:07 UTC (permalink / raw) To: buildroot This patchset updates the Freescale gstreamer and GPU libraries to the latest release (3.0.35_4.1.0). Note that the package names refer to and are compatible with an alpha release of kernel version 3.5.7. The primary release package (based on LTIB) is available here and includes documentation. https://community.freescale.com/docs/DOC-95560 The 3.5.7 kernel sources are available here: http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/log/?h=imx_3.5.7_1.0.0_alpha Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> --- package/freescale-imx/freescale-imx.mk | 4 ++-- .../gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk | 8 +++++--- package/freescale-imx/imx-lib/imx-lib.mk | 11 ++++++++++- .../gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk | 4 ++-- package/libfslcodec/libfslcodec.mk | 4 ++-- package/libfslparser/libfslparser.mk | 8 +++++--- package/libfslvpuwrap/libfslvpuwrap.mk | 22 ++++++++++++++++++++-- 7 files changed, 46 insertions(+), 15 deletions(-) diff --git a/package/freescale-imx/freescale-imx.mk b/package/freescale-imx/freescale-imx.mk index d35a62e..42ef11c 100644 --- a/package/freescale-imx/freescale-imx.mk +++ b/package/freescale-imx/freescale-imx.mk @@ -4,9 +4,9 @@ # ################################################################################ -FREESCALE_IMX_VERSION = 1.1.0 +FREESCALE_IMX_VERSION = 3.5.7-1.0.0 # No official download site from freescale, just this mirror -FREESCALE_IMX_MIRROR_SITE = http://download.ossystems.com.br/bsp/freescale/source +FREESCALE_IMX_MIRROR_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ include $(sort $(wildcard package/freescale-imx/*/*.mk)) diff --git a/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk b/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk index c48d108..499f617 100644 --- a/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk +++ b/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk @@ -3,8 +3,12 @@ # gpu-viv-bin-mx6q # ################################################################################ +ifeq ($(BR2_ARM_EABIHF),y) +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-hfp +else +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-sfp +endif -GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION) GPU_VIV_BIN_MX6Q_SITE = $(FREESCALE_IMX_MIRROR_SITE) GPU_VIV_BIN_MX6Q_SOURCE = gpu-viv-bin-mx6q-$(GPU_VIV_BIN_MX6Q_VERSION).bin @@ -30,8 +34,6 @@ endif # The archive is a shell-self-extractor of a bzipped tar. It happens # to extract in the correct directory (gpu-viv-bin-mx6q-x.y.z) # The --force makes sure it doesn't fail if the source dir already exists. -# The --auto-accept skips the license check - not needed for us -# because we have legal-info. define GPU_VIV_BIN_MX6Q_EXTRACT_CMDS (cd $(BUILD_DIR); \ sh $(DL_DIR)/$(GPU_VIV_BIN_MX6Q_SOURCE) --force --auto-accept) diff --git a/package/freescale-imx/imx-lib/imx-lib.mk b/package/freescale-imx/imx-lib/imx-lib.mk index bd7e174..e3c29b1 100644 --- a/package/freescale-imx/imx-lib/imx-lib.mk +++ b/package/freescale-imx/imx-lib/imx-lib.mk @@ -6,7 +6,8 @@ IMX_LIB_VERSION = $(FREESCALE_IMX_VERSION) IMX_LIB_SITE = $(FREESCALE_IMX_MIRROR_SITE) -IMX_LIB_LICENSE = LGPLv2.1+ +IMX_LIB_SOURCE = imx-lib-$(IMX_LIB_VERSION).bin +IMX_LIB_LICENSE = Freescale Semiconductor Software License Agreement # No license file included IMX_LIB_INSTALL_STAGING = YES @@ -25,6 +26,14 @@ IMX_LIB_MAKE_ENV = \ PLATFORM=$(BR2_PACKAGE_IMX_LIB_PLATFORM) \ INCLUDE="$(IMX_LIB_INCLUDE)" +# The archive is a shell-self-extractor of a bzipped tar. It happens +# to extract in the correct directory (imx-lib-x.y.z) +# The --force makes sure it doesn't fail if the source dir already exists. +define IMX_LIB_EXTRACT_CMDS + (cd $(BUILD_DIR); \ + sh $(DL_DIR)/$(IMX_LIB_SOURCE) --force --auto-accept) +endef + define IMX_LIB_BUILD_CMDS $(IMX_LIB_MAKE_ENV) $(MAKE1) -C $(@D) endef diff --git a/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk b/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk index ccdb521..c46073d 100644 --- a/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk +++ b/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk @@ -4,9 +4,9 @@ # ################################################################################ -GST_FSL_PLUGINS_VERSION = 3.0.1 +GST_FSL_PLUGINS_VERSION = 3.5.7-1.0.0 # No official download site from freescale, just this mirror -GST_FSL_PLUGINS_SITE = http://download.ossystems.com.br/bsp/freescale/source +GST_FSL_PLUGINS_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ # Most is LGPLv2+, but some sources are copied from upstream and are # LGPLv2.1+, which essentially makes it LGPLv2.1+ diff --git a/package/libfslcodec/libfslcodec.mk b/package/libfslcodec/libfslcodec.mk index c18332a..b1991f8 100644 --- a/package/libfslcodec/libfslcodec.mk +++ b/package/libfslcodec/libfslcodec.mk @@ -4,9 +4,9 @@ # ################################################################################ -LIBFSLCODEC_VERSION = 3.0.1 +LIBFSLCODEC_VERSION = 3.5.7-1.0.0 # No official download site from freescale, just this mirror -LIBFSLCODEC_SITE = http://download.ossystems.com.br/bsp/freescale/source +LIBFSLCODEC_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ LIBFSLCODEC_SOURCE = libfslcodec-$(LIBFSLCODEC_VERSION).bin LIBFSLCODEC_LICENSE = Freescale Semiconductor Software License Agreement, BSD-3c (flac, ogg headers) LIBFSLCODEC_LICENSE_FILES = EULA EULA.txt diff --git a/package/libfslparser/libfslparser.mk b/package/libfslparser/libfslparser.mk index 3e27a6a..66068fe 100644 --- a/package/libfslparser/libfslparser.mk +++ b/package/libfslparser/libfslparser.mk @@ -4,9 +4,9 @@ # ################################################################################ -LIBFSLPARSER_VERSION = 3.0.1 +LIBFSLPARSER_VERSION = 3.5.7-1.0.0 # No official download site from freescale, just this mirror -LIBFSLPARSER_SITE = http://download.ossystems.com.br/bsp/freescale/source +LIBFSLPARSER_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ LIBFSLPARSER_SOURCE = libfslparser-$(LIBFSLPARSER_VERSION).bin LIBFSLPARSER_LICENSE = Freescale Semiconductor Software License Agreement LIBFSLPARSER_LICENSE_FILES = EULA EULA.txt @@ -31,7 +31,9 @@ define LIBFSLPARSER_EXTRACT_CMDS /<<EOEULA/ { start=1; }'\ $(DL_DIR)/$(LIBFSLPARSER_SOURCE) > $(@D)/EULA cd $(BUILD_DIR); \ - sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept + sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept ; \ + cd libfslparser-3.5.7-1.0.0; \ + pwd && sh ./autogen.sh endef # The Makefile installs several versions of the libraries, but we only diff --git a/package/libfslvpuwrap/libfslvpuwrap.mk b/package/libfslvpuwrap/libfslvpuwrap.mk index 1bc7129..cf11968 100644 --- a/package/libfslvpuwrap/libfslvpuwrap.mk +++ b/package/libfslvpuwrap/libfslvpuwrap.mk @@ -4,9 +4,10 @@ # ################################################################################ -LIBFSLVPUWRAP_VERSION = 1.0.17 +LIBFSLVPUWRAP_VERSION = 3.5.7-1.0.0 # No official download site from freescale, just this mirror -LIBFSLVPUWRAP_SITE = http://download.ossystems.com.br/bsp/freescale/source +LIBFSLVPUWRAP_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ +LIBFSLVPUWRAP_SOURCE = libfslvpuwrap-$(LIBFSLVPUWRAP_VERSION).bin LIBFSLVPUWRAP_LICENSE = Freescale Semiconductor Software License Agreement LIBFSLVPUWRAP_LICENSE_FILES = EULA.txt LIBFSLVPUWRAP_REDISTRIBUTE = NO @@ -15,4 +16,21 @@ LIBFSLVPUWRAP_INSTALL_STAGING = YES LIBFSLVPUWRAP_DEPENDENCIES += imx-lib +# The archive is a shell-self-extractor of a bzipped tar. It happens +# to extract in the correct directory (libfslvpuwrap-x.y.z) +# The --force makes sure it doesn't fail if the source dir already exists. +# The --auto-accept skips the license check - not needed for us +# because we have legal-info +# Since the EULA in the bin file differs from the one in the tar file, +# extract the one from the bin file as well. +define LIBFSLVPUWRAP_EXTRACT_CMDS + awk 'BEGIN { start=0; } \ + /^EOEULA/ { start = 0; } \ + { if (start) print; } \ + /<<EOEULA/ { start=1; }'\ + $(DL_DIR)/$(LIBFSLVPUWRAP_SOURCE) > $(@D)/EULA + cd $(BUILD_DIR); \ + sh $(DL_DIR)/$(LIBFSLVPUWRAP_SOURCE) --force --auto-accept +endef + $(eval $(autotools-package)) -- 1.8.1.2 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-09-25 19:07 ` [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions Eric Nelson @ 2013-09-26 19:36 ` Peter Korsgaard 2013-09-26 20:41 ` Eric Nelson 0 siblings, 1 reply; 17+ messages in thread From: Peter Korsgaard @ 2013-09-26 19:36 UTC (permalink / raw) To: buildroot >>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: Eric> This patchset updates the Freescale gstreamer and GPU libraries Eric> to the latest release (3.0.35_4.1.0). Eric> Note that the package names refer to and are compatible with Eric> an alpha release of kernel version 3.5.7. Eric> The primary release package (based on LTIB) is available Eric> here and includes documentation. Eric> https://community.freescale.com/docs/DOC-95560 Eric> The 3.5.7 kernel sources are available here: Eric> http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/log/?h=imx_3.5.7_1.0.0_alpha Eric> # No official download site from freescale, just this mirror Eric> -FREESCALE_IMX_MIRROR_SITE = http://download.ossystems.com.br/bsp/freescale/source Eric> +FREESCALE_IMX_MIRROR_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ That sounds pretty official to me, so the comment should probably go and the variable perhaps renamed to FREESCALE_IMX_SITE? The page btw seems to be down at the moment as I just get: The system is currently unavailable. Please accept our apologies for any inconvenience this may cause. Feel free to contact our Technical Helpline at one of the following numbers: ... Eric> +++ b/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk Eric> @@ -3,8 +3,12 @@ Eric> # gpu-viv-bin-mx6q Eric> # Eric> ################################################################################ Eric> +ifeq ($(BR2_ARM_EABIHF),y) Eric> +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-hfp Eric> +else Eric> +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-sfp Eric> +endif NIT: There should be an empty line after the ##### line. Eric> -GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION) Eric> GPU_VIV_BIN_MX6Q_SITE = $(FREESCALE_IMX_MIRROR_SITE) Eric> GPU_VIV_BIN_MX6Q_SOURCE = gpu-viv-bin-mx6q-$(GPU_VIV_BIN_MX6Q_VERSION).bin Eric> @@ -30,8 +34,6 @@ endif Eric> # The archive is a shell-self-extractor of a bzipped tar. It happens Eric> # to extract in the correct directory (gpu-viv-bin-mx6q-x.y.z) Eric> # The --force makes sure it doesn't fail if the source dir already exists. Eric> -# The --auto-accept skips the license check - not needed for us Eric> -# because we have legal-info. Eric> define GPU_VIV_BIN_MX6Q_EXTRACT_CMDS Eric> (cd $(BUILD_DIR); \ Eric> sh $(DL_DIR)/$(GPU_VIV_BIN_MX6Q_SOURCE) --force --auto-accept) Did you remove this on purpose? If the --auto-accept option is gone then it should also be removed from here. Eric> diff --git a/package/freescale-imx/imx-lib/imx-lib.mk b/package/freescale-imx/imx-lib/imx-lib.mk Eric> index bd7e174..e3c29b1 100644 Eric> --- a/package/freescale-imx/imx-lib/imx-lib.mk Eric> +++ b/package/freescale-imx/imx-lib/imx-lib.mk Eric> @@ -6,7 +6,8 @@ Eric> IMX_LIB_VERSION = $(FREESCALE_IMX_VERSION) Eric> IMX_LIB_SITE = $(FREESCALE_IMX_MIRROR_SITE) Eric> -IMX_LIB_LICENSE = LGPLv2.1+ Eric> +IMX_LIB_SOURCE = imx-lib-$(IMX_LIB_VERSION).bin Eric> +IMX_LIB_LICENSE = Freescale Semiconductor Software License Agreement Eric> # No license file included Sigh, back to self extracting binaries? :/ Eric> IMX_LIB_INSTALL_STAGING = YES Eric> @@ -25,6 +26,14 @@ IMX_LIB_MAKE_ENV = \ Eric> PLATFORM=$(BR2_PACKAGE_IMX_LIB_PLATFORM) \ Eric> INCLUDE="$(IMX_LIB_INCLUDE)" Eric> +# The archive is a shell-self-extractor of a bzipped tar. It happens Eric> +# to extract in the correct directory (imx-lib-x.y.z) Eric> +# The --force makes sure it doesn't fail if the source dir already exists. Eric> +define IMX_LIB_EXTRACT_CMDS Eric> + (cd $(BUILD_DIR); \ Eric> + sh $(DL_DIR)/$(IMX_LIB_SOURCE) --force --auto-accept) Eric> +endef Does it have an --auto-accept option? If so, please document it, if not remove it here. Eric> +++ b/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk Eric> @@ -4,9 +4,9 @@ Eric> # Eric> ################################################################################ Eric> -GST_FSL_PLUGINS_VERSION = 3.0.1 Eric> +GST_FSL_PLUGINS_VERSION = 3.5.7-1.0.0 Eric> # No official download site from freescale, just this mirror Eric> -GST_FSL_PLUGINS_SITE = http://download.ossystems.com.br/bsp/freescale/source Eric> +GST_FSL_PLUGINS_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ Is this still for gstreamer 0.10? Eric> +++ b/package/libfslcodec/libfslcodec.mk Eric> @@ -4,9 +4,9 @@ Eric> # Eric> ################################################################################ Eric> -LIBFSLCODEC_VERSION = 3.0.1 Eric> +LIBFSLCODEC_VERSION = 3.5.7-1.0.0 Eric> # No official download site from freescale, just this mirror Eric> -LIBFSLCODEC_SITE = http://download.ossystems.com.br/bsp/freescale/source Eric> +LIBFSLCODEC_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ Maybe this should use FREESCALE_IMX_SITE Eric> LIBFSLCODEC_SOURCE = libfslcodec-$(LIBFSLCODEC_VERSION).bin Eric> LIBFSLCODEC_LICENSE = Freescale Semiconductor Software License Agreement, BSD-3c (flac, ogg headers) Eric> LIBFSLCODEC_LICENSE_FILES = EULA EULA.txt Eric> diff --git a/package/libfslparser/libfslparser.mk b/package/libfslparser/libfslparser.mk Eric> index 3e27a6a..66068fe 100644 Eric> --- a/package/libfslparser/libfslparser.mk Eric> +++ b/package/libfslparser/libfslparser.mk Eric> @@ -4,9 +4,9 @@ Eric> # Eric> ################################################################################ Eric> -LIBFSLPARSER_VERSION = 3.0.1 Eric> +LIBFSLPARSER_VERSION = 3.5.7-1.0.0 Eric> # No official download site from freescale, just this mirror Eric> -LIBFSLPARSER_SITE = http://download.ossystems.com.br/bsp/freescale/source Eric> +LIBFSLPARSER_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ Same here (and comment dropped). Eric> LIBFSLPARSER_SOURCE = libfslparser-$(LIBFSLPARSER_VERSION).bin Eric> LIBFSLPARSER_LICENSE = Freescale Semiconductor Software License Agreement Eric> LIBFSLPARSER_LICENSE_FILES = EULA EULA.txt Eric> @@ -31,7 +31,9 @@ define LIBFSLPARSER_EXTRACT_CMDS Eric> /<<EOEULA/ { start=1; }'\ Eric> $(DL_DIR)/$(LIBFSLPARSER_SOURCE) > $(@D)/EULA Eric> cd $(BUILD_DIR); \ Eric> - sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept Eric> + sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept ; \ Eric> + cd libfslparser-3.5.7-1.0.0; \ Eric> + pwd && sh ./autogen.sh Eric> endef Why the pwd? We do have generic autoreconf support in Buildroot. Normally you can just set: LIBFSLPARSERS_AUTORECONF = YES And Buildroot will take care of building the proper auto* packages for the host and running autoreconf. Eric> +++ b/package/libfslvpuwrap/libfslvpuwrap.mk Eric> @@ -4,9 +4,10 @@ Eric> # Eric> ################################################################################ Eric> -LIBFSLVPUWRAP_VERSION = 1.0.17 Eric> +LIBFSLVPUWRAP_VERSION = 3.5.7-1.0.0 Eric> # No official download site from freescale, just this mirror Eric> -LIBFSLVPUWRAP_SITE = http://download.ossystems.com.br/bsp/freescale/source Eric> +LIBFSLVPUWRAP_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ Same comment as elsewhere. Care to fix these issues and resubmit? Thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-09-26 19:36 ` Peter Korsgaard @ 2013-09-26 20:41 ` Eric Nelson 2013-10-01 6:20 ` Arnout Vandecappelle 0 siblings, 1 reply; 17+ messages in thread From: Eric Nelson @ 2013-09-26 20:41 UTC (permalink / raw) To: buildroot Hi Peter, On 09/26/2013 12:36 PM, Peter Korsgaard wrote: >>>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: > > Eric> This patchset updates the Freescale gstreamer and GPU libraries > Eric> to the latest release (3.0.35_4.1.0). > > Eric> Note that the package names refer to and are compatible with > Eric> an alpha release of kernel version 3.5.7. > > Eric> The primary release package (based on LTIB) is available > Eric> here and includes documentation. > Eric> https://community.freescale.com/docs/DOC-95560 > > Eric> The 3.5.7 kernel sources are available here: > Eric> http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/log/?h=imx_3.5.7_1.0.0_alpha > > > Eric> # No official download site from freescale, just this mirror > Eric> -FREESCALE_IMX_MIRROR_SITE = http://download.ossystems.com.br/bsp/freescale/source > Eric> +FREESCALE_IMX_MIRROR_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ > OT: How do you get this nifty formatting? It's about time for me to retire my crusty Mozilla Thunderbird... > That sounds pretty official to me, so the comment should probably go and > the variable perhaps renamed to FREESCALE_IMX_SITE? > > The page btw seems to be down at the moment as I just get: > The system is currently unavailable. > > Please accept our apologies for any inconvenience this may cause. > Feel free to contact our Technical Helpline at one of the following numbers: > ... > There's no page at http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/. This is simply the repository used to feed packages to the Yocto project, with a mirror at O.S. Systems: https://github.com/Freescale/meta-fsl-arm/blob/master/conf/layer.conf#L14 > > Eric> +++ b/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk > Eric> @@ -3,8 +3,12 @@ > Eric> # gpu-viv-bin-mx6q > Eric> # > Eric> ################################################################################ > Eric> +ifeq ($(BR2_ARM_EABIHF),y) > Eric> +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-hfp > Eric> +else > Eric> +GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION)-sfp > Eric> +endif > > NIT: There should be an empty line after the ##### line. > Ok. > Eric> -GPU_VIV_BIN_MX6Q_VERSION = $(FREESCALE_IMX_VERSION) > Eric> GPU_VIV_BIN_MX6Q_SITE = $(FREESCALE_IMX_MIRROR_SITE) > Eric> GPU_VIV_BIN_MX6Q_SOURCE = gpu-viv-bin-mx6q-$(GPU_VIV_BIN_MX6Q_VERSION).bin > > Eric> @@ -30,8 +34,6 @@ endif > Eric> # The archive is a shell-self-extractor of a bzipped tar. It happens > Eric> # to extract in the correct directory (gpu-viv-bin-mx6q-x.y.z) > Eric> # The --force makes sure it doesn't fail if the source dir already exists. > Eric> -# The --auto-accept skips the license check - not needed for us > Eric> -# because we have legal-info. > Eric> define GPU_VIV_BIN_MX6Q_EXTRACT_CMDS > Eric> (cd $(BUILD_DIR); \ > Eric> sh $(DL_DIR)/$(GPU_VIV_BIN_MX6Q_SOURCE) --force --auto-accept) > > Did you remove this on purpose? If the --auto-accept option is gone then > it should also be removed from here. > I actually meant to, but hadn't yet started a discussion on the topic. We should be able to get rid of these comments: http://git.buildroot.net/buildroot/tree/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk#n15 By following the lead of the meta-fsl-arm project and allowing the first of any packages with the Freescale EULA to prompt the user for acceptance, but I'm not sure how to accomplish that. In the Yocto world, a tag ("ACCEPT_FREESCALE_EULA") placed in local.conf is sufficient to bypass the prompt, which requires action by the user. > > Eric> diff --git a/package/freescale-imx/imx-lib/imx-lib.mk b/package/freescale-imx/imx-lib/imx-lib.mk > Eric> index bd7e174..e3c29b1 100644 > Eric> --- a/package/freescale-imx/imx-lib/imx-lib.mk > Eric> +++ b/package/freescale-imx/imx-lib/imx-lib.mk > Eric> @@ -6,7 +6,8 @@ > > Eric> IMX_LIB_VERSION = $(FREESCALE_IMX_VERSION) > Eric> IMX_LIB_SITE = $(FREESCALE_IMX_MIRROR_SITE) > Eric> -IMX_LIB_LICENSE = LGPLv2.1+ > Eric> +IMX_LIB_SOURCE = imx-lib-$(IMX_LIB_VERSION).bin > Eric> +IMX_LIB_LICENSE = Freescale Semiconductor Software License Agreement > Eric> # No license file included > > Sigh, back to self extracting binaries? :/ > Yep. The tar-balls are gone. This one (imx-lib) is a bit misleading, since the extracted files say very clearly that all of the components are released under LGPL. I think somebody in the packaging department was in the "EULA zone"... > > Eric> IMX_LIB_INSTALL_STAGING = YES > Eric> @@ -25,6 +26,14 @@ IMX_LIB_MAKE_ENV = \ > Eric> PLATFORM=$(BR2_PACKAGE_IMX_LIB_PLATFORM) \ > Eric> INCLUDE="$(IMX_LIB_INCLUDE)" > > Eric> +# The archive is a shell-self-extractor of a bzipped tar. It happens > Eric> +# to extract in the correct directory (imx-lib-x.y.z) > Eric> +# The --force makes sure it doesn't fail if the source dir already exists. > Eric> +define IMX_LIB_EXTRACT_CMDS > Eric> + (cd $(BUILD_DIR); \ > Eric> + sh $(DL_DIR)/$(IMX_LIB_SOURCE) --force --auto-accept) > Eric> +endef > > Does it have an --auto-accept option? If so, please document it, if not > remove it here. > Okay, will do. > Eric> +++ b/package/gstreamer/gst-fsl-plugins/gst-fsl-plugins.mk > Eric> @@ -4,9 +4,9 @@ > Eric> # > Eric> ################################################################################ > > Eric> -GST_FSL_PLUGINS_VERSION = 3.0.1 > Eric> +GST_FSL_PLUGINS_VERSION = 3.5.7-1.0.0 > Eric> # No official download site from freescale, just this mirror > Eric> -GST_FSL_PLUGINS_SITE = http://download.ossystems.com.br/bsp/freescale/source > Eric> +GST_FSL_PLUGINS_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ > > Is this still for gstreamer 0.10? > Yes, but version "1.0.0" of the "3.5.7" package and to make things even more confusing, this is the binary that matches up with kernel version 3.0.35_4.1.0.... Have you noticed how the Freescale packages often name things "MXC-something"... I think the "MXC" stands for "Mobile eXtreme Confusion"... > > Eric> +++ b/package/libfslcodec/libfslcodec.mk > Eric> @@ -4,9 +4,9 @@ > Eric> # > Eric> ################################################################################ > > Eric> -LIBFSLCODEC_VERSION = 3.0.1 > Eric> +LIBFSLCODEC_VERSION = 3.5.7-1.0.0 > Eric> # No official download site from freescale, just this mirror > Eric> -LIBFSLCODEC_SITE = http://download.ossystems.com.br/bsp/freescale/source > Eric> +LIBFSLCODEC_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ > > Maybe this should use FREESCALE_IMX_SITE > Agreed. I'm also confused about why this isn't in the freescale-imx/ directory. > > Eric> LIBFSLCODEC_SOURCE = libfslcodec-$(LIBFSLCODEC_VERSION).bin > Eric> LIBFSLCODEC_LICENSE = Freescale Semiconductor Software License Agreement, BSD-3c (flac, ogg headers) > Eric> LIBFSLCODEC_LICENSE_FILES = EULA EULA.txt > Eric> diff --git a/package/libfslparser/libfslparser.mk b/package/libfslparser/libfslparser.mk > Eric> index 3e27a6a..66068fe 100644 > Eric> --- a/package/libfslparser/libfslparser.mk > Eric> +++ b/package/libfslparser/libfslparser.mk > Eric> @@ -4,9 +4,9 @@ > Eric> # > Eric> ################################################################################ > > Eric> -LIBFSLPARSER_VERSION = 3.0.1 > Eric> +LIBFSLPARSER_VERSION = 3.5.7-1.0.0 > Eric> # No official download site from freescale, just this mirror > Eric> -LIBFSLPARSER_SITE = http://download.ossystems.com.br/bsp/freescale/source > Eric> +LIBFSLPARSER_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ > > Same here (and comment dropped). > > > Eric> LIBFSLPARSER_SOURCE = libfslparser-$(LIBFSLPARSER_VERSION).bin > Eric> LIBFSLPARSER_LICENSE = Freescale Semiconductor Software License Agreement > Eric> LIBFSLPARSER_LICENSE_FILES = EULA EULA.txt > Eric> @@ -31,7 +31,9 @@ define LIBFSLPARSER_EXTRACT_CMDS > Eric> /<<EOEULA/ { start=1; }'\ > Eric> $(DL_DIR)/$(LIBFSLPARSER_SOURCE) > $(@D)/EULA > Eric> cd $(BUILD_DIR); \ > Eric> - sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept > Eric> + sh $(DL_DIR)/$(LIBFSLPARSER_SOURCE) --force --auto-accept ; \ > Eric> + cd libfslparser-3.5.7-1.0.0; \ > Eric> + pwd && sh ./autogen.sh > Eric> endef > > Why the pwd? > 'Cause I was debugging ;) > We do have generic autoreconf support in Buildroot. Normally you can > just set: > > LIBFSLPARSERS_AUTORECONF = YES > > And Buildroot will take care of building the proper auto* packages for > the host and running autoreconf. > Thanks for the tip. I'll re-try with that bit. > > Eric> +++ b/package/libfslvpuwrap/libfslvpuwrap.mk > Eric> @@ -4,9 +4,10 @@ > Eric> # > Eric> ################################################################################ > > Eric> -LIBFSLVPUWRAP_VERSION = 1.0.17 > Eric> +LIBFSLVPUWRAP_VERSION = 3.5.7-1.0.0 > Eric> # No official download site from freescale, just this mirror > Eric> -LIBFSLVPUWRAP_SITE = http://download.ossystems.com.br/bsp/freescale/source > Eric> +LIBFSLVPUWRAP_SITE = http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ > > Same comment as elsewhere. > > Care to fix these issues and resubmit? Thanks. > Will do. Regards, Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-09-26 20:41 ` Eric Nelson @ 2013-10-01 6:20 ` Arnout Vandecappelle 2013-10-01 14:07 ` Eric Nelson 2013-10-01 19:42 ` Thomas Petazzoni 0 siblings, 2 replies; 17+ messages in thread From: Arnout Vandecappelle @ 2013-10-01 6:20 UTC (permalink / raw) To: buildroot On 09/26/13 22:41, Eric Nelson wrote: > Hi Peter, > > On 09/26/2013 12:36 PM, Peter Korsgaard wrote: >>>>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: [snip] >> Eric> @@ -30,8 +34,6 @@ endif >> Eric> # The archive is a shell-self-extractor of a bzipped tar. It happens >> Eric> # to extract in the correct directory (gpu-viv-bin-mx6q-x.y.z) >> Eric> # The --force makes sure it doesn't fail if the source dir already exists. >> Eric> -# The --auto-accept skips the license check - not needed for us >> Eric> -# because we have legal-info. >> Eric> define GPU_VIV_BIN_MX6Q_EXTRACT_CMDS >> Eric> (cd $(BUILD_DIR); \ >> Eric> sh $(DL_DIR)/$(GPU_VIV_BIN_MX6Q_SOURCE) --force --auto-accept) >> >> Did you remove this on purpose? If the --auto-accept option is gone then >> it should also be removed from here. >> > > I actually meant to, but hadn't yet started a discussion on the topic. > > We should be able to get rid of these comments: > http://git.buildroot.net/buildroot/tree/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk#n15 > > By following the lead of the meta-fsl-arm project and allowing the first > of any packages with the Freescale EULA to prompt the user for > acceptance, but I'm not sure how to accomplish that. > > In the Yocto world, a tag ("ACCEPT_FREESCALE_EULA") placed in local.conf > is sufficient to bypass the prompt, which requires action by the user. We prefer to avoid that the build stops after half an hour or so because it is waiting for user input. Also, I don't see a reason why we have to explicitly ask the user to accept this license. If we do that, we should probably do the same for all the other packages. If we do want explicit acceptance, then it can be done similar to the Qt license, i.e. add a config option that enables the --auto-accept. > >> >> Eric> diff --git a/package/freescale-imx/imx-lib/imx-lib.mk >> b/package/freescale-imx/imx-lib/imx-lib.mk >> Eric> index bd7e174..e3c29b1 100644 >> Eric> --- a/package/freescale-imx/imx-lib/imx-lib.mk >> Eric> +++ b/package/freescale-imx/imx-lib/imx-lib.mk >> Eric> @@ -6,7 +6,8 @@ >> >> Eric> IMX_LIB_VERSION = $(FREESCALE_IMX_VERSION) >> Eric> IMX_LIB_SITE = $(FREESCALE_IMX_MIRROR_SITE) >> Eric> -IMX_LIB_LICENSE = LGPLv2.1+ >> Eric> +IMX_LIB_SOURCE = imx-lib-$(IMX_LIB_VERSION).bin >> Eric> +IMX_LIB_LICENSE = Freescale Semiconductor Software License >> Agreement >> Eric> # No license file included >> >> Sigh, back to self extracting binaries? :/ >> > > Yep. The tar-balls are gone. This one (imx-lib) is a bit misleading, > since the extracted files say very clearly that all of the components > are released under LGPL. In that case, the license is still LGPL. The EULA text specifies: This Agreement does not concern your use of any files that are subject to an "open source" software license, such as the Free Software Foundation "General Public License," or "GNU GPL." Since only GPL stuff is included in imx-lib, the rest of the EULA essentially doesn't apply. [snip] >> Eric> -LIBFSLCODEC_VERSION = 3.0.1 >> Eric> +LIBFSLCODEC_VERSION = 3.5.7-1.0.0 >> Eric> # No official download site from freescale, just this mirror >> Eric> -LIBFSLCODEC_SITE = >> http://download.ossystems.com.br/bsp/freescale/source >> Eric> +LIBFSLCODEC_SITE = >> http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ >> >> Maybe this should use FREESCALE_IMX_SITE >> > > Agreed. I'm also confused about why this isn't in the freescale-imx/ > directory. Because the libfslcodec fits in the multimedia libraries menu more than in the hardware handling menu. Note that it is not necessary to put the libfslcodec package under the freescale-imx directory for it to use the FREESCALE_IMX_SITE variable. At least, I think so :-) Regards, Arnout [snip] -- 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 ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-10-01 6:20 ` Arnout Vandecappelle @ 2013-10-01 14:07 ` Eric Nelson 2013-10-01 19:42 ` Thomas Petazzoni 1 sibling, 0 replies; 17+ messages in thread From: Eric Nelson @ 2013-10-01 14:07 UTC (permalink / raw) To: buildroot Hi Arnout, On 09/30/2013 11:20 PM, Arnout Vandecappelle wrote: > On 09/26/13 22:41, Eric Nelson wrote: >> Hi Peter, >> >> On 09/26/2013 12:36 PM, Peter Korsgaard wrote: >>>>>>>> "Eric" == Eric Nelson <eric.nelson@boundarydevices.com> writes: > [snip] >>> Eric> @@ -30,8 +34,6 @@ endif >>> Eric> # The archive is a shell-self-extractor of a bzipped tar. It >>> happens >>> Eric> # to extract in the correct directory (gpu-viv-bin-mx6q-x.y.z) >>> Eric> # The --force makes sure it doesn't fail if the source dir >>> already exists. >>> Eric> -# The --auto-accept skips the license check - not needed for us >>> Eric> -# because we have legal-info. >>> Eric> define GPU_VIV_BIN_MX6Q_EXTRACT_CMDS >>> Eric> (cd $(BUILD_DIR); \ >>> Eric> sh $(DL_DIR)/$(GPU_VIV_BIN_MX6Q_SOURCE) --force >>> --auto-accept) >>> >>> Did you remove this on purpose? If the --auto-accept option is gone then >>> it should also be removed from here. >>> >> >> I actually meant to, but hadn't yet started a discussion on the topic. >> >> We should be able to get rid of these comments: >> >> http://git.buildroot.net/buildroot/tree/package/freescale-imx/gpu-viv-bin-mx6q/gpu-viv-bin-mx6q.mk#n15 >> >> >> By following the lead of the meta-fsl-arm project and allowing the first >> of any packages with the Freescale EULA to prompt the user for >> acceptance, but I'm not sure how to accomplish that. >> >> In the Yocto world, a tag ("ACCEPT_FREESCALE_EULA") placed in local.conf >> is sufficient to bypass the prompt, which requires action by the user. > > We prefer to avoid that the build stops after half an hour or so > because it is waiting for user input. Also, I don't see a reason why we > have to explicitly ask the user to accept this license. If we do that, > we should probably do the same for all the other packages. > > If we do want explicit acceptance, then it can be done similar to the > Qt license, i.e. add a config option that enables the --auto-accept. > I'll take a whack at that, but it may take a week or so. >> >>> >>> Eric> diff --git a/package/freescale-imx/imx-lib/imx-lib.mk >>> b/package/freescale-imx/imx-lib/imx-lib.mk >>> Eric> index bd7e174..e3c29b1 100644 >>> Eric> --- a/package/freescale-imx/imx-lib/imx-lib.mk >>> Eric> +++ b/package/freescale-imx/imx-lib/imx-lib.mk >>> Eric> @@ -6,7 +6,8 @@ >>> >>> Eric> IMX_LIB_VERSION = $(FREESCALE_IMX_VERSION) >>> Eric> IMX_LIB_SITE = $(FREESCALE_IMX_MIRROR_SITE) >>> Eric> -IMX_LIB_LICENSE = LGPLv2.1+ >>> Eric> +IMX_LIB_SOURCE = imx-lib-$(IMX_LIB_VERSION).bin >>> Eric> +IMX_LIB_LICENSE = Freescale Semiconductor Software License >>> Agreement >>> Eric> # No license file included >>> >>> Sigh, back to self extracting binaries? :/ >>> >> >> Yep. The tar-balls are gone. This one (imx-lib) is a bit misleading, >> since the extracted files say very clearly that all of the components >> are released under LGPL. > > In that case, the license is still LGPL. The EULA text specifies: > > This Agreement does not concern your use of any files that > are subject to an "open source" software license, such as the Free > Software Foundation "General Public License," or "GNU GPL." > > Since only GPL stuff is included in imx-lib, the rest of the EULA > essentially doesn't apply. > Right. > [snip] >>> Eric> -LIBFSLCODEC_VERSION = 3.0.1 >>> Eric> +LIBFSLCODEC_VERSION = 3.5.7-1.0.0 >>> Eric> # No official download site from freescale, just this mirror >>> Eric> -LIBFSLCODEC_SITE = >>> http://download.ossystems.com.br/bsp/freescale/source >>> Eric> +LIBFSLCODEC_SITE = >>> http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/ >>> >>> Maybe this should use FREESCALE_IMX_SITE >>> >> >> Agreed. I'm also confused about why this isn't in the freescale-imx/ >> directory. > > Because the libfslcodec fits in the multimedia libraries menu more > than in the hardware handling menu. > I'm not sure. It shows up under Packages/Libraries/Multimedia and I found the packages difficult to locate. > Note that it is not necessary to put the libfslcodec package under the > freescale-imx directory for it to use the FREESCALE_IMX_SITE variable. > At least, I think so :-) > That variable (currently FREESCALE_IMX_MIRROR_SITE) is defined in the parent makefile: http://git.buildroot.net/buildroot/tree/package/freescale-imx/freescale-imx.mk ... so it's not appropriate to include from the other tree. Can you take a look at the more recent patch set? Things seem much cleaner by having this all in the same tree. Thanks, Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions 2013-10-01 6:20 ` Arnout Vandecappelle 2013-10-01 14:07 ` Eric Nelson @ 2013-10-01 19:42 ` Thomas Petazzoni 1 sibling, 0 replies; 17+ messages in thread From: Thomas Petazzoni @ 2013-10-01 19:42 UTC (permalink / raw) To: buildroot Dear Arnout Vandecappelle, On Tue, 01 Oct 2013 08:20:47 +0200, Arnout Vandecappelle wrote: > > By following the lead of the meta-fsl-arm project and allowing the first > > of any packages with the Freescale EULA to prompt the user for > > acceptance, but I'm not sure how to accomplish that. > > > > In the Yocto world, a tag ("ACCEPT_FREESCALE_EULA") placed in local.conf > > is sufficient to bypass the prompt, which requires action by the user. > > We prefer to avoid that the build stops after half an hour or so > because it is waiting for user input. Also, I don't see a reason why we > have to explicitly ask the user to accept this license. If we do that, we > should probably do the same for all the other packages. > > If we do want explicit acceptance, then it can be done similar to the > Qt license, i.e. add a config option that enables the --auto-accept. Agreed. > > Agreed. I'm also confused about why this isn't in the freescale-imx/ > > directory. > > Because the libfslcodec fits in the multimedia libraries menu more than > in the hardware handling menu. I don't quite agree here. The location of the package in the menu has nothing to do with the location of the package .mk file and directory. So if the libfslcodec is downloaded from the same site and with the same version as the other package/freescale-imx/ stuff and therefore re-use the same variables, it could very well be moved under this directory. Best regards, Thomas -- Thomas Petazzoni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2013-10-01 19:42 UTC | newest] Thread overview: 17+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-09-25 19:07 [Buildroot] [PATCH 0/4] i.MX6 and Nitrogen6x updates Eric Nelson 2013-09-25 19:07 ` [Buildroot] [PATCH 1/4] nitrogen6x: bump U-Boot to latest production version (2013.07) Eric Nelson 2013-09-26 19:19 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 2/4] nitrogen6x: use 6x_bootscript/6x_upgrade instead of older 6q_ versions Eric Nelson 2013-09-26 14:37 ` Peter Korsgaard 2013-09-26 15:13 ` Eric Nelson 2013-09-26 19:10 ` Peter Korsgaard 2013-09-26 20:20 ` Eric Nelson 2013-09-26 19:20 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 3/4] nitrogen6x: bump kernel to latest version (3.0.35-4.1.0) Eric Nelson 2013-09-26 19:20 ` Peter Korsgaard 2013-09-25 19:07 ` [Buildroot] [PATCH 4/4] i.MX6: Update Freescale hardware-acceleration bits to latest versions Eric Nelson 2013-09-26 19:36 ` Peter Korsgaard 2013-09-26 20:41 ` Eric Nelson 2013-10-01 6:20 ` Arnout Vandecappelle 2013-10-01 14:07 ` Eric Nelson 2013-10-01 19:42 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox