All of lore.kernel.org
 help / color / mirror / Atom feed
From: smitha.t@samsung.com (Smitha T Murthy)
To: linux-arm-kernel@lists.infradead.org
Subject: [Patch v6 00/12] Add MFC v10.10 support
Date: Tue, 23 Jan 2018 11:11:06 +0530	[thread overview]
Message-ID: <1516686066.12482.148.camel@smitha-fedora> (raw)
In-Reply-To: <126f1982-060a-6fdb-e1b3-b5c81d65ca13@xs4all.nl>

On Mon, 2018-01-22 at 13:18 +0100, Hans Verkuil wrote:
> Hi Smitha,
> 
> Thank you for this v6 series!
> 
> You can add my:
> 
> Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
> 
> to patches 1-9 and 11. See my review for patches 10 and 12. The comments
> are minor, so I hope I can Ack v7 once it's posted and this can be merged
> for 4.17.
> 
> Regards,
> 
> 	Hans
> 
Thank you so much for the review.
I will update the same by this week and post.

Regards,
Smitha
> On 08/12/17 10:08, Smitha T Murthy wrote:
> > This patch series adds MFC v10.10 support. MFC v10.10 is used in some
> > of Exynos7 variants.
> > 
> > This adds support for following:
> > 
> > * Add support for HEVC encoder and decoder
> > * Add support for VP9 decoder
> > * Update Documentation for control id definitions
> > * Update computation of min scratch buffer size requirement for V8 onwards
> > 
> > Changes since v5:
> >  - Addressed review comments by Kamil Debski <kamil@wypas.org>.
> >  - Addressed review comments by 
> >    Stanimir Varbanov <stanimir.varbanov@linaro.org>.
> >  - Addressed review comments by Hans Verkuil <hverkuil@xs4all.nl>.
> >  - Rebased on latest git://linuxtv.org/snawrocki/samsung.git
> >    for-v4.15/media/next.
> >  - Applied r-o-b from Andrzej, Stanimir on respective patches.
> >  - Applied acked-by from Kamil, Hans on respective patches.
> > 
> > Smitha T Murthy (12):
> >   [media] s5p-mfc: Rename IS_MFCV8 macro
> >   [media] s5p-mfc: Adding initial support for MFC v10.10
> >   [media] s5p-mfc: Use min scratch buffer size as provided by F/W
> >   [media] s5p-mfc: Support MFCv10.10 buffer requirements
> >   [media] videodev2.h: Add v4l2 definition for HEVC
> >   [media] v4l2-ioctl: add HEVC format description
> >   Documentation: v4l: Documentation for HEVC v4l2 definition
> >   [media] s5p-mfc: Add support for HEVC decoder
> >   [media] s5p-mfc: Add VP9 decoder support
> >   [media] v4l2: Add v4l2 control IDs for HEVC encoder
> >   [media] s5p-mfc: Add support for HEVC encoder
> >   Documention: v4l: Documentation for HEVC CIDs
> > 
> >  .../devicetree/bindings/media/s5p-mfc.txt          |   1 +
> >  Documentation/media/uapi/v4l/extended-controls.rst | 395 +++++++++++++++
> >  Documentation/media/uapi/v4l/pixfmt-compressed.rst |   5 +
> >  drivers/media/platform/s5p-mfc/regs-mfc-v10.h      |  88 ++++
> >  drivers/media/platform/s5p-mfc/regs-mfc-v8.h       |   2 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc.c           |  28 ++
> >  drivers/media/platform/s5p-mfc/s5p_mfc_cmd_v6.c    |   9 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc_common.h    |  68 ++-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c      |   6 +-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_dec.c       |  48 +-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_enc.c       | 555 ++++++++++++++++++++-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr.h       |  14 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c    | 397 +++++++++++++--
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.h    |  15 +
> >  drivers/media/v4l2-core/v4l2-ctrls.c               | 118 +++++
> >  drivers/media/v4l2-core/v4l2-ioctl.c               |   1 +
> >  include/uapi/linux/v4l2-controls.h                 |  92 +++-
> >  include/uapi/linux/videodev2.h                     |   1 +
> >  18 files changed, 1765 insertions(+), 78 deletions(-)
> >  create mode 100644 drivers/media/platform/s5p-mfc/regs-mfc-v10.h
> > 
> 
> 
> 

