* [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance
@ 2019-09-27 14:27 Hans Verkuil
2019-09-27 14:44 ` Jenkins
2019-10-01 19:33 ` Mauro Carvalho Chehab
0 siblings, 2 replies; 3+ messages in thread
From: Hans Verkuil @ 2019-09-27 14:27 UTC (permalink / raw)
To: Linux Media Mailing List; +Cc: Benoit Parrot
Various fixes for am437x-vpfe.
One special note: the last three patches adds new macros to be able to
print a V4L2 fourcc in a standard way, both for kernel and userspace,
and uses them in v4l2-ioctl.c and am437x.
If you have concerns about this and do not want to merge those patches
without discussing this some more, then please just drop these last three
patches.
Regards,
Hans
The following changes since commit 6f51fdfd8229d5358c2d6e272cf73478866e8ddc:
media: videobuf-core.c: poll_wait needs a non-NULL buf pointer (2019-09-05 06:26:57 -0300)
are available in the Git repository at:
git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.5c2
for you to fetch changes up to 743d13880c0749ca61a40ec4c57ebeb60d06f9c6:
media: am437x-vpfe: Remove print_fourcc helper (2019-09-27 16:24:49 +0200)
----------------------------------------------------------------
Tag branch
----------------------------------------------------------------
Benoit Parrot (12):
media: am437x-vpfe: Fix missing first line
media: am437x-vpfe: Rework ISR routine for clarity
media: am437x-vpfe: Wait for end of frame before tear-down
media: am437x-vpfe: fix start streaming error path
media: am437x-vpfe: Streamlined vb2 buffer cleanup
media: am437x-vpfe: Setting STD to current value is not an error
media: am437x-vpfe: Use a per instance format array instead of a static one
media: am437x-vpfe: fix function trace debug log
media: am437x-vpfe: TRY_FMT ioctl is not really trying anything
media: am437x-vpfe: Remove per bus width static data
media: am437x-vpfe: Switch to SPDX Licensing
media: am437x-vpfe: Remove print_fourcc helper
Dave Gerlach (1):
media: am437x-vpfe: Fix suspend path to always handle pinctrl config
Hans Verkuil (1):
v4l2-ioctl.c: use new v4l2_fourcc_conv/args macros
Sakari Ailus (1):
v4l: Add macros for printing V4L fourcc values
Documentation/media/videodev2.h.rst.exceptions | 2 +
drivers/media/platform/am437x/am437x-vpfe.c | 880 +++++++++++++++++++++++++----------------------------------
drivers/media/platform/am437x/am437x-vpfe.h | 43 ++-
drivers/media/platform/am437x/am437x-vpfe_regs.h | 10 +-
drivers/media/v4l2-core/v4l2-ioctl.c | 58 ++--
include/uapi/linux/videodev2.h | 27 ++
6 files changed, 453 insertions(+), 567 deletions(-)
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance
2019-09-27 14:27 [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance Hans Verkuil
@ 2019-09-27 14:44 ` Jenkins
2019-10-01 19:33 ` Mauro Carvalho Chehab
1 sibling, 0 replies; 3+ messages in thread
From: Jenkins @ 2019-09-27 14:44 UTC (permalink / raw)
To: mchehab+samsung, linux-media; +Cc: builder
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/patch/58971/
Build log: https://builder.linuxtv.org/job/patchwork/17976/
Build time: 00:13:55
Link: https://lore.kernel.org/linux-media/95a1c6a9-636f-66f4-0360-66105f22af57@xs4all.nl
gpg: Signature made Fri 27 Sep 2019 02:25:10 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]
Summary: 3 patches and/or PDF generation with issues, being 0 at build time
Error/warnings:
patches/0013-v4l-Add-macros-for-printing-V4L-fourcc-values.patch:70: ERROR: Macros with complex values should be enclosed in parentheses
patches/0013-v4l-Add-macros-for-printing-V4L-fourcc-values.patch:70: CHECK: Macro argument reuse 'fourcc' - possible side-effects?
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0013-v4l-Add-macros-for-printing-V4L-fourcc-values.patch
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:24: WARNING: line over 80 characters
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:24: WARNING: Avoid logging continuation uses where feasible
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:39: WARNING: line over 80 characters
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:39: WARNING: Avoid logging continuation uses where feasible
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:54: WARNING: line over 80 characters
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:54: WARNING: Avoid logging continuation uses where feasible
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:73: WARNING: Avoid logging continuation uses where feasible
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:84: WARNING: Avoid logging continuation uses where feasible
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:102: WARNING: line over 80 characters
patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch:102: WARNING: Avoid logging continuation uses where feasible
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0014-v4l2-ioctl.c-use-new-v4l2_fourcc_conv-args-macros.patch
patches/0015-media-am437x-vpfe-Remove-print_fourcc-helper.patch:76: WARNING: line over 80 characters
patches/0015-media-am437x-vpfe-Remove-print_fourcc-helper.patch:87: WARNING: line over 80 characters
patches/0015-media-am437x-vpfe-Remove-print_fourcc-helper.patch:98: WARNING: line over 80 characters
patches/0015-media-am437x-vpfe-Remove-print_fourcc-helper.patch:109: WARNING: line over 80 characters
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0015-media-am437x-vpfe-Remove-print_fourcc-helper.patch
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance
2019-09-27 14:27 [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance Hans Verkuil
2019-09-27 14:44 ` Jenkins
@ 2019-10-01 19:33 ` Mauro Carvalho Chehab
1 sibling, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2019-10-01 19:33 UTC (permalink / raw)
To: Hans Verkuil; +Cc: Linux Media Mailing List, Benoit Parrot
Em Fri, 27 Sep 2019 16:27:00 +0200
Hans Verkuil <hverkuil-cisco@xs4all.nl> escreveu:
> Various fixes for am437x-vpfe.
>
> One special note: the last three patches adds new macros to be able to
> print a V4L2 fourcc in a standard way, both for kernel and userspace,
> and uses them in v4l2-ioctl.c and am437x.
>
> If you have concerns about this and do not want to merge those patches
> without discussing this some more, then please just drop these last three
> patches.
I looked at the patch with introduced the fourcc. While I like the idea,
IMHO, the implementation should be improved. Instead of adding obscure
subsystem-specific magic strings to be added at printk() lines, it should,
instead, add a new macro to be handled by printk, properly documenting
it at:
Documentation/core-api/printk-formats.rst
There are other subsystems with macros there, like the network subsystem.
So, I suspect that it shouldn't be hard to add something like "%pCC"
with would properly print the fourcc.
So, I'm dropping the last 3 patches on this series applying the
remaining ones.
Regards,
Mauro
>
> Regards,
>
> Hans
>
> The following changes since commit 6f51fdfd8229d5358c2d6e272cf73478866e8ddc:
>
> media: videobuf-core.c: poll_wait needs a non-NULL buf pointer (2019-09-05 06:26:57 -0300)
>
> are available in the Git repository at:
>
> git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.5c2
>
> for you to fetch changes up to 743d13880c0749ca61a40ec4c57ebeb60d06f9c6:
>
> media: am437x-vpfe: Remove print_fourcc helper (2019-09-27 16:24:49 +0200)
>
> ----------------------------------------------------------------
> Tag branch
>
> ----------------------------------------------------------------
> Benoit Parrot (12):
> media: am437x-vpfe: Fix missing first line
> media: am437x-vpfe: Rework ISR routine for clarity
> media: am437x-vpfe: Wait for end of frame before tear-down
> media: am437x-vpfe: fix start streaming error path
> media: am437x-vpfe: Streamlined vb2 buffer cleanup
> media: am437x-vpfe: Setting STD to current value is not an error
> media: am437x-vpfe: Use a per instance format array instead of a static one
> media: am437x-vpfe: fix function trace debug log
> media: am437x-vpfe: TRY_FMT ioctl is not really trying anything
> media: am437x-vpfe: Remove per bus width static data
> media: am437x-vpfe: Switch to SPDX Licensing
> media: am437x-vpfe: Remove print_fourcc helper
>
> Dave Gerlach (1):
> media: am437x-vpfe: Fix suspend path to always handle pinctrl config
>
> Hans Verkuil (1):
> v4l2-ioctl.c: use new v4l2_fourcc_conv/args macros
>
> Sakari Ailus (1):
> v4l: Add macros for printing V4L fourcc values
>
> Documentation/media/videodev2.h.rst.exceptions | 2 +
> drivers/media/platform/am437x/am437x-vpfe.c | 880 +++++++++++++++++++++++++----------------------------------
> drivers/media/platform/am437x/am437x-vpfe.h | 43 ++-
> drivers/media/platform/am437x/am437x-vpfe_regs.h | 10 +-
> drivers/media/v4l2-core/v4l2-ioctl.c | 58 ++--
> include/uapi/linux/videodev2.h | 27 ++
> 6 files changed, 453 insertions(+), 567 deletions(-)
Thanks,
Mauro
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-10-01 19:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-27 14:27 [GIT PULL FOR v5.5] am437x-vpfe: overdue maintenance Hans Verkuil
2019-09-27 14:44 ` Jenkins
2019-10-01 19:33 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).