All of lore.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs
@ 2020-11-26 13:43 Hans Verkuil
  2020-11-26 14:27 ` [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs (#69390) Jenkins
  0 siblings, 1 reply; 2+ messages in thread
From: Hans Verkuil @ 2020-11-26 13:43 UTC (permalink / raw)
  To: Linux Media Mailing List; +Cc: Ezequiel Garcia

This series de-stages the H.264 and FWHT (as used in vicodec) stateless codec APIs.

This means that the APIs for these stateless codecs are now considered stable.

Many thanks to Ezequiel and others who worked hard on making this possible.
The H.264 is probably the most commonly used codec, so having this as part of
the public API is a major step forward.

The de-staging of the FWHT codec is primarily useful for regression testing of
the stateless codec API with the vicodec driver.

The expectation is that MPEG-2 and VP8 will follow soon (although likely not
for 5.11). HEVC and VP9 will take (a lot?) longer.

Besides the patches for H.264 and FWHT there are also some related patches
for doc fixes and a fix for VP8 type initialization (fixes a compliance issue).

Regards,

	Hans

The following changes since commit 4863b93cfd2dfe88557f820b3399c3fa2163ec43:

  media: camss: csiphy: Set rate on csiX_phy clock on SDM630/660 (2020-11-25 15:02:44 +0100)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.11b

for you to fetch changes up to b7f6989ec69a25b37350a1a210a74c5823e1f090:

  userspace-api/media: finalize stateless FWHT codec docs (2020-11-26 14:26:17 +0100)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Ezequiel Garcia (13):
      media: controls: Add VP8 stateless type initialization
      media: controls: Add validate failure debug message
      media: cedrus: h264: Support profile controls
      media: Rename stateful codec control macros
      media: Clean stateless control includes
      media: controls: Validate H264 stateless controls
      media: controls: Add the stateless codec control class
      media: uapi: Move parsed H264 pixel format out of staging
      media: doc: Replace symbol for V4L2_CTRL_TYPE_H264_PRED_WEIGHTS
      media: uapi: Move the H264 stateless control types out of staging
      media: controls: Log H264 stateless controls in .std_log
      media: uapi: move H264 stateless controls out of staging
      media: docs: Move the H264 stateless codec uAPI

Hans Verkuil (8):
      media: vidioc-g-ext-ctrls.rst: document V4L2_CTRL_CLASS_DETECT
      media: ext-ctrls-image-source.rst: document v4l2_area
      vicodec: add V4L2_ prefix before FWHT_VERSION and FWHT_FL_*
      vicodec: mark the stateless FWHT API as stable
      ext-ctrls-codec.rst: move FWHT docs to ext-ctrls-codec-stateless.rst
      pixfmt-compressed.rst: fix 'bullet' formatting
      vidioc-g-ext-ctrls.rst: add missing 'struct' before the types
      userspace-api/media: finalize stateless FWHT codec docs

Helen Koike (1):
      media: admin-guide/pixfmt-meta-rkisp1.rst: pixfmt reference conforming with macro

Jonas Karlman (1):
      media: rkvdec: h264: Support profile and level controls

 Documentation/admin-guide/media/rkisp1.rst                          |   4 +-
 Documentation/userspace-api/media/v4l/common.rst                    |   1 +
 Documentation/userspace-api/media/v4l/dev-mem2mem.rst               |   2 +-
 Documentation/userspace-api/media/v4l/ext-ctrls-codec-stateless.rst | 793 +++++++++++++++++++++++++++++
 Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst           | 817 +-----------------------------
 Documentation/userspace-api/media/v4l/ext-ctrls-image-source.rst    |  16 +
 Documentation/userspace-api/media/v4l/extended-controls.rst         |   8 +-
 Documentation/userspace-api/media/v4l/pixfmt-compressed.rst         |  37 +-
 Documentation/userspace-api/media/v4l/pixfmt-meta-rkisp1.rst        |   4 +-
 Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst        |  44 +-
 Documentation/userspace-api/media/v4l/vidioc-queryctrl.rst          |   6 +
 Documentation/userspace-api/media/videodev2.h.rst.exceptions        |   2 +
 drivers/media/common/cx2341x.c                                      |   4 +-
 drivers/media/platform/s5p-mfc/s5p_mfc_dec.c                        |   2 +-
 drivers/media/platform/s5p-mfc/s5p_mfc_enc.c                        |   2 +-
 drivers/media/test-drivers/vicodec/codec-fwht.c                     |  13 +-
 drivers/media/test-drivers/vicodec/codec-fwht.h                     |  32 --
 drivers/media/test-drivers/vicodec/codec-v4l2-fwht.c                |  88 ++--
 drivers/media/test-drivers/vicodec/vicodec-core.c                   |  46 +-
 drivers/media/v4l2-core/v4l2-ctrls.c                                | 229 ++++++++-
 drivers/staging/media/hantro/hantro_drv.c                           |  26 +-
 drivers/staging/media/hantro/hantro_h264.c                          |   8 +-
 drivers/staging/media/hantro/hantro_hw.h                            |   4 +-
 drivers/staging/media/rkvdec/rkvdec-h264.c                          |   8 +-
 drivers/staging/media/rkvdec/rkvdec.c                               |  39 +-
 drivers/staging/media/sunxi/cedrus/cedrus.c                         |  43 +-
 drivers/staging/media/sunxi/cedrus/cedrus_dec.c                     |  12 +-
 include/media/fwht-ctrls.h                                          |  31 --
 include/media/h264-ctrls.h                                          | 406 ---------------
 include/media/hevc-ctrls.h                                          |  10 +-
 include/media/mpeg2-ctrls.h                                         |   4 +-
 include/media/v4l2-ctrls.h                                          |   2 -
 include/media/v4l2-h264.h                                           |   2 +-
 include/media/vp8-ctrls.h                                           |   2 +-
 include/uapi/linux/v4l2-controls.h                                  | 867 ++++++++++++++++++++++++--------
 include/uapi/linux/videodev2.h                                      |  17 +
 36 files changed, 1948 insertions(+), 1683 deletions(-)
 create mode 100644 Documentation/userspace-api/media/v4l/ext-ctrls-codec-stateless.rst
 delete mode 100644 include/media/fwht-ctrls.h
 delete mode 100644 include/media/h264-ctrls.h

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs (#69390)
  2020-11-26 13:43 [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs Hans Verkuil
@ 2020-11-26 14:27 ` Jenkins
  0 siblings, 0 replies; 2+ messages in thread
