Message ID | 20230802082733.21862-1-stanimir.k.varbanov@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Delegated to: | Hans Verkuil |
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 1qR7Cs-008nx1-KV; Wed, 02 Aug 2023 08:27:46 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233943AbjHBI1o (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Wed, 2 Aug 2023 04:27:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45094 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233941AbjHBI1n (ORCPT <rfc822;linux-media@vger.kernel.org>); Wed, 2 Aug 2023 04:27:43 -0400 Received: from mail-ed1-x536.google.com (mail-ed1-x536.google.com [IPv6:2a00:1450:4864:20::536]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F1129E48 for <linux-media@vger.kernel.org>; Wed, 2 Aug 2023 01:27:41 -0700 (PDT) Received: by mail-ed1-x536.google.com with SMTP id 4fb4d7f45d1cf-51a52a7d859so1439126a12.0 for <linux-media@vger.kernel.org>; Wed, 02 Aug 2023 01:27:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1690964860; x=1691569660; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=f9CvipQeFtHOcLEbAWVrFmu35ePajL0eMH9tb1EZJHI=; b=fgufiO1de7SuiJ95JmK92r6fssprswiDfX319XUgAZnl+lhFFa0R6z7sVOJR3yPPcY LF/ulR48W2E9Soi2XtcNXdRtOVNZFDog46Pw5h6xpJcXQ8qJyFwffal5st3l1RBhNAhW w9WvcmkYQtwAp20gQqzm9N7R6pAimyx+JL2kHMfKs3dHUw6a507SkyQFXVGwqMcIP/jl EDQIZ7RneDiFPKeAOHPV8DfyXlq4WOWjvS8O+eUOGqHkfDsykcYABiUNH7SwlBGuXLj6 28u6Psm0RYjggmxvQcNT4KhCtbv72OzaVOIsOlldSpNA14bvzi8bMzc/xGs2RK4xbf0H hRJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690964860; x=1691569660; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=f9CvipQeFtHOcLEbAWVrFmu35ePajL0eMH9tb1EZJHI=; b=I0Xq67SBG2cBsk1fun7HSkA9ie19EETHIWsnQdk+Ng7DQJblOwvbu2PNPWtd8fPNM3 tXJ6hxFo4FPlrmuuCn5dbFR26FdkG0y/PiUzARY1b7BYNFdc1t35lTTfWGHOcEnbKLv9 By8b2GlGICrDmvhgunMov4KlKzxd3+SGoszCMov3L3U9xY0GEsuk/KfICHkxDu9fO0Dn hqyt+RzfuM3VszlA0RFeNhy/D31QFk8gkkqtaWie942PhMMWhu3KGpVJGGrBvXiXiXmd pmH8gtLbB339lD7Hgx4S0z06JBFrrv0nL5C0i1YFEU4VqoX48gJU1eJjF7+H7YduoZ4D Y4cA== X-Gm-Message-State: ABy/qLYwBhS5VmWDkPmp61/QQQbi3p6+qRWzkgzKlohjvOYjL4Y3EUrL /gLMTubTGMh3RAMGmBla50vduPuem98= X-Google-Smtp-Source: APBJJlFsmeD1LzX/AtIeCTzzyHRDy8bCa9uILobzUQqiWJfeoMqlj41FYJvGmDZGWf2aevw6KXHX9Q== X-Received: by 2002:a17:906:64ce:b0:99b:484b:21b4 with SMTP id p14-20020a17090664ce00b0099b484b21b4mr6499480ejn.8.1690964860361; Wed, 02 Aug 2023 01:27:40 -0700 (PDT) Received: from aero-laptop.. (62-73-122-38.ip.btc-net.bg. [62.73.122.38]) by smtp.gmail.com with ESMTPSA id y26-20020a170906071a00b009737b8d47b6sm8667052ejb.203.2023.08.02.01.27.39 for <linux-media@vger.kernel.org> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Aug 2023 01:27:39 -0700 (PDT) From: Stanimir Varbanov <stanimir.k.varbanov@gmail.com> To: linux-media@vger.kernel.org Subject: [GIT PULL FOR v6.6] Venus updates Date: Wed, 2 Aug 2023 11:27:33 +0300 Message-Id: <20230802082733.21862-1-stanimir.k.varbanov@gmail.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_BL_SPAMCOP_NET,RCVD_IN_DNSWL_BLOCKED,RCVD_IN_SBL_CSS, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: ** 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.2 (++) X-LSpam-Report: No, score=2.2 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,FREEMAIL_FORGED_FROMDOMAIN=0.001,FREEMAIL_FROM=0.001,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,RCVD_IN_BL_SPAMCOP_NET=1.347,RCVD_IN_DNSWL_NONE=-0.0001,RCVD_IN_SBL_CSS=3.335 autolearn=no autolearn_force=no |
Series |
[GIT,PULL,FOR,v6.6] Venus updates
|
|
Pull-request
git://linuxtv.org/svarbanov/media_tree.git tags/tag-venus-for-v6.6Message
Stanimir Varbanov
Aug. 2, 2023, 8:27 a.m. UTC
Hi Mauro, This time the pull request includes following changes: - A long patchset from Konrad to start differentiating venus by VPU (Video Processing Unit) instead of HFI firmware interface versions - this will allow us to start the support for IRIS1 and AR50 Lite. - Few fixes. Please pull. regards, Stan The following changes since commit 06c2afb862f9da8dc5efa4b6076a0e48c3fbaaa5: Linux 6.5-rc1 (2023-07-09 13:53:13 -0700) are available in the Git repository at: git://linuxtv.org/svarbanov/media_tree.git tags/tag-venus-for-v6.6 for you to fetch changes up to 210fefeb11b4bf5d4c5597f126425c2d3fea1aa9: media: venus: core: Set up secure memory ranges for SC7180 (2023-08-02 10:39:44 +0300) ---------------------------------------------------------------- Venus updates for v6.6 ---------------------------------------------------------------- Konrad Dybcio (18): media: venus: hfi_venus: Only consider sys_idle_indicator on V1 media: venus: hfi_venus: Write to VIDC_CTRL_INIT after unmasking interrupts media: venus: Introduce VPU version distinction media: venus: Add vpu_version to most SoCs media: venus: firmware: Leave a clue about obtaining CP VARs media: venus: hfi_venus: Sanitize venus_boot_core() per-VPU-version media: venus: core: Assign registers based on VPU version media: venus: hfi_venus: Sanitize venus_halt_axi() per-VPU-version media: venus: hfi_venus: Sanitize venus_isr() per-VPU-version media: venus: hfi_venus: Sanitize venus_cpu_and_video_core_idle() per-VPU-version media: venus: hfi_venus: Sanitize venus_cpu_idle_and_pc_ready() per-VPU-version media: venus: firmware: Sanitize per-VPU-version media: venus: hfi_platform: Check vpu_version instead of device compatible media: venus: vdec: Sanitize vdec_set_work_route() per-VPU-version media: venus: Introduce accessors for remapped hfi_buffer_reqs members media: venus: Use newly-introduced hfi_buffer_requirements accessors media: venus: hfi_venus: Restrict writing SCIACMDARG3 to Venus V1/V2 media: venus: core: Set up secure memory ranges for SC7180 drivers/media/platform/qcom/venus/core.c | 11 +++- drivers/media/platform/qcom/venus/core.h | 15 ++++++ drivers/media/platform/qcom/venus/firmware.c | 18 +++++-- drivers/media/platform/qcom/venus/helpers.c | 7 +-- drivers/media/platform/qcom/venus/hfi_helper.h | 61 +++++++++++++++++++--- drivers/media/platform/qcom/venus/hfi_msgs.c | 2 +- .../media/platform/qcom/venus/hfi_plat_bufs_v6.c | 22 ++++---- drivers/media/platform/qcom/venus/hfi_platform.c | 2 +- drivers/media/platform/qcom/venus/hfi_venus.c | 42 +++++++-------- drivers/media/platform/qcom/venus/vdec.c | 10 ++-- drivers/media/platform/qcom/venus/vdec_ctrls.c | 2 +- drivers/media/platform/qcom/venus/venc.c | 4 +- drivers/media/platform/qcom/venus/venc_ctrls.c | 2 +- 13 files changed, 137 insertions(+), 61 deletions(-)
Comments
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20230802082733.21862-1-stanimir.k.varbanov@gmail.com/
Build log: https://builder.linuxtv.org/job/patchwork/327285/
Build time: 00:26:07
Link: https://lore.kernel.org/linux-media/20230802082733.21862-1-stanimir.k.varbanov@gmail.com
gpg: Signature made Wed 02 Aug 2023 07:40:32 AM UTC
gpg: using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [expired]
gpg: Note: This key has expired!
Primary key fingerprint: 34CF E039 8A16 AD93 18FD D5E8 A6D0 26D8 E358 14D4
Subkey fingerprint: E155 8C24 97CE 3CCC 2B5A A30F 25B5 5FC8 1B70 35F2
Summary: got 1/18 patches with issues, being 1 at build time
Error/warnings:
patches/0001-media-venus-hfi_venus-Only-consider-sys_idle_indicat.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
../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret'
../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468)
../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret'
../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto?
../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto?
../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto?
allyesconfig: return code #0:
../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v1'.
../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v2'.
../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'.
Locked on : 326,387
Unlocked on: 465,467
../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_local_data'.
../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_input_ctrl'.
../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_pin_sts'.
../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'.
SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1971102
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() warn: Function too hairy. No more merges.
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: __split_smt: function too hairy. Giving up after 54 seconds
../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687.
../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2772 mxc_jpeg_probe() warn: missing unwind goto?
../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h):
SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...):
SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression
../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2890 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting
checkpatch.pl:
$ cat patches/0001-media-venus-hfi_venus-Only-consider-sys_idle_indicat.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:17: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)