omap4xxx: camera: timing issue with autoFocus/cancelAutoFocus

Fixes b/5332895

If cancelAutoFocus comes when Ducati is already sending focus
callback, CameraHal will go ahead and send focus callback to
the application. This can cause the subsequent autoFocus call
to be acked almost immediately with the previous autoFocus call's
notification. Application will think autoFocus is done while
CamearHal is still focusing, takePicture from the application
will return an error.

To fix this issue, do not always send focus notification. We are
going to internally enable/disable auto focus message. Enable focus
message when autoFocus call comes and disable focus message after
notification or if cancelAutoFocus call comes. This will prevent us
from sending notify message if application calls cancelAutoFocus.

Change-Id: I8c8892f14894e23a9d73fa1c2b058389d196eefa
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Iliyan Malchev <malchev@google.com>

1 files changed, 2 insertions, 0 deletions

diff --git a/camera/AppCallbackNotifier.cpp b/camera/AppCallbackNotifier.cpp
index a69cfc0..cc53fb4 100644
--- a/camera/AppCallbackNotifier.cpp
+++ b/camera/AppCallbackNotifier.cpp
@@ -352,6 +352,7 @@ void AppCallbackNotifier::notifyEvent()
                           ( mCameraHal->msgTypeEnabled(CAMERA_MSG_FOCUS) ) )
                         {
                          mNotifyCb(CAMERA_MSG_FOCUS, true, 0, mCallbackCookie);
+                         mCameraHal->disableMsgType(CAMERA_MSG_FOCUS);
                         }
                     else if ( focusEvtData->focusError &&
                                 ( NULL != mCameraHal ) &&
@@ -359,6 +360,7 @@ void AppCallbackNotifier::notifyEvent()
                                 ( mCameraHal->msgTypeEnabled(CAMERA_MSG_FOCUS) ) )
                         {
                          mNotifyCb(CAMERA_MSG_FOCUS, false, 0, mCallbackCookie);
+                         mCameraHal->disableMsgType(CAMERA_MSG_FOCUS);
                         }
 
                     break;