From: Jenkins @ 2020-11-26 14:27 UTC (permalink / raw)
  To: mchehab+samsung, linux-media; +Cc: builder

From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/d68da172-b251-000f-653d-38a8a4c7b715@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/78715/
Build time: 00:34:42
Link: https://lore.kernel.org/linux-media/d68da172-b251-000f-653d-38a8a4c7b715@xs4all.nl

gpg: Signature made Thu 26 Nov 2020 01:32:56 PM UTC
gpg:                using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg:                 aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]
gpg: Note: This key has expired!
Primary key fingerprint: 052C DE7B C215 053B 689F  1BCA BD2D 6148 6614 3B4C
     Subkey fingerprint: AAA7 FFBA 4D2D 77EF 4CAE  A142 1326 E0CD 23AB DCE5

Summary: got 8/23 patches with issues, being 0 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0006-media-Rename-stateful-codec-control-macros.patch:

   checkpatch.pl:
	$ cat patches/0006-media-Rename-stateful-codec-control-macros.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:142: CHECK: Macro argument reuse 'x' - possible side-effects?
	-:155: CHECK: Macro argument reuse 'x' - possible side-effects?
	-:210: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:211: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:212: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:213: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:214: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:215: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:216: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:217: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:252: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:253: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:294: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:309: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:309: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:310: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:311: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:312: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:313: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:314: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:315: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:323: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:330: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:330: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:339: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:339: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:348: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:348: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:357: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:357: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:366: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:366: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:374: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:374: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:382: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:382: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:389: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:389: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:397: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:397: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:398: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:399: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:408: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:408: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:417: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:417: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:421: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:428: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:428: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:440: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:440: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:441: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:442: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:443: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:444: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:460: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:460: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:461: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:462: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:463: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:464: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:465: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:466: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:467: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:468: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:469: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:480: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:480: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:481: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:482: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:483: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:484: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:500: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:500: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:501: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:502: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:503: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:504: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:505: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:506: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:507: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:511: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:519: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:519: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:546: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:547: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:548: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:549: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:550: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:551: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:552: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:553: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:554: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:555: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:556: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:557: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:558: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:565: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:565: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:566: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:577: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:577: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:578: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:579: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:586: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:586: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:598: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:598: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:599: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:600: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:601: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:612: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:612: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:613: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:614: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:624: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:624: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:625: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:635: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:635: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:636: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:647: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:647: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:648: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:649: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:650: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:651: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:652: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:671: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:671: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:672: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:673: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:674: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:675: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:676: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:677: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:678: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:679: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:680: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:681: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:682: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:683: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:684: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:693: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:693: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:702: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:702: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:709: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:718: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:718: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:719: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:729: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:729: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:730: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:731: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:732: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:741: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:741: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:742: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:743: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:744: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:747: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:756: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:764: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:764: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:799: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:814: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:824: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:832: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:842: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:862: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:891: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:912: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:919: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:919: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:920: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:929: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:929: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:935: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:935: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:942: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:942: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:943: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:956: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:956: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:957: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:958: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:959: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:960: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:969: WARNING: line length of 101 exceeds 100 columns
	-:969: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:970: WARNING: line length of 101 exceeds 100 columns
	-:970: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:971: WARNING: line length of 101 exceeds 100 columns
	-:971: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:978: WARNING: line length of 101 exceeds 100 columns
	-:978: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:978: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:993: WARNING: line length of 101 exceeds 100 columns
	-:993: CHECK: Please use a blank line after function/struct/union/enum declarations
	-:993: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:994: WARNING: line length of 101 exceeds 100 columns
	-:994: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:995: WARNING: line length of 101 exceeds 100 columns
	-:995: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:996: WARNING: line length of 101 exceeds 100 columns
	-:996: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:997: WARNING: line length of 102 exceeds 100 columns
	-:997: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:998: WARNING: line length of 102 exceeds 100 columns
	-:998: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:999: WARNING: line length of 102 exceeds 100 columns
	-:999: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:1000: WARNING: line length of 102 exceeds 100 columns
	-:1000: CHECK: spaces preferred around that '+' (ctx:VxV)
	-:1001: WARNING: line length of 102 exceeds 100 columns
	-:1001: CHECK: spaces preferred around that '+' (ctx:VxV)

