From 7544e052e6ed6a5d1e0fd783bd04f4b8aa0d45ce Mon Sep 17 00:00:00 2001
From: Andreas Monzner <andreas.monzner@multimedia-labs.de>
Date: Mon, 15 Aug 2005 23:44:52 +0000
Subject: [PATCH] fix block artifacts when zap with pallas/vulcan hardware

---
 lib/dvb/decoder.cpp | 87 ++++++++++++++++++++++++++++++++++++++++++++++++-----
 lib/dvb/decoder.h   |  8 +++++
 2 files changed, 87 insertions(+), 8 deletions(-)

diff --git a/lib/dvb/decoder.cpp b/lib/dvb/decoder.cpp
index 8d1a4b1..5f528b8 100644
--- a/lib/dvb/decoder.cpp
+++ b/lib/dvb/decoder.cpp
@@ -65,12 +65,11 @@ int eDVBAudio::startPid(int pid)
 		eWarning("audio: DMX_SET_PES_FILTER: %m");
 		return -errno;
 	}
-	if (::ioctl(m_fd_demux, DMX_START, &pes) < 0)
+	if (::ioctl(m_fd_demux, DMX_START) < 0)
 	{
 		eWarning("audio: DMX_START: %m");
 		return -errno;
 	}
-	
 	if (::ioctl(m_fd, AUDIO_PLAY) < 0)
 		eWarning("audio: AUDIO_PLAY: %m");
 	return 0;
@@ -80,10 +79,20 @@ void eDVBAudio::stop()
 {
 	if (::ioctl(m_fd, AUDIO_STOP) < 0)
 		eWarning("audio: AUDIO_STOP: %m");
+#if HAVE_DVB_API_VERSION > 2
 	if (::ioctl(m_fd_demux, DMX_STOP) < 0)
 		eWarning("audio: DMX_STOP: %m");
+#endif
 }
 	
+#if HAVE_DVB_API_VERSION < 3
+void eDVBAudio::stopPid()
+{
+	if (::ioctl(m_fd_demux, DMX_STOP) < 0)
+		eWarning("audio: DMX_STOP: %m");
+}
+#endif
+	
 eDVBAudio::~eDVBAudio()
 {
 	if (m_fd >= 0)
@@ -117,6 +126,7 @@ eDVBVideo::eDVBVideo(eDVBDemux *demux, int dev): m_demux(demux)
 	
 int eDVBVideo::startPid(int pid)
 {	
+	eDebug("setting video pid to %x", pid);
 	if ((m_fd < 0) || (m_fd_demux < 0))
 		return -1;
 	dmx_pes_filter_params pes;
@@ -131,12 +141,11 @@ int eDVBVideo::startPid(int pid)
 		eWarning("video: DMX_SET_PES_FILTER: %m");
 		return -errno;
 	}
-	if (::ioctl(m_fd_demux, DMX_START, &pes) < 0)
+	if (::ioctl(m_fd_demux, DMX_START) < 0)
 	{
 		eWarning("video: DMX_START: %m");
 		return -errno;
 	}
-	
 	if (::ioctl(m_fd, VIDEO_PLAY) < 0)
 		eWarning("video: VIDEO_PLAY: %m");
 	else
@@ -148,10 +157,20 @@ void eDVBVideo::stop()
 {
 	if (::ioctl(m_fd, VIDEO_STOP) < 0)
 		eWarning("video: VIDEO_STOP: %m");
+#if HAVE_DVB_API_VERSION > 2
 	if (::ioctl(m_fd_demux, DMX_STOP) < 0)
 		eWarning("video: DMX_STOP: %m");
+#endif
 }
-	
+
+#if HAVE_DVB_API_VERSION < 3
+void eDVBVideo::stopPid()
+{
+	if (::ioctl(m_fd_demux, DMX_STOP) < 0)
+		eWarning("video: DMX_STOP: %m");
+}
+#endif
+
 eDVBVideo::~eDVBVideo()
 {
 	if (m_fd >= 0)
@@ -177,6 +196,7 @@ eDVBPCR::eDVBPCR(eDVBDemux *demux): m_demux(demux)
 
 int eDVBPCR::startPid(int pid)
 {
+	eDebug("setting pcr pid to %x", pid);
 	if (m_fd_demux < 0)
 		return -1;
 	dmx_pes_filter_params pes;
@@ -191,7 +211,7 @@ int eDVBPCR::startPid(int pid)
 		eWarning("video: DMX_SET_PES_FILTER: %m");
 		return -errno;
 	}
-	if (::ioctl(m_fd_demux, DMX_START, &pes) < 0)
+	if (::ioctl(m_fd_demux, DMX_START) < 0)
 	{
 		eWarning("video: DMX_START: %m");
 		return -errno;
@@ -217,7 +237,57 @@ int eTSMPEGDecoder::setState()
 {
 	int res = 0;
 	eDebug("changed %x", m_changed);
-
+#if HAVE_DVB_API_VERSION < 3
+	if (m_changed & changeAudio && m_audio)
+		m_audio->stopPid();
+	if (m_changed & changeVideo && m_video)
+		m_video->stopPid();
+	if (m_changed & changePCR && m_pcr)
+	{
+		m_pcr->stop();
+		m_pcr=0;
+	}
+	if (m_changed & changeAudio && m_audio)
+	{
+		m_audio->stop();
+		m_audio=0;
+	}
+	if (m_changed & changeVideo && m_video)
+	{
+		m_video->stop();
+		m_video=0;
+	}
+	if (m_changed & changePCR)
+	{
+		m_pcr = new eDVBPCR(m_demux);
+		if (m_pcr->startPid(m_pcrpid))
+		{
+			eWarning("video: startpid failed!");
+			res = -1;
+		}
+		m_changed &= ~changePCR;
+	}
+	if (m_changed & changeVideo)
+	{
+		m_video = new eDVBVideo(m_demux, 0);
+		if (m_video->startPid(m_vpid))
+		{
+			eWarning("video: startpid failed!");
+			res = -1;
+		}
+		m_changed &= ~changeVideo;
+	}
+	if (m_changed & changeAudio)
+	{
+		m_audio = new eDVBAudio(m_demux, 0);
+		if (m_audio->startPid(m_apid))
+		{
+			eWarning("audio: startpid failed!");
+			res = -1;
+		}
+		m_changed &= ~changeAudio;
+	}
+#else
 	if (m_changed & changePCR)
 	{
 		if (m_pcr)
@@ -257,10 +327,11 @@ int eTSMPEGDecoder::setState()
 		}
 		m_changed &= ~changeAudio;
 	}
+#endif
 	return res;
 }
 
-eTSMPEGDecoder::eTSMPEGDecoder(eDVBDemux *demux, int decoder): m_demux(demux)
+eTSMPEGDecoder::eTSMPEGDecoder(eDVBDemux *demux, int decoder): m_demux(demux), m_changed(0)
 {
 }
 
diff --git a/lib/dvb/decoder.h b/lib/dvb/decoder.h
index 67a8c50..df63955 100644
--- a/lib/dvb/decoder.h
+++ b/lib/dvb/decoder.h
@@ -14,6 +14,10 @@ public:
 	eDVBAudio(eDVBDemux *demux, int dev);
 	int startPid(int pid);
 	void stop();
+#if HAVE_DVB_API_VERSION < 3
+	void start();
+	void stopPid();
+#endif
 	virtual ~eDVBAudio();
 };
 
@@ -27,6 +31,10 @@ public:
 	eDVBVideo(eDVBDemux *demux, int dev);
 	int startPid(int pid);
 	void stop();
+#if HAVE_DVB_API_VERSION < 3
+	void start();
+	void stopPid();
+#endif
 	virtual ~eDVBVideo();
 };
 
-- 
2.7.4