mbox series

[GIT,PULL,FOR,6.7] Mediatek Vcodec: Codec profile and level setting

Message ID 20231027085058.k65rwgxepmx5ivu5@basti-XPS-13-9310
State New
Headers show
Series [GIT,PULL,FOR,6.7] Mediatek Vcodec: Codec profile and level setting | expand

Pull-request

https://gitlab.collabora.com/sebastianfricke/linux tags/for-6.7-signed

Message

Sebastian Fricke Oct. 27, 2023, 8:50 a.m. UTC 
Hey Mauro and Hans,

first attempt at doing a PR, could you please check and pull this?

The following changes since commit 19e67e01eb1e84f3529770d084b93f16a4894c42:

   media: venus: Fix firmware path for resources (2023-10-23 12:24:53 +0200)

are available in the Git repository at:

   https://gitlab.collabora.com/sebastianfricke/linux tags/for-6.7-signed

for you to fetch changes up to ccd08a80f5e1844819874b01c5ff61e18aa9e133:

   media: mediatek: vcodec: Set the supported vp9 profile for each platform (2023-10-26 09:09:21 +0200)

----------------------------------------------------------------
Mediatek Codec patches for 6.7

----------------------------------------------------------------
Yunfei Dong (7):
       media: mediatek: vcodec: Get the chip name for each platform
       media: mediatek: vcodec: Set the supported h264 level for each platform
       media: mediatek: vcodec: Set the supported h265 level for each platform
       media: mediatek: vcodec: Set the supported h264 profile for each platform
       media: mediatek: vcodec: Set the supported h265 profile for each platform
       media: mediatek: vcodec: Set the supported vp9 level for each platform
       media: mediatek: vcodec: Set the supported vp9 profile for each platform

  drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c           |  24 +-----------------
  drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.c       |  26 ++++++++++++++++++++
  drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.h       |  14 +++++++++++
  drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_stateless.c | 168 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
  4 files changed, 208 insertions(+), 24 deletions(-)

Greetings,
Sebastian

Comments

Chen-Yu Tsai Nov. 23, 2023, 10:38 a.m. UTC | #1 
On Thu, Nov 23, 2023 at 6:02 PM Mauro Carvalho Chehab
<mchehab@kernel.org> wrote:
>
> Em Fri, 27 Oct 2023 10:50:58 +0200
> Sebastian Fricke <sebastian.fricke@collabora.com> escreveu:
>
> > Hey Mauro and Hans,
> >
> > first attempt at doing a PR, could you please check and pull this?
> >
> > The following changes since commit 19e67e01eb1e84f3529770d084b93f16a4894c42:
> >
> >    media: venus: Fix firmware path for resources (2023-10-23 12:24:53 +0200)
> >
> > are available in the Git repository at:
> >
> >    https://gitlab.collabora.com/sebastianfricke/linux tags/for-6.7-signed
> >
> > for you to fetch changes up to ccd08a80f5e1844819874b01c5ff61e18aa9e133:
> >
> >    media: mediatek: vcodec: Set the supported vp9 profile for each platform (2023-10-26 09:09:21 +0200)
>
>
> Checkpatch is producing some warnings here:
>
> WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
> #32:
> The following changes since commit 19e67e01eb1e84f3529770d084b93f16a4894c42:
>
> ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 19e67e01eb1e ("media: venus: Fix firmware path for resources")'
> #32:
> The following changes since commit 19e67e01eb1e84f3529770d084b93f16a4894c42:
>
> ERROR:NOT_UNIFIED_DIFF: Does not appear to be a unified-diff format patch
>
> Could you please double-check those?

It seems like you ran checkpatch against this pull request email, not
the patches to be pulled in?

ChenYu

> Regards,
> Mauro
>
> >
> > ----------------------------------------------------------------
> > Mediatek Codec patches for 6.7
> >
> > ----------------------------------------------------------------
> > Yunfei Dong (7):
> >        media: mediatek: vcodec: Get the chip name for each platform
> >        media: mediatek: vcodec: Set the supported h264 level for each platform
> >        media: mediatek: vcodec: Set the supported h265 level for each platform
> >        media: mediatek: vcodec: Set the supported h264 profile for each platform
> >        media: mediatek: vcodec: Set the supported h265 profile for each platform
> >        media: mediatek: vcodec: Set the supported vp9 level for each platform
> >        media: mediatek: vcodec: Set the supported vp9 profile for each platform
> >
> >   drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec.c           |  24 +-----------------
> >   drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.c       |  26 ++++++++++++++++++++
> >   drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.h       |  14 +++++++++++
> >   drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_stateless.c | 168 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> >   4 files changed, 208 insertions(+), 24 deletions(-)
> >
> > Greetings,
> > Sebastian
>
>
>
> Thanks,
> Mauro
>