Re: [PATCH v3 0/8] [media] s5p-jpeg: Various fixes and improvements

From: Jacek Anaszewski
Date: Thu Jun 29 2017 - 16:14:56 EST


Hi Thierry,

For the whole series:

Acked-by: Jacek Anaszewski <jacek.anaszewski@xxxxxxxxx>

Best regards,
Jacek Anaszewski

On 06/27/2017 06:08 PM, Thierry Escande wrote:
> Hi,
>
> This series contains various fixes and improvements for the Samsung
> s5p-jpeg driver. Most of these patches come from the Chromium v3.8
> kernel tree.
>
> In this v3:
> - Remove codec reset patch (Not needed based on documentation and no
> use case described in original patch commit message).
> - Check for Exynos5420 variant in stream error handling patch.
> - Add use case for resolution change event support in commit message.
> - Move subsampling value decoding in a separate function.
> - Check Exynos variant for 4:1:1 subsampling support.
>
> v2:
> - Remove IOMMU support patch (mapping now created automatically for
> single JPEG CODEC device).
> - Remove "Change sclk_jpeg to 166MHz" patch (can be set through DT
> properties).
> - Remove support for multi-planar APIs (Not needed).
> - Add comment regarding call to jpeg_bound_align_image() after qbuf.
> - Remove unrelated code from resolution change event support patch.
>
> Thierry Escande (3):
> [media] s5p-jpeg: Handle parsing error in s5p_jpeg_parse_hdr()
> [media] s5p-jpeg: Don't use temporary structure in s5p_jpeg_buf_queue
> [media] s5p-jpeg: Split s5p_jpeg_parse_hdr()
>
> Tony K Nadackal (3):
> [media] s5p-jpeg: Call jpeg_bound_align_image after qbuf
> [media] s5p-jpeg: Correct WARN_ON statement for checking subsampling
> [media] s5p-jpeg: Decode 4:1:1 chroma subsampling format
>
> henryhsu (2):
> [media] s5p-jpeg: Add support for resolution change event
> [media] s5p-jpeg: Add stream error handling for Exynos5420
>
> drivers/media/platform/s5p-jpeg/jpeg-core.c | 186 +++++++++++++++++++++-------
> drivers/media/platform/s5p-jpeg/jpeg-core.h | 7 ++
> 2 files changed, 148 insertions(+), 45 deletions(-)
>