All of lore.kernel.org
 help / color / mirror / Atom feed
From: "mirela.rabulea@oss.nxp.com" <mirela.rabulea@oss.nxp.com>
To: Ming Qian <ming.qian@nxp.com>,
	mchehab@kernel.org, hverkuil-cisco@xs4all.nl
Cc: shawnguo@kernel.org, s.hauer@pengutronix.de,
	kernel@pengutronix.de, festevam@gmail.com, xiahong.bao@nxp.com,
	linux-imx@nxp.com, linux-media@vger.kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended
Date: Thu, 22 Dec 2022 23:47:00 +0200	[thread overview]
Message-ID: <281a508b-fb88-4bd4-c473-d329aaa3e332@oss.nxp.com> (raw)
In-Reply-To: <cover.1671071730.git.ming.qian@nxp.com>

Hi,
for the patches in the imx-jpeg driver, in the commit description, 
sentences should begin with capital letter.

Other than that, for the entire patch-set:
Reviewed-by: Mirela Rabulea <mirela.rabulea@nxp.com>
Tested-by: Mirela Rabulea <mirela.rabulea@nxp.com>

Regards,
Mirela

On 20.12.2022 05:11, Ming Qian wrote:
> the imx8 jpeg codec support ISO/IEC 10918-1 Standard Baseline and
> Extended Sequential DCT modes.
> it can support 8-bit and 12-bit per color samples.
> Currently we only enable the 8-bit jpeg,
> and we can enable the 12-bit extended jpeg
> with the new defined 12 bit pixel format.
> 
> 
> Ming Qian (10):
>    media: Add P012 and P012M video format
>    media: Add Y012 video format
>    media: Add Y212 video format
>    media: Add Y312 video format
>    media: Add B312 video format
>    media: Add B412 video format
>    media: imx-jpeg: Refine the function mxc_jpeg_find_format
>    media: imx-jpeg: Clear slot next desc ptr if config error
>    media: imx-jpeg: Decoder add support for 12bit jpeg
>    media: imx-jpeg: Encoder add support for 12bit jpeg
> 
>   .../media/v4l/pixfmt-packed-yuv.rst           |  70 ++++
>   .../userspace-api/media/v4l/pixfmt-rgb.rst    |  44 +++
>   .../media/v4l/pixfmt-yuv-luma.rst             |  11 +
>   .../media/v4l/pixfmt-yuv-planar.rst           |  94 +++++
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c |  19 +-
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h |   5 +-
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg.c    | 326 ++++++++++++++++--
>   drivers/media/v4l2-core/v4l2-common.c         |   6 +
>   drivers/media/v4l2-core/v4l2-ioctl.c          |   7 +
>   include/uapi/linux/videodev2.h                |   9 +
>   10 files changed, 565 insertions(+), 26 deletions(-)
> 

WARNING: multiple messages have this Message-ID (diff)
From: "mirela.rabulea@oss.nxp.com" <mirela.rabulea@oss.nxp.com>
To: Ming Qian <ming.qian@nxp.com>,
	mchehab@kernel.org, hverkuil-cisco@xs4all.nl
Cc: shawnguo@kernel.org, s.hauer@pengutronix.de,
	kernel@pengutronix.de, festevam@gmail.com, xiahong.bao@nxp.com,
	linux-imx@nxp.com, linux-media@vger.kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended
Date: Thu, 22 Dec 2022 23:47:00 +0200	[thread overview]
Message-ID: <281a508b-fb88-4bd4-c473-d329aaa3e332@oss.nxp.com> (raw)
In-Reply-To: <cover.1671071730.git.ming.qian@nxp.com>

Hi,
for the patches in the imx-jpeg driver, in the commit description, 
sentences should begin with capital letter.

Other than that, for the entire patch-set:
Reviewed-by: Mirela Rabulea <mirela.rabulea@nxp.com>
Tested-by: Mirela Rabulea <mirela.rabulea@nxp.com>

Regards,
Mirela

