[GIT,PULL,FOR,v5.10] DVB & RC fixes (v2)

Message ID 20200830122609.GA22247@gofer.mess.org (mailing list archive)
State Accepted, archived
Headers
Series [GIT,PULL,FOR,v5.10] DVB & RC fixes (v2) |

Pull-request

git://linuxtv.org/syoung/media_tree.git tags/v5.10b

Message

Sean Young Aug. 30, 2020, 12:26 p.m. UTC
  Hi Mauro,

I've tested most of my IR hardware to make sure this patch set does not
regress.

Thanks,
Sean

The following changes since commit 07d999f0d9463ea0d1f76e0f8921ea363d805767:

  media: open.rst: document mc-centric and video-node-centric (2020-08-29 10:41:03 +0200)

are available in the Git repository at:

  git://linuxtv.org/syoung/media_tree.git tags/v5.10b

for you to fetch changes up to ee96e0b01023305386e8e6912c5ef47febce069f:

  media: rc: rename ir_lirc_* functions to lirc_* (2020-08-30 12:30:32 +0100)

----------------------------------------------------------------
v5.10b

----------------------------------------------------------------
Alex Bee (1):
      media: dvb_usb_pctv452e: use ISL6423 voltage regulator per default

Jason Yan (1):
      media: mxl5xx: remove unused including <linux/version.h>

Sean Young (3):
      media: rc: rename lirc char dev region to "lirc"
      media: rc: harmonize infrared durations to microseconds
      media: rc: rename ir_lirc_* functions to lirc_*

Tom Rix (1):
      media: dvb-frontends/cxd2099: report errors

 drivers/hid/hid-picolcd_cir.c              | 10 ++--
 drivers/media/cec/core/cec-core.c          |  2 +-
 drivers/media/cec/platform/seco/seco-cec.c |  2 +-
 drivers/media/common/siano/smsir.c         |  4 +-
 drivers/media/dvb-frontends/cxd2099.c      | 14 +++--
 drivers/media/dvb-frontends/mxl5xx.c       |  1 -
 drivers/media/i2c/cx25840/cx25840-ir.c     |  2 +-
 drivers/media/pci/cx23885/cx23888-ir.c     |  2 +-
 drivers/media/pci/cx88/cx88-input.c        |  4 +-
 drivers/media/pci/smipcie/smipcie-ir.c     |  7 +--
 drivers/media/rc/ene_ir.c                  | 18 +++---
 drivers/media/rc/fintek-cir.c              |  8 +--
 drivers/media/rc/igorplugusb.c             |  6 +-
 drivers/media/rc/iguanair.c                |  4 +-
 drivers/media/rc/imon_raw.c                |  2 +-
 drivers/media/rc/ir-hix5hd2.c              |  8 +--
 drivers/media/rc/ir-imon-decoder.c         | 10 ++--
 drivers/media/rc/ir-jvc-decoder.c          |  6 +-
 drivers/media/rc/ir-mce_kbd-decoder.c      |  8 +--
 drivers/media/rc/ir-nec-decoder.c          |  6 +-
 drivers/media/rc/ir-rc5-decoder.c          |  6 +-
 drivers/media/rc/ir-rc6-decoder.c          |  6 +-
 drivers/media/rc/ir-rcmm-decoder.c         | 18 +++---
 drivers/media/rc/ir-sanyo-decoder.c        |  6 +-
 drivers/media/rc/ir-sharp-decoder.c        |  6 +-
 drivers/media/rc/ir-sony-decoder.c         |  6 +-
 drivers/media/rc/ir-xmp-decoder.c          | 15 ++---
 drivers/media/rc/ir_toy.c                  | 14 ++---
 drivers/media/rc/ite-cir.c                 | 10 ++--
 drivers/media/rc/ite-cir.h                 |  4 +-
 drivers/media/rc/lirc_dev.c                | 95 ++++++++++++++----------------
 drivers/media/rc/mceusb.c                  | 12 ++--
 drivers/media/rc/meson-ir.c                |  4 +-
 drivers/media/rc/mtk-cir.c                 |  4 +-
 drivers/media/rc/nuvoton-cir.c             | 10 ++--
 drivers/media/rc/nuvoton-cir.h             |  2 +-
 drivers/media/rc/rc-core-priv.h            | 21 ++++---
 drivers/media/rc/rc-ir-raw.c               | 16 ++---
 drivers/media/rc/rc-loopback.c             |  8 +--
 drivers/media/rc/rc-main.c                 | 10 ++--
 drivers/media/rc/redrat3.c                 | 17 +++---
 drivers/media/rc/serial_ir.c               | 10 ++--
 drivers/media/rc/sir_ir.c                  |  2 +-
 drivers/media/rc/st_rc.c                   |  6 +-
 drivers/media/rc/streamzap.c               | 10 +---
 drivers/media/rc/sunxi-cir.c               |  4 +-
 drivers/media/rc/ttusbir.c                 | 18 +++---
 drivers/media/rc/winbond-cir.c             | 10 ++--
 drivers/media/rc/xbox_remote.c             |  2 +-
 drivers/media/usb/au0828/au0828-input.c    |  8 +--
 drivers/media/usb/dvb-usb-v2/rtl28xxu.c    |  4 +-
 drivers/media/usb/dvb-usb/Kconfig          |  1 +
 drivers/media/usb/dvb-usb/pctv452e.c       | 26 ++++++--
 drivers/media/usb/dvb-usb/technisat-usb2.c |  4 +-
 include/media/rc-core.h                    | 11 ++--
 55 files changed, 265 insertions(+), 265 deletions(-)
  

