[Buildroot] [pull request] Pull request for branch yem/misc-updates

[Buildroot] [pull request] Pull request for branch yem/misc-updates

[Yann E. MORIN]

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Hello All!

Peter, here are a few changes I've been sitting on since before I
switched into holidays mode (and in redo-the-kitchen-from-scrath mode,
which ultimately kicked in the has-no-time-to-get-in-front-of-computer
mode) three weeks ago.

I'm now back in operational condition! :-]

  - two changes RPI-related: bump kernel (rpi_defconfig), bunp firmware
  - further cleanup top-level config by moving target options to their
    own sub-menu
  - optionally print toolchain wrapper's args one-per-line in DEBUG mode

I still have to dequeue all my mails, but I'm still on holidays until
the end of the week, so... ;-)

Regards,
Yann E. MORIN.


The following changes since commit 5128cc6602d56a2c3af4cf6a0fe072d39d6c1aca:

  gcc: pass MAKEINFO=missing in the environment rather than as a ./configure arg (2013-09-04 16:22:30 +0200)

are available in the git repository at:

  git://gitorious.org/buildroot/buildroot.git 

for you to fetch changes up to e1d03e22b10b073d5906a4b6df22c72d38bfd8c2:

  toolchain/wrapper: add option to print one argument per line (2013-09-04 23:27:37 +0200)

----------------------------------------------------------------
Yann E. MORIN (4):
      package/rpi-firmware: bump version
      configs: bump kernel use for RPi, use simpler defconfig
      targets: move target options to their own sub-menu
      toolchain/wrapper: add option to print one argument per line

 arch/Config.in                                     |  4 ++++
 configs/rpi_defconfig                              |  4 ++--
 package/rpi-firmware/rpi-firmware.mk               |  2 +-
 .../toolchain-external/ext-toolchain-wrapper.c     | 25 +++++++++++++++-------
 4 files changed, 24 insertions(+), 11 deletions(-)

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH 1/4] package/rpi-firmware: bump version

[Yann E. MORIN]

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Fixes and improvements for USB, audio sync, camera support,
codecs, HDMI framerate and audio, and GPU freezes.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/rpi-firmware/rpi-firmware.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/rpi-firmware/rpi-firmware.mk b/package/rpi-firmware/rpi-firmware.mk
index 6ea621f..55c1ca4 100644
--- a/package/rpi-firmware/rpi-firmware.mk
+++ b/package/rpi-firmware/rpi-firmware.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-RPI_FIRMWARE_VERSION = 320084a
+RPI_FIRMWARE_VERSION = d4f5315
 RPI_FIRMWARE_SITE = http://github.com/raspberrypi/firmware/tarball/$(RPI_FIRMWARE_VERSION)
 RPI_FIRMWARE_LICENSE = BSD-3c
 RPI_FIRMWARE_LICENSE_FILES = boot/LICENCE.broadcom
-- 
1.8.1.2

[Buildroot] [PATCH 1/4] package/rpi-firmware: bump version

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 Yann> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
 Yann> Fixes and improvements for USB, audio sync, camera support,
 Yann> codecs, HDMI framerate and audio, and GPU freezes.

Committed, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 2/4] configs: bump kernel use for RPi, use simpler defconfig

[Yann E. MORIN]

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Bump for a slew of USB-related and other fixes.

The bcmrpi defconfig is a huge beast with lots of superfluous drivers.
The brcmrpi_quick defconfig is a stripped down minimalist defconfig with
all that is needed to bring up a RPi with complete HW support, and
nothing superfluous, which is what we need for a board's defconfig in
buildroot.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 configs/rpi_defconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/configs/rpi_defconfig b/configs/rpi_defconfig
index 9d608f1..2b19673 100644
--- a/configs/rpi_defconfig
+++ b/configs/rpi_defconfig
@@ -17,7 +17,7 @@ BR2_DEFAULT_KERNEL_VERSION="3.6.11"
 BR2_LINUX_KERNEL=y
 BR2_LINUX_KERNEL_CUSTOM_GIT=y
 BR2_LINUX_KERNEL_CUSTOM_GIT_REPO_URL="git://github.com/raspberrypi/linux.git"
-BR2_LINUX_KERNEL_CUSTOM_GIT_VERSION="0006326"
+BR2_LINUX_KERNEL_CUSTOM_GIT_VERSION="1587f77"
 BR2_LINUX_KERNEL_USE_DEFCONFIG=y
-BR2_LINUX_KERNEL_DEFCONFIG="bcmrpi"
+BR2_LINUX_KERNEL_DEFCONFIG="bcmrpi_quick"
 BR2_LINUX_KERNEL_ZIMAGE=y
-- 
1.8.1.2

[Buildroot] [PATCH 2/4] configs: bump kernel use for RPi, use simpler defconfig

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 Yann> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
 Yann> Bump for a slew of USB-related and other fixes.

 Yann> The bcmrpi defconfig is a huge beast with lots of superfluous drivers.
 Yann> The brcmrpi_quick defconfig is a stripped down minimalist defconfig with
 Yann> all that is needed to bring up a RPi with complete HW support, and
 Yann> nothing superfluous, which is what we need for a board's defconfig in
 Yann> buildroot.

Committed, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 3/4] targets: move target options to their own sub-menu

[Yann E. MORIN]

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Currently, all target options lie in the top-level menu.
This looks a bit clumsy, since all other options are neatly
folded into their own sematic sub-menus each.

Move the target option into their own sub-menu too.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
---
 arch/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/Config.in b/arch/Config.in
index 14899ca..3fd8c1c 100644
--- a/arch/Config.in
+++ b/arch/Config.in
@@ -1,3 +1,5 @@
+menu "Target options"
+
 config BR2_ARCH_IS_64
 	bool
 
@@ -335,3 +337,5 @@ endif
 if BR2_xtensa
 source "arch/Config.in.xtensa"
 endif
+
+endmenu # Target options
-- 
1.8.1.2

[Buildroot] [PATCH 3/4] targets: move target options to their own sub-menu

[Thomas De Schampheleire]

Op 4-sep.-2013 23:36 schreef "Yann E. MORIN" <yann.morin.1998@free.fr> het
volgende:
>
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> Currently, all target options lie in the top-level menu.
> This looks a bit clumsy, since all other options are neatly
> folded into their own sematic sub-menus each.
>
> Move the target option into their own sub-menu too.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

This is an Acked-by...

> ---
>  arch/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/arch/Config.in b/arch/Config.in
> index 14899ca..3fd8c1c 100644
> --- a/arch/Config.in
> +++ b/arch/Config.in
> @@ -1,3 +1,5 @@
> +menu "Target options"
> +
>  config BR2_ARCH_IS_64
>         bool
>
> @@ -335,3 +337,5 @@ endif
>  if BR2_xtensa
>  source "arch/Config.in.xtensa"
>  endif
> +
> +endmenu # Target options
> --
> 1.8.1.2
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20130905/3ee98aeb/attachment-0001.html>

[Buildroot] [PATCH 3/4] targets: move target options to their own sub-menu

[Yann E. MORIN]

Thomas, All,

On 2013-09-05 04:25 +0200, Thomas De Schampheleire spake thusly:
> Op 4-sep.-2013 23:36 schreef "Yann E. MORIN" <yann.morin.1998@free.fr> het
> volgende:
> >
> > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> >
> > Currently, all target options lie in the top-level menu.
> > This looks a bit clumsy, since all other options are neatly
> > folded into their own sematic sub-menus each.
> >
> > Move the target option into their own sub-menu too.
> >
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
> 
> This is an Acked-by...

Doh. Yes, bad cut-n-paste, sorry.

It seems I have to re-train my fingers after being away from the kbd for
15+ days... :-/

Regrads,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH 3/4] targets: move target options to their own sub-menu

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 Yann> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
 Yann> Currently, all target options lie in the top-level menu.
 Yann> This looks a bit clumsy, since all other options are neatly
 Yann> folded into their own sematic sub-menus each.

 Yann> Move the target option into their own sub-menu too.

 Yann> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 Yann> Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

Committed, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 4/4] toolchain/wrapper: add option to print one argument per line

[Yann E. MORIN]

From: "Yann E. MORIN" <yann.morin.1998@free.fr>

In case there are many arguments passed to the tools, the command line
can get very long, and difficult to parse visually.

For example, the Linux kernel passes a lot of arguments to gcc (at least
45, which gives 53 with our hard-coded args). Looking at such a command
line is daunting.