On 20.12.2022 05:11, Ming Qian wrote:
> the imx8 jpeg codec support ISO/IEC 10918-1 Standard Baseline and
> Extended Sequential DCT modes.
> it can support 8-bit and 12-bit per color samples.
> Currently we only enable the 8-bit jpeg,
> and we can enable the 12-bit extended jpeg
> with the new defined 12 bit pixel format.
> 
> 
> Ming Qian (10):
>    media: Add P012 and P012M video format
>    media: Add Y012 video format
>    media: Add Y212 video format
>    media: Add Y312 video format
>    media: Add B312 video format
>    media: Add B412 video format
>    media: imx-jpeg: Refine the function mxc_jpeg_find_format
>    media: imx-jpeg: Clear slot next desc ptr if config error
>    media: imx-jpeg: Decoder add support for 12bit jpeg
>    media: imx-jpeg: Encoder add support for 12bit jpeg
> 
>   .../media/v4l/pixfmt-packed-yuv.rst           |  70 ++++
>   .../userspace-api/media/v4l/pixfmt-rgb.rst    |  44 +++
>   .../media/v4l/pixfmt-yuv-luma.rst             |  11 +
>   .../media/v4l/pixfmt-yuv-planar.rst           |  94 +++++
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.c |  19 +-
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg-hw.h |   5 +-
>   .../media/platform/nxp/imx-jpeg/mxc-jpeg.c    | 326 ++++++++++++++++--
>   drivers/media/v4l2-core/v4l2-common.c         |   6 +
>   drivers/media/v4l2-core/v4l2-ioctl.c          |   7 +
>   include/uapi/linux/videodev2.h                |   9 +
>   10 files changed, 565 insertions(+), 26 deletions(-)
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-12-22 21:47 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-20  3:11 [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended Ming Qian
2022-12-20  3:11 ` Ming Qian
2022-12-20  3:11 ` [PATCH 01/10] media: Add P012 and P012M video format Ming Qian
2022-12-20  3:11   ` Ming Qian
2023-01-27 14:31   ` Hans Verkuil
2023-01-27 14:31     ` Hans Verkuil
2023-01-30  8:15     ` [EXT] " Ming Qian
2023-01-30  8:15       ` Ming Qian
2023-02-10 15:08       ` Nicolas Dufresne
2023-02-10 15:08         ` Nicolas Dufresne
2023-02-13  9:16         ` Ming Qian
2023-02-13  9:16           ` Ming Qian
2022-12-20  3:11 ` [PATCH 02/10] media: Add Y012 " Ming Qian
2022-12-20  3:11   ` Ming Qian
2022-12-20  3:11 ` [PATCH 03/10] media: Add Y212 " Ming Qian
2022-12-20  3:11   ` Ming Qian
2023-01-27 14:37   ` Hans Verkuil
2023-01-27 14:37     ` Hans Verkuil
2023-02-22 15:31     ` Nicolas Dufresne
2023-02-22 15:31       ` Nicolas Dufresne
2023-02-23  1:35       ` [EXT] " Ming Qian
2023-02-23  1:35         ` Ming Qian
2022-12-20  3:11 ` [PATCH 04/10] media: Add Y312 " Ming Qian
2022-12-20  3:11   ` Ming Qian
2023-01-27 14:42   ` Hans Verkuil
2023-01-27 14:42     ` Hans Verkuil
2022-12-20  3:11 ` [PATCH 05/10] media: Add B312 " Ming Qian
2022-12-20  3:11   ` Ming Qian
2023-01-27 14:46   ` Hans Verkuil
2023-01-27 14:46     ` Hans Verkuil
2022-12-20  3:11 ` [PATCH 06/10] media: Add B412 " Ming Qian
2022-12-20  3:11   ` Ming Qian
2023-01-27 14:53   ` Hans Verkuil
2023-01-27 14:53     ` Hans Verkuil
2022-12-20  3:11 ` [PATCH 07/10] media: imx-jpeg: Refine the function mxc_jpeg_find_format Ming Qian
2022-12-20  3:11   ` Ming Qian
2022-12-20  3:11 ` [PATCH 08/10] media: imx-jpeg: Clear slot next desc ptr if config error Ming Qian
2022-12-20  3:11   ` Ming Qian
2022-12-20  3:11 ` [PATCH 09/10] media: imx-jpeg: Decoder add support for 12bit jpeg Ming Qian
2022-12-20  3:11   ` Ming Qian
2022-12-20  3:11 ` [PATCH 10/10] media: imx-jpeg: Encoder " Ming Qian
2022-12-20  3:11   ` Ming Qian
2022-12-22 21:47 ` mirela.rabulea [this message]
2022-12-22 21:47   ` [PATCH 00/10] media: imx-jpeg: Add support for 12 bit extended mirela.rabulea

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=281a508b-fb88-4bd4-c473-d329aaa3e332@oss.nxp.com \
    --to=mirela.rabulea@oss.nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=ming.qian@nxp.com \
    --cc=s.hauer@pengutronix.de \
    --cc=shawnguo@kernel.org \
    --cc=xiahong.bao@nxp.com \
    /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.