Release Version 0.3.1.

Move objects before $LIBS to avoid link failures with --as-needed.
Do not override CFLAGS for video test.
Rearrange *FLAGS incl. some minor fixes.
Don't override VDRDIR, LIBDIR and TMPDIR in makefile.
Don't abuse LDFLAGS in makefile.
Define CC in makefile.
Include GL/gl.h for the GL_COLOR_BUFFER_BIT definition.
VideoInit() needs an argument.

ChangeLog

@@ -1,5 +1,58 @@
 User johns
-Date:
+Date: Sun Jan 15 16:56:04 CET 2012
+
+    Release Version 0.3.1
+    Fix bug: AudioFreeBytes didn't check if audio running/compiled.
+    Fix bug: snd_pcm_state: Assertion `pcm' failed.
+    Add support for fullscreen and fullscreen toogle.
+    Instant update deinterlace configuration changes.
+    Fix subtitle position.
+    Add SVDRP support.
+    Suspend when user is inactive.
+
+User Christian Rupper
+Date: Tue Jan 10 22:33:14 CET 2012
+
+    Move objects before $LIBS to avoid link failures with --as-needed.
+    Do not override CFLAGS for video test.
+    Rearrange *FLAGS incl. some minor fixes.
+    Don't override VDRDIR, LIBDIR and TMPDIR in makefile.
+    Don't abuse LDFLAGS in makefile.
+    Define CC in makefile.
+    Include GL/gl.h for the GL_COLOR_BUFFER_BIT definition.
+    VideoInit() needs an argument.
+
+User johns
+Date: Tue Jan 10 22:32:50 CET 2012
+
+    Add main menu entry, which suspends the plugin.
+    Add support for resize window.
+    Close window sends "close" as remote key press.
+
+Date: Mon Jan  9 22:09:38 CET 2012
+
+    Release Version 0.3.0
+    Add support of resolution dependend video parameters (deint, scale, ...).
+    Add support for recording play back.
+    Add workaround for alsa crash in snd_pcm_prepare.
+    Fix bug: audio crash on exit.
+    Fix build with vdr without yaepg support.
+    Support yaepghd video picture output position change.
+
+Date: Sat Jan  7 13:20:07 CET 2012
+
+    Release Version 0.2.0
+    Add support for ac3 audio pass through.
+    Add workaround for alsa not playing hdmi sound.
+    Fix bug: broken device plugin stop and exit.
+    Show transparent cursor to hide cursor.
+    VDPAU: Add color standard support.
+    VDPAU: Add denoise and sharpness support.
+    VDPAU: Add skip chroma deinterlace support.
+    VDPAU: Show OSD only if something is to display, improves performance.
+    VDPAU: Add deinterlace with only 4 surfaces.
+
+Date: Thu Jan 4 17:00:00 CET 2012
 
     Release Version 0.1.5
     Adds OSS mixer support.

Makefile

@@ -26,40 +26,17 @@ CONFIG += $(shell pkg-config --exists alsa && echo "-DUSE_ALSA")
 
 ### The C++ compiler and options:
 
+CC       ?= gcc
 CXX      ?= g++
-CXXFLAGS ?= -g -O2 -W -Wall -Wextra -Woverloaded-virtual -fPIC
-override CXXFLAGS += $(DEFINES) $(INCLUDES)
 CFLAGS   ?=	-g -O2 -W -Wall -Wextra -Winit-self \
-		-Wdeclaration-after-statement -fPIC
+		-Wdeclaration-after-statement
-#CFLAGS	+=	-Werror
+CXXFLAGS ?= -g -O2 -W -Wall -Wextra -Woverloaded-virtual
-override CFLAGS   +=	$(DEFINES) $(INCLUDES) \
-	$(shell pkg-config --cflags libavcodec libavformat) \
-	`pkg-config --cflags x11 x11-xcb xcb xcb-xv xcb-shm xcb-dpms xcb-atom\
-		xcb-screensaver xcb-randr xcb-glx xcb-icccm xcb-keysyms`\
-	`pkg-config --cflags gl glu` \
-	$(if $(findstring USE_VDPAU,$(CONFIG)), \
-	            `pkg-config --cflags vdpau`) \
-	$(if $(findstring USE_VAAPI,$(CONFIG)), \
-	            `pkg-config --cflags libva-x11 libva-glx libva`) \
-	$(if $(findstring USE_ALSA,$(CONFIG)), \
-	            `pkg-config --cflags alsa`)
-override LDFLAGS  += -lrt \
-	$(shell pkg-config --libs libavcodec libavformat) \
-	`pkg-config --libs x11 x11-xcb xcb xcb-xv xcb-shm xcb-dpms xcb-atom\
-		xcb-screensaver xcb-randr xcb-glx xcb-icccm xcb-keysyms`\
-	`pkg-config --libs gl glu` \
-	$(if $(findstring USE_VDPAU,$(CONFIG)), \
-	            `pkg-config --libs vdpau`) \
-	$(if $(findstring USE_VAAPI,$(CONFIG)), \
-	            `pkg-config --libs libva-x11 libva-glx libva`) \
-	$(if $(findstring USE_ALSA,$(CONFIG)), \
-	            `pkg-config --libs alsa`)
 
 ### The directory environment:
 
-VDRDIR = ../../..
+VDRDIR ?= ../../..
-LIBDIR = ../../lib
+LIBDIR ?= ../../lib
-TMPDIR = /tmp
+TMPDIR ?= /tmp
 
 ### Make sure that necessary options are included:
 
@@ -78,12 +55,40 @@ APIVERSION = $(shell sed -ne '/define APIVERSION/s/^.*"\(.*\)".*$$/\1/p' $(VDRDI
 ARCHIVE = $(PLUGIN)-$(VERSION)
 PACKAGE = vdr-$(ARCHIVE)
 
-### Includes and Defines (add further entries here):
+### Includes, Defines and dependencies (add further entries here):
 
 INCLUDES += -I$(VDRDIR)/include
 
 DEFINES += $(CONFIG) -D_GNU_SOURCE -DPLUGIN_NAME_I18N='"$(PLUGIN)"'
 
+_CFLAGS = $(DEFINES) $(INCLUDES) \
+	$(shell pkg-config --cflags libavcodec libavformat) \
+	`pkg-config --cflags x11 x11-xcb xcb xcb-xv xcb-shm xcb-dpms xcb-atom\
+		xcb-screensaver xcb-randr xcb-glx xcb-icccm xcb-keysyms`\
+	`pkg-config --cflags gl glu` \
+	$(if $(findstring USE_VDPAU,$(CONFIG)), \
+	            `pkg-config --cflags vdpau`) \
+	$(if $(findstring USE_VAAPI,$(CONFIG)), \
+	            `pkg-config --cflags libva-x11 libva-glx libva`) \
+	$(if $(findstring USE_ALSA,$(CONFIG)), \
+	            `pkg-config --cflags alsa`)
+
+#override _CFLAGS  += -Werror
+override CXXFLAGS += $(_CFLAGS)
+override CFLAGS   += $(_CFLAGS)
+
+LIBS += -lrt \
+	$(shell pkg-config --libs libavcodec libavformat) \
+	`pkg-config --libs x11 x11-xcb xcb xcb-xv xcb-shm xcb-dpms xcb-atom\
+		xcb-screensaver xcb-randr xcb-glx xcb-icccm xcb-keysyms`\
+	`pkg-config --libs gl glu` \
+	$(if $(findstring USE_VDPAU,$(CONFIG)), \
+	            `pkg-config --libs vdpau`) \
+	$(if $(findstring USE_VAAPI,$(CONFIG)), \
+	            `pkg-config --libs libva-x11 libva-glx libva`) \
+	$(if $(findstring USE_ALSA,$(CONFIG)), \
+	            `pkg-config --libs alsa`)
+
 ### The object files (add further files here):
 
 OBJS = $(PLUGIN).o softhddev.o video.o audio.o codec.o ringbuffer.o
@@ -105,6 +110,8 @@ DEPFILE = .dependencies
 $(DEPFILE): Makefile
 	@$(MAKEDEP) $(DEFINES) $(INCLUDES) $(SRCS) >$@
 
+$(OBJS): Makefile
+
 -include $(DEPFILE)
 
 ### Internationalization (I18N):
@@ -137,7 +144,7 @@ i18n: $(I18Nmsgs) $(I18Npot)
 ### Targets:
 
 libvdr-$(PLUGIN).so: $(OBJS) Makefile
-	$(CXX) $(CXXFLAGS) -shared $(OBJS) -o $@ $(LDFLAGS)
+	$(CXX) $(CXXFLAGS) $(LDFLAGS) -shared -fPIC $(OBJS) -o $@ $(LIBS)
 	@cp --remove-destination $@ $(LIBDIR)/$@.$(APIVERSION)
 
 dist: $(I18Npo) clean
@@ -163,5 +170,5 @@ indent:
 	done
 
 video_test: video.c
-	$(CC) -DVIDEO_TEST -DVERSION='"$(VERSION)"' $(CFLAGS) $(LDFLAGS) $(LIBS) \
+	$(CC) -DVIDEO_TEST -DVERSION='"$(VERSION)"' $(CFLAGS) $(LDFLAGS) $< $(LIBS) \
-	-O0 -g -o $@ $<
+	-o $@

README.txt

@@ -33,7 +33,7 @@ A software and GPU emulated HD output device plugin for VDR.
     o Audio FFMpeg/Alsa/Analog
     o Audio FFMpeg/Alsa/Digital
     o Audio FFMpeg/OSS/Analog
-    o planned: Alsa HDMI/SPDIF Passthrough
+    o Alsa HDMI/SPDIF Passthrough
     o planned: OSS HDMI/SPDIF Passthrough
 
 To compile you must have the 'requires' installed.
@@ -92,24 +92,66 @@ Setup: /etc/vdr/setup.conf
 ------
 	Following is supported:
 
-	softhddevice.Deinterlace = 0
-	0 = bob, 1 = weave, 2 = temporal, 3 = temporal_spatial, 4 = software
-	(only 0, 1 supported with vaapi)
-
 	softhddevice.MakePrimary = 1
 	0 = no change, 1 make softhddevice primary at start
 
-	softhddevice.Scaling = 0
+	softhddevice.HideMainMenuEntry = 0
+	0 = show softhddevice main menu entry, 1 = hide entry
+
+	<res> of the next parameters is 567i, 720p, 1080i_fake or 1080i.
+	1080i_fake is 1280x1080 or 1440x1080
+	1080i is "real" 1920x1080
+
+	softhddevice.<res>.Scaling = 0
 	0 = normal, 1 = fast, 2 = HQ, 3 = anamorphic
 
+	softhddevice.<res>.Deinterlace = 0
+	0 = bob, 1 = weave, 2 = temporal, 3 = temporal_spatial, 4 = software
+	(only 0, 1 supported with vaapi)
+
+	softhddevice.<res>.SkipChromaDeinterlace = 0
+	0 = disabled, 1 = enabled (for slower cards, poor qualit<69>t)
+
+	softhddevice.<res>.Denoise = 0
+	0 .. 1000 noise reduction level (0 off, 1000 max)
+
+	softhddevice.<res>.Sharpness = 0
+	-1000 .. 1000 noise reduction level (0 off, -1000 max blur,
+	    1000 max sharp)
+
 	softhddevice.AudioDelay = 0
 	+n or -n ms
 
+	softhddevice.AudioPassthrough = 0
+	0 = none, 1 = AC-3
+
+Setup: /etc/vdr/remote.conf
+------
+
+	Add "XKeySym." definitions to /etc/vdr/remote.conf to control
+	the vdr and plugin with the connected input device.
+
+	fe.
+	XKeySym.Up	Up
+	XKeySym.Down	Down
+	...
+
+	Additional to the x11 input sends the window close button "Close".
+
+	fe.
+	XKeySym.Power	Close
+
 Commandline:
 ------------
 
 	Use vdr -h to see the command line arguments support by the plugin.
 
+Running:
+--------
+
+	Click into video window to toggle fullscreen/window mode, only if you
+	have a window manager running.
+
 Warning:
 --------
 	libav is not supported, expect many bugs with it.

Todo

@@ -19,22 +19,26 @@ GNU Affero General Public License for more details.
 $Id: $
 
 missing:
-    video out with xv
-    video out with opengl
-    software decoder for xv / opengl
     software deinterlace
     auto crop
-    atmolight
+    zoom/fit-zoom 4:3 (SetVideoDisplayFormat, SetVideoFormat?)
-    zoom/fit-zoom 4:3
+    ITU BT601, ITU BT709 (HD), RGB studio levels (16-235)?
-    multistream handling
+    suspend output / energie saver: stop audio, stop video, configurable
-    HDMI/SPDIF Passthrough
+    Option deinterlace off / deinterlace force!
-    disable screensaver
+    Make output drivers better moduluar.
-    disable window cursor
 
 vdpau:
-    1080i with temporal spatial and level 1 scaling too slow with GT 520
+    1080i with temporal spatial and level 1 scaling too slow with my GT 520
-    1080i with temporal spatial too slow with GT 520 on some channels
+    1080i with temporal spatial too slow with my GT 520 on some channels
+    SkipChromaDeinterlace improves performance
+    Improve OSD handling, show only what is used. Big OSD costs performance
     VdpPreemptionCallback handling
+    hard channel switch
+    suspendoutput didn't show logo or black picture.
+
+libva:
+    hard channel switch
+    yaepghd (VaapiSetOutputPosition) support
 
 libva-intel-driver:
     intel still has hangups most with 1080i
@@ -50,16 +54,14 @@ libva-vdpau-driver:
 libva-xvba-driver:
 
 x11:
-    support resize of x11 window
+    disable screensaver
-    support fullscreen window
-    support fullscreen / window toggle
-    close window should send power button
-    disable cursor
 
 audio/alsa:
     done? video/audio asyncron
     random crashes in av_parser_parse2, when switching channels
     sometimes alsa hangs
+    fixed? snd_pcm_state: Assertion `pcm' failed. while switching channels
+    (thread problem)
 
     better downmix of >2 channels on 2 channel hardware
     remix support of unsupported sample rates
@@ -67,17 +69,38 @@ audio/alsa:
     ffmpeg didn't support resample of 5 to 2 channels
     CodecAudioOpen can fail "can't open audio codec" and does Fatal exit.
 
