mbox series

[GIT,PULL,FOR,v6.4] saa7146: convert to vb2

Message ID ced694ce-8d93-282c-960f-927c60de1a26@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v6.4] saa7146: convert to vb2 | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.4o

Message

Hans Verkuil April 14, 2023, 11:35 a.m. UTC
Hi Mauro,

This PR converts saa7146 to vb2.

Please note that the first patch (media: common: saa7146: disable clipping)
must be picked up for 6.4 since it fixes a regression from the removal of the
overlay support.

The remainder is up to you whether you want take it of 6.4 or 6.5.

Regards,

	Hans

The following changes since commit f100ce3bbd6aa0093075b20b9dbd006686f6aedf:

  media: verisilicon: Fix crash when probing encoder (2023-04-13 12:58:55 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v6.4o

for you to fetch changes up to ebfa0c8cf135bddaee0384c8350181075809c795:

  staging: media: av7110: fix VBI output support (2023-04-14 12:48:25 +0200)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Hans Verkuil (18):
      media: common: saa7146: disable clipping
      common/saa7146: fix VFL direction for vbi output
      media: pci: saa7146: hexium_orion: initialize input 0
      media: saa7146: drop 'dev' and 'resources' from struct saa7146_fh
      media: common: saa7146: drop 'fmt' from struct saa7146_buf
      media: common: saa7146: replace BUG_ON by WARN_ON
      staging: media: av7110: replace BUG_ON by WARN_ON
      media: common: saa7146: fix broken V4L2_PIX_FMT_YUV422P support
      media: common: saa7146: use for_each_sg_dma_page
      media: saa7146: convert to vb2
      media: common: saa7146: fix compliance problems with field handling
      media: common: saa7146: check minimum video format size
      media: common: saa7146: allow S_STD(G_STD)
      media: mxb: update the tvnorms when changing input
      media: common: saa7146: add support for missing .vidioc_try_fmt_vbi_cap
      media: mxb: allow tuner/input/audio ioctls for vbi
      media: pci: saa7146: advertise only those TV standard that are supported
      staging: media: av7110: fix VBI output support

 drivers/media/common/saa7146/Kconfig         |   2 +-
 drivers/media/common/saa7146/saa7146_core.c  |  40 ++--
 drivers/media/common/saa7146/saa7146_fops.c  | 349 ++++++++--------------------------
 drivers/media/common/saa7146/saa7146_hlp.c   |  61 ++++--
 drivers/media/common/saa7146/saa7146_vbi.c   | 287 ++++++++++++----------------
 drivers/media/common/saa7146/saa7146_video.c | 637 ++++++++++++++++++++-------------------------------------------
 drivers/media/pci/saa7146/hexium_gemini.c    |  23 +--
 drivers/media/pci/saa7146/hexium_orion.c     |  24 +--
 drivers/media/pci/saa7146/mxb.c              |  53 +++---
 drivers/media/pci/ttpci/budget-av.c          |   4 +-
 drivers/staging/media/av7110/av7110.c        |   6 +-
 drivers/staging/media/av7110/av7110_hw.c     |   3 +-
 drivers/staging/media/av7110/av7110_v4l.c    | 131 +++++++------
 include/media/drv-intf/saa7146_vv.h          |  44 +----
 14 files changed, 592 insertions(+), 1072 deletions(-)

Comments

Jenkins April 14, 2023, 12:12 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/ced694ce-8d93-282c-960f-927c60de1a26@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/297637/
Build time: 00:28:24
Link: https://lore.kernel.org/linux-media/ced694ce-8d93-282c-960f-927c60de1a26@xs4all.nl

gpg: Signature made Fri 14 Apr 2023 11:30:12 AM 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 3/18 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-common-saa7146-disable-clipping.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-ov2680.c:415 ov2680_s_stream() warn: missing error code 'ret'
	../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:3357 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:3456 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/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2489 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: 3000016Kb sm_state_count = 1974493
	../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/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting
	../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

patches/0009-media-common-saa7146-use-for_each_sg_dma_page.patch:

   checkpatch.pl:
	$ cat patches/0009-media-common-saa7146-use-for_each_sg_dma_page.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:136: CHECK: spaces preferred around that '*' (ctx:ExV)

patches/0018-staging-media-av7110-fix-VBI-output-support.patch:

   checkpatch.pl:
	$ cat patches/0018-staging-media-av7110-fix-VBI-output-support.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:96: CHECK: Alignment should match open parenthesis
	-:114: CHECK: Avoid CamelCase: <wssMode>
	-:115: CHECK: Avoid CamelCase: <wssData>
	-:121: CHECK: Avoid CamelCase: <SetWSSConfig>


Error #512 when building PDF docs