8000 Enable HDMI audio outputs for Rock 5B by detlev.casanova@collabora.com by q4a · Pull Request #7925 · armbian/build · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Enable HDMI audio outputs for Rock 5B by detlev.casanova@collabora.com #7925

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 8, 2025
Merged

Enable HDMI audio outputs for Rock 5B by detlev.casanova@collabora.com #7925

merged 1 commit into from
Mar 8, 2025

Conversation

q4a
Copy link
Contributor
@q4a q4a commented Mar 8, 2025 •
edited
Loading

Just this small patch:
https://lore.kernel.org/all/20250217215641.372723-4-detlev.casanova@collabora.com/
Already merged to linux-rockchip (for-next branch)
https://web.git.kernel.org/pub/scm/linux/kernel/git/mmind/linux-rockchip.git/commit/?h=for-next&id=97aa62ed1e970bf8aa9f57e87c946a95fa3d5bef

Description

HDMI audio is available on the Rock 5B HDMI TX ports.
Enable it for both ports.

How Has This Been Tested?

I added patch, then build kernel with ./compile.sh kernel BOARD=rock-5b BRANCH=edge and install builded debs on my Rock 5B. After that I got Analog Output - HDMI0 Audio and now I can hear music =)

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 size/medium PR with more then 50 and less then 250 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... Patches Patches related to kernel, U-Boot, ... labels Mar 8, 2025
paolosabatino
< 8000 div data-view-component="true" class="TimelineItem-body d-flex flex-column flex-md-row flex-justify-start">
paolosabatino approved these changes Mar 8, 2025
View reviewed changes
Copy link
Contributor
@paolosabatino paolosabatino left a comment

Choose a reason for hiding this comment

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

Legit to me

@paolosabatino paolosabatino added the Ready to merge Reviewed, tested and ready for merge label Mar 8, 2025
@github-actions github-actions bot removed the Ready to merge Reviewed, tested and ready for merge label Mar 8, 2025
@SuperKali
Copy link
Member

@q4a can you use ./compile.sh rewrite-kernel-patches and push the patch?

@igorpecovnik igorpecovnik added Ready to merge Reviewed, tested and ready for merge 05 Milestone: Second quarter release and removed Needs review Seeking for review labels Mar 8, 2025
@igorpecovnik igorpecovnik merged commit bf9ffa6 into armbian:main Mar 8, 2025
@igorpecovnik
Copy link
Member

can you use ./compile.sh rewrite-kernel-patches and push the patch?

Ups :) Saw it too late.

@SuperKali
Copy link
Member

Ups :) Saw it too late.

Nevermind, i will open a new PR for this :)

@SuperKali SuperKali mentioned this pull request Mar 8, 2025
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paolosabatino paolosabatino paolosabatino approved these changes

@ColorfulRhino ColorfulRhino Awaiting requested review from ColorfulRhino ColorfulRhino is a code owner

@JohnTheCoolingFan JohnTheCoolingFan Awaiting requested review from JohnTheCoolingFan JohnTheCoolingFan is a code owner

@Manouchehri Manouchehri Awaiting requested review from Manouchehri

@SuperKali SuperKali Awaiting requested review from SuperKali SuperKali is a code owner

@TRSx80 TRSx80 Awaiting requested review from TRSx80 TRSx80 is a code owner

@TheSnowfield TheSnowfield Awaiting requested review from TheSnowfield TheSnowfield is a code owner

@Tonymac32 Tonymac32 Awaiting requested review from Tonymac32 Tonymac32 is a code owner

@ahoneybun ahoneybun Awaiting requested review from ahoneybun ahoneybun is a code owner

@amazingfate amazingfate Awaiting requested review from amazingfate amazingfate is a code owner

@brentr brentr Awaiting requested review from brentr brentr is a code owner

@catalinii catalinii Awaiting requested review from catalinii catalinii is a code owner

@clee clee Awaiting requested review from clee clee is a code owner

@efectn efectn Awaiting requested review from efectn efectn is a code owner

@igorpecovnik igorpecovnik Awaiting requested review from igorpecovnik igorpecovnik is a code owner

@joekhoobyar joekhoobyar Awaiting requested review from joekhoobyar joekhoobyar is a code owner

@krachlatte krachlatte Awaiting requested review from krachlatte krachlatte is a code owner

@lanefu lanefu Awaiting requested review from lanefu lanefu is a code owner

@rpardini rpardini Awaiting requested review from rpardini rpardini is a code owner

@schwar3kat schwar3kat Awaiting requested review from schwar3kat schwar3kat is a code owner

Assignees
No one assigned
Labels
05 Milestone: Second 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/medium PR with more then 50 and less then 250 lines
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@q4a @SuperKali @igorpecovnik @paolosabatino
0