vdr May 2016

vdr@linuxtv.org

10 participants
16 discussions

VDR ISDB-T support
by Christian Rodriguez 14 Sep '18

14 Sep '18

Hello everyone! I'm from Argentina, and up to last month I've been using VDR 1.7.262 with an ISDB-T USB adapter. It works really good, but the last week I've been trying to upgrade to VDR 2.2 and I've found a problem with ISDB-T cards: they are not supported. As I'm really interested in using VDR, I was digging inside the code and found a partial solution for this problem: as ISDBT is the same as DVBT, I've changed the way delivery system is queried. My changes are applied in a cloned repo at github: https://github.com/chrodriguez/vdr/commit/dadbca57c2dc55c1c53b4fa8f68b56728… That's the only change I've made. Now I can enjoy my fresh upgraded VDR installation!! I think a better refactor should be made, but for now it's working perfectly deployed as a docker container: https://registry.hub.docker.com/u/chrodriguez/vdr Thanks for this wonderfull project! -- Lic. Christian A. Rodriguez @car_unlp

4 7

[PATCH] cDevice::GetDeviceForTransponder(): fix a typo
by glenvt18 01 Jan '17

01 Jan '17

d->MaySwitchTransponder(Channel) is always false here Please review, Sergey Chernyavskiy. --- device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/device.c b/device.c index 18867cd..542d120 100644 --- a/device.c +++ b/device.c @@ -342,7 +342,7 @@ cDevice *cDevice::GetDeviceForTransponder(const cChannel *Channel, int Priority) if (d->ProvidesTransponder(Channel)) { if (d->MaySwitchTransponder(Channel)) Device = d; // this device may switch to the transponder without disturbing any receiver or live view - else if (!d->Occupied() && d->MaySwitchTransponder(Channel)) { // MaySwitchTransponder() implicitly calls Occupied() + else if (!d->Occupied()) { // MaySwitchTransponder() implicitly calls Occupied() if (d->Priority() < Priority && (!Device || d->Priority() < Device->Priority())) Device = d; // use this one only if no other with less impact can be found } -- 1.9.1

2 5

"TS packets to determine frame type"
by Richard F 11 Sep '16

11 Sep '16

Hi Klaus, I'm 99% of the way to finishing my H264 ffmpeg transcoding script, I'm just getting 2 of these warnings when reindexing transcoded files May 30 21:08:09 ha-server vdr: [31785] WARNING: required 5 video TS packets to determine frame type May 30 21:08:09 ha-server vdr: [31785] WARNING: required 5 video TS packets to determine frame type Sometimes the number is 4. Fortunately not over 6 yet. Is this a functional problem ? (files play in everything I need them to). I reduced the I-frame gap which also improves seeking performance, but that's not the issue. Looking at the code I'm not quite sure what it's looking for - can you help? I may be able to adjust codec / muxer to avoid warnings Thanks Richard

4 9

[PATCH] Device power saving feature
by glenvt18 20 Aug '16

20 Aug '16

Hi folks. This patch introduces a feature which allows an idle device (a device which is not currently recording or streaming) to enter a power-down mode after some period of time. Given two timeout values, PowerdownTimeoutM and PowerdownWakeupH, it works like this: when a device becomes idle, it is kept powered up for PowerdownTimeoutM minutes doing, for instance, an EPG scan before it is powered down. If the device is still idle and has been powered down for PowerdownWakeupH hours it is powered up for PowerdownTimeoutM minutes and so on. When recording, streaming or a forced EPG scan starts, the device is powered up and it's idle timer is disabled. This implies that PowerdownTimeoutM should be enough for a full round of EPG scanning (20 seconds * number_of_transponders). Another option is to run EPG scans from cron (at night) and use SVDRP SCAN command. Actual implementation of power saving facilities is left to a derived device class. In the case of a DVB device it is implemented by closing it's frontend device. For a DVB-S/S2 tuner this usually means powering the LNB off. My measurements show 3-4W power consumption drops per tuner for various DVB-S/S2 tuners. So, this feature (together with HDD spin-down) is especially valuable while running a headless 24/7 VDR server and/or using several tuners. A SATIP device can also implement power saving if it is supported by a server. I know about the dynamite plugin, but 1) it does much more then this, 2) still requires a VDR patch, which is bigger, 3) doesn't work reliably for me and 4) I think this functionality should be part of the VDR core. A copy of the patch is here: http://pastebin.com/FRi0kTjf Please review, Sergey Chernyavskiy. --- config.c | 9 ++++++ config.h | 3 ++ device.c | 96 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-- device.h | 29 +++++++++++++++++++ dvbdevice.c | 39 +++++++++++++++++++++++++ dvbdevice.h | 7 +++++ eitscan.c | 7 ++++- menu.c | 9 +++++- vdr.c | 6 ++++ 9 files changed, 201 insertions(+), 4 deletions(-) diff --git a/config.c b/config.c index e5f5463..794c9f8 100644 --- a/config.c +++ b/config.c @@ -395,6 +395,9 @@ cSetup::cSetup(void) PositionerSpeed = 15; PositionerSwing = 650; PositionerLastLon = 0; + PowerdownEnabled = 0; + PowerdownTimeoutM = 15; + PowerdownWakeupH = 4; SetSystemTime = 0; TimeSource = 0; TimeTransponder = 0; @@ -622,6 +625,9 @@ bool cSetup::Parse(const char *Name, const char *Value) else if (!strcasecmp(Name, "PositionerSpeed")) PositionerSpeed = atoi(Value); else if (!strcasecmp(Name, "PositionerSwing")) PositionerSwing = atoi(Value); else if (!strcasecmp(Name, "PositionerLastLon")) PositionerLastLon = atoi(Value); + else if (!strcasecmp(Name, "PowerdownEnabled")) PowerdownEnabled = atoi(Value); + else if (!strcasecmp(Name, "PowerdownTimeoutM")) PowerdownTimeoutM = atoi(Value); + else if (!strcasecmp(Name, "PowerdownWakeupH")) PowerdownWakeupH = atoi(Value); else if (!strcasecmp(Name, "SetSystemTime")) SetSystemTime = atoi(Value); else if (!strcasecmp(Name, "TimeSource")) TimeSource = cSource::FromString(Value); else if (!strcasecmp(Name, "TimeTransponder")) TimeTransponder = atoi(Value); @@ -753,6 +759,9 @@ bool cSetup::Save(void) Store("PositionerSpeed", PositionerSpeed); Store("PositionerSwing", PositionerSwing); Store("PositionerLastLon", PositionerLastLon); + Store("PowerdownEnabled", PowerdownEnabled); + Store("PowerdownTimeoutM", PowerdownTimeoutM); + Store("PowerdownWakeupH", PowerdownWakeupH); Store("SetSystemTime", SetSystemTime); Store("TimeSource", cSource::ToString(TimeSource)); Store("TimeTransponder", TimeTransponder); diff --git a/config.h b/config.h index e5565da..7a73d9d 100644 --- a/config.h +++ b/config.h @@ -273,6 +273,9 @@ public: int PositionerSpeed; int PositionerSwing; int PositionerLastLon; + int PowerdownEnabled; + int PowerdownTimeoutM; + int PowerdownWakeupH; int SetSystemTime; int TimeSource; int TimeTransponder; diff --git a/device.c b/device.c index 542d120..9306adb 100644 --- a/device.c +++ b/device.c @@ -104,6 +104,9 @@ cDevice::cDevice(void) dvbSubtitleConverter = NULL; autoSelectPreferredSubtitleLanguage = true; + idleTimerExpires = time(NULL) + Setup.PowerdownTimeoutM * 60; + wakeupTimerExpires = 0; + for (int i = 0; i < MAXRECEIVERS; i++) receiver[i] = NULL; @@ -745,6 +748,11 @@ bool cDevice::SwitchChannel(int Direction) return result; } +// While switching to a channel, the device will be kept powered up +// for at least this number of seconds before a receiver is attached. +// Must be less than cEITScanner::ScanTimeout. +#define CHANNEL_SWITCH_POWERUP_TIMEOUT 10 + eSetChannelResult cDevice::SetChannel(const cChannel *Channel, bool LiveView) { cStatus::MsgChannelSwitch(this, 0, LiveView); @@ -778,6 +786,8 @@ eSetChannelResult cDevice::SetChannel(const cChannel *Channel, bool LiveView) Result = scrNotAvailable; } else { + // Power up the device + PowerUp(CHANNEL_SWITCH_POWERUP_TIMEOUT); // Stop section handling: if (sectionHandler) { sectionHandler->SetStatus(false); @@ -843,8 +853,11 @@ int cDevice::Occupied(void) const void cDevice::SetOccupied(int Seconds) { - if (Seconds >= 0) + if (Seconds >= 0) { occupiedTimeout = time(NULL) + min(Seconds, MAXOCCUPIEDTIMEOUT); + // avoid short power-down/power-up cycles + SetIdleTimer(true, Seconds + 30); + } } bool cDevice::SetChannelDevice(const cChannel *Channel, bool LiveView) @@ -1675,6 +1688,7 @@ bool cDevice::AttachReceiver(cReceiver *Receiver) startScrambleDetection = time(NULL); } Start(); + SetIdleTimer(false); return true; } } @@ -1708,8 +1722,10 @@ void cDevice::Detach(cReceiver *Receiver) camSlot->Assign(NULL); } } - if (!receiversLeft) + if (!receiversLeft) { Cancel(-1); + SetIdleTimer(true); + } } void cDevice::DetachAll(int Pid) @@ -1731,6 +1747,82 @@ void cDevice::DetachAllReceivers(void) Detach(receiver[i]); } +void cDevice::CheckIdle(void) +{ + if (!SupportsPowerDown() || !Setup.PowerdownEnabled) + return; + cMutexLock MutexLock(&mutexPowerSaving); + if (idleTimerExpires != 0 && time(NULL) > idleTimerExpires) { + // idle, powered up + dsyslog("power saving: device %d idle timer expired", CardIndex() + 1); + SetIdleTimer(false); + if (Setup.PowerdownWakeupH != 0) + wakeupTimerExpires = time(NULL) + Setup.PowerdownWakeupH * 3600; + else + dsyslog("power saving: waking up is disabled"); + if (!IsPoweredDown()) { + dsyslog("power saving: powering device %d down", CardIndex() + 1); + if (sectionHandler) { + sectionHandler->SetStatus(false); + sectionHandler->SetChannel(NULL); + } + PowerDown(true); + } + } + if (wakeupTimerExpires != 0 && time(NULL) > wakeupTimerExpires) { + // idle, powered down + dsyslog("power saving: device %d wakeup timer expired", CardIndex() + 1); + SetIdleTimer(true); + if (IsPoweredDown()) { + dsyslog("power saving: waking up device %d", CardIndex() + 1); + PowerDown(false); + } + } +} + +void cDevice::SetIdleTimer(bool On, int ExtraTimeoutS) +{ + if (!SupportsPowerDown()) + return; + cMutexLock MutexLock(&mutexPowerSaving); + if (On) { + int Tout = Setup.PowerdownTimeoutM * 60; + time_t Now = time(NULL); + if (ExtraTimeoutS > 0) { + if (idleTimerExpires >= Now + ExtraTimeoutS) + return; + Tout = ExtraTimeoutS; + } + idleTimerExpires = Now + Tout; + if (Setup.PowerdownEnabled) + dsyslog("power saving: set device %d idle timer to %d sec", CardIndex() + 1, Tout); + } + else { + idleTimerExpires = 0; + if (Setup.PowerdownEnabled) + dsyslog("power saving: disable device %d idle timer", CardIndex() + 1); + } + wakeupTimerExpires = 0; +} + +bool cDevice::PoweredDown(void) +{ + if (SupportsPowerDown() && Setup.PowerdownEnabled) { + cMutexLock MutexLock(&mutexPowerSaving); + return IsPoweredDown(); + } + else + return false; +} + +void cDevice::PowerUp(int ExtraTimeoutS) +{ + cMutexLock MutexLock(&mutexPowerSaving); + SetIdleTimer(true, ExtraTimeoutS); + if (SupportsPowerDown() && IsPoweredDown()) + PowerDown(false); +} + // --- cTSBuffer ------------------------------------------------------------- cTSBuffer::cTSBuffer(int File, int Size, int CardIndex) diff --git a/device.h b/device.h index 31ee303..6c1fdfb 100644 --- a/device.h +++ b/device.h @@ -821,6 +821,35 @@ public: ///< Detaches all receivers from this device for this pid. virtual void DetachAllReceivers(void); ///< Detaches all receivers from this device. + +// Power saving facilities + +private: + cMutex mutexPowerSaving; + time_t idleTimerExpires, wakeupTimerExpires; + void PowerUp(int ExtraTimeoutS); + ///< If the device is powered down, powers it up and keeps it + ///< powered up for at least ExtraTimeoutS seconds (see + ///< cDevice::SetIdleTimer()). +public: + void CheckIdle(void); + ///< Should be called periodically in the main loop. + bool PoweredDown(void); + ///< Returns true if the device is powered down "logically", that is, + ///< idle tasks like EPG scanning are disabled. + void SetIdleTimer(bool On, int ExtraTimeoutS = 0); + ///< Starts/disables the idle timer. This timer must be started when + ///< a device gets idle and must be disabled when it is receiving. + ///< If ExtraTimeoutS is greater than zero and On is true, a new timer + ///< won't be set, but the device will be kept powered up for at least + ///< ExtraTimeoutS seconds. +protected: + virtual bool IsPoweredDown(void) {return false;} + ///< Returns true if the device is powered down "physically". + virtual void PowerDown(bool On) {}; + ///< Actually powers the device down/up. + virtual bool SupportsPowerDown() {return false;} + ///< Returns true if a derived device supports power saving. }; /// Derived cDevice classes that can receive channels will have to provide diff --git a/dvbdevice.c b/dvbdevice.c index 63af52e..cdadbff 100644 --- a/dvbdevice.c +++ b/dvbdevice.c @@ -348,6 +348,8 @@ public: const cPositioner *Positioner(void) const { return positioner; } int GetSignalStrength(void) const; int GetSignalQuality(void) const; + bool IsPoweredDown(void) {return fd_frontend < 0;} + void PowerDown(bool On); }; cMutex cDvbTuner::bondMutex; @@ -544,6 +546,8 @@ void cDvbTuner::ClearEventQueue(void) const bool cDvbTuner::GetFrontendStatus(fe_status_t &Status) const { + if (fd_frontend < 0) + return false; ClearEventQueue(); while (1) { if (ioctl(fd_frontend, FE_READ_STATUS, &Status) != -1) @@ -559,6 +563,8 @@ bool cDvbTuner::GetFrontendStatus(fe_status_t &Status) const int cDvbTuner::GetSignalStrength(void) const { + if (fd_frontend < 0) + return -1; ClearEventQueue(); uint16_t Signal; while (1) { @@ -1001,6 +1007,26 @@ void cDvbTuner::Action(void) } } +void cDvbTuner::PowerDown(bool On) +{ + cMutexLock MutexLock(&mutex); + if (On && fd_frontend >= 0) { + isyslog("dvb tuner: power-down - closing frontend %d/%d", adapter, frontend); + tunerStatus = tsIdle; + close(fd_frontend); + fd_frontend = -1; + } + if (!On && fd_frontend < 0) { + cString Filename = cString::sprintf("%s/%s%d/%s%d", + DEV_DVB_BASE, DEV_DVB_ADAPTER, adapter, DEV_DVB_FRONTEND, frontend); + isyslog("dvb tuner: power-up - opening frontend %d/%d", adapter, frontend); + fd_frontend = open(Filename, O_RDWR | O_NONBLOCK); + if (fd_frontend < 0) + esyslog("ERROR: can't open DVB device frontend %d/%d", adapter, frontend); + tunerStatus = tsIdle; + } +} + // --- cDvbSourceParam ------------------------------------------------------- class cDvbSourceParam : public cSourceParam { @@ -1712,6 +1738,19 @@ void cDvbDevice::DetachAllReceivers(void) needsDetachBondedReceivers = false; } +bool cDvbDevice::IsPoweredDown(void) +{ + if (dvbTuner) + return dvbTuner->IsPoweredDown(); + return false; +} + +void cDvbDevice::PowerDown(bool On) +{ + if (dvbTuner) + dvbTuner->PowerDown(On); +} + // --- cDvbDeviceProbe ------------------------------------------------------- cList<cDvbDeviceProbe> DvbDeviceProbes; diff --git a/dvbdevice.h b/dvbdevice.h index 5ae4952..15d1ff8 100644 --- a/dvbdevice.h +++ b/dvbdevice.h @@ -290,6 +290,13 @@ protected: virtual void CloseDvr(void); virtual bool GetTSPacket(uchar *&Data); virtual void DetachAllReceivers(void); + +// Power saving facilities + +protected: + virtual bool IsPoweredDown(void); + virtual void PowerDown(bool On); + virtual bool SupportsPowerDown() {return true;} }; // A plugin that implements a DVB device derived from cDvbDevice needs to create diff --git a/eitscan.c b/eitscan.c index 41ac25e..765055c 100644 --- a/eitscan.c +++ b/eitscan.c @@ -144,7 +144,8 @@ void cEITScanner::Process(void) bool AnyDeviceSwitched = false; for (int i = 0; i < cDevice::NumDevices(); i++) { cDevice *Device = cDevice::GetDevice(i); - if (Device && Device->ProvidesEIT()) { + if (Device && Device->ProvidesEIT() + && (!Device->PoweredDown() || lastActivity == 0)) { // powered up or forced scan for (cScanData *ScanData = scanList->First(); ScanData; ScanData = scanList->Next(ScanData)) { const cChannel *Channel = ScanData->GetChannel(); if (Channel) { @@ -165,6 +166,10 @@ void cEITScanner::Process(void) } } //dsyslog("EIT scan: device %d source %-8s tp %5d", Device->DeviceNumber() + 1, *cSource::ToString(Channel->Source()), Channel->Transponder()); + if (lastActivity == 0) + // forced scan - set idle timer for each channel switch; + // this prevents powering down while scanning a transponder + Device->SetIdleTimer(true, ScanTimeout + 5); Device->SwitchChannel(Channel, false); scanList->Del(ScanData); AnyDeviceSwitched = true; diff --git a/menu.c b/menu.c index 569900c..5a89771 100644 --- a/menu.c +++ b/menu.c @@ -3715,6 +3715,12 @@ void cMenuSetupLNB::Setup(void) Add(new cMenuEditIntxItem(tr("Setup.LNB$Positioner speed (degrees/s)"), &data.PositionerSpeed, 1, 1800, 10)); } + Add(new cMenuEditBoolItem(tr("Setup.LNB$Enable power saving"), &data.PowerdownEnabled)); + if (data.PowerdownEnabled) { + Add(new cMenuEditIntItem(tr("Setup.LNB$Power down an idle device after (min)"), &data.PowerdownTimeoutM)); + Add(new cMenuEditIntItem(tr("Setup.LNB$Wake up from power-down after (h)"), &data.PowerdownWakeupH)); + } + SetCurrent(Get(current)); Display(); } @@ -3723,6 +3729,7 @@ eOSState cMenuSetupLNB::ProcessKey(eKeys Key) { int oldDiSEqC = data.DiSEqC; int oldUsePositioner = data.UsePositioner; + int oldPowerdownEnabled = data.PowerdownEnabled; bool DeviceBondingsChanged = false; if (Key == kOk) { cString NewDeviceBondings = satCableNumbers.ToString(); @@ -3731,7 +3738,7 @@ eOSState cMenuSetupLNB::ProcessKey(eKeys Key) } eOSState state = cMenuSetupBase::ProcessKey(Key); - if (Key != kNone && (data.DiSEqC != oldDiSEqC || data.UsePositioner != oldUsePositioner)) + if (Key != kNone && (data.DiSEqC != oldDiSEqC || data.UsePositioner != oldUsePositioner || data.PowerdownEnabled != oldPowerdownEnabled)) Setup(); else if (DeviceBondingsChanged) cDvbDevice::BondDevices(data.DeviceBondings); diff --git a/vdr.c b/vdr.c index 6b0bf2b..c8de702 100644 --- a/vdr.c +++ b/vdr.c @@ -1515,6 +1515,12 @@ int main(int argc, char *argv[]) ReportEpgBugFixStats(); + for (int i = 0; i < cDevice::NumDevices(); i++) { + cDevice *d = cDevice::GetDevice(i); + if (d) + d->CheckIdle(); + } + // Main thread hooks of plugins: PluginManager.MainThreadHook(); } -- 1.9.1

