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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 23, 2023 at 6:02 PM Mauro Carvalho Chehab
<mchehab@xxxxxxxxxx> wrote:
>
> Em Fri, 27 Oct 2023 10:50:58 +0200
> Sebastian Fricke <sebastian.fricke@xxxxxxxxxxxxx> 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
>





[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux