mbox series

[GIT,PULL,for,v5.6] Venus updates

Message ID 20200110144017.16648-1-stanimir.varbanov@linaro.org (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,for,v5.6] Venus updates | expand

Pull-request

git://linuxtv.org/svarbanov/media_tree.git tags/venus-for-v5.6

Message

Stanimir Varbanov Jan. 10, 2020, 2:40 p.m. UTC 
Hi Mauro,

The Venus driver updates include:
 - re-design pm-domain and clocks handling
 - added core assingment for Venus IPs with more than one vcodec pipeline
 - dt-binding convertion to yaml DT schema
 - added decoder handling of 10bit bitstreams
 - few fixes

Please pull.

regards,
Stan

The following changes since commit 2a0a0bc7020ef7e66c9569d8229d79fa72e3d659:

  media: MAINTAINERS: add entry for Rockchip ISP1 driver (2020-01-09 16:45:01 +0100)

are available in the Git repository at:

  git://linuxtv.org/svarbanov/media_tree.git tags/venus-for-v5.6

for you to fetch changes up to e91ebb35b94ef8cf5988cfb2f4802422fbd5f4ec:

  dt-bindings: media: venus: delete old binding document (2020-01-10 12:37:29 +0200)

----------------------------------------------------------------
Venus updates for v5.6

----------------------------------------------------------------
Andy Shevchenko (1):
      media: venus: firmware: Use %pR to print IO resource

Aniket Masule (2):
      media: venus: introduce core selection
      media: venus: vdec: handle 10bit bitstreams

Stanimir Varbanov (8):
      venus: redesign clocks and pm domains control
      venus: venc: blacklist two encoder properties
      dt-bindings: media: venus: Convert msm8916 to DT schema
      dt-bindings: media: venus: Convert msm8996 to DT schema
      dt-bindings: media: venus: Convert sdm845 to DT schema
      dt-bindings: media: venus: Add sdm845v2 DT schema
      venus: core: add sdm845-v2 DT compatible and resource struct
      dt-bindings: media: venus: delete old binding document

Stephan Gerhold (1):
      media: venus: hfi_parser: Ignore HEVC encoding for V1

 .../bindings/media/qcom,msm8916-venus.yaml         | 119 +++
 .../bindings/media/qcom,msm8996-venus.yaml         | 153 ++++
 .../bindings/media/qcom,sdm845-venus-v2.yaml       | 140 +++
 .../bindings/media/qcom,sdm845-venus.yaml          | 156 ++++
 .../devicetree/bindings/media/qcom,venus.txt       | 120 ---
 drivers/media/platform/qcom/venus/Makefile         |   2 +-
 drivers/media/platform/qcom/venus/core.c           | 122 ++-
 drivers/media/platform/qcom/venus/core.h           |  31 +-
 drivers/media/platform/qcom/venus/firmware.c       |   3 +-
 drivers/media/platform/qcom/venus/helpers.c        | 437 +++-------
 drivers/media/platform/qcom/venus/helpers.h        |   4 -
 drivers/media/platform/qcom/venus/hfi_cmds.c       |   2 +
 drivers/media/platform/qcom/venus/hfi_helper.h     |   6 +
 drivers/media/platform/qcom/venus/hfi_parser.c     |   1 +
 drivers/media/platform/qcom/venus/hfi_parser.h     |   5 +
 drivers/media/platform/qcom/venus/pm_helpers.c     | 964 +++++++++++++++++++++
 drivers/media/platform/qcom/venus/pm_helpers.h     |  65 ++
 drivers/media/platform/qcom/venus/vdec.c           |  84 +-
 drivers/media/platform/qcom/venus/venc.c           |  75 +-
 19 files changed, 1854 insertions(+), 635 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
 create mode 100644 Documentation/devicetree/bindings/media/qcom,msm8996-venus.yaml
 create mode 100644 Documentation/devicetree/bindings/media/qcom,sdm845-venus-v2.yaml
 create mode 100644 Documentation/devicetree/bindings/media/qcom,sdm845-venus.yaml
 delete mode 100644 Documentation/devicetree/bindings/media/qcom,venus.txt
 create mode 100644 drivers/media/platform/qcom/venus/pm_helpers.c
 create mode 100644 drivers/media/platform/qcom/venus/pm_helpers.h