patches/0008-media-controls-Validate-H264-stateless-controls.patch:

   checkpatch.pl:
	$ cat patches/0008-media-controls-Validate-H264-stateless-controls.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:191: CHECK: Macro argument reuse 'sps' - possible side-effects?

patches/0009-media-controls-Add-the-stateless-codec-control-class.patch:

   checkpatch.pl:
	$ cat patches/0009-media-controls-Add-the-stateless-codec-control-class.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:72: CHECK: Please don't use multiple blank lines
	-:77: CHECK: Please don't use multiple blank lines

patches/0013-media-controls-Log-H264-stateless-controls-in-.std_l.patch:

   checkpatch.pl:
	$ cat patches/0013-media-controls-Log-H264-stateless-controls-in-.std_l.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:26: WARNING: Avoid logging continuation uses where feasible
	-:29: WARNING: Avoid logging continuation uses where feasible
	-:32: WARNING: Avoid logging continuation uses where feasible
	-:35: WARNING: Avoid logging continuation uses where feasible
	-:38: WARNING: Avoid logging continuation uses where feasible
	-:41: WARNING: Avoid logging continuation uses where feasible

patches/0014-media-uapi-move-H264-stateless-controls-out-of-stagi.patch:

   checkpatch.pl:
	$ cat patches/0014-media-uapi-move-H264-stateless-controls-out-of-stagi.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:68: WARNING: line length of 101 exceeds 100 columns
	-:71: WARNING: line length of 102 exceeds 100 columns
	-:408: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:891: CHECK: Macro argument reuse 'sps' - possible side-effects?
	-:1026: CHECK: Macro argument reuse 'pps' - possible side-effects?
	-:1026: CHECK: Macro argument reuse 'slice' - possible side-effects?

patches/0015-media-docs-Move-the-H264-stateless-codec-uAPI.patch:

   checkpatch.pl:
	$ cat patches/0015-media-docs-Move-the-H264-stateless-codec-uAPI.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:34: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0018-vicodec-add-V4L2_-prefix-before-FWHT_VERSION-and-FWH.patch:

   checkpatch.pl:
	$ cat patches/0018-vicodec-add-V4L2_-prefix-before-FWHT_VERSION-and-FWH.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:195: CHECK: Prefer using the BIT macro

patches/0019-vicodec-mark-the-stateless-FWHT-API-as-stable.patch:

   checkpatch.pl:
	$ cat patches/0019-vicodec-mark-the-stateless-FWHT-API-as-stable.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:174: WARNING: Avoid logging continuation uses where feasible
	-:200: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?


Error #512 when building PDF docs


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-11-26 14:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-26 13:43 [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs Hans Verkuil
2020-11-26 14:27 ` [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs (#69390) Jenkins

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.