Re: Better reviews for the same cost in gcc-linaro

2011-03-22 Thread Michael Hope
Hi Robert. The builds end up at: http://builds.linaro.org/toolchain/ and include the times for each step such as: http://builds.linaro.org/toolchain/gcc-linaro-4.5+bzr99475/logs/armv7l-maverick-cbuild49-carina6-cortexa9r1/gcc-build.time http://builds.linaro.org/toolchain/gcc-linaro-4.5+bzr99

Re: How do you see a patch series?

2011-03-22 Thread John Rigby
On Tue, Mar 22, 2011 at 11:10 AM, Peter Maydell wrote: > On 22 March 2011 16:56, Andy Doan wrote: >> On 03/22/2011 08:05 AM, Guilherme Salgado wrote: >>> That's to decide whether we should count the patches that belong to a >>> series individually or not. >>> >> >> Could you generate two sets of

Re: How do you see a patch series?

2011-03-22 Thread Peter Maydell
On 22 March 2011 16:56, Andy Doan wrote: > On 03/22/2011 08:05 AM, Guilherme Salgado wrote: >> That's to decide whether we should count the patches that belong to a >> series individually or not. >> > > Could you generate two sets of stats like "Total Patches Per Author" and > "Total Patch-Series

Re: [PATCH android/device/linaro/beagleboard] Add vold.fstab

2011-03-22 Thread Alexander Sack
On Tue, Mar 22, 2011 at 8:07 AM, Jeremy Chang wrote: > Same time, we could rethink the partition layout, > Partition layout is wikied at > https://wiki.linaro.org/Platform/Android/DevBoards. > The original partition layout define is from > https://bugs.launchpad.net/linaro-image-tools/+bug/724207

Re: [PATCH android/device/linaro/beagleboard] Add vold.fstab

2011-03-22 Thread Alexander Sack
On Tue, Mar 22, 2011 at 8:07 AM, Jeremy Chang wrote: > Same time, we could rethink the partition layout, > Partition layout is wikied at > https://wiki.linaro.org/Platform/Android/DevBoards. > The original partition layout define is from > https://bugs.launchpad.net/linaro-image-tools/+bug/724207

Re: How do you see a patch series?

2011-03-22 Thread Andy Doan
On 03/22/2011 08:05 AM, Guilherme Salgado wrote: > That's to decide whether we should count the patches that belong to a > series individually or not. > Could you generate two sets of stats like "Total Patches Per Author" and "Total Patch-Series Per Author" or possibly try and give more weight to

Re: [PATCH v2 android/system/vold] Promote the max partition support to 8

2011-03-22 Thread Patrik Ryd
Looks good to me. /Patrik On 22 March 2011 13:58, Jeremy Chang wrote: > This is needed for supporting multiple partition layout. > > Delete MAX_PARTS define since it's not used and also to make it look > consistent. > > Change-Id: Ic4e1b5f7219cb51f02d9e0d2c14046da9f7b51f1 > > Signed-off-by: Je

Re: How do you see a patch series?

2011-03-22 Thread Alexander Sack
On Tue, Mar 22, 2011 at 4:05 PM, Guilherme Salgado wrote: > On Mon, 2011-03-21 at 18:24 +, Peter Maydell wrote: >> On 18 March 2011 23:29, Guilherme Salgado >> wrote: >> > I'm currently working on the PatchTracking[1] spec and, wrt patch >> > series, I'm wondering whether you consider the pa

Re: How do you see a patch series?

2011-03-22 Thread Guilherme Salgado
On Mon, 2011-03-21 at 18:24 +, Peter Maydell wrote: > On 18 March 2011 23:29, Guilherme Salgado > wrote: > > I'm currently working on the PatchTracking[1] spec and, wrt patch > > series, I'm wondering whether you consider the patches that are part of > > a series to be separate or the series

[PATCH] arm: handle ptrace requests to change PC during interrupted system calls

