From bb7922f7f22bd99a61bec846efca9dab25fcf666 Mon Sep 17 00:00:00 2001 From: Kyle Repinski Date: Tue, 20 Oct 2015 12:07:04 -0500 Subject: liblight: Build with -Wall -Werror. Change-Id: I576d40023e7d1981a2da6ce5d6dd999067418c89 --- liblight/Android.mk | 2 ++ liblight/lights.c | 9 +++++---- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/liblight/Android.mk b/liblight/Android.mk index 159d91f..34465e0 100755 --- a/liblight/Android.mk +++ b/liblight/Android.mk @@ -30,4 +30,6 @@ LOCAL_C_INCLUDES := $(LOCAL_PATH)/../kernel-headers LOCAL_MODULE_TAGS := optional +LOCAL_CFLAGS := -Wall -Werror + include $(BUILD_SHARED_LIBRARY) diff --git a/liblight/lights.c b/liblight/lights.c index e155d70..943fc2f 100644 --- a/liblight/lights.c +++ b/liblight/lights.c @@ -16,6 +16,7 @@ #define LOG_TAG "lights" #include +#include #include #include #include @@ -91,7 +92,7 @@ static int rgb_to_brightness(struct light_state_t const *state) + (150*((color>>8) & 0x00ff)) + (29*(color & 0x00ff))) >> 8; } -static int set_light_backlight(struct light_device_t *dev, +static int set_light_backlight(struct light_device_t *dev __unused, struct light_state_t const *state) { int err = 0; @@ -207,13 +208,13 @@ static int set_light_leds(struct light_state_t const *state, int type) return write_leds_priority(); } -static int set_light_leds_notifications(struct light_device_t *dev, +static int set_light_leds_notifications(struct light_device_t *dev __unused, struct light_state_t const *state) { return set_light_leds(state, LED_TYPE_NOTIFICATION); } -static int set_light_leds_attention(struct light_device_t *dev, +static int set_light_leds_attention(struct light_device_t *dev __unused, struct light_state_t const *state) { struct light_state_t attention_state = *state; @@ -224,7 +225,7 @@ static int set_light_leds_attention(struct light_device_t *dev, return set_light_leds(&attention_state, LED_TYPE_ATTENTION); } -static int set_light_leds_battery(struct light_device_t *dev, +static int set_light_leds_battery(struct light_device_t *dev __unused, struct light_state_t const *state) { return set_light_leds(state, LED_TYPE_CHARGING); -- cgit v1.1