Comments

Jenkins Aug. 30, 2020, 1:58 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20200830122609.GA22247@gofer.mess.org/
Build log: https://builder.linuxtv.org/job/patchwork/66226/
Build time: 01:88:16
Link: https://lore.kernel.org/linux-media/20200830122609.GA22247@gofer.mess.org

gpg: Signature made Sun 30 Aug 2020 12:16:46 PM UTC
gpg:                using RSA key A624251A26084A9ED9E4C8B6425F639D3960FA9E
gpg:                issuer "sean@mess.org"
gpg: Good signature from "Sean Young <sean@mess.org>" [full]

Summary: got 6/6 patches with issues, being 0 at build time and one fatal error when buinding PDF document

Error/warnings:


Error #512 when building PDF docs
  
Jenkins Aug. 31, 2020, 5:04 p.m. UTC | #2
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20200830122609.GA22247@gofer.mess.org/
Build log: https://builder.linuxtv.org/job/patchwork/66330/
Build time: 00:23:45
Link: https://lore.kernel.org/linux-media/20200830122609.GA22247@gofer.mess.org

gpg: Signature made Sun 30 Aug 2020 12:16:46 PM UTC
gpg:                using RSA key A624251A26084A9ED9E4C8B6425F639D3960FA9E
gpg:                issuer "sean@mess.org"
gpg: Good signature from "Sean Young <sean@mess.org>" [full]

Summary: got 2/6 patches with issues, being 2 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-mxl5xx-remove-unused-including-linux-version.h.patch:

    allyesconfig: return code #0:
	  struct gdc_warp_param_mem_s *isp_data_ptr;

    allyesconfig: return code #0:
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:3204 video_put_user() warn: check that 'ev32' doesn't leak information (struct has a hole after 'type')
	../drivers/media/v4l2-core/v4l2-ioctl.c: ../drivers/media/v4l2-core/v4l2-ioctl.c:3229 video_put_user() warn: check that 'vb32' doesn't leak information (struct has a hole after 'memory')
	../drivers/media/tuners/qt1010.c: ../drivers/media/tuners/qt1010.c:239 qt1010_init_meas1() error: uninitialized symbol 'val2'.
	../drivers/media/tuners/qt1010.c: ../drivers/media/tuners/qt1010.c:273 qt1010_init_meas2() error: uninitialized symbol 'val'.
	../drivers/media/dvb-frontends/tda10021.c: ../drivers/media/dvb-frontends/tda10021.c:153 tda10021_set_symbolrate() warn: unsigned 'symbolrate' is never less than zero.
	../drivers/media/dvb-frontends/tda10021.c: ../drivers/media/dvb-frontends/tda10021.c:155 tda10021_set_symbolrate() warn: unsigned 'symbolrate' is never less than zero.
	../drivers/media/dvb-frontends/tda10021.c: ../drivers/media/dvb-frontends/tda10021.c:157 tda10021_set_symbolrate() warn: unsigned 'symbolrate' is never less than zero.
	../drivers/media/dvb-frontends/tda10021.c: ../drivers/media/dvb-frontends/tda10021.c:159 tda10021_set_symbolrate() warn: unsigned 'symbolrate' is never less than zero.
	../drivers/media/v4l2-core/videobuf-dma-sg.c: ../drivers/media/v4l2-core/videobuf-dma-sg.c:245 videobuf_dma_init_kernel() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/pci/ttpci/av7110_v4l.c: ../drivers/media/pci/ttpci/av7110_v4l.c:163 ves1820_set_tv_freq() warn: unsigned 'freq' is never less than zero.
	../drivers/media/pci/ttpci/av7110_v4l.c: ../drivers/media/pci/ttpci/av7110_v4l.c:165 ves1820_set_tv_freq() warn: unsigned 'freq' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:300 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:303 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:306 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:309 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:312 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:315 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:318 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:321 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:324 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/dvb-frontends/tda10086.c: ../drivers/media/dvb-frontends/tda10086.c:327 tda10086_set_symbol_rate() warn: unsigned 'symbol_rate' is never less than zero.
	../drivers/media/rc/serial_ir.c: ../drivers/media/rc/serial_ir.c:550 serial_ir_probe() warn: should '8 << ioshift' be a 64 bit type?
	SPARSE:../drivers/media/platform/qcom/venus/hfi_venus.c ../drivers/media/platform/qcom/venus/hfi_venus.c:133:5:  warning: symbol 'venus_fw_debug' was not declared. Should it be static?
	../drivers/media/pci/cx23885/cx23885-alsa.c: ../drivers/media/pci/cx23885/cx23885-alsa.c:83 cx23885_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/pci/cx25821/cx25821-alsa.c: ../drivers/media/pci/cx25821/cx25821-alsa.c:146 cx25821_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/radio/wl128x/fmdrv_common.c: ../drivers/media/radio/wl128x/fmdrv_common.c:736 fm_irq_handle_rdsdata_getcmd_resp() warn: potential spectre issue 'rds_fmt.data.groupdatabuff.buff' [w]
	../drivers/media/pci/cx88/cx88-alsa.c: ../drivers/media/pci/cx88/cx88-alsa.c:286 cx88_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/pci/saa7134/saa7134-tvaudio.c: ../drivers/media/pci/saa7134/saa7134-tvaudio.c:686 saa_dsp_writel() warn: should 'reg << 2' be a 64 bit type?
	../drivers/media/pci/saa7134/saa7134-alsa.c: ../drivers/media/pci/saa7134/saa7134-alsa.c:267 saa7134_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2841 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1200 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1460 vivid_create_instance() parse error: __split_smt: function too hairy.  Giving up after 308 seconds