2011-03-22 Thread Arnd Bergmann
From: Ulrich Weigand GDB's interrupt.exp test cases currenly fail on ARM. The problem is how do_signal handled restarting interrupted system calls: The entry.S assembler code determines that we come from a system call; and that information is passed as "syscall" parameter to do_signal. That r

Re: gcc: Thumb interworking and weakly linked functions

2011-03-22 Thread Ulrich Weigand
Aneesh V wrote: > On Wednesday 16 March 2011 07:14 PM, Ulrich Weigand wrote: > > The R_ARM_THM_CALL marks the branch instruction as still to be processed > > by the final link step. And once you actually *perform* the final link, > > e.g. via "gcc -o final a.out", the branch gets indeed converted

[PATCH v2 android/system/vold] Promote the max partition support to 8

2011-03-22 Thread Jeremy Chang
This is needed for supporting multiple partition layout. Delete MAX_PARTS define since it's not used and also to make it look consistent. Change-Id: Ic4e1b5f7219cb51f02d9e0d2c14046da9f7b51f1 Signed-off-by: Jeremy Chang --- DirectVolume.h |4 +--- 1 files changed, 1 insertions(+), 3 deletio

[PATCH] ARM: EXYNOS4: Fix addruart macro

2011-03-22 Thread Thomas Abraham
Fix incorrect conditional execution of ldr instructions in addruart macro. Signed-off-by: Thomas Abraham Signed-off-by: Abhilash Kesavan --- arch/arm/mach-exynos4/include/mach/debug-macro.S |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-exynos4/include

[powerdebug 07/14] save some bytes by using the 'bool' type

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano sizeof(int) : 4 sizeof(bool): 1 Signed-off-by: Daniel Lezcano --- clocks.h |1 - powerdebug.c | 50 ++ powerdebug.h |2 +- regulator.h |1 - 4 files changed, 27 insertions(+), 27 deletions(-) diff --git a/

[powerdebug 10/14] make 'dump' variable static

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- clocks.c | 142 +- clocks.h |1 - powerdebug.c |8 ++-- powerdebug.h |7 +-- 4 files changed, 78 insertions(+), 80 deletions(-) diff --git a/clocks.c b/clocks.c i

[powerdebug 14/14] Add configure framework

2011-03-22 Thread Daniel Lezcano
Signed-off-by: Daniel Lezcano --- AUTHORS |1 + ChangeLog|1 + Makefile.am | 13 + NEWS |1 + autogen.sh |9 + configure.ac | 26 ++ 6 files changed, 51 insertions(+), 0 deletions(-) create mode 100644 AUTHORS c

[powerdebug 13/14] Remove Makefile

2011-03-22 Thread Daniel Lezcano
Remove the Makefile to put in place the next patch where it is generated Signed-off-by: Daniel Lezcano --- Makefile | 29 - 1 files changed, 0 insertions(+), 29 deletions(-) delete mode 100644 Makefile diff --git a/Makefile b/Makefile deleted file mode 100644 inde

[powerdebug 12/14] make 'selectedwindow' variable static

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- clocks.c |6 +++--- display.c|6 +++--- powerdebug.c | 50 ++ powerdebug.h | 10 -- 4 files changed, 36 insertions(+), 36 deletions(-) diff --git a/clocks.c b/cloc

[powerdebug 06/14] encapsulate the keystroke callback

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.c | 161 +++--- 1 files changed, 86 insertions(+), 75 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 12c0c56..b9b1ac5 100644 --- a/powerdebug.c +++ b/powerdebug.c

[powerdebug 11/14] move the 'dump' variable to the options structure

2011-03-22 Thread Daniel Lezcano
Signed-off-by: Daniel Lezcano --- powerdebug.c | 29 +++-- 1 files changed, 15 insertions(+), 14 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 2f3a77d..db19e16 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -18,7 +18,6 @@ #include #include "powerdebug.

[powerdebug 01/14] encapsulate the options code

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Create a single structure for powerdebug options. Signed-off-by: Daniel Lezcano --- powerdebug.c | 175 +- 1 files changed, 99 insertions(+), 76 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 197f255..4

