* [GIT PULL FOR v5.14] More Outreachy cleanups (v2)
@ 2021-04-30 10:11 Hans Verkuil
2021-04-30 10:30 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) (#73995) Jenkins
2021-05-17 13:57 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Mauro Carvalho Chehab
0 siblings, 2 replies; 3+ messages in thread
From: Hans Verkuil @ 2021-04-30 10:11 UTC (permalink / raw)
To: Linux Media Mailing List
Cc: Aline Santana Cordeiro, Deepak R Varma, Martiros Shakhzadyan
This supersedes an earlier PR. This v2 adds more patches from Martiros and
Deepak.
Regards,
Hans
The following changes since commit 6756ae29420f849de4dc5c41f9fe948450b09285:
media: staging: media: atomisp: pci: Format comments according to coding-style in file atomisp_cmd.h (2021-04-21 14:07:01 +0200)
are available in the Git repository at:
git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.14-out1
for you to fetch changes up to 68dc2d762b637dd980f8dcc9e762da8cb788b57e:
staging: media: atomisp: remove unwanted dev_*() calls (2021-04-30 11:49:40 +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
Deepak R Varma (8):
staging: media: atomisp: balance braces around if...else block
staging: media: atomisp: remove unnecessary braces
staging: media: atomisp: use __func__ over function names
staging: media: atomisp: reformat code comment blocks
staging: media: atomisp: fix CamelCase variable naming
staging: media: atomisp: replace raw pr_*() by dev_dbg()
staging: media: atomisp: remove unnecessary pr_info calls
staging: media: atomisp: remove unwanted dev_*() calls
Martiros Shakhzadyan (9):
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
staging: media: atomisp: Refactor ia_css_stream_load()
staging: media: atomisp: Fix line split style issues
drivers/staging/media/atomisp/i2c/atomisp-gc0310.c | 49 +-
drivers/staging/media/atomisp/i2c/atomisp-gc2235.c | 25 +-
drivers/staging/media/atomisp/i2c/atomisp-libmsrlisthelper.c | 6 +-
drivers/staging/media/atomisp/i2c/atomisp-lm3554.c | 2 +-
drivers/staging/media/atomisp/i2c/atomisp-mt9m114.c | 108 +-
drivers/staging/media/atomisp/i2c/atomisp-ov2680.c | 41 +-
drivers/staging/media/atomisp/i2c/mt9m114.h | 6 +-
drivers/staging/media/atomisp/i2c/ov2680.h | 10 +-
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 | 2089 ++++++++++++++++++--------------------
drivers/staging/media/tegra-video/vi.c | 4 +-
14 files changed, 1100 insertions(+), 1271 deletions(-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [GIT PULL FOR v5.14] More Outreachy cleanups (v2) (#73995)
2021-04-30 10:11 [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Hans Verkuil
@ 2021-04-30 10:30 ` Jenkins
2021-05-17 13:57 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Mauro Carvalho Chehab
1 sibling, 0 replies; 3+ messages in thread
From: Jenkins @ 2021-04-30 10:30 UTC (permalink / raw)
To: mchehab+samsung, linux-media; +Cc: builder
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/706ba93e-fa1c-34ce-8c54-d8a1c442811e@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/102415/
Build time: 00:09:24
Link: https://lore.kernel.org/linux-media/706ba93e-fa1c-34ce-8c54-d8a1c442811e@xs4all.nl
gpg: Signature made Fri 30 Apr 2021 10:00:43 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 3/22 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
-:281: WARNING: line length of 107 exceeds 100 columns
-:293: WARNING: line length of 110 exceeds 100 columns
-:390: WARNING: line length of 107 exceeds 100 columns
-:402: WARNING: line length of 110 exceeds 100 columns
patches/0017-staging-media-atomisp-use-__func__-over-function-nam.patch:
checkpatch.pl:
$ cat patches/0017-staging-media-atomisp-use-__func__-over-function-nam.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:30: WARNING: Unnecessary ftrace-like logging - prefer using ftrace
-:43: WARNING: Unnecessary ftrace-like logging - prefer using ftrace
-:119: WARNING: Unnecessary ftrace-like logging - prefer using ftrace
Error #512 when building PDF docs
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [GIT PULL FOR v5.14] More Outreachy cleanups (v2)
2021-04-30 10:11 [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Hans Verkuil
2021-04-30 10:30 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) (#73995) Jenkins
@ 2021-05-17 13:57 ` Mauro Carvalho Chehab
1 sibling, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2021-05-17 13:57 UTC (permalink / raw)
To: Hans Verkuil
Cc: Linux Media Mailing List, Aline Santana Cordeiro, Deepak R Varma,
Martiros Shakhzadyan
Em Fri, 30 Apr 2021 12:11:22 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> This supersedes an earlier PR. This v2 adds more patches from Martiros and
> Deepak.
>
> Regards,
>
> Hans
>
> The following changes since commit 6756ae29420f849de4dc5c41f9fe948450b09285:
>
> media: staging: media: atomisp: pci: Format comments according to coding-style in file atomisp_cmd.h (2021-04-21 14:07:01 +0200)
>
> are available in the Git repository at:
>
> git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.14-out1
>
> for you to fetch changes up to 68dc2d762b637dd980f8dcc9e762da8cb788b57e:
>
> staging: media: atomisp: remove unwanted dev_*() calls (2021-04-30 11:49:40 +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
>
> Deepak R Varma (8):
> staging: media: atomisp: balance braces around if...else block
> staging: media: atomisp: remove unnecessary braces
> staging: media: atomisp: use __func__ over function names
> staging: media: atomisp: reformat code comment blocks
> staging: media: atomisp: fix CamelCase variable naming
> staging: media: atomisp: replace raw pr_*() by dev_dbg()
> staging: media: atomisp: remove unnecessary pr_info calls
> staging: media: atomisp: remove unwanted dev_*() calls
>
> Martiros Shakhzadyan (9):
> 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
> staging: media: atomisp: Refactor ia_css_stream_load()
> staging: media: atomisp: Fix line split style issues
I ended not applying two patches from this series:
- staging: media: atomisp: use __func__ over function names
Dan requested some changes on it, as it has several dev_dbg() calls
used just for tracing. I sent a reply with a few additional issues;
- staging: media: atomisp: remove unwanted dev_*() calls
This one didn't apply. Is it touching the same lines the first patch
changed? If so, please merge both into a single one, after applying
the fixes I mentioned on my e-mail:
https://lore.kernel.org/linux-media/20210517154448.7bb8be17@coco.lan/T/#ma9ded10c22aaa86de74babf1bd2a2320c3dac70b
The remaining ones were applied.
Thanks!
Mauro
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-05-17 13:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-04-30 10:11 [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Hans Verkuil
2021-04-30 10:30 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) (#73995) Jenkins
2021-05-17 13:57 ` [GIT PULL FOR v5.14] More Outreachy cleanups (v2) Mauro Carvalho Chehab
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.