Em Fri, 30 Apr 2021 12:11:22 +0200 Hans Verkuil <hverkuil@xxxxxxxxx> 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@xxxxxxxx/T/#ma9ded10c22aaa86de74babf1bd2a2320c3dac70b The remaining ones were applied. Thanks! Mauro