[powerdebug 08/14] Remove pointless declarations

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.h |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/powerdebug.h b/powerdebug.h index 5bf7084..f23bbba 100644 --- a/powerdebug.h +++ b/powerdebug.h @@ -36,9 +36,6 @@ extern int selectedwindow; extern i

[powerdebug 05/14] encapsulate the mainloop code

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.c | 48 +--- 1 files changed, 29 insertions(+), 19 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index f2d02c8..12c0c56 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -162,3

[powerdebug 09/14] reduce scope and make win_names static

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- display.c|5 + powerdebug.c |5 - powerdebug.h |1 - 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/display.c b/display.c index 5cdf78a..fbdce80 100644 --- a/display.c +++ b/display.c @@ -31,6 +31,11

[powerdebug 03/14] use an int instead of a double

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 3529459..44f1f02 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -125,7 +125,7 @@ int getoptions(int argc, char *argv[]

[powerdebug 02/14] save some bytes by allocating dynamically the string option

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 4c2400b..3529459 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -85,7 +85,7 @@ struct powerdebug_options {

[powerdebug 04/14] check the return value of init_regulator_ds and fix prototype

2011-03-22 Thread Daniel Lezcano
From: Daniel Lezcano Signed-off-by: Daniel Lezcano --- powerdebug.c |3 ++- powerdebug.h |2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 44f1f02..f2d02c8 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -185,7 +185,8 @@ int main(int

Re: Yet another memory provider: can linaro organize a meeting?

2011-03-22 Thread Hans Verkuil
On Wednesday, March 16, 2011 09:14:54 Kyungmin Park wrote: > Rough schedules. > > 1. Warsaw meetings (3/16~3/18): mostly v4l2 person and some SoC vendors > Make a consensence at media developers. and share the information. > Please note that it's v4l2 brainstorming meeting. so memory > managem

Re: Yet another memory provider: can linaro organize a meeting?

2011-03-22 Thread Laurent Pinchart
On Monday 21 March 2011 19:03:38 Hans Verkuil wrote: > On Wednesday, March 16, 2011 09:14:54 Kyungmin Park wrote: > > Rough schedules. > > > > 1. Warsaw meetings (3/16~3/18): mostly v4l2 person and some SoC vendors > > > > Make a consensence at media developers. and share the information. > >

Re: [PATCH android/device/linaro/beagleboard] Add vold.fstab

2011-03-22 Thread Jeremy Chang
On Tue, Mar 22, 2011 at 6:25 PM, Patrik Ryd wrote: > On 21 March 2011 19:35, Jeremy Chang wrote: >> >> vold.fstab is the vold configuration file to tell what the device to >> mount. >> >> Add device.mk, the device-specific product definition file for copy >> device-specific drivers, configuration

Re: [PATCH android/device/linaro/beagleboard] Add vold.fstab

2011-03-22 Thread Patrik Ryd
On 21 March 2011 19:35, Jeremy Chang wrote: > vold.fstab is the vold configuration file to tell what the device to > mount. > > Add device.mk, the device-specific product definition file for copy > device-specific drivers, configuration files, settings, etc. > > Signed-off-by: Jeremy Chang > ---

Re: Device Tree on ARM status report - Mar 20

2011-03-22 Thread Tixy
Hi Grant On Mon, 2011-03-21 at 03:25 -0600, Grant Likely wrote: > For each board, I need an engineer to do the following: > > 1) Enable CONFIG_OF_LIBFDT and CONFIG_SYS_BOOTMAPSZ against the Linaro > u-boot tree. Which tree should I use, u-boot-linaro-stable or u-boot-linaro-next ? -- Tixy __

Re: [PATCH android/device/linaro/beagleboard] Add vold.fstab

2011-03-22 Thread Jeremy Chang
Same time, we could rethink the partition layout, Partition layout is wikied at https://wiki.linaro.org/Platform/Android/DevBoards. The original partition layout define is from https://bugs.launchpad.net/linaro-image-tools/+bug/724207 Regarding to the partition offset for our validation farm insta