patches/0005-media-rc-harmonize-infrared-durations-to-microsecond.patch:

    allyesconfig: return code #0:
	../drivers/media/rc/serial_ir.c: ../drivers/media/rc/serial_ir.c:550 serial_ir_probe() warn: should '8 << ioshift' be a 64 bit type?
	../drivers/media/pci/saa7134/saa7134-tvaudio.c: ../drivers/media/pci/saa7134/saa7134-tvaudio.c:686 saa_dsp_writel() warn: should 'reg << 2' be a 64 bit type?
	../drivers/media/pci/cx88/cx88-alsa.c: ../drivers/media/pci/cx88/cx88-alsa.c:286 cx88_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/pci/saa7134/saa7134-alsa.c: ../drivers/media/pci/saa7134/saa7134-alsa.c:267 saa7134_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/pci/cx23885/cx23885-alsa.c: ../drivers/media/pci/cx23885/cx23885-alsa.c:83 cx23885_alsa_dma_init() warn: should 'nr_pages << 12' be a 64 bit type?
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2841 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1200 vivid_create_instance() parse error: turning off implications after 60 seconds
	../drivers/media/test-drivers/vivid/vivid-core.c: ../drivers/media/test-drivers/vivid/vivid-core.c:1456 vivid_create_instance() parse error: __split_smt: function too hairy.  Giving up after 304 seconds


Error #512 when building PDF docs
  
Mauro Carvalho Chehab Sept. 3, 2020, 2:20 p.m. UTC | #3
Em Sun, 30 Aug 2020 13:26:10 +0100
Sean Young <sean@mess.org> escreveu:

> Hi Mauro,
> 
> I've tested most of my IR hardware to make sure this patch set does not
> regress.
> 
> Thanks,
> Sean
> 
> The following changes since commit 07d999f0d9463ea0d1f76e0f8921ea363d805767:
> 
>   media: open.rst: document mc-centric and video-node-centric (2020-08-29 10:41:03 +0200)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/syoung/media_tree.git tags/v5.10b
> 
> for you to fetch changes up to ee96e0b01023305386e8e6912c5ef47febce069f:
> 
>   media: rc: rename ir_lirc_* functions to lirc_* (2020-08-30 12:30:32 +0100)
> 
> ----------------------------------------------------------------
> v5.10b
> 
> ----------------------------------------------------------------
> Alex Bee (1):
>       media: dvb_usb_pctv452e: use ISL6423 voltage regulator per default
> 
> Jason Yan (1):
>       media: mxl5xx: remove unused including <linux/version.h>
> 
> Sean Young (3):
>       media: rc: rename lirc char dev region to "lirc"
>       media: rc: harmonize infrared durations to microseconds
>       media: rc: rename ir_lirc_* functions to lirc_*

Patches merged, thanks!

> Tom Rix (1):
>       media: dvb-frontends/cxd2099: report errors

I'm not sure if this patch is ok. So, I opted to not merge
this one for now.

Thanks,
Mauro