Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

LightService: log errors.

Browse Source 
Test: lights work
Bug: 32022100
Change-Id: I7ac7de63e378ca37665859e4d45b09bf99313d10
This commit is contained in:
Steven Moreland
2016-11-02 14:29:46 -07:00
parent 7fdf38180c
commit da183ac9c3
1 changed files with 28 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
services/core/jni/com_android_server_lights_LightsService.cpp
View File

@@ -34,6 +34,9 @@ using Brightness = ::android::hardware::light::V2_0::Brightness;
using Flash = ::android::hardware::light::V2_0::Flash;
using Type = ::android::hardware::light::V2_0::Type;
using LightState = ::android::hardware::light::V2_0::LightState;
using Status = ::android::hardware::light::V2_0::Status;
template<typename T>
using Return = ::android::hardware::Return<T>;
static sp<ILight> gLight;
@@ -108,9 +111,33 @@ static void setLight_native(
state.brightnessMode = brightness;
Status status;
{
ALOGD_IF_SLOW(50, "Excessive delay setting light");
gLight->setLight(type, state);
Return<Status> ret = gLight->setLight(type, state);
// TODO(b/31348667): this is transport specific status
if (!ret.getStatus().isOk()) {
ALOGE("Failed to issue set light command.");
return;
}
status = static_cast<Status>(ret); // hal status
}
switch (status) {
case Status::SUCCESS:
break;
case Status::LIGHT_NOT_SUPPORTED:
ALOGE("Light requested not availale on this device.");
break;
case Status::BRIGHTNESS_NOT_SUPPORTED:
ALOGE("Brightness parameter not supported on this device.");
break;
case Status::UNKNOWN:
default:
ALOGE("Unknown error setting light.");
}
}