mbox series

[GIT,PULL,v2,FOR,v5.5] ti-vpe maintenance, add V4L2_CTRL_TYPE_AREA/V4L2_CID_UNIT_CELL_SIZE

Message ID 494470cf-7de0-d5bd-a448-390f4b2d14dc@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,v2,FOR,v5.5] ti-vpe maintenance, add V4L2_CTRL_TYPE_AREA/V4L2_CID_UNIT_CELL_SIZE | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git for-v5.5h

Message

Hans Verkuil Oct. 10, 2019, 12:30 p.m. UTC
This PR adds support for V4L2_CID_UNIT_CELL_SIZE and the V4L2_CTRL_TYPE_AREA
control type. It also adds support for providing a default value for compound
controls (will be useful for stateless codec support as well).

It also adds a maintenance series for ti-vpe. Note the addition of a pixel
encoding enum for v4l2_format_info.

Changes since the previous PR:

- Fixed up some mismatched email addresses
- Fixed up some incorrect commit IDs in 'Fixes:' lines

Regards,

	Hans

The following changes since commit 219031a6e7dff52a066e8b074adc0697f501e3d3:

  media: venus: fix build on 32bit environments (2019-10-08 13:46:36 -0300)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git for-v5.5h

for you to fetch changes up to 2558d9369e6430360bcf14236f7b0c52c4d66094:

  media: ti-vpe: vpe: don't rely on colorspace member for conversion (2019-10-10 14:26:45 +0200)

----------------------------------------------------------------
Benoit Parrot (18):
      media: ti-vpe: Fix a parallel build issue
      media: ti-vpe: vpe: Fix Motion Vector vpdma stride
      media: ti-vpe: vpe: Add missing null pointer checks
      media: ti-vpe: vpe: Remove unnecessary use of container_of
      media: ti-vpe: vpe: fix a v4l2-compliance failure causing a kernel panic
      media: ti-vpe: vpe: fix a v4l2-compliance warning about invalid pixel format
      media: ti-vpe: vpe: Make sure YUYV is set as default format
      media: ti-vpe: vpe: fix a v4l2-compliance failure about invalid sizeimage
      media: ti-vpe: vpe: fix a v4l2-compliance failure about frame sequence number
      media: ti-vpe: vpe: ensure buffers are cleaned up properly in abort cases
      media: ti-vpe: vpdma: Use fixed type for address in descriptor
      media: ti-vpe: Set the DMA mask and coherent mask
      media: ti-vpe: vpe: use standard struct instead of duplicating fields
      media: ti-vpe: vpe: fix v4l2_compliance issue related to xfer_func
      media: ti-vpe: csc: rgb-to-yuv HD full range coeff are wrong
      media: v4l2-common: add pixel encoding support
      media: v4l2-common: add RGB565 and RGB55 to v4l2_format_info
      media: ti-vpe: vpe: don't rely on colorspace member for conversion

Nikhil Devshatwar (2):
      media: ti-vpe: Add support for SEQ_BT
      media: ti-vpe: Add support for NV21 format

Ram Prasad (1):
      media: ti-vpe: Set MAX height supported to 2048 pixels

Ricardo Ribalda Delgado (8):
      media: v4l2-core: Implement v4l2_ctrl_new_std_compound
      Documentation: v4l2_ctrl_new_std_compound
      media: add V4L2_CTRL_TYPE_AREA control type
      Documentation: media: Document V4L2_CTRL_TYPE_AREA
      media: add V4L2_CID_UNIT_CELL_SIZE control
      Documentation: media: Describe V4L2_CID_UNIT_CELL_SIZE
      media: v4l2-ctrl: Add new helper v4l2_ctrl_ptr_create
      media: imx214: Add new control with V4L2_CID_UNIT_CELL_SIZE

 Documentation/media/kapi/v4l2-controls.rst              |   9 ++
 Documentation/media/uapi/v4l/ext-ctrls-image-source.rst |  10 ++
 Documentation/media/uapi/v4l/vidioc-queryctrl.rst       |   6 +
 Documentation/media/videodev2.h.rst.exceptions          |   1 +
 drivers/media/i2c/imx214.c                              |   9 ++
 drivers/media/platform/Makefile                         |   4 +-
 drivers/media/platform/ti-vpe/csc.c                     | 254 ++++++++++++++++++++----------
 drivers/media/platform/ti-vpe/csc.h                     |   4 +-
 drivers/media/platform/ti-vpe/vpdma.c                   |  11 +-
 drivers/media/platform/ti-vpe/vpdma.h                   |   2 +
 drivers/media/platform/ti-vpe/vpdma_priv.h              |   5 +-
 drivers/media/platform/ti-vpe/vpe.c                     | 394 ++++++++++++++++++++++++++++-------------------
 drivers/media/v4l2-core/v4l2-common.c                   | 128 +++++++--------
 drivers/media/v4l2-core/v4l2-ctrls.c                    |  76 ++++++++-
 include/media/v4l2-common.h                             |  33 +++-
 include/media/v4l2-ctrls.h                              |  76 +++++++++
 include/uapi/linux/v4l2-controls.h                      |   1 +
 include/uapi/linux/videodev2.h                          |   6 +
 18 files changed, 707 insertions(+), 322 deletions(-)

