commit b4eec7eb5e0408ecc0e450025dbb5ab3fe6cbc5c
author Daniel Lezcano <daniel.lezcano@linaro.org> Thu Feb 18 16:44:55 2016 +0000
committer Daniel Lezcano <daniel.lezcano@linaro.org> Thu Feb 18 16:47:34 2016 +0000
tree ef61a1289d04949023979220791302ec0655bdc4
parent 30f014726a0614f0e331c67604729723306007e1

Pass options to init functions.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

clocks.c

diff --git a/clocks.c b/clocks.c
index c751ec1..b57de3d 100644
--- a/clocks.c
+++ b/clocks.c
@@ -447,10 +447,13 @@
 /*
  * Initialize the clock framework
  */
-int clock_init(void)
+int clock_init(struct powerdebug_options *options)
 {
 	char clk_dir_path[MAX+1][PATH_MAX];
 
+	if (!(options->flags & CLOCK_OPTION))
+		return 0;
+
 	if (locate_debugfs(clk_dir_path[CCF]) || locate_debugfs(clk_dir_path[OCF]))
 		return -1;
 

gpio.c

diff --git a/gpio.c b/gpio.c
index 8fd3e5c..36dedd1 100644
--- a/gpio.c
+++ b/gpio.c
@@ -375,10 +375,13 @@
 /*
  * Initialize the gpio framework
  */
-int gpio_init(void)
+int gpio_init(struct powerdebug_options *options)
 {
 	int ret = 0;
 
+	if (!(options->flags & GPIO_OPTION))
+		return 0;
+
 	ret = display_register(GPIO, &gpio_ops);
 	if (ret)
 		printf("error: gpio display register failed");

powerdebug.c

diff --git a/powerdebug.c b/powerdebug.c
index 0f5c98a..b626b05 100644
--- a/powerdebug.c
+++ b/powerdebug.c
@@ -33,17 +33,6 @@
 
 extern void sigwinch_handler(int);
 
-#define REGULATOR_OPTION	0x01
-#define SENSOR_OPTION		0x02
-#define CLOCK_OPTION		0x04
-#define GPIO_OPTION		0x08
-
-#define VERBOSE_OPTION		0x10
-#define DUMP_OPTION		0x20
-
-#define DEFAULT_OPTION (REGULATOR_OPTION | SENSOR_OPTION | \
-			CLOCK_OPTION | GPIO_OPTION)
-
 void usage(void)
 {
 	printf("Usage: powerdebug [OPTIONS]\n");
@@ -243,22 +232,22 @@
 		return 1;
 	}
 
-	if ((options->flags & REGULATOR_OPTION) && regulator_init()) {
+	if (regulator_init(options)) {
 		printf("failed to initialize regulator\n");
 		options->flags &= ~REGULATOR_OPTION;
 	}
 
-	if ((options->flags & CLOCK_OPTION) && clock_init()) {
+	if (clock_init(options)) {
 		printf("failed to initialize clock details (check debugfs)\n");
 		options->flags &= ~CLOCK_OPTION;
 	}
 
-	if ((options->flags & SENSOR_OPTION) && sensor_init()) {
+	if (sensor_init(options)) {
 		printf("failed to initialize sensors\n");
 		options->flags &= SENSOR_OPTION;
 	}
 
-	if ((options->flags & GPIO_OPTION) && gpio_init()) {
+	if (gpio_init(options)) {
 		printf("failed to initialize gpios\n");
 		options->flags &= GPIO_OPTION;
 	}

powerdebug.h

diff --git a/powerdebug.h b/powerdebug.h
index 33463a0..2df0a27 100644
--- a/powerdebug.h
+++ b/powerdebug.h
@@ -21,6 +21,17 @@
 
 #define VERSION "0.7.3"
 
+#define REGULATOR_OPTION	0x01
+#define SENSOR_OPTION		0x02
+#define CLOCK_OPTION		0x04
+#define GPIO_OPTION		0x08
+
+#define VERBOSE_OPTION		0x10
+#define DUMP_OPTION		0x20
+
+#define DEFAULT_OPTION (REGULATOR_OPTION | SENSOR_OPTION | \
+			CLOCK_OPTION | GPIO_OPTION)
+
 struct powerdebug_options {
 	int flags;
 	unsigned int ticktime;
@@ -28,14 +39,15 @@
 	char *clkname;
 };
 
-extern int clock_init(void);
+extern int clock_init(struct powerdebug_options *options);
 extern int clock_dump(char *clk);
 
-extern int regulator_init(void);
+extern int regulator_init(struct powerdebug_options *options);
 extern int regulator_dump(void);
 
-extern int gpio_init(void);
+extern int gpio_init(struct powerdebug_options *options);
 extern int gpio_dump(void);
 
+extern int sensor_init(struct powerdebug_options *options);
 extern int sensor_dump(void);
-extern int sensor_init(void);
+

regulator.c

diff --git a/regulator.c b/regulator.c
index b38ddab..549a744 100644
--- a/regulator.c
+++ b/regulator.c
@@ -265,10 +265,13 @@
 	.display = regulator_display,
 };
 
-int regulator_init(void)
+int regulator_init(struct powerdebug_options *options)
 {
 	int ret = 0;
 
+	if (!(options->flags & REGULATOR_OPTION))
+		return 0;
+
 	ret = display_register(REGULATOR, &regulator_ops);
 	if (ret)
 		printf("error: regulator display register failed");

sensor.c

diff --git a/sensor.c b/sensor.c
index 008b662..3b37bc6 100644
--- a/sensor.c
+++ b/sensor.c
@@ -295,10 +295,13 @@
 	.display = sensor_display,
 };
 
-int sensor_init(void)
+int sensor_init(struct powerdebug_options *options)
 {
 	int ret = 0;
 
+	if (!(options->flags & SENSOR_OPTION))
+		return 0;
+
 	ret = display_register(SENSOR, &sensor_ops);
 	if (ret)
 		printf("error: sensor display register failed");