5 24

Re: [vdr] TBS driver breaks lirc_serial
by Birgit & Andreas Böttger 27 May '16

27 May '16

Hi Karim, Karim schrieb am 26.05.2016 um 22:39: > I tried today with Jessie 8.4.0 x64, exactly same problem :-( > I think we tried all the ways. I opened a case at TBS support a few days ago. > They asked me to send them "lirc_serial.ko working" to check, it seems they could integrate it in their driver. > If so, it should be great ! > > Many thanks for your help ! > Of course, if I have some good news from TBS, I will post here. You mean that the (current) TBS driver is incompatible to Debian? Maybe .. OK, let's check the TBS support :) Best regards Andreas Böttger

2 2

Re: [vdr] TBS driver breaks lirc_serial
by Birgit & Andreas Böttger 26 May '16

26 May '16

Hi Karim, Karim schrieb am 25.05.2016 um 00:24: > LIRC is a daemon (/usr/sbin/lircd on my system) and /etc/init.d/lirc is the script to start this daemon. > If I restart LIRC I see the following in syslog (on my systemd system via journalctl -f): > > Unfortunately I don't have journalctl, and I didn't afford to install it on Debian Wheezy. You do not need journalctl, because your system is no systemd-system obviously. On my system there is no syslog or other logfile in directory /var/log :) > Your line "[FAIL] Starting remote control..." means that LIRC has any problem. > Please try to find any other entries in any logfiles that may be related with this problem. > This failed start should be visible in syslog... > > You've right. But curiously, I didnt find error : > > grep lirc /var/log/syslog > May 24 23:01:00 pctest lircd-0.9.0-pre1[4396]: lircd(default) ready, using /var/run/lirc/lircd > May 24 23:02:00 pctest lircd-0.9.0-pre1[4396]: caught signal > May 24 23:09:01 pctest lircd-0.9.0-pre1[4414]: lircd(default) ready, using /var/run/lirc/lircd > May 24 23:09:09 pctest lircd-0.9.0-pre1[4414]: caught signal > May 24 23:35:51 pctest kernel: [ 4171.326315] lirc_serial: Manually using active low receiver > May 24 23:35:51 pctest kernel: [ 4171.326409] lirc_serial lirc_serial.0: lirc_dev: driver lirc_serial registered at minor = 2 > > grep lirc /var/log/user.log > nothing. Strange... >> 3/ >> During workflow, theses commands returns "no file", I don't know if it's normal : >> mv /lib/modules/`uname -r`/kernel/drivers/media/* $SICDIR/media/ >> mv /lib/modules/`uname -r`/kernel/drivers/staging/media/* >> $SICDIR/staging/media/ > On my system the package from TBS installs in different directories than the original modules are. > This way I had a mixture of original modules and modules from TBS. After deleting the original "media" subdirs and reinstalling TBS stuff all was ok. But if it become necessary to restore the originals you need a backup... > Maybe your system has a different directory structure? > > Yes, I think so. I see 2 path : > /lib/modules/3.2.0-4-amd64/kernel/drivers/media/ > /lib/modules/3.2.0-4-amd64/kernel/drivers/staging/media/ > > (I am afraid that if I restore media from backup it erase news modules from TBS).Theses points (path and need/or don't need to restore backup) are not very clear for me. It is not important in which directory structure the modules are installed. But it is important, that you have no mixture of modules, that are build from different configurations or that are incompatible in any other way. The package from TBS install its own version of v4l (Video4Linux) and your system should use only these modules, not the originals or both of them. Normally you will not need backups, but to have it is better :) Maybe it is a good idea to reinstall your Debian system to be sure, that all modules are clean. Than create a backup of the entire modules tree (/lib/modules/*) and install TBS stuff without deleting any directories. After that you are able to compare the mixed modules to the backup (diff -Nqr /lib/modules /your.backup) and find out, what you should delete before installing TBS. >> 4/ >> I can't check this because "/etc/sysconfig" doesn't exist in my Debian 7.7.0. >> Do you know the path for Debian ? >> vdr:~ # grep '^[A-Z]' /etc/sysconfig/lirc LIRCD_DEV_PERMISSIONS="660" >> LIRCD_DEV_OWNER="root:video" >> LIRCD_DRIVER="default" >> LIRCD_DEVICE="/dev/lirc0" >> LIRC_MODULE="lirc_serial" >> LIRCD_LISTENPORT= >> LIRCD_CONNECT= > vdr:~ # find /etc -name 'lirc*' > /etc/lirc > /etc/lirc/lircd.conf > /etc/init.d/lirc > /etc/sysconfig/lirc > > Try this on your system... > > Unfortunately I didn't find this file :-( > > > >> 5/ >> /etc/lirc/hardware.conf >> # /etc/lirc/hardware.conf >> # >> # Arguments which will be used when launching lircd LIRCD_ARGS="" >> >> #Don't start lircmd even if there seems to be a good config file >> #START_LIRCMD=false >> >> #Don't start irexec, even if a good config file seems to exist. >> #START_IREXEC=false >> >> #Try to load appropriate kernel modules LOAD_MODULES=true >> >> # Run "lircd --driver=help" for a list of supported drivers. >> DRIVER="" >> # If DEVICE is set to /dev/lirc and udev is in use /dev/lirc0 will be >> # automatically used instead DEVICE="" >> MODULES="lirc_serial" >> >> # Default configuration files for your hardware if any LIRCD_CONF="" >> LIRCMD_CONF="" > > Your /etc/lirc/hardware.conf could be somewhat that /etc/sysconfig/lirc is on my system. > > > Yes, that's why I did'nt find /etc/sysconfig. > > > At this time, I think **the point** is to clarify is the configuration file : > path, name and content. > > In case I can't find/fix this, do you think it could be interesting to try with a fresh install of Debian Jessie **8.4.0** which kernel 3.16 ? Debian and OpenSUSE are different. Logging is different, configuration is different, ... but both are linux :) Do you have an old system with running DVB and LIRC? If so, try to find the LIRC configuration... I do not know Debian and which version is stable or not. But why not - "latest is greatest" :) Best regards Andreas Böttger

2 1

[PATCH] cSkinXXXXDisplayMenu::SetEvent(): fix date buffer size
by glenvt18 26 May '16

26 May '16

32 bytes is not enough for multi-byte utf8 strings Date string is truncated. Please review, Sergey Chernyavskiy. --- PLUGINS/src/skincurses/skincurses.c | 2 +- skinclassic.c | 2 +- skinlcars.c | 2 +- skinsttng.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/PLUGINS/src/skincurses/skincurses.c b/PLUGINS/src/skincurses/skincurses.c index 358035e..fcf183d 100644 --- a/PLUGINS/src/skincurses/skincurses.c +++ b/PLUGINS/src/skincurses/skincurses.c @@ -407,7 +407,7 @@ void cSkinCursesDisplayMenu::SetEvent(const cEvent *Event) return; int y = 2; cTextScroller ts; - char t[32]; + char t[64]; snprintf(t, sizeof(t), "%s %s - %s", *Event->GetDateString(), *Event->GetTimeString(), *Event->GetEndTimeString()); ts.Set(osd, 0, y, ScOsdWidth, ScOsdHeight - y - 2, t, &Font, clrYellow, clrBackground); if (Event->Vps() && Event->Vps() != Event->StartTime()) { diff --git a/skinclassic.c b/skinclassic.c index b6d183b..410230d 100644 --- a/skinclassic.c +++ b/skinclassic.c @@ -352,7 +352,7 @@ void cSkinClassicDisplayMenu::SetEvent(const cEvent *Event) const cFont *font = cFont::GetFont(fontOsd); int y = y2; cTextScroller ts; - char t[32]; + char t[64]; snprintf(t, sizeof(t), "%s %s - %s", *Event->GetDateString(), *Event->GetTimeString(), *Event->GetEndTimeString()); ts.Set(osd, x1, y, x2 - x1, y3 - y, t, font, Theme.Color(clrMenuEventTime), Theme.Color(clrBackground)); if (Event->Vps() && Event->Vps() != Event->StartTime()) { diff --git a/skinlcars.c b/skinlcars.c index d84a753..7c5561c 100644 --- a/skinlcars.c +++ b/skinlcars.c @@ -1636,7 +1636,7 @@ void cSkinLCARSDisplayMenu::SetEvent(const cEvent *Event) int xl = xi00; int y = yi00; cTextScroller ts; - char t[32]; + char t[64]; snprintf(t, sizeof(t), "%s %s - %s", *Event->GetDateString(), *Event->GetTimeString(), *Event->GetEndTimeString()); ts.Set(osd, xl, y, xi01 - xl, yi01 - y, t, font, Theme.Color(clrEventTime), Theme.Color(clrBackground)); if (Event->Vps() && Event->Vps() != Event->StartTime()) { diff --git a/skinsttng.c b/skinsttng.c index f10f120..f81b13e 100644 --- a/skinsttng.c +++ b/skinsttng.c @@ -655,7 +655,7 @@ void cSkinSTTNGDisplayMenu::SetEvent(const cEvent *Event) int xl = x3 + TextSpacing; int y = y3; cTextScroller ts; - char t[32]; + char t[64]; snprintf(t, sizeof(t), "%s %s - %s", *Event->GetDateString(), *Event->GetTimeString(), *Event->GetEndTimeString()); ts.Set(osd, xl, y, x4 - xl, y4 - y, t, font, Theme.Color(clrMenuEventTime), Theme.Color(clrBackground)); if (Event->Vps() && Event->Vps() != Event->StartTime()) { -- 1.9.1

1 0

Re: [vdr] TBS driver breaks lirc_serial
by Birgit & Andreas Böttger 24 May '16

24 May '16

Hi Karim, Karim schrieb am 21.05.2016 um 23:29: > grep lirc /var/log/syslog > May 21 22:40:05 pctest kernel: [ 3.835944] lirc_dev: IR Remote Control driver registered, major 251 > May 21 22:40:05 pctest kernel: [ 3.836499] rc rc0: lirc_dev: driver ir-lirc-codec (saa716x) registered at minor = 0 > May 21 22:40:05 pctest kernel: [ 4.402162] rc rc1: lirc_dev: driver ir-lirc-codec (saa716x) registered at minor = 1 > May 21 22:40:05 pctest kernel: [ 7.364112] lirc_serial: Manually using active low receiver > May 21 22:40:05 pctest kernel: [ 7.364198] lirc_serial lirc_serial.0: lirc_dev: driver lirc_serial registered at minor = 2 > May 21 22:46:27 pctest kernel: [ 388.677309] lirc_dev: module unloaded > May 21 22:48:32 pctest kernel: [ 513.779622] lirc_dev: IR Remote Control driver registered, major 251 > May 21 22:48:32 pctest kernel: [ 513.780127] lirc_serial: Manually using active low receiver > May 21 22:48:32 pctest kernel: [ 513.780224] lirc_serial lirc_serial.0: lirc_dev: driver lirc_serial registered at minor = 0 > > Questions : > > 1/ > There is **2 modules**, but I think it's not a problem because modinfo is OK. > Are you OK ? It seems to me that there are two rc-devices on receiver cards - rc0 and rc1. After restart of LIRC only lirc_dev and lirc_serial are running - ok. > locate lirc_serial.ko > /lib/modules/3.2.0-4-amd64/kernel/drivers/linux/drivers/staging/lirc/lirc_serial.ko > /lib/modules/sic/2016-05-20_22:51/3.2.0-4-amd64/kernel/drivers/media/staging/media/lirc/lirc_serial.ko .. the new one from TBS and a backup - ok. > 2/ > I found a problem with lirc (I don't know if there is a relation with mode2 : > setserial /dev/ttyS0 uart none > /etc/init.d/lirc start > [ ok ] Loading LIRC modules:. > [FAIL] Starting remote control daemon(s) : LIRC : failed! > You mean lirc or lircd ? (I use lircd in my vdr) LIRC is a daemon (/usr/sbin/lircd on my system) and /etc/init.d/lirc is the script to start this daemon. If I restart LIRC I see the following in syslog (on my systemd system via journalctl -f): Mai 22 14:52:10 vdr systemd[1]: Stopping LSB: lirc daemon... Mai 22 14:52:10 vdr lircd-0.9.0[980]: caught signal Mai 22 14:52:10 vdr vdr[2522]: [2540] ERROR: lircd connection broken, trying to reconnect every 3,0 seconds Mai 22 14:52:10 vdr lirc[1024]: Shutting down lircd ..done Mai 22 14:52:10 vdr systemd[1]: Stopped LSB: lirc daemon. Mai 22 14:52:12 vdr systemd[1]: Starting LSB: lirc daemon... Mai 22 14:52:12 vdr lircd-0.9.0[1071]: lircd(default) ready, using /var/run/lirc/lircd Mai 22 14:52:12 vdr lirc[1044]: Starting lircd (/dev/lirc0)..done Mai 22 14:52:12 vdr systemd[1]: Started LSB: lirc daemon. Mai 22 14:52:13 vdr vdr[2522]: [2540] reconnected to lircd Mai 22 14:52:13 vdr lircd-0.9.0[1071]: accepted new client on /var/run/lirc/lircd Your line "[FAIL] Starting remote control..." means that LIRC has any problem. Please try to find any other entries in any logfiles that may be related with this problem. This failed start should be visible in syslog... > 3/ > During workflow, theses commands returns "no file", I don't know if it's normal : > mv /lib/modules/`uname -r`/kernel/drivers/media/* $SICDIR/media/ > mv /lib/modules/`uname -r`/kernel/drivers/staging/media/* $SICDIR/staging/media/ On my system the package from TBS installs in different directories than the original modules are. This way I had a mixture of original modules and modules from TBS. After deleting the original "media" subdirs and reinstalling TBS stuff all was ok. But if it become necessary to restore the originals you need a backup... Maybe your system has a different directory structure? > 4/ > I can't check this because "/etc/sysconfig" doesn't exist in my Debian 7.7.0. > Do you know the path for Debian ? > vdr:~ # grep '^[A-Z]' /etc/sysconfig/lirc LIRCD_DEV_PERMISSIONS="660" > LIRCD_DEV_OWNER="root:video" > LIRCD_DRIVER="default" > LIRCD_DEVICE="/dev/lirc0" > LIRC_MODULE="lirc_serial" > LIRCD_LISTENPORT= > LIRCD_CONNECT= vdr:~ # find /etc -name 'lirc*' /etc/lirc /etc/lirc/lircd.conf /etc/init.d/lirc /etc/sysconfig/lirc Try this on your system... > 5/ > I use Homebrew too. > Could you confirm that parameters must be sent with theses 3 files ? > > /etc/lirc/lircd.conf > > > /etc/serial.conf > /dev/ttyS0 uart none > > /etc/lirc/hardware.conf > # /etc/lirc/hardware.conf > # > # Arguments which will be used when launching lircd > LIRCD_ARGS="" > > #Don't start lircmd even if there seems to be a good config file > #START_LIRCMD=false > > #Don't start irexec, even if a good config file seems to exist. > #START_IREXEC=false > > #Try to load appropriate kernel modules > LOAD_MODULES=true > > # Run "lircd --driver=help" for a list of supported drivers. > DRIVER="" > # If DEVICE is set to /dev/lirc and udev is in use /dev/lirc0 will be > # automatically used instead > DEVICE="" > MODULES="lirc_serial" > > # Default configuration files for your hardware if any > LIRCD_CONF="" > LIRCMD_CONF="" On my system there is no /etc/serial.conf. Your /etc/lirc/hardware.conf could be somewhat that /etc/sysconfig/lirc is on my system. Best regards Andreas Böttger

2 1

Re: [vdr] TBS driver breaks lirc_serial
by Birgit & Andreas Böttger 21 May '16

21 May '16

Hi Karim, Karim schrieb am 20.05.2016 um 22:59: > lirc_serial is now loading : > > dmesg > [7.492111] lirc_serial: Manually using active low receiver > [7.492378] lirc_serial lirc_serial.0: lirc_dev: driver lirc_serial registered at minor = 2 > > Unfortunately, remote doesn't work (tested with mode2). If you try it with your old setup, original lirc modules and the same config files - is then all ok? Are there any errors in syslog? > I noticed in ".config" file : > # CONFIG_LIRC_SERIAL_TRANSMITTER is not set > > I changed to : > CONFIG_LIRC_SERIAL_TRANSMITTER=y I think that you will need this only for any infrared sender. > Did I missed something ? Hmm... I restart lirc in my runvdr script like this: /etc/init.d/lirc stop sleep 1 rmmod lirc_serial lirc_dev # to be sure sleep 1 setserial /dev/ttyS0 uart none sleep 1 /etc/init.d/lirc start vdr:~ # dmesg | grep lirc lirc_dev: IR Remote Control driver registered, major 247 lirc_serial: Manually using active low receiver lirc_serial lirc_serial.0: lirc_dev: driver lirc_serial registered at minor = 0 vdr:~ # lsmod | grep lirc lirc_serial 18982 3 lirc_dev 19166 1 lirc_serial vdr:~ # grep '^[A-Z]' /etc/sysconfig/lirc LIRCD_DEV_PERMISSIONS="660" LIRCD_DEV_OWNER="root:video" LIRCD_DRIVER="default" LIRCD_DEVICE="/dev/lirc0" LIRC_MODULE="lirc_serial" LIRCD_LISTENPORT= LIRCD_CONNECT= Best regards. Andreas Böttger

2 2

Re: [vdr] TBS driver breaks lirc_serial
by Birgit & Andreas Böttger 20 May '16

20 May '16

Hi Karim, Karim schrieb am 19.05.2016 um 22:36: > I have an issue with TBS drivers (even with last version from april 2016). > As soon as I install TBS Linux drivers and reboot, lirc_serial doesn't load anymore. I use TBS drivers v160405 with OpenSuse 13.2 (kernel 3.16.7-35) but lirc from TBS driver package. My workflow to do this is: cd linux-tbs-drivers make distclean ./v4l/tbs-x86_64.sh make # but only some seconds to initialize build and generate v4l/.config vi v4l/.config # CONFIG_LIRC_STAGING=y and CONFIG_LIRC_SERIAL=m make # remove (with or without backup) original media SICDIR=/lib/modules/sic/`date +%Y-%m-%d_%H:%M`/`uname -r`/kernel/drivers/media/ mkdir -p $SICDIR/media $SICDIR/staging/media mv /lib/modules/`uname -r`/kernel/drivers/media/* $SICDIR/media/ mv /lib/modules/`uname -r`/kernel/drivers/staging/media/* $SICDIR/staging/media/ make install reboot Best regards. Andreas Böttger

2 1

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

vdr May 2016