Skip to content

rockchip64-6.17: add rkvdec patch for rk35xx back#8730

Merged
igorpecovnik merged 1 commit intoarmbian:mainfrom
amazingfate:6.17-rkvdec
Oct 8, 2025
Merged

rockchip64-6.17: add rkvdec patch for rk35xx back#8730
igorpecovnik merged 1 commit intoarmbian:mainfrom
amazingfate:6.17-rkvdec

Conversation

@amazingfate
Copy link
Contributor

Description

#8703 has dropped the patch for rk35xx rkvdec, and introduced media-rkvdec-from-kernel-mailing-lists.patch. While the dropped patch has already included the new one. So add it back.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • ./compile.sh kernel BOARD=armsom-sige7 BRANCH=edge KERNEL_CONFIGURE=no DEB_COMPRESS=xz KERNEL_BTF=yes KERNEL_GIT=shallow
  • rkvdec with H264/H265 support on rk3588
$ v4l2-ctl -d1 -l

Codec Controls

                     h264_level 0x00990a67 (menu)   : min=0 max=15 default=0 value=0 (1) flags=0x00001000
                   h264_profile 0x00990a6b (menu)   : min=1 max=9 default=2 value=2 (Main) flags=0x00001000
                   hevc_profile 0x00990b67 (menu)   : min=0 max=2 default=0 value=0 (Main) flags=0x00001000
                     hevc_level 0x00990b68 (menu)   : min=0 max=11 default=0 value=0 (1) flags=0x00001000

Stateless Codec Controls

               h264_decode_mode 0x00a40900 (menu)   : min=1 max=1 default=1 value=1 (Frame-Based) flags=0x00001000
                h264_start_code 0x00a40901 (menu)   : min=1 max=1 default=1 value=1 (Annex B Start Code) flags=0x00001000
    h264_sequence_parameter_set 0x00a40902 (h264-sps): value=unsupported payload type flags=has-payload
     h264_picture_parameter_set 0x00a40903 (h264-pps): value=unsupported payload type flags=has-payload
            h264_scaling_matrix 0x00a40904 (h264-scaling-matrix): value=unsupported payload type flags=has-payload
         h264_decode_parameters 0x00a40907 (h264-decode-params): value=unsupported payload type flags=has-payload
    hevc_sequence_parameter_set 0x00a40a90 (hevc-sps): value=unsupported payload type flags=has-payload
     hevc_picture_parameter_set 0x00a40a91 (hevc-sps): value=unsupported payload type flags=has-payload
            hevc_scaling_matrix 0x00a40a93 (hevc-scaling-matrix): value=unsupported payload type flags=has-payload
         hevc_decode_parameters 0x00a40a94 (hevc-decode-params): value=unsupported payload type flags=has-payload
               hevc_decode_mode 0x00a40a95 (menu)   : min=1 max=1 default=1 value=1 (Frame-Based) flags=0x00001000
                hevc_start_code 0x00a40a96 (menu)   : min=1 max=1 default=1 value=1 (Annex B Start Code) flags=0x00001000
       hevc_short_term_ref_sets 0x00a40a98 (unknown): type=275 elems=1 dims=[65] flags=has-payload, dynamic-array
        hevc_long_term_ref_sets 0x00a40a99 (unknown): type=276 elems=1 dims=[65] flags=has-payload, dynamic-array

Checklist:

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added 11 Milestone: Fourth quarter release size/large PR with 250 lines or more Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Oct 8, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 8, 2025

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • patch/kernel/archive/rockchip64-6.17/media-0001-Add-rkvdec-Support-v5.patch is excluded by !patch/**
  • patch/kernel/archive/rockchip64-6.17/media-rkvdec-from-kernel-mailing-lists.patch is excluded by !patch/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@EvilOlaf EvilOlaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

@github-actions github-actions bot added the Ready to merge Reviewed, tested and ready for merge label Oct 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

✅ This PR has been reviewed and approved — all set for merge!

@github-actions github-actions bot removed the Needs review Seeking for review label Oct 8, 2025
@igorpecovnik igorpecovnik merged commit 0aa0216 into armbian:main Oct 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

11 Milestone: Fourth quarter release Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... Ready to merge Reviewed, tested and ready for merge size/large PR with 250 lines or more

Development

Successfully merging this pull request may close these issues.

5 participants