Comments

Jenkins Jan. 10, 2020, 2:55 p.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/patch/61112/
Build log: https://builder.linuxtv.org/job/patchwork/33062/
Build time: 00:05:03
Link: https://lore.kernel.org/linux-media/20200110144017.16648-1-stanimir.varbanov@linaro.org

gpg: Signature made Fri 10 Jan 2020 02:33:09 PM UTC
gpg:                using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [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: 34CF E039 8A16 AD93 18FD  D5E8 A6D0 26D8 E358 14D4
     Subkey fingerprint: E155 8C24 97CE 3CCC 2B5A  A30F 25B5 5FC8 1B70 35F2

Summary: 6 patches and/or PDF generation with issues, being 0 at build time

Error/warnings:


Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0003-venus-redesign-clocks-and-pm-domains-control.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0003-venus-redesign-clocks-and-pm-domains-control.patch
patches/0003-venus-redesign-clocks-and-pm-domains-control.patch:679: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch
patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch
patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch
patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch
patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:
$ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch
patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:17: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:20: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.txt
Stanimir Varbanov Jan. 15, 2020, 12:58 p.m. UTC | #2 
Hi Mauro,

On 1/10/20 4:55 PM, Jenkins wrote:
> From: builder@linuxtv.org
> 
> Pull request: https://patchwork.linuxtv.org/patch/61112/
> Build log: https://builder.linuxtv.org/job/patchwork/33062/
> Build time: 00:05:03
> Link: https://lore.kernel.org/linux-media/20200110144017.16648-1-stanimir.varbanov@linaro.org
> 
> gpg: Signature made Fri 10 Jan 2020 02:33:09 PM UTC
> gpg:                using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
> gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [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: 34CF E039 8A16 AD93 18FD  D5E8 A6D0 26D8 E358 14D4
>      Subkey fingerprint: E155 8C24 97CE 3CCC 2B5A  A30F 25B5 5FC8 1B70 35F2
> 
> Summary: 6 patches and/or PDF generation with issues, being 0 at build time
> 
> Error/warnings:
> 
> 

I'm not sure how to fix bellow warnings.

I found [1], which seems describe the issue.

[1] https://lore.kernel.org/patchwork/patch/527353/

> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0003-venus-redesign-clocks-and-pm-domains-control.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0003-venus-redesign-clocks-and-pm-domains-control.patch
> patches/0003-venus-redesign-clocks-and-pm-domains-control.patch:679: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> 
> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch
> patches/0007-dt-bindings-media-venus-Convert-msm8916-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> 
> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch
> patches/0008-dt-bindings-media-venus-Convert-msm8996-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> 
> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch
> patches/0009-dt-bindings-media-venus-Convert-sdm845-to-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> 
> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch
> patches/0010-dt-bindings-media-venus-Add-sdm845v2-DT-schema.patch:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> 
> Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:
> $ ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch
> patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:17: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> patches/0012-dt-bindings-media-venus-delete-old-binding-document.patch:20: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.txt
>
Stanimir Varbanov Feb. 13, 2020, 10:58 a.m. UTC | #3 
Hi Mauro,

As this PR did not get merged, I will send a new one for v5.7 with some
more commits when the media_tree master is updated to v5.6-rc1.

On 1/10/20 4:40 PM, Stanimir Varbanov wrote:
> Hi Mauro,
> 
> The Venus driver updates include:
>  - re-design pm-domain and clocks handling
>  - added core assingment for Venus IPs with more than one vcodec pipeline
>  - dt-binding convertion to yaml DT schema
>  - added decoder handling of 10bit bitstreams
>  - few fixes
> 
> Please pull.
> 
> regards,
> Stan
> 
> The following changes since commit 2a0a0bc7020ef7e66c9569d8229d79fa72e3d659:
> 
>   media: MAINTAINERS: add entry for Rockchip ISP1 driver (2020-01-09 16:45:01 +0100)
> 
> are available in the Git repository at:
> 
>   git://linuxtv.org/svarbanov/media_tree.git tags/venus-for-v5.6
> 
> for you to fetch changes up to e91ebb35b94ef8cf5988cfb2f4802422fbd5f4ec:
> 
>   dt-bindings: media: venus: delete old binding document (2020-01-10 12:37:29 +0200)
> 
> ----------------------------------------------------------------
> Venus updates for v5.6
> 
> ----------------------------------------------------------------
> Andy Shevchenko (1):
>       media: venus: firmware: Use %pR to print IO resource
> 
> Aniket Masule (2):
>       media: venus: introduce core selection
>       media: venus: vdec: handle 10bit bitstreams
> 
> Stanimir Varbanov (8):
>       venus: redesign clocks and pm domains control
>       venus: venc: blacklist two encoder properties
>       dt-bindings: media: venus: Convert msm8916 to DT schema
>       dt-bindings: media: venus: Convert msm8996 to DT schema
>       dt-bindings: media: venus: Convert sdm845 to DT schema
>       dt-bindings: media: venus: Add sdm845v2 DT schema
>       venus: core: add sdm845-v2 DT compatible and resource struct
>       dt-bindings: media: venus: delete old binding document
> 
> Stephan Gerhold (1):
>       media: venus: hfi_parser: Ignore HEVC encoding for V1
> 
>  .../bindings/media/qcom,msm8916-venus.yaml         | 119 +++
>  .../bindings/media/qcom,msm8996-venus.yaml         | 153 ++++
>  .../bindings/media/qcom,sdm845-venus-v2.yaml       | 140 +++
>  .../bindings/media/qcom,sdm845-venus.yaml          | 156 ++++
>  .../devicetree/bindings/media/qcom,venus.txt       | 120 ---
>  drivers/media/platform/qcom/venus/Makefile         |   2 +-
>  drivers/media/platform/qcom/venus/core.c           | 122 ++-
>  drivers/media/platform/qcom/venus/core.h           |  31 +-
>  drivers/media/platform/qcom/venus/firmware.c       |   3 +-
>  drivers/media/platform/qcom/venus/helpers.c        | 437 +++-------
>  drivers/media/platform/qcom/venus/helpers.h        |   4 -
>  drivers/media/platform/qcom/venus/hfi_cmds.c       |   2 +
>  drivers/media/platform/qcom/venus/hfi_helper.h     |   6 +
>  drivers/media/platform/qcom/venus/hfi_parser.c     |   1 +
>  drivers/media/platform/qcom/venus/hfi_parser.h     |   5 +
>  drivers/media/platform/qcom/venus/pm_helpers.c     | 964 +++++++++++++++++++++
>  drivers/media/platform/qcom/venus/pm_helpers.h     |  65 ++
>  drivers/media/platform/qcom/venus/vdec.c           |  84 +-
>  drivers/media/platform/qcom/venus/venc.c           |  75 +-
>  19 files changed, 1854 insertions(+), 635 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/media/qcom,msm8916-venus.yaml
>  create mode 100644 Documentation/devicetree/bindings/media/qcom,msm8996-venus.yaml
>  create mode 100644 Documentation/devicetree/bindings/media/qcom,sdm845-venus-v2.yaml
>  create mode 100644 Documentation/devicetree/bindings/media/qcom,sdm845-venus.yaml
>  delete mode 100644 Documentation/devicetree/bindings/media/qcom,venus.txt
>  create mode 100644 drivers/media/platform/qcom/venus/pm_helpers.c
>  create mode 100644 drivers/media/platform/qcom/venus/pm_helpers.h
>