From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 52C5FC2BB84 for ; Mon, 7 Sep 2020 10:19:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id F0A58207C3 for ; Mon, 7 Sep 2020 10:19:17 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=xs4all.nl header.i=@xs4all.nl header.b="OIZC81IM" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728687AbgIGKTO (ORCPT ); Mon, 7 Sep 2020 06:19:14 -0400 Received: from lb3-smtp-cloud8.xs4all.net ([194.109.24.29]:53327 "EHLO lb3-smtp-cloud8.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728819AbgIGKRk (ORCPT ); Mon, 7 Sep 2020 06:17:40 -0400 Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud8.xs4all.net with ESMTPA id FEDLktozZgJnQFEDMkcAgg; Mon, 07 Sep 2020 12:17:34 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s1; t=1599473854; bh=HgbncFYAzk/LirXAJ2xU3kPm9hShrC3K8k0R1FKZBag=; h=Subject:To:From:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=OIZC81IMuhwqFKUNoBTQKdWY+UP4jp8/nZPMcg1TbTvkPRBkNKpPVjQdEDsILM/1U N5vlSp5f2+jBagHguCCl8ygvXW1G5C/r7jqkmJQMyyd+IC6R0qbIcGTC6qVPvqYNAR gPfBVkRL/QmI4c1IIjIuCRvuUCx7bFs0ITqrYLkoMdYO6RyGQNdzCD3rW6hkA0m4Lx pqOBkjlbeK4i0rTyk0xnFGUYqCW8PaygVm7wvgq9XfAOYoV+ErPfwf1ssWs3P4BVZ1 S0l3704zui3deYbXCWGEv66L+Sa3UKTI4Mpy+auBZBWzlo8v+6RynmwWmjYRC1G3vF kkrrEwRN9UGvg== Subject: Re: [PATCH 00/38] media sparse/smatch warn fixes To: Mauro Carvalho Chehab Cc: Sean Young , Andrew Morton , Laurent Pinchart , linux-kernel@vger.kernel.org, Arnd Bergmann , =?UTF-8?Q?Niklas_S=c3=b6derlund?= , Nicolas Boichat , Stanimir Varbanov , Mike Rapoport , Michel Lespinasse , linux-media@vger.kernel.org, Allen Pais , Bjorn Andersson , Greg Kroah-Hartman , Sakari Ailus , Nathan Chancellor , Forest Crossman , Hans Verkuil , Colin Ian King , Antti Palosaari , Romain Perier , linux-arm-msm@vger.kernel.org, Andy Gross , Vandana BN , devel@driverdev.osuosl.org, Takashi Iwai , John Hubbard , Andy Shevchenko References: From: Hans Verkuil Message-ID: <5ced9e99-c3d9-e81f-db17-e13f33ed70e5@xs4all.nl> Date: Mon, 7 Sep 2020 12:17:31 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4wfOc6uePBCSHhyIyCiqDkng0D9SBU74ntHiDAyiBpwbwiPXns6jguVb2g4Yt6hvx4lTRj4RgBM4IbW6MyVr6PembmE5RwzGsuGoc7cTCRDZb6V8gMooxA EZlGkIsmIqfeIxltkTPpGt390a/cyTC4DCt/FC1zzFb+uFR8T7zdiaR9P1+pQ9d/r0NYKYirzqDYRlCRjZdprBEtIgiEO9pCkTTEZd5M4S2+jew7iQU5KuGP p755uCKj7M4THjQkj4Aq2kmBPsL6EhnfkLbJMSJ6mnMpx2Dha0u4BFjDYZzC2PPcducBWrhScvuDKHrlX5GfmEb/MwbMa38M0bz4za5A1vQxUTb6/n5lMNkE nbbbMygQw37lKUXlShZGbu0DR9viqI9z+9E9qbKUTPM375eWTE3sctetTr96Dp11wnHOhyHjyYgZZmih4DoPlHjsgLvfSd/8szDYRldVZzVjumql0EPaWf0z JHP9zbJulruaA72Q3dD9TAXAR1YwMK4UhP6EAiPa/M1Sfrl7OZZQMySnmk+Ka71w6FjDDvXoO7xtgzNvEfEDE4zxzpTzXUNLEO6N4zm82GN/g2+2jDPcObLo TMCbV7bgA5eUQQf1h4kRLGScaoVp22A+7y/sBLcRha0VAu7okgpZYfaQmJBeRN75gOu69VV4qZRBkR/MIzGUxnNM2xMD77FeW3iqQnPCNGfxq/rw/rWNzGea u2/cYPnOyfQVl2Dh+HUs9WxN+g6b1dS6o8r3/xb41Osk+qasE8UlLLDo1815ogLizDJEy1HRC57yAmlkPqo8E12wFBdmAKxBQHkKP/BznwYYkX2xe49RhTDi bMUceJa98gLo1iYayGEdfaD2ITQ1V88vfi3EoTsqA8vYkSseWmd5jgcCV/7FS4QRsEthYkAnK1BSFSSncPpZ746DGVt40KDmT15eWzCdQxlGV8Soo6L5arh7 xfzH32ilsgG1c2KKRSRhelR9L+EWyBcIMPtltmyaGhRzoZlnAZtAMCuQ3qeEct2LW0z5Im5oQBm2NWGKLlDy/RUj4c8cfsmSo8IIWTq/1gwyrfEAwyEGW3JF WrslmCQiEgQAS0wR67beYHCv0jVibzB0pVr5raa3ob1SUklcVfUTachqVLuRG6kqaMxbTI9rmGiKrRy91glOINh7ETdWacM0I+1KQxVb9Fdv27lRKGTMFnpA P+rjPo66gFubD63ljV9FhJupctUq4mwSMRfODM8Pkhgsa0yNV07fis4+tPICUTLwVYHFCC59axC3eoM0CvN1Yw== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/09/2020 18:10, Mauro Carvalho Chehab wrote: > While I was lately relying on Jenkins to report smatch/sparse issues, > I discovered this week that the reports were missing such warnings > > I got it fixed already at builder.linuxtv.org, but there are several new > warnings that got added over time. > > This series is meant to solve the ones that are reported on x86_64 > without pre-building the smatch database. > > It should allow receiving cleaner error reports. > > While most of the patches are meant to shut up the static analizers, > some contain real bug fixes. > > Mauro Carvalho Chehab (38): > media: tda10086: cleanup symbol_rate setting logic > media: v4l2-ioctl: avoid memory leaks on some time32 compat functions > media: qt1010: fix usage of unititialized value > media: av7110_v4l: avoid a typecast > media: wl128x: get rid of a potential spectre issue > media: venus: place extern venus_fw_debug on a header file > media: tda10021: avoid casts when using symbol_rate > media: serial_ir: use the right type for a dma address > media: vivid: move the detection part out of vivid_create_instance > media: vivid: place the logic which disables ioctl on a separate > function > media: vivid: move set_capabilities logic to a separate function > media: vivid: place dt timings init code on a separate function > media: vivid: move the create queues to a separate function > media: vivid: move the devnode creation logic to a separate function > media: vivid: fix error path For these vivid patches and after fixing the "unused variable 'i'" warning as reported by the kernel test robot: Reviewed-by: Hans Verkuil Thank you for working on this! Regards, Hans > media: videobuf-dma-sg: number of pages should be unsigned long > media: cx25821-alsa: number of pages should be unsigned long > media: cx23885-alsa: number of pages should be unsigned long > media: cx88-alsa: number of pages should be unsigned long > media: saa7134-alsa.c: number of pages should be unsigned long > media: dvb-ttusb-budget: don't use stack for USB transfers > media: dvb-ttusb-budget: cleanup printk logic > media: saa7134: avoid a shift overflow > media: atomisp: fix casts at atomisp_compat_ioctl32.c > media: atomisp: get rid of some unused code > media: atomisp: cleanup ifdefs from ia_css_debug.c > media: atomisp: get rid of version-dependent globals > media: atomisp: get rid of isys_dma.h and isys_dma_local.h > media: atomisp: get rid of ibuf_ctrl abstraction > media: atomisp: don't check for ISP version for includes > media: atomisp: unify INPUT error return type > media: atomisp: de-duplicate names at *_input_system_global.h > media: atomisp: reorder functions at pixelgen_private.h > media: atomisp: remove compile-time tests from input_system_global.h > media: atomisp: fix some bad indents > media: atomisp: csi_rx.c: add a missing includes > media: atomisp: atomisp_gmin_platform: check before use > media: atomisp: cleanup isys_irq headers > > drivers/media/dvb-frontends/tda10021.c | 38 +- > drivers/media/dvb-frontends/tda10086.c | 22 +- > drivers/media/pci/cx23885/cx23885-alsa.c | 7 +- > drivers/media/pci/cx23885/cx23885.h | 4 +- > drivers/media/pci/cx25821/cx25821-alsa.c | 11 +- > drivers/media/pci/cx88/cx88-alsa.c | 13 +- > drivers/media/pci/saa7134/saa7134-alsa.c | 7 +- > drivers/media/pci/saa7134/saa7134-tvaudio.c | 3 +- > drivers/media/pci/saa7134/saa7134.h | 2 +- > drivers/media/pci/ttpci/av7110_v4l.c | 4 +- > drivers/media/platform/qcom/venus/core.h | 2 + > drivers/media/platform/qcom/venus/dbgfs.c | 2 - > drivers/media/radio/wl128x/fmdrv_common.c | 16 +- > drivers/media/rc/serial_ir.c | 2 +- > drivers/media/test-drivers/vivid/vivid-core.c | 645 ++++++------ > drivers/media/tuners/qt1010.c | 25 +- > .../media/usb/ttusb-budget/dvb-ttusb-budget.c | 197 ++-- > drivers/media/v4l2-core/v4l2-ioctl.c | 48 +- > drivers/media/v4l2-core/videobuf-dma-sg.c | 22 +- > drivers/staging/media/atomisp/Makefile | 12 +- > .../atomisp/pci/atomisp_compat_ioctl32.c | 84 +- > .../media/atomisp/pci/atomisp_gmin_platform.c | 5 +- > .../atomisp/pci/css_2401_system/host/csi_rx.c | 1 + > .../pci/css_2401_system/host/ibuf_ctrl.c | 1 + > .../css_2401_system/host/ibuf_ctrl_local.h | 1 + > .../css_2401_system/host/ibuf_ctrl_private.h | 268 ----- > .../pci/css_2401_system/host/isys_dma.c | 10 +- > .../pci/css_2401_system/host/isys_dma_local.h | 21 - > .../css_2401_system/host/isys_dma_private.h | 12 +- > .../pci/css_2401_system/host/isys_irq.c | 3 +- > .../pci/css_2401_system/host/isys_irq_local.h | 4 +- > .../css_2401_system/host/isys_irq_private.h | 12 +- > .../css_2401_system/host/pixelgen_private.h | 73 +- > .../pci/css_2401_system/ibuf_ctrl_global.h | 6 +- > .../pci/css_2401_system/isys_irq_global.h | 4 +- > .../pci/css_2401_system/pixelgen_global.h | 8 +- > .../host/input_formatter.c | 2 +- > .../hive_isp_css_common/host/input_system.c | 70 +- > .../hive_isp_css_include/host/csi_rx_public.h | 4 +- > .../host/ibuf_ctrl_public.h | 94 -- > .../host/isys_dma_public.h | 8 +- > .../host/isys_irq_public.h | 29 +- > .../hive_isp_css_include/host/isys_public.h | 4 +- > .../host/pixelgen_public.h | 4 +- > .../pci/hive_isp_css_include/ibuf_ctrl.h | 47 - > .../pci/hive_isp_css_include/isys_dma.h | 47 - > .../pci/hive_isp_css_include/isys_irq.h | 16 +- > drivers/staging/media/atomisp/pci/hmm/hmm.c | 6 +- > .../staging/media/atomisp/pci/ia_css_mipi.h | 2 - > .../staging/media/atomisp/pci/ia_css_stream.h | 4 +- > .../media/atomisp/pci/input_system_global.h | 29 +- > .../kernels/ctc/ctc_1.0/ia_css_ctc_param.h | 7 - > .../ctc/ctc_1.0/ia_css_ctc_table.host.c | 145 --- > .../kernels/gc/gc_1.0/ia_css_gc_table.host.c | 144 --- > .../kernels/gc/gc_2/ia_css_gc2_table.host.c | 53 - > .../isp/kernels/raw/raw_1.0/ia_css_raw.host.c | 4 +- > .../kernels/xnr/xnr_1.0/ia_css_xnr_param.h | 7 - > .../xnr/xnr_1.0/ia_css_xnr_table.host.c | 31 - > .../atomisp/pci/isp2400_input_system_global.h | 21 +- > .../atomisp/pci/isp2400_input_system_local.h | 17 +- > .../atomisp/pci/isp2400_input_system_public.h | 26 +- > .../media/atomisp/pci/isp2400_system_global.h | 16 - > .../atomisp/pci/isp2401_input_system_global.h | 30 +- > .../atomisp/pci/isp2401_input_system_local.h | 3 - > .../pci/isp2401_input_system_private.h | 224 ++++- > .../media/atomisp/pci/isp2401_system_global.h | 19 - > .../atomisp/pci/runtime/binary/src/binary.c | 919 +++++++++--------- > .../runtime/bufq/interface/ia_css_bufq_comm.h | 10 - > .../media/atomisp/pci/runtime/bufq/src/bufq.c | 26 +- > .../runtime/debug/interface/ia_css_debug.h | 6 - > .../pci/runtime/debug/src/ia_css_debug.c | 193 +--- > .../atomisp/pci/runtime/ifmtr/src/ifmtr.c | 2 +- > .../pci/runtime/inputfifo/src/inputfifo.c | 10 - > .../pci/runtime/isys/interface/ia_css_isys.h | 18 +- > .../runtime/isys/interface/ia_css_isys_comm.h | 4 +- > .../pci/runtime/isys/src/csi_rx_rmgr.c | 2 +- > .../pci/runtime/isys/src/isys_dma_rmgr.c | 2 +- > .../atomisp/pci/runtime/isys/src/isys_init.c | 20 +- > .../runtime/isys/src/isys_stream2mmio_rmgr.c | 2 +- > .../media/atomisp/pci/runtime/isys/src/rx.c | 24 +- > .../pci/runtime/isys/src/virtual_isys.c | 58 +- > .../pipeline/interface/ia_css_pipeline.h | 2 +- > .../pci/runtime/pipeline/src/pipeline.c | 4 +- > drivers/staging/media/atomisp/pci/sh_css.c | 296 ++---- > .../staging/media/atomisp/pci/sh_css_defs.h | 4 - > .../staging/media/atomisp/pci/sh_css_hrt.c | 4 - > .../media/atomisp/pci/sh_css_internal.h | 36 +- > .../staging/media/atomisp/pci/sh_css_mipi.c | 34 +- > .../media/atomisp/pci/sh_css_param_shading.c | 2 +- > .../staging/media/atomisp/pci/sh_css_params.c | 538 ---------- > .../media/atomisp/pci/sh_css_properties.c | 10 - > drivers/staging/media/atomisp/pci/sh_css_sp.c | 39 +- > drivers/staging/media/atomisp/pci/sh_css_sp.h | 8 +- > .../staging/media/atomisp/pci/sh_css_struct.h | 2 - > .../staging/media/atomisp/pci/system_global.h | 23 - > include/media/videobuf-dma-sg.h | 2 +- > 96 files changed, 1708 insertions(+), 3300 deletions(-) > delete mode 100644 drivers/staging/media/atomisp/pci/css_2401_system/host/ibuf_ctrl_private.h > delete mode 100644 drivers/staging/media/atomisp/pci/css_2401_system/host/isys_dma_local.h > delete mode 100644 drivers/staging/media/atomisp/pci/hive_isp_css_include/host/ibuf_ctrl_public.h > delete mode 100644 drivers/staging/media/atomisp/pci/hive_isp_css_include/ibuf_ctrl.h > delete mode 100644 drivers/staging/media/atomisp/pci/hive_isp_css_include/isys_dma.h > delete mode 100644 drivers/staging/media/atomisp/pci/isp2400_system_global.h > delete mode 100644 drivers/staging/media/atomisp/pci/isp2401_system_global.h >