Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1475103 > unrolled thread

[PATCH v5 0/9] Add MT8173 Video Decoder Driver

Started byTiffany Lin <tiffany.lin@mediatek.com>
First post2016-09-02 14:30 +0200
Last post2016-09-05 13:40 +0200
Articles 9 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH v5 0/9] Add MT8173 Video Decoder Driver Tiffany Lin <tiffany.lin@mediatek.com> - 2016-09-02 14:30 +0200
    Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-09-05 11:40 +0200
      Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Tiffany Lin <tiffany.lin@mediatek.com> - 2016-09-05 12:00 +0200
        Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-09-05 12:00 +0200
          Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-09-05 12:10 +0200
            Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Tiffany Lin <tiffany.lin@mediatek.com> - 2016-09-05 13:00 +0200
              Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-09-05 13:30 +0200
                Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-09-05 13:40 +0200
                Re: [PATCH v5 0/9] Add MT8173 Video Decoder Driver Tiffany Lin <tiffany.lin@mediatek.com> - 2016-09-05 13:40 +0200

#1475103 — [PATCH v5 0/9] Add MT8173 Video Decoder Driver

FromTiffany Lin <tiffany.lin@mediatek.com>
Date2016-09-02 14:30 +0200
Subject[PATCH v5 0/9] Add MT8173 Video Decoder Driver
Message-ID<scVLz-7Si-3@gated-at.bofh.it>
==============
 Introduction
==============

The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.

This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.

Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.

[1]https://chromium-review.googlesource.com/#/c/245241/

==================
 Device interface
==================

In principle the driver bases on v4l2 memory-to-memory framework:
it provides a single video node and each opened file handle gets its own private context with separate buffer queues. Each context consist of 2 buffer 
queues: OUTPUT (for source buffers, i.e. bitstream) and CAPTURE (for destination buffers, i.e. decoded video frames).
OUTPUT and CAPTURE buffer could be MMAP or DMABUF memory type.

Change in v5:
1. Merge wucheng's V4L2_PIX_FMT_VP9 series to this series
   V4L: add VP9 format documentation
   v4l2-ioctl: add VP9 format description.
   videodev2.h: add V4L2_PIX_FMT_VP9 format.
2. Remove V4L2_PIX_FMT_MT21C to another series

Change in v4:
1.Change V4L2_PIX_FMT_MT21 to v4l2_PIX_FMT_MT21C
2.Fix g/s_selection implementation
3.Refine code according to review comments and pass V4l2-compliance test

-/bin/sh: v4l2: not found
# v4l2-compliance -d /dev/video0
v4l2-compliance SHA   : fc45cdc502065bc1dfb4ef9ceb9a822bb9877bce

Driver Info:
        Driver name   : mtk-vcodec-dec
        Card type     : platform:mt8173
        Bus info      : platform:mt8173
        Driver version: 4.8.0
        Capabilities  : 0x84204000
                Video Memory-to-Memory Multiplanar
                Streaming
                Extended Pix Format
                Device Capabilities
        Device Caps   : 0x04204000
                Video Memory-to-Memory Multiplanar
                Streaming
                Extended Pix Format

Compliance test for device /dev/video0 (not using libv4l2):

Required ioctls:
        test VIDIOC_QUERYCAP: OK

Allow for multiple opens:
        test second video open: OK
        test VIDIOC_QUERYCAP: OK
        test VIDIOC_G/S_PRIORITY: OK
        test for unlimited opens: OK

Debug ioctls:
        test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported)
        test VIDIOC_LOG_STATUS: OK (Not Supported)

Input ioctls:
        test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported)
        test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
        test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported)
        test VIDIOC_ENUMAUDIO: OK (Not Supported)
        test VIDIOC_G/S/ENUMINPUT: OK (Not Supported)
        test VIDIOC_G/S_AUDIO: OK (Not Supported)
        Inputs: 0 Audio Inputs: 0 Tuners: 0

Output ioctls:
        test VIDIOC_G/S_MODULATOR: OK (Not Supported)
        test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
        test VIDIOC_ENUMAUDOUT: OK (Not Supported)
        test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported)
        test VIDIOC_G/S_AUDOUT: OK (Not Supported)
        Outputs: 0 Audio Outputs: 0 Modulators: 0

Input/Output configuration ioctls:
        test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported)
        test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported)
        test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
        test VIDIOC_G/S_EDID: OK (Not Supported)

        Control ioctls:
                test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
                test VIDIOC_QUERYCTRL: OK
                test VIDIOC_G/S_CTRL: OK
                test VIDIOC_G/S/TRY_EXT_CTRLS: OK
                test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
                test VIDIOC_G/S_JPEGCOMP: OK (Not Supported)
                Standard Controls: 2 Private Controls: 0

        Format ioctls:
                test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
                test VIDIOC_G/S_PARM: OK (Not Supported)
                test VIDIOC_G_FBUF: OK (Not Supported)
                test VIDIOC_G_FMT: OK
                test VIDIOC_TRY_FMT: OK
                test VIDIOC_S_FMT: OK
                test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported)
                test Cropping: OK (Not Supported)
                test Composing: OK
                test Scaling: OK

        Codec ioctls:
                test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
                test VIDIOC_G_ENC_INDEX: OK (Not Supported)
                test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported)

        Buffer ioctls:
                test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK
                test VIDIOC_EXPBUF: OK

Test input 0:

Total: 43, Succeeded: 43, Failed: 0, Warnings: 0


Change in v3:
1. Refine vdec hw clock setting
2. Refine vp9 codec driver
3. Refine v4l2 codec driver

