diff options
| author | Eino-Ville Talvala <etalvala@google.com> | 2014-07-23 17:17:59 -0700 |
|---|---|---|
| committer | Eino-Ville Talvala <etalvala@google.com> | 2014-07-28 20:20:36 +0000 |
| commit | f67e23ef637d0b53a0d4bebb68c654234df3da94 (patch) | |
| tree | 3b61f96b8fa4a85b3a1780e03985d15e8e2e3d5c /services/camera/libcameraservice/device2 | |
| parent | ef9c5ade2308f9eb6f222db3c4796e196f0ab76c (diff) | |
| download | frameworks_av-f67e23ef637d0b53a0d4bebb68c654234df3da94.zip frameworks_av-f67e23ef637d0b53a0d4bebb68c654234df3da94.tar.gz frameworks_av-f67e23ef637d0b53a0d4bebb68c654234df3da94.tar.bz2 | |
CameraService: Clean up availability listeners and HAL error codes
- Refactor where availability listeners are called to centralize behavior,
ensuring that all client creation/destruction invokes the listeners
- Clean up some of the client hierarchy
- Filter error codes from key HAL calls to ensure proper reporting
Bug: 16514157
Bug: 16483222
Change-Id: I59875a865b6a508b47423946c78862da8df34cd1
Diffstat (limited to 'services/camera/libcameraservice/device2')
| -rw-r--r-- | services/camera/libcameraservice/device2/Camera2Device.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/services/camera/libcameraservice/device2/Camera2Device.cpp b/services/camera/libcameraservice/device2/Camera2Device.cpp index 89c6b10..c1f77fa 100644 --- a/services/camera/libcameraservice/device2/Camera2Device.cpp +++ b/services/camera/libcameraservice/device2/Camera2Device.cpp @@ -30,6 +30,7 @@ #include <utils/Trace.h> #include <utils/Timers.h> #include "Camera2Device.h" +#include "CameraService.h" namespace android { @@ -67,8 +68,8 @@ status_t Camera2Device::initialize(camera_module_t *module) camera2_device_t *device; - res = module->common.methods->open(&module->common, name, - reinterpret_cast<hw_device_t**>(&device)); + res = CameraService::filterOpenErrorCode(module->common.methods->open( + &module->common, name, reinterpret_cast<hw_device_t**>(&device))); if (res != OK) { ALOGE("%s: Could not open camera %d: %s (%d)", __FUNCTION__, |