+audio:
+    write TS -> PES parser, which feeds audio before the next start packet
+
 audio/oss:
     alsa oss emulation mixer "pcm" not working
     ring buffer overflow with alsa oss emulation
 
+HDMI/SPDIF Passthrough:
+    only AC-3 written
+    Channels are wrong setup, if changing setting during operation.
+    split pcm and ac-3 out into two devices
+
 playback of recording
-    play back is too fast
+    pause is not reset, when replay exit
+    replay/pause need 100% cpu
 
 setup:
     Setup of decoder type.
     Setup of output type.
     Setup of display type.
     Setup 4:3 zoom type
-    Setup parameters are not used until restart.
+    Some setup parameters are not used until restart.
 	Can a notice be added to the setup menu?
+    576i, 720p, fake 1080i, 1080i
+
+future features (not planed for 1.0 - 1.5)
+
+    video out with xv
+    video out with opengl
+    video out with xvba
+    software decoder for xv / opengl
+    atmolight support
+    multistream handling
+
+    upmix stereo to AC-3

audio.c

@@ -43,6 +43,8 @@
 #endif
 //#define USE_ALSA			///< enable alsa support
 //#define USE_OSS			///< enable oss support
+#define noSEARCH_HDMI_BUG
+#define noSEARCH_HDMI_BUG2
 
 #include <stdio.h>
 #include <stdint.h>
@@ -88,6 +90,8 @@
 #endif
 #endif
 
+#include <alsa/iatomic.h>		// portable atomic_t
+
 #include "ringbuffer.h"
 #include "misc.h"
 #include "audio.h"
@@ -110,6 +114,95 @@ static int64_t AudioPTS;		///< audio pts clock
 static pthread_cond_t AudioStartCond;	///< condition variable
 #endif
 
+#ifdef SEARCH_HDMI_BUG2
+
+//----------------------------------------------------------------------------
+//	ring buffer
+//----------------------------------------------------------------------------
+
+// FIXME: use this code, to combine alsa&oss ring buffers
+
+#define AUDIO_RING_MAX 8		///< number of audio ring buffers
+
+/**
+**	Audio ring buffer.
+*/
+typedef struct _audio_ring_ring_
+{
+    char FlushBuffers;			///< flag: flush buffers
+    unsigned SampleRate;		///< sample rate in hz
+    unsigned Channels;			///< number of channels
+} AudioRingRing;
+
+    /// ring of audio ring buffers
+static AudioRingRing AudioRing[AUDIO_RING_MAX];
+static int AudioRingWrite;		///< audio ring write pointer
+static int AudioRingRead;		///< audio ring read pointer
+static atomic_t AudioRingFilled;	///< how many of the ring is used
+
+/**
+**	Add sample rate, number of channel change to ring.
+**
+**	@param freq	sample frequency
+**	@param channels	number of channels
+*/
+static int AudioRingAdd(int freq, int channels)
+{
+    int filled;
+
+    filled = atomic_read(&AudioRingFilled);
+    if (filled == AUDIO_RING_MAX) {	// no free slot
+	// FIXME: can wait for ring buffer empty
+	Error(_("audio: out of ring buffers\n"));
+	return -1;
+    }
+    AudioRing[AudioRingWrite].FlushBuffers = 1;
+    AudioRing[AudioRingWrite].SampleRate = freq;
+    AudioRing[AudioRingWrite].Channels = channels;
+
+    AudioRingWrite = (AudioRingWrite + 1) % AUDIO_RING_MAX;
+    atomic_inc(&AudioRingFilled);
+
+#ifdef USE_AUDIO_THREAD
+    // tell thread, that something todo
+    AudioRunning = 1;
+    pthread_cond_signal(&AudioStartCond);
+#endif
+
+    return 0;
+}
+
+/**
+**	Setup audio ring.
+*/
+static void AudioRingInit(void)
+{
+    int i;
+
+    for (i = 0; i < AUDIO_RING_MAX; ++i) {
+	// FIXME:
+	//AlsaRingBuffer = RingBufferNew(48000 * 8 * 2);	// ~1s 8ch 16bit
+    }
+    // one slot always reservered
+    AudioRingWrite = 1;
+    atomic_set(&AudioRingFilled, 1);
+}
+
+/**
+**	Cleanup audio ring.
+*/
+static void AudioRingExit(void)
+{
+    int i;
+
+    for (i = 0; i < AUDIO_RING_MAX; ++i) {
+	// FIXME:
+	//RingBufferDel(AlsaRingBuffer);
+    }
+}
+
+#endif
+
 #ifdef USE_ALSA
 
 //============================================================================
@@ -152,6 +245,7 @@ static int AlsaAddToRingbuffer(const void *samples, int count)
     if (n != count) {
 	Error(_("audio/alsa: can't place %d samples in ring buffer\n"), count);
 	// too many bytes are lost
+	// FIXME: should skip more, longer skip, but less often?
     }
     // Update audio clock
     AudioPTS +=
@@ -202,12 +296,25 @@ static int AlsaPlayRingbuffer(void)
 	    if (first) {
 		// happens with broken alsa drivers
 		Error(_("audio/alsa: broken driver %d\n"), avail);
+		usleep(5 * 1000);
 	    }
 	    Debug(4, "audio/alsa: break state %s\n",
 		snd_pcm_state_name(snd_pcm_state(AlsaPCMHandle)));
 	    break;
 	}
+#ifdef SEARCH_HDMI_BUG
+	{
+	    uint16_t buf[8192];
+	    unsigned u;
 
+	    for (u = 0; u < sizeof(buf) / 2; u++) {
+		buf[u] = random() & 0xffff;
+	    }
+
+	    n = sizeof(buf);
+	    p = buf;
+	}
+#else
 	n = RingBufferGetReadPointer(AlsaRingBuffer, &p);
 	if (!n) {			// ring buffer empty
 	    if (first) {		// only error on first loop
@@ -215,6 +322,7 @@ static int AlsaPlayRingbuffer(void)
 	    }
 	    return 0;
 	}
+#endif
 	if (n < avail) {		// not enough bytes in ring buffer
 	    avail = n;
 	}
@@ -229,12 +337,17 @@ static int AlsaPlayRingbuffer(void)
 	} else {
 	    err = snd_pcm_writei(AlsaPCMHandle, p, frames);
 	}
-	Debug(4, "audio/alsa: wrote %d/%d frames\n", err, frames);
+	//Debug(3, "audio/alsa: wrote %d/%d frames\n", err, frames);
 	if (err != frames) {
 	    if (err < 0) {
 		if (err == -EAGAIN) {
 		    goto again;
 		}
+		/*
+		   if (err == -EBADFD) {
+		   goto again;
+		   }
+		 */
 		Error(_("audio/alsa: underrun error?\n"));
 		err = snd_pcm_recover(AlsaPCMHandle, err, 0);
 		if (err >= 0) {
@@ -255,6 +368,29 @@ static int AlsaPlayRingbuffer(void)
     return 0;
 }
 
+/**
+**	Flush alsa buffers.
+*/
+static void AlsaFlushBuffers(void)
+{
+    int err;
+    snd_pcm_state_t state;
+
+    RingBufferReadAdvance(AlsaRingBuffer, RingBufferUsedBytes(AlsaRingBuffer));
+    state = snd_pcm_state(AlsaPCMHandle);
+    Debug(3, "audio/alsa: state %d - %s\n", state, snd_pcm_state_name(state));
+    if (state != SND_PCM_STATE_OPEN) {
+	if ((err = snd_pcm_drop(AlsaPCMHandle)) < 0) {
+	    Error(_("audio: snd_pcm_drop(): %s\n"), snd_strerror(err));
+	}
+	// ****ing alsa crash, when in open state here
+	if ((err = snd_pcm_prepare(AlsaPCMHandle)) < 0) {
+	    Error(_("audio: snd_pcm_prepare(): %s\n"), snd_strerror(err));
+	}
+    }
+    AudioPTS = INT64_C(0x8000000000000000);
+}
+
 #if 0
 
 //----------------------------------------------------------------------------
@@ -447,8 +583,20 @@ static void AlsaThread(void)
     for (;;) {
 	int err;
 
-	Debug(4, "audio: play loop\n");
 	pthread_testcancel();
+	if (AlsaFlushBuffer) {
+	    // we can flush too many, but wo cares
+	    Debug(3, "audio/alsa: flushing buffers\n");
+	    AlsaFlushBuffers();
+	    /*
+	       if ((err = snd_pcm_prepare(AlsaPCMHandle))) {
+	       Error(_("audio: snd_pcm_prepare(): %s\n"), snd_strerror(err));
+	       }
+	     */
+	    AlsaFlushBuffer = 0;
+	    break;
+	}
+	// wait for space in kernel buffers
 	if ((err = snd_pcm_wait(AlsaPCMHandle, 100)) < 0) {
 	    Error(_("audio/alsa: wait underrun error?\n"));
 	    err = snd_pcm_recover(AlsaPCMHandle, err, 0);
@@ -460,20 +608,7 @@ static void AlsaThread(void)
 	    continue;
 	}
 	if (AlsaFlushBuffer) {
-	    // we can flush too many, but wo cares
+	    continue;
-	    Debug(3, "audio/alsa: flushing buffers\n");
-	    RingBufferReadAdvance(AlsaRingBuffer,
-		RingBufferUsedBytes(AlsaRingBuffer));
-#if 1
-	    if ((err = snd_pcm_drop(AlsaPCMHandle))) {
-		Error(_("audio: snd_pcm_drop(): %s\n"), snd_strerror(err));
-	    }
-	    if ((err = snd_pcm_prepare(AlsaPCMHandle))) {
-		Error(_("audio: snd_pcm_prepare(): %s\n"), snd_strerror(err));
-	    }
-#endif
-	    AlsaFlushBuffer = 0;
-	    break;
 	}
 	if ((err = AlsaPlayRingbuffer())) {	// empty / error
 	    snd_pcm_state_t state;
@@ -486,7 +621,8 @@ static void AlsaThread(void)
 		Debug(3, "audio/alsa: stopping play\n");
 		break;
 	    }
-	    usleep(20 * 1000);
+	    pthread_yield();
+	    usleep(20 * 1000);		// let fill the buffers
 	}
     }
 }