Change in v2:
1. Add documentation for V4L2_PIX_FMT_MT21 2. Remove DRM_FORMAT_MT21 2. Refine code according to review comments


Andrew-CT Chen (1):
  VPU: mediatek: Add decode support

Tiffany Lin (10):
  dt-bindings: Add a binding for Mediatek Video Decoder
  vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
  vcodec: mediatek: Add Mediatek H264 Video Decoder Drive
  vcodec: mediatek: Add Mediatek VP8 Video Decoder Driver
  Add documentation for V4L2_PIX_FMT_VP9.
  vcodec: mediatek: Add Mediatek VP9 Video Decoder Driver
  v4l: add Mediatek compressed video block format
  docs-rst: Add compressed video formats used on MT8173 codec driver
  vcodec: mediatek: Add V4L2_PIX_FMT_MT21C support for v4l2 decoder
  arm64: dts: mediatek: Add Video Decoder for MT8173

Wu-Cheng Li (2):
  videodev2.h: add V4L2_PIX_FMT_VP9 format.
  v4l2-ioctl: add VP9 format description.

 .../devicetree/bindings/media/mediatek-vcodec.txt  |   57 +-
 Documentation/media/uapi/v4l/pixfmt-013.rst        |    8 +
 Documentation/media/uapi/v4l/pixfmt-reserved.rst   |    6 +
 arch/arm64/boot/dts/mediatek/mt8173.dtsi           |   44 +
 drivers/media/platform/mtk-vcodec/Makefile         |   15 +-
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 1448 ++++++++++++++++++++
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h |   88 ++
 .../media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c |  394 ++++++
 .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c  |  205 +++
 .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h  |   28 +
 drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h |   62 +-
 .../media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c |    8 +-
 .../media/platform/mtk-vcodec/mtk_vcodec_intr.c    |    3 +-
 .../media/platform/mtk-vcodec/mtk_vcodec_util.c    |   33 +-
 .../media/platform/mtk-vcodec/mtk_vcodec_util.h    |    5 +
 .../media/platform/mtk-vcodec/vdec/vdec_h264_if.c  |  506 +++++++
 .../media/platform/mtk-vcodec/vdec/vdec_vp8_if.c   |  633 +++++++++
 .../media/platform/mtk-vcodec/vdec/vdec_vp9_if.c   |  967 +++++++++++++
 drivers/media/platform/mtk-vcodec/vdec_drv_base.h  |   56 +
 drivers/media/platform/mtk-vcodec/vdec_drv_if.c    |  122 ++
 drivers/media/platform/mtk-vcodec/vdec_drv_if.h    |  101 ++
 drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h   |  103 ++
 drivers/media/platform/mtk-vcodec/vdec_vpu_if.c    |  168 +++
 drivers/media/platform/mtk-vcodec/vdec_vpu_if.h    |   96 ++
 drivers/media/platform/mtk-vpu/mtk_vpu.c           |   12 +
 drivers/media/platform/mtk-vpu/mtk_vpu.h           |   27 +
 drivers/media/v4l2-core/v4l2-ioctl.c               |    2 +
 include/uapi/linux/videodev2.h                     |    2 +
 28 files changed, 5173 insertions(+), 26 deletions(-)
 create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
 create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h
 create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c
 create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c
 create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_h264_if.c
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp8_if.c
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp9_if.c
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_base.h
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.c
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.h
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.c
 create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.h

-- 
1.7.9.5

[toc] | [next] | [standalone]

#1476235

FromHans Verkuil <hverkuil@xs4all.nl>
Date2016-09-05 11:40 +0200
Message-ID<sdYxI-1S9-31@gated-at.bofh.it>
In reply to#1475103
On 09/02/2016 02:19 PM, Tiffany Lin wrote:
> ==============
>  Introduction
> ==============
> 
> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
> 
> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
> 
> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
> 
> [1]https://chromium-review.googlesource.com/#/c/245241/

This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?

I'm ready to make a pull request for this, so I hope you can fix this soon.

Thanks!

	Hans

