Re: [PATCH v3 00/10] Add support for DELTA video decoder of STMicroelectronics STiH4xx SoC series

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Hans, I will test with new compliance.
BR,
Hugues.

On 12/05/2016 11:18 AM, Hans Verkuil wrote:
> Hi Hugues,
>
> On 11/22/2016 04:53 PM, Hugues Fruchet wrote:
>> This patchset introduces a basic support for DELTA multi-format video
>> decoder of STMicroelectronics STiH4xx SoC series.
>>
>> DELTA hardware IP is controlled by a remote firmware loaded in a ST231
>> coprocessor. Communication with firmware is done within an IPC layer
>> using rpmsg kernel framework and a shared memory for messages handling.
>> This driver is compatible with firmware version 21.1-3.
>> While a single firmware is loaded in ST231 coprocessor, it is composed
>> of several firmwares, one per video format family.
>>
>> This DELTA V4L2 driver is designed around files:
>>   - delta-v4l2.c   : handles V4L2 APIs using M2M framework and calls decoder ops
>>   - delta-<codec>* : implements <codec> decoder calling its associated
>>                      video firmware (for ex. MJPEG) using IPC layer
>>   - delta-ipc.c    : IPC layer which handles communication with firmware using rpmsg
>>
>> This first basic support implements only MJPEG hardware acceleration but
>> the driver structure is in place to support all the features of the
>> DELTA video decoder hardware IP.
>>
>> This driver depends on:
>>   - ST remoteproc/rpmsg: patchset posted at https://lkml.org/lkml/2016/9/6/77
>>   - ST DELTA firmware: its license is under review. When available,
>>     pull request will be done on linux-firmware.
>>
>> ===========
>> = history =
>> ===========
>> version 3
>>   - update after v2 review:
>>     - fixed m2m_buf_done missing on start_streaming error case
>>     - fixed q->dev missing in queue_init()
>>     - removed unsupported s_selection
>>     - refactored string namings in delta-debug.c
>>     - fixed space before comment
>>     - all commits have commit messages
>>     - reword memory allocator helper commit
>>
>> version 2
>>   - update after v1 review:
>>     - simplified tracing
>>     - G_/S_SELECTION reworked to fit COMPOSE(CAPTURE)
>>     - fixed m2m_buf_done missing on start_streaming error case
>>     - fixed q->dev missing in queue_init()
>>   - switch to kernel-4.9 rpmsg API
>>   - DELTA support added in multi_v7_defconfig
>>   - minor typo fixes & code cleanup
>>
>> version 1:
>>   - Initial submission
>>
>> ===================
>> = v4l2-compliance =
>> ===================
>> Below is the v4l2-compliance report for the version 3 of the DELTA video
>> decoder driver. v4l2-compliance has been build from SHA1:
>> 600492351ddf40cc524aab73802153674d7d287b (libdvb5: Fix multiple definition of dvb_dev_remote_init linking error)
>
> Can you update v4l-utils and run this test again? The S_SELECTION compliance
> test has been fixed to allow for ENOTTY as the error code.
>
> If the output looks good, then I'll merge this driver (will be for 4.11).
>
> Regards,
>
> 	Hans
>--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux