mbox series

[GIT,PULL,FOR,v5.13] v2: Venus updates

Message ID 20210312085716.2435846-1-stanimir.varbanov@linaro.org (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.13] v2: Venus updates | expand

Pull-request

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

Message

Stanimir Varbanov March 12, 2021, 8:57 a.m. UTC
Fixed checkpatch warn/errors.

Supersedes https://patchwork.linuxtv.org/project/linux-media/patch/20210306162903.221668-1-stanimir.varbanov@linaro.org/

Venus driver updates for v5.13 includes:

 * Make display-delay and display-delay-enable standard controls and support
 for them in Venus driver.
 * Add new AUD generation control and support for it in Venus driver.
 * Parse firmware-name DT property in the Venus driver.
 * Fix probe dependency error in case of built-in Venus driver.
 * Fix interconnect resource in .probe.

Please pull.

regards,
Stan

The following changes since commit 1cb3ff130f4d97dec2d1bee968a87f5d6e24dd60:

  media: imx: imx7_mipi_csis: Print shadow registers in mipi_csis_dump_regs() (2021-03-11 11:59:53 +0100)

are available in the Git repository at:

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

for you to fetch changes up to a7a0b3a31d6d270721b7e563272a1647da4522f1:

  venus: core: Parse firmware-name DT property (2021-03-12 10:43:18 +0200)

----------------------------------------------------------------
Venus updates for v5.13 v2

----------------------------------------------------------------
Bryan O'Donoghue (1):
      media: venus: core, venc, vdec: Fix probe dependency error

Christophe JAILLET (1):
      media: venus: core: Fix some resource leaks in the error path of 'venus_probe()'

Stanimir Varbanov (7):
      v4l2-ctrl: Make display delay and display enable std controls
      venus: vdec: Add support for display delay and delay enable controls
      s5p-mfc: Use display delay and display enable std controls
      docs: Deprecate mfc display delay controls
      media: v4l2-ctrls: Add control for AUD generation
      venus: venc: Add support for AUD NALU control
      venus: core: Parse firmware-name DT property

Stephen Boyd (1):
      media: venus: Include io.h for memremap()

 .../userspace-api/media/v4l/ext-ctrls-codec.rst    | 30 +++++++++++++++++
 drivers/media/platform/qcom/venus/core.c           | 38 ++++++++++++----------
 drivers/media/platform/qcom/venus/core.h           |  3 ++
 drivers/media/platform/qcom/venus/firmware.c       |  8 ++++-
 drivers/media/platform/qcom/venus/pm_helpers.c     | 30 ++++++++---------
 drivers/media/platform/qcom/venus/pm_helpers.h     |  7 ++--
 drivers/media/platform/qcom/venus/vdec.c           | 10 +++++-
 drivers/media/platform/qcom/venus/vdec_ctrls.c     | 16 ++++++++-
 drivers/media/platform/qcom/venus/venc.c           | 14 ++++++++
 drivers/media/platform/qcom/venus/venc_ctrls.c     |  8 ++++-
 drivers/media/platform/s5p-mfc/s5p_mfc_dec.c       | 16 +++++++++
 drivers/media/v4l2-core/v4l2-ctrls.c               |  6 ++++
 include/uapi/linux/v4l2-controls.h                 |  4 +++
 13 files changed, 148 insertions(+), 42 deletions(-)

Comments

Jenkins March 12, 2021, 9:23 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20210312085716.2435846-1-stanimir.varbanov@linaro.org/
Build log: https://builder.linuxtv.org/job/patchwork/93925/
Build time: 00:11:32
Link: https://lore.kernel.org/linux-media/20210312085716.2435846-1-stanimir.varbanov@linaro.org

gpg: Signature made Fri 12 Mar 2021 08:44:25 AM UTC
gpg:                using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [expired]
gpg: Note: This key has expired!
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: got 1/10 patches with issues, being 0 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0005-media-v4l2-ctrls-Add-control-for-AUD-generation.patch:

   checkpatch.pl:
	$ cat patches/0005-media-v4l2-ctrls-Add-control-for-AUD-generation.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:41: WARNING: line length of 105 exceeds 100 columns
	-:61: CHECK: spaces preferred around that '+' (ctx:VxV)


Error #512 when building PDF docs
Mauro Carvalho Chehab March 22, 2021, 10:13 a.m. UTC | #2
Em Fri, 12 Mar 2021 10:57:16 +0200
Stanimir Varbanov <stanimir.varbanov@linaro.org> escreveu:

> Fixed checkpatch warn/errors.
> 
> Supersedes https://patchwork.linuxtv.org/project/linux-media/patch/20210306162903.221668-1-stanimir.varbanov@linaro.org/

Don't do that. Instead, reply to the patch you sent earlier, or change
it at patchwork to "Superseed", as I handle pull requests in the order
they arrive. 

So, by the time I handled this PR, the previous one was already
merged, and another PR was added afterwards.

So, too late to revert it.

Please rebase your PR on the top of upstream and resend it.

Thanks,
Mauro
Stanimir Varbanov March 22, 2021, 11 a.m. UTC | #3
On 3/22/21 12:13 PM, Mauro Carvalho Chehab wrote:
> Em Fri, 12 Mar 2021 10:57:16 +0200
> Stanimir Varbanov <stanimir.varbanov@linaro.org> escreveu:
> 
>> Fixed checkpatch warn/errors.
>>
>> Supersedes https://patchwork.linuxtv.org/project/linux-media/patch/20210306162903.221668-1-stanimir.varbanov@linaro.org/
> 
> Don't do that. Instead, reply to the patch you sent earlier, or change
> it at patchwork to "Superseed", as I handle pull requests in the order
> they arrive. 
> 
> So, by the time I handled this PR, the previous one was already
> merged, and another PR was added afterwards.

Sorry about that. Note taken.

> 
> So, too late to revert it.
> 
> Please rebase your PR on the top of upstream and resend it.

Sure will do it. Thanks!

> 
> Thanks,
> Mauro
>