WARNING: multiple messages have this Message-ID (diff)
From: Smitha T Murthy <smitha.t@samsung.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
	kyungmin.park@samsung.com, kamil@wypas.org, jtp.park@samsung.com,
	a.hajda@samsung.com, mchehab@kernel.org,
	pankaj.dubey@samsung.com, krzk@kernel.org,
	m.szyprowski@samsung.com, s.nawrocki@samsung.com
Subject: Re: [Patch v6 00/12] Add MFC v10.10 support
Date: Tue, 23 Jan 2018 11:11:06 +0530	[thread overview]
Message-ID: <1516686066.12482.148.camel@smitha-fedora> (raw)
In-Reply-To: <126f1982-060a-6fdb-e1b3-b5c81d65ca13@xs4all.nl>

On Mon, 2018-01-22 at 13:18 +0100, Hans Verkuil wrote:
> Hi Smitha,
> 
> Thank you for this v6 series!
> 
> You can add my:
> 
> Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
> 
> to patches 1-9 and 11. See my review for patches 10 and 12. The comments
> are minor, so I hope I can Ack v7 once it's posted and this can be merged
> for 4.17.
> 
> Regards,
> 
> 	Hans
> 
Thank you so much for the review.
I will update the same by this week and post.

Regards,
Smitha
> On 08/12/17 10:08, Smitha T Murthy wrote:
> > This patch series adds MFC v10.10 support. MFC v10.10 is used in some
> > of Exynos7 variants.
> > 
> > This adds support for following:
> > 
> > * Add support for HEVC encoder and decoder
> > * Add support for VP9 decoder
> > * Update Documentation for control id definitions
> > * Update computation of min scratch buffer size requirement for V8 onwards
> > 
> > Changes since v5:
> >  - Addressed review comments by Kamil Debski <kamil@wypas.org>.
> >  - Addressed review comments by 
> >    Stanimir Varbanov <stanimir.varbanov@linaro.org>.
> >  - Addressed review comments by Hans Verkuil <hverkuil@xs4all.nl>.
> >  - Rebased on latest git://linuxtv.org/snawrocki/samsung.git
> >    for-v4.15/media/next.
> >  - Applied r-o-b from Andrzej, Stanimir on respective patches.
> >  - Applied acked-by from Kamil, Hans on respective patches.
> > 
> > Smitha T Murthy (12):
> >   [media] s5p-mfc: Rename IS_MFCV8 macro
> >   [media] s5p-mfc: Adding initial support for MFC v10.10
> >   [media] s5p-mfc: Use min scratch buffer size as provided by F/W
> >   [media] s5p-mfc: Support MFCv10.10 buffer requirements
> >   [media] videodev2.h: Add v4l2 definition for HEVC
> >   [media] v4l2-ioctl: add HEVC format description
> >   Documentation: v4l: Documentation for HEVC v4l2 definition
> >   [media] s5p-mfc: Add support for HEVC decoder
> >   [media] s5p-mfc: Add VP9 decoder support
> >   [media] v4l2: Add v4l2 control IDs for HEVC encoder
> >   [media] s5p-mfc: Add support for HEVC encoder
> >   Documention: v4l: Documentation for HEVC CIDs
> > 
> >  .../devicetree/bindings/media/s5p-mfc.txt          |   1 +
> >  Documentation/media/uapi/v4l/extended-controls.rst | 395 +++++++++++++++
> >  Documentation/media/uapi/v4l/pixfmt-compressed.rst |   5 +
> >  drivers/media/platform/s5p-mfc/regs-mfc-v10.h      |  88 ++++
> >  drivers/media/platform/s5p-mfc/regs-mfc-v8.h       |   2 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc.c           |  28 ++
> >  drivers/media/platform/s5p-mfc/s5p_mfc_cmd_v6.c    |   9 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc_common.h    |  68 ++-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c      |   6 +-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_dec.c       |  48 +-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_enc.c       | 555 ++++++++++++++++++++-
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr.h       |  14 +
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c    | 397 +++++++++++++--
> >  drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.h    |  15 +
> >  drivers/media/v4l2-core/v4l2-ctrls.c               | 118 +++++
> >  drivers/media/v4l2-core/v4l2-ioctl.c               |   1 +
> >  include/uapi/linux/v4l2-controls.h                 |  92 +++-
> >  include/uapi/linux/videodev2.h                     |   1 +
> >  18 files changed, 1765 insertions(+), 78 deletions(-)
> >  create mode 100644 drivers/media/platform/s5p-mfc/regs-mfc-v10.h
> > 
> 
> 
> 

  reply	other threads:[~2018-01-23  5:41 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20171208093612epcas1p1eda138655cf5397893fe1f2b2152bd1f@epcas1p1.samsung.com>
2017-12-08  9:08 ` [Patch v6 00/12] Add MFC v10.10 support Smitha T Murthy
2017-12-08  9:08   ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 01/12] [media] s5p-mfc: Rename IS_MFCV8 macro Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 02/12] [media] s5p-mfc: Adding initial support for MFC v10.10 Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08 13:25     ` Philippe Ombredanne
2017-12-08 13:25       ` Philippe Ombredanne
2017-12-08 13:25       ` Philippe Ombredanne
2018-01-18 12:36       ` Smitha T Murthy
2018-01-18 12:36         ` Smitha T Murthy
2018-01-18 12:36         ` Smitha T Murthy
2018-01-18 13:19         ` Hans Verkuil
2018-01-18 13:19           ` Hans Verkuil
2017-12-08  9:08   ` [Patch v6 03/12] [media] s5p-mfc: Use min scratch buffer size as provided by F/W Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 04/12] [media] s5p-mfc: Support MFCv10.10 buffer requirements Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 05/12] [media] videodev2.h: Add v4l2 definition for HEVC Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08 15:47     ` Nicolas Dufresne
2017-12-08 15:47       ` Nicolas Dufresne
2017-12-08 16:07       ` Mauro Carvalho Chehab
2017-12-08 16:07         ` Mauro Carvalho Chehab
2017-12-08  9:08   ` [Patch v6 06/12] [media] v4l2-ioctl: add HEVC format description Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 07/12] Documentation: v4l: Documentation for HEVC v4l2 definition Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 08/12] [media] s5p-mfc: Add support for HEVC decoder Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 09/12] [media] s5p-mfc: Add VP9 decoder support Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-09 18:48     ` Stanimir Varbanov
2017-12-09 18:48       ` Stanimir Varbanov
2017-12-12  2:34       ` Smitha T Murthy
2017-12-12  2:34         ` Smitha T Murthy
2017-12-12  9:46         ` Sylwester Nawrocki
2017-12-12  9:46           ` Sylwester Nawrocki
2017-12-13  8:30           ` Smitha T Murthy
2017-12-13  8:30             ` Smitha T Murthy
2018-01-22 11:08     ` Hans Verkuil
2018-01-22 11:08       ` Hans Verkuil
2018-01-23  4:24       ` Smitha T Murthy
2018-01-23  4:24         ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 11/12] [media] s5p-mfc: Add support " Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2017-12-08  9:08   ` [Patch v6 12/12] Documention: v4l: Documentation for HEVC CIDs Smitha T Murthy
2017-12-08  9:08     ` Smitha T Murthy
2018-01-22 12:15     ` Hans Verkuil
2018-01-22 12:15       ` Hans Verkuil
2018-01-23  5:40       ` Smitha T Murthy
2018-01-23  5:40         ` Smitha T Murthy
2018-01-22 12:18   ` [Patch v6 00/12] Add MFC v10.10 support Hans Verkuil
2018-01-22 12:18     ` Hans Verkuil
2018-01-23  5:41     ` Smitha T Murthy [this message]
2018-01-23  5:41       ` Smitha T Murthy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1516686066.12482.148.camel@smitha-fedora \
    --to=smitha.t@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.