Re: Device Tree on ARM status report - Mar 20

On Sat, Mar 26, 2011 at 9:33 AM, Tixy wrote: > On Fri, 2011-03-25 at 22:46 -0600, Grant Likely wrote: > [...] >> It appears that when U-Boot >> relocates the .dtb, it either moves it to a location that the kernel >> cannot read during early boot, or it corrupts it when it is moved. >> Either way,

[powerdebug 35/35] Don't exit when the regulator init fails

We just disable the option Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 86f51bc..0f86e0c 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -377,7 +377,7 @@ int main(int argc, char **argv)

[powerdebug 34/35] Make some variable static

These variable are only used in the sensor.c file. Signed-off-by: Daniel Lezcano --- sensor.c |9 + sensor.h |8 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/sensor.c b/sensor.c index f5fcd40..df07593 100644 --- a/sensor.c +++ b/sensor.c @@ -16,6 +16,15

[powerdebug 32/35] Remove pointless function definitions

By moving the functions in the right order in the file, we can get ride of their definitions and we can set them static. Signed-off-by: Daniel Lezcano --- clocks.c | 195 -- clocks.h | 18 -- 2 files changed, 101 insertions(+), 1

[powerdebug 33/35] create a specific function for display

In order to have the code more clear, let's create a function for the display like what we did with the dump function. Signed-off-by: Daniel Lezcano --- powerdebug.c | 45 - 1 files changed, 24 insertions(+), 21 deletions(-) diff --git a/powerdebug.

[powerdebug 31/35] simplify clock_init function

Signed-off-by: Daniel Lezcano --- clocks.c | 39 --- clocks.h |2 ++ powerdebug.c | 12 ++-- powerdebug.h |1 - 4 files changed, 12 insertions(+), 42 deletions(-) diff --git a/clocks.c b/clocks.c index 9577267..c83055f 100644 --- a/

[powerdebug 30/35] Cleanup the code by removing weird initialization

