mbox series

[GIT,PULL,FOR,v5.20] cedrus fix, hantro fixes and destaging

Message ID fe899b13-8e1c-4550-4b1a-a2b125e19cc9@xs4all.nl
State New
Headers show
Series [GIT,PULL,FOR,v5.20] cedrus fix, hantro fixes and destaging | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20m

Message

Hans Verkuil July 19, 2022, 9:34 a.m. UTC
Hi Mauro,

This PR has three cedrus/hantro fixes and destages hantro in the final patch.
That last patch just moves code into the new directory and has no other changes.

Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21.
It's just an improvement, and the cedrus changes are a bit larger than I'd like
so close to the code freeze. I'd like to have those tested first.

Regards,

	Hans

The following changes since commit 8bd1dbf8d580c425605fb8936309a4e9745a7a95:

  media: rkisp1: debug: Add dump file in debugfs for MI main path registers (2022-07-17 12:34:43 +0100)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20m

for you to fetch changes up to 10b516fc3544cc9057495d981e43df3953bd264d:

  hantro: Remove incorrect HEVC SPS validation (2022-07-19 11:14:41 +0200)

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

----------------------------------------------------------------
Ezequiel Garcia (3):
      media: destage Hantro VPU driver
      hantro: Remove dedicated control documentation
      hantro: Remove incorrect HEVC SPS validation

Jernej Skrabec (1):
      media: cedrus: hevc: Add check for invalid timestamp

 Documentation/userspace-api/media/drivers/hantro.rst                                      | 19 -------------------
 Documentation/userspace-api/media/drivers/index.rst                                       |  1 -
 MAINTAINERS                                                                               |  2 +-
 drivers/media/platform/Kconfig                                                            |  1 +
 drivers/media/platform/Makefile                                                           |  1 +
 drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig                      |  6 +++++-
 drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile                     |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h                     |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c                 | 12 ++++++------
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c                  |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c         |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c        |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h             |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c          |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c                  |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c         |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h             |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c          |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c         |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h             |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c                |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c                |  9 +--------
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h                  |  1 -
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c                |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h                |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c               |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c            |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c                |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h                |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c                 |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c                 |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h                 |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c               |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c  |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c  |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c   |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h         |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c            |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c            |  0
 drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c               |  0
 drivers/staging/media/Kconfig                                                             |  2 --
 drivers/staging/media/Makefile                                                            |  1 -
 drivers/staging/media/hantro/TODO                                                         |  2 --
 drivers/staging/media/sunxi/cedrus/cedrus_h265.c                                          |  3 +++
 45 files changed, 18 insertions(+), 42 deletions(-)
 delete mode 100644 Documentation/userspace-api/media/drivers/hantro.rst
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig (91%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c (99%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c (96%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h (99%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c (100%)
 rename drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c (100%)
 delete mode 100644 drivers/staging/media/hantro/TODO

Comments

Mauro Carvalho Chehab July 27, 2022, 6:35 p.m. UTC | #1
Em Tue, 19 Jul 2022 11:34:55 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> Hi Mauro,
> 
> This PR has three cedrus/hantro fixes and destages hantro in the final patch.
> That last patch just moves code into the new directory and has no other changes.
> 
> Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21.
> It's just an improvement, and the cedrus changes are a bit larger than I'd like
> so close to the code freeze. I'd like to have those tested first.

It sounds too late to merge it for the upcoming merge window.

So, I applied only patch 1/4:

	media: cedrus: hevc: Add check for invalid timestamp

Let's merge the remaining ones after -rc1.

Regards,
Mauro

> 
> Regards,
> 
> 	Hans
> 
> The following changes since commit 8bd1dbf8d580c425605fb8936309a4e9745a7a95:
> 
>   media: rkisp1: debug: Add dump file in debugfs for MI main path registers (2022-07-17 12:34:43 +0100)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20m
> 
> for you to fetch changes up to 10b516fc3544cc9057495d981e43df3953bd264d:
> 
>   hantro: Remove incorrect HEVC SPS validation (2022-07-19 11:14:41 +0200)
> 
> ----------------------------------------------------------------
> Tag branch
> 
> ----------------------------------------------------------------
> Ezequiel Garcia (3):
>       media: destage Hantro VPU driver
>       hantro: Remove dedicated control documentation
>       hantro: Remove incorrect HEVC SPS validation
> 
> Jernej Skrabec (1):
>       media: cedrus: hevc: Add check for invalid timestamp
> 
>  Documentation/userspace-api/media/drivers/hantro.rst                                      | 19 -------------------
>  Documentation/userspace-api/media/drivers/index.rst                                       |  1 -
>  MAINTAINERS                                                                               |  2 +-
>  drivers/media/platform/Kconfig                                                            |  1 +
>  drivers/media/platform/Makefile                                                           |  1 +
>  drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig                      |  6 +++++-
>  drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile                     |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h                     |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c                 | 12 ++++++------
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c                  |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c         |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c        |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h             |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c          |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c                  |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c         |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h             |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c          |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c         |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h             |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c                |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c                |  9 +--------
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h                  |  1 -
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c                |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h                |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c               |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c            |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c                |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h                |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c                 |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c                 |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h                 |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c               |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c  |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c  |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c   |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h         |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c            |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c            |  0
>  drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c               |  0
>  drivers/staging/media/Kconfig                                                             |  2 --
>  drivers/staging/media/Makefile                                                            |  1 -
>  drivers/staging/media/hantro/TODO                                                         |  2 --
>  drivers/staging/media/sunxi/cedrus/cedrus_h265.c                                          |  3 +++
>  45 files changed, 18 insertions(+), 42 deletions(-)
>  delete mode 100644 Documentation/userspace-api/media/drivers/hantro.rst
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig (91%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c (99%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c (96%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h (99%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c (100%)
>  rename drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c (100%)
>  delete mode 100644 drivers/staging/media/hantro/TODO



Thanks,
Mauro
Mauro Carvalho Chehab July 27, 2022, 6:57 p.m. UTC | #2
Em Wed, 27 Jul 2022 20:35:51 +0200
Mauro Carvalho Chehab <mchehab@kernel.org> escreveu:

> Em Tue, 19 Jul 2022 11:34:55 +0200
> Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> 
> > Hi Mauro,
> > 
> > This PR has three cedrus/hantro fixes and destages hantro in the final patch.
> > That last patch just moves code into the new directory and has no other changes.
> > 
> > Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21.
> > It's just an improvement, and the cedrus changes are a bit larger than I'd like
> > so close to the code freeze. I'd like to have those tested first.  
> 
> It sounds too late to merge it for the upcoming merge window.
> 
> So, I applied only patch 1/4:
> 
> 	media: cedrus: hevc: Add check for invalid timestamp
> 
> Let's merge the remaining ones after -rc1.

Applied patch 4/4 as well, as this is actually a fix:

	hantro: Remove incorrect HEVC SPS validation

Regards,
Mauro