Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RK3588: add mpp and rkvenc2 nodes #7294

Closed
wants to merge 1 commit into from
Closed

Conversation

benhoff
Copy link
Collaborator

@benhoff benhoff commented Sep 24, 2024

Description

This ports Collabora's patches from 6.8 to 6.10 for the 580 encoding nodes. This should allow ffmpeg-rockchip to be used with an up to date kernel

How Has This Been Tested?

To be tested using OP5+

Checklist:

Please delete options that are not relevant.

  • I have performed a self-review of my own code
  • My changes generate no new warnings

@benhoff benhoff added the Work in progress Unfinished / work in progress label Sep 24, 2024
@benhoff benhoff changed the title add mpp and rkvenc2 nodes RK3588: add mpp and rkvenc2 nodes Sep 24, 2024
@github-actions github-actions bot added size/large PR with 250 lines or more Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Sep 24, 2024
@hbiyik
Copy link

hbiyik commented Sep 24, 2024

@benhoff you might want to check below branch.

https://github.com/hbiyik/linux/tree/panthor%2Bmpp%2Brga

Bsp mpp and rga kernel drivers are ported to mainline linux there.

starts at commit d98096a3cab8486905a7db064fb820078c3d35c4 and ends in ad3bc03d168ebdaff8aba161f0b992dc2912391c

@benhoff
Copy link
Collaborator Author

benhoff commented Sep 24, 2024

@hbiyik , will your branch run rockchip-ffmpeg?

I just want HW accelerated h264 encoding. Maybe h265 encoding if I'm feeling frisky.

@hbiyik
Copy link

hbiyik commented Sep 24, 2024

will your branch run rockchip-ffmpeg?

yep, i tested that it was working at that time

@amazingfate
Copy link
Contributor

Mpp is a driver from vendor kernel, and I think it is not a good idea to integrate it into mainline kernel by default.
This driver patch is super large and hard to maintain. Is it possible to install the driver as dkms?
And it's also better to use devicetree overlay.

@benhoff
Copy link
Collaborator Author

benhoff commented Sep 25, 2024

@amazingfate , I'll open a pr for the device tree overlay tomorrow. I've got one kicking around that I've applied locally.

Makes sense on the dkms bit.

I haven't looked and I can dig into it tomorrow, but do you know off the cuff how that normally gets distributed? I assume not apt.

@EvilOlaf
Copy link
Member

Not sure if this is worth the effort overall. 6.10 will probably be abandoned for 6.11.y as soon as 6.12-rc hits which then likely will become new edge for this family. Also 6.10 is not an upstream LTS kernel, so this is kind of riding a dead horse.

@amazingfate
Copy link
Contributor

@amazingfate , I'll open a pr for the device tree overlay tomorrow. I've got one kicking around that I've applied locally.

Makes sense on the dkms bit.

I haven't looked and I can dig into it tomorrow, but do you know off the cuff how that normally gets distributed? I assume not apt.

Dkms has to get packaged to deb. I have pacakged bcmdhd driver at https://github.com/armbian/bcmdhd-dkms, in which deb is packaged at github workflow: https://github.com/armbian/bcmdhd-dkms/blob/main/.github/workflows/release.yml
And in armbian dkms package is installed with extension: https://github.com/armbian/build/blob/main/extensions/bcmdhd.sh

@benhoff
Copy link
Collaborator Author

benhoff commented Sep 25, 2024

Not sure if this is worth the effort overall. 6.10 will probably be abandoned for 6.11.y as soon as 6.12-rc hits which then likely will become new edge for this family. Also 6.10 is not an upstream LTS kernel, so this is kind of riding a dead horse.

I have a 6.10 kernel that I'm actively developing with so its easier for me to test locally. However, whatever is worth keeping of this patch (if anything) should be pretty portable to any kernel version.

I just need to burn an updated image on an SD which I haven't in awhile.

@benhoff
Copy link
Collaborator Author

benhoff commented Sep 25, 2024

@amazingfate , any value in pulling in the entire driver from @hbiyik 's linux branch into dkms for Armbian?

I must have H264 encoding (and probably decoding) for my own project, so I know I must do something here.

If there's value to Armbian to grab the whole driver it might not be much more additional effort.

Disclaimer, I haven't looked though.

@benhoff
Copy link
Collaborator Author

benhoff commented Sep 25, 2024

Device tree overlay in this branch: #7298

I'll explore DKMS packaging as suggested in this thread rather than modifying the kernel.

@benhoff benhoff closed this Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... size/large PR with 250 lines or more Work in progress Unfinished / work in progress
Development

Successfully merging this pull request may close these issues.

4 participants