> 
> ==================
>  Device interface
> ==================
> 
> In principle the driver bases on v4l2 memory-to-memory framework:
> it provides a single video node and each opened file handle gets its own private context with separate buffer queues. Each context consist of 2 buffer 
> queues: OUTPUT (for source buffers, i.e. bitstream) and CAPTURE (for destination buffers, i.e. decoded video frames).
> OUTPUT and CAPTURE buffer could be MMAP or DMABUF memory type.
> 
> Change in v5:
> 1. Merge wucheng's V4L2_PIX_FMT_VP9 series to this series
>    V4L: add VP9 format documentation
>    v4l2-ioctl: add VP9 format description.
>    videodev2.h: add V4L2_PIX_FMT_VP9 format.
> 2. Remove V4L2_PIX_FMT_MT21C to another series
> 
> Change in v4:
> 1.Change V4L2_PIX_FMT_MT21 to v4l2_PIX_FMT_MT21C
> 2.Fix g/s_selection implementation
> 3.Refine code according to review comments and pass V4l2-compliance test
> 
> -/bin/sh: v4l2: not found
> # v4l2-compliance -d /dev/video0
> v4l2-compliance SHA   : fc45cdc502065bc1dfb4ef9ceb9a822bb9877bce
> 
> Driver Info:
>         Driver name   : mtk-vcodec-dec
>         Card type     : platform:mt8173
>         Bus info      : platform:mt8173
>         Driver version: 4.8.0
>         Capabilities  : 0x84204000
>                 Video Memory-to-Memory Multiplanar
>                 Streaming
>                 Extended Pix Format
>                 Device Capabilities
>         Device Caps   : 0x04204000
>                 Video Memory-to-Memory Multiplanar
>                 Streaming
>                 Extended Pix Format
> 
> Compliance test for device /dev/video0 (not using libv4l2):
> 
> Required ioctls:
>         test VIDIOC_QUERYCAP: OK
> 
> Allow for multiple opens:
>         test second video open: OK
>         test VIDIOC_QUERYCAP: OK
>         test VIDIOC_G/S_PRIORITY: OK
>         test for unlimited opens: OK
> 
> Debug ioctls:
>         test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported)
>         test VIDIOC_LOG_STATUS: OK (Not Supported)
> 
> Input ioctls:
>         test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported)
>         test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
>         test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported)
>         test VIDIOC_ENUMAUDIO: OK (Not Supported)
>         test VIDIOC_G/S/ENUMINPUT: OK (Not Supported)
>         test VIDIOC_G/S_AUDIO: OK (Not Supported)
>         Inputs: 0 Audio Inputs: 0 Tuners: 0
> 
> Output ioctls:
>         test VIDIOC_G/S_MODULATOR: OK (Not Supported)
>         test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
>         test VIDIOC_ENUMAUDOUT: OK (Not Supported)
>         test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported)
>         test VIDIOC_G/S_AUDOUT: OK (Not Supported)
>         Outputs: 0 Audio Outputs: 0 Modulators: 0
> 
> Input/Output configuration ioctls:
>         test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported)
>         test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported)
>         test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
>         test VIDIOC_G/S_EDID: OK (Not Supported)
> 
>         Control ioctls:
>                 test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
>                 test VIDIOC_QUERYCTRL: OK
>                 test VIDIOC_G/S_CTRL: OK
>                 test VIDIOC_G/S/TRY_EXT_CTRLS: OK
>                 test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
>                 test VIDIOC_G/S_JPEGCOMP: OK (Not Supported)
>                 Standard Controls: 2 Private Controls: 0
> 
>         Format ioctls:
>                 test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
>                 test VIDIOC_G/S_PARM: OK (Not Supported)
>                 test VIDIOC_G_FBUF: OK (Not Supported)
>                 test VIDIOC_G_FMT: OK
>                 test VIDIOC_TRY_FMT: OK
>                 test VIDIOC_S_FMT: OK
>                 test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported)
>                 test Cropping: OK (Not Supported)
>                 test Composing: OK
>                 test Scaling: OK
> 
>         Codec ioctls:
>                 test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
>                 test VIDIOC_G_ENC_INDEX: OK (Not Supported)
>                 test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported)
> 
>         Buffer ioctls:
>                 test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK
>                 test VIDIOC_EXPBUF: OK
> 
> Test input 0:
> 
> Total: 43, Succeeded: 43, Failed: 0, Warnings: 0
> 
> 
> Change in v3:
> 1. Refine vdec hw clock setting
> 2. Refine vp9 codec driver
> 3. Refine v4l2 codec driver
> 
> Change in v2:
> 1. Add documentation for V4L2_PIX_FMT_MT21 2. Remove DRM_FORMAT_MT21 2. Refine code according to review comments
> 
> 
> Andrew-CT Chen (1):
>   VPU: mediatek: Add decode support
> 
> Tiffany Lin (10):
>   dt-bindings: Add a binding for Mediatek Video Decoder
>   vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
>   vcodec: mediatek: Add Mediatek H264 Video Decoder Drive
>   vcodec: mediatek: Add Mediatek VP8 Video Decoder Driver
>   Add documentation for V4L2_PIX_FMT_VP9.
>   vcodec: mediatek: Add Mediatek VP9 Video Decoder Driver
>   v4l: add Mediatek compressed video block format
>   docs-rst: Add compressed video formats used on MT8173 codec driver
>   vcodec: mediatek: Add V4L2_PIX_FMT_MT21C support for v4l2 decoder
>   arm64: dts: mediatek: Add Video Decoder for MT8173
> 
> Wu-Cheng Li (2):
>   videodev2.h: add V4L2_PIX_FMT_VP9 format.
>   v4l2-ioctl: add VP9 format description.
> 
>  .../devicetree/bindings/media/mediatek-vcodec.txt  |   57 +-
>  Documentation/media/uapi/v4l/pixfmt-013.rst        |    8 +
>  Documentation/media/uapi/v4l/pixfmt-reserved.rst   |    6 +
>  arch/arm64/boot/dts/mediatek/mt8173.dtsi           |   44 +
>  drivers/media/platform/mtk-vcodec/Makefile         |   15 +-
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 1448 ++++++++++++++++++++
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h |   88 ++
>  .../media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c |  394 ++++++
>  .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c  |  205 +++
>  .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h  |   28 +
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h |   62 +-
>  .../media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c |    8 +-
>  .../media/platform/mtk-vcodec/mtk_vcodec_intr.c    |    3 +-
>  .../media/platform/mtk-vcodec/mtk_vcodec_util.c    |   33 +-
>  .../media/platform/mtk-vcodec/mtk_vcodec_util.h    |    5 +
>  .../media/platform/mtk-vcodec/vdec/vdec_h264_if.c  |  506 +++++++
>  .../media/platform/mtk-vcodec/vdec/vdec_vp8_if.c   |  633 +++++++++
>  .../media/platform/mtk-vcodec/vdec/vdec_vp9_if.c   |  967 +++++++++++++
>  drivers/media/platform/mtk-vcodec/vdec_drv_base.h  |   56 +
>  drivers/media/platform/mtk-vcodec/vdec_drv_if.c    |  122 ++
>  drivers/media/platform/mtk-vcodec/vdec_drv_if.h    |  101 ++
>  drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h   |  103 ++
>  drivers/media/platform/mtk-vcodec/vdec_vpu_if.c    |  168 +++
>  drivers/media/platform/mtk-vcodec/vdec_vpu_if.h    |   96 ++
>  drivers/media/platform/mtk-vpu/mtk_vpu.c           |   12 +
>  drivers/media/platform/mtk-vpu/mtk_vpu.h           |   27 +
>  drivers/media/v4l2-core/v4l2-ioctl.c               |    2 +
>  include/uapi/linux/videodev2.h                     |    2 +
>  28 files changed, 5173 insertions(+), 26 deletions(-)
>  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h
>  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_h264_if.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp8_if.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp9_if.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_base.h
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.h
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.c
>  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.h
>

[toc] | [prev] | [next] | [standalone]

#1476255

FromTiffany Lin <tiffany.lin@mediatek.com>
Date2016-09-05 12:00 +0200
Message-ID<sdYR6-209-73@gated-at.bofh.it>
In reply to#1476235
Hi Hans,

On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
> > ==============
> >  Introduction
> > ==============
> > 
> > The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
> > Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
> > 
> > This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
> > Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
> > 
> > Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
> > 
> > [1]https://chromium-review.googlesource.com/#/c/245241/
> 
> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
> 
> I'm ready to make a pull request for this, so I hope you can fix this soon.
> 
I saw Encoder fix patches in fixes branch.
This patch series is base on Encoder fix patches. 

[media] vcodec:mediatek: Refine VP8 encoder driver
[media] vcodec:mediatek: Refine H264 encoder driver
[media] vcodec:mediatek: change H264 profile default to profile high
[media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
Encoder
[media] vcodec:mediatek: Fix visible_height larger than coded_height
issue in s_fmt_out
[media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
[media] vcodec:mediatek:code refine for v4l2 Encoder driver
[media] dvb_frontend: Use memdup_user() rather than duplicating its
implementation


If I do not rebase decoder patch series base on Encoder fix pathces.
Then it will fail after Encoder fix patches merged.

May I know what parent patch I need to rebase to to fix this issue?


best regards,
Tiffany

> Thanks!
> 
> 	Hans
> 
> > 
> > ==================
> >  Device interface
> > ==================
> > 
> > In principle the driver bases on v4l2 memory-to-memory framework:
> > it provides a single video node and each opened file handle gets its own private context with separate buffer queues. Each context consist of 2 buffer 
> > queues: OUTPUT (for source buffers, i.e. bitstream) and CAPTURE (for destination buffers, i.e. decoded video frames).
> > OUTPUT and CAPTURE buffer could be MMAP or DMABUF memory type.
> > 
> > Change in v5:
> > 1. Merge wucheng's V4L2_PIX_FMT_VP9 series to this series
> >    V4L: add VP9 format documentation
> >    v4l2-ioctl: add VP9 format description.
> >    videodev2.h: add V4L2_PIX_FMT_VP9 format.
> > 2. Remove V4L2_PIX_FMT_MT21C to another series
> > 
> > Change in v4:
> > 1.Change V4L2_PIX_FMT_MT21 to v4l2_PIX_FMT_MT21C
> > 2.Fix g/s_selection implementation
> > 3.Refine code according to review comments and pass V4l2-compliance test
> > 
> > -/bin/sh: v4l2: not found
> > # v4l2-compliance -d /dev/video0
> > v4l2-compliance SHA   : fc45cdc502065bc1dfb4ef9ceb9a822bb9877bce
> > 
> > Driver Info:
> >         Driver name   : mtk-vcodec-dec
> >         Card type     : platform:mt8173
> >         Bus info      : platform:mt8173
> >         Driver version: 4.8.0
> >         Capabilities  : 0x84204000
> >                 Video Memory-to-Memory Multiplanar
> >                 Streaming
> >                 Extended Pix Format
> >                 Device Capabilities
> >         Device Caps   : 0x04204000
> >                 Video Memory-to-Memory Multiplanar
> >                 Streaming
> >                 Extended Pix Format
> > 
> > Compliance test for device /dev/video0 (not using libv4l2):
> > 
> > Required ioctls:
> >         test VIDIOC_QUERYCAP: OK
> > 
> > Allow for multiple opens:
> >         test second video open: OK
> >         test VIDIOC_QUERYCAP: OK
> >         test VIDIOC_G/S_PRIORITY: OK
> >         test for unlimited opens: OK
> > 
> > Debug ioctls:
> >         test VIDIOC_DBG_G/S_REGISTER: OK (Not Supported)
> >         test VIDIOC_LOG_STATUS: OK (Not Supported)
> > 
> > Input ioctls:
> >         test VIDIOC_G/S_TUNER/ENUM_FREQ_BANDS: OK (Not Supported)
> >         test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
> >         test VIDIOC_S_HW_FREQ_SEEK: OK (Not Supported)
> >         test VIDIOC_ENUMAUDIO: OK (Not Supported)
> >         test VIDIOC_G/S/ENUMINPUT: OK (Not Supported)
> >         test VIDIOC_G/S_AUDIO: OK (Not Supported)
> >         Inputs: 0 Audio Inputs: 0 Tuners: 0
> > 
> > Output ioctls:
> >         test VIDIOC_G/S_MODULATOR: OK (Not Supported)
> >         test VIDIOC_G/S_FREQUENCY: OK (Not Supported)
> >         test VIDIOC_ENUMAUDOUT: OK (Not Supported)
> >         test VIDIOC_G/S/ENUMOUTPUT: OK (Not Supported)
> >         test VIDIOC_G/S_AUDOUT: OK (Not Supported)
> >         Outputs: 0 Audio Outputs: 0 Modulators: 0
> > 
> > Input/Output configuration ioctls:
> >         test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported)
> >         test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported)
> >         test VIDIOC_DV_TIMINGS_CAP: OK (Not Supported)
> >         test VIDIOC_G/S_EDID: OK (Not Supported)
> > 
> >         Control ioctls:
> >                 test VIDIOC_QUERY_EXT_CTRL/QUERYMENU: OK
> >                 test VIDIOC_QUERYCTRL: OK
> >                 test VIDIOC_G/S_CTRL: OK
> >                 test VIDIOC_G/S/TRY_EXT_CTRLS: OK
> >                 test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
> >                 test VIDIOC_G/S_JPEGCOMP: OK (Not Supported)
> >                 Standard Controls: 2 Private Controls: 0
> > 
> >         Format ioctls:
> >                 test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
> >                 test VIDIOC_G/S_PARM: OK (Not Supported)
> >                 test VIDIOC_G_FBUF: OK (Not Supported)
> >                 test VIDIOC_G_FMT: OK
> >                 test VIDIOC_TRY_FMT: OK
> >                 test VIDIOC_S_FMT: OK
> >                 test VIDIOC_G_SLICED_VBI_CAP: OK (Not Supported)
> >                 test Cropping: OK (Not Supported)
> >                 test Composing: OK
> >                 test Scaling: OK
> > 
> >         Codec ioctls:
> >                 test VIDIOC_(TRY_)ENCODER_CMD: OK (Not Supported)
> >                 test VIDIOC_G_ENC_INDEX: OK (Not Supported)
> >                 test VIDIOC_(TRY_)DECODER_CMD: OK (Not Supported)
> > 
> >         Buffer ioctls:
> >                 test VIDIOC_REQBUFS/CREATE_BUFS/QUERYBUF: OK
> >                 test VIDIOC_EXPBUF: OK
> > 
> > Test input 0:
> > 
> > Total: 43, Succeeded: 43, Failed: 0, Warnings: 0
> > 
> > 
> > Change in v3:
> > 1. Refine vdec hw clock setting
> > 2. Refine vp9 codec driver
> > 3. Refine v4l2 codec driver
> > 
> > Change in v2:
> > 1. Add documentation for V4L2_PIX_FMT_MT21 2. Remove DRM_FORMAT_MT21 2. Refine code according to review comments
> > 
> > 
> > Andrew-CT Chen (1):
> >   VPU: mediatek: Add decode support
> > 
> > Tiffany Lin (10):
> >   dt-bindings: Add a binding for Mediatek Video Decoder
> >   vcodec: mediatek: Add Mediatek V4L2 Video Decoder Driver
> >   vcodec: mediatek: Add Mediatek H264 Video Decoder Drive
> >   vcodec: mediatek: Add Mediatek VP8 Video Decoder Driver
> >   Add documentation for V4L2_PIX_FMT_VP9.
> >   vcodec: mediatek: Add Mediatek VP9 Video Decoder Driver
> >   v4l: add Mediatek compressed video block format
> >   docs-rst: Add compressed video formats used on MT8173 codec driver
> >   vcodec: mediatek: Add V4L2_PIX_FMT_MT21C support for v4l2 decoder
> >   arm64: dts: mediatek: Add Video Decoder for MT8173
> > 
> > Wu-Cheng Li (2):
> >   videodev2.h: add V4L2_PIX_FMT_VP9 format.
> >   v4l2-ioctl: add VP9 format description.
> > 
> >  .../devicetree/bindings/media/mediatek-vcodec.txt  |   57 +-
> >  Documentation/media/uapi/v4l/pixfmt-013.rst        |    8 +
> >  Documentation/media/uapi/v4l/pixfmt-reserved.rst   |    6 +
> >  arch/arm64/boot/dts/mediatek/mt8173.dtsi           |   44 +
> >  drivers/media/platform/mtk-vcodec/Makefile         |   15 +-
> >  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 1448 ++++++++++++++++++++
> >  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h |   88 ++
> >  .../media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c |  394 ++++++
> >  .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c  |  205 +++
> >  .../media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h  |   28 +
> >  drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h |   62 +-
> >  .../media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c |    8 +-
> >  .../media/platform/mtk-vcodec/mtk_vcodec_intr.c    |    3 +-
> >  .../media/platform/mtk-vcodec/mtk_vcodec_util.c    |   33 +-
> >  .../media/platform/mtk-vcodec/mtk_vcodec_util.h    |    5 +
> >  .../media/platform/mtk-vcodec/vdec/vdec_h264_if.c  |  506 +++++++
> >  .../media/platform/mtk-vcodec/vdec/vdec_vp8_if.c   |  633 +++++++++
> >  .../media/platform/mtk-vcodec/vdec/vdec_vp9_if.c   |  967 +++++++++++++
> >  drivers/media/platform/mtk-vcodec/vdec_drv_base.h  |   56 +
> >  drivers/media/platform/mtk-vcodec/vdec_drv_if.c    |  122 ++
> >  drivers/media/platform/mtk-vcodec/vdec_drv_if.h    |  101 ++
> >  drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h   |  103 ++
> >  drivers/media/platform/mtk-vcodec/vdec_vpu_if.c    |  168 +++
> >  drivers/media/platform/mtk-vcodec/vdec_vpu_if.h    |   96 ++
> >  drivers/media/platform/mtk-vpu/mtk_vpu.c           |   12 +
> >  drivers/media/platform/mtk-vpu/mtk_vpu.h           |   27 +
> >  drivers/media/v4l2-core/v4l2-ioctl.c               |    2 +
> >  include/uapi/linux/videodev2.h                     |    2 +
> >  28 files changed, 5173 insertions(+), 26 deletions(-)
> >  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.h
> >  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_h264_if.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp8_if.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_vp9_if.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_base.h
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_drv_if.h
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_ipi_msg.h
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.c
> >  create mode 100644 drivers/media/platform/mtk-vcodec/vdec_vpu_if.h
> >

