summaryrefslogtreecommitdiffstats
path: root/camera
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | CameraHAL: Fixed CPCAM regressionDaniel Levin2012-07-251-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Patch which added regression: http://review.omapzoom.org/#/c/23319/ Change-Id: If79a40e62548f535263995b62b296a8f561491b0 Signed-off-by: Daniel Levin <dendy@ti.com>
| * | | | | | | | CameraHAL: Limit the sensor/caps detection in the absence of primary cameraSaravanan Solaiyappan2012-07-251-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the absence of primary sensor OR some error in the primary sensor simply exit from the sensor detection of other sensor ids. Because with the current implementation, if the primary sensor is not present application can not switch to other sensors. Change-Id: I9e00f91d3fb34af6c70d4b9146014926218ae2c0 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHAL: Wrapped sources with OMAP_ENHANCEMENT_* macrosDaniel Levin2012-07-2511-68/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Wrapped the enhanced code with OMAP_ENHANCEMENT_* macros. Currently next enhancing modules dedicated: - CPCAM: OMAP_ENHANCEMENT_CPCAM - VTC: OMAP_ENHANCEMENT_VTC - S3D: OMAP_ENHANCEMENT_S3D - Burst capture: OMAP_ENHANCEMENT Disabled the VTC test. Disabled the camera_test. Must be used together with: http://review.omapzoom.org/#/c/23318/ Change-Id: Ie23aa6b7e0a72dabe529f2828442afa0464d4af4 Signed-off-by: Daniel Levin <dendy@ti.com>
| * | | | | | | | Camera HAL: Second stereo camera addedPavel Nedev2012-07-254-46/+103
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supported number of cameras is increased to 4 accordingly. The way number of cameras is detected from the OMX camera adapter is changed as not every camera has its own camera index. Stereo cameras are distinguished by the stereo operating mode but have the same indexes as the mono cameras. On OMX init time when capabilities are gotten a container is filled that maps each Android camera index to the corresponding OMX camera settings - OMX cam index and operating mode. OMX_TI_StereoSensor index is left for backward compatibility with older Ducati images. CameraService max number of cameras must also be increased to 4. There is another framework/base patch that does just that. Change-Id: I9ea9fb8454d09380612c19f6240457ab36c773e9 Signed-off-by: Pavel Nedev <pnedev@mm-sol.com>
| * | | | | | | | cameraHAL: V4LCameraAdapter: few bug fixes.Saravanan Solaiyappan2012-07-252-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds few more properties in to capabilities. Eventhough these properties are unsupported by USB camera, it fixes some issues with the usb camera usecases. Change-Id: I01753e5c781472bd45f1f2669db3dc46c4e291ee Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHAL: Fix for RawCapture & YuvCapture failure.Saravanan Solaiyappan2012-07-252-6/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the issues with UseBuffer failure for RawCapture. This is due to the CPCAM change with the buffer handling. pBufferHdr-pBuffer will no more hold the actual raw buffer. pBufferHdr-pAppPrivate should be used to get the CameraBuffer address. This patch also fixes the yuvCapture (/data/misc/camera/YuV_PiCtUrEs/). Change-Id: Ic4d9a25221d1916b33cd643ed810fa2a863d8cf9 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHal: Fixes for temporal bracketingEmilian Peev2012-07-254-33/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Recent modifications in the way burst capture operates require coressponding changes for temporal bracketing. - Fixes a deadlock which arises during 'stopBracketing()' if 'stopImageCapture()' is called from a context holding 'mBracketingLock'. The deadlock arises when waiting for flush buffers condition and incoming buffers try to acquire the same 'mBracketingLock'. - Fixes an issue with CameraHal overriding the client set capture mode after a stopPreview(), startPreview() sequence. Change-Id: I245b11c102b24c22a9f760067bcdba507d539f27 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: Optimization in color conversion.Saravanan Solaiyappan2012-07-251-31/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch optimize the color convertion by copying the NV12 converted frame directly in preview buffer (Tiler memory). This avoids one additional memory copy in the existing implementation. This patch also include the PPM calculation for the color conversion per frame (Average of 30 frames) and placed under PPM_PER_FRAME_CONVERSION macro. Change-Id: I5332050047c2b611d1093322c0a18f1b4e057d17 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | Revert "[TEMP] CameraHAL:OMAP5: WA for sensor detection."Saravanan Solaiyappan2012-07-252-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 2cd10f50ec3e470362a828b80d756e9983f7cd68. Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com> Change-Id: I8c41a03efabf9c06e16c62007e673d9728fde3d3
| * | | | | | | | Camera: Use Tiler1D heap if ION 1D allocation failsDevaraj Rangasamy2012-07-251-25/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default, Camera today uses ION 1D heap for buffer allocation. In 512MB system configuration, ION 1D heap might not be available, and hence use Tiler 1D heap as fallback mechanism. This patch is applicable for both 1GB and 512MB systems Change-Id: I59dddfa8e6567d0ec8881fdff6bfe41047deb3b5 Signed-off-by: Devaraj Rangasamy <dev@ti.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: catch v4lInitMmap errorSergio Aguirre2012-07-251-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I6c7d57f816a03336157b6e3b4ea5f246ec1671dd Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: Catch getCaps error properlySergio Aguirre2012-07-251-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Iada3006b091a297505903063319077f0f5b9662e Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
| * | | | | | | | CameraHAL: Fixed memory leak because of missing ion_close()Tyler Luu2012-07-256-20/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch prevents ION device from infinite openings without close. Added missing MemoryManager::initialize() calls through the code. Change-Id: I053bfd2076d355439c5cb8adc4a041a6650eac5c Signed-off-by: Anatolii.Shuba <x0158321@ti.com> Signed-off-by: Daniel Levin <dendy@ti.com>
| * | | | | | | | CameraHal: Queries picture sizes dynamicallyEmilian Peev2012-07-259-107/+236
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Currently the picture sizes are static and will not change with the capture mode. For some use-cases this is not sufficient. To overcome this the Properties class is extended to support an operating mode, which will switch the supported properties depending on the current setting. Additionally the Ducati capabilities are queried not only for one capture mode alone but for all use-cases in 2D and 3D. As a result once the camera client switches the capture mode, the supported picture sizes will change inside CameraParameters retrieved through a subsequent "getParameters()" call. Related UI patch: http://review.omapzoom.org/#/c/21685 Signed-off-by: Emilian Peev <epeev@mm-sol.com> Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: Ib9f02ede94ff76feb86b3422dc453dfd8782ee43
| * | | | | | | | CameraHAL: Recording HINT param handling optimizationPavel Nedev2012-07-252-143/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimizations regarding recording HINT setting - preview size and framerate dependency fixes, required restart decision fixes. Change-Id: I10abd1af31d812002d4dc6ac76c1221390ec481c Signed-off-by: Pavel Nedev <pnedev@mm-sol.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: Take queueable buffer count properlySergio Aguirre2012-07-252-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ibae272f0a8580688e84f25ea7f87bc8f8c704726 Signed-off-by: Sergio Aguirre <saaguirre@ti.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: Enable Video record.Saravanan Solaiyappan2012-07-253-70/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Comment] This patch enables the basic video preview and record with USB camera. [DR] OMAPS00267530 Change-Id: I1f0e7eebe91515384ba02b3de1ed51cbb9bed818 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | V4LCameraAdapter: Neon conversion from yuv422i to NV12.Saravanan Solaiyappan2012-07-251-19/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch converts the prview frames from yuv422i to NV12 using neon instructions. Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com> Change-Id: Id30e27d860057063a12aa96187f6da297903660a
| * | | | | | | | CameraHal: V4LCapabilities: Fixes for OMAPCamera to start previewDimitar Borisov2012-07-251-5/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added some of the not supported modes in capabilities. - Some minor issues fixes. Change-Id: I44fa53ec2bbb35ea688006b940d660989aeb29a6 Signed-off-by: Dimitar Borisov <x0152684@ti.com>
| * | | | | | | | CameraHAL: V4LCameraAdapter: Enable Image capture.Saravanan Solaiyappan2012-07-254-34/+387
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Comment] This patch enables the basic Image capture with USB camera. [DR] OMAPS00267530 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com> Change-Id: Ib8119a4514d3eb102cf4ee4d147c790ccc1bdd76
| * | | | | | | | CameraHAL: V4LCameraAdapter: Enable 30fps preview alwaysSaravanan Solaiyappan2012-07-253-166/+318
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Comment] This patch contains the following changes: 1. Finds the best preview resolution which supports 30fps and populates that resolution through capabilities. 2. Enables printing the FPS value when the below property is set. setprop debug.camera.showfps 1 3. Adds some wrapper routines for v4l ioctl. Change-Id: I7971ba42f83087bd85bd955374e568735d735426 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHAL: Extends pixel format YUV422i-uyvy for video snapshot.Saravanan Solaiyappan2012-07-256-9/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Comment] Ducati camera always gives CbYCrY(uyvy) pixel format for YUV422I. But CameraParameters class only supports YUV422-yuyv (YCbYCr). This patch adds the YUV422-uyvy pixel format and updates the jpeg encoder for video snapshot capture. This also adds the support for jpeg encoding from YUV422i-yuyv pixel format. This will be used in V4L/USB camera capture. Change-Id: I564bd00d1a12efba1490190926efa30441298181 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | [TEMP] CameraHAL:OMAP5: WA for sensor detection.Saravanan Solaiyappan2012-07-252-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch limits the MAX_CAMERAS_SUPPORTED to 2 to avoid querying S3D capabilities. This is done due to some issues in ducati while querying S3D capabilities. This is noticed with kernel3.4. Change-Id: I319c42992e674fcbe954d89a77346f19da3feb5d Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHal: WA for VectShotInfo enableTyler Luu2012-07-252-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, Ducati has following issue: If DOMX client disables VectShotInfo metadata on the Image port, this cause VectShotInfo to be disabled internally on preview port also. This is simple work-aroud to handle that. Change-Id: I95e385cab4475dc0e43c531f8f9770f4556770dc Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | camera: add exp/gain preview metadataTyler Luu2012-07-259-143/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add ability to send current preview exposure time and analog gain to application. Preview metadata implementation is extended to support photometry data along with face detection data. Depends on Change-Id Ia4fcfbeb: camera: Add Exp./Gain preview metadata Change-Id: I3a4c7548d0e98f3a8c29e305fd746f0241cb45e4 Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | Revert "CameraHAL: Adds support for controlling the FD update interval"Vladimir Petrov2012-07-252-30/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3a658e8e38839397ce018d5ae04ede2c1ca5dd89. Just reverting change Ie93c6ae5: CameraHAL: Adds support for controlling the FD update interval Because it conflicts with next change I3a4c754: CameraHal: Add Exp./Gain preview metadata Change-Id: I3e5eec25985197213ef089efe4181b646d2662a3 Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | camera: cleanups regarding extra data and setMetaData()Vladimir Petrov2012-07-255-110/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid code duplication in OMXCameraAdapter::sniffDccFileDataSave() and OMXCameraAdapter::detectFaces() Checking of OMX_TI_PLATFORMPRIVATE was moved inside OMXCameraAdapter::setMetaData() Change-Id: I0ab560addffec066316b4f36063119005ea2042d Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | camera: add lsc, exposure, gain, and frame # metadataTyler Luu2012-07-252-2/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | adding support for new metadata keys Depends on: Change-Id: I1cf4bdb5: CameraMetadata: Add LSC and exposure and gai... Change-Id: Id60213837a187fe4e68712e90d37662f87baa510 Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | CameraHAL: Reworked frame rate collectionDaniel Levin2012-07-253-197/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | United encodeFramerateCap() and encodeVFramerateCap() into single encodeFrameRates() method that populates frame rates. Returned list contains both variable frame rates from Ducati and fixed frame rates from predefined values, clipped by min/max range. Min/max values also added to the list as fixed frame rates. List is ordered according to Android requirements. List does not contain duplicated values. Caller uses generated frame rate list to populate separately variable and fixed frame rate parameters. Must be used together with: http://review.omapzoom.org/#/c/22250/ Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: I07df14da648da46d7bef1d648fcd75649ebd4e54
| * | | | | | | | CameraHAL: Optimized image coding format logicPavel Nedev2012-07-251-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The image coding format logic in OMXCameraAdapter::setFormat() optimized. Change-Id: I92c8178d1ad4701fa854b4298147463ffaf4a1f8 Signed-off-by: Pavel Nedev <pnedev@mm-sol.com>
| * | | | | | | | CameraHAL: Removed misleading bufferSize assignmentPavel Nedev2012-07-251-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OMX camera image port bufferSize assignment is removed from setFormat - this size comes from Ducati. Change-Id: Id48628f978e14ccf9ccbeaaf78b4a03f87d37403 Signed-off-by: Pavel Nedev <pnedev@mm-sol.com>
| * | | | | | | | CameraHal: Handles preview and snapshot buffers separatelyEmilian Peev2012-07-255-7/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Hal should differentiate between preview and snapshot frames. The camera client can expect postview callbacks even if the preview frame notifications are internally disabled during image capture. Change-Id: I5ab7ecf51860989bf983e4dc116f6816ee57e8f7 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | camera: change defaults for some processing defaultsVladimir Petrov2012-07-252-6/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change defaults for additional control of fixed gamma, NSF1, NSF2, sharpening, three color line map, and GIC. All are set to ENABLED, and also fixed missing initialization. Change-Id: Ic6543be456bae3e844629a0e56fa6da705b0e78f Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | camera: absolute forced exp/gain and EV compensationVladimir Petrov2012-07-253-27/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add absolute forced Exp./Gain shot config. Application can set a "KEY_EXP_GAIN_PAIRS" keyed parameter in format (exposure,gain): (12500,50)F,(25000,50)F,(12500,300)F,(25000,300)F Exposure time should be in microseconds and analog gain should be in EV units x100. Added total EV compensation shot config. Application can set a "KEY_EXP_COMPENSATION" keyed parameter in format: -200,-100,0,+100,+200,+100,0,-100,-200 Added forced total EV compensation shot config. Application can set a "KEY_EXP_COMPENSATION" keyed parameter in format: -200F,-100F,+0F,+100F,+200F,+100F,+0F,-100F,-200F Note: A "KEY_EXP_COMPENSATION" keyed parameter should not be used along with a "KEY_EXP_GAIN_PAIRS" keyed parameter. If application needs to configure a sequence with some frames in total exposure mode and other frames in "exposure,gain" mode, it should be done with sequence of separate "takePicture" calls. Note: Application should set a "KEY_BURST" keyed shot parameter accordingly. Change-Id: Ic23a56ae421bfb45c22fc819e911ef1094f70fb8 Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | CameraHal: Proper buffer tracking with Shot config flushVladimir Petrov2012-07-251-3/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reset buffer tracking if application sends flush shot config queue shot parameter. Avoid queuing too many capture buffers to DOMX Camera. Change-Id: I0a6c1a38ea8ab75c1176d84fc4e3ccdb36521d8c Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | camera: add shot config flushTyler Luu2012-07-256-3/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a mechanism to configure shot config flushing. Also ensure that shot config flush flag can be sent to Ducati only once on first iteration. Requires: Change I11e6a7a7: ShotParameters: Adding flush config and EV compe... Change-Id: Ic06c28e7634b3e8fc3a474869504e47a6ad56217 Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
| * | | | | | | | CameraHal: Add support for BAYER reprocessingTyler Luu2012-07-256-9/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Add support for allocating bitstream buffers for bayer format through buffersourceadapter 2. Add support for setting up reprocessing port (video-in) in OMXCameraAdapter for bayer format Change-Id: I0d9c1d6ec4ca09b4f9d19b36321abef0495d7405 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHal: Do full AF sweep during lost AF statusEmilian Peev2012-07-252-14/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Previously only "OMX_FocusStatusUnableToReach" status was activating the full AF scan. Currently with CAF callbacks updates in Ducati, this is no longer the case and "OMX_FocusStatusLost" should be considered as well. Change-Id: Ib296c6d7197cd275a9e5106f791451f4179be3e9 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHal: Add support for YUY2 preview callbacksEmilian Peev2012-07-252-24/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - The currently used Gralloc buffers for preview are limited to only NV12 pixelformat. In order to support this type of preview callbacks, CameraHal will internally convert from NV12 to YUY2. Change-Id: I879e32d4b2142bdd0a1f89fe7c5496d250039b73 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHAL: Aligns UV planes during YV12 preview callbacksEmilian Peev2012-07-251-5/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - UV planes stride should be multiple of 16. The buffer size should also be updated accordingly. Change-Id: If436801fbb2750523a3207878f611c8094667367 Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHAL: Simplified mechanical misalignment parametersDaniel Levin2012-07-259-48/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Turned mechanical misalignment parameters from list to boolean parameter: "mechanical-misalignment-correction-supported" Replaced custom "enable" and "disable" tokens with standard Android "true" and "false". Must be used together with: http://review.omapzoom.org/#/c/21700/ Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: I45dbe39eb03d3642e127175b2ebc6021fb38e9a8
| * | | | | | | | CameraHAL: Fix for disable port time out.Saravanan Solaiyappan2012-07-252-3/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [Comment] There was a port disable timeout in switchToExecuting() as the port is not enabled state. So while switching the preview from image preview to video preview with 60fps other than 1080p resolution, >5sec delay is noticed because of this port disable time out issue. This issue is noticed after the below change. http://review.omapzoom.org/#/c/22417/ This patch enables the preview port during switchToLoaded if the sensor overclock is set. So that portDisable will not time out in doSwitchToExecuting(). Change-Id: I1501520337f9c41d68761ddce9abd81328fb7443 Signed-off-by: Solaiyappan Saravanan <saravanan.s@ti.com>
| * | | | | | | | CameraHal: Changes necessitated by slice based processingPrasanna Kumar M.R2012-07-258-190/+461
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch reverts the revert commit http://review.omapzoom.org/#/c/22000/ and enables the Slice mode in camera hal as in original patch http://review.omapzoom.org/#/c/19749/ Couple of changes are present in addition to original patch: The initial revert was to due to 60fps issue on OMAP5. The issue was debugged and confirmed to be caused by combination of existing code for handling state change when sensor overclocking is enabled in combination with port enable/disable changes of this patch. Mutex handling has been updated to improve the scenario (another patch expected from camera team to manage port status) Another change has also been incorporated in this patch to address the logging related compilation issue when debug flags are enabled in camera hal. Change-Id: I5b78d2bdc3f82133be8447176bebde7f4673441b Signed-off-by: Prasanna Kumar M.R. <prasanna.kumarmr@ti.com> Signed-off-by: Anu Sundararajan <sanuradha@ti.com>
| * | | | | | | | CameraHAL: Renamed scene mode "sport" to "sports"Daniel Levin2012-07-253-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Android has standard "sports" scene mode since API Level 5, which was not used in HAL. This patch removes custom "sport" scene mode, replacing it with standard "sports" token. Depends on UI patch: http://review.omapzoom.org/#/c/22365/ Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: I3228bcec480a2b8b36130d35d51c3ba5cede2b95
| * | | | | | | | CameraHAL: Prevent possible deadlock scenarioTyler Luu2012-07-251-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hold scope of returnFrame lock for just the calculation of refCount. Otherwise, we could have a scenario where endImageCapture and stopRecording could cause a deadlock competing for mReturnFrameLock and mLock. Issue arises mainly because endImageCapture is signalled from fillThisBuffer which stopRecording will need to get into. Change-Id: I3a125711632fa311fd794e300cd5008321a84c40 Signed-off-by: Tyler Luu <tluu@ti.com> Signed-off-by: Emilian Peev <epeev@mm-sol.com> Signed-off-by: Daniel Levin <dendy@ti.com>
| * | | | | | | | CameraHal: Speedup face detection startEmilian Peev2012-07-253-10/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Currently when face detection gets enabled the face priority for AE and AF is getting configured as well. This configuration takes around 18 ms. on average and can delay the overall startup time. The optimization will move the algo setup away from face detection enablement and in to the FillBufferDone callback. Change-Id: I2b0e8df9fdead7a5d761a1efbcbf57d39e27ae3c Signed-off-by: Emilian Peev <epeev@mm-sol.com>
| * | | | | | | | CameraHAL: Fixed portrait preview resolutionsDaniel Levin2012-07-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: I917f84b863738073567b501bd9dac7b7d153b6ad
| * | | | | | | | CameraHAL: Unused code cleanupDaniel Levin2012-07-252-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed: - DEFAULT_MAX_FOCUS_AREAS - DEFAULT_MAX_NUM_METERING_AREAS - DEFAULT_FACING_SUPPORTED - DEFAULT_FRAMERATE_RANGE_IMAGE - DEFAULT_FRAMERATE_RANGE_VIDEO - CapEVComp typedef Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: Ie469c592a5e43bbe6eff8f1cb48c84b84d3aec24
| * | | | | | | | CameraHAL: Removed preferred video sizeDaniel Levin2012-07-255-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed token are not used anymore. Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: Id36633758bdbd978bd2d34ed683264ed4da09c7b
| * | | | | | | | CameraHAL: Added checking for sensor orientationDaniel Levin2012-07-253-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sensor orientation must be one of 0,90,180,270, otherwise error is returned from setParameters(). Also removed unused KEY_SENSOR_ORIENTATION_VALUES token. Also removed duplicated insertion of KEY_SENSOR_ORIENTATION. Signed-off-by: Daniel Levin <dendy@ti.com> Change-Id: I2589bdf0225c9de8d568c288a9dde95eaa9007c2