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

Intel: Fix EDID overriding (makes audio work) #4042

Merged
merged 1 commit into from
Mar 26, 2015

Conversation

fritsch
Copy link
Contributor

@fritsch fritsch commented Mar 26, 2015

This should fix a long standing issue when users want to boot with their AVRs disabled or resume from suspend and end without audio.

It's perhaps worth to integrate a working EDID file into OE for easy usage. Was only verified with kms_helper for now.

Edid (Full HD + All bitstream audio codecs): https://dl.dropboxusercontent.com/u/55728161/edid-samsung-marantz.bin

fritsch added a commit that referenced this pull request Mar 26, 2015
Intel: Fix EDID overriding (makes audio work)
@fritsch fritsch merged commit b87a7ed into OpenELEC:master Mar 26, 2015
@fritsch fritsch deleted the intel-edid branch March 26, 2015 12:23
@bkuhls
Copy link

bkuhls commented Mar 26, 2015

Works great when AVR+TV (HDMI: PC -> AVR -> TV) are powered off during boot using i3-4330T CPU with a custom edid extracted from my system, all AVR capabilities are available in Kodi!

Now I only have to revert a CEC-related commit so I can power on my TV hours after boot and still have a working CEC-connected remote control, now I have 30s ;)
opdenkamp/xbmc@85d5bd4

@fritsch
Copy link
Contributor Author

fritsch commented Mar 27, 2015

@bkuhls Thanks for your feedback!

Big thank you goes to @jnikula for implementing it. If you have something on your amazon wish list - feel free to post a link :-)

@fritsch
Copy link
Contributor Author

fritsch commented Mar 27, 2015

If it applies, it can go to OE 5.0.x - as it is used by a minority and was also CCed to stable-kernel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants