Permalink
Browse files

AE: don't disable devices - as detection does not work reliable

  • Loading branch information...
1 parent dbb0daa commit fb9af19585820ce1827ffb320a97faf04933f2ad @fritsch fritsch committed with fritsch Nov 13, 2012
Showing with 7 additions and 14 deletions.
  1. +7 −14 xbmc/cores/AudioEngine/Sinks/AESinkALSA.cpp
@@ -952,20 +952,13 @@ void CAESinkALSA::EnumerateDevice(AEDeviceInfoList &list, const std::string &dev
if (badHDMI)
{
- /* only trust badHDMI (= unconnected or non-existent port) on Intel
- * and NVIDIA where it has been confirmed to work, show the empty
- * port on other systems */
- if (info.m_displayName.compare(0, 9, "HDA Intel") == 0 || info.m_displayName.compare(0, 10, "HDA NVidia") == 0)
- {
- /* unconnected HDMI port */
- CLog::Log(LOGDEBUG, "CAESinkALSA - Skipping HDMI device \"%s\" as it has no ELD data", device.c_str());
- snd_pcm_close(pcmhandle);
- return;
- }
- else
- {
- CLog::Log(LOGDEBUG, "CAESinkALSA - HDMI device \"%s\" may be unconnected (no ELD data)", device.c_str());
- }
+ /*
+ * Warn about disconnected devices, but keep them enabled
+ * Detection can go wrong on Intel, Nvidia and on all
+ * AMD (fglrx) hardware, so it is not safe to close those
+ * handles
+ */
+ CLog::Log(LOGDEBUG, "CAESinkALSA - HDMI device \"%s\" may be unconnected (no ELD data)", device.c_str());
}
}
else

0 comments on commit fb9af19

Please sign in to comment.