@@ -499,8 +635,9 @@ static void AlsaThread(void)
 */
 static void AlsaEnqueue(const void *samples, int count)
 {
-    if (!AlsaRingBuffer || !AlsaPCMHandle) {
+    if (!AlsaRingBuffer || !AlsaPCMHandle || !AudioSampleRate) {
-	Debug(3, "audio/alsa: not ready\n");
+	printf("%p %p %d\n", AlsaRingBuffer, AlsaPCMHandle, AudioSampleRate);
+	Debug(3, "audio/alsa: enqueue not ready\n");
 	return;
     }
     if (AlsaAddToRingbuffer(samples, count)) {
@@ -518,35 +655,48 @@ static void AlsaEnqueue(const void *samples, int count)
 #endif
 
 /**
-**	Initialize alsa pcm device.
+**	Open alsa pcm device.
-**
-**	@see AudioPCMDevice
 */
-static void AlsaInitPCM(void)
+static snd_pcm_t *AlsaOpenPCM(void)
 {
     const char *device;
     snd_pcm_t *handle;
-    snd_pcm_hw_params_t *hw_params;
     int err;
-    snd_pcm_uframes_t buffer_size;
 
     if (!(device = AudioPCMDevice)) {
 	if (!(device = getenv("ALSA_DEVICE"))) {
 	    device = "default";
 	}
     }
-    // FIXME: must set alsa error output to /dev/null
     if ((err =
 	    snd_pcm_open(&handle, device, SND_PCM_STREAM_PLAYBACK,
 		SND_PCM_NONBLOCK)) < 0) {
-	Fatal(_("audio/alsa: playback open '%s' error: %s\n"), device,
+	Error(_("audio/alsa: playback open '%s' error: %s\n"), device,
 	    snd_strerror(err));
-	// FIXME: no fatal error for plugins!
+	return NULL;
     }
 
     if ((err = snd_pcm_nonblock(handle, 0)) < 0) {
 	Error(_("audio/alsa: can't set block mode: %s\n"), snd_strerror(err));
     }
+    return handle;
+}
+
+/**
+**	Initialize alsa pcm device.
+**
+**	@see AudioPCMDevice
+*/
+static void AlsaInitPCM(void)
+{
+    snd_pcm_t *handle;
+    snd_pcm_hw_params_t *hw_params;
+    int err;
+    snd_pcm_uframes_t buffer_size;
+
+    if (!(handle = AlsaOpenPCM())) {
+	return;
+    }
 
     snd_pcm_hw_params_alloca(&hw_params);
     // choose all parameters
@@ -556,8 +706,7 @@ static void AlsaInitPCM(void)
 	    snd_strerror(err));
     }
     AlsaCanPause = snd_pcm_hw_params_can_pause(hw_params);
-    Info(_("audio/alsa: hw '%s' supports pause: %s\n"), device,
+    Info(_("audio/alsa: supports pause: %s\n"), AlsaCanPause ? "yes" : "no");
-	AlsaCanPause ? "yes" : "no");
     snd_pcm_hw_params_get_buffer_size_max(hw_params, &buffer_size);
     Info(_("audio/alsa: max buffer size %lu\n"), buffer_size);
 
@@ -656,7 +805,7 @@ static uint64_t AlsaGetDelay(void)
     snd_pcm_sframes_t delay;
     uint64_t pts;
 
-    if (!AlsaPCMHandle) {
+    if (!AlsaPCMHandle || !AudioSampleRate) {
 	return 0UL;
     }
     // FIXME: thread safe? __assert_fail_base in snd_pcm_delay
@@ -702,26 +851,24 @@ static int AlsaSetup(int *freq, int *channels)
     snd_pcm_uframes_t period_size;
     int err;
     int ret;
+    snd_pcm_t *handle;
 
     if (!AlsaPCMHandle) {		// alsa not running yet
 	return -1;
     }
-#if 1
+#if 1					// easy alsa hw setup way
     // flush any buffered data
-#ifdef USE_AUDIO_THREAD
+    AudioFlushBuffers();
-    if (AudioRunning) {
+
-	while (AudioRunning) {
+    if (1) {				// close+open to fix hdmi no sound bugs
-	    AlsaFlushBuffer = 1;
+	handle = AlsaPCMHandle;
-	    usleep(1 * 1000);
+	AlsaPCMHandle = NULL;
+	snd_pcm_close(handle);
+	if (!(handle = AlsaOpenPCM())) {
+	    return -1;
 	}
-	AlsaFlushBuffer = 0;
+	AlsaPCMHandle = handle;
-    } else
-#endif
-    {
-	RingBufferReadAdvance(AlsaRingBuffer,
-	    RingBufferUsedBytes(AlsaRingBuffer));
     }
-    AudioPTS = INT64_C(0x8000000000000000);
 
     ret = 0;
   try_again:
@@ -844,6 +991,41 @@ static int AlsaSetup(int *freq, int *channels)
     // FIXME: use hw_params for buffer_size period_size
 #endif
 
+#if 1
+    if (0) {				// no underruns allowed, play silence
+	snd_pcm_sw_params_t *sw_params;
+	snd_pcm_uframes_t boundary;
+
+	snd_pcm_sw_params_alloca(&sw_params);
+	err = snd_pcm_sw_params_current(AlsaPCMHandle, sw_params);
+	if (err < 0) {
+	    Error(_("audio: snd_pcm_sw_params_current failed: %s\n"),
+		snd_strerror(err));
+	}
+	if ((err = snd_pcm_sw_params_get_boundary(sw_params, &boundary)) < 0) {
+	    Error(_("audio: snd_pcm_sw_params_get_boundary failed: %s\n"),
+		snd_strerror(err));
+	}
+	Debug(4, "audio/alsa: boundary %lu frames\n", boundary);
+	if ((err =
+		snd_pcm_sw_params_set_stop_threshold(AlsaPCMHandle, sw_params,
+		    boundary)) < 0) {
+	    Error(_("audio: snd_pcm_sw_params_set_silence_size failed: %s\n"),
+		snd_strerror(err));
+	}
+	if ((err =
+		snd_pcm_sw_params_set_silence_size(AlsaPCMHandle, sw_params,
+		    boundary)) < 0) {
+	    Error(_("audio: snd_pcm_sw_params_set_silence_size failed: %s\n"),
+		snd_strerror(err));
+	}
+	if ((err = snd_pcm_sw_params(AlsaPCMHandle, sw_params)) < 0) {
+	    Error(_("audio: snd_pcm_sw_params failed: %s\n"),
+		snd_strerror(err));
+	}
+    }
+#endif
+
     // update buffer
 
     snd_pcm_get_params(AlsaPCMHandle, &buffer_size, &period_size);
@@ -954,6 +1136,7 @@ static int OssAddToRingbuffer(const void *samples, int count)
     if (n != count) {
 	Error(_("audio/oss: can't place %d samples in ring buffer\n"), count);
 	// too many bytes are lost
+	// FIXME: should skip more, longer skip, but less often?
     }
     // Update audio clock
     AudioPTS +=
@@ -1020,6 +1203,21 @@ static int OssPlayRingbuffer(void)
     return 0;
 }
 
+/**
+**	Flush oss buffers.
+*/
+static void OssFlushBuffers(void)
+{
+    RingBufferReadAdvance(OssRingBuffer, RingBufferUsedBytes(OssRingBuffer));
+    // flush kernel buffers
+    if (ioctl(OssPcmFildes, SNDCTL_DSP_HALT_OUTPUT, NULL) < 0) {
+	Error(_("audio/oss: ioctl(SNDCTL_DSP_HALT_OUTPUT): %s\n"),
+	    strerror(errno));
+	return;
+    }
+    AudioPTS = INT64_C(0x8000000000000000);
+}
+
 //----------------------------------------------------------------------------
 //	OSS pcm polled
 //----------------------------------------------------------------------------
@@ -1235,16 +1433,8 @@ static int OssSetup(int *freq, int *channels)
     // flush any buffered data
     {
 	AudioRunning = 0;
-	RingBufferReadAdvance(OssRingBuffer,
+	OssFlushBuffers();
-	    RingBufferUsedBytes(OssRingBuffer));
-	// flush kernel buffers
-	if (ioctl(OssPcmFildes, SNDCTL_DSP_HALT_OUTPUT, NULL) == -1) {
-	    Error(_("audio/oss: ioctl(SNDCTL_DSP_HALT_OUTPUT): %s\n"),
-		strerror(errno));
-	    return -1;
-	}
     }
-    AudioPTS = INT64_C(0x8000000000000000);
 
     ret = 0;
 
@@ -1375,12 +1565,59 @@ static void *AudioPlayHandlerThread(void *dummy)
 	Debug(3, "audio: wait on start condition\n");
 	pthread_mutex_lock(&AudioMutex);
 	AudioRunning = 0;
+#ifndef SEARCH_HDMI_BUG
 	do {
 	    pthread_cond_wait(&AudioStartCond, &AudioMutex);
 	    // cond_wait can return, without signal!
 	} while (!AudioRunning);
+#else
+	usleep(1 * 1000);
+	AudioRunning = 1;
+#endif
 	pthread_mutex_unlock(&AudioMutex);
 
+#ifdef SEARCH_HDMI_BUG2
+	if (atomic_read(&AudioRingFilled) > 1) {
+	    int sample_rate;
+	    int channels;
+
+	    // skip all sample changes between
+	    while (atomic_read(&AudioRingFilled) > 1) {
+		Debug(3, "audio: skip ring buffer\n");
+		AudioRingRead = (AudioRingRead + 1) % AUDIO_RING_MAX;
+		atomic_dec(&AudioRingFilled);
+	    }
+
+#ifdef USE_ALSA
+	    // FIXME: flush only if there is something to flush
+	    AlsaFlushBuffers();
+
+	    sample_rate = AudioRing[AudioRingRead].SampleRate;
+	    channels = AudioRing[AudioRingRead].Channels;
+	    Debug(3, "audio: thread channels %d sample-rate %d hz\n", channels,
+		sample_rate);
+
+	    if (AlsaSetup(&sample_rate, &channels)) {
+		Error(_("audio: can't set channels %d sample-rate %d hz\n"),
+		    channels, sample_rate);
+	    }
+	    Debug(3, "audio: thread channels %d sample-rate %d hz\n",
+		AudioChannels, AudioSampleRate);
+	    if (1) {
+		int16_t buf[6144 / 2];
+
+		buf[0] = htole16(0xF872);	// iec 61937 sync word
+		buf[1] = htole16(0x4E1F);
+		buf[2] = htole16((7 << 5) << 8 | 0x00);
+		buf[3] = htole16(0x0000);
+		memset(buf + 4, 0, 6144 - 8);
+
+		AlsaEnqueue(buf, 6144);
+	    }
+#endif
+	}
+#endif
+
 	Debug(3, "audio: play start\n");
 #ifdef USE_ALSA
 	AlsaThread();
@@ -1406,6 +1643,8 @@ static void AudioInitThread(void)
 	pthread_yield();
     } while (!AlsaPCMHandle);
 #endif
+    pthread_yield();
+    usleep(5 * 1000);
 }
 
 /**
@@ -1415,14 +1654,16 @@ static void AudioExitThread(void)
 {
     void *retval;
 
-    if (pthread_cancel(AudioThread)) {
+    if (AudioThread) {
-	Error(_("audio: can't queue cancel play thread\n"));
+	if (pthread_cancel(AudioThread)) {
+	    Error(_("audio: can't queue cancel play thread\n"));
+	}
+	if (pthread_join(AudioThread, &retval) || retval != PTHREAD_CANCELED) {
+	    Error(_("audio: can't cancel play thread\n"));
+	}
+	pthread_cond_destroy(&AudioStartCond);
+	pthread_mutex_destroy(&AudioMutex);
     }
-    if (pthread_join(AudioThread, &retval) || retval != PTHREAD_CANCELED) {
-	Error(_("audio: can't cancel play thread\n"));
-    }
-    pthread_cond_destroy(&AudioStartCond);
-    pthread_mutex_destroy(&AudioMutex);
 }
 
 #endif
@@ -1448,6 +1689,31 @@ void AudioEnqueue(const void *samples, int count)
     (void)count;
 }
 
+/**
+**	Flush audio buffers.
+*/
+void AudioFlushBuffers(void)
+{
+#ifdef USE_ALSA
+#ifdef USE_AUDIO_THREAD
+    // signal thread to flush buffers
+    if (AudioThread) {
+	AlsaFlushBuffer = 1;
+	do {
+	    AudioRunning = 1;		// wakeup in case of sleeping
+	    pthread_cond_signal(&AudioStartCond);
+	    usleep(1 * 1000);
+	} while (AlsaFlushBuffer);	// wait until flushed
+    }
+#else
+    AlsaFlushBuffers();
+#endif
+#endif
+#ifdef USE_OSS
+    OssFlushBuffers();
+#endif
+}
+
 /**
 **	Call back to play audio polled.
 */
@@ -1463,6 +1729,20 @@ void AudioPoller(void)
 #endif
 }
 
+/**
+**	Get free bytes in audio output.
+*/
+int AudioFreeBytes(void)
+{
+#ifdef USE_ALSA
+    return AlsaRingBuffer ? RingBufferFreeBytes(AlsaRingBuffer) : INT32_MAX;
+#endif
+#ifdef USE_OSS
+    return OssRingBuffer ? RingBufferFreeBytes(OssRingBuffer) : INT32_MAX;
+#endif
+    return INT32_MAX;			// no driver, much space
+}
+
 /**
 **	Set audio clock base.
 **
@@ -1550,6 +1830,10 @@ int AudioSetup(int *freq, int *channels)
 	// FIXME: set flag invalid setup
 	return -1;
     }
+#if defined(SEARCH_HDMI_BUG) || defined(SEARCH_HDMI_BUG2)
+    // FIXME: need to store possible combination and report this
+    return AudioRingAdd(*freq, *channels);
+#endif
 #ifdef USE_ALSA
     return AlsaSetup(freq, channels);
 #endif
@@ -1577,6 +1861,9 @@ void AudioInit(void)
     int freq;
     int chan;
 
+#ifdef SEARCH_HDMI_BUG2
+    AudioRingInit();
+#endif
 #ifdef USE_ALSA
     AlsaInit();
 #endif
@@ -1609,6 +1896,9 @@ void AudioExit(void)
 #ifdef USE_OSS
     OssExit();
 #endif
+#ifdef SEARCH_HDMI_BUG2
+    AudioRingExit();
+#endif
 }
 
 #ifdef AUDIO_TEST

audio.h

@@ -1,7 +1,7 @@
 ///
 ///	@file audio.h		@brief Audio module headerfile
 ///
-///	Copyright (c) 2009 - 2011 by Johns.  All Rights Reserved.
+///	Copyright (c) 2009 - 2012 by Johns.  All Rights Reserved.
 ///
 ///	Contributor(s):
 ///
@@ -28,12 +28,14 @@
 //----------------------------------------------------------------------------
 
 extern void AudioEnqueue(const void *, int);	///< buffer audio samples
+extern void AudioFlushBuffers(void);	///< flush audio buffers
+extern void AudioPoller(void);		///< poll audio events/handling
+
+extern int AudioFreeBytes(void);	///< free bytes in audio output
 
-//extern int AudioFreeBytes(void);	///< free bytes in audio output
 //extern int AudioUsedBytes(void);	///< used bytes in audio output
 extern void AudioSetClock(int64_t);	///< set audio clock base
 extern int64_t AudioGetClock();		///< get current audio clock
-
 extern uint64_t AudioGetDelay(void);	///< get current audio delay
 
 extern int AudioSetup(int *, int *);	///< setup audio output

codec.c

@@ -35,8 +35,12 @@
     */
 #define USE_AVPARSER
 
+    /// compile with passthrough support (experimental)
+#define USE_PASSTHROUGH
+
 #include <stdio.h>
 #include <unistd.h>
+#include <endian.h>
 
 #include <sys/types.h>
 #include <sys/stat.h>
@@ -557,6 +561,16 @@ void CodecVideoDecode(VideoDecoder * decoder, const AVPacket * avpkt)
     }
 }
 
+/**
+**	Flush the video decoder.
+**
+**	@param decoder	video decoder data
+*/
+void CodecVideoFlushBuffers(VideoDecoder * decoder)
+{
+    avcodec_flush_buffers(decoder->VideoCtx);
+}
+
 //----------------------------------------------------------------------------
 //	Audio
 //----------------------------------------------------------------------------
@@ -585,8 +599,17 @@ struct _audio_decoder_
     int HwChannels;			///< hw channels
 
     ReSampleContext *ReSample;		///< audio resampling context
+
 };
 
+#ifdef USE_PASSTHROUGH
+//static char CodecPassthroughPCM;	///< pass pcm through (unsupported)
+static char CodecPassthroughAC3;	///< pass ac3 through
+
+//static char CodecPassthroughDTS;	///< pass dts through (unsupported)
+//static char CodecPassthroughMPA;	///< pass mpa through (unsupported)
+#endif
+
 /**
 **	Allocate a new audio decoder context.
 **
@@ -683,6 +706,18 @@ void CodecAudioClose(AudioDecoder * audio_decoder)
     }
 }
 
+/**
+**	Set audio pass-through.
+*/
+void CodecSetAudioPassthrough(int mask)
+{
+#ifdef USE_PASSTHROUGH
+    CodecPassthroughAC3 = mask & 1 ? 1 : 0;
+#endif
+    // FIXME: must update audio decoder (nr. of channels wrong)
+    (void)mask;
+}
+
 #ifdef USE_AVPARSER
 
 /**
@@ -713,9 +748,11 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
     spkt->pts = avpkt->pts;
     spkt->dts = avpkt->dts;
 #endif
+#ifdef DEBUG
     if (!audio_decoder->AudioParser) {
 	Fatal(_("codec: internal error parser freeded while running\n"));
     }
+#endif
 
     audio_ctx = audio_decoder->AudioCtx;
     index = 0;
@@ -730,6 +767,7 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
 	    !index ? (uint64_t) spkt->pts : AV_NOPTS_VALUE,
 	    !index ? (uint64_t) spkt->dts : AV_NOPTS_VALUE, -1);
 
+	// FIXME: make this a function for both #ifdef cases
 	if (dpkt->size) {
 	    int buf_sz;
 
@@ -765,7 +803,15 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
 		audio_decoder->SampleRate = audio_ctx->sample_rate;
 		audio_decoder->HwSampleRate = audio_ctx->sample_rate;
 		audio_decoder->Channels = audio_ctx->channels;
-		audio_decoder->HwChannels = audio_ctx->channels;
+#ifdef USE_PASSTHROUGH
+		// SPDIF/HDMI passthrough
+		if (CodecPassthroughAC3 && audio_ctx->codec_id == CODEC_ID_AC3) {
+		    audio_decoder->HwChannels = 2;
+		} else
+#endif
+		{
+		    audio_decoder->HwChannels = audio_ctx->channels;
+		}
 
 		// channels not support?
 		if ((err =
@@ -826,6 +872,81 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
 			AudioEnqueue(outbuf, outlen);
 		    }
 		} else {
+#ifdef USE_PASSTHROUGH
+		    // SPDIF/HDMI passthrough
+		    if (CodecPassthroughAC3
+			&& audio_ctx->codec_id == CODEC_ID_AC3) {
+			// build SPDIF header and append A52 audio to it
+			// dpkt is the original data
+			buf_sz = 6144;
+			if (buf_sz < dpkt->size + 8) {
+			    Error(_
+				("codec/audio: decoded data smaller than encoded\n"));
+			    break;
+			}
+			// copy original data for output
+			// FIXME: not 100% sure, if endian is correct
+			buf[0] = htole16(0xF872);	// iec 61937 sync word
+			buf[1] = htole16(0x4E1F);
+			buf[2] = htole16(0x01 | (dpkt->data[5] & 0x07) << 8);
+			buf[3] = htole16(dpkt->size * 8);
+			swab(dpkt->data, buf + 4, dpkt->size);
+			memset(buf + 4 + dpkt->size / 2, 0,
+			    buf_sz - 8 - dpkt->size);
+		    }
+#if 0
+		    //
+		    //	old experimental code
+		    //
+		    if (1) {
+			// FIXME: need to detect dts
+			// copy original data for output
+			// FIXME: buf is sint
+			buf[0] = 0x72;
+			buf[1] = 0xF8;
+			buf[2] = 0x1F;
+			buf[3] = 0x4E;
+			buf[4] = 0x00;
+			switch (dpkt->size) {
+			    case 512:
+				buf[5] = 0x0B;
+				break;
+			    case 1024:
+				buf[5] = 0x0C;
+				break;
+			    case 2048:
+				buf[5] = 0x0D;
+				break;
+			    default:
+				Debug(3,
+				    "codec/audio: dts sample burst not supported\n");
+				buf[5] = 0x00;
+				break;
+			}
+			buf[6] = (dpkt->size * 8);
+			buf[7] = (dpkt->size * 8) >> 8;
+			//buf[8] = 0x0B;
+			//buf[9] = 0x77;
+			//printf("%x %x\n", dpkt->data[0],dpkt->data[1]);
+			// swab?
+			memcpy(buf + 8, dpkt->data, dpkt->size);
+			memset(buf + 8 + dpkt->size, 0,
+			    buf_sz - 8 - dpkt->size);
+		    } else if (1) {
+			// FIXME: need to detect mp2
+			// FIXME: mp2 passthrough
+			// see softhddev.c version/layer
+			// 0x04 mpeg1 layer1
+			// 0x05 mpeg1 layer23
+			// 0x06 mpeg2 ext
+			// 0x07 mpeg2.5 layer 1
+			// 0x08 mpeg2.5 layer 2
+			// 0x09 mpeg2.5 layer 3
+		    }
+		    // DTS HD?
+		    // True HD?
+#endif
+#endif
 		    AudioEnqueue(buf, buf_sz);
 		}
 	    }
@@ -904,90 +1025,7 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
 	avcodec_decode_audio4(audio_ctx, frame, &got_frame, dpkt);
 #else
 #endif
-	if (buf_sz > 0) {		// something decoded
+	// FIXME: see above, old code removed
-	    // Update audio clock
-	    if ((uint64_t) spkt->pts != AV_NOPTS_VALUE) {
-		AudioSetClock(spkt->pts);
-		spkt->pts = AV_NOPTS_VALUE;
-	    }
-	    // FIXME: must first play remainings bytes, than change and play new.
-	    if (audio_decoder->SampleRate != audio_ctx->sample_rate
-		|| audio_decoder->Channels != audio_ctx->channels) {
-		int err;
-
-		if (audio_decoder->ReSample) {
-		    audio_resample_close(audio_decoder->ReSample);
-		    audio_decoder->ReSample = NULL;
-		}
-
-		audio_decoder->SampleRate = audio_ctx->sample_rate;
-		audio_decoder->HwSampleRate = audio_ctx->sample_rate;
-		audio_decoder->Channels = audio_ctx->channels;
-		audio_decoder->HwChannels = audio_ctx->channels;
-
-		// channels not support?
-		if ((err =
-			AudioSetup(&audio_decoder->HwSampleRate,
-			    &audio_decoder->HwChannels))) {
-		    Debug(3, "codec/audio: resample %dHz *%d -> %dHz *%d\n",
-			audio_ctx->sample_rate, audio_ctx->channels,
-			audio_decoder->HwSampleRate,
-			audio_decoder->HwChannels);
-
-		    if (err == 1) {
-			audio_decoder->ReSample =
-			    av_audio_resample_init(audio_decoder->HwChannels,
-			    audio_ctx->channels, audio_decoder->HwSampleRate,
-			    audio_ctx->sample_rate, audio_ctx->sample_fmt,
-			    audio_ctx->sample_fmt, 16, 10, 0, 0.8);
-			// libav-0.8_pre didn't support 6 -> 2 channels
-			if (!audio_decoder->ReSample) {
-			    Error(_("codec/audio: resample setup error\n"));
-			    audio_decoder->HwChannels = 0;
-			    audio_decoder->HwSampleRate = 0;
-			}
-		    } else {
-			// FIXME: handle errors
-			Debug(3, "codec/audio: audio setup error\n");
-			audio_decoder->HwChannels = 0;
-			audio_decoder->HwSampleRate = 0;
-			break;
-		    }
-		}
-	    }
-
-	    if (audio_decoder->HwSampleRate && audio_decoder->HwChannels) {
-		// need to resample audio
-		if (audio_decoder->ReSample) {
-		    int16_t outbuf[(AVCODEC_MAX_AUDIO_FRAME_SIZE * 3) / 4 +
-			FF_INPUT_BUFFER_PADDING_SIZE]
-			__attribute__ ((aligned(16)));
-		    int outlen;
-
-		    // FIXME: libav-0.7.2 crash here
-		    outlen =
-			audio_resample(audio_decoder->ReSample, outbuf, buf,
-			buf_sz);
-#ifdef DEBUG
-		    if (outlen != buf_sz) {
-			Debug(3, "codec/audio: possible fixed ffmpeg\n");
-		    }
-#endif
-		    if (outlen) {
-			// outlen seems to be wrong in ffmpeg-0.9
-			outlen /= audio_ctx->channels *
-			    av_get_bytes_per_sample(audio_ctx->sample_fmt);
-			outlen *=
-			    audio_decoder->HwChannels *
-			    av_get_bytes_per_sample(audio_ctx->sample_fmt);
-			Debug(4, "codec/audio: %d -> %d\n", buf_sz, outlen);
-			AudioEnqueue(outbuf, outlen);
-		    }
-		} else {
-		    AudioEnqueue(buf, buf_sz);
-		}
-	    }
-	}
 
 	index += n;
     }
@@ -1000,6 +1038,17 @@ void CodecAudioDecode(AudioDecoder * audio_decoder, const AVPacket * avpkt)
 
 #endif
 
+/**
+**	Flush the audio decoder.
+**
+**	@param decoder	audio decoder data
+*/
+void CodecAudioFlushBuffers(AudioDecoder * decoder)
+{
+    // FIXME: reset audio parser
+    avcodec_flush_buffers(decoder->AudioCtx);
+}
+
 //----------------------------------------------------------------------------
 //	Codec
 //----------------------------------------------------------------------------

codec.h

@@ -1,7 +1,7 @@
 ///
 ///	@file codec.h	@brief Codec module headerfile
 ///
-///	Copyright (c) 2009 - 2011 by Johns.  All Rights Reserved.
+///	Copyright (c) 2009 - 2012 by Johns.  All Rights Reserved.
 ///
 ///	Contributor(s):
 ///
@@ -40,26 +40,32 @@ typedef struct _audio_decoder_ AudioDecoder;
     /// Allocate a new video decoder context.
 extern VideoDecoder *CodecVideoNewDecoder(VideoHwDecoder *);
 
-    /// Open video codec
+    /// Open video codec.
 extern void CodecVideoOpen(VideoDecoder *, const char *, int);
 
-    /// Close video codec
+    /// Close video codec.
 extern void CodecVideoClose(VideoDecoder *);
 
-    /// Decode a video packet
+    /// Decode a video packet.
-extern void CodecVideoDecode(VideoDecoder *, const AVPacket * pkt);
+extern void CodecVideoDecode(VideoDecoder *, const AVPacket *);
+
+    /// Flush video buffers.
+extern void CodecVideoFlushBuffers(VideoDecoder *);
 
     /// Allocate a new audio decoder context.
 extern AudioDecoder *CodecAudioNewDecoder(void);
 
-    /// Open audio codec
+    /// Open audio codec.
 extern void CodecAudioOpen(AudioDecoder *, const char *, int);
 
-    /// Close audio codec
+    /// Close audio codec.
 extern void CodecAudioClose(AudioDecoder *);
 
-    /// Decode an audio packet
+    /// Decode an audio packet.
-extern void CodecAudioDecode(AudioDecoder *, const AVPacket * pkt);
+extern void CodecAudioDecode(AudioDecoder *, const AVPacket *);
+
+    /// Flush audio buffers.
+extern void CodecAudioFlushBuffers(AudioDecoder *);
 
     /// Setup and initialize codec module.
 extern void CodecInit(void);

softhddev.c

@@ -35,6 +35,11 @@
 
 #include <libavcodec/avcodec.h>
 
+#ifndef __USE_GNU
+#define __USE_GNU
+#endif
+#include <pthread.h>
+
 #include "misc.h"
 #include "softhddev.h"
 
@@ -42,7 +47,9 @@
 #include "video.h"
 #include "codec.h"
 
-static char BrokenThreadsAndPlugins;	///< broken vdr threads and plugins
+//////////////////////////////////////////////////////////////////////////////
+//	Variables
+//////////////////////////////////////////////////////////////////////////////
 
 #ifdef USE_VDPAU
 static char ConfigVdpauDecoder = 1;	///< use vdpau decoder, if possible
@@ -50,16 +57,23 @@ static char ConfigVdpauDecoder = 1;	///< use vdpau decoder, if possible
 #define ConfigVdpauDecoder 0		///< no vdpau decoder configured
 #endif
 
+static char ConfigFullscreen;		///< fullscreen modus
+static char ConfigSuspendClose = 1;	///< suspend should close devices
+static char ConfigSuspendX11 = 1;	///< suspend should stop x11
+
+static pthread_mutex_t SuspendLockMutex;	///< suspend lock mutex
+
+static volatile char VideoFreezed;	///< video freezed
+
 //////////////////////////////////////////////////////////////////////////////
 //	Audio
 //////////////////////////////////////////////////////////////////////////////
 
 static volatile char NewAudioStream;	///< new audio stream
+static volatile char SkipAudio;		///< skip audio stream
 static AudioDecoder *MyAudioDecoder;	///< audio decoder
 static enum CodecID AudioCodecID;	///< current codec id
 
-extern void AudioTest(void);		// FIXME:
-
 /**
 **	mpeg bitrate table.
 **
@@ -183,29 +197,37 @@ static int FindAudioSync(const AVPacket * avpkt)
 **	@param size	size of PES packet
 **	@param id	PES packet type
 */
