| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Add missing usage flag in case of UYVY BS buffer allocation.
Should be squashed with change Ia82e0836 if porting to
another branch:
a4c9c61 CameraHal: Add support for UYVY reprocessing
Change-Id: I6a28377c1941504639b3375989c6f41fad55871c
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |\
| | |
| | |
| | | |
Change-Id: Id3028c34bfcbbe8b0e3817f29ea62025db30e92f
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- The capture mode could be reset to the default "HQ"
if the preview is restarted by new client camera
parameters ( for instance when preview resolution changes).
Change-Id: Ic7e361ba0b7b1347fd3cbf08eca27f3928a2ba85
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Some camera's don't write Huffman Table to MJPEG buffers.
In case of HW MJPEG decoder usage DHT will be added
if it does not exists in buffer.
Change-Id: I12c47d6fdb555a32eeac789b58b481b06c3f82d6
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | | |
Change-Id: I705149d5ce274671385e9f465c0fd3b556d4c7ee
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This patch fix preview data callbacks for V4LCameraAdapter,
when used HW MJPEG Decoder.
Change-Id: I8994549f4706a2a704fc5e1eb30c4c3afd987d3f
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Tis patch adds Ducati decoder support on CameraHAL level.
- Added FrameDecoder - that used as base class for
hw/sw decoders.
- SwFrameDecoder - class that capable for sw MJPEG decoding
- OmxFrameDecoder - class that capable for hardware
MJPEG/H264 decoding (using Ducati decoder)
- FrameDecoder is now used generic interface
for decoder in V4l camera adapter
- Added ability to change video stream type for
V4L camera.
Change-Id: Ia632cb0e202752b05a6994acc90c7f10ee851a34
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | |
| | | |
Synchronize public interface of V4LCameraAdapter.
Change-Id: I5670151a69a58cd7db4e2eda401b3aac7bef18cb
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Added FRAMERATE_RANGE_SUPPORTED parameter fill.
Also added dynamic fps change handling.
Change-Id: I8752987dd51301d1022081f7bd633cd425b866e4
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This mechanism is added to support components where
locking mechanism is handled on DOMX layer.
As example is using ducati decoder with buffers allocated
via ANativeWindowDisplayAdapter.
Change-Id: I33ae0f0766473a6d8e85699c31dcad9345b4df70
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- This patch fix continuous buffer reallocation bug
in Decoder_libjpeg in case of resolution changes on
more greater that was on startup.
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
Change-Id: If62359af9ceca87800f8b819b4da0aa1b40b58b1
|
| |/
| |
| |
| | |
This reverts commit 6520993a1f2c3f4c2c73f79a6949c4ef565cf74c.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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>
|
| |
| |
| |
| |
| |
| |
| | |
This patch enables the FPS information print.
Change-Id: I9f679dba10bdb5100c79436929f1da474233f185
Signed-off-by: Saravanan Solaiyappan <saravanan.s@ti.com>
|
| |
| |
| |
| |
| |
| |
| | |
This reverts commit c02cd8fb7a0b8c9f8bb1f065f5cb086c802a7088.
Change-Id: I70433150f7f5384a28ba9c05ab2dec80d7398da5
Signed-off-by: Saravanan Solaiyappan <saravanan.s@ti.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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- ST can be abandoned by the client, so we
need to handle this case inside 'handleFrameCallback'
without crashing.
- 'freeBufferList()' should only release buffers
associated with it.
Change-Id: Ib052aa6a6c2b5cb495d4e8bcc804cd3a3b87c95f
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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Instead of having to have CameraHal deal with logic to
queue shots, move the code flow to state machine and
respective camera adapters, so they can interally handle it.
Now call flow for queuing shots and regular capture
should be the same from CameraHal perspective.
Change-Id: I9a74008aeaabc8dd169c78f6a85201f50580d25c
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Locking inside 'takePicture()' is sometimes possible
when re-using the same SurfaceTexture. This is due
to the blocking nature of the dequeue call inside
'BufferSourceAdapter::handleFrameReturn()' which
does acquire 'mLock' upon entering. The solution
is to handle this particular case by reusing the
buffers that were previously allocated and avoiding
any calls to the Adapter which might try to lock
'mLock'.
Change-Id: I6c4e49fd84df2659c0466b6f89c88916f93a6b8d
Signed-off-by: Emilian Peev <epeev@mm-sol.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- We were freeing image buffers erroneously when camera adapter
called freeImageBufs during the buffer size query call after
we already cleared mBufferSourceAdapter_Out. This caused
freeImageBufs to take an incorrect path.
- When trying to figure out number of tap out buffers to allocate,
use NO_BUFFERS_IMAGE_CAPTURE_SYSTEM_HEAP as the default when
tap out hasn't been configured yet.
Change-Id: I045ef357578c7eafd12fef58ef50542a711c8233
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Allocate the buffers for a particular surface when it is
set. This allows applications to set multiple surfaces
before capturing or reprocessing, so they don't have to
take the penalty of buffer allocation during.
- Partialy revert change I0c1b8854.
Depends on hardware/libhardware change I01831e05:
http://review.omapzoom.org/#/c/28152/
Change-Id: I13aa2b19387e2a21f8ca08203b132154c14d481b
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| | |
Fix crash if IPP not set by default
Change-Id: I329f3bf49fef2db2ec136e7755c27a01bcab373f
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
USBCameras support higher resolutions(above VGA) in MJPEG format only.
V4LCameraAdapter has been modified to use MJPEG as the default pixel format.
The incoming jpeg frames are decoded(into NV12) using libjpeg and
then sent for rendering.
Change-Id: Ie7f62abd7a7fad3bc1f01e048fe8f17f67890301
Signed-off-by: Saravanan Solaiyappan <saravanan.s@ti.com>
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
Signed-off-by: Anu Sundararajan <sanuradha@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Most of the USBCameras support higher resolutions in MJPEG format only.
To decode MJPEG frames, this patch adds support for libjpeg.
Change-Id: Ifef19a16fddcc8bd56894ed67829f76a63e86a97
Signed-off-by: Saravanan Solaiyappan <saravanan.s@ti.com>
Signed-off-by: Andriy Chepurnyy <x0155536@ti.com>
Signed-off-by: Anu Sundararajan <sanuradha@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Do not allow image buffer size query while in initialized
state. OMXCameraAdapter doesn't support this...
Change-Id: Id66ce67f5cb3667ce883c0ebbd482e958146be08
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
- In Bayer format set GRALLOC_USAGE_PRIVATE_0 usage flag
to the graphics buffer allocator.
Change-Id: I629803110ec6f18cc824c75c6d81450bb108dcff
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- The Thread responsible for retrieving free buffers
from the tapout is possible to miss some incoming
signals and wait while buffers are available in the
ST. Solution is to count all outgoing buffers and
not wait on the condition when buffers are available.
Change-Id: I31526dd334a08f52f0c6069cea847ae14e2cffbc
Signed-off-by: Emilian Peev <epeev@mm-sol.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>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Providing a way to camera client to tell camera hardware
that tap in/outs previously set are not longer valid.
Depends on frameworks/av change I68f61303:
http://review.omapzoom.org/#/c/28914/
Depends on hardware/libhardware change I8f6c377a:
http://review.omapzoom.org/#/c/28984/
Change-Id: I9e7faf1a8cb9bd4e9f3bfd98ed4ccc56c1e9b10e
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Added a method in BufferProvider which can provide
the CameraBuffer list
Change-Id: Ia9ea8e0ac7e59e969fd5bedcfaf057f3607ab963
Signed-off-by: Sundar Raman <a0393242@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Small clean up. Just splitting setBufferSource
into two functions.
Change-Id: Idb160bf26ca0a7f11b4a675c311ad14f978232f1
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Adding support for setting multiple tap in and tap out points
that the application can set at any time. Applications need
to provide name of tap-in and tap-outs with their shot config
for us to find the tap-in or tap-out point from the vector we
are keeping.
- Patch for releasing tap in/outs to follow
Depends on frameworks/av change Ia21c4a26:
http://review.omapzoom.org/#/c/28899/
Change-Id: I6fa91de68c7bf71525596ee5e1a5ebb2a7b3b5da
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- API has changed so applications need to specify the name of the tap
in/out surfaces they want to use for a particular shot/reprocess
call.
Depends on frameworks/av change I7c8c17cf:
http://review.omapzoom.org/#/c/28898/
Depends on hardware/libhardware change I859c6e87:
http://review.omapzoom.org/#/c/28141/
Change-Id: I785286be77970679f8d9cc8bfd9befe4a63191ac
Signed-off-by: Tyler Luu <tluu@ti.com>
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- Check that passed buffer is in current buffer
source list. If no, return the buffer to caller
instead of getting out of range.
Change-Id: I3acef6514049d1e1009f91ec4ec62688974223a1
Signed-off-by: Vladimir Petrov <vppetrov@mm-sol.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
It is impossible to use some V4L devices (e.g. uvc) using blocking API.
In order to avoid deadlocks while recording video we have to use
non-blocking dequeue in a loop.
Change-Id: I7d41170c188b412a0a06553ab64ae53eb56ba3af
Signed-off-by: Nikita Borodikhin <nikita.borodikhin@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
Weaken camera strong dependency on orientation sensor which causes it to
crash in case the sensor is absent, e.g. on Panda.
Change-Id: I276c862878e0a99abb52122fa98c50e067f54f22
Signed-off-by: Nikita Borodikhin <nikita.borodikhin@ti.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This fixes a problem introduced in commit e9bf3ac2. Instead of
specialized method getPictureBufferSize base classes stub is being
always. That led to allocating zero-length snapshot data buffer and
failure to get snapshot.
Change-Id: I36585d7801a788c19be753e2fdbe0f2d056509d9
Signed-off-by: Nikita Borodikhin <nikita.borodikhin@ti.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>
|