mbox series

[GIT,PULL,FOR,v5.17] Mediatek improvements

Message ID e85c97e7-cdd1-db38-5c9a-5455cd83e6ad@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.17] Mediatek improvements | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.17f

Message

Hans Verkuil Dec. 6, 2021, 2:52 p.m. UTC
Hi Mauro,

This PR contains the patches from these two series (minus the dts patches):

https://patchwork.linuxtv.org/project/linux-media/list/?series=6887
https://patchwork.linuxtv.org/project/linux-media/list/?series=6698

The second series depends on the first.

The second series contains patches for different subsystems, but since they
all depend on the iommu changes, and since most of the patches are media
related, this series goes in through the media subsystem. That made the most
sense. I believe all the correct Acks are collected for this.

Regards,

	Hans

The following changes since commit b0f7d9b6d81209f87cabd45c91458dd745d84924:

  media: docs: media: Fix imbalance of LaTeX group (2021-12-06 13:23:03 +0100)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.17f

for you to fetch changes up to 4a3482c6afb179e282353cdb343545b8ad985d95:

  memory: mtk-smi: Get rid of mtk_smi_larb_get/put (2021-12-06 15:41:56 +0100)

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

----------------------------------------------------------------
Hsin-Yi Wang (2):
      dt-bindings: mediatek: convert mtk jpeg decoder/encoder to yaml
      dt-bindings: mediatek: Add mediatek, mt8183-jpgenc compatible

Yong Wu (12):
      dt-binding: mediatek: Get rid of mediatek,larb for multimedia HW
      iommu/mediatek-v1: Free the existed fwspec if the master dev already has
      iommu/mediatek: Return ENODEV if the device is NULL
      iommu/mediatek: Add probe_defer for smi-larb
      iommu/mediatek: Add device_link between the consumer and the larb devices
      media: mtk-jpeg: Get rid of mtk_smi_larb_get/put
      media: mtk-mdp: Get rid of mtk_smi_larb_get/put
      drm/mediatek: Get rid of mtk_smi_larb_get/put
      media: mtk-vcodec: Get rid of mtk_smi_larb_get/put
      media: mtk-vcodec: dec: Remove mtk_vcodec_release_dec_pm
      media: mtk-vcodec: enc: Remove mtk_vcodec_release_enc_pm
      memory: mtk-smi: Get rid of mtk_smi_larb_get/put

Yongqiang Niu (1):
      drm/mediatek: Add pm runtime support for ovl and rdma

 Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt |  9 -----
 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt    | 38 ---------------------
 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.yaml   | 80 ++++++++++++++++++++++++++++++++++++++++++++
 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt    | 35 -------------------
 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml   | 72 +++++++++++++++++++++++++++++++++++++++
 Documentation/devicetree/bindings/media/mediatek-mdp.txt             |  8 -----
 Documentation/devicetree/bindings/media/mediatek-vcodec.txt          |  4 ---
 drivers/gpu/drm/mediatek/mtk_disp_ovl.c                              |  8 ++++-
 drivers/gpu/drm/mediatek/mtk_disp_rdma.c                             |  9 ++++-
 drivers/gpu/drm/mediatek/mtk_drm_crtc.c                              | 15 +++++----
 drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c                          | 36 ++------------------
 drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h                          |  1 -
 drivers/gpu/drm/mediatek/mtk_drm_drv.c                               |  5 +--
 drivers/iommu/mtk_iommu.c                                            | 34 +++++++++++++++++++
 drivers/iommu/mtk_iommu_v1.c                                         | 42 ++++++++++++++++++++++-
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c                      | 45 ++-----------------------
 drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h                      |  2 --
 drivers/media/platform/mtk-mdp/mtk_mdp_comp.c                        | 40 ----------------------
 drivers/media/platform/mtk-mdp/mtk_mdp_comp.h                        |  2 --
 drivers/media/platform/mtk-mdp/mtk_mdp_core.c                        |  1 -
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c               |  8 +++--
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c                | 46 ++++---------------------
 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h                |  3 +-
 drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h                   |  3 --
 drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c                   |  1 -
 drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c               |  6 ++--
 drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_pm.c                | 52 ++++------------------------
 drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_pm.h                |  3 +-
 drivers/memory/mtk-smi.c                                             | 14 --------
 include/soc/mediatek/smi.h                                           | 20 -----------
 30 files changed, 277 insertions(+), 365 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt
 create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.yaml
 delete mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt
 create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml

Comments

Jenkins Dec. 6, 2021, 3:24 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/e85c97e7-cdd1-db38-5c9a-5455cd83e6ad@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/163085/
Build time: 00:20:36
Link: https://lore.kernel.org/linux-media/e85c97e7-cdd1-db38-5c9a-5455cd83e6ad@xs4all.nl

gpg: Signature made Mon 06 Dec 2021 02:43:51 PM UTC
gpg:                using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg:                 aka "Hans Verkuil <hverkuil@xs4all.nl>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 052C DE7B C215 053B 689F  1BCA BD2D 6148 6614 3B4C
     Subkey fingerprint: AAA7 FFBA 4D2D 77EF 4CAE  A142 1326 E0CD 23AB DCE5

Summary: got 3/15 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-dt-bindings-mediatek-convert-mtk-jpeg-decoder-encode.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

    allyesconfig: return code #0:
	../drivers/media/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
	SPARSE:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: warning: array of flexible structures
	../drivers/media/i2c/ov8865.c: ../drivers/media/i2c/ov8865.c:2841 ov8865_get_selection() warn: inconsistent indenting
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2625 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:658 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 654)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2874 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0001-dt-bindings-mediatek-convert-mtk-jpeg-decoder-encode.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:23: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:26: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:66: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:165: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:202: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst

