mbox series

[GIT,PULL,FOR,v5.14] More Outreachy patches

Message ID 89431df2-8159-f45e-1aa5-4e9f0836c53d@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.14] More Outreachy patches | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.13-out3

Message

Hans Verkuil April 23, 2021, 9:44 a.m. UTC 
The following changes since commit 0b276e470a4d43e1365d3eb53c608a3d208cabd4:

  media: coda: fix macroblocks count control usage (2021-04-15 13:23:26 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.13-out3

for you to fetch changes up to 3d5a3ac0e5d35c0582fc87558027b25c95a143f8:

  staging: media: atomisp: Fix alignment and line length issues (2021-04-23 11:30:41 +0200)

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

----------------------------------------------------------------
Aline Santana Cordeiro (5):
      staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_cmd.c
      staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_compat_css20.c
      staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_subdev.c
      staging: media: atomisp: pci: Balance braces around conditional statements in file atomisp_v4l2.c
      staging: media: tegra-video: Align line break to match with the open parenthesis in file vi.c

Martiros Shakhzadyan (7):
      staging: media: atomisp: Fix sh_css.c brace coding style issues
      staging: media: atomisp: Remove redundant assertions in sh_css.c
      staging: media: atomisp: Fix the rest of sh_css.c brace issues
      staging: media: atomisp: Remove all redundant assertions in sh_css.c
      staging: media: atomisp: Remove a superfluous else clause in sh_css.c
      staging: media: atomisp: Replace if else clause with a ternary
      staging: media: atomisp: Fix alignment and line length issues

 drivers/staging/media/atomisp/pci/atomisp_cmd.c          |   19 +-
 drivers/staging/media/atomisp/pci/atomisp_compat_css20.c |    4 +-
 drivers/staging/media/atomisp/pci/atomisp_subdev.c       |    4 +-
 drivers/staging/media/atomisp/pci/atomisp_v4l2.c         |    4 +-
 drivers/staging/media/atomisp/pci/sh_css.c               | 1961 +++++++++++++++++++-----------------------
 drivers/staging/media/tegra-video/vi.c                   |    4 +-
 6 files changed, 918 insertions(+), 1078 deletions(-)

Comments

Jenkins April 23, 2021, 10 a.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/89431df2-8159-f45e-1aa5-4e9f0836c53d@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/100399/
Build time: 00:07:28
Link: https://lore.kernel.org/linux-media/89431df2-8159-f45e-1aa5-4e9f0836c53d@xs4all.nl

gpg: Signature made Fri 23 Apr 2021 09:37:44 AM 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 2/12 patches with issues, being 0 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0008-staging-media-atomisp-Fix-the-rest-of-sh_css.c-brace.patch:

   checkpatch.pl:
	$ cat patches/0008-staging-media-atomisp-Fix-the-rest-of-sh_css.c-brace.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:130: WARNING: please, no spaces at the start of a line
	-:141: CHECK: Unnecessary parentheses around '!pipe'
	-:141: CHECK: Unnecessary parentheses around '!pipe->stream'
	-:269: CHECK: Unnecessary parentheses around '!pipe'
	-:269: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_PREVIEW'
	-:278: CHECK: Unnecessary parentheses around 'fw == last_fw'
	-:278: CHECK: Unnecessary parentheses around 'fw->info.isp.sp.enable.out_frame  != 0'
	-:339: CHECK: Unnecessary parentheses around '!pipe'
	-:339: CHECK: Unnecessary parentheses around '!pipe->stream'
	-:377: CHECK: Unnecessary parentheses around '!pipe'
	-:377: CHECK: Unnecessary parentheses around '!pipe->stream'
	-:377: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_PREVIEW'
	-:469: CHECK: Unnecessary parentheses around '!pipe'
	-:469: CHECK: Unnecessary parentheses around '!pipe->stream'
	-:469: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_PREVIEW'
	-:529: CHECK: Unnecessary parentheses around '!pipe'
	-:529: CHECK: Unnecessary parentheses around '!buffer'
	-:582: CHECK: Unnecessary parentheses around 'queue_id <= SH_CSS_INVALID_QUEUE_ID'
	-:582: CHECK: Unnecessary parentheses around 'queue_id >= SH_CSS_MAX_NUM_QUEUES'
	-:627: CHECK: Logical continuations should be on the previous line
	-:637: CHECK: Unnecessary parentheses around '!h_vbuf'
	-:637: CHECK: Unnecessary parentheses around 'h_vbuf->vptr == 0x0'
	-:647: CHECK: Logical continuations should be on the previous line
	-:657: CHECK: Logical continuations should be on the previous line
	-:707: CHECK: Unnecessary parentheses around '!pipe'
	-:707: CHECK: Unnecessary parentheses around '!buffer'
	-:732: CHECK: Unnecessary parentheses around 'queue_id <= SH_CSS_INVALID_QUEUE_ID'
	-:732: CHECK: Unnecessary parentheses around 'queue_id >= SH_CSS_MAX_NUM_QUEUES'
	-:1197: CHECK: Unnecessary parentheses around '!pipe'
	-:1197: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_VIDEO'
	-:1283: WARNING: please, no spaces at the start of a line
	-:1293: WARNING: please, no spaces at the start of a line
	-:1303: WARNING: please, no spaces at the start of a line
	-:1313: WARNING: please, no spaces at the start of a line
	-:1331: WARNING: line length of 105 exceeds 100 columns
	-:1331: CHECK: Unnecessary parentheses around '!pipe'
	-:1331: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_CAPTURE'
	-:1331: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_COPY'
	-:1360: WARNING: please, no spaces at the start of a line
	-:1370: WARNING: please, no spaces at the start of a line
	-:1513: CHECK: Unnecessary parentheses around '!pipe'
	-:1513: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_YUVPP'
	-:1608: CHECK: Unnecessary parentheses around '!pipe'
	-:1608: CHECK: Unnecessary parentheses around '!pipe->stream'
	-:1608: CHECK: Unnecessary parentheses around 'pipe->mode != IA_CSS_PIPE_ID_YUVPP'
	-:1775: CHECK: Unnecessary parentheses around 'num_primary_stage == 0'
	-:1775: CHECK: Unnecessary parentheses around 'mode == IA_CSS_CAPTURE_MODE_PRIMARY'
	-:1880: WARNING: line length of 112 exceeds 100 columns
	-:1909: WARNING: please, no spaces at the start of a line
	-:2052: CHECK: Unnecessary parentheses around '!firmware'
	-:2052: CHECK: Unnecessary parentheses around '!pipe'
	-:2111: WARNING: please, no spaces at the start of a line
	-:2206: CHECK: Unnecessary parentheses around '!pipe'
	-:2206: CHECK: Unnecessary parentheses around '!config'
	-:2764: CHECK: Unnecessary parentheses around '!stream'
	-:2764: CHECK: Unnecessary parentheses around '!stream->last_pipe'

patches/0012-staging-media-atomisp-Fix-alignment-and-line-length-.patch:

   checkpatch.pl:
	$ cat patches/0012-staging-media-atomisp-Fix-alignment-and-line-length-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:135: CHECK: Unnecessary parentheses around 'buf_type == IA_CSS_BUFFER_TYPE_3A_STATISTICS'
	-:135: CHECK: Unnecessary parentheses around 'buf_type == IA_CSS_BUFFER_TYPE_DIS_STATISTICS'
	-:135: CHECK: Unnecessary parentheses around 'buf_type == IA_CSS_BUFFER_TYPE_LACE_STATISTICS'
	-:287: WARNING: line length of 107 exceeds 100 columns
	-:299: WARNING: line length of 110 exceeds 100 columns
	-:396: WARNING: line length of 107 exceeds 100 columns
	-:408: WARNING: line length of 110 exceeds 100 columns


Error #512 when building PDF docs