So, add the possibility to print each argument on its own line.

Also, enclose all args between single quotes, so the command line
can be safely copy-pasted without special chars (spaces, $) being
inrerpreted by the shell.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 .../toolchain-external/ext-toolchain-wrapper.c     | 25 +++++++++++++++-------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/toolchain/toolchain-external/ext-toolchain-wrapper.c b/toolchain/toolchain-external/ext-toolchain-wrapper.c
index 565e36b..dfdfcff 100644
--- a/toolchain/toolchain-external/ext-toolchain-wrapper.c
+++ b/toolchain/toolchain-external/ext-toolchain-wrapper.c
@@ -74,7 +74,8 @@ int main(int argc, char **argv)
 	char *relbasedir, *absbasedir;
 	char *progpath = argv[0];
 	char *basename;
-	int ret, i, count = 0;
+	char *env_debug;
+	int ret, i, count = 0, debug;
 
 	/* Calculate the relative paths */
 	basename = strrchr(progpath, '/');
@@ -157,13 +158,21 @@ int main(int argc, char **argv)
 	/* finish with NULL termination */
 	*cur = NULL;
 
-	if (getenv("BR_DEBUG_WRAPPER")) {
-		fprintf(stderr, "Executing");
-
-		for (i = 0; args[i]; i++)
-			fprintf(stderr, " %s", args[i]);
-
-		fprintf(stderr, "\n");
+	/* Debug the wrapper to see actual arguments passed to
+	 * the compiler:
+	 * unset, empty, or 0: do not trace
+	 * set to 1          : trace all arguments on a single line
+	 * set to 2          : trace one argument per line
+	 */
+	if ((env_debug = getenv("BR_DEBUG_WRAPPER"))) {
+		debug = atoi(env_debug);
+		if (debug > 0) {
+			fprintf(stderr, "Toolchain wrapper executing:");
+			for (i = 0; args[i]; i++)
+				fprintf(stderr, "%s'%s'",
+					(debug == 2)?"\n    ":" ", args[i]);
+			fprintf(stderr, "\n");
+		}
 	}
 
 	if (execv(path, args))
-- 
1.8.1.2

[Buildroot] [PATCH 4/4] toolchain/wrapper: add option to print one argument per line

[Luca Ceresoli]

Yann E. MORIN wrote:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> In case there are many arguments passed to the tools, the command line
> can get very long, and difficult to parse visually.
>
> For example, the Linux kernel passes a lot of arguments to gcc (at least
> 45, which gives 53 with our hard-coded args). Looking at such a command
> line is daunting.
>
> So, add the possibility to print each argument on its own line.
>
> Also, enclose all args between single quotes, so the command line
> can be safely copy-pasted without special chars (spaces, $) being
> inrerpreted by the shell.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

This feature should be mentioned in the docs.

Apart for that:

Acked-by: Luca Ceresoli <luca@lucaceresoli.net>
Tested-by: Luca Ceresoli <luca@lucaceresoli.net>
[Tested on arm_foundationv8_defconfig]

Luca

[Buildroot] [PATCH 4/4] toolchain/wrapper: add option to print one argument per line

[Yann E. MORIN]

Luca, All,

On 2013-09-05 14:48 +0200, Luca Ceresoli spake thusly:
> Yann E. MORIN wrote:
> >From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> >
> >In case there are many arguments passed to the tools, the command line
> >can get very long, and difficult to parse visually.
> >
> >For example, the Linux kernel passes a lot of arguments to gcc (at least
> >45, which gives 53 with our hard-coded args). Looking at such a command
> >line is daunting.
> >
> >So, add the possibility to print each argument on its own line.
> >
> >Also, enclose all args between single quotes, so the command line
> >can be safely copy-pasted without special chars (spaces, $) being
> >inrerpreted by the shell.
> >
> >Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> This feature should be mentioned in the docs.

Well, for now, BR_DEBUG_WRAPPER is not even documented.
But I agree this is missing; will add.

> Apart for that:
> 
> Acked-by: Luca Ceresoli <luca@lucaceresoli.net>
> Tested-by: Luca Ceresoli <luca@lucaceresoli.net>
> [Tested on arm_foundationv8_defconfig]

I'll add those when I respin the patch. Thank you! :-)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'