power: Compile with -Wall -Wextra -Werror

* And fix the build errors

Change-Id: I183203edfd92d4672893d74aa7428bc9f5e17772
This commit is contained in:
Michael Bestas
2018-03-26 01:00:20 +03:00
parent 0c54d07956
commit 39e18570bf
5 changed files with 8 additions and 13 deletions

View File

@@ -14,6 +14,8 @@ LOCAL_HEADER_LIBRARIES += libutils_headers
LOCAL_HEADER_LIBRARIES += libhardware_headers
LOCAL_SRC_FILES := power-common.c metadata-parser.c utils.c list.c hint-data.c Power.cpp main.cpp PowerHintSession.cpp
LOCAL_CFLAGS += -Wall -Wextra -Werror
ifneq ($(BOARD_POWER_CUSTOM_BOARD_LIB),)
LOCAL_WHOLE_STATIC_LIBRARIES += $(BOARD_POWER_CUSTOM_BOARD_LIB)
else

View File

@@ -53,10 +53,8 @@
static int display_hint_sent;
static int video_encode_hint_sent;
static int cam_preview_hint_sent;
static void process_video_encode_hint(void *metadata);
//static void process_cam_preview_hint(void *metadata);
/* Returns true is target is SDM630/SDM455 else false*/
static bool is_target_SDM630()
@@ -91,6 +89,8 @@ int power_hint_override(power_hint_t hint, void *data)
process_video_encode_hint(data);
return HINT_HANDLED;
}
default:
break;
}
return HINT_NONE;
}
@@ -98,11 +98,8 @@ int power_hint_override(power_hint_t hint, void *data)
int set_interactive_override(int on)
{
char governor[80];
char tmp_str[NODE_MAX];
int resource_values[20];
int num_resources;
struct video_encode_metadata_t video_encode_metadata;
int rc;
ALOGI("Got set_interactive hint");

View File

@@ -87,6 +87,8 @@ int power_hint_override(power_hint_t hint, void *data)
process_video_encode_hint(data);
return HINT_HANDLED;
}
default:
break;
}
return HINT_NONE;
}
@@ -94,8 +96,6 @@ int power_hint_override(power_hint_t hint, void *data)
int set_interactive_override(int on)
{
char governor[80];
char tmp_str[NODE_MAX];
struct video_encode_metadata_t video_encode_metadata;
static int set_i_count = 0;
ALOGI("Got set_interactive hint");

View File

@@ -53,10 +53,8 @@
static int display_hint_sent;
static int video_encode_hint_sent;
static int cam_preview_hint_sent;
static void process_video_encode_hint(void *metadata);
//static void process_cam_preview_hint(void *metadata);
static bool is_target_SDM632() /* Returns value=632 if target is SDM632 else value 0 */
{
@@ -90,6 +88,8 @@ int power_hint_override(power_hint_t hint, void *data)
process_video_encode_hint(data);
return HINT_HANDLED;
}
default:
break;
}
return HINT_NONE;
}
@@ -97,8 +97,6 @@ int power_hint_override(power_hint_t hint, void *data)
int set_interactive_override(int on)
{
char governor[80];
char tmp_str[NODE_MAX];
struct video_encode_metadata_t video_encode_metadata;
static int set_i_count = 0;
ALOGI("Got set_interactive hint");

View File

@@ -33,8 +33,6 @@
extern "C" {
#endif
#define NODE_MAX (64)
#define SCALING_GOVERNOR_PATH "/sys/devices/system/cpu/cpu0/cpufreq/scaling_governor"
#define INTERACTIVE_GOVERNOR "interactive"
#define SCHEDUTIL_GOVERNOR "schedutil"