[toc] | [prev] | [next] | [standalone]

#1476257

FromHans Verkuil <hverkuil@xs4all.nl>
Date2016-09-05 12:00 +0200
Message-ID<sdYR6-209-83@gated-at.bofh.it>
In reply to#1476255
On 09/05/2016 11:50 AM, Tiffany Lin wrote:
> Hi Hans,
> 
> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
>>> ==============
>>>  Introduction
>>> ==============
>>>
>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
>>>
>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
>>>
>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
>>>
>>> [1]https://chromium-review.googlesource.com/#/c/245241/
>>
>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
>>
>> I'm ready to make a pull request for this, so I hope you can fix this soon.
>>
> I saw Encoder fix patches in fixes branch.
> This patch series is base on Encoder fix patches. 
> 
> [media] vcodec:mediatek: Refine VP8 encoder driver
> [media] vcodec:mediatek: Refine H264 encoder driver
> [media] vcodec:mediatek: change H264 profile default to profile high
> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
> Encoder
> [media] vcodec:mediatek: Fix visible_height larger than coded_height
> issue in s_fmt_out
> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
> [media] dvb_frontend: Use memdup_user() rather than duplicating its
> implementation
> 
> 
> If I do not rebase decoder patch series base on Encoder fix pathces.
> Then it will fail after Encoder fix patches merged.
> 
> May I know what parent patch I need to rebase to to fix this issue?

