VE7UWU/sdrangel
1
0
Fork 0
mirror of https://github.com/f4exb/sdrangel.git synced 2025-11-28 11:03:18 -05:00
Code Issues Projects Releases Wiki Activity

USRP: Call setBuddySharedPtr even if open fails. Fix one of the bugs in #2552

Browse Source
This commit is contained in:
Jon Beniston 2025-11-26 15:48:45 +00:00
parent b6c8940345
commit 3e912352f6
2 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
plugins/samplesink/usrpoutput/usrpoutput.cpp
View File

@ -100,6 +100,8 @@ void USRPOutput::destroy()
bool USRPOutput::openDevice() bool USRPOutput::openDevice()
{ {
bool ret = true;
int requestedChannel = m_deviceAPI->getDeviceItemIndex(); int requestedChannel = m_deviceAPI->getDeviceItemIndex();
// look for Tx buddies and get reference to common parameters // look for Tx buddies and get reference to common parameters
@ -198,14 +200,15 @@ bool USRPOutput::openDevice()
if (!m_deviceShared.m_deviceParams->open(deviceStr, false)) if (!m_deviceShared.m_deviceParams->open(deviceStr, false))
{ {
qCritical("USRPOutput::openDevice: failed to open device"); qCritical("USRPOutput::openDevice: failed to open device");
return false; // We need to set setBuddySharedPtr below even if open fails
ret = false;
} }
m_deviceShared.m_channel = requestedChannel; // acknowledge the requested channel m_deviceShared.m_channel = requestedChannel; // acknowledge the requested channel
} }
m_deviceAPI->setBuddySharedPtr(&m_deviceShared); // propagate common parameters to API m_deviceAPI->setBuddySharedPtr(&m_deviceShared); // propagate common parameters to API
return true; return ret;
} }
void USRPOutput::suspendRxBuddies() void USRPOutput::suspendRxBuddies()

7
plugins/samplesource/usrpinput/usrpinput.cpp
View File

@ -104,6 +104,8 @@ void USRPInput::destroy()
bool USRPInput::openDevice() bool USRPInput::openDevice()
{ {
bool ret = true;
// B210 supports up to 50MSa/s, so a fairly large FIFO is probably a good idea // B210 supports up to 50MSa/s, so a fairly large FIFO is probably a good idea
// Should it be bigger still? // Should it be bigger still?
if (!m_sampleFifo.setSize(2000000)) if (!m_sampleFifo.setSize(2000000))
@ -228,14 +230,15 @@ bool USRPInput::openDevice()
if (!m_deviceShared.m_deviceParams->open(deviceStr, false)) if (!m_deviceShared.m_deviceParams->open(deviceStr, false))
{ {
qCritical("USRPInput::openDevice: failed to open device"); qCritical("USRPInput::openDevice: failed to open device");
return false; // We need to set setBuddySharedPtr below even if open fails
ret = false;
} }
m_deviceShared.m_channel = requestedChannel; // acknowledge the requested channel m_deviceShared.m_channel = requestedChannel; // acknowledge the requested channel
} }
m_deviceAPI->setBuddySharedPtr(&m_deviceShared); // propagate common parameters to API m_deviceAPI->setBuddySharedPtr(&m_deviceShared); // propagate common parameters to API
return true; return ret;
} }
void USRPInput::suspendRxBuddies() void USRPInput::suspendRxBuddies()