From: builder@xxxxxxxxxxx Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/d68da172-b251-000f-653d-38a8a4c7b715@xxxxxxxxx/ 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@xxxxxxxxx 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@xxxxxxxxx>" [unknown] gpg: aka "Hans Verkuil <hverkuil@xxxxxxxxx>" [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
