Message ID | 7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl (mailing list archive) |
---|---|
State | Accepted |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1oCMqs-00C697-AR; Fri, 15 Jul 2022 15:03:35 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232055AbiGOPDc (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 15 Jul 2022 11:03:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44598 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232133AbiGOPDa (ORCPT <rfc822;linux-media@vger.kernel.org>); Fri, 15 Jul 2022 11:03:30 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D79ED7BE1A for <linux-media@vger.kernel.org>; Fri, 15 Jul 2022 08:03:29 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 7319961FD8 for <linux-media@vger.kernel.org>; Fri, 15 Jul 2022 15:03:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 49BE0C34115; Fri, 15 Jul 2022 15:03:28 +0000 (UTC) Message-ID: <7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl> Date: Fri, 15 Jul 2022 17:03:26 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Content-Language: en-US To: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org> Cc: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>, =?utf-8?q?Jernej_=C5=A0?= =?utf-8?q?krabec?= <jernej.skrabec@gmail.com> From: Hans Verkuil <hverkuil@xs4all.nl> Subject: [GIT PULL FOR v5.20] v3: hantro/cedrus/vb2 updates Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-6.7 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.4 (--) X-LSpam-Report: No, score=-2.4 required=5.0 tests=BAYES_00=-1.9,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
[GIT,PULL,FOR,v5.20] v3: hantro/cedrus/vb2 updates
|
|
Pull-request
git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20jMessage
Hans Verkuil
July 15, 2022, 3:03 p.m. UTC
This PR sits on top of the stateless HEVC uAPI series: https://patchwork.linuxtv.org/project/linux-media/patch/8409c576-e2f3-489a-23df-dc16386aac13@xs4all.nl/ This series adds hantro 10 bit encoding, various cedrus fixes, and the series replacing vb2_find_timestamp with vb2_find_buffer. Changes since v2: dropped "cedrus: Use vb2_find_buffer" and "videobuf2: Remove vb2_find_timestamp()" since the first cedrus patch introduced a regression. This can be added later. Changes since v1: added the last three patches of the cedrus series (https://patchwork.linuxtv.org/project/linux-media/list/?series=8237) after Ezequiel reviewed those as well. Regards, Hans The following changes since commit bb0b9dfa3ba6b403d2f8ce39a024668e6b6e63c7: media: uapi: move HEVC stateless controls out of staging (2022-07-15 16:49:13 +0200) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20j for you to fetch changes up to 185b34bc5169424c65f3e7aa742cb96f955ebe42: rkvdec: Use vb2_find_buffer (2022-07-15 16:49:14 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Ezequiel Garcia (7): media: Add P010 tiled format videobuf2: Introduce vb2_find_buffer() mediatek: vcodec: Use vb2_find_buffer tegra-vde: Use vb2_find_buffer vicodec: Use vb2_find_buffer hantro: Use vb2_find_buffer rkvdec: Use vb2_find_buffer Jernej Skrabec (13): media: hantro: Support format filtering by depth media: hantro: postproc: Fix buffer size calculation media: hantro: postproc: Fix legacy regs configuration media: hantro: postproc: Properly calculate chroma offset media: hantro: Store VP9 bit depth in context media: hantro: sunxi: Enable 10-bit decoding media: cedrus: h265: Fix flag name media: cedrus: h265: Fix logic for not low delay flag media: cedrus: Improve error messages for controls media: cedrus: Add error handling for failed setup media: cedrus: h265: Add a couple of error checks media: cedrus: Add helper for determining number of elements media: cedrus: h265: Implement support for tiles Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst | 18 ++++++-- drivers/media/common/videobuf2/videobuf2-v4l2.c | 12 +++++ drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.c | 7 ++- drivers/media/platform/mediatek/vcodec/vdec/vdec_vp8_req_if.c | 7 ++- drivers/media/platform/mediatek/vcodec/vdec/vdec_vp9_req_lat_if.c | 8 ++-- drivers/media/platform/nvidia/tegra-vde/h264.c | 9 ++-- drivers/media/test-drivers/vicodec/vicodec-core.c | 8 +--- drivers/media/v4l2-core/v4l2-common.c | 1 + drivers/media/v4l2-core/v4l2-ioctl.c | 1 + drivers/staging/media/hantro/hantro.h | 4 ++ drivers/staging/media/hantro/hantro_drv.c | 29 ++++++++++-- drivers/staging/media/hantro/hantro_g2_vp9_dec.c | 18 +++----- drivers/staging/media/hantro/hantro_postproc.c | 38 ++++++++++++---- drivers/staging/media/hantro/hantro_v4l2.c | 50 +++++++++++++++++++-- drivers/staging/media/hantro/hantro_v4l2.h | 3 ++ drivers/staging/media/hantro/sunxi_vpu_hw.c | 27 +++++++++++ drivers/staging/media/rkvdec/rkvdec-h264.c | 41 ++++++----------- drivers/staging/media/rkvdec/rkvdec-vp9.c | 10 ++--- drivers/staging/media/sunxi/cedrus/cedrus.c | 28 +++++++++++- drivers/staging/media/sunxi/cedrus/cedrus.h | 7 ++- drivers/staging/media/sunxi/cedrus/cedrus_dec.c | 27 ++++++++--- drivers/staging/media/sunxi/cedrus/cedrus_h264.c | 5 ++- drivers/staging/media/sunxi/cedrus/cedrus_h265.c | 154 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----- drivers/staging/media/sunxi/cedrus/cedrus_mpeg2.c | 4 +- drivers/staging/media/sunxi/cedrus/cedrus_regs.h | 3 +- drivers/staging/media/sunxi/cedrus/cedrus_vp8.c | 5 ++- include/media/videobuf2-v4l2.h | 10 +++++ include/uapi/linux/videodev2.h | 1 + 28 files changed, 421 insertions(+), 114 deletions(-)
Comments
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/225090/
Build time: 00:00:00
Link: https://lore.kernel.org/linux-media/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl
Build aborted due to a fatal error:
FAILED: probably the tree doesn't exist
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/225499/
Build time: 00:46:47
Link: https://lore.kernel.org/linux-media/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl
gpg: Signature made Fri 15 Jul 2022 02:49:56 PM UTC
gpg: using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg: aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]
Summary: got 4/20 patches with issues, being 2 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-Add-P010-tiled-format.patch:
allyesconfig: return code #0:
../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr]
../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr]
../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples
allyesconfig: return code #0:
../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5495 cx23885_dif_setup() parse error: turning off implications after 60 seconds
SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
SPARSE:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: warning: array of flexible structures
../drivers/media/dvb-frontends/sp887x.c: ../drivers/media/dvb-frontends/sp887x.c:178 sp887x_initial_setup() error: __memcpy() '&buf[2]' too small (30 vs 16384)
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000028Kb sm_state_count = 1725742
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy. No more merges.
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy. Giving up after 50 seconds
../drivers/media/test-drivers/vimc/vimc-core.c: ../drivers/media/test-drivers/vimc/vimc-core.c:214 vimc_create_links() warn: passing a valid pointer to 'PTR_ERR'
../drivers/media/platform/sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c:193 sun6i_mipi_csi2_s_stream() warn: missing error code 'ret'
../drivers/media/platform/sunxi/sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c:225 sun8i_a83t_mipi_csi2_s_stream() warn: missing error code 'ret'
../drivers/media/usb/uvc/uvc_ctrl.c: ../drivers/media/usb/uvc/uvc_ctrl.c:2472 uvc_ctrl_init_ctrl() error: we previously assumed 'mapping' could be null (see line 2458)
../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2884 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
checkpatch.pl:
$ cat patches/0001-media-Add-P010-tiled-format.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:73: WARNING: line length of 165 exceeds 100 columns
-:85: ERROR: trailing statements should be on next line
-:97: WARNING: line length of 107 exceeds 100 columns
patches/0014-media-cedrus-h265-Implement-support-for-tiles.patch:
checkpatch.pl:
$ cat patches/0014-media-cedrus-h265-Implement-support-for-tiles.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:72: CHECK: Alignment should match open parenthesis
-:74: CHECK: Alignment should match open parenthesis
patches/0015-videobuf2-Introduce-vb2_find_buffer.patch:
allyesconfig: return code #0:
../drivers/media/usb/uvc/uvc_ctrl.c: ../drivers/media/usb/uvc/uvc_ctrl.c:2472 uvc_ctrl_init_ctrl() error: we previously assumed 'mapping' could be null (see line 2458)
../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2858 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000024Kb sm_state_count = 1725741
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy. No more merges.
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy. Giving up after 52 seconds
patches/0020-rkvdec-Use-vb2_find_buffer.patch:
checkpatch.pl:
$ cat patches/0020-rkvdec-Use-vb2_find_buffer.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:57: CHECK: Comparison to NULL could be written "run->ref_buf[ref->index]"
Error #512 when building PDF docs