Ah, OK. I will retest placing this on top of those fixes. I forgot about the
pending fixes. I'll handle this.

Regards,

	Hans

[toc] | [prev] | [next] | [standalone]

#1476269

FromHans Verkuil <hverkuil@xs4all.nl>
Date2016-09-05 12:10 +0200
Message-ID<sdZ0K-2iZ-31@gated-at.bofh.it>
In reply to#1476257
On 09/05/2016 11:53 AM, Hans Verkuil wrote:
> On 09/05/2016 11:50 AM, Tiffany Lin wrote:
>> Hi Hans,
>>
>> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
>>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
>>>> ==============
>>>>  Introduction
>>>> ==============
>>>>
>>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
>>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
>>>>
>>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
>>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
>>>>
>>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
>>>>
>>>> [1]https://chromium-review.googlesource.com/#/c/245241/
>>>
>>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
>>>
>>> I'm ready to make a pull request for this, so I hope you can fix this soon.
>>>
>> I saw Encoder fix patches in fixes branch.
>> This patch series is base on Encoder fix patches. 
>>
>> [media] vcodec:mediatek: Refine VP8 encoder driver
>> [media] vcodec:mediatek: Refine H264 encoder driver
>> [media] vcodec:mediatek: change H264 profile default to profile high
>> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
>> Encoder
>> [media] vcodec:mediatek: Fix visible_height larger than coded_height
>> issue in s_fmt_out
>> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
>> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
>> [media] dvb_frontend: Use memdup_user() rather than duplicating its
>> implementation
>>
>>
>> If I do not rebase decoder patch series base on Encoder fix pathces.
>> Then it will fail after Encoder fix patches merged.
>>
>> May I know what parent patch I need to rebase to to fix this issue?
> 
> Ah, OK. I will retest placing this on top of those fixes. I forgot about the
> pending fixes. I'll handle this.

OK, that works much better when it is on top of the fixes.

Only one thing is missing: an entry in the MAINTAINERS file. Strange that that
wasn't done for the encoder driver, we must have missed that. So can you make a
patch adding the en/decoder driver to this file? I'm ready to make the pull
request once I have that.

Regards,

	Hans

[toc] | [prev] | [next] | [standalone]

#1476301

FromTiffany Lin <tiffany.lin@mediatek.com>
Date2016-09-05 13:00 +0200
Message-ID<sdZN7-2Fz-9@gated-at.bofh.it>
In reply to#1476269
Hi Hans,

On Mon, 2016-09-05 at 12:09 +0200, Hans Verkuil wrote:
> On 09/05/2016 11:53 AM, Hans Verkuil wrote:
> > On 09/05/2016 11:50 AM, Tiffany Lin wrote:
> >> Hi Hans,
> >>
> >> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
> >>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
> >>>> ==============
> >>>>  Introduction
> >>>> ==============
> >>>>
> >>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
> >>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
> >>>>
> >>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
> >>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
> >>>>
> >>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
> >>>>
> >>>> [1]https://chromium-review.googlesource.com/#/c/245241/
> >>>
> >>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
> >>>
> >>> I'm ready to make a pull request for this, so I hope you can fix this soon.
> >>>
> >> I saw Encoder fix patches in fixes branch.
> >> This patch series is base on Encoder fix patches. 
> >>
> >> [media] vcodec:mediatek: Refine VP8 encoder driver
> >> [media] vcodec:mediatek: Refine H264 encoder driver
> >> [media] vcodec:mediatek: change H264 profile default to profile high
> >> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
> >> Encoder
> >> [media] vcodec:mediatek: Fix visible_height larger than coded_height
> >> issue in s_fmt_out
> >> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
> >> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
> >> [media] dvb_frontend: Use memdup_user() rather than duplicating its
> >> implementation
> >>
> >>
> >> If I do not rebase decoder patch series base on Encoder fix pathces.
> >> Then it will fail after Encoder fix patches merged.
> >>
> >> May I know what parent patch I need to rebase to to fix this issue?
> > 
> > Ah, OK. I will retest placing this on top of those fixes. I forgot about the
> > pending fixes. I'll handle this.
> 
> OK, that works much better when it is on top of the fixes.
> 
> Only one thing is missing: an entry in the MAINTAINERS file. Strange that that
> wasn't done for the encoder driver, we must have missed that. So can you make a
> patch adding the en/decoder driver to this file? I'm ready to make the pull
> request once I have that.
> 
Got it. I will check and update it.

