VE7UWU/sdrangel
1
0
Fork 0
mirror of https://github.com/f4exb/sdrangel.git synced 2024-11-25 09:18:54 -05:00
Code Issues Projects Releases Wiki Activity

USRP: Don't call get_tx/rx methods if tx/rx channels is 0, as may result in KeyError exception

Browse Source
This commit is contained in:
Jon Beniston 2022-06-13 08:36:31 +01:00
parent 9b2c7511fe
commit 964835e2f2
1 changed files with 63 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

91
devices/usrp/deviceusrpparam.cpp
View File

@ -37,17 +37,23 @@ bool DeviceUSRPParams::open(const QString &deviceStr, bool channelNumOnly)
m_nbRxChannels = m_dev->get_rx_num_channels(); m_nbRxChannels = m_dev->get_rx_num_channels();
m_nbTxChannels = m_dev->get_tx_num_channels(); m_nbTxChannels = m_dev->get_tx_num_channels();
qDebug() << "DeviceUSRPParams::open: m_nbRxChannels: " << m_nbRxChannels << " m_nbTxChannels: " << m_nbTxChannels;
// Speed up program initialisation, by not getting all properties // Speed up program initialisation, by not getting all properties
// If we could find out number of channles without ::make ing the device // If we could find out number of channles without ::make ing the device
// that would be even better // that would be even better
if (!channelNumOnly) if (!channelNumOnly)
{ {
m_lpfRangeRx = m_dev->get_rx_bandwidth_range(); if (m_nbRxChannels > 0)
m_lpfRangeTx = m_dev->get_tx_bandwidth_range(); {
m_lpfRangeRx = m_dev->get_rx_bandwidth_range();
m_loRangeRx = m_dev->get_fe_rx_freq_range(); m_loRangeRx = m_dev->get_fe_rx_freq_range();
m_loRangeTx = m_dev->get_fe_tx_freq_range(); }
if (m_nbTxChannels > 0)
{
m_lpfRangeTx = m_dev->get_tx_bandwidth_range();
m_loRangeTx = m_dev->get_fe_tx_freq_range();
}
// For some devices (B210), rx/tx_rates vary with master_clock_rate // For some devices (B210), rx/tx_rates vary with master_clock_rate
// which can be set automatically by UHD. For other devices, // which can be set automatically by UHD. For other devices,
@ -58,22 +64,41 @@ bool DeviceUSRPParams::open(const QString &deviceStr, bool channelNumOnly)
uhd::property_tree::sptr properties = m_dev->get_device()->get_tree(); uhd::property_tree::sptr properties = m_dev->get_device()->get_tree();
if ((clockRange.start() == clockRange.stop()) || !properties->exists("/mboards/0/auto_tick_rate")) if ((clockRange.start() == clockRange.stop()) || !properties->exists("/mboards/0/auto_tick_rate"))
{ {
m_srRangeRx = m_dev->get_rx_rates(); if (m_nbRxChannels > 0) {
m_srRangeTx = m_dev->get_tx_rates(); m_srRangeRx = m_dev->get_rx_rates();
}
if (m_nbTxChannels > 0) {
m_srRangeTx = m_dev->get_tx_rates();
}
} }
else else
{ {
// Find max and min sample rate, for max and min master clock rates // Find max and min sample rate, for max and min master clock rates
m_dev->set_master_clock_rate(clockRange.start()); m_dev->set_master_clock_rate(clockRange.start());
uhd::meta_range_t rxLow = m_dev->get_rx_rates(); uhd::meta_range_t rxLow;
uhd::meta_range_t txLow = m_dev->get_tx_rates(); uhd::meta_range_t txLow;
if (m_nbRxChannels > 0) {
rxLow = m_dev->get_rx_rates();
}
if (m_nbTxChannels > 0) {
txLow = m_dev->get_tx_rates();
}
m_dev->set_master_clock_rate(clockRange.stop()); m_dev->set_master_clock_rate(clockRange.stop());
uhd::meta_range_t rxHigh = m_dev->get_rx_rates(); uhd::meta_range_t rxHigh;
uhd::meta_range_t txHigh = m_dev->get_tx_rates(); uhd::meta_range_t txHigh;
m_srRangeRx = uhd::meta_range_t(std::min(rxLow.start(), rxHigh.start()), std::max(rxLow.stop(), rxHigh.stop())); if (m_nbRxChannels > 0)
m_srRangeTx = uhd::meta_range_t(std::min(txLow.start(), txHigh.start()), std::max(txLow.stop(), txHigh.stop())); {
rxHigh = m_dev->get_rx_rates();
m_srRangeRx = uhd::meta_range_t(std::min(rxLow.start(), rxHigh.start()), std::max(rxLow.stop(), rxHigh.stop()));
}
if (m_nbTxChannels > 0)
{
txHigh = m_dev->get_tx_rates();
m_srRangeTx = uhd::meta_range_t(std::min(txLow.start(), txHigh.start()), std::max(txLow.stop(), txHigh.stop()));
}
// Need to restore automatic clock rate // Need to restore automatic clock rate
if (properties->exists("/mboards/0/auto_tick_rate")) { if (properties->exists("/mboards/0/auto_tick_rate")) {
@ -81,28 +106,38 @@ bool DeviceUSRPParams::open(const QString &deviceStr, bool channelNumOnly)
} }
} }
m_gainRangeRx = m_dev->get_rx_gain_range(); if (m_nbRxChannels > 0)
m_gainRangeTx = m_dev->get_tx_gain_range(); {
m_gainRangeRx = m_dev->get_rx_gain_range();
std::vector<std::string> txAntennas = m_dev->get_tx_antennas(); std::vector<std::string> rxAntennas = m_dev->get_rx_antennas();
m_txAntennas.reserve(txAntennas.size()); m_rxAntennas.reserve(rxAntennas.size());
for(size_t i = 0, l = txAntennas.size(); i < l; ++i) for(size_t i = 0, l = rxAntennas.size(); i < l; ++i) {
m_txAntennas << QString::fromStdString(txAntennas[i]); m_rxAntennas << QString::fromStdString(rxAntennas[i]);
}
std::vector<std::string> rxAntennas = m_dev->get_rx_antennas(); std::vector<std::string> rxGainNames = m_dev->get_rx_gain_names();
m_rxAntennas.reserve(rxAntennas.size()); m_rxGainNames.reserve(rxGainNames.size());
for(size_t i = 0, l = rxAntennas.size(); i < l; ++i) for(size_t i = 0, l = rxGainNames.size(); i < l; ++i) {
m_rxAntennas << QString::fromStdString(rxAntennas[i]); m_rxGainNames << QString::fromStdString(rxGainNames[i]);
}
}
if (m_nbTxChannels > 0)
{
m_gainRangeTx = m_dev->get_tx_gain_range();
std::vector<std::string> rxGainNames = m_dev->get_rx_gain_names(); std::vector<std::string> txAntennas = m_dev->get_tx_antennas();
m_rxGainNames.reserve(rxGainNames.size()); m_txAntennas.reserve(txAntennas.size());
for(size_t i = 0, l = rxGainNames.size(); i < l; ++i) for(size_t i = 0, l = txAntennas.size(); i < l; ++i) {
m_rxGainNames << QString::fromStdString(rxGainNames[i]); m_txAntennas << QString::fromStdString(txAntennas[i]);
}
}
std::vector<std::string> clockSources = m_dev->get_clock_sources(0); std::vector<std::string> clockSources = m_dev->get_clock_sources(0);
m_clockSources.reserve(clockSources.size()); m_clockSources.reserve(clockSources.size());
for(size_t i = 0, l = clockSources.size(); i < l; ++i) for(size_t i = 0, l = clockSources.size(); i < l; ++i) {
m_clockSources << QString::fromStdString(clockSources[i]); m_clockSources << QString::fromStdString(clockSources[i]);
}
} }
return true; return true;