-void PlayAudio(const uint8_t * data, int size,
+int PlayAudio(const uint8_t * data, int size,
     __attribute__ ((unused)) uint8_t id)
 {
     int n;
+    int osize;
     AVPacket avpkt[1];
 
-    if (BrokenThreadsAndPlugins) {
-	return;
-    }
     // channel switch: SetAudioChannelDevice: SetDigitalAudioDevice:
 
+    if (VideoFreezed) {			// video freezed
+	return 0;
+    }
     if (NewAudioStream) {
 	// FIXME: does this clear the audio ringbuffer?
 	CodecAudioClose(MyAudioDecoder);
 	AudioCodecID = CODEC_ID_NONE;
 	NewAudioStream = 0;
     }
+    if (SkipAudio) {			// skip audio
+	return size;
+    }
     // PES header 0x00 0x00 0x01 ID
     // ID 0xBD 0xC0-0xCF
 
     if (size < 9) {
 	Error(_("[softhddev] invalid audio packet\n"));
-	return;
+	return size;
+    }
+    // Don't overrun audio buffers on replay
+    if (AudioFreeBytes() < 3072 * 8 * 8) {	// 8 channels 8 packets
+	return 0;
     }
 
     n = data[8];			// header size
@@ -226,11 +248,12 @@ void PlayAudio(const uint8_t * data, int size,
 	}
     }
 
+    osize = size;
     data += 9 + n;
     size -= 9 + n;			// skip pes header
     if (size <= 0) {
 	Error(_("[softhddev] invalid audio packet\n"));
-	return;
+	return osize;
     }
     // Detect audio code
     // MPEG-PS mp2 MPEG1, MPEG2, AC3
@@ -272,7 +295,7 @@ void PlayAudio(const uint8_t * data, int size,
 	    avpkt->size = size;
 	    n = FindAudioSync(avpkt);
 	    if (n < 0) {
-		return;
+		return osize;
 	    }
 	    if (!MyAudioDecoder) {
 		MyAudioDecoder = CodecAudioNewDecoder();
@@ -287,13 +310,15 @@ void PlayAudio(const uint8_t * data, int size,
 
     // no decoder or codec known
     if (!MyAudioDecoder || AudioCodecID == CODEC_ID_NONE) {
-	return;
+	return osize;
     }
 
     avpkt->data = (void *)data;
     avpkt->size = size;
     //memset(avpkt->data + avpkt->size, 0, FF_INPUT_BUFFER_PADDING_SIZE);
     CodecAudioDecode(MyAudioDecoder, avpkt);
+
+    return osize;
 }
 
 /**
@@ -301,10 +326,8 @@ void PlayAudio(const uint8_t * data, int size,
 */
 void Mute(void)
 {
-    if (BrokenThreadsAndPlugins) {
+    SkipAudio = 1;
-	return;
+    //AudioSetVolume(0);
-    }
-    AudioSetVolume(0);
 }
 
 /**
@@ -314,9 +337,6 @@ void Mute(void)
 */
 void SetVolumeDevice(int volume)
 {
-    if (BrokenThreadsAndPlugins) {
-	return;
-    }
     AudioSetVolume((volume * 100) / 255);
 }
 
@@ -342,8 +362,8 @@ static AVPacket VideoPacketRb[VIDEO_PACKET_MAX];
 static int VideoPacketWrite;		///< write pointer
 static int VideoPacketRead;		///< read pointer
 static atomic_t VideoPacketsFilled;	///< how many of the buffer is used
-static volatile char VideoFreezed;	///< video freezed
 static volatile char VideoClearBuffers;	///< clear video buffers
+static volatile char SkipVideo;		///< skip video
 
 #ifdef DEBUG
 static int VideoMaxPacketSize;		///< biggest used packet buffer
@@ -356,8 +376,6 @@ static void VideoPacketInit(void)
 {
     int i;
 
-    Debug(4, "[softhddev]: %s\n", __FUNCTION__);
-
     for (i = 0; i < VIDEO_PACKET_MAX; ++i) {
 	AVPacket *avpkt;
 
@@ -379,16 +397,10 @@ static void VideoPacketExit(void)
 {
     int i;
 
-    Debug(4, "[softhddev]: %s\n", __FUNCTION__);
-
     atomic_set(&VideoPacketsFilled, 0);
 
     for (i = 0; i < VIDEO_PACKET_MAX; ++i) {
-	AVPacket *avpkt;
+	av_free_packet(&VideoPacketRb[i]);
-
-	avpkt = &VideoPacketRb[i];
-	// build a clean ffmpeg av packet
-	av_free_packet(avpkt);
     }
 }
 
@@ -439,6 +451,8 @@ static void VideoEnqueue(int64_t pts, const void *data, int size)
 
 /**
 **	Finish current packet advance to next.
+**
+**	@param codec_id	codec id of packet (MPEG/H264)
 */
 static void VideoNextPacket(int codec_id)
 {
@@ -469,13 +483,13 @@ static void VideoNextPacket(int codec_id)
     VideoPacketWrite = (VideoPacketWrite + 1) % VIDEO_PACKET_MAX;
     atomic_inc(&VideoPacketsFilled);
 
+    VideoDisplayWakeup();
+
     // intialize next package to use
     avpkt = &VideoPacketRb[VideoPacketWrite];
     avpkt->stream_index = 0;
     avpkt->pts = AV_NOPTS_VALUE;
     avpkt->dts = AV_NOPTS_VALUE;
-
-    VideoDisplayHandler();
 }
 
 /**
@@ -493,6 +507,10 @@ int VideoDecode(void)
     }
     if (VideoClearBuffers) {
 	atomic_set(&VideoPacketsFilled, 0);
+	VideoPacketRead = VideoPacketWrite;
+	if (MyVideoDecoder) {
+	    CodecVideoFlushBuffers(MyVideoDecoder);
+	}
 	VideoClearBuffers = 0;
 	return 1;
     }
@@ -563,19 +581,23 @@ int VideoDecode(void)
 /**
 **	Try video start.
 **
-**	Could be called, when already started.
+**	NOT TRUE: Could be called, when already started.
 */
 static void StartVideo(void)
 {
     VideoInit(X11DisplayName);
+    if (ConfigFullscreen) {
+	// FIXME: not good looking, mapped and then resized.
+	VideoSetFullscreen(1);
+    }
     VideoOsdInit();
     if (!MyVideoDecoder) {
 	VideoHwDecoder *hw_decoder;
 
 	if ((hw_decoder = VideoNewHwDecoder())) {
 	    MyVideoDecoder = CodecVideoNewDecoder(hw_decoder);
-	    VideoCodecID = CODEC_ID_NONE;
 	}
+	VideoCodecID = CODEC_ID_NONE;
     }
     VideoPacketInit();
 }
@@ -637,9 +659,6 @@ int PlayVideo(const uint8_t * data, int size)
     int64_t pts;
     int n;
 
-    if (BrokenThreadsAndPlugins) {
-	return size;
-    }
     if (Usr1Signal) {			// x11 server ready
 	Usr1Signal = 0;
 	StartVideo();
@@ -647,6 +666,12 @@ int PlayVideo(const uint8_t * data, int size)
     if (!MyVideoDecoder) {		// no x11 video started
 	return size;
     }
+    if (SkipVideo) {			// skip video
+	return size;
+    }
+    if (VideoFreezed) {			// video freezed
+	return 0;
+    }
     if (NewVideoStream) {		// channel switched
 	Debug(3, "video: new stream %d\n", GetMsTicks() - VideoSwitch);
 	// FIXME: hack to test results
@@ -667,7 +692,7 @@ int PlayVideo(const uint8_t * data, int size)
     n = data[8];			// header size
     // wrong size
     if (size < 9 + n + 4) {
-	Error(_("[softhddev] invalid video packet\n"));
+	Error(_("[softhddev] invalid video packet %d bytes\n"), size);
 	return size;
     }
     // FIXME: hack to test results
@@ -734,6 +759,16 @@ int PlayVideo(const uint8_t * data, int size)
 	    Debug(3, "video: h264 detected\n");
 	    VideoCodecID = CODEC_ID_H264;
 	}
+	// Access Unit Delimiter (BBC-HD)
+	// FIXME: the 4 offset are try & error selected
+    } else if ((data[6] & 0xC0) == 0x80 && !check[4 + 0] && !check[4 + 1]
+	&& !check[4 + 2] && check[4 + 3] == 0x1 && check[4 + 4] == 0x09) {
+	if (VideoCodecID == CODEC_ID_H264) {
+	    VideoNextPacket(CODEC_ID_H264);
+	} else {
+	    Debug(3, "video: h264 detected\n");
+	    VideoCodecID = CODEC_ID_H264;
+	}
     } else {
 	// this happens when vdr sends incomplete packets
 	if (VideoCodecID == CODEC_ID_NONE) {
@@ -760,9 +795,7 @@ int PlayVideo(const uint8_t * data, int size)
 */
 void SetPlayMode(void)
 {
-    if (BrokenThreadsAndPlugins) {
+    Resume();
-	return;
-    }
     if (MyVideoDecoder) {
 	if (VideoCodecID != CODEC_ID_NONE) {
 	    NewVideoStream = 1;
@@ -770,8 +803,13 @@ void SetPlayMode(void)
 	}
     }
     if (MyAudioDecoder) {
-	NewAudioStream = 1;
+	if (AudioCodecID != CODEC_ID_NONE) {
+	    NewAudioStream = 1;
+	}
     }
+    VideoFreezed = 0;
+    SkipAudio = 0;
+    SkipVideo = 0;
 }
 
 /**
@@ -779,9 +817,16 @@ void SetPlayMode(void)
 */
 void Clear(void)
 {
+    int i;
+
+    VideoNextPacket(VideoCodecID);	// terminate work
     VideoClearBuffers = 1;
     // FIXME: avcodec_flush_buffers
-    // FIXME: flush audio buffers
+    AudioFlushBuffers();
+
+    for (i = 0; VideoClearBuffers && i < 20; ++i) {
+	usleep(1 * 1000);
+    }
 }
 
 /**
@@ -790,6 +835,7 @@ void Clear(void)
 void Play(void)
 {
     VideoFreezed = 0;
+    SkipAudio = 0;
     // FIXME: restart audio
 }
 
@@ -800,16 +846,38 @@ void Freeze(void)
 {
     VideoFreezed = 1;
     // FIXME: freeze audio
+    AudioFlushBuffers();
+}
+
+/**
+**	Display the given I-frame as a still picture.
+*/
+void StillPicture(const uint8_t * data, int size)
+{
+    int i;
+
+    // must be a PES start code
+    if (size < 9 || !data || data[0] || data[1] || data[2] != 0x01) {
+	Error(_("[softhddev] invalid PES video packet\n"));
+	return;
+    }
+    Clear();				// flush video buffers
+    // +1 future for deinterlace
+    for (i = -1; i < (VideoCodecID == CODEC_ID_MPEG2VIDEO ? 3 : 17); ++i) {
+	PlayVideo(data, size);		// reference frames
+    }
+    VideoNextPacket(VideoCodecID);	// terminate last packet
 }
 
 /**
 **	Poll if device is ready.  Called by replay.
+**
+**	@param timeout	timeout to become ready in ms
 */
 int Poll(int timeout)
 {
     // buffers are too full
     if (atomic_read(&VideoPacketsFilled) >= VIDEO_PACKET_MAX / 2) {
-	Debug(3, "replay: poll %d\n", timeout);
 	if (timeout) {
 	    // let display thread work
 	    usleep(timeout * 1000);
@@ -819,6 +887,22 @@ int Poll(int timeout)
     return 0;
 }
 
+/**
+**	Flush the device output buffers.
+**
+**	@param timeout	timeout to flush in ms
+*/
+int Flush(int timeout)
+{
+    if (atomic_read(&VideoPacketsFilled)) {
+	if (timeout) {			// let display thread work
+	    usleep(timeout * 1000);
+	}
+	return !atomic_read(&VideoPacketsFilled);
+    }
+    return 1;
+}
+
 //////////////////////////////////////////////////////////////////////////////
 //	OSD
 //////////////////////////////////////////////////////////////////////////////
@@ -828,7 +912,9 @@ int Poll(int timeout)
 */
 void GetOsdSize(int *width, int *height, double *aspect)
 {
+#ifdef DEBUG
     static char done;
+#endif
 
     // FIXME: should be configured!
     *width = 1920;
@@ -838,11 +924,13 @@ void GetOsdSize(int *width, int *height, double *aspect)
 
     *aspect = 16.0 / 9.0 / (double)*width * (double)*height;
 
+#ifdef DEBUG
     if (!done) {
 	Debug(3, "[softhddev]%s: %dx%d %g\n", __FUNCTION__, *width, *height,
 	    *aspect);
 	done = 1;
     }
+#endif
 }
 
 /**
@@ -850,9 +938,6 @@ void GetOsdSize(int *width, int *height, double *aspect)
 */
 void OsdClose(void)
 {
-    if (BrokenThreadsAndPlugins) {
-	return;
-    }
     VideoOsdClear();
 }
 
@@ -861,25 +946,24 @@ void OsdClose(void)
 */
 void OsdDrawARGB(int x, int y, int height, int width, const uint8_t * argb)
 {
-    if (BrokenThreadsAndPlugins) {
+    Resume();
-	return;
-    }
     VideoOsdDrawARGB(x, y, height, width, argb);
 }
 
 //////////////////////////////////////////////////////////////////////////////
 
-static char StartX11Server;		///< flag start the x11 server
+static char ConfigStartX11Server;	///< flag start the x11 server
 
 /**
 **	Return command line help string.
 */
 const char *CommandLineHelp(void)
 {
-    return "  -a device\talsa audio device (fe. hw:0,0)\n"
+    return "  -a device\taudio device (fe. alsa: hw:0,0 oss: /dev/dsp)\n"
-	"  -d display\tdisplay of x11 server (f.e :0.0)\n"
+	"  -d display\tdisplay of x11 server (fe. :0.0)\n"
+	"  -f\t\tstart with fullscreen window (only with window manager)\n"
 	"  -g geometry\tx11 window geometry wxh+x+y\n"
-	"  -x\tstart x11 server\n";
+	"  -x\t\tstart x11 server\n";
 }
 
 /**
@@ -894,13 +978,16 @@ int ProcessArgs(int argc, char *const argv[])
     //	Parse arguments.
     //
     for (;;) {
-	switch (getopt(argc, argv, "-a:d:g:x")) {
+	switch (getopt(argc, argv, "-a:d:fg:x")) {
 	    case 'a':			// audio device
 		AudioSetDevice(optarg);
 		continue;
 	    case 'd':			// x11 display name
 		X11DisplayName = optarg;
 		continue;
+	    case 'f':			// fullscreen mode
+		ConfigFullscreen = 1;
+		continue;
 	    case 'g':			// geometry
 		if (VideoSetGeometry(optarg) < 0) {
 		    fprintf(stderr,
@@ -910,7 +997,7 @@ int ProcessArgs(int argc, char *const argv[])
 		}
 		continue;
 	    case 'x':			// x11 server
-		StartX11Server = 1;
+		ConfigStartX11Server = 1;
 		continue;
 	    case EOF:
 		break;
@@ -1030,39 +1117,6 @@ static void StartXServer(void)
 */
 void SoftHdDeviceExit(void)
 {
-}
-
-/**
-**	Prepare plugin.
-*/
-void Start(void)
-{
-    if (StartX11Server) {
-	StartXServer();
-    }
-    CodecInit();
-    // FIXME: AudioInit for HDMI after X11 startup
-    AudioInit();
-    if (!StartX11Server) {
-	StartVideo();
-    }
-}
-
-/**
-**	Stop plugin.
-*/
-void Stop(void)
-{
-#ifdef DEBUG
-    Debug(3, "video: max used PES packet size: %d\n", VideoMaxPacketSize);
-#endif
-
-    // FIXME:
-    // don't let any thread enter our plugin, but can still crash, when
-    // a thread has called any function, while Stop is called.
-    BrokenThreadsAndPlugins = 1;
-    usleep(2 * 1000);
-
     // lets hope that vdr does a good thead cleanup
     // no it doesn't do a good thread cleanup
     if (MyVideoDecoder) {
@@ -1082,13 +1136,46 @@ void Stop(void)
     CodecExit();
     VideoPacketExit();
 
-    if (StartX11Server) {
+    if (ConfigStartX11Server) {
 	Debug(3, "x-setup: Stop x11 server\n");
 
 	if (X11ServerPid) {
 	    kill(X11ServerPid, SIGTERM);
+	    // FIXME: wait for x11 finishing
 	}
     }
+
+    pthread_mutex_destroy(&SuspendLockMutex);
+}
+
+/**
+**	Prepare plugin.
+*/
+void Start(void)
+{
+    if (ConfigStartX11Server) {
+	StartXServer();
+    }
+    CodecInit();
+    // FIXME: AudioInit for HDMI after X11 startup
+    AudioInit();
+    if (!ConfigStartX11Server) {
+	StartVideo();
+    }
+
+    pthread_mutex_init(&SuspendLockMutex, NULL);
+}
+
+/**
+**	Stop plugin.
+**
+**	@note stop everything, but don't cleanup, module is still called.
+*/
+void Stop(void)
+{
+#ifdef DEBUG
+    Debug(3, "video: max used PES packet size: %d\n", VideoMaxPacketSize);
+#endif
 }
 
 /**
@@ -1096,5 +1183,58 @@ void Stop(void)
 */
 void MainThreadHook(void)
 {
-    VideoDisplayHandler();
+}
+
+//////////////////////////////////////////////////////////////////////////////
+//	Suspend/Resume
+//////////////////////////////////////////////////////////////////////////////
+
+/**
+**	Suspend plugin.
+*/
+void Suspend(void)
+{
+    pthread_mutex_lock(&SuspendLockMutex);
+    if (SkipVideo && SkipAudio) {	// already suspended
+	pthread_mutex_unlock(&SuspendLockMutex);
+	return;
+    }
+
+    Debug(3, "[softhddev]%s:\n", __FUNCTION__);
+
+    SkipVideo = 1;
+    SkipAudio = 1;
+    pthread_mutex_unlock(&SuspendLockMutex);
+
+    if (ConfigSuspendClose) {
+	pthread_mutex_lock(&SuspendLockMutex);
+	// FIXME: close audio
+	// FIXME: close video
+	pthread_mutex_unlock(&SuspendLockMutex);
+    }
+    if (ConfigSuspendX11) {
+	// FIXME: stop x11, if started
+    }
+}
+
+/**
+**	Resume plugin.
+*/
+void Resume(void)
+{
+    if (!SkipVideo && !SkipAudio) {	// we are not suspended
+	return;
+    }
+
+    Debug(3, "[softhddev]%s:\n", __FUNCTION__);
+
+    if (ConfigSuspendX11) {
+    }
+    if (ConfigSuspendClose) {
+	pthread_mutex_lock(&SuspendLockMutex);
+	pthread_mutex_unlock(&SuspendLockMutex);
+    }
+
+    SkipVideo = 0;
+    SkipAudio = 0;
 }

softhddev.h

@@ -1,7 +1,7 @@
 ///
 ///	@file softhddev.h	@brief software HD device plugin header file.
 ///
-///	Copyright (c) 2011 by Johns.  All Rights Reserved.
+///	Copyright (c) 2011 - 2012 by Johns.  All Rights Reserved.
 ///
 ///	Contributor(s):
 ///
@@ -36,7 +36,7 @@ extern "C"
     extern void OsdDrawARGB(int, int, int, int, const uint8_t *);
 
     /// C plugin play audio packet
-    extern void PlayAudio(const uint8_t *, int, uint8_t);
+    extern int PlayAudio(const uint8_t *, int, uint8_t);
     /// C plugin mute audio
     extern void Mute(void);
     /// C plugin set audio volume
@@ -55,8 +55,12 @@ extern "C"
     extern void Play(void);
     /// C plugin sets the device into "freeze frame" mode
     extern void Freeze(void);
+    /// C plugin display I-frame as a still picture.
+    extern void StillPicture(const uint8_t *, int);
     /// C plugin poll if ready
     extern int Poll(int);
+    /// C plugin flush output buffers
+    extern int Flush(int);
 
     /// C plugin command line help
     extern const char *CommandLineHelp(void);
@@ -72,6 +76,10 @@ extern "C"
     /// C plugin main thread hook
     extern void MainThreadHook(void);
 
+    /// Suspend plugin
+    extern void Suspend(void);
+    /// Resume plugin
+    extern void Resume(void);
 #ifdef __cplusplus
 }
 #endif

softhddevice.cpp

@@ -33,27 +33,52 @@
 
 #include "softhddev.h"
 #include "softhddevice.h"
-extern "C" {
+extern "C"
-    #include "video.h"
+{
+#include "video.h"
     extern void AudioPoller(void);
+    extern void CodecSetAudioPassthrough(int);
 }
 
 //////////////////////////////////////////////////////////////////////////////
 
-static const char *const VERSION = "0.1.5";
+static const char *const VERSION = "0.3.1";
 static const char *const DESCRIPTION =
 trNOOP("A software and GPU emulated HD device");
 
-//static const char *MAINMENUENTRY = trNOOP("Soft-HD-Device");
+static const char *MAINMENUENTRY = trNOOP("Suspend Soft-HD-Device");
 static class cSoftHdDevice *MyDevice;
 
 //////////////////////////////////////////////////////////////////////////////
 
-static char ConfigMakePrimary;			///< config primary wanted
+#define RESOLUTIONS 4			///< number of resolutions
-static char ConfigVideoDeinterlace;		///< config deinterlace
+
-static char ConfigVideoScaling;			///< config scaling
+static const char *const Resolution[RESOLUTIONS] = {
-static int ConfigVideoAudioDelay;		///< config audio delay
+    "576i", "720p", "1080i_fake", "1080i"
-static char DoMakePrimary;			///< flag switch primary
+};
+
+static char ConfigMakePrimary;		///< config primary wanted
+static char ConfigHideMainMenuEntry;	///< config hide main menu entry
+
+    /// config deinterlace
+static int ConfigVideoDeinterlace[RESOLUTIONS];
+
+    /// config skip chroma
+static int ConfigVideoSkipChromaDeinterlace[RESOLUTIONS];
+
+    /// config denoise
+static int ConfigVideoDenoise[RESOLUTIONS];
+
+    /// config sharpen
+static int ConfigVideoSharpen[RESOLUTIONS];
+
+    /// config scaling
+static int ConfigVideoScaling[RESOLUTIONS];
+
+static int ConfigVideoAudioDelay;	///< config audio delay
+static int ConfigAudioPassthrough;	///< config audio pass-through
+
+static volatile char DoMakePrimary;	///< flag switch primary
 
 //////////////////////////////////////////////////////////////////////////////
 
@@ -107,8 +132,10 @@ extern "C" void FeedKeyPress(const char *keymap, const char *key, int repeat,
 
 class cSoftOsd:public cOsd
 {
+    int Level;				///< level: subtitle
+
   public:
-    cSoftOsd(int, int, uint);
+     cSoftOsd(int, int, uint);
      virtual ~ cSoftOsd(void);
     virtual void Flush(void);
     // virtual void SetActive(bool);
@@ -121,14 +148,20 @@ cSoftOsd::cSoftOsd(int left, int top, uint level)
     dsyslog("[softhddev]%s: %dx%d+%d+%d, %d\n", __FUNCTION__, OsdWidth(),
 	OsdHeight(), left, top, level);
 
+    this->Level = level;
     //SetActive(true);
 }
 
 cSoftOsd::~cSoftOsd(void)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
     SetActive(false);
 
+#ifdef USE_YAEPG
+    if (vidWin.bpp) {
+	VideoSetOutputPosition(0, 0, 1920, 1080);
+    }
+#endif
     OsdClose();
 }
 
@@ -142,7 +175,18 @@ void cSoftOsd::Flush(void)
     if (!Active()) {
 	return;
     }
-    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    // support yaepghd, video window
+#ifdef USE_YAEPG
+    if (vidWin.bpp) {
+	dsyslog("[softhddev]%s: %dx%d+%d+%d\n", __FUNCTION__, vidWin.Width(),
+	    vidWin.Height(), vidWin.x1, vidWin.y2);
+
+	// FIXME: vidWin is OSD relative not video window.
+	VideoSetOutputPosition(Left() + vidWin.x1, Top() + vidWin.y1,
+	    vidWin.Width(), vidWin.Height());
+    }
+#endif
+
     if (!IsTrueColor()) {
 	static char warned;
 	cBitmap *bitmap;
@@ -182,8 +226,24 @@ void cSoftOsd::Flush(void)
 		}
 	    }
 
-	    OsdDrawARGB(Left() + bitmap->X0(), Top() + bitmap->Y0(),
+	    // check if subtitles
-		bitmap->Width(), bitmap->Height(), argb);
+	    if (this->Level == OSD_LEVEL_SUBTITLES) {
+		int video_width;
+		int video_height;
+
+		if (0) {
+		    dsyslog("[softhddev]%s: subtitle %d, %d\n", __FUNCTION__,
+			Left() + bitmap->X0(), Top() + bitmap->Y0());
+		}
+		video_width = 1920;
+		video_height = 1080;
+		OsdDrawARGB((1920 - video_width) / 2 + Left() + bitmap->X0(),
+		    1080 - video_height + Top() + bitmap->Y0(),
+		    bitmap->Width(), bitmap->Height(), argb);
+	    } else {
+		OsdDrawARGB(Left() + bitmap->X0(), Top() + bitmap->Y0(),
+		    bitmap->Width(), bitmap->Height(), argb);
+	    }
 
 	    bitmap->Clean();
 	    free(argb);
@@ -203,8 +263,10 @@ void cSoftOsd::Flush(void)
 	w = pm->ViewPort().Width();
 	h = pm->ViewPort().Height();
 
-	dsyslog("[softhddev]%s: draw %dx%d+%d+%d %p\n", __FUNCTION__, w, h, x,
+	/*
-	    y, pm->Data());
+	   dsyslog("[softhddev]%s: draw %dx%d+%d+%d %p\n", __FUNCTION__, w, h, x,
+	   y, pm->Data());
+	 */
 
 	OsdDrawARGB(x, y, w, h, pm->Data());
 
@@ -226,14 +288,14 @@ class cSoftOsdProvider:public cOsdProvider
     cSoftOsdProvider(void);
 };
 
-cOsd *cSoftOsdProvider::Osd;			///< single osd
+cOsd *cSoftOsdProvider::Osd;		///< single osd
 
 /**
 **	Create a new OSD.
 */
 cOsd *cSoftOsdProvider::CreateOsd(int left, int top, uint level)
 {
-    dsyslog("[softhddev]%s: %d, %d, %d\n", __FUNCTION__, left, top, level);
+    //dsyslog("[softhddev]%s: %d, %d, %d\n", __FUNCTION__, left, top, level);
 
     Osd = new cSoftOsd(left, top, level);
     return Osd;
@@ -247,10 +309,13 @@ bool cSoftOsdProvider::ProvidesTrueColor(void)
     return true;
 }
 
+/**
+**	Create cOsdProvider class.
+*/
 cSoftOsdProvider::cSoftOsdProvider(void)
 :  cOsdProvider()
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 }
 
 //////////////////////////////////////////////////////////////////////////////
@@ -261,36 +326,91 @@ class cMenuSetupSoft:public cMenuSetupPage
 {
   protected:
     int MakePrimary;
-    int Deinterlace;
+    int HideMainMenuEntry;
-    int Scaling;
+    int Scaling[RESOLUTIONS];
+    int Deinterlace[RESOLUTIONS];
+    int SkipChromaDeinterlace[RESOLUTIONS];
+    int Denoise[RESOLUTIONS];
+    int Sharpen[RESOLUTIONS];
     int AudioDelay;
+    int AudioPassthrough;
   protected:
      virtual void Store(void);
   public:
      cMenuSetupSoft(void);
 };
 
+/**
+**	Create a seperator item.
+*/
+static inline cOsdItem *SeparatorItem(const char *label)
+{
+    cOsdItem *item;
+
+    item = new cOsdItem(cString::sprintf("* %s: ", label));
+    item->SetSelectable(false);
+
+    return item;
+}
+
 /**
 **	Constructor setup menu.
 */
 cMenuSetupSoft::cMenuSetupSoft(void)
 {
-    static const char * const deinterlace[] = {
+    static const char *const deinterlace[] = {
-	"Bob", "Weave", "Temporal", "TemporalSpatial", "Software" };
+	"Bob", "Weave/None", "Temporal", "TemporalSpatial", "Software"
-    static const char * const scaling[] = {
+    };
-	"Normal", "Fast", "HQ", "Anamorphic" };
+    static const char *const scaling[] = {
+	"Normal", "Fast", "HQ", "Anamorphic"
+    };
+    static const char *const passthrough[] = {
+	"None", "AC-3"
+    };
+    static const char *const resolution[RESOLUTIONS] = {
+	"576i", "720p", "fake 1080i", "1080i"
+    };
+    int i;
 
     // cMenuEditBoolItem cMenuEditBitItem cMenuEditNumItem
     // cMenuEditStrItem cMenuEditStraItem cMenuEditIntItem
     MakePrimary = ConfigMakePrimary;
     Add(new cMenuEditBoolItem(tr("Make primary device"), &MakePrimary,
 	    tr("no"), tr("yes")));
-    Deinterlace = ConfigVideoDeinterlace;
+    HideMainMenuEntry = ConfigHideMainMenuEntry;
-    Add(new cMenuEditStraItem(tr("Deinterlace"), &Deinterlace, 5, deinterlace));
+    Add(new cMenuEditBoolItem(tr("Hide main menu entry"), &HideMainMenuEntry,
-    Scaling = ConfigVideoScaling;
+	    tr("no"), tr("yes")));
-    Add(new cMenuEditStraItem(tr("Scaling"), &Scaling, 4, scaling));
+    //
+    //	video
+    //
+    Add(SeparatorItem(tr("Video")));
+    for (i = 0; i < RESOLUTIONS; ++i) {
+	Add(SeparatorItem(resolution[i]));
+	Scaling[i] = ConfigVideoScaling[i];
+	Add(new cMenuEditStraItem(tr("Scaling"), &Scaling[i], 4, scaling));
+	Deinterlace[i] = ConfigVideoDeinterlace[i];
+	Add(new cMenuEditStraItem(tr("Deinterlace"), &Deinterlace[i], 5,
+		deinterlace));
+	SkipChromaDeinterlace[i] = ConfigVideoSkipChromaDeinterlace[i];
+	Add(new cMenuEditBoolItem(tr("SkipChromaDeinterlace (vdpau)"),
+		&SkipChromaDeinterlace[i], tr("no"), tr("yes")));
+	Denoise[i] = ConfigVideoDenoise[i];
+	Add(new cMenuEditIntItem(tr("Denoise (0..1000) (vdpau)"), &Denoise[i],
+		0, 1000));
+	Sharpen[i] = ConfigVideoSharpen[i];
+	Add(new cMenuEditIntItem(tr("Sharpen (-1000..1000) (vdpau)"),
+		&Sharpen[i], -1000, 1000));
+    }
+    //
+    //	audio
+    //
+    Add(SeparatorItem(tr("Audio")));
     AudioDelay = ConfigVideoAudioDelay;
-    Add(new cMenuEditIntItem(tr("Audio delay (ms)"), &AudioDelay, -1000, 1000));
+    Add(new cMenuEditIntItem(tr("Audio delay (ms)"), &AudioDelay, -1000,
+	    1000));
+    AudioPassthrough = ConfigAudioPassthrough;
+    Add(new cMenuEditStraItem(tr("Audio pass-through"), &AudioPassthrough, 2,
+	    passthrough));
 }
 
 /**
@@ -298,13 +418,38 @@ cMenuSetupSoft::cMenuSetupSoft(void)
 */
 void cMenuSetupSoft::Store(void)
 {
+    int i;
+
     SetupStore("MakePrimary", ConfigMakePrimary = MakePrimary);
-    SetupStore("Deinterlace", ConfigVideoDeinterlace = Deinterlace);
+    SetupStore("HideMainMenuEntry", ConfigHideMainMenuEntry =
-    VideoSetDeinterlace(ConfigVideoDeinterlace);
+	HideMainMenuEntry);
-    SetupStore("Scaling", ConfigVideoScaling = Scaling);
+
+    for (i = 0; i < RESOLUTIONS; ++i) {
+	char buf[128];
+
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Scaling");
+	SetupStore(buf, ConfigVideoScaling[i] = Scaling[i]);
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Deinterlace");
+	SetupStore(buf, ConfigVideoDeinterlace[i] = Deinterlace[i]);
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i],
+	    "SkipChromaDeinterlace");
+	SetupStore(buf, ConfigVideoSkipChromaDeinterlace[i] =
+	    SkipChromaDeinterlace[i]);
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Denoise");
+	SetupStore(buf, ConfigVideoDenoise[i] = Denoise[i]);
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Sharpen");
+	SetupStore(buf, ConfigVideoSharpen[i] = Sharpen[i]);
+    }
     VideoSetScaling(ConfigVideoScaling);
+    VideoSetDeinterlace(ConfigVideoDeinterlace);
+    VideoSetSkipChromaDeinterlace(ConfigVideoSkipChromaDeinterlace);
+    VideoSetDenoise(ConfigVideoDenoise);
+    VideoSetSharpen(ConfigVideoSharpen);
+
     SetupStore("AudioDelay", ConfigVideoAudioDelay = AudioDelay);
     VideoSetAudioDelay(ConfigVideoAudioDelay);
+    SetupStore("AudioPassthrough", ConfigAudioPassthrough = AudioPassthrough);
+    CodecSetAudioPassthrough(ConfigAudioPassthrough);
 }
 
 //////////////////////////////////////////////////////////////////////////////
@@ -330,10 +475,17 @@ class cSoftHdDevice:public cDevice
     virtual bool Poll(cPoller &, int = 0);
     virtual bool Flush(int = 0);
     virtual int64_t GetSTC(void);
+    virtual void GetVideoSize(int &width, int &height, double &aspect)
+    {
+	width = 1920;
+	height = 1080;
+	aspect = (double)width / height;
+    }
     virtual void GetOsdSize(int &, int &, double &);
     virtual int PlayVideo(const uchar *, int);
+
     //virtual int PlayTsVideo(const uchar *, int);
-#ifdef USE_OSS			// FIXME: testing only oss
+#ifdef USE_OSS				// FIXME: testing only oss
     virtual int PlayTsAudio(const uchar *, int);
 #endif
     virtual void SetAudioChannelDevice(int);
@@ -348,11 +500,13 @@ class cSoftHdDevice:public cDevice
 
     virtual int ProvidesCa(const cChannel *) const;
 
+#if 0
 // SPU facilities
   private:
     cDvbSpuDecoder * spuDecoder;
   public:
     virtual cSpuDecoder * GetSpuDecoder(void);
+#endif
 
   protected:
     virtual void MakePrimaryDevice(bool);
@@ -360,14 +514,16 @@ class cSoftHdDevice:public cDevice
 
 cSoftHdDevice::cSoftHdDevice(void)
 {
-    dsyslog("[softhddev]%s\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s\n", __FUNCTION__);
 
+#if 0
     spuDecoder = NULL;
+#endif
 }
 
 cSoftHdDevice::~cSoftHdDevice(void)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 }
 
 void cSoftHdDevice::MakePrimaryDevice(bool on)
@@ -380,15 +536,18 @@ void cSoftHdDevice::MakePrimaryDevice(bool on)
     }
 }
 
-int cSoftHdDevice::ProvidesCa(
+    int cSoftHdDevice::ProvidesCa(
-    __attribute__ ((unused)) const cChannel * channel) const
+    __attribute__ ((unused)) const cChannel *
-{
+    channel) const
-    dsyslog("[softhddev]%s: %p\n", __FUNCTION__, channel);
+    {
+	//dsyslog("[softhddev]%s: %p\n", __FUNCTION__, channel);
 
-    return 0;
+	return 0;
-}
+    }
 
-cSpuDecoder *cSoftHdDevice::GetSpuDecoder(void)
+#if 0
+
+    cSpuDecoder *cSoftHdDevice::GetSpuDecoder(void)
 {
     dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
@@ -398,6 +557,8 @@ cSpuDecoder *cSoftHdDevice::GetSpuDecoder(void)
     return spuDecoder;
 }
 
+#endif
+
 bool cSoftHdDevice::HasDecoder(void) const
 {
     return true;
@@ -421,7 +582,7 @@ bool cSoftHdDevice::SetPlayMode(ePlayMode PlayMode)
 	case pmVideoOnly:
 	    break;
 	case pmNone:
-	    break;
+	    return true;
 	case pmExtern_THIS_SHOULD_BE_AVOIDED:
 	    break;
 	default:
@@ -436,12 +597,17 @@ int64_t cSoftHdDevice::GetSTC(void)
 {
     // dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
-    return ::VideoGetClock();
+    return::VideoGetClock();
 }
 
-void cSoftHdDevice::TrickSpeed(int Speed)
+/**
+**	Set trick play speed.
+**
+**	@param speed	trick speed
+*/
+void cSoftHdDevice::TrickSpeed(int speed)
 {
-    dsyslog("[softhddev]%s: %d\n", __FUNCTION__, Speed);
+    dsyslog("[softhddev]%s: %d\n", __FUNCTION__, speed);
 }
 
 void cSoftHdDevice::Clear(void)
@@ -460,6 +626,9 @@ void cSoftHdDevice::Play(void)
     ::Play();
 }
 
+/**
+**	Puts the device into "freeze frame" mode.
+*/
 void cSoftHdDevice::Freeze(void)
 {
     dsyslog("[softhddev]%s:\n", __FUNCTION__);
@@ -473,7 +642,6 @@ void cSoftHdDevice::Mute(void)
     dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     cDevice::Mute();
-
     ::Mute();
 }
 
@@ -484,26 +652,46 @@ void cSoftHdDevice::SetVolumeDevice(int volume)
     ::SetVolumeDevice(volume);
 }
 
-void cSoftHdDevice::StillPicture(
+/**
-    __attribute__ ((unused)) const uchar * data, __attribute__ ((unused))
+**	Display the given I-frame as a still picture.
-    int length)
+*/
+void cSoftHdDevice::StillPicture(const uchar * data, int length)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    dsyslog("[softhddev]%s: %s\n", __FUNCTION__,
+	data[0] == 0x47 ? "ts" : "pes");
+
+    if (data[0] == 0x47) {		// ts sync
+	cDevice::StillPicture(data, length);
+	return;
+    }
+
+    ::StillPicture(data, length);
 }
 
+/**
+**	Check if the device is ready for further action.
+**
+**	@param poller		file handles (unused)
+**	@param timeout_ms	timeout in ms to become ready
+*/
 bool cSoftHdDevice::Poll(
     __attribute__ ((unused)) cPoller & poller, int timeout_ms)
 {
     // dsyslog("[softhddev]%s: %d\n", __FUNCTION__, timeout_ms);
 
-    return ::Poll(timeout_ms);
+    return::Poll(timeout_ms);
 }
 
+/**
+**	Flush the device output buffers.
+**
+**	@param timeout_ms	timeout in ms to become ready
+*/
 bool cSoftHdDevice::Flush(int timeout_ms)
 {
     dsyslog("[softhddev]%s: %d ms\n", __FUNCTION__, timeout_ms);
 
-    return true;
+    return::Flush(timeout_ms);
 }
 
 // ----------------------------------------------------------------------------
@@ -520,34 +708,36 @@ void cSoftHdDevice::GetOsdSize(int &width, int &height, double &pixel_aspect)
 
 // ----------------------------------------------------------------------------
 
+/**
+**	Play a audio packet.
+*/
 int cSoftHdDevice::PlayAudio(const uchar * data, int length, uchar id)
 {
     //dsyslog("[softhddev]%s: %p %p %d %d\n", __FUNCTION__, this, data, length, id);
 
-    ::PlayAudio(data, length, id);
+    return::PlayAudio(data, length, id);
-
-    return length;
 }
 
 void cSoftHdDevice::SetAudioTrackDevice(
     __attribute__ ((unused)) eTrackType type)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 }
 
-void cSoftHdDevice::SetDigitalAudioDevice(bool on)
+void cSoftHdDevice::SetDigitalAudioDevice( __attribute__ ((unused)) bool on)
 {
-    dsyslog("[softhddev]%s: %s\n", __FUNCTION__, on ? "true" : "false");
+    //dsyslog("[softhddev]%s: %s\n", __FUNCTION__, on ? "true" : "false");
 }
 
-void cSoftHdDevice::SetAudioChannelDevice(int audio_channel)
+void cSoftHdDevice::SetAudioChannelDevice( __attribute__ ((unused))
+    int audio_channel)
 {
-    dsyslog("[softhddev]%s: %d\n", __FUNCTION__, audio_channel);
+    //dsyslog("[softhddev]%s: %d\n", __FUNCTION__, audio_channel);
 }
 
 int cSoftHdDevice::GetAudioChannelDevice(void)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
     return 0;
 }
 
@@ -560,7 +750,7 @@ int cSoftHdDevice::PlayVideo(const uchar * data, int length)
 {
     //dsyslog("[softhddev]%s: %p %d\n", __FUNCTION__, data, length);
 
-    return ::PlayVideo(data, length);
+    return::PlayVideo(data, length);
 }
 
 #if 0
@@ -573,7 +763,7 @@ int cSoftHdDevice::PlayTsVideo(const uchar * Data, int Length)
 }
 #endif
 