Comments

Jenkins Oct. 10, 2019, 1:07 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/patch/59388/
Build log: https://builder.linuxtv.org/job/patchwork/19812/
Build time: 00:27:21
Link: https://lore.kernel.org/linux-media/494470cf-7de0-d5bd-a448-390f4b2d14dc@xs4all.nl

error: FETCH_HEAD: cannot verify a non-tag object of type commit.

Summary: 7 patches and/or PDF generation with issues, being 0 at build time

Error/warnings:


Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0001-media-v4l2-core-Implement-v4l2_ctrl_new_std_compound.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0001-media-v4l2-core-Implement-v4l2_ctrl_new_std_compound.patch
patches/0001-media-v4l2-core-Implement-v4l2_ctrl_new_std_compound.patch:119: CHECK: Alignment should match open parenthesis

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0005-media-add-V4L2_CID_UNIT_CELL_SIZE-control.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0005-media-add-V4L2_CID_UNIT_CELL_SIZE-control.patch
patches/0005-media-add-V4L2_CID_UNIT_CELL_SIZE-control.patch:50: WARNING: line over 80 characters

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-media-imx214-Add-new-control-with-V4L2_CID_UNIT_CELL.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-media-imx214-Add-new-control-with-V4L2_CID_UNIT_CELL.patch
patches/0008-media-imx214-Add-new-control-with-V4L2_CID_UNIT_CELL.patch:8: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
patches/0008-media-imx214-Add-new-control-with-V4L2_CID_UNIT_CELL.patch:44: CHECK: Alignment should match open parenthesis

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0019-media-ti-vpe-vpe-fix-a-v4l2-compliance-failure-about.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0019-media-ti-vpe-vpe-fix-a-v4l2-compliance-failure-about.patch
patches/0019-media-ti-vpe-vpe-fix-a-v4l2-compliance-failure-about.patch:38: WARNING: line over 80 characters

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0024-media-ti-vpe-vpe-use-standard-struct-instead-of-dupl.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0024-media-ti-vpe-vpe-use-standard-struct-instead-of-dupl.patch
patches/0024-media-ti-vpe-vpe-use-standard-struct-instead-of-dupl.patch:386: CHECK: Alignment should match open parenthesis

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0027-media-v4l2-common-add-pixel-encoding-support.patch
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:47: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:48: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:49: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:50: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:51: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:52: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:53: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:54: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:55: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:56: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:57: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:58: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:59: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:60: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:61: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:68: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:69: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:70: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:71: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:87: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:88: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:89: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:90: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:91: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:92: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:94: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:95: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:96: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:97: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:98: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:99: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:113: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:114: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:115: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:116: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:117: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:118: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:120: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:121: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:122: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:123: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:146: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:147: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:148: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:149: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:150: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:151: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:152: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:153: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:154: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:155: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:156: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:157: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:158: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:159: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:160: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:161: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:162: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:163: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:164: WARNING: line over 80 characters
patches/0027-media-v4l2-common-add-pixel-encoding-support.patch:165: WARNING: line over 80 characters

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0028-media-v4l2-common-add-RGB565-and-RGB55-to-v4l2_forma.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0028-media-v4l2-common-add-RGB565-and-RGB55-to-v4l2_forma.patch
patches/0028-media-v4l2-common-add-RGB565-and-RGB55-to-v4l2_forma.patch:22: WARNING: line over 80 characters
patches/0028-media-v4l2-common-add-RGB565-and-RGB55-to-v4l2_forma.patch:23: WARNING: line over 80 characters