| Message ID | YBBZzXcB8R1xd3Hm@pendragon.ideasonboard.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,v5.12] UVC driver changes | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/YBBZzXcB8R1xd3Hm@pendragon.ideasonboard.com/
Build log: https://builder.linuxtv.org/job/patchwork/87535/
Build time: 00:06:54
Link: https://lore.kernel.org/linux-media/YBBZzXcB8R1xd3Hm@pendragon.ideasonboard.com
gpg: Signature made Tue 26 Jan 2021 06:03:22 PM UTC
gpg: using RSA key CB9D6877529820CD53099B1B65F89C37BC54210D
gpg: issuer "laurent.pinchart@ideasonboard.com"
gpg: Can't check signature: No public key
Summary: got 7/13 patches with issues, being 0 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0004-media-uvcvideo-Allow-extra-entities.patch:
checkpatch.pl:
$ cat patches/0004-media-uvcvideo-Allow-extra-entities.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:31: CHECK: Alignment should match open parenthesis
patches/0007-media-uvcvideo-Allow-entity-defined-get_info-and-get.patch:
checkpatch.pl:
$ cat patches/0007-media-uvcvideo-Allow-entity-defined-get_info-and-get.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:36: CHECK: Alignment should match open parenthesis
patches/0008-media-uvcvideo-Implement-UVC_EXT_GPIO_UNIT.patch:
checkpatch.pl:
$ cat patches/0008-media-uvcvideo-Implement-UVC_EXT_GPIO_UNIT.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:31: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)
patches/0010-media-uvcvideo-Use-dev_-printk-aliases.patch:
checkpatch.pl:
$ cat patches/0010-media-uvcvideo-Use-dev_-printk-aliases.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:387: CHECK: Macro argument 'flag' may be better as '(flag)' to avoid precedence issues
-:390: WARNING: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(... to printk(KERN_DEBUG ...
-:393: CHECK: Macro argument reuse '_dev' - possible side-effects?
patches/0011-media-uvcvideo-New-macro-uvc_trace_cont.patch:
checkpatch.pl:
$ cat patches/0011-media-uvcvideo-New-macro-uvc_trace_cont.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:163: CHECK: Macro argument 'flag' may be better as '(flag)' to avoid precedence issues
-:166: WARNING: Avoid logging continuation uses where feasible
patches/0012-media-uvcvideo-use-dev_printk-for-uvc_trace.patch:
checkpatch.pl:
$ cat patches/0012-media-uvcvideo-use-dev_printk-for-uvc_trace.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:820: CHECK: Alignment should match open parenthesis
-:1332: CHECK: Macro argument 'flag' may be better as '(flag)' to avoid precedence issues
-:1336: WARNING: Prefer dev_dbg(... to dev_printk(KERN_DEBUG, ...
patches/0013-media-uvcvideo-Rename-debug-functions.patch:
checkpatch.pl:
$ cat patches/0013-media-uvcvideo-Rename-debug-functions.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:975: CHECK: spaces preferred around that '|' (ctx:VxV)
-:975: WARNING: Symbolic permissions 'S_IRUGO|S_IWUSR' are not preferred. Consider using octal permissions '0644'.
-:1065: CHECK: Avoid CamelCase: <bValue>
-:1072: CHECK: Avoid CamelCase: <bEvent>
-:1090: CHECK: Avoid CamelCase: <bSelector>
-:1509: CHECK: Prefer using the BIT macro
-:1510: CHECK: Prefer using the BIT macro
-:1511: CHECK: Prefer using the BIT macro
-:1512: CHECK: Prefer using the BIT macro
-:1513: CHECK: Prefer using the BIT macro
-:1514: CHECK: Prefer using the BIT macro
-:1515: CHECK: Prefer using the BIT macro
-:1516: CHECK: Prefer using the BIT macro
-:1517: CHECK: Prefer using the BIT macro
-:1518: CHECK: Prefer using the BIT macro
-:1519: CHECK: Prefer using the BIT macro
-:1520: CHECK: Prefer using the BIT macro
Error #512 when building PDF docs
Hi Mauro, The following changes since commit 066a94e28a23e04c0e9cb293f9ead56d409d7e41: media: dt-bindings: media: Use graph and video-interfaces schemas (2021-01-22 22:40:15 +0100) are available in the Git repository at: git://linuxtv.org/pinchartl/media.git tags/uvc-next-20210126 for you to fetch changes up to 9a71d1f0628599823d9233e62aa2fa17d5322013: media: uvcvideo: Rename debug functions (2021-01-26 19:07:18 +0200) ---------------------------------------------------------------- UVC changes for v5.12: - UVC privacy GPIO support - Fix format/size setting regression with some devices - Support 1bcf:0b40 ---------------------------------------------------------------- Joe Perches (1): media: uvcvideo: Rename debug functions Laurent Pinchart (2): media: uvcvideo: Force UVC version to 1.0a for 1bcf:0b40 media: uvcvideo: Accept invalid bFormatIndex and bFrameIndex values Ricardo Ribalda (10): media: uvcvideo: Move guid to entity media: uvcvideo: Allow extra entities media: uvcvideo: Allow entities with no pads media: uvcvideo: Provide sync and async uvc_ctrl_status_event media: uvcvideo: Allow entity-defined get_info and get_cur media: uvcvideo: Implement UVC_EXT_GPIO_UNIT media: uvcvideo: Add Privacy control based on EXT_GPIO media: uvcvideo: Use dev_ printk aliases media: uvcvideo: New macro uvc_trace_cont media: uvcvideo: use dev_printk() for uvc_trace() drivers/media/usb/uvc/uvc_ctrl.c | 179 ++++++------ drivers/media/usb/uvc/uvc_driver.c | 552 ++++++++++++++++++++++++------------- drivers/media/usb/uvc/uvc_entity.c | 11 +- drivers/media/usb/uvc/uvc_isight.c | 17 +- drivers/media/usb/uvc/uvc_queue.c | 9 +- drivers/media/usb/uvc/uvc_status.c | 44 +-- drivers/media/usb/uvc/uvc_v4l2.c | 62 +++-- drivers/media/usb/uvc/uvc_video.c | 162 +++++------ drivers/media/usb/uvc/uvcvideo.h | 95 +++++-- 9 files changed, 680 insertions(+), 451 deletions(-)