Message ID | d68da172-b251-000f-653d-38a8a4c7b715@xs4all.nl (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1kiHYr-00BIyw-25; Thu, 26 Nov 2020 13:43:49 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2390890AbgKZNnR (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Thu, 26 Nov 2020 08:43:17 -0500 Received: from lb2-smtp-cloud7.xs4all.net ([194.109.24.28]:44353 "EHLO lb2-smtp-cloud7.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2390852AbgKZNnQ (ORCPT <rfc822;linux-media@vger.kernel.org>); Thu, 26 Nov 2020 08:43:16 -0500 Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud7.xs4all.net with ESMTPA id iHYEkPGqVN7XgiHYIkYuBb; Thu, 26 Nov 2020 14:43:14 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s2; t=1606398194; bh=n6ExoS7yvmPB1/EfgzBEXRHusmyGXXucfBRt/PjJREQ=; h=From:Subject:To:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=IeM61rFREtgLlGiL2Ucj5sxwxbEwRwkhgEsiM9edWJjAXbfbQzkWRRuyJJnL9hL1v N0CXDtG2uKs27OoKwQCo75rxscRvpBdJLU5b1yonYQIEp96Qah0RngNA17pzs+AwUG BPVZsJRG3oI9epFbQCocStT3mtg/GVak4wBCPKWM27Lm3karDJzqgW+4B7Exydn96h LANPVBKs21ae2IqlIVqxRRwHPEhUjTM733zzWKvQn0tt3huaZ7PFnQfWMDsDxZdN6P PTKSnO2onPs3ND2elW8JwQue0yOV6ZfSY2/o23QPyPPdi8yCdeXxvEjrxLvPS0/rRP 9wkb7uogjzNqA== From: Hans Verkuil <hverkuil@xs4all.nl> Subject: [GIT PULL FOR v5.11] De-stage H.264 and FWHT stateless codec APIs To: Linux Media Mailing List <linux-media@vger.kernel.org> Cc: Ezequiel Garcia <ezequiel@collabora.com> Message-ID: <d68da172-b251-000f-653d-38a8a4c7b715@xs4all.nl> Date: Thu, 26 Nov 2020 14:43:10 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4xfAfmXei734nzcDDRZKXQMMzUIR0TLIi6TwSRQrA9qzddXNyemDc1tLNNL1z0K9yP/8zUjTgToC2bpbtPxA/X+ohrRACk9qlceQ4QAQiAVSfR3Ev0+pYz yA1ecRgNFiSiglzUpq4a9hnrTKNoVLoQvXpi23IvL0g2YtNLReX4Rm9QwooES62kGQIw9/GOMgVyMCCCH0daHDlTG5IWc07bXEc45xnIkoopZWtwHgqk7elv Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.5 (--) X-LSpam-Report: No, score=-2.5 required=5.0 tests=BAYES_00=-1.9,DKIMWL_WL_MED=0.001,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
[GIT,PULL,FOR,v5.11] De-stage H.264 and FWHT stateless codec APIs
|
|
Pull-request
git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.11bMessage
Hans Verkuil
Nov. 26, 2020, 1:43 p.m. UTC
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
Comments
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