I have a question about decoder device node.
In v5 decoder patch series, I do not add device node patch at end of
patch series.
I plain to put it in MT21C format patch series, is it ok?


best regards,
Tiffany



> Regards,
> 
> 	Hans

[toc] | [prev] | [next] | [standalone]

#1476316

FromHans Verkuil <hverkuil@xs4all.nl>
Date2016-09-05 13:30 +0200
Message-ID<se0ga-34H-19@gated-at.bofh.it>
In reply to#1476301
On 09/05/2016 12:52 PM, Tiffany Lin wrote:
> Hi Hans,
> 
> On Mon, 2016-09-05 at 12:09 +0200, Hans Verkuil wrote:
>> On 09/05/2016 11:53 AM, Hans Verkuil wrote:
>>> On 09/05/2016 11:50 AM, Tiffany Lin wrote:
>>>> Hi Hans,
>>>>
>>>> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
>>>>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
>>>>>> ==============
>>>>>>  Introduction
>>>>>> ==============
>>>>>>
>>>>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
>>>>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
>>>>>>
>>>>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
>>>>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
>>>>>>
>>>>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
>>>>>>
>>>>>> [1]https://chromium-review.googlesource.com/#/c/245241/
>>>>>
>>>>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
>>>>>
>>>>> I'm ready to make a pull request for this, so I hope you can fix this soon.
>>>>>
>>>> I saw Encoder fix patches in fixes branch.
>>>> This patch series is base on Encoder fix patches. 
>>>>
>>>> [media] vcodec:mediatek: Refine VP8 encoder driver
>>>> [media] vcodec:mediatek: Refine H264 encoder driver
>>>> [media] vcodec:mediatek: change H264 profile default to profile high
>>>> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
>>>> Encoder
>>>> [media] vcodec:mediatek: Fix visible_height larger than coded_height
>>>> issue in s_fmt_out
>>>> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
>>>> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
>>>> [media] dvb_frontend: Use memdup_user() rather than duplicating its
>>>> implementation
>>>>
>>>>
>>>> If I do not rebase decoder patch series base on Encoder fix pathces.
>>>> Then it will fail after Encoder fix patches merged.
>>>>
>>>> May I know what parent patch I need to rebase to to fix this issue?
>>>
>>> Ah, OK. I will retest placing this on top of those fixes. I forgot about the
>>> pending fixes. I'll handle this.
>>
>> OK, that works much better when it is on top of the fixes.
>>
>> Only one thing is missing: an entry in the MAINTAINERS file. Strange that that
>> wasn't done for the encoder driver, we must have missed that. So can you make a
>> patch adding the en/decoder driver to this file? I'm ready to make the pull
>> request once I have that.
>>
> Got it. I will check and update it.
> 
> I have a question about decoder device node.
> In v5 decoder patch series, I do not add device node patch at end of
> patch series.
> I plain to put it in MT21C format patch series, is it ok?

What is the reason for not adding it in the decoder patch series?

Regards,

	Hans

[toc] | [prev] | [next] | [standalone]

#1476322

FromHans Verkuil <hverkuil@xs4all.nl>
Date2016-09-05 13:40 +0200
Message-ID<se0pQ-381-25@gated-at.bofh.it>
In reply to#1476316
On 09/05/2016 01:30 PM, Tiffany Lin wrote:
> Hi Hans,
> 
> On Mon, 2016-09-05 at 13:26 +0200, Hans Verkuil wrote:
>> On 09/05/2016 12:52 PM, Tiffany Lin wrote:
>>> Hi Hans,
>>>
>>> On Mon, 2016-09-05 at 12:09 +0200, Hans Verkuil wrote:
>>>> On 09/05/2016 11:53 AM, Hans Verkuil wrote:
>>>>> On 09/05/2016 11:50 AM, Tiffany Lin wrote:
>>>>>> Hi Hans,
>>>>>>
>>>>>> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
>>>>>>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
>>>>>>>> ==============
>>>>>>>>  Introduction
>>>>>>>> ==============
>>>>>>>>
>>>>>>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
>>>>>>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
>>>>>>>>
>>>>>>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
>>>>>>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
>>>>>>>>
>>>>>>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
>>>>>>>>
>>>>>>>> [1]https://chromium-review.googlesource.com/#/c/245241/
>>>>>>>
>>>>>>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
>>>>>>>
>>>>>>> I'm ready to make a pull request for this, so I hope you can fix this soon.
>>>>>>>
>>>>>> I saw Encoder fix patches in fixes branch.
>>>>>> This patch series is base on Encoder fix patches. 
>>>>>>
>>>>>> [media] vcodec:mediatek: Refine VP8 encoder driver
>>>>>> [media] vcodec:mediatek: Refine H264 encoder driver
>>>>>> [media] vcodec:mediatek: change H264 profile default to profile high
>>>>>> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
>>>>>> Encoder
>>>>>> [media] vcodec:mediatek: Fix visible_height larger than coded_height
>>>>>> issue in s_fmt_out
>>>>>> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
>>>>>> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
>>>>>> [media] dvb_frontend: Use memdup_user() rather than duplicating its
>>>>>> implementation
>>>>>>
>>>>>>
>>>>>> If I do not rebase decoder patch series base on Encoder fix pathces.
>>>>>> Then it will fail after Encoder fix patches merged.
>>>>>>
>>>>>> May I know what parent patch I need to rebase to to fix this issue?
>>>>>
>>>>> Ah, OK. I will retest placing this on top of those fixes. I forgot about the
>>>>> pending fixes. I'll handle this.
>>>>
>>>> OK, that works much better when it is on top of the fixes.
>>>>
>>>> Only one thing is missing: an entry in the MAINTAINERS file. Strange that that
>>>> wasn't done for the encoder driver, we must have missed that. So can you make a
>>>> patch adding the en/decoder driver to this file? I'm ready to make the pull
>>>> request once I have that.
>>>>
>>> Got it. I will check and update it.
>>>
>>> I have a question about decoder device node.
>>> In v5 decoder patch series, I do not add device node patch at end of
>>> patch series.
>>> I plain to put it in MT21C format patch series, is it ok?
>>
>> What is the reason for not adding it in the decoder patch series?
>>
> 
> It is because our decoder HW only support MT21C format.