-#ifdef USE_OSS			// FIXME: testing only oss
+#ifdef USE_OSS				// FIXME: testing only oss
 ///
 ///	Play a TS audio packet.
 ///
@@ -583,7 +773,7 @@ int cSoftHdDevice::PlayTsAudio(const uchar * data, int length)
 {
     AudioPoller();
 
-    return cDevice::PlayTsAudio(data,length);
+    return cDevice::PlayTsAudio(data, length);
 }
 #endif
 
@@ -614,11 +804,13 @@ class cPluginSoftHdDevice:public cPlugin
     virtual void Stop(void);
 //    virtual void Housekeeping(void);
     virtual void MainThreadHook(void);
-//    virtual const char *MainMenuEntry(void);
+    virtual const char *MainMenuEntry(void);
-//    virtual cOsdObject *MainMenuAction(void);
+    virtual cOsdObject *MainMenuAction(void);
     virtual cMenuSetupPage *SetupMenu(void);
     virtual bool SetupParse(const char *, const char *);
-//    virtual bool Service(const char *Id, void *Data = NULL);
+//    virtual bool Service(const char *, void * = NULL);
+    virtual const char **SVDRPHelpPages(void);
+    virtual cString SVDRPCommand(const char *, const char *, int &);
 };
 
 cPluginSoftHdDevice::cPluginSoftHdDevice(void)