Signed-off-by: Daniel Lezcano --- powerdebug.c | 42 -- 1 files changed, 12 insertions(+), 30 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 215b0fe..ef2098c 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -142,19 +142,9 @@ int getoptions(in

[powerdebug 28/35] Do more error check in display init

Signed-off-by: Daniel Lezcano --- display.c| 28 display.h|2 +- powerdebug.c | 10 -- 3 files changed, 25 insertions(+), 15 deletions(-) diff --git a/display.c b/display.c index cf1d5d7..aee5503 100644 --- a/display.c +++ b/display.c @@ -66,2

[powerdebug 29/35] Replace colors definition by enum

and reduce the scope by moving it to the c file. Signed-off-by: Daniel Lezcano --- display.c | 10 ++ display.h |9 - 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/display.c b/display.c index aee5503..2d63908 100644 --- a/display.c +++ b/display.c @@ -20,6

[powerdebug 26/35] Unify the init function name for display

Signed-off-by: Daniel Lezcano --- display.c|2 +- display.h|2 ++ powerdebug.c |3 ++- powerdebug.h |1 - 4 files changed, 5 insertions(+), 3 deletions(-) diff --git a/display.c b/display.c index fb252cb..8ea796a 100644 --- a/display.c +++ b/display.c @@ -65,7 +65,7 @@ vo

[powerdebug 25/35] Remove unused variables and definitions

Signed-off-by: Daniel Lezcano --- display.h |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/display.h b/display.h index f00a4f8..a73ae43 100644 --- a/display.h +++ b/display.h @@ -13,10 +13,6 @@ * - initial API and implementation

[powerdebug 27/35] mindless - change function name for consistency

Signed-off-by: Daniel Lezcano --- display.c| 12 ++-- powerdebug.h |1 - 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/display.c b/display.c index 8ea796a..cf1d5d7 100644 --- a/display.c +++ b/display.c @@ -36,10 +36,6 @@ static char *win_names[TOTAL_FEATURE_WIN

[powerdebug 23/35] Remove unused clk_name variable

Signed-off-by: Daniel Lezcano --- clocks.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/clocks.c b/clocks.c index 2c9597d..5c5c24f 100644 --- a/clocks.c +++ b/clocks.c @@ -20,7 +20,6 @@ #include "clocks.h" static char clk_dir_path[PATH_MAX]; -static char clk_na

[powerdebug 05/35] make regulator_init to return a regulators_info pointer

The regulator_init function does no longer use the global defined function. Signed-off-by: Daniel Lezcano --- powerdebug.c |5 - regulator.c | 18 ++ regulator.h |2 +- 3 files changed, 11 insertions(+), 14 deletions(-) diff --git a/powerdebug.c b/powerdebug.c in

[powerdebug 16/35] split dump and display

Let's create a specific function to dump instead of managing with ncurses Signed-off-by: Daniel Lezcano Index: powerdebug/powerdebug.c === --- powerdebug.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) dif

[powerdebug 22/35] Reduce dump usage

Signed-off-by: Daniel Lezcano --- powerdebug.c | 18 +- 1 files changed, 13 insertions(+), 5 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 7ef0a22..d337cc8 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -141,13 +141,21 @@ int getoptions(int argc, char *argv[], st

[powerdebug 03/35] browse /sys/class/regulator with a generic name

All the files found in the /sys/class/regulator directory should be regulators. Don't assume their name begin with 'regulator'. Signed-off-by: Daniel Lezcano --- regulator.c | 18 ++ 1 files changed, 14 insertions(+), 4 deletions(-) diff --git a/regulator.c b/regulator.c index

[powerdebug 24/35] Return an error instead of exiting deeper in the code

Signed-off-by: Daniel Lezcano --- clocks.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/clocks.c b/clocks.c index 5c5c24f..9577267 100644 --- a/clocks.c +++ b/clocks.c @@ -64,7 +64,7 @@ int init_clock_details(bool dump, int selectedwindow) fp

[powerdebug 21/35] improve function to read the file

Use a simple function to read the content of the file. Fix the format of the flags which were always zero by reading in heaxdecimal mode. Signed-off-by: Daniel Lezcano --- clocks.c | 37 ++--- 1 files changed, 22 insertions(+), 15 deletions(-) diff --git a/cloc

[powerdebug 20/35] change debugfs finder function

Change the debugfs_locate_mpoint function to be nicer and more efficient. Another effect is that fixes a segmentation fault when calling powerdebug --dump Signed-off-by: Daniel Lezcano --- clocks.c | 68 + clocks.h |7 --

[powerdebug 08/35] regulator-8

The show_regulator_info function does no longer rely on the global regulators_info variable. Signed-off-by: Daniel Lezcano --- display.c| 20 ++-- powerdebug.c |2 +- powerdebug.h |2 +- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/display.c b/di

[powerdebug 06/35] pass regulators_info parameter to read_info_from_dirent

The read_info_from_dirent does no longer rely on the regulators_info global variable. Signed-off-by: Daniel Lezcano --- regulator.c | 33 ++--- 1 files changed, 18 insertions(+), 15 deletions(-) diff --git a/regulator.c b/regulator.c index ddbeea3..fc40f27 100644 -

[powerdebug 18/35] Reduce dump variable usage

The modified function are used on the display mode context, we no longer need to pass the dump option. Signed-off-by: Daniel Lezcano --- clocks.c |4 ++-- powerdebug.c |3 +-- powerdebug.h |2 +- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/clocks.c b/clocks.c i

[powerdebug 02/35] change regulator functions name

Move the corresponding functions to the right header file and rename the function with _function_name. Signed-off-by: Daniel Lezcano --- powerdebug.c |4 ++-- powerdebug.h |4 regulator.c |8 regulator.h |2 ++ 4 files changed, 8 insertions(+), 10 deletions(-) di

[powerdebug 01/35] change regulator init function

Move the corresponding function to the right header file and rename the function with _function_name. Signed-off-by: Daniel Lezcano --- powerdebug.c |3 ++- powerdebug.h |1 - regulator.c |2 +- regulator.h |2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/p

[powerdebug 19/35] Remove duplicate findparent variable

If we allocate a clock name, that means we can just check with the clock name pointer, no need of the findparent boolean. Signed-off-by: Daniel Lezcano --- powerdebug.c | 39 +-- 1 files changed, 25 insertions(+), 14 deletions(-) diff --git a/powerdebug.c b

[powerdebug 17/35] Reduce the usage of the dump variable

When we reach the mainloop, that means we are not in dump-mode, so we can remove the test with the dump mode. In the other side, let's use the dump function when we are in dump-mode. Signed-off-by: Daniel Lezcano --- powerdebug.c | 57 -

[powerdebug 04/35] Pass regulators_info parameter to regulator_print_info

The objective is to make the regulators_info variable local to the main function. This first patch pass the regulators_info variable to the function instead of having this one relying on a global variable definition. Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- regulator.c | 52 +

[powerdebug 07/35] pass regulators_info parameter to regulator_read_info

The regulator_read_info function does no longer rely on the regulators_info global variable. Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- regulator.c | 13 ++--- regulator.h |2 +- 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/powerdebug.c b/powerdeb

[powerdebug 10/35] pass numregulators parameter to regulator_print_info

The regulator_print_info does no longer rely on the numregulators global variable. Signed-off-by: Daniel Lezcano --- powerdebug.c |3 ++- regulator.c |6 +++--- regulator.h |3 ++- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 7d91

[powerdebug 11/35] Pass numregulators paramter to regulator_read_info

The regulator_read_info does no longer depend no the global variable numregulators. Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- regulator.c |4 ++-- regulator.h |2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index d310d5c

[powerdebug 09/35] make regulators_info static in powerdebug.c

As no function rely on the global variable regulators_info, let's make it static and reduce the scope to powerdebug.c Signed-off-by: Daniel Lezcano --- powerdebug.c |2 ++ powerdebug.h |4 ++-- regulator.h |2 +- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/powerde

[powerdebug 14/35] make regulators_info and numregulators local to main

Kill the global variables and make them local to the main function. Signed-off-by: Daniel Lezcano --- powerdebug.c | 16 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 6dd4b8c..0b26678 100644 --- a/powerdebug.c +++ b/powerdebug

[powerdebug 15/35] Make highlighted_row variable static

Signed-off-by: Daniel Lezcano --- powerdebug.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 0b26678..7eb9c5e 100644 --- a/powerdebug.c +++ b/powerdebug.c @@ -18,7 +18,7 @@ #include "regulator.h" #include "powerdebug.h" -int highli

[powerdebug 13/35] make numregulators static in powerdebug.c

Reduce the scope of the numregulators to powerdebug.c Signed-off-by: Daniel Lezcano --- powerdebug.c |1 + powerdebug.h |2 -- regulator.h |2 -- 3 files changed, 1 insertions(+), 4 deletions(-) diff --git a/powerdebug.c b/powerdebug.c index 6e9e896..6dd4b8c 100644 --- a/powerdebug

[powerdebug 12/35] Pass numregulators paramter to show_regulator_info

The show_regulator_info function does no longer depend no the global variable numregulators. Signed-off-by: Daniel Lezcano --- display.c|4 ++-- powerdebug.c |3 ++- powerdebug.h |3 ++- 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/display.c b/display.c index b4

Re: Device Tree on ARM status report - Mar 20

On Fri, 2011-03-25 at 22:46 -0600, Grant Likely wrote: [...] > It appears that when U-Boot > relocates the .dtb, it either moves it to a location that the kernel > cannot read during early boot, or it corrupts it when it is moved. > Either way, the kernel is hooped when it tries to parse the device

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

On Wed, Mar 16, 2011 at 3:14 AM, 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 > management i

Re: Device Tree on ARM status report - Mar 20

For those of you working on bringing up arm device tree support against the Linaro u-boot * kernel trees, I've just pushed out a couple of branches that work on Panda board. It's not complete yet because there are some bugs I found in both the kernel and in u-boot, but it should be useful to base

Re: ELC and memory management

Andy, good idea. Jesse will be there, so would be good to agree the problem and start thinking of some directions... Dave Sent from yet another ARM powered mobile device On 25 Mar 2011, at 21:08, "Gross, Andy" wrote: > All, > > Are there plans of having a meeting to discuss memory manag