| Message ID | ZJIVvXtDlP2OtS3X@valkosipuli.retiisi.eu (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,6.6] V4L2 patches of all kind plus more | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/ZJIVvXtDlP2OtS3X@valkosipuli.retiisi.eu/
Build log: https://builder.linuxtv.org/job/patchwork/316365/
Build time: 00:39:06
Link: https://lore.kernel.org/linux-media/ZJIVvXtDlP2OtS3X@valkosipuli.retiisi.eu
gpg: Signature made Tue 20 Jun 2023 08:55:21 PM UTC
gpg: using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg: issuer "sakari.ailus@linux.intel.com"
gpg: Good signature from "Sakari Ailus <sakari.ailus@linux.intel.com>" [full]
Summary: got 13/56 patches with issues, being 4 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-cec-ch7322-drop-of_match_ptr-for-ID-table.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
SPARSE:../drivers/staging/media/tegra-video/vip.c ../drivers/staging/media/tegra-video/vip.c:280:24: warning: symbol 'tegra_vip_driver' was not declared. Should it be static?
../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/i2c/atomisp-gc0310.c:447 gc0310_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?
../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:4283 atomisp_set_fmt_to_snr() warn: variable dereferenced before check 'asd' (see line 4268)
allyesconfig: return code #0:
../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto?
../drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c: ../drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c:347 vdec_msg_queue_init() warn: missing unwind goto?
../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2775 mxc_jpeg_probe() warn: missing unwind goto?
../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/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: OOM: 3000028Kb sm_state_count = 1963540
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() warn: Function too hairy. No more merges.
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2570 dvb_register() parse error: __split_smt: function too hairy. Giving up after 54 seconds
SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
../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/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/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2878 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
patches/0026-media-dt-bindings-cadence-csi2rx-Convert-to-DT-schem.patch:
checkpatch.pl:
$ cat patches/0026-media-dt-bindings-cadence-csi2rx-Convert-to-DT-schem.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:29: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
-:131: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
patches/0031-media-ipu3-cio2-rename-cio2-bridge-to-ipu-bridge-and.patch:
checkpatch.pl:
$ cat patches/0031-media-ipu3-cio2-rename-cio2-bridge-to-ipu-bridge-and.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
-:188: CHECK: Lines should not end with a '('
-:213: CHECK: Lines should not end with a '('
-:218: CHECK: Lines should not end with a '('
-:270: CHECK: Lines should not end with a '('
patches/0033-media-ipu3-cio2-rename-ipu3-cio2-main.c-back-to-ipu3.patch:
checkpatch.pl:
$ cat patches/0033-media-ipu3-cio2-rename-ipu3-cio2-main.c-back-to-ipu3.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:14: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0034-media-MAINTAINERS-Assign-Shawn-Tu-s-sensor-drivers-t.patch:
checkpatch.pl:
$ cat patches/0034-media-MAINTAINERS-Assign-Shawn-Tu-s-sensor-drivers-t.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:14: WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report
patches/0035-media-MAINTAINERS-Add-an-entry-for-V4L2-sensor-and-l.patch:
checkpatch.pl:
$ cat patches/0035-media-MAINTAINERS-Add-an-entry-for-V4L2-sensor-and-l.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:35: WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
-:36: WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order
patches/0048-dt-bindings-i2c-Add-I2C-Address-Translator-ATR.patch:
checkpatch.pl:
$ cat patches/0048-dt-bindings-i2c-Add-I2C-Address-Translator-ATR.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:20: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0050-dt-bindings-media-add-TI-DS90UB913-FPD-Link-III-Seri.patch:
checkpatch.pl:
$ cat patches/0050-dt-bindings-media-add-TI-DS90UB913-FPD-Link-III-Seri.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0051-dt-bindings-media-add-TI-DS90UB953-FPD-Link-III-Seri.patch:
checkpatch.pl:
$ cat patches/0051-dt-bindings-media-add-TI-DS90UB953-FPD-Link-III-Seri.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0052-dt-bindings-media-add-TI-DS90UB960-FPD-Link-III-Dese.patch:
checkpatch.pl:
$ cat patches/0052-dt-bindings-media-add-TI-DS90UB960-FPD-Link-III-Dese.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0053-media-i2c-add-DS90UB960-driver.patch:
allyesconfig: return code #0:
../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1671 ub960_rxport_add_serializer() error: 'rxport->ser.client' dereferencing possible ERR_PTR()
../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'.
checkpatch.pl:
$ cat patches/0053-media-i2c-add-DS90UB960-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:54: WARNING: please write a help paragraph that fully describes the config symbol
-:1949: CHECK: Alignment should match open parenthesis
-:2575: CHECK: Alignment should match open parenthesis
-:2578: CHECK: Alignment should match open parenthesis
-:2579: WARNING: line length of 101 exceeds 100 columns
-:2694: CHECK: Lines should not end with a '('
-:2721: CHECK: Lines should not end with a '('
-:2777: CHECK: Lines should not end with a '('
patches/0054-media-i2c-add-DS90UB913-driver.patch:
allyesconfig: return code #0:
../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:475 ub913_log_status() error: uninitialized symbol 'v'.
../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'.
checkpatch.pl:
$ cat patches/0054-media-i2c-add-DS90UB913-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:27: WARNING: please write a help paragraph that fully describes the config symbol
-:56: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
-:673: CHECK: Alignment should match open parenthesis
patches/0055-media-i2c-add-DS90UB953-driver.patch:
allyesconfig: return code #0:
../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'.
checkpatch.pl:
$ cat patches/0055-media-i2c-add-DS90UB953-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:27: WARNING: please write a help paragraph that fully describes the config symbol
-:56: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
Error #512 when building PDF docs
Hi Mauro, Here's a large number of V4L2 patches for 6.6, in particular: - Rename of the CIO2 bridge as IPU bridge (expect to be used with IPU6). - OV13B10 driver fixes. - Miscellaneous fixes (V4L2 fwnode) as well as drivers. - External PHY support for cadence CSI2RX. - Removal of of_match_ptr() macro patches from Krzysztof. - I²C address translator support plus the dependent FPD-link drivers. - MAINTAINERS changes. Please pull. The following changes since commit be9aac187433af6abba5fcc2e73d91d0794ba360: Merge tag 'v6.4-rc5' into media_stage (2023-06-09 10:12:41 +0100) are available in the Git repository at: git://linuxtv.org/sailus/media_tree.git tags/for-6.6-1.4-signed for you to fetch changes up to 08784f9591c8a894046b489a8b456c39cf8caa22: media: v4l2-core: Fix a potential resource leak in v4l2_fwnode_parse_link() (2023-06-20 23:54:15 +0300) ---------------------------------------------------------------- V4L2 patches for 6.6 ---------------------------------------------------------------- Bingbu Cao (6): media: ipu3-cio2: rename cio2 bridge to ipu bridge and move out of ipu3 media: ipu-bridge: use IPU_MAX_PORTS for bridge instead of CIO2_NUM_PORTS media: ipu3-cio2: rename ipu3-cio2-main.c back to ipu3-cio2.c media: ov13b10: Defer probe if no endpoint found media: ov13b10: support new ACPI HID 'OVTI13B1' media: ov13b10: add PM control support based on power resources Christophe JAILLET (1): media: v4l2-core: Fix a potential resource leak in v4l2_fwnode_parse_link() Claudiu Beznea (1): media: i2c: tvp5150: check return value of devm_kasprintf() Guoniu.zhou (1): media: ov5640: fix low resolution image abnormal issue Hans de Goede (1): media: ad5820: Drop unsupported ad5823 from i2c_ and of_device_id tables Jack Zhu (5): media: dt-bindings: cadence-csi2rx: Convert to DT schema media: dt-bindings: cadence-csi2rx: Add resets property media: cadence: Add operation on reset media: cadence: Add support for external dphy media: cadence: Add support for JH7110 SoC Jason Chen (1): media: ov08x40: Fix hblank out of range issue Krzysztof Kozlowski (25): media: cec: ch7322: drop of_match_ptr for ID table media: cec: meson: drop of_match_ptr for ID table media: cec: tegra: drop of_match_ptr for ID table media: rc: gpio-ir-recv: drop of_match_ptr for ID table media: rc: gpio-ir-tx: drop of_match_ptr for ID table media: rc: ir-rx51: drop of_match_ptr for ID table media: platform: allegro-dvt: drop of_match_ptr for ID table media: platform: intel: pxa: drop of_match_ptr for ID table media: platform: samsung: s5p-jpeg: drop of_match_ptr for ID table media: platform: ti: am437x: drop of_match_ptr for ID table media: platform: verisilicon: drop of_match_ptr for ID table media: platform: marvell: drop of_match_ptr for ID table media: platform: mdp3: drop of_match_ptr for ID table media: platform: mdp3: mark OF related data as maybe unused media: platform: sun6i-csi: drop of_match_ptr for ID table media: platform: sun6i-mipi-csi2: drop of_match_ptr for ID table media: platform: sun8i-a83t-mipi-csi2: drop of_match_ptr for ID table media: platform: mdp: mark OF related data as maybe unused media: platform: jpeg: always reference OF data media: i2c: isl7998x: drop of_match_ptr for ID table media: i2c: mt9m111: drop of_match_ptr for ID table media: i2c: ov2640: drop of_match_ptr for ID table media: i2c: ov2680: drop of_match_ptr for ID table media: i2c: ov7740: drop of_match_ptr for ID table media: i2c: max9286: drop of_match_ptr for ID table Luca Ceresoli (1): i2c: add I2C Address Translator (ATR) support Nicolas Ferre (1): MAINTAINERS: Orphan the OV7740 driver Sakari Ailus (5): media: MAINTAINERS: Assign Shawn Tu's sensor drivers to myself media: MAINTAINERS: Add an entry for V4L2 sensor and lens drivers media: MAINTAINERS: Orphan dw9768 and ov02a10 drivers media: MAINTAINERS: Pick ov5670 maintenance media: i2c: Remove Shawn's and Chiranjeevi's e-mail addresses Tomi Valkeinen (7): dt-bindings: i2c: Add I2C Address Translator (ATR) dt-bindings: media: add TI DS90UB913 FPD-Link III Serializer dt-bindings: media: add TI DS90UB953 FPD-Link III Serializer dt-bindings: media: add TI DS90UB960 FPD-Link III Deserializer media: i2c: add DS90UB960 driver media: i2c: add DS90UB913 driver media: i2c: add DS90UB953 driver Tommaso Merciai (1): media: i2c: imx290: drop format param from imx290_ctrl_update Documentation/devicetree/bindings/i2c/i2c-atr.yaml | 34 + .../devicetree/bindings/media/cdns,csi2rx.txt | 100 - .../devicetree/bindings/media/cdns,csi2rx.yaml | 201 + .../bindings/media/i2c/ti,ds90ub913.yaml | 133 + .../bindings/media/i2c/ti,ds90ub953.yaml | 134 + .../bindings/media/i2c/ti,ds90ub960.yaml | 427 +++ Documentation/i2c/i2c-address-translators.rst | 96 + Documentation/i2c/index.rst | 1 + MAINTAINERS | 54 +- drivers/i2c/Kconfig | 9 + drivers/i2c/Makefile | 1 + drivers/i2c/i2c-atr.c | 710 ++++ drivers/media/cec/i2c/ch7322.c | 2 +- drivers/media/cec/platform/meson/ao-cec.c | 2 +- drivers/media/cec/platform/tegra/tegra_cec.c | 2 +- drivers/media/i2c/Kconfig | 47 + drivers/media/i2c/Makefile | 3 + drivers/media/i2c/ad5820.c | 2 - drivers/media/i2c/ds90ub913.c | 906 +++++ drivers/media/i2c/ds90ub953.c | 1400 +++++++ drivers/media/i2c/ds90ub960.c | 4051 ++++++++++++++++++++ drivers/media/i2c/hi556.c | 2 +- drivers/media/i2c/hi847.c | 2 +- drivers/media/i2c/imx208.c | 2 +- drivers/media/i2c/imx290.c | 7 +- drivers/media/i2c/imx319.c | 2 +- drivers/media/i2c/imx355.c | 2 +- drivers/media/i2c/isl7998x.c | 2 +- drivers/media/i2c/max9286.c | 2 +- drivers/media/i2c/mt9m111.c | 2 +- drivers/media/i2c/og01a1b.c | 2 +- drivers/media/i2c/ov08x40.c | 16 +- drivers/media/i2c/ov13858.c | 2 +- drivers/media/i2c/ov13b10.c | 129 +- drivers/media/i2c/ov2640.c | 2 +- drivers/media/i2c/ov2680.c | 2 +- drivers/media/i2c/ov2740.c | 2 +- drivers/media/i2c/ov5640.c | 7 +- drivers/media/i2c/ov5670.c | 2 +- drivers/media/i2c/ov5675.c | 2 +- drivers/media/i2c/ov7740.c | 2 +- drivers/media/i2c/tvp5150.c | 4 + drivers/media/pci/Kconfig | 2 +- drivers/media/pci/intel/Kconfig | 21 + drivers/media/pci/intel/Makefile | 4 +- .../pci/intel/{ipu3/cio2-bridge.c => ipu-bridge.c} | 179 +- .../pci/intel/{ipu3/cio2-bridge.h => ipu-bridge.h} | 57 +- drivers/media/pci/intel/ipu3/Kconfig | 19 - drivers/media/pci/intel/ipu3/Makefile | 3 - .../intel/ipu3/{ipu3-cio2-main.c => ipu3-cio2.c} | 4 +- drivers/media/pci/intel/ipu3/ipu3-cio2.h | 6 - drivers/media/platform/allegro-dvt/allegro-core.c | 2 +- drivers/media/platform/cadence/cdns-csi2rx.c | 107 +- drivers/media/platform/intel/pxa_camera.c | 2 +- drivers/media/platform/marvell/mmp-driver.c | 2 +- .../media/platform/mediatek/jpeg/mtk_jpeg_core.c | 6 +- .../media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c | 4 +- .../media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c | 4 +- drivers/media/platform/mediatek/mdp/mtk_mdp_core.c | 2 +- .../media/platform/mediatek/mdp3/mtk-mdp3-comp.c | 2 +- .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 2 +- .../media/platform/samsung/s5p-jpeg/jpeg-core.c | 2 +- drivers/media/platform/sunxi/sun6i-csi/sun6i_csi.c | 2 +- .../sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c | 2 +- .../sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c | 2 +- drivers/media/platform/ti/am437x/am437x-vpfe.c | 2 +- drivers/media/platform/verisilicon/hantro_drv.c | 2 +- drivers/media/rc/gpio-ir-recv.c | 2 +- drivers/media/rc/gpio-ir-tx.c | 2 +- drivers/media/rc/ir-rx51.c | 2 +- drivers/media/v4l2-core/v4l2-fwnode.c | 18 +- include/linux/i2c-atr.h | 116 + include/media/i2c/ds90ub9xx.h | 22 + 73 files changed, 8756 insertions(+), 356 deletions(-) create mode 100644 Documentation/devicetree/bindings/i2c/i2c-atr.yaml delete mode 100644 Documentation/devicetree/bindings/media/cdns,csi2rx.txt create mode 100644 Documentation/devicetree/bindings/media/cdns,csi2rx.yaml create mode 100644 Documentation/devicetree/bindings/media/i2c/ti,ds90ub913.yaml create mode 100644 Documentation/devicetree/bindings/media/i2c/ti,ds90ub953.yaml create mode 100644 Documentation/devicetree/bindings/media/i2c/ti,ds90ub960.yaml create mode 100644 Documentation/i2c/i2c-address-translators.rst create mode 100644 drivers/i2c/i2c-atr.c create mode 100644 drivers/media/i2c/ds90ub913.c create mode 100644 drivers/media/i2c/ds90ub953.c create mode 100644 drivers/media/i2c/ds90ub960.c create mode 100644 drivers/media/pci/intel/Kconfig rename drivers/media/pci/intel/{ipu3/cio2-bridge.c => ipu-bridge.c} (68%) rename drivers/media/pci/intel/{ipu3/cio2-bridge.h => ipu-bridge.h} (72%) rename drivers/media/pci/intel/ipu3/{ipu3-cio2-main.c => ipu3-cio2.c} (99%) create mode 100644 include/linux/i2c-atr.h create mode 100644 include/media/i2c/ds90ub9xx.h