patches/0004-iommu-mediatek-v1-Free-the-existed-fwspec-if-the-mas.patch:

   checkpatch.pl:
	$ cat patches/0004-iommu-mediatek-v1-Free-the-existed-fwspec-if-the-mas.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:36: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)

patches/0007-iommu-mediatek-Add-device_link-between-the-consumer-.patch:

   checkpatch.pl:
	$ cat patches/0007-iommu-mediatek-Add-device_link-between-the-consumer-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:47: WARNING: Possible unwrapped commit description (prefer a maximum 75 chars per line)


Error #512 when building PDF docs
Hans Verkuil Dec. 7, 2021, 12:39 p.m. UTC | #2
Hi all,

I'm withdrawing this PR until this issue that I only stumbled on today is
clarified:

https://lore.kernel.org/linux-media/a0b3a10d-9313-7ac5-692c-822aeff50858@xs4all.nl/T/#u

Regards,

	Hans

On 12/6/21 3:52 PM, Hans Verkuil wrote:
> Hi Mauro,
> 
> This PR contains the patches from these two series (minus the dts patches):
> 
> https://patchwork.linuxtv.org/project/linux-media/list/?series=6887
> https://patchwork.linuxtv.org/project/linux-media/list/?series=6698
> 
> The second series depends on the first.
> 
> The second series contains patches for different subsystems, but since they
> all depend on the iommu changes, and since most of the patches are media
> related, this series goes in through the media subsystem. That made the most
> sense. I believe all the correct Acks are collected for this.
> 
> Regards,
> 
> 	Hans
> 
> The following changes since commit b0f7d9b6d81209f87cabd45c91458dd745d84924:
> 
>   media: docs: media: Fix imbalance of LaTeX group (2021-12-06 13:23:03 +0100)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.17f
> 
> for you to fetch changes up to 4a3482c6afb179e282353cdb343545b8ad985d95:
> 
>   memory: mtk-smi: Get rid of mtk_smi_larb_get/put (2021-12-06 15:41:56 +0100)
> 
> ----------------------------------------------------------------
> Tag branch
> 
> ----------------------------------------------------------------
> Hsin-Yi Wang (2):
>       dt-bindings: mediatek: convert mtk jpeg decoder/encoder to yaml
>       dt-bindings: mediatek: Add mediatek, mt8183-jpgenc compatible
> 
> Yong Wu (12):
>       dt-binding: mediatek: Get rid of mediatek,larb for multimedia HW
>       iommu/mediatek-v1: Free the existed fwspec if the master dev already has
>       iommu/mediatek: Return ENODEV if the device is NULL
>       iommu/mediatek: Add probe_defer for smi-larb
>       iommu/mediatek: Add device_link between the consumer and the larb devices
>       media: mtk-jpeg: Get rid of mtk_smi_larb_get/put
>       media: mtk-mdp: Get rid of mtk_smi_larb_get/put
>       drm/mediatek: Get rid of mtk_smi_larb_get/put
>       media: mtk-vcodec: Get rid of mtk_smi_larb_get/put
>       media: mtk-vcodec: dec: Remove mtk_vcodec_release_dec_pm
>       media: mtk-vcodec: enc: Remove mtk_vcodec_release_enc_pm
>       memory: mtk-smi: Get rid of mtk_smi_larb_get/put
> 
> Yongqiang Niu (1):
>       drm/mediatek: Add pm runtime support for ovl and rdma
> 
>  Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt |  9 -----
>  Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt    | 38 ---------------------
>  Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.yaml   | 80 ++++++++++++++++++++++++++++++++++++++++++++
>  Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt    | 35 -------------------
>  Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml   | 72 +++++++++++++++++++++++++++++++++++++++
>  Documentation/devicetree/bindings/media/mediatek-mdp.txt             |  8 -----
>  Documentation/devicetree/bindings/media/mediatek-vcodec.txt          |  4 ---
>  drivers/gpu/drm/mediatek/mtk_disp_ovl.c                              |  8 ++++-
>  drivers/gpu/drm/mediatek/mtk_disp_rdma.c                             |  9 ++++-
>  drivers/gpu/drm/mediatek/mtk_drm_crtc.c                              | 15 +++++----
>  drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c                          | 36 ++------------------
>  drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h                          |  1 -
>  drivers/gpu/drm/mediatek/mtk_drm_drv.c                               |  5 +--
>  drivers/iommu/mtk_iommu.c                                            | 34 +++++++++++++++++++
>  drivers/iommu/mtk_iommu_v1.c                                         | 42 ++++++++++++++++++++++-
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_core.c                      | 45 ++-----------------------
>  drivers/media/platform/mtk-jpeg/mtk_jpeg_core.h                      |  2 --
>  drivers/media/platform/mtk-mdp/mtk_mdp_comp.c                        | 40 ----------------------
>  drivers/media/platform/mtk-mdp/mtk_mdp_comp.h                        |  2 --
>  drivers/media/platform/mtk-mdp/mtk_mdp_core.c                        |  1 -
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_drv.c               |  8 +++--
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.c                | 46 ++++---------------------
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_pm.h                |  3 +-
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h                   |  3 --
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c                   |  1 -
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_drv.c               |  6 ++--
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_pm.c                | 52 ++++------------------------
>  drivers/media/platform/mtk-vcodec/mtk_vcodec_enc_pm.h                |  3 +-
>  drivers/memory/mtk-smi.c                                             | 14 --------
>  include/soc/mediatek/smi.h                                           | 20 -----------
>  30 files changed, 277 insertions(+), 365 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.txt
>  create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-decoder.yaml
>  delete mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.txt
>  create mode 100644 Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml
>