Good reason :-)

OK, that's fine.

Regards,

	Hans

[toc] | [prev] | [next] | [standalone]

#1476328

FromTiffany Lin <tiffany.lin@mediatek.com>
Date2016-09-05 13:40 +0200
Message-ID<se0pQ-381-27@gated-at.bofh.it>
In reply to#1476316
Hi Hans,

On Mon, 2016-09-05 at 13:26 +0200, Hans Verkuil wrote:
> On 09/05/2016 12:52 PM, Tiffany Lin wrote:
> > Hi Hans,
> > 
> > On Mon, 2016-09-05 at 12:09 +0200, Hans Verkuil wrote:
> >> On 09/05/2016 11:53 AM, Hans Verkuil wrote:
> >>> On 09/05/2016 11:50 AM, Tiffany Lin wrote:
> >>>> Hi Hans,
> >>>>
> >>>> On Mon, 2016-09-05 at 11:33 +0200, Hans Verkuil wrote:
> >>>>> On 09/02/2016 02:19 PM, Tiffany Lin wrote:
> >>>>>> ==============
> >>>>>>  Introduction
> >>>>>> ==============
> >>>>>>
> >>>>>> The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs.
> >>>>>> Mediatek Video Codec is able to handle video decoding and encoding of in a range of formats.
> >>>>>>
> >>>>>> This patch series rely on MTK VPU driver that have been merged in v4.8-rc1.
> >>>>>> Mediatek Video Decoder driver rely on VPU driver to load, communicate with VPU.
> >>>>>>
> >>>>>> Internally the driver uses videobuf2 framework, and MTK IOMMU and MTK SMI both have been merged in v4.6-rc1.
> >>>>>>
> >>>>>> [1]https://chromium-review.googlesource.com/#/c/245241/
> >>>>>
> >>>>> This patch series fails to apply to the media_tree master (patch 3/9). Can you rebase and repost?
> >>>>>
> >>>>> I'm ready to make a pull request for this, so I hope you can fix this soon.
> >>>>>
> >>>> I saw Encoder fix patches in fixes branch.
> >>>> This patch series is base on Encoder fix patches. 
> >>>>
> >>>> [media] vcodec:mediatek: Refine VP8 encoder driver
> >>>> [media] vcodec:mediatek: Refine H264 encoder driver
> >>>> [media] vcodec:mediatek: change H264 profile default to profile high
> >>>> [media] vcodec:mediatek: Add timestamp and timecode copy for V4L2
> >>>> Encoder
> >>>> [media] vcodec:mediatek: Fix visible_height larger than coded_height
> >>>> issue in s_fmt_out
> >>>> [media] vcodec:mediatek: Fix fops_vcodec_release flow for V4L2 Encoder
> >>>> [media] vcodec:mediatek:code refine for v4l2 Encoder driver
> >>>> [media] dvb_frontend: Use memdup_user() rather than duplicating its
> >>>> implementation
> >>>>
> >>>>
> >>>> If I do not rebase decoder patch series base on Encoder fix pathces.
> >>>> Then it will fail after Encoder fix patches merged.
> >>>>
> >>>> May I know what parent patch I need to rebase to to fix this issue?
> >>>
> >>> Ah, OK. I will retest placing this on top of those fixes. I forgot about the
> >>> pending fixes. I'll handle this.
> >>
> >> OK, that works much better when it is on top of the fixes.
> >>
> >> Only one thing is missing: an entry in the MAINTAINERS file. Strange that that
> >> wasn't done for the encoder driver, we must have missed that. So can you make a
> >> patch adding the en/decoder driver to this file? I'm ready to make the pull
> >> request once I have that.
> >>
> > Got it. I will check and update it.
> > 
> > I have a question about decoder device node.
> > In v5 decoder patch series, I do not add device node patch at end of
> > patch series.
> > I plain to put it in MT21C format patch series, is it ok?
> 
> What is the reason for not adding it in the decoder patch series?
> 

It is because our decoder HW only support MT21C format.


best regards,
Tiffany

> Regards,
> 
> 	Hans

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web