@@ -626,13 +818,13 @@ cPluginSoftHdDevice::cPluginSoftHdDevice(void)
     // Initialize any member variables here.
     // DON'T DO ANYTHING ELSE THAT MAY HAVE SIDE EFFECTS, REQUIRE GLOBAL
     // VDR OBJECTS TO EXIST OR PRODUCE ANY OUTPUT!
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 }
 
 cPluginSoftHdDevice::~cPluginSoftHdDevice(void)
 {
     // Clean up after yourself!
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     ::SoftHdDeviceExit();
 }
@@ -660,7 +852,7 @@ const char *cPluginSoftHdDevice::CommandLineHelp(void)
 */
 bool cPluginSoftHdDevice::ProcessArgs(int argc, char *argv[])
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     return::ProcessArgs(argc, argv);
 }
@@ -668,22 +860,21 @@ bool cPluginSoftHdDevice::ProcessArgs(int argc, char *argv[])
 bool cPluginSoftHdDevice::Initialize(void)
 {
     // Start any background activities the plugin shall perform.
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     MyDevice = new cSoftHdDevice();
 
     return true;
 }
 
+/**
+**	 Start any background activities the plugin shall perform.
+*/
 bool cPluginSoftHdDevice::Start(void)
 {
-    const cDevice *primary;
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
-    // Start any background activities the plugin shall perform.
+    if (!MyDevice->IsPrimaryDevice()) {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
-
-    primary = cDevice::PrimaryDevice();
-    if (MyDevice != primary) {
 	isyslog("[softhddev] softhddevice is not the primary device!");
 	if (ConfigMakePrimary) {
 	    // Must be done in the main thread
@@ -703,7 +894,7 @@ bool cPluginSoftHdDevice::Start(void)
 
 void cPluginSoftHdDevice::Stop(void)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     ::Stop();
 }
@@ -715,14 +906,31 @@ void cPluginSoftHdDevice::Housekeeping(void)
     // Perform any cleanup or other regular tasks.
 }
 
+#endif
+
+/**
+**	Create main menu entry.
+*/
 const char *cPluginSoftHdDevice::MainMenuEntry(void)
 {
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
-    return tr(MAINMENUENTRY);
+
-    return NULL;
+    return ConfigHideMainMenuEntry ? NULL : tr(MAINMENUENTRY);
 }
 
-#endif
+/**
+**	Perform the action when selected from the main VDR menu.
+*/
+cOsdObject *cPluginSoftHdDevice::MainMenuAction(void)
+{
+    dsyslog("[softhddev]%s:\n", __FUNCTION__);
+
+    cDevice::PrimaryDevice()->StopReplay();
+    Suspend();
+    ShutdownHandler.SetUserInactive();
+
+    return NULL;
+}
 
 /**
 **	Called for every plugin once during every cycle of VDR's main program
@@ -730,17 +938,95 @@ const char *cPluginSoftHdDevice::MainMenuEntry(void)
 */
 void cPluginSoftHdDevice::MainThreadHook(void)
 {
-    // dsyslog("[softhddev]%s:\n", __FUNCTION__);
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
     if (DoMakePrimary && MyDevice) {
 	dsyslog("[softhddev]%s: switching primary device\n", __FUNCTION__);
 	cDevice::SetPrimaryDevice(MyDevice->DeviceNumber() + 1);
 	DoMakePrimary = 0;
     }
+    // check if user is inactive, automatic enter suspend mode
+    if (ShutdownHandler.IsUserInactive()) {
+	// this is regular called, but guarded against double calls
+	Suspend();
+    }
 
     ::MainThreadHook();
 }
 
+/**
+**	Return our setup menu.
+*/
+cMenuSetupPage *cPluginSoftHdDevice::SetupMenu(void)
+{
+    //dsyslog("[softhddev]%s:\n", __FUNCTION__);
+
+    return new cMenuSetupSoft;
+}
+
+/**
+**	Parse setup parameters
+*/
+bool cPluginSoftHdDevice::SetupParse(const char *name, const char *value)
+{
+    int i;
+    char buf[128];
+
+    //dsyslog("[softhddev]%s: '%s' = '%s'\n", __FUNCTION__, name, value);
+
+    if (!strcmp(name, "MakePrimary")) {
+	ConfigMakePrimary = atoi(value);
+	return true;
+    }
+    if (!strcmp(name, "HideMainMenuEntry")) {
+	ConfigHideMainMenuEntry = atoi(value);
+	return true;
+    }
+    for (i = 0; i < RESOLUTIONS; ++i) {
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Scaling");
+	if (!strcmp(name, buf)) {
+	    ConfigVideoScaling[i] = atoi(value);
+	    VideoSetScaling(ConfigVideoScaling);
+	    return true;
+	}
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Deinterlace");
+	if (!strcmp(name, buf)) {
+	    ConfigVideoDeinterlace[i] = atoi(value);
+	    VideoSetDeinterlace(ConfigVideoDeinterlace);
+	    return true;
+	}
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i],
+	    "SkipChromaDeinterlace");
+	if (!strcmp(name, buf)) {
+	    ConfigVideoSkipChromaDeinterlace[i] = atoi(value);
+	    VideoSetSkipChromaDeinterlace(ConfigVideoSkipChromaDeinterlace);
+	    return true;
+	}
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Denoise");
+	if (!strcmp(name, buf)) {
+	    ConfigVideoDenoise[i] = atoi(value);
+	    VideoSetDenoise(ConfigVideoDenoise);
+	    return true;
+	}
+	snprintf(buf, sizeof(buf), "%s.%s", Resolution[i], "Sharpen");
+	if (!strcmp(name, buf)) {
+	    ConfigVideoSharpen[i] = atoi(value);
+	    VideoSetSharpen(ConfigVideoSharpen);
+	    return true;
+	}
+    }
+    if (!strcmp(name, "AudioDelay")) {
+	VideoSetAudioDelay(ConfigVideoAudioDelay = atoi(value));
+	return true;
+    }
+    if (!strcmp(name, "AudioPassthrough")) {
+	CodecSetAudioPassthrough(ConfigAudioPassthrough = atoi(value));
+	return true;
+    }
+
+    return false;
+}
+
 #if 0
 
 bool cPluginSoftHdDevice::Service(const char *Id, void *Data)
