| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
These tags are needed by the CM13 camera to not crop images to square
when shooting portrait. As the camera HAL already rotated the picture,
always set orientation to 0.
Change-Id: Id8b6aac5369ae3adfdfec0ec7528edc849bd3b10
|
|
|
|
|
|
| |
Adds support for camera on the Galaxy Tab 2.
Change-Id: I8580dac218593b31e789f372ebb1f294d66cd5c3
|
|
|
|
| |
Change-Id: I5d055f7da992c9c0fd71823c6ca0f18c95fc3be8
|
|
|
|
|
|
|
| |
Missed this bit when migrating to hardware/ti/omap4. Without it,
the front-facing camera can't be used.
Change-Id: I1a47d2991afa71840225f0cab9f27a155eaed780
|
|
|
|
|
|
|
| |
This adds flash-support for motorola devices and disables
unsupported functionality.
Change-Id: Ib1107d78641ca1ded61e8adb88ab7e8951e4ec74
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Change-Id: If580892f35bdfb4ae7071d34b6194fff3773f30a
|
| |
|
|
|
|
|
|
|
|
|
|
| |
- Add MAX_CAMERAS board config to limit scanning of omx cameras
(Jem has 2 extra configs which don't exist)
- Add TREAT_FRONT_AS_BACK board config to pass the front camera
off to the framework as a back camera to fix Panorama modes
- ifdef VT slices code which is only used in CPCAM mode
Change-Id: I273ec01403e216c544722e681c96ae7e1220b8d7
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch fix reference calculation during AV record
usecase. Previously there was some logic that counts
reference depends what type of frame passed. That
leads to race condition in case of video snapshot.
Now all possible types of subscriptions will be
counted for particular frame.
Change-Id: If183fdc425a25424fdb3ca5d31a0030c3669f322
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
|
|
|
|
|
|
|
|
| |
There is no real need to deliver AF callbacks if
autofocus is not triggered from client(i.e. adapter
in non AF state).
Change-Id: Ie724922c705d749820b6984144bd5390dcd3ff03
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the destructor of OMXCameraAdapter, OMX_Deinit() is called after OMX state
is switched to Loaded via switchToLoaded function.
But if some error happens during the state change, OMX_Deinit() is called in
switchToLoaded function.
As the result, OMX_Deinit() is called twice in swtchToLoaded and the destructor.
This causes the ducati crash.
This patch adds checking for the mOmxInitialized after switchToLoaded() to
avoid double deinitialization.
Change-Id: Ie30170de2bf032a1861a00b2ca370266fe12d339
Signed-off-by: Nanako Baba <baba@ti.com>
Signed-off-by: Daniel Levin <dendy@ti.com>
|
|
|
|
|
|
|
|
|
|
| |
In case performCleanupAfterError() is called without further HAL crash
OMX component will be in deinitialized state. Thus it must be marked as
uninialized to avoid double deinitialization attempt in OMXCameraAdapter
destructor.
Change-Id: I65bf7e6ca0acef4b3740d967b6846b5e91136874
Signed-off-by: Daniel Levin <dendy@ti.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since Camera API supports 2 mechanisms that
used to set FPS : setPreviewFpsRange and setPreviewFrameRate
HAL should correctly handle situation when only one
of that call were made. CameraHAL should not
modify fps range or fps that was set by client,
but it should inform camera adpter somehow what
exactly was modified and what value should be used
for configuration of preview port. So therefore
new paramter KEY_PREVIEW_FRAME_RATE_RANGE was added.
Now it will be used by camera adapters for
preview configuration and camera hal will be
capable for value of this parameter.
Possible situations:
1) User had changed FPS range and HAL will use it for fps port
value (don't care about changed or not const FPS).
2) User had changed single FPS and had not changed FPS
range - will be applyed single FPS value to port.
3) Both FPS range and const FPS are unchanged - FPS range
will be applied to port.
This patch also fixed several CTS tests from
android.media.cts.MediaRecorderTest : testRecorderCamera,
testRecorderVideo, testSetMaxFileSize.
Change-Id: I31badcdc62a1a33474cb8e0e893f50378e9f6114
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
|
|
|
|
|
|
|
| |
Prevent waiting for callback of failed AF in ducati. Just
retriggering it.
Change-Id: Ie7305bc98a6391adbdc410f60bfe23222b0d1e39
Signed-off-by: Dmytro Vasylenko <x0178436@ti.com>
|
|
|
|
|
|
|
| |
In stereo mode picture rotation is always set to 0.
Change-Id: Ia71ddce063c90ac66eea1b2b0b5cc2ad58fbed79
Signed-off-by: Ivaylo Todorov <x0177393@ti.com>
|
|\
| |
| |
| |
| | |
Change-Id: I3feec53fce7f847398420b89e2e8cc2b1045cca3
Signed-off-by: Daniel Levin <dendy@ti.com>
|
| |
| |
| |
| |
| | |
Change-Id: Icbca29fb3af86cd0e4e454ad87add5f5561dceab
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
|
|\ \
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
Conflicts:
camera/OMXCameraAdapter/OMXCameraAdapter.cpp
test/CameraHal/camera_test_bufferqueue.h
test/CameraHal/camera_test_surfacetexture.cpp
Change-Id: I1f13c6a5b6369e943773d04a650406a79eb95750
Signed-off-by: Daniel Levin <dendy@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Added support to set external gamma table.
Application can set camera parameter with key "gamma-table"
and value in the form:
"(OffsetR0:SlopeR0,OffsetR1:SlopeR1,...,OffsetRN:SlopeRN),"
"(OffsetG0:SlopeG0,OffsetG1:SlopeG1,...,OffsetGN:SlopeGN),"
"(OffsetB0:SlopeB0,OffsetB1:SlopeB1,...,OffsetBN:SlopeBN)"
Depends on hardware/ti/domx change I199d28e2:
http://review.omapzoom.org/#/c/27885/
Change-Id: I5dabde9b693cf8cb501ec8584729cd4fa174939c
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
NOTE: Currently UYVY is mapped to
'CameraParameters::PIXEL_FORMAT_YUV422I' which
is not entirely correct. The Google pixelformat
expects YUYV ordering of the components.
Change-Id: Ia82e08363d6804e51c447eb845da2ca23fc29ae9
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- The header of the processed buffer should be
set correctly in order for the camera component
to properly match the start of the valid data.
Depends on frameworks/av change I788dd738:
http://review.omapzoom.org/#/c/29677/
Depends on hardware/libhardware change Ie37ab311:
http://review.omapzoom.org/#/c/29676/
Change-Id: I003b7cfae41ddf6bb46661e0803a4fb1b60a99e6
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Image port was disabled twice. There is no
need for that since this is done during
'stopImageCapture()'.
- Re-configuring extra data on the image port
during image capture stop seems unneeded as
well. The extra data on this port is always
configured during image capture start.
- Omit vector shot and extra data configurations
during reprocessing.
Change-Id: I4d766fff581c8043b4cc436882a35f0a87731a3c
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Preview frame wait condition is adding around 30-40 ms.
to the reprocess execution time and this doesn't seem
to be required by Ducati.
Change-Id: I3b708c9d395aa8b9d85cc0289a705b254a5e09a0
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| | |
Change-Id: I019247d8175ae46be307616ae2011938f0b604b1
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- When queued shot is running and parameters like
tap-out/ins don't change we could skip some of
the overhead involved in triggering image capture
using the new shots.
Change-Id: I9738ed0abbc266da0354c216f852bc2f8971d58c
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- With new logic in startImageCapture we have to set the
buffer as IDLE before signaling end image capture.
Change-Id: Ica4d1c222f9ce5f3bef8ada7c3cb8e6ca306d8d5
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Since flow through base camera adapter has changed to make
call flow for accumulating shots similar to regular capture,
OMX capture needs to change it's slow a bit. We should be
accumlating burst counters in UseBuffer and go through whole
list of buffers trying to find an IDLE one when starting capture.
Change-Id: I91bebbb176d29504631095b29ce2a15a44b79f39
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Moving initializing for capture reference count to
derived adapters since useBuffers call for capture
is going to called queuing shots as well.
- This patch needs to include V4L adapter before merging
on mainline.
Change-Id: Ifc4986a68173234ec636ef943b62009f7df1e041
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Fixed deadlock in the case when application has ommiting
to set shot parameters with any bracketing configuration
in CPCAM mode. Should handle properly single shot and
burst shot.
Change-Id: I4f3e2b4ee5bc0aa9072d03da6f9edc4ba86253d8
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 1080p and 720p as well as resolutions which
are 5%, 10%, 20% and 25% bigger along the horizontal&
vertical axes are added.
Change-Id: If255a52995a626cf81ef73ebebe58f938a430a79
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch restores
"CameraHal: Omit flush buffers when disabling image capture"
which was lost after merge of p-jb-release into d-jb-release
because of conflict in OMXCapture.cpp.
Change-Id: I6c9bc02bc6c08dba72f44c4e028b3244dd8d6401
Signed-off-by: Andrii Kulian <andrii.kulian@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Introduce mPendingReprocessSettings, so we can setFormat inside
UseBuffersReprocess instead of setParametersReprocess. We will
later use this mechanism to determine if port settings need to
be reconfigured in NPA-like mode.
Change-Id: I8fc485fc0f84c11bb293a49894090ea6a28b3c47
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Move decision to disable capture port from setParameters to functions
called in the state machine (getPictureBufferSize and
UseBuffersCapture). This will help avoid calling the disable port code
multiple times if user changes settings. It will also keep the capture
from being disabled asynchronously from state changes.
Since decision to disable port is moved to UseBuffersCapture, logic to
skip UseBuffers for subsequent captures changes slightly. Instead of
skipping if capture port is configured, we skip if no settings have
changes that require SetParam have changed.
As part of code movement, capture is no longer disabling reprocess.
The decision to disable reprocess will be moved to the reprocess
file.
Change-Id: I878c4cc1cc4985a762afcae686ba51a072c25836
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
- Added missing initialization of the buffer tracking counters:
'mCapturedFrames', 'mBurstFramesAccum' and 'mBurstFramesQueued'.
Change-Id: I2788c7f9f82c7e806c2817f603381ad2e095eecb
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |\
| | |
| | |
| | | |
Change-Id: I5846453a07de05891544684fd524c546579d957c
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This reverts commit cd6eeab5eada7354dad580d91e801f9681514f5d.
Fixes an instability seen with image capture robustness tests
"flushBuffers - Exiting function flushBuffers because of ret -1
eError=0"
Change-Id: Ic1dee320429397d28d48d16faa91991b843c9572
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Initially queued preview buffers were triggering false
"Buffer was never with Ducati" error in debug build.
Change-Id: I70c310d02bd284c0c532a3b72157c651eb68a20b
Signed-off-by: Daniel Levin <dendy@ti.com>
|
| |\ \
| | | |
| | | |
| | | | |
Change-Id: I0760b135e56d22fc02e49d1344af734e0dcf81be
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This patch improves the PPM numbers for Camera
Change-Id: I1ca1aaaee1ea37992a2717530361429b012377c9
Signed-off-by: Georgi Georgiev <x0154504@ti.com>
|
| |\ \ \
| | | | |
| | | | |
| | | | | |
Change-Id: I649b0c53d1c0e8f56e059a3cb24a38089d8fccc4
|
| |\ \ \ \
| | | | | |
| | | | | |
| | | | | | |
Change-Id: Iec5798f854fdcac8bdbe6f5ab2ef8d17c7224696
|
| |\ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Conflicts:
camera/inc/CameraHal.h
Change-Id: I826ef084fe01e8b5682d8a78c7cec76b39043d45
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Use a common MemoryManager instance in the various OMXCameraAdapter
modules that use it.
Change-Id: I5a2e957345dafb3ccbd305b8e589ed1822a289fe
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
- The parameters to 'getExtraData()' did change
recently so the OMX profiling modification needs
be updated accordingly.
Change-Id: I4d051ec0836af2c47d15446825d835baf40402a0
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
- Fixes an issue with CTS test testSceneMode
Change-Id: I2aebb2f9894d5f31351b29de161b8f5dade035a8
Signed-off-by: Dimitar Borisov <x0152684@ti.com>
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Fixes regression introduced with change I3e5eec25:
Revert "CameraHAL: Adds support for controlling
the FD update interval"
Currently, CameraHal is emmiting preview metadata callbacks
to the Camera application with every preview frame.
But in most cases, there is no any difference between the metadata
of two consecutive frames. So, tracking of metadata changes
would be used in order to send metadata callbacks only if
metadata is different compared to one already sent to
the application. Ie93c6ae5 (Adds support for controlling the FD
update interval) had a disadvantage to add undesired delay when
sending metadata in range (0 ~ 400 ms).
Change-Id: I481a78020a040153f792a3ef9e095b59ea7003e0
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|