@@ -750,52 +1036,42 @@ bool cPluginSoftHdDevice::Service(const char *Id, void *Data)
     return false;
 }
 
-cOsdObject *cPluginSoftHdDevice::MainMenuAction(void)
+#endif
-{
-    // Perform the action when selected from the main VDR menu.
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
 
+//----------------------------------------------------------------------------
+//	cPlugin SVDRP
+//----------------------------------------------------------------------------
+
+/**
+**	Return SVDRP commands help pages.
+**
+**	return a pointer to a list of help strings for all of the plugin's
+**	SVDRP commands.
+*/
+const char **cPluginSoftHdDevice::SVDRPHelpPages(void)
+{
+    // FIXME: translation?
+    static const char *text[] = {
+	"SUSP\n",
+	"    Suspend plugin",
+	NULL
+    };
+
+    return text;
+}
+
+/**
+**	Handle SVDRP commands.
+*/
+cString cPluginSoftHdDevice::SVDRPCommand(const char *command,
+    __attribute__ ((unused)) const char *option,
+    __attribute__ ((unused)) int &reply_code)
+{
+    if (!strcasecmp(command, "SUSP")) {
+	Suspend();
+	return "SoftHdDevice is suspended";
+    }
     return NULL;
 }
 
-#endif
-
-/**
-**	Return our setup menu.
-*/
-cMenuSetupPage *cPluginSoftHdDevice::SetupMenu(void)
-{
-    dsyslog("[softhddev]%s:\n", __FUNCTION__);
-
-    return new cMenuSetupSoft;
-}
-
-/**
-**	Parse setup parameters
-*/
-bool cPluginSoftHdDevice::SetupParse(const char *name, const char *value)
-{
-    dsyslog("[softhddev]%s: '%s' = '%s'\n", __FUNCTION__, name, value);
-
-    // FIXME: handle the values
-    if (!strcmp(name, "MakePrimary")) {
-	ConfigMakePrimary = atoi(value);
-	return true;
-    }
-    if (!strcmp(name, "Deinterlace")) {
-	VideoSetDeinterlace(ConfigVideoDeinterlace = atoi(value));
-	return true;
-    }
-    if (!strcmp(name, "Scaling")) {
-	VideoSetScaling(ConfigVideoScaling = atoi(value));
-	return true;
-    }
-    if (!strcmp(name, "AudioDelay")) {
-	VideoSetAudioDelay(ConfigVideoAudioDelay = atoi(value));
-	return true;
-    }
-
-    return false;
-}
-
 VDRPLUGINCREATOR(cPluginSoftHdDevice);	// Don't touch this!

vdr-softhddevice-9999.ebuild

@@ -0,0 +1,71 @@
+# Copyright 1999-2012 Gentoo Foundation
+# Distributed under the terms of the GNU General Public License v2
+# $Header: $
+
+EAPI="3"
+
+inherit eutils vdr-plugin
+
+if [[ ${PV} == "9999" ]] ; then
+		inherit git-2
+		EGIT_REPO_URI="git://projects.vdr-developer.org/vdr-plugin-softhddevice.git"
+else
+		SRC_URI="http://projects.vdr-developer.org/attachments/download/838/${P}.tgz"
+fi
+
+
+DESCRIPTION="A software and GPU emulated HD output device plugin for VDR."
+HOMEPAGE="http://projects.vdr-developer.org/projects/show/plg-softhddevice"
+SRC_URI=""
+
+LICENSE="AGPL-3"
+SLOT="0"
+KEYWORDS="~x86 ~amd64"
+IUSE="vaapi vdpau alsa oss yaepg"
+
+DEPEND=">=x11-libs/libxcb-1.7
+		x11-libs/xcb-util
+		x11-libs/xcb-util-wm
+		x11-libs/xcb-util-wm
+		x11-libs/xcb-util-keysyms
+		x11-libs/xcb-util-renderutil
+		x11-libs/libX11
+		>=media-video/ffmpeg-0.7
+		sys-devel/gettext
+		sys-devel/make
+		dev-util/pkgconfig
+		yaepg? ( >=media-video/vdr-1.7[yaepg] )
+		!yaepg? ( >=media-video/vdr-1.7 )
+		vdpau? ( x11-libs/libvdpau )
+		vaapi? ( x11-libs/libva )
+		alsa? ( media-libs/alsa-lib )
+"
+
+src_prepare() {
+		vdr-plugin_src_prepare
+}
+
+src_compile() {
+		local myconf
+
+		myconf=""
+		use vdpau && myconf="${myconf} -DUSE_VDPAU"
+		use vaapi && myconf="${myconf} -DUSE_VAAPI"
+		use alsa && myconf="${myconf} -DUSE_ALSA"
+		use oss && myconf="${myconf} -DUSE_OSS"
+
+		emake all CC="$(tc-getCC)" CFLAGS="${CFLAGS}" \
+			LDFLAGS="${LDFLAGS}" CONFIG="${myconf}" LIBDIR="." || die
+}
+
+src_install() {
+		vdr-plugin_src_install
+
+		dodir /etc/vdr/plugins || die
+
+		insinto /etc/vdr/plugins
+		fowners -R vdr:vdr /etc/vdr || die
+
+		#insinto /etc/conf.d
+		#doins vdr.softhddevice
+}

video.h

@@ -1,7 +1,7 @@
 ///
 ///	@file video.h	@brief Video module header file
 ///
-///	Copyright (c) 2009 - 2011 by Johns.  All Rights Reserved.
+///	Copyright (c) 2009 - 2012 by Johns.  All Rights Reserved.
 ///
 ///	Contributor(s):
 ///
@@ -30,13 +30,6 @@
     /// Video hardware decoder typedef
 typedef struct _video_hw_decoder_ VideoHwDecoder;
 
-//----------------------------------------------------------------------------
-//	Variables
-//----------------------------------------------------------------------------
-
-//extern unsigned VideoWindowWidth;	///< current video output width
-//extern unsigned VideoWindowHeight;	///< current video output height
-
 //----------------------------------------------------------------------------
 //	Prototypes
 //----------------------------------------------------------------------------
@@ -47,14 +40,14 @@ extern VideoHwDecoder *VideoNewHwDecoder(void);
     /// Get and allocate a video hardware surface.
 extern unsigned VideoGetSurface(VideoHwDecoder *);
 
-    /// Release a video hardware surface.
+    /// Release a video hardware surface
 extern void VideoReleaseSurface(VideoHwDecoder *, unsigned);
 
 #ifdef LIBAVCODEC_VERSION
-    /// Render a ffmpeg frame
+    /// Render a ffmpeg frame.
 extern void VideoRenderFrame(VideoHwDecoder *, AVCodecContext *, AVFrame *);
 
-    /// Get ffmpeg vaapi context
+    /// Get ffmpeg vaapi context.
 extern struct vaapi_context *VideoGetVaapiContext(VideoHwDecoder *);
 
     /// Callback to negotiate the PixelFormat.
@@ -62,48 +55,63 @@ extern enum PixelFormat Video_get_format(VideoHwDecoder *, AVCodecContext *,
     const enum PixelFormat *);
 
 #ifdef AVCODEC_VDPAU_H
-    /// Draw vdpau render state
+    /// Draw vdpau render state.
 extern void VideoDrawRenderState(VideoHwDecoder *,
     struct vdpau_render_state *);
 #endif
 #endif
 
-    /// Display video TEST
+    /// Poll video events.
-extern void VideoDisplayHandler(void);
-
-    /// Poll video events
 extern void VideoPollEvent(void);
 
-    /// set video mode
+    /// Wakeup display handler.
-//extern void VideoSetVideoMode(int, int, int, int);
+extern void VideoDisplayWakeup(void);
 
-    /// set video geometry
+    /// Set video geometry.
 extern int VideoSetGeometry(const char *);
 
-    /// set deinterlace
+    /// Set video output position.
-extern void VideoSetDeinterlace(int);
+extern void VideoSetOutputPosition(int, int, int, int);
 
-    /// set scaling
+    /// Set video mode.
-extern void VideoSetScaling(int);
+extern void VideoSetVideoMode(int, int, int, int);
 
-    /// set audio delay
+    /// Set video fullscreen mode.
+extern void VideoSetFullscreen(int);
+
+    /// Set deinterlace.
+extern void VideoSetDeinterlace(int[]);
+
+    /// Set skip chroma deinterlace.
+extern void VideoSetSkipChromaDeinterlace(int[]);
+
+    /// Set scaling.
+extern void VideoSetScaling(int[]);
+
+    /// Set denoise.
+extern void VideoSetDenoise(int[]);
+
+    /// Set sharpen.
+extern void VideoSetSharpen(int[]);
+
+    /// Set audio delay.
 extern void VideoSetAudioDelay(int);
 
-    /// Clear OSD
+    /// Clear OSD.
 extern void VideoOsdClear(void);
 
-    /// Draw an OSD ARGB image
+    /// Draw an OSD ARGB image.
 extern void VideoOsdDrawARGB(int, int, int, int, const uint8_t *);
 
-extern int64_t VideoGetClock(void);	///< get video clock
+extern int64_t VideoGetClock(void);	///< Get video clock.
 
-extern void VideoOsdInit(void);		///< setup osd
+extern void VideoOsdInit(void);		///< Setup osd.
-extern void VideoOsdExit(void);		///< cleanup osd
+extern void VideoOsdExit(void);		///< Cleanup osd.
 
-extern void VideoInit(const char *);	///< setup video module
+extern void VideoInit(const char *);	///< Setup video module.
-extern void VideoExit(void);		///< cleanup and exit video module
+extern void VideoExit(void);		///< Cleanup and exit video module.
 
-extern void VideoFlushInput(void);	///< flush codec input buffers
+extern void VideoFlushInput(void);	///< Flush video input buffers.
-extern int VideoDecode(void);		///< decode
+extern int VideoDecode(void);		